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

Side by Side Diff: chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc

Issue 2177743002: Migrate page_load_metrics out of components. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: migrate page_load_metrics_messages to common message generator Created 4 years, 4 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/observers/from_gws_page_load_metrics_ observer.h" 5 #include "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_ observer.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #include "chrome/browser/page_load_metrics/observers/page_load_metrics_observer_ test_harness.h" 9 #include "chrome/browser/page_load_metrics/observers/page_load_metrics_observer_ test_harness.h"
10 #include "components/page_load_metrics/browser/page_load_metrics_util.h" 10 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
11 11
12 namespace { 12 namespace {
13 const char kExampleUrl[] = "http://www.example.com/"; 13 const char kExampleUrl[] = "http://www.example.com/";
14 const char kGoogleSearchResultsUrl[] = "https://www.google.com/webhp?q=d"; 14 const char kGoogleSearchResultsUrl[] = "https://www.google.com/webhp?q=d";
15 } // namespace 15 } // namespace
16 16
17 class FromGWSPageLoadMetricsObserverTest 17 class FromGWSPageLoadMetricsObserverTest
18 : public page_load_metrics::PageLoadMetricsObserverTestHarness { 18 : public page_load_metrics::PageLoadMetricsObserverTestHarness {
19 public: 19 public:
20 void RegisterObservers(page_load_metrics::PageLoadTracker* tracker) override { 20 void RegisterObservers(page_load_metrics::PageLoadTracker* tracker) override {
(...skipping 891 matching lines...) Expand 10 before | Expand all | Expand 10 after
912 logger.ShouldLogPostCommitMetrics(GURL("https://www.google.com/about/"))); 912 logger.ShouldLogPostCommitMetrics(GURL("https://www.google.com/about/")));
913 } 913 }
914 914
915 TEST_F(FromGWSPageLoadMetricsLoggerTest, NavigationFromSearchRedirector) { 915 TEST_F(FromGWSPageLoadMetricsLoggerTest, NavigationFromSearchRedirector) {
916 FromGWSPageLoadMetricsLogger logger; 916 FromGWSPageLoadMetricsLogger logger;
917 logger.SetPreviouslyCommittedUrl( 917 logger.SetPreviouslyCommittedUrl(
918 GURL("https://www.google.com/url?source=web")); 918 GURL("https://www.google.com/url?source=web"));
919 logger.set_navigation_initiated_via_link(true); 919 logger.set_navigation_initiated_via_link(true);
920 ASSERT_TRUE(logger.ShouldLogPostCommitMetrics(GURL(kExampleUrl))); 920 ASSERT_TRUE(logger.ShouldLogPostCommitMetrics(GURL(kExampleUrl)));
921 } 921 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698