OLD | NEW |
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 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/extensions/extension_browsertest.h" | 5 #include "chrome/browser/extensions/extension_browsertest.h" |
6 | 6 |
7 #include <vector> | 7 #include <vector> |
8 | 8 |
9 #include "base/command_line.h" | 9 #include "base/command_line.h" |
10 #include "base/file_path.h" | 10 #include "base/file_path.h" |
11 #include "base/file_util.h" | 11 #include "base/file_util.h" |
12 #include "base/path_service.h" | 12 #include "base/path_service.h" |
13 #include "base/scoped_temp_dir.h" | 13 #include "base/scoped_temp_dir.h" |
14 #include "base/string_number_conversions.h" | 14 #include "base/string_number_conversions.h" |
15 #include "chrome/browser/extensions/crx_installer.h" | 15 #include "chrome/browser/extensions/crx_installer.h" |
16 #include "chrome/browser/extensions/extension_creator.h" | 16 #include "chrome/browser/extensions/extension_creator.h" |
17 #include "chrome/browser/extensions/extension_error_reporter.h" | 17 #include "chrome/browser/extensions/extension_error_reporter.h" |
18 #include "chrome/browser/extensions/extension_host.h" | 18 #include "chrome/browser/extensions/extension_host.h" |
19 #include "chrome/browser/extensions/extension_install_ui.h" | 19 #include "chrome/browser/extensions/extension_install_ui.h" |
20 #include "chrome/browser/extensions/extension_service.h" | 20 #include "chrome/browser/extensions/extension_service.h" |
21 #include "chrome/browser/profiles/profile.h" | 21 #include "chrome/browser/profiles/profile.h" |
22 #include "chrome/browser/ui/browser.h" | 22 #include "chrome/browser/ui/browser.h" |
23 #include "chrome/browser/ui/browser_window.h" | 23 #include "chrome/browser/ui/browser_window.h" |
24 #include "chrome/browser/ui/omnibox/location_bar.h" | 24 #include "chrome/browser/ui/omnibox/location_bar.h" |
| 25 #include "chrome/common/chrome_notification_types.h" |
25 #include "chrome/common/chrome_paths.h" | 26 #include "chrome/common/chrome_paths.h" |
26 #include "chrome/common/chrome_switches.h" | 27 #include "chrome/common/chrome_switches.h" |
27 #include "chrome/test/ui_test_utils.h" | 28 #include "chrome/test/ui_test_utils.h" |
28 #include "content/common/notification_registrar.h" | 29 #include "content/common/notification_registrar.h" |
29 #include "content/common/notification_service.h" | 30 #include "content/common/notification_service.h" |
30 #include "content/common/notification_type.h" | |
31 | 31 |
32 ExtensionBrowserTest::ExtensionBrowserTest() | 32 ExtensionBrowserTest::ExtensionBrowserTest() |
33 : loaded_(false), | 33 : loaded_(false), |
34 installed_(false), | 34 installed_(false), |
35 extension_installs_observed_(0), | 35 extension_installs_observed_(0), |
36 target_page_action_count_(-1), | 36 target_page_action_count_(-1), |
37 target_visible_page_action_count_(-1) { | 37 target_visible_page_action_count_(-1) { |
38 EXPECT_TRUE(temp_dir_.CreateUniqueTempDir()); | 38 EXPECT_TRUE(temp_dir_.CreateUniqueTempDir()); |
39 } | 39 } |
40 | 40 |
(...skipping 16 matching lines...) Expand all Loading... |
57 command_line->AppendSwitchASCII(switches::kLoginProfile, "user"); | 57 command_line->AppendSwitchASCII(switches::kLoginProfile, "user"); |
58 command_line->AppendSwitch(switches::kNoFirstRun); | 58 command_line->AppendSwitch(switches::kNoFirstRun); |
59 #endif | 59 #endif |
60 } | 60 } |
61 | 61 |
62 const Extension* ExtensionBrowserTest::LoadExtensionImpl( | 62 const Extension* ExtensionBrowserTest::LoadExtensionImpl( |
63 const FilePath& path, bool incognito_enabled, bool fileaccess_enabled) { | 63 const FilePath& path, bool incognito_enabled, bool fileaccess_enabled) { |
64 ExtensionService* service = browser()->profile()->GetExtensionService(); | 64 ExtensionService* service = browser()->profile()->GetExtensionService(); |
65 { | 65 { |
66 NotificationRegistrar registrar; | 66 NotificationRegistrar registrar; |
67 registrar.Add(this, NotificationType::EXTENSION_LOADED, | 67 registrar.Add(this, chrome::NOTIFICATION_EXTENSION_LOADED, |
68 NotificationService::AllSources()); | 68 NotificationService::AllSources()); |
69 service->LoadExtension(path); | 69 service->LoadExtension(path); |
70 ui_test_utils::RunMessageLoop(); | 70 ui_test_utils::RunMessageLoop(); |
71 } | 71 } |
72 | 72 |
73 // Find the extension by iterating backwards since it is likely last. | 73 // Find the extension by iterating backwards since it is likely last. |
74 FilePath extension_path = path; | 74 FilePath extension_path = path; |
75 file_util::AbsolutePath(&extension_path); | 75 file_util::AbsolutePath(&extension_path); |
76 const Extension* extension = NULL; | 76 const Extension* extension = NULL; |
77 for (ExtensionList::const_reverse_iterator iter = | 77 for (ExtensionList::const_reverse_iterator iter = |
78 service->extensions()->rbegin(); | 78 service->extensions()->rbegin(); |
79 iter != service->extensions()->rend(); ++iter) { | 79 iter != service->extensions()->rend(); ++iter) { |
80 if ((*iter)->path() == extension_path) { | 80 if ((*iter)->path() == extension_path) { |
81 extension = *iter; | 81 extension = *iter; |
82 break; | 82 break; |
83 } | 83 } |
84 } | 84 } |
85 if (!extension) | 85 if (!extension) |
86 return NULL; | 86 return NULL; |
87 | 87 |
88 const std::string extension_id = extension->id(); | 88 const std::string extension_id = extension->id(); |
89 | 89 |
90 // The call to OnExtensionInstalled ensures the other extension prefs | 90 // The call to OnExtensionInstalled ensures the other extension prefs |
91 // are set up with the defaults. | 91 // are set up with the defaults. |
92 service->extension_prefs()->OnExtensionInstalled( | 92 service->extension_prefs()->OnExtensionInstalled( |
93 extension, Extension::ENABLED); | 93 extension, Extension::ENABLED); |
94 service->SetIsIncognitoEnabled(extension_id, incognito_enabled); | 94 service->SetIsIncognitoEnabled(extension_id, incognito_enabled); |
95 | 95 |
96 ui_test_utils::WindowedNotificationObserver extension_loaded_signal( | 96 ui_test_utils::WindowedNotificationObserver extension_loaded_signal( |
97 NotificationType::EXTENSION_LOADED, | 97 chrome::NOTIFICATION_EXTENSION_LOADED, |
98 Source<Profile>(browser()->profile())); | 98 Source<Profile>(browser()->profile())); |
99 service->SetAllowFileAccess(extension, fileaccess_enabled); | 99 service->SetAllowFileAccess(extension, fileaccess_enabled); |
100 | 100 |
101 // Disabling file access (it's enabled by default for unpacked extensions) | 101 // Disabling file access (it's enabled by default for unpacked extensions) |
102 // ends up reloading the extension, so we need to wait for that and make sure | 102 // ends up reloading the extension, so we need to wait for that and make sure |
103 // that we have the most up to date extension instance. | 103 // that we have the most up to date extension instance. |
104 if (!fileaccess_enabled) { | 104 if (!fileaccess_enabled) { |
105 extension_loaded_signal.Wait(); | 105 extension_loaded_signal.Wait(); |
106 extension = service->GetExtensionById(extension_id, false); | 106 extension = service->GetExtensionById(extension_id, false); |
107 CHECK(extension) << extension_id << " not found after reloading."; | 107 CHECK(extension) << extension_id << " not found after reloading."; |
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
219 const FilePath& path, | 219 const FilePath& path, |
220 InstallUIType ui_type, | 220 InstallUIType ui_type, |
221 int expected_change, | 221 int expected_change, |
222 Profile* profile) { | 222 Profile* profile) { |
223 ExtensionService* service = profile->GetExtensionService(); | 223 ExtensionService* service = profile->GetExtensionService(); |
224 service->set_show_extensions_prompts(false); | 224 service->set_show_extensions_prompts(false); |
225 size_t num_before = service->extensions()->size(); | 225 size_t num_before = service->extensions()->size(); |
226 | 226 |
227 { | 227 { |
228 NotificationRegistrar registrar; | 228 NotificationRegistrar registrar; |
229 registrar.Add(this, NotificationType::EXTENSION_LOADED, | 229 registrar.Add(this, chrome::NOTIFICATION_EXTENSION_LOADED, |
230 NotificationService::AllSources()); | 230 NotificationService::AllSources()); |
231 registrar.Add(this, NotificationType::EXTENSION_UPDATE_DISABLED, | 231 registrar.Add(this, chrome::NOTIFICATION_EXTENSION_UPDATE_DISABLED, |
232 NotificationService::AllSources()); | 232 NotificationService::AllSources()); |
233 registrar.Add(this, NotificationType::EXTENSION_INSTALL_ERROR, | 233 registrar.Add(this, chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR, |
234 NotificationService::AllSources()); | 234 NotificationService::AllSources()); |
235 | 235 |
236 ExtensionInstallUI* install_ui = NULL; | 236 ExtensionInstallUI* install_ui = NULL; |
237 if (ui_type == INSTALL_UI_TYPE_CANCEL) | 237 if (ui_type == INSTALL_UI_TYPE_CANCEL) |
238 install_ui = new MockAbortExtensionInstallUI(); | 238 install_ui = new MockAbortExtensionInstallUI(); |
239 else if (ui_type == INSTALL_UI_TYPE_NORMAL) | 239 else if (ui_type == INSTALL_UI_TYPE_NORMAL) |
240 install_ui = new ExtensionInstallUI(profile); | 240 install_ui = new ExtensionInstallUI(profile); |
241 else if (ui_type == INSTALL_UI_TYPE_AUTO_CONFIRM) | 241 else if (ui_type == INSTALL_UI_TYPE_AUTO_CONFIRM) |
242 install_ui = new MockAutoConfirmExtensionInstallUI(profile); | 242 install_ui = new MockAutoConfirmExtensionInstallUI(profile); |
243 | 243 |
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
277 return false; | 277 return false; |
278 } | 278 } |
279 | 279 |
280 return WaitForExtensionHostsToLoad(); | 280 return WaitForExtensionHostsToLoad(); |
281 } | 281 } |
282 | 282 |
283 void ExtensionBrowserTest::ReloadExtension(const std::string& extension_id) { | 283 void ExtensionBrowserTest::ReloadExtension(const std::string& extension_id) { |
284 ExtensionService* service = browser()->profile()->GetExtensionService(); | 284 ExtensionService* service = browser()->profile()->GetExtensionService(); |
285 service->ReloadExtension(extension_id); | 285 service->ReloadExtension(extension_id); |
286 ui_test_utils::RegisterAndWait(this, | 286 ui_test_utils::RegisterAndWait(this, |
287 NotificationType::EXTENSION_LOADED, | 287 chrome::NOTIFICATION_EXTENSION_LOADED, |
288 NotificationService::AllSources()); | 288 NotificationService::AllSources()); |
289 } | 289 } |
290 | 290 |
291 void ExtensionBrowserTest::UnloadExtension(const std::string& extension_id) { | 291 void ExtensionBrowserTest::UnloadExtension(const std::string& extension_id) { |
292 ExtensionService* service = browser()->profile()->GetExtensionService(); | 292 ExtensionService* service = browser()->profile()->GetExtensionService(); |
293 service->UnloadExtension(extension_id, UnloadedExtensionInfo::DISABLE); | 293 service->UnloadExtension(extension_id, UnloadedExtensionInfo::DISABLE); |
294 } | 294 } |
295 | 295 |
296 void ExtensionBrowserTest::UninstallExtension(const std::string& extension_id) { | 296 void ExtensionBrowserTest::UninstallExtension(const std::string& extension_id) { |
297 ExtensionService* service = browser()->profile()->GetExtensionService(); | 297 ExtensionService* service = browser()->profile()->GetExtensionService(); |
298 service->UninstallExtension(extension_id, false, NULL); | 298 service->UninstallExtension(extension_id, false, NULL); |
299 } | 299 } |
300 | 300 |
301 void ExtensionBrowserTest::DisableExtension(const std::string& extension_id) { | 301 void ExtensionBrowserTest::DisableExtension(const std::string& extension_id) { |
302 ExtensionService* service = browser()->profile()->GetExtensionService(); | 302 ExtensionService* service = browser()->profile()->GetExtensionService(); |
303 service->DisableExtension(extension_id); | 303 service->DisableExtension(extension_id); |
304 } | 304 } |
305 | 305 |
306 void ExtensionBrowserTest::EnableExtension(const std::string& extension_id) { | 306 void ExtensionBrowserTest::EnableExtension(const std::string& extension_id) { |
307 ExtensionService* service = browser()->profile()->GetExtensionService(); | 307 ExtensionService* service = browser()->profile()->GetExtensionService(); |
308 service->EnableExtension(extension_id); | 308 service->EnableExtension(extension_id); |
309 } | 309 } |
310 | 310 |
311 bool ExtensionBrowserTest::WaitForPageActionCountChangeTo(int count) { | 311 bool ExtensionBrowserTest::WaitForPageActionCountChangeTo(int count) { |
312 LocationBarTesting* location_bar = | 312 LocationBarTesting* location_bar = |
313 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); | 313 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); |
314 if (location_bar->PageActionCount() != count) { | 314 if (location_bar->PageActionCount() != count) { |
315 target_page_action_count_ = count; | 315 target_page_action_count_ = count; |
316 ui_test_utils::RegisterAndWait(this, | 316 ui_test_utils::RegisterAndWait(this, |
317 NotificationType::EXTENSION_PAGE_ACTION_COUNT_CHANGED, | 317 chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_COUNT_CHANGED, |
318 NotificationService::AllSources()); | 318 NotificationService::AllSources()); |
319 } | 319 } |
320 return location_bar->PageActionCount() == count; | 320 return location_bar->PageActionCount() == count; |
321 } | 321 } |
322 | 322 |
323 bool ExtensionBrowserTest::WaitForPageActionVisibilityChangeTo(int count) { | 323 bool ExtensionBrowserTest::WaitForPageActionVisibilityChangeTo(int count) { |
324 LocationBarTesting* location_bar = | 324 LocationBarTesting* location_bar = |
325 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); | 325 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); |
326 if (location_bar->PageActionVisibleCount() != count) { | 326 if (location_bar->PageActionVisibleCount() != count) { |
327 target_visible_page_action_count_ = count; | 327 target_visible_page_action_count_ = count; |
328 ui_test_utils::RegisterAndWait(this, | 328 ui_test_utils::RegisterAndWait(this, |
329 NotificationType::EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED, | 329 chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED, |
330 NotificationService::AllSources()); | 330 NotificationService::AllSources()); |
331 } | 331 } |
332 return location_bar->PageActionVisibleCount() == count; | 332 return location_bar->PageActionVisibleCount() == count; |
333 } | 333 } |
334 | 334 |
335 bool ExtensionBrowserTest::WaitForExtensionHostsToLoad() { | 335 bool ExtensionBrowserTest::WaitForExtensionHostsToLoad() { |
336 // Wait for all the extension hosts that exist to finish loading. | 336 // Wait for all the extension hosts that exist to finish loading. |
337 NotificationRegistrar registrar; | 337 NotificationRegistrar registrar; |
338 registrar.Add(this, NotificationType::EXTENSION_HOST_DID_STOP_LOADING, | 338 registrar.Add(this, chrome::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING, |
339 NotificationService::AllSources()); | 339 NotificationService::AllSources()); |
340 | 340 |
341 ExtensionProcessManager* manager = | 341 ExtensionProcessManager* manager = |
342 browser()->profile()->GetExtensionProcessManager(); | 342 browser()->profile()->GetExtensionProcessManager(); |
343 for (ExtensionProcessManager::const_iterator iter = manager->begin(); | 343 for (ExtensionProcessManager::const_iterator iter = manager->begin(); |
344 iter != manager->end();) { | 344 iter != manager->end();) { |
345 if ((*iter)->did_stop_loading()) { | 345 if ((*iter)->did_stop_loading()) { |
346 ++iter; | 346 ++iter; |
347 } else { | 347 } else { |
348 ui_test_utils::RunMessageLoop(); | 348 ui_test_utils::RunMessageLoop(); |
349 | 349 |
350 // Test activity may have modified the set of extension processes during | 350 // Test activity may have modified the set of extension processes during |
351 // message processing, so re-start the iteration to catch added/removed | 351 // message processing, so re-start the iteration to catch added/removed |
352 // processes. | 352 // processes. |
353 iter = manager->begin(); | 353 iter = manager->begin(); |
354 } | 354 } |
355 } | 355 } |
356 return true; | 356 return true; |
357 } | 357 } |
358 | 358 |
359 bool ExtensionBrowserTest::WaitForExtensionInstall() { | 359 bool ExtensionBrowserTest::WaitForExtensionInstall() { |
360 int before = extension_installs_observed_; | 360 int before = extension_installs_observed_; |
361 ui_test_utils::RegisterAndWait(this, NotificationType::EXTENSION_INSTALLED, | 361 ui_test_utils::RegisterAndWait(this, |
| 362 chrome::NOTIFICATION_EXTENSION_INSTALLED, |
362 NotificationService::AllSources()); | 363 NotificationService::AllSources()); |
363 return extension_installs_observed_ == (before + 1); | 364 return extension_installs_observed_ == (before + 1); |
364 } | 365 } |
365 | 366 |
366 bool ExtensionBrowserTest::WaitForExtensionInstallError() { | 367 bool ExtensionBrowserTest::WaitForExtensionInstallError() { |
367 int before = extension_installs_observed_; | 368 int before = extension_installs_observed_; |
368 ui_test_utils::RegisterAndWait(this, | 369 ui_test_utils::RegisterAndWait(this, |
369 NotificationType::EXTENSION_INSTALL_ERROR, | 370 chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR, |
370 NotificationService::AllSources()); | 371 NotificationService::AllSources()); |
371 return extension_installs_observed_ == before; | 372 return extension_installs_observed_ == before; |
372 } | 373 } |
373 | 374 |
374 void ExtensionBrowserTest::WaitForExtensionLoad() { | 375 void ExtensionBrowserTest::WaitForExtensionLoad() { |
375 ui_test_utils::RegisterAndWait(this, NotificationType::EXTENSION_LOADED, | 376 ui_test_utils::RegisterAndWait(this, chrome::NOTIFICATION_EXTENSION_LOADED, |
376 NotificationService::AllSources()); | 377 NotificationService::AllSources()); |
377 WaitForExtensionHostsToLoad(); | 378 WaitForExtensionHostsToLoad(); |
378 } | 379 } |
379 | 380 |
380 bool ExtensionBrowserTest::WaitForExtensionCrash( | 381 bool ExtensionBrowserTest::WaitForExtensionCrash( |
381 const std::string& extension_id) { | 382 const std::string& extension_id) { |
382 ExtensionService* service = browser()->profile()->GetExtensionService(); | 383 ExtensionService* service = browser()->profile()->GetExtensionService(); |
383 | 384 |
384 if (!service->GetExtensionById(extension_id, true)) { | 385 if (!service->GetExtensionById(extension_id, true)) { |
385 // The extension is already unloaded, presumably due to a crash. | 386 // The extension is already unloaded, presumably due to a crash. |
386 return true; | 387 return true; |
387 } | 388 } |
388 ui_test_utils::RegisterAndWait(this, | 389 ui_test_utils::RegisterAndWait( |
389 NotificationType::EXTENSION_PROCESS_TERMINATED, | 390 this, chrome::NOTIFICATION_EXTENSION_PROCESS_TERMINATED, |
390 NotificationService::AllSources()); | 391 NotificationService::AllSources()); |
391 return (service->GetExtensionById(extension_id, true) == NULL); | 392 return (service->GetExtensionById(extension_id, true) == NULL); |
392 } | 393 } |
393 | 394 |
394 void ExtensionBrowserTest::Observe(NotificationType type, | 395 void ExtensionBrowserTest::Observe(int type, |
395 const NotificationSource& source, | 396 const NotificationSource& source, |
396 const NotificationDetails& details) { | 397 const NotificationDetails& details) { |
397 switch (type.value) { | 398 switch (type) { |
398 case NotificationType::EXTENSION_LOADED: | 399 case chrome::NOTIFICATION_EXTENSION_LOADED: |
399 last_loaded_extension_id_ = Details<const Extension>(details).ptr()->id(); | 400 last_loaded_extension_id_ = Details<const Extension>(details).ptr()->id(); |
400 VLOG(1) << "Got EXTENSION_LOADED notification."; | 401 VLOG(1) << "Got EXTENSION_LOADED notification."; |
401 MessageLoopForUI::current()->Quit(); | 402 MessageLoopForUI::current()->Quit(); |
402 break; | 403 break; |
403 | 404 |
404 case NotificationType::EXTENSION_UPDATE_DISABLED: | 405 case chrome::NOTIFICATION_EXTENSION_UPDATE_DISABLED: |
405 VLOG(1) << "Got EXTENSION_UPDATE_DISABLED notification."; | 406 VLOG(1) << "Got EXTENSION_UPDATE_DISABLED notification."; |
406 MessageLoopForUI::current()->Quit(); | 407 MessageLoopForUI::current()->Quit(); |
407 break; | 408 break; |
408 | 409 |
409 case NotificationType::EXTENSION_HOST_DID_STOP_LOADING: | 410 case chrome::NOTIFICATION_EXTENSION_HOST_DID_STOP_LOADING: |
410 VLOG(1) << "Got EXTENSION_HOST_DID_STOP_LOADING notification."; | 411 VLOG(1) << "Got EXTENSION_HOST_DID_STOP_LOADING notification."; |
411 MessageLoopForUI::current()->Quit(); | 412 MessageLoopForUI::current()->Quit(); |
412 break; | 413 break; |
413 | 414 |
414 case NotificationType::EXTENSION_INSTALLED: | 415 case chrome::NOTIFICATION_EXTENSION_INSTALLED: |
415 VLOG(1) << "Got EXTENSION_INSTALLED notification."; | 416 VLOG(1) << "Got EXTENSION_INSTALLED notification."; |
416 ++extension_installs_observed_; | 417 ++extension_installs_observed_; |
417 MessageLoopForUI::current()->Quit(); | 418 MessageLoopForUI::current()->Quit(); |
418 break; | 419 break; |
419 | 420 |
420 case NotificationType::EXTENSION_INSTALL_ERROR: | 421 case chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR: |
421 VLOG(1) << "Got EXTENSION_INSTALL_ERROR notification."; | 422 VLOG(1) << "Got EXTENSION_INSTALL_ERROR notification."; |
422 MessageLoopForUI::current()->Quit(); | 423 MessageLoopForUI::current()->Quit(); |
423 break; | 424 break; |
424 | 425 |
425 case NotificationType::EXTENSION_PROCESS_CREATED: | 426 case chrome::NOTIFICATION_EXTENSION_PROCESS_CREATED: |
426 VLOG(1) << "Got EXTENSION_PROCESS_CREATED notification."; | 427 VLOG(1) << "Got EXTENSION_PROCESS_CREATED notification."; |
427 MessageLoopForUI::current()->Quit(); | 428 MessageLoopForUI::current()->Quit(); |
428 break; | 429 break; |
429 | 430 |
430 case NotificationType::EXTENSION_PROCESS_TERMINATED: | 431 case chrome::NOTIFICATION_EXTENSION_PROCESS_TERMINATED: |
431 VLOG(1) << "Got EXTENSION_PROCESS_TERMINATED notification."; | 432 VLOG(1) << "Got EXTENSION_PROCESS_TERMINATED notification."; |
432 MessageLoopForUI::current()->Quit(); | 433 MessageLoopForUI::current()->Quit(); |
433 break; | 434 break; |
434 | 435 |
435 case NotificationType::EXTENSION_PAGE_ACTION_COUNT_CHANGED: { | 436 case chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_COUNT_CHANGED: { |
436 LocationBarTesting* location_bar = | 437 LocationBarTesting* location_bar = |
437 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); | 438 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); |
438 VLOG(1) << "Got EXTENSION_PAGE_ACTION_COUNT_CHANGED notification. Number " | 439 VLOG(1) << "Got EXTENSION_PAGE_ACTION_COUNT_CHANGED notification. Number " |
439 "of page actions: " << location_bar->PageActionCount(); | 440 "of page actions: " << location_bar->PageActionCount(); |
440 if (location_bar->PageActionCount() == | 441 if (location_bar->PageActionCount() == |
441 target_page_action_count_) { | 442 target_page_action_count_) { |
442 target_page_action_count_ = -1; | 443 target_page_action_count_ = -1; |
443 MessageLoopForUI::current()->Quit(); | 444 MessageLoopForUI::current()->Quit(); |
444 } | 445 } |
445 break; | 446 break; |
446 } | 447 } |
447 | 448 |
448 case NotificationType::EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED: { | 449 case chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED: { |
449 LocationBarTesting* location_bar = | 450 LocationBarTesting* location_bar = |
450 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); | 451 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); |
451 VLOG(1) << "Got EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED notification. " | 452 VLOG(1) << "Got EXTENSION_PAGE_ACTION_VISIBILITY_CHANGED notification. " |
452 "Number of visible page actions: " | 453 "Number of visible page actions: " |
453 << location_bar->PageActionVisibleCount(); | 454 << location_bar->PageActionVisibleCount(); |
454 if (location_bar->PageActionVisibleCount() == | 455 if (location_bar->PageActionVisibleCount() == |
455 target_visible_page_action_count_) { | 456 target_visible_page_action_count_) { |
456 target_visible_page_action_count_ = -1; | 457 target_visible_page_action_count_ = -1; |
457 MessageLoopForUI::current()->Quit(); | 458 MessageLoopForUI::current()->Quit(); |
458 } | 459 } |
459 break; | 460 break; |
460 } | 461 } |
461 | 462 |
462 default: | 463 default: |
463 NOTREACHED(); | 464 NOTREACHED(); |
464 break; | 465 break; |
465 } | 466 } |
466 } | 467 } |
OLD | NEW |