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

Side by Side Diff: chrome/browser/performance_monitor/performance_monitor_browsertest.cc

Issue 10829078: CPM Page Load timing (Closed) Base URL: http://git.chromium.org/chromium/src.git@dc_event_refactor
Patch Set: Created 8 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/test/base/in_process_browser_test.h" 5 #include "chrome/test/base/in_process_browser_test.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/file_path.h" 8 #include "base/file_path.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/path_service.h" 10 #include "base/path_service.h"
(...skipping 621 matching lines...) Expand 10 before | Expand all | Expand 10 after
632 632
633 Database::MetricInfoVector metrics = GetStats(METRIC_TEST_STARTUP_TIME); 633 Database::MetricInfoVector metrics = GetStats(METRIC_TEST_STARTUP_TIME);
634 ASSERT_EQ(1u, metrics.size()); 634 ASSERT_EQ(1u, metrics.size());
635 ASSERT_LT(metrics[0].value, kMaxStartupTime.ToInternalValue()); 635 ASSERT_LT(metrics[0].value, kMaxStartupTime.ToInternalValue());
636 636
637 metrics = GetStats(METRIC_SESSION_RESTORE_TIME); 637 metrics = GetStats(METRIC_SESSION_RESTORE_TIME);
638 ASSERT_EQ(1u, metrics.size()); 638 ASSERT_EQ(1u, metrics.size());
639 ASSERT_LT(metrics[0].value, kMaxStartupTime.ToInternalValue()); 639 ASSERT_LT(metrics[0].value, kMaxStartupTime.ToInternalValue());
640 } 640 }
641 641
642 IN_PROC_BROWSER_TEST_F(PerformanceMonitorBrowserTest, PageLoadTime) {
643 const base::TimeDelta kMaxLoadTime= base::TimeDelta::FromSeconds(30);
Yoyo Zhou 2012/08/10 18:34:04 nit: space before =
Devlin 2012/08/11 18:22:25 Done.
644
645 ui_test_utils::NavigateToURL(
646 browser(),
647 ui_test_utils::GetTestUrl(FilePath(FilePath::kCurrentDirectory),
648 FilePath(FILE_PATH_LITERAL("title1.html"))));
649
650 ui_test_utils::NavigateToURL(
651 browser(),
652 ui_test_utils::GetTestUrl(FilePath(FilePath::kCurrentDirectory),
653 FilePath(FILE_PATH_LITERAL("title2.html"))));
654
655 Database::MetricInfoVector metrics = GetStats(METRIC_PAGE_LOAD_TIME);
656
657 ASSERT_EQ(2u, metrics.size());
658 ASSERT_LT(metrics[0].value, kMaxLoadTime.ToInternalValue());
659 ASSERT_LT(metrics[1].value, kMaxLoadTime.ToInternalValue());
660 }
661
642 } // namespace performance_monitor 662 } // namespace performance_monitor
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698