OLD | NEW |
---|---|
1 // Copyright 2017 The Chromium Authors. All rights reserved. | 1 // Copyright 2017 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 #ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOAD_M ETRICS_OBSERVER_H_ | 5 #ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOAD_M ETRICS_OBSERVER_H_ |
6 #define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOAD_M ETRICS_OBSERVER_H_ | 6 #define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOAD_M ETRICS_OBSERVER_H_ |
7 | 7 |
8 #include <stdint.h> | 8 #include <stdint.h> |
9 | 9 |
10 #include "chrome/browser/page_load_metrics/page_load_metrics_observer.h" | 10 #include "chrome/browser/page_load_metrics/page_load_metrics_observer.h" |
11 #include "content/public/common/resource_type.h" | |
12 | |
13 namespace offline_pages { | |
14 class ResourceTrackerObserver; | |
15 } | |
11 | 16 |
12 namespace page_load_metrics { | 17 namespace page_load_metrics { |
13 | 18 |
14 class ResourceTrackingPageLoadMetricsObserver | 19 class ResourceTrackingPageLoadMetricsObserver |
15 : public page_load_metrics::PageLoadMetricsObserver { | 20 : public page_load_metrics::PageLoadMetricsObserver { |
16 public: | 21 public: |
17 ResourceTrackingPageLoadMetricsObserver(); | 22 ResourceTrackingPageLoadMetricsObserver( |
23 offline_pages::ResourceTrackerObserver* request_coordinator); | |
18 ~ResourceTrackingPageLoadMetricsObserver() override; | 24 ~ResourceTrackingPageLoadMetricsObserver() override; |
19 | 25 |
20 // Called by the PageLoadMetrics framework when we start a new request, so we | 26 // Called by the PageLoadMetrics framework when we start a new request, so we |
21 // can update our data structures to be able to calculate a resource done | 27 // can update our data structures to be able to calculate a resource done |
22 // percentage. | 28 // percentage. |
23 void OnStartedResource( | 29 void OnStartedResource( |
24 const ExtraRequestStartInfo& extra_request_start_info) override; | 30 const ExtraRequestStartInfo& extra_request_start_info) override; |
25 | 31 |
26 // Called by the PageLoadMetrics framework when we start a new request, so we | 32 // Called by the PageLoadMetrics framework when we start a new request, so we |
27 // can update our data structures to be able to calculate a resource done | 33 // can update our data structures to be able to calculate a resource done |
28 // percentage. | 34 // percentage. |
29 void OnLoadedResource( | 35 void OnLoadedResource( |
30 const ExtraRequestCompleteInfo& extra_request_complete_info) override; | 36 const ExtraRequestCompleteInfo& extra_request_complete_info) override; |
31 | 37 |
32 // For the specified type, get the count of requests started and completed. | 38 // Called when the page load commits, we return STOP_OBSERVING if the page is |
33 // TODO(petewil) Note that type is not used yet, code to use it is coming | 39 // not part of an offlining attempt. |
34 // soon. | 40 ObservePolicy OnCommit(content::NavigationHandle* navigation_handle) override; |
35 void GetCountsForTypeForTesting(const content::ResourceType type, | |
36 int64_t* started_count, | |
37 int64_t* completed_count); | |
38 | 41 |
39 private: | 42 private: |
43 void InformObservers(const content::ResourceType type, | |
Dmitry Titov
2017/05/18 02:28:10
Since there is only one 'observer', perhaps it's m
Bryan McQuade
2017/05/23 05:00:01
Agree - given that the containing class is also it
Pete Williamson
2017/05/24 01:01:50
Renamed this function to InformClient.
We can saf
| |
44 int64_t started_count, | |
45 int64_t completed_count); | |
46 | |
40 // TODO(petewil): Some way to keep track of what we've seen | 47 // TODO(petewil): Some way to keep track of what we've seen |
41 // TODO(petewil): Some way to inform our keeper of aggregate results when they | 48 // TODO(petewil): Some way to inform our keeper of aggregate results when they |
42 // change. | 49 // change. |
43 int64_t started_count_; | 50 int64_t started_count_; |
44 int64_t completed_count_; | 51 int64_t completed_count_; |
52 // Unowned pointer to the ResourceTrackerObserver for this profile, it must be | |
53 // kept valid by the caller for the lifetime of the PageLoadMetricsObserver. | |
54 offline_pages::ResourceTrackerObserver* request_coordinator_; | |
45 }; | 55 }; |
46 | 56 |
47 } // namespace page_load_metrics | 57 } // namespace page_load_metrics |
48 | 58 |
49 #endif // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOA D_METRICS_OBSERVER_H_ | 59 #endif // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_RESOURCE_TRACKING_PAGE_LOA D_METRICS_OBSERVER_H_ |
OLD | NEW |