New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 661411 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Expand OfflinePageModelEventLogger

Project Member Reported by romax@chromium.org, Nov 2 2016

Issue description

OfflinePageModelEventLogger could be used on a larger scope, providing information to test harness and evaluation test.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 6 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0

commit d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0
Author: romax <romax@chromium.org>
Date: Tue Dec 06 20:49:50 2016

[Offline Pages] Expanding event logger to be used by harness.

Added some functionality in event logger to write log into files during
test harness run.

BUG= 661411 

Review-Url: https://codereview.chromium.org/2529603004
Cr-Commit-Position: refs/heads/master@{#436712}

[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/evaluation/OfflinePageEvaluationBridge.java
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/android/javatests/src/org/chromium/chrome/browser/offlinepages/OfflinePageSavePageLaterEvaluationTest.java
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/browser/android/offline_pages/evaluation/evaluation_test_scheduler.cc
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/browser/android/offline_pages/evaluation/evaluation_test_scheduler.h
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/browser/android/offline_pages/evaluation/offline_page_evaluation_bridge.cc
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/chrome/browser/android/offline_pages/evaluation/offline_page_evaluation_bridge.h
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/components/offline_pages/BUILD.gn
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/components/offline_pages/offline_event_logger.cc
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/components/offline_pages/offline_event_logger.h
[add] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/components/offline_pages/offline_event_logger_unittest.cc
[modify] https://crrev.com/d7ce8a5c3be5ebe48eac93746c617f0ea9d45cd0/components/offline_pages/offline_page_model_event_logger_unittest.cc

Comment 2 by romax@chromium.org, Dec 6 2016

Status: Fixed (was: Assigned)

Sign in to add a comment