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

Unified Diff: chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc

Issue 2614203002: Record data use of user traffic by different core page transition types (Closed)
Patch Set: rebased Created 3 years, 9 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: chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
diff --git a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
index 0bb0c0f49952993cc9b84fd96b1d81530c40d250..8ef1560841b1f00779ed8b10beb25e8b913baebd 100644
--- a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
+++ b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
@@ -358,6 +358,16 @@ void ChromeDataUseAscriber::ReadyToCommitMainFrameNavigation(
}
}
+void ChromeDataUseAscriber::DidFinishNavigation(int render_process_id,
+ int render_frame_id,
+ uint32_t page_transition) {
+ auto frame_it = main_render_frame_data_use_map_.find(
+ RenderFrameHostID(render_process_id, render_frame_id));
+ if (frame_it != main_render_frame_data_use_map_.end()) {
+ frame_it->second->set_page_transition(page_transition);
+ }
+}
+
void ChromeDataUseAscriber::OnDataUseCompleted(DataUseRecorderEntry entry) {
// TODO(ryansturm): Notify observers that data use is complete.
}

Powered by Google App Engine
This is Rietveld 408576698