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

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: rebase 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 a511e19e5fb580cbaa1f6de5ef8957ff102ed01f..d7798c4c5b26e9bbda7b181d86aec2a11033d958 100644
--- a/components/offline_pages/background/request_coordinator.cc
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -150,6 +150,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 @@ RequestCoordinator::GetTriggerConditionsForUserRequest() {
}
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