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

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

Issue 5089001: Cleanup: Include browser.h -> ui/browser.h [Part 3]. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixes. Created 10 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) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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/browser.h"
6 #include "chrome/browser/extensions/extension_apitest.h" 5 #include "chrome/browser/extensions/extension_apitest.h"
6 #include "chrome/browser/extensions/extensions_service.h"
7 #include "chrome/browser/extensions/extension_test_message_listener.h" 7 #include "chrome/browser/extensions/extension_test_message_listener.h"
8 #include "chrome/browser/extensions/extensions_service.h"
9 #include "chrome/browser/profile.h" 8 #include "chrome/browser/profile.h"
9 #include "chrome/browser/ui/browser.h"
10 #include "chrome/common/extensions/extension.h" 10 #include "chrome/common/extensions/extension.h"
11 #include "chrome/test/ui_test_utils.h" 11 #include "chrome/test/ui_test_utils.h"
12 12
13 const std::string kAllUrlsTarget = 13 const std::string kAllUrlsTarget =
14 "files/extensions/api_test/all_urls/index.html"; 14 "files/extensions/api_test/all_urls/index.html";
15 15
16 typedef ExtensionApiTest AllUrlsApiTest; 16 typedef ExtensionApiTest AllUrlsApiTest;
17 17
18 18
19 // Note: This test is flaky, but is actively being worked on. 19 // Note: This test is flaky, but is actively being worked on.
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 // Now verify we can script a regular http page. 123 // Now verify we can script a regular http page.
124 ASSERT_TRUE(test_server()->Start()); 124 ASSERT_TRUE(test_server()->Start());
125 GURL page_url = test_server()->GetURL(kAllUrlsTarget); 125 GURL page_url = test_server()->GetURL(kAllUrlsTarget);
126 ExtensionTestMessageListener listener1a("content script: " + page_url.spec(), 126 ExtensionTestMessageListener listener1a("content script: " + page_url.spec(),
127 false); 127 false);
128 ExtensionTestMessageListener listener1b("execute: " + page_url.spec(), false); 128 ExtensionTestMessageListener listener1b("execute: " + page_url.spec(), false);
129 ui_test_utils::NavigateToURL(browser(), page_url); 129 ui_test_utils::NavigateToURL(browser(), page_url);
130 ASSERT_TRUE(listener1a.WaitUntilSatisfied()); 130 ASSERT_TRUE(listener1a.WaitUntilSatisfied());
131 ASSERT_TRUE(listener1b.WaitUntilSatisfied()); 131 ASSERT_TRUE(listener1b.WaitUntilSatisfied());
132 } 132 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/alert_apitest.cc ('k') | chrome/browser/extensions/app_background_page_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698