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

Unified Diff: chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h

Issue 2798953002: [PageLoadMetrics] Keep track of Ad Sizes on Pages (Closed)
Patch Set: Add 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h
diff --git a/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..50c213b87e9b65c7c079274755f748b2c2e52d87
--- /dev/null
+++ b/chrome/browser/page_load_metrics/observers/ads_page_load_metrics_observer.h
@@ -0,0 +1,54 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_ADS_PAGE_LOAD_METRICS_OBSERVER_H_
+#define CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_ADS_PAGE_LOAD_METRICS_OBSERVER_H_
+
+#include <unordered_map>
+#include <unordered_set>
+
+#include "base/macros.h"
+#include "chrome/browser/page_load_metrics/page_load_metrics_observer.h"
+#include "net/http/http_response_info.h"
+
+class AdsPageLoadMetricsObserver
+ : public page_load_metrics::PageLoadMetricsObserver {
+ public:
+ AdsPageLoadMetricsObserver();
+ ~AdsPageLoadMetricsObserver() override;
+
+ // page_load_metrics::PageLoadMetricsObserver implementation:
+ ObservePolicy OnCommit(content::NavigationHandle* navigation_handle) override;
+ ObservePolicy OnCommitSubFrame(
+ content::NavigationHandle* navigation_handle) override;
+ void OnLoadedResource(
+ const page_load_metrics::ExtraRequestInfo& extra_request_info) override;
+ void OnComplete(const page_load_metrics::PageLoadTiming& timing,
+ const page_load_metrics::PageLoadExtraInfo& info) override;
+
+ private:
+ struct AdFrameData {
+ size_t frame_bytes;
+ size_t frame_bytes_uncached;
+ };
+
+ using FrameTreeNodeId = int;
+ const FrameTreeNodeId kInvalidFrameTreeNodeId = -1;
+
+ friend class AdsPageLoadMetricsObserverTest;
+
+ int FindAdAncestor(content::NavigationHandle* navigation_handle);
+ void ProcessDelayedResources(FrameTreeNodeId frame_tree_node_id);
+
+ std::unordered_map<FrameTreeNodeId, AdFrameData> ad_frames_;
+ std::unordered_map<FrameTreeNodeId, FrameTreeNodeId> ad_ancestors_;
+ std::unordered_map<FrameTreeNodeId, page_load_metrics::ExtraRequestInfo>
+ delayed_resources_;
+ size_t page_bytes_ = 0;
+ size_t uncached_page_bytes_ = 0;
+
+ DISALLOW_COPY_AND_ASSIGN(AdsPageLoadMetricsObserver);
+};
+
+#endif // CHROME_BROWSER_PAGE_LOAD_METRICS_OBSERVERS_ADS_PAGE_LOAD_METRICS_OBSERVER_H_

Powered by Google App Engine
This is Rietveld 408576698