Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(433)

Unified Diff: components/offline_pages/background/request_coordinator.cc

Issue 2089413002: [Offline Pages] Create a event/activity logger (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add ability to retrieve whether we are currently logging Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/background/request_coordinator.cc
diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc
index 05ce9ffe9f656143119f54df05814fd7d91a7ccf..4c5671acb47959ecd971b6e3e9c97f0df4cf98ee 100644
--- a/components/offline_pages/background/request_coordinator.cc
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -155,6 +155,10 @@ void RequestCoordinator::OfflinerDoneCallback(const SavePageRequest& request,
DVLOG(2) << "offliner finished, saved: "
<< (status == Offliner::RequestStatus::SAVED) << ", status: "
<< (int) status << ", " << __FUNCTION__;
+ event_logger_.RecordSavePageRequestUpdated(
+ request.client_id().name_space,
+ "Saved",
+ request.request_id());
last_offlining_status_ = status;
is_busy_ = false;
@@ -174,9 +178,9 @@ void RequestCoordinator::OfflinerDoneCallback(const SavePageRequest& request,
}
void RequestCoordinator::GetOffliner() {
- if (!offliner_) {
- offliner_ = factory_->GetOffliner(policy_.get());
- }
+ if (!offliner_) {
+ offliner_ = factory_->GetOffliner(policy_.get());
+ }
}
} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698