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

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

Issue 472383002: Add ProcessPowerCollector to audit power information. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Missed one move. Created 6 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "chrome/browser/power/process_power_collector.h"
6
7 #include "apps/app_window_contents.h"
8 #include "apps/app_window_registry.h"
9 #include "chrome/browser/profiles/profile_manager.h"
10 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
11 #include "chrome/browser/ui/browser_commands.h"
12 #include "chrome/browser/ui/tabs/tab_strip_model.h"
13 #include "chrome/test/base/browser_with_test_window_test.h"
14 #include "chrome/test/base/testing_browser_process.h"
15 #include "chrome/test/base/testing_profile_manager.h"
16 #include "components/power/origin_power_map.h"
17 #include "components/power/origin_power_map_factory.h"
18 #include "content/public/browser/site_instance.h"
19 #include "content/public/test/browser_test_utils.h"
20 #include "content/public/test/mock_render_process_host.h"
21 #include "extensions/browser/app_window/native_app_window.h"
22 #include "extensions/common/extension.h"
23 #include "testing/gtest/include/gtest/gtest.h"
24 #include "url/gurl.h"
25
26 #if defined(OS_CHROMEOS)
27 #include "chromeos/dbus/power_manager/power_supply_properties.pb.h"
28 #endif
29
30 using power::OriginPowerMap;
31 using power::OriginPowerMapFactory;
32
33 class BrowserProcessPowerTest : public BrowserWithTestWindowTest {
34 public:
35 BrowserProcessPowerTest() {}
36 virtual ~BrowserProcessPowerTest() {}
37
38 virtual void SetUp() OVERRIDE {
39 BrowserWithTestWindowTest::SetUp();
40 collector.reset(new ProcessPowerCollector);
41
42 #if defined(OS_CHROMEOS)
43 power_manager::PowerSupplyProperties prop;
44 prop.set_external_power(power_manager::PowerSupplyProperties::AC);
45 prop.set_battery_state(power_manager::PowerSupplyProperties::DISCHARGING);
46 prop.set_battery_percent(20.00);
47 prop.set_battery_discharge_rate(1);
48 collector->PowerChanged(prop);
49 #endif
50
51 profile_manager_.reset(
52 new TestingProfileManager(TestingBrowserProcess::GetGlobal()));
53 ASSERT_TRUE(profile_manager_->SetUp());
54 }
55
56 virtual void TearDown() OVERRIDE {
57 collector.reset();
58 BrowserWithTestWindowTest::TearDown();
59 }
60
61 // Mocks out CPU usage for all processes as |value| percent.
62 double ReturnCpuAsConstant(double value, base::ProcessHandle handle) {
63 return value;
64 }
65
66 protected:
67 content::MockRenderProcessHost* GetProcess(Browser* browser) {
68 return static_cast<content::MockRenderProcessHost*>(
69 browser->tab_strip_model()
70 ->GetActiveWebContents()
71 ->GetRenderViewHost()
72 ->GetProcess());
73 }
74
75 scoped_ptr<base::ProcessHandle> MakeProcessHandle(int process_id) {
76 scoped_ptr<base::ProcessHandle> proc_handle(new base::ProcessHandle(
77 #if defined(OS_WIN)
78 reinterpret_cast<HANDLE>(process_id))
79 #else
80 process_id)
81 #endif
82 );
83 return proc_handle.Pass();
84 }
85
86 scoped_ptr<ProcessPowerCollector> collector;
87 scoped_ptr<TestingProfileManager> profile_manager_;
88 };
89
90 class TestAppWindowContents : public apps::AppWindowContents {
91 public:
92 explicit TestAppWindowContents(content::WebContents* web_contents)
93 : web_contents_(web_contents) {}
94
95 // apps:AppWindowContents
96 virtual void Initialize(content::BrowserContext* context,
97 const GURL& url) OVERRIDE {}
98 virtual void LoadContents(int32 creator_process_id) OVERRIDE {}
99 virtual void NativeWindowChanged(
100 extensions::NativeAppWindow* native_app_window) OVERRIDE {}
101 virtual void NativeWindowClosed() OVERRIDE {}
102 virtual void DispatchWindowShownForTests() const OVERRIDE {}
103 virtual content::WebContents* GetWebContents() const OVERRIDE {
104 return web_contents_.get();
105 }
106
107 private:
108 scoped_ptr<content::WebContents> web_contents_;
109 };
110
111 TEST_F(BrowserProcessPowerTest, NoSite) {
112 collector->UpdatePowerConsumptionForTesting();
113 EXPECT_EQ(0u, collector->metrics_map_for_testing()->size());
114 }
115
116 TEST_F(BrowserProcessPowerTest, OneSite) {
117 GURL url("http://www.google.com");
118 AddTab(browser(), url);
119 collector->UpdatePowerConsumptionForTesting();
120 ProcessPowerCollector::ProcessMetricsMap* metrics_map =
121 collector->metrics_map_for_testing();
122 EXPECT_EQ(1u, metrics_map->size());
123
124 // Create fake process numbers.
125 GetProcess(browser())->SetProcessHandle(MakeProcessHandle(1).Pass());
126
127 OriginPowerMap* origin_power_map =
128 OriginPowerMapFactory::GetForBrowserContext(profile());
129 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(url));
130
131 collector->set_cpu_usage_callback_for_testing(
132 base::Bind(&BrowserProcessPowerTest::ReturnCpuAsConstant,
133 base::Unretained(this),
134 5));
135 EXPECT_DOUBLE_EQ(5, collector->UpdatePowerConsumptionForTesting());
136 EXPECT_EQ(100, origin_power_map->GetPowerForOrigin(url));
137 }
138
139 TEST_F(BrowserProcessPowerTest, MultipleSites) {
140 Browser::CreateParams native_params(profile(),
141 chrome::HOST_DESKTOP_TYPE_NATIVE);
142 GURL url1("http://www.google.com");
143 GURL url2("http://www.example.com");
144 GURL url3("https://www.google.com");
145 scoped_ptr<Browser> browser2(
146 chrome::CreateBrowserWithTestWindowForParams(&native_params));
147 scoped_ptr<Browser> browser3(
148 chrome::CreateBrowserWithTestWindowForParams(&native_params));
149 AddTab(browser(), url1);
150 AddTab(browser2.get(), url2);
151 AddTab(browser3.get(), url3);
152
153 // Create fake process numbers.
154 GetProcess(browser())->SetProcessHandle(MakeProcessHandle(1).Pass());
155 GetProcess(browser2.get())->SetProcessHandle(MakeProcessHandle(2).Pass());
156 GetProcess(browser3.get())->SetProcessHandle(MakeProcessHandle(3).Pass());
157
158 collector->UpdatePowerConsumptionForTesting();
159 ProcessPowerCollector::ProcessMetricsMap* metrics_map =
160 collector->metrics_map_for_testing();
161 EXPECT_EQ(3u, metrics_map->size());
162
163 // Since all handlers are uninitialized, this should be 0.
164 EXPECT_DOUBLE_EQ(0, collector->UpdatePowerConsumptionForTesting());
165 OriginPowerMap* origin_power_map =
166 OriginPowerMapFactory::GetForBrowserContext(profile());
167 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(url1));
168 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(url2));
169 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(url3));
170
171 collector->set_cpu_usage_callback_for_testing(
172 base::Bind(&BrowserProcessPowerTest::ReturnCpuAsConstant,
173 base::Unretained(this),
174 5));
175 EXPECT_DOUBLE_EQ(15, collector->UpdatePowerConsumptionForTesting());
176 EXPECT_EQ(33, origin_power_map->GetPowerForOrigin(url1));
177 EXPECT_EQ(33, origin_power_map->GetPowerForOrigin(url2));
178 EXPECT_EQ(33, origin_power_map->GetPowerForOrigin(url3));
179
180 // Close some tabs and verify that they are removed from the metrics map.
181 chrome::CloseTab(browser2.get());
182 chrome::CloseTab(browser3.get());
183
184 collector->UpdatePowerConsumptionForTesting();
185 EXPECT_EQ(1u, metrics_map->size());
186 }
187
188 TEST_F(BrowserProcessPowerTest, IncognitoDoesntRecordPowerUsage) {
189 Browser::CreateParams native_params(profile()->GetOffTheRecordProfile(),
190 chrome::HOST_DESKTOP_TYPE_NATIVE);
191 scoped_ptr<Browser> incognito_browser(
192 chrome::CreateBrowserWithTestWindowForParams(&native_params));
193 GURL url("http://www.google.com");
194 AddTab(browser(), url);
195
196 GURL hidden_url("http://foo.com");
197 AddTab(incognito_browser.get(), hidden_url);
198
199 // Create fake process numbers.
200 GetProcess(browser())->SetProcessHandle(MakeProcessHandle(1).Pass());
201 GetProcess(incognito_browser.get())
202 ->SetProcessHandle(MakeProcessHandle(2).Pass());
203
204 EXPECT_DOUBLE_EQ(0, collector->UpdatePowerConsumptionForTesting());
205 ProcessPowerCollector::ProcessMetricsMap* metrics_map =
206 collector->metrics_map_for_testing();
207 EXPECT_EQ(1u, metrics_map->size());
208
209 OriginPowerMap* origin_power_map =
210 OriginPowerMapFactory::GetForBrowserContext(profile());
211 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(url));
212
213 collector->set_cpu_usage_callback_for_testing(
214 base::Bind(&BrowserProcessPowerTest::ReturnCpuAsConstant,
215 base::Unretained(this),
216 5));
217 EXPECT_DOUBLE_EQ(5, collector->UpdatePowerConsumptionForTesting());
218
219 // Verify that the incognito data was not stored.
220 EXPECT_EQ(100, origin_power_map->GetPowerForOrigin(url));
221 EXPECT_EQ(0, origin_power_map->GetPowerForOrigin(hidden_url));
222
223 chrome::CloseTab(incognito_browser.get());
224 }
225
226 TEST_F(BrowserProcessPowerTest, MultipleProfilesRecordSeparately) {
227 scoped_ptr<Profile> other_profile(CreateProfile());
228 Browser::CreateParams native_params(other_profile.get(),
229 chrome::HOST_DESKTOP_TYPE_NATIVE);
230 scoped_ptr<Browser> other_user(
231 chrome::CreateBrowserWithTestWindowForParams(&native_params));
232
233 GURL url("http://www.google.com");
234 AddTab(browser(), url);
235
236 GURL hidden_url("http://foo.com");
237 AddTab(other_user.get(), hidden_url);
238
239 // Create fake process numbers.
240 GetProcess(browser())->SetProcessHandle(MakeProcessHandle(1).Pass());
241 GetProcess(other_user.get())->SetProcessHandle(MakeProcessHandle(2).Pass());
242
243 EXPECT_DOUBLE_EQ(0, collector->UpdatePowerConsumptionForTesting());
244 EXPECT_EQ(2u, collector->metrics_map_for_testing()->size());
245
246 collector->set_cpu_usage_callback_for_testing(
247 base::Bind(&BrowserProcessPowerTest::ReturnCpuAsConstant,
248 base::Unretained(this),
249 5));
250 EXPECT_DOUBLE_EQ(10, collector->UpdatePowerConsumptionForTesting());
251
252 // profile() should have an entry for |url| but not |hidden_url|.
253 OriginPowerMap* origin_power_map_first =
254 OriginPowerMapFactory::GetForBrowserContext(profile());
255 EXPECT_EQ(100, origin_power_map_first->GetPowerForOrigin(url));
256 EXPECT_EQ(0, origin_power_map_first->GetPowerForOrigin(hidden_url));
257
258 // |other_profile| should have an entry for |hidden_url| but not |url|.
259 OriginPowerMap* origin_power_map_second =
260 OriginPowerMapFactory::GetForBrowserContext(other_profile.get());
261 EXPECT_EQ(0, origin_power_map_second->GetPowerForOrigin(url));
262 EXPECT_EQ(100, origin_power_map_second->GetPowerForOrigin(hidden_url));
263
264 // Clean up
265 chrome::CloseTab(other_user.get());
266 }
267
268 TEST_F(BrowserProcessPowerTest, AppsRecordPowerUsage) {
269 // Install an app (an extension*).
270 #if defined(OS_WIN)
271 base::FilePath extension_path(FILE_PATH_LITERAL("c:\\foo"));
272 #elif defined(OS_POSIX)
273 base::FilePath extension_path(FILE_PATH_LITERAL("/foo"));
274 #endif
275 base::DictionaryValue manifest;
276 manifest.SetString("name", "Fake Name");
277 manifest.SetString("version", "1");
278 std::string error;
279 char kTestAppId[] = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa";
280 scoped_refptr<extensions::Extension> extension(
281 extensions::Extension::Create(extension_path,
282 extensions::Manifest::INTERNAL,
283 manifest,
284 extensions::Extension::NO_FLAGS,
285 kTestAppId,
286 &error));
287 EXPECT_TRUE(extension.get()) << error;
288
289 Profile* current_profile =
290 profile_manager_->CreateTestingProfile("Test user");
291 GURL url("chrome-extension://aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa");
292 apps::AppWindow* window =
293 new apps::AppWindow(current_profile, new ChromeAppDelegate(), extension);
294 scoped_ptr<content::WebContents> web_contents(
295 content::WebContents::Create(content::WebContents::CreateParams(
296 current_profile,
297 content::SiteInstance::CreateForURL(current_profile, url))));
298 window->SetAppWindowContentsForTesting(scoped_ptr<apps::AppWindowContents>(
299 new TestAppWindowContents(web_contents.get())));
300 apps::AppWindowRegistry* app_registry =
301 apps::AppWindowRegistry::Get(current_profile);
302 app_registry->AddAppWindow(window);
303
304 collector->set_cpu_usage_callback_for_testing(
305 base::Bind(&BrowserProcessPowerTest::ReturnCpuAsConstant,
306 base::Unretained(this),
307 5));
308 collector->UpdatePowerConsumptionForTesting();
309 EXPECT_EQ(1u, collector->metrics_map_for_testing()->size());
310
311 app_registry->RemoveAppWindow(window);
312 collector->UpdatePowerConsumptionForTesting();
313 EXPECT_EQ(0u, collector->metrics_map_for_testing()->size());
314 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698