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

Unified Diff: chrome/browser/page_load_metrics/browser_page_track_decider.cc

Issue 2331053003: Add common page filtering logic for page load metrics. (Closed)
Patch Set: fix method names to be consistent with style guide Created 4 years, 3 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/browser_page_track_decider.cc
diff --git a/chrome/browser/page_load_metrics/browser_page_track_decider.cc b/chrome/browser/page_load_metrics/browser_page_track_decider.cc
new file mode 100644
index 0000000000000000000000000000000000000000..8b09eae2a2535ccdd9b3b850fd135faa2e45b4ff
--- /dev/null
+++ b/chrome/browser/page_load_metrics/browser_page_track_decider.cc
@@ -0,0 +1,58 @@
+// Copyright 2016 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.
+
+#include "chrome/browser/page_load_metrics/browser_page_track_decider.h"
+
+#include <string>
+
+#include "chrome/browser/page_load_metrics/metrics_web_contents_observer.h"
+#include "content/public/browser/navigation_handle.h"
+#include "content/public/browser/web_contents.h"
+#include "net/http/http_response_headers.h"
+
+namespace page_load_metrics {
+
+BrowserPageTrackDecider::BrowserPageTrackDecider(
+ PageLoadMetricsEmbedderInterface* embedder_interface,
+ content::WebContents* web_contents,
+ content::NavigationHandle* navigation_handle)
+ : embedder_interface_(embedder_interface),
+ web_contents_(web_contents),
+ navigation_handle_(navigation_handle) {}
+
+BrowserPageTrackDecider::~BrowserPageTrackDecider() {}
+
+bool BrowserPageTrackDecider::HasCommitted() {
+ return navigation_handle_->HasCommitted();
+}
+
+bool BrowserPageTrackDecider::IsHttpOrHttpsUrl() {
+ return navigation_handle_->GetURL().SchemeIsHTTPOrHTTPS();
+}
+
+bool BrowserPageTrackDecider::IsNewTabPageUrl() {
+ return embedder_interface_->IsNewTabPageUrl(navigation_handle_->GetURL());
+}
+
+bool BrowserPageTrackDecider::IsChromeErrorPage() {
+ DCHECK(HasCommitted());
+ return navigation_handle_->IsErrorPage();
+}
+
+int BrowserPageTrackDecider::GetHttpStatusCode() {
+ DCHECK(HasCommitted());
+ const net::HttpResponseHeaders* response_headers =
+ navigation_handle_->GetResponseHeaders();
+ if (!response_headers)
+ return -1;
+ return response_headers->response_code();
+}
+
+bool BrowserPageTrackDecider::IsHtmlOrXhtmlPage() {
+ DCHECK(HasCommitted());
+ const std::string& mime_type = web_contents_->GetContentsMimeType();
+ return mime_type == "text/html" || mime_type == "application/xhtml+xml";
+}
+
+} // namespace page_load_metrics

Powered by Google App Engine
This is Rietveld 408576698