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

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

Issue 2947973002: Support moving pending requests from one DataUseRecorder to another (Closed)
Patch Set: Created 3 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: 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 3dec687dfb3b6b9ba4ab85c17758fa00796f799e..55376ccbecd30de20fb7fb2a91630b08b6c891e5 100644
--- a/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
+++ b/chrome/browser/data_use_measurement/chrome_data_use_ascriber.cc
@@ -372,8 +372,8 @@ void ChromeDataUseAscriber::DidFinishNavigation(int render_process_id,
if (is_same_page_navigation) {
old_frame_entry->MergeFrom(&(*entry));
- for (auto* request : entry->pending_url_requests())
- AscribeRecorderWithRequest(request, old_frame_entry);
+ for (auto& request : entry->pending_url_requests())
+ AscribeRecorderWithRequest(request.first, old_frame_entry);
entry->RemoveAllPendingURLRequests();
data_use_recorders_.erase(entry);
@@ -431,11 +431,10 @@ ChromeDataUseAscriber::CreateNewDataUseRecorder(
void ChromeDataUseAscriber::AscribeRecorderWithRequest(
net::URLRequest* request,
- DataUseRecorderEntry recorder) {
- recorder->AddPendingURLRequest(request);
- request->SetUserData(
- DataUseRecorderEntryAsUserData::kUserDataKey,
- base::MakeUnique<DataUseRecorderEntryAsUserData>(recorder));
+ DataUseRecorderEntry entry) {
+ entry->AddPendingURLRequest(request);
+ request->SetUserData(DataUseRecorderEntryAsUserData::kUserDataKey,
+ base::MakeUnique<DataUseRecorderEntryAsUserData>(entry));
}
void ChromeDataUseAscriber::WasShownOrHidden(int main_render_process_id,

Powered by Google App Engine
This is Rietveld 408576698