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

Side by Side Diff: chrome/browser/power/process_power_collector_unittest.cc

Issue 684613002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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/power/process_power_collector.h" 5 #include "chrome/browser/power/process_power_collector.h"
6 6
7 #include "chrome/browser/apps/scoped_keep_alive.h" 7 #include "chrome/browser/apps/scoped_keep_alive.h"
8 #include "chrome/browser/profiles/profile_manager.h" 8 #include "chrome/browser/profiles/profile_manager.h"
9 #include "chrome/browser/ui/apps/chrome_app_delegate.h" 9 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
10 #include "chrome/browser/ui/browser_commands.h" 10 #include "chrome/browser/ui/browser_commands.h"
(...skipping 17 matching lines...) Expand all
28 #if defined(OS_CHROMEOS) 28 #if defined(OS_CHROMEOS)
29 #include "chromeos/dbus/power_manager/power_supply_properties.pb.h" 29 #include "chromeos/dbus/power_manager/power_supply_properties.pb.h"
30 #endif 30 #endif
31 31
32 using power::OriginPowerMap; 32 using power::OriginPowerMap;
33 using power::OriginPowerMapFactory; 33 using power::OriginPowerMapFactory;
34 34
35 class BrowserProcessPowerTest : public BrowserWithTestWindowTest { 35 class BrowserProcessPowerTest : public BrowserWithTestWindowTest {
36 public: 36 public:
37 BrowserProcessPowerTest() {} 37 BrowserProcessPowerTest() {}
38 virtual ~BrowserProcessPowerTest() {} 38 ~BrowserProcessPowerTest() override {}
39 39
40 virtual void SetUp() override { 40 void SetUp() override {
41 BrowserWithTestWindowTest::SetUp(); 41 BrowserWithTestWindowTest::SetUp();
42 collector.reset(new ProcessPowerCollector); 42 collector.reset(new ProcessPowerCollector);
43 43
44 #if defined(OS_CHROMEOS) 44 #if defined(OS_CHROMEOS)
45 power_manager::PowerSupplyProperties prop; 45 power_manager::PowerSupplyProperties prop;
46 prop.set_external_power(power_manager::PowerSupplyProperties::AC); 46 prop.set_external_power(power_manager::PowerSupplyProperties::AC);
47 prop.set_battery_state(power_manager::PowerSupplyProperties::DISCHARGING); 47 prop.set_battery_state(power_manager::PowerSupplyProperties::DISCHARGING);
48 prop.set_battery_percent(20.00); 48 prop.set_battery_percent(20.00);
49 prop.set_battery_discharge_rate(1); 49 prop.set_battery_discharge_rate(1);
50 collector->PowerChanged(prop); 50 collector->PowerChanged(prop);
51 #endif 51 #endif
52 52
53 profile_manager_.reset( 53 profile_manager_.reset(
54 new TestingProfileManager(TestingBrowserProcess::GetGlobal())); 54 new TestingProfileManager(TestingBrowserProcess::GetGlobal()));
55 ASSERT_TRUE(profile_manager_->SetUp()); 55 ASSERT_TRUE(profile_manager_->SetUp());
56 } 56 }
57 57
58 virtual void TearDown() override { 58 void TearDown() override {
59 collector.reset(); 59 collector.reset();
60 BrowserWithTestWindowTest::TearDown(); 60 BrowserWithTestWindowTest::TearDown();
61 } 61 }
62 62
63 // Mocks out CPU usage for all processes as |value| percent. 63 // Mocks out CPU usage for all processes as |value| percent.
64 double ReturnCpuAsConstant(double value, base::ProcessHandle handle) { 64 double ReturnCpuAsConstant(double value, base::ProcessHandle handle) {
65 return value; 65 return value;
66 } 66 }
67 67
68 protected: 68 protected:
(...skipping 242 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 collector->UpdatePowerConsumptionForTesting(); 311 collector->UpdatePowerConsumptionForTesting();
312 EXPECT_EQ(1u, collector->metrics_map_for_testing()->size()); 312 EXPECT_EQ(1u, collector->metrics_map_for_testing()->size());
313 313
314 // Clear the AppWindowContents before trying to close. 314 // Clear the AppWindowContents before trying to close.
315 window->SetAppWindowContentsForTesting( 315 window->SetAppWindowContentsForTesting(
316 scoped_ptr<extensions::AppWindowContents>()); 316 scoped_ptr<extensions::AppWindowContents>());
317 window->OnNativeClose(); 317 window->OnNativeClose();
318 collector->UpdatePowerConsumptionForTesting(); 318 collector->UpdatePowerConsumptionForTesting();
319 EXPECT_EQ(0u, collector->metrics_map_for_testing()->size()); 319 EXPECT_EQ(0u, collector->metrics_map_for_testing()->size());
320 } 320 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698