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

Unified Diff: chrome/browser/extensions/platform_app_browsertest_util.cc

Issue 11190016: Do not add TabContents from packaged apps to the task manager. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: rebase Created 8 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/platform_app_browsertest_util.cc
diff --git a/chrome/browser/extensions/platform_app_browsertest_util.cc b/chrome/browser/extensions/platform_app_browsertest_util.cc
index 5b26f9bb58c9f3b0dc82ad4b43a5328372ba576f..281fae18f34ffae71271b8e82202fec717c7b7ab 100644
--- a/chrome/browser/extensions/platform_app_browsertest_util.cc
+++ b/chrome/browser/extensions/platform_app_browsertest_util.cc
@@ -32,17 +32,19 @@ void PlatformAppBrowserTest::SetUpCommandLine(CommandLine* command_line) {
}
const Extension* PlatformAppBrowserTest::LoadAndLaunchPlatformApp(
- const char* name) {
+ Browser* browser, NotificationObserver* observer,
+ FilePath test_data_dir, const char* name) {
content::WindowedNotificationObserver app_loaded_observer(
content::NOTIFICATION_LOAD_COMPLETED_MAIN_FRAME,
content::NotificationService::AllSources());
const Extension* extension = LoadExtension(
- test_data_dir_.AppendASCII("platform_apps").AppendASCII(name));
+ browser, observer,
+ test_data_dir.AppendASCII("platform_apps").AppendASCII(name));
EXPECT_TRUE(extension);
application_launch::OpenApplication(application_launch::LaunchParams(
- browser()->profile(), extension, extension_misc::LAUNCH_NONE,
+ browser->profile(), extension, extension_misc::LAUNCH_NONE,
NEW_WINDOW));
app_loaded_observer.Wait();
« no previous file with comments | « chrome/browser/extensions/platform_app_browsertest_util.h ('k') | chrome/browser/task_manager/task_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698