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

Side by Side Diff: chrome/browser/page_load_metrics/page_load_tracker.cc

Issue 2780003003: Send an event to the page load metrics to track resource starting. (Closed)
Patch Set: Fix browser tests Created 3 years, 8 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "chrome/browser/page_load_metrics/page_load_tracker.h" 5 #include "chrome/browser/page_load_metrics/page_load_tracker.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <ostream> 8 #include <ostream>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 566 matching lines...) Expand 10 before | Expand all | Expand 10 after
577 577
578 const PageLoadMetadata last_metadata = main_frame_metadata_; 578 const PageLoadMetadata last_metadata = main_frame_metadata_;
579 main_frame_metadata_ = new_metadata; 579 main_frame_metadata_ = new_metadata;
580 const PageLoadExtraInfo info = ComputePageLoadExtraInfo(); 580 const PageLoadExtraInfo info = ComputePageLoadExtraInfo();
581 for (const auto& observer : observers_) { 581 for (const auto& observer : observers_) {
582 DispatchObserverTimingCallbacks(observer.get(), last_timing, new_timing, 582 DispatchObserverTimingCallbacks(observer.get(), last_timing, new_timing,
583 last_metadata, info); 583 last_metadata, info);
584 } 584 }
585 } 585 }
586 586
587 void PageLoadTracker::OnStartedResource(
588 const ExtraRequestStartInfo& extra_request_info) {
589 for (const auto& observer : observers_) {
590 observer->OnStartedResource(extra_request_info);
591 }
592 }
593
587 void PageLoadTracker::OnLoadedResource( 594 void PageLoadTracker::OnLoadedResource(
588 const ExtraRequestInfo& extra_request_info) { 595 const ExtraRequestCompleteInfo& extra_request_info) {
589 for (const auto& observer : observers_) { 596 for (const auto& observer : observers_) {
590 observer->OnLoadedResource(extra_request_info); 597 observer->OnLoadedResource(extra_request_info);
591 } 598 }
592 } 599 }
593 600
594 void PageLoadTracker::StopTracking() { 601 void PageLoadTracker::StopTracking() {
595 did_stop_tracking_ = true; 602 did_stop_tracking_ = true;
596 observers_.clear(); 603 observers_.clear();
597 } 604 }
598 605
(...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after
762 observer->MediaStartedPlaying(video_type, is_in_main_frame); 769 observer->MediaStartedPlaying(video_type, is_in_main_frame);
763 } 770 }
764 771
765 void PageLoadTracker::OnNavigationDelayComplete(base::TimeDelta scheduled_delay, 772 void PageLoadTracker::OnNavigationDelayComplete(base::TimeDelta scheduled_delay,
766 base::TimeDelta actual_delay) { 773 base::TimeDelta actual_delay) {
767 for (const auto& observer : observers_) 774 for (const auto& observer : observers_)
768 observer->OnNavigationDelayComplete(scheduled_delay, actual_delay); 775 observer->OnNavigationDelayComplete(scheduled_delay, actual_delay);
769 } 776 }
770 777
771 } // namespace page_load_metrics 778 } // namespace page_load_metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698