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

Side by Side Diff: content/browser/loader/async_resource_handler.cc

Issue 2136223003: ResourceDispatcherHostDelegate::OnResponseStarted doesn't need IPC::Sender (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/async_resource_handler.h" 5 #include "content/browser/loader/async_resource_handler.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 320 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 331
332 // We want to send a final upload progress message prior to sending the 332 // We want to send a final upload progress message prior to sending the
333 // response complete message even if we're waiting for an ack to to a 333 // response complete message even if we're waiting for an ack to to a
334 // previous upload progress message. 334 // previous upload progress message.
335 if (info->is_upload_progress_enabled()) { 335 if (info->is_upload_progress_enabled()) {
336 waiting_for_upload_progress_ack_ = false; 336 waiting_for_upload_progress_ack_ = false;
337 ReportUploadProgress(); 337 ReportUploadProgress();
338 } 338 }
339 339
340 if (rdh_->delegate()) { 340 if (rdh_->delegate()) {
341 rdh_->delegate()->OnResponseStarted( 341 rdh_->delegate()->OnResponseStarted(request(), info->GetContext(),
342 request(), info->GetContext(), response, info->filter()); 342 response);
343 } 343 }
344 344
345 NetLogObserver::PopulateResponseInfo(request(), response); 345 NetLogObserver::PopulateResponseInfo(request(), response);
346 346
347 const HostZoomMapImpl* host_zoom_map = 347 const HostZoomMapImpl* host_zoom_map =
348 static_cast<const HostZoomMapImpl*>(info->filter()->GetHostZoomMap()); 348 static_cast<const HostZoomMapImpl*>(info->filter()->GetHostZoomMap());
349 349
350 if (info->GetResourceType() == RESOURCE_TYPE_MAIN_FRAME && host_zoom_map) { 350 if (info->GetResourceType() == RESOURCE_TYPE_MAIN_FRAME && host_zoom_map) {
351 const GURL& request_url = request()->url(); 351 const GURL& request_url = request()->url();
352 int render_process_id = info->GetChildID(); 352 int render_process_id = info->GetChildID();
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
601 } else { 601 } else {
602 UMA_HISTOGRAM_CUSTOM_COUNTS( 602 UMA_HISTOGRAM_CUSTOM_COUNTS(
603 "Net.ResourceLoader.ResponseStartToEnd.Over_512kB", 603 "Net.ResourceLoader.ResponseStartToEnd.Over_512kB",
604 elapsed_time, 1, 100000, 100); 604 elapsed_time, 1, 100000, 100);
605 } 605 }
606 606
607 inlining_helper_->RecordHistogram(elapsed_time); 607 inlining_helper_->RecordHistogram(elapsed_time);
608 } 608 }
609 609
610 } // namespace content 610 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698