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

Side by Side Diff: chrome/browser/extensions/extension_browsertest.cc

Issue 8480007: Change EXTENSION_PROCESS_CREATED observers to use EXTENSION_HOST_CREATED. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: host it Created 9 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 | Annotate | Revision Log
OLDNEW
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"
(...skipping 448 matching lines...) Expand 10 before | Expand all | Expand 10 after
459 case chrome::NOTIFICATION_EXTENSION_LOAD_ERROR: 459 case chrome::NOTIFICATION_EXTENSION_LOAD_ERROR:
460 VLOG(1) << "Got EXTENSION_LOAD_ERROR notification."; 460 VLOG(1) << "Got EXTENSION_LOAD_ERROR notification.";
461 MessageLoopForUI::current()->Quit(); 461 MessageLoopForUI::current()->Quit();
462 break; 462 break;
463 463
464 case chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR: 464 case chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR:
465 VLOG(1) << "Got EXTENSION_INSTALL_ERROR notification."; 465 VLOG(1) << "Got EXTENSION_INSTALL_ERROR notification.";
466 MessageLoopForUI::current()->Quit(); 466 MessageLoopForUI::current()->Quit();
467 break; 467 break;
468 468
469 case chrome::NOTIFICATION_EXTENSION_PROCESS_CREATED:
470 VLOG(1) << "Got EXTENSION_PROCESS_CREATED notification.";
471 MessageLoopForUI::current()->Quit();
472 break;
473
474 case chrome::NOTIFICATION_EXTENSION_PROCESS_TERMINATED: 469 case chrome::NOTIFICATION_EXTENSION_PROCESS_TERMINATED:
475 VLOG(1) << "Got EXTENSION_PROCESS_TERMINATED notification."; 470 VLOG(1) << "Got EXTENSION_PROCESS_TERMINATED notification.";
476 MessageLoopForUI::current()->Quit(); 471 MessageLoopForUI::current()->Quit();
477 break; 472 break;
478 473
479 case chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_COUNT_CHANGED: { 474 case chrome::NOTIFICATION_EXTENSION_PAGE_ACTION_COUNT_CHANGED: {
480 LocationBarTesting* location_bar = 475 LocationBarTesting* location_bar =
481 browser()->window()->GetLocationBar()->GetLocationBarForTesting(); 476 browser()->window()->GetLocationBar()->GetLocationBarForTesting();
482 VLOG(1) << "Got EXTENSION_PAGE_ACTION_COUNT_CHANGED notification. Number " 477 VLOG(1) << "Got EXTENSION_PAGE_ACTION_COUNT_CHANGED notification. Number "
483 "of page actions: " << location_bar->PageActionCount(); 478 "of page actions: " << location_bar->PageActionCount();
(...skipping 17 matching lines...) Expand all
501 MessageLoopForUI::current()->Quit(); 496 MessageLoopForUI::current()->Quit();
502 } 497 }
503 break; 498 break;
504 } 499 }
505 500
506 default: 501 default:
507 NOTREACHED(); 502 NOTREACHED();
508 break; 503 break;
509 } 504 }
510 } 505 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/extensions/extension_host.cc » ('j') | chrome/browser/extensions/extension_host.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698