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

Side by Side Diff: chrome/browser/extensions/api/omnibox/omnibox_apitest.cc

Issue 10807047: Create content\public\test\test_utils.h to hold common test classes that are used by unit and brows… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 5 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "base/format_macros.h" 5 #include "base/format_macros.h"
6 #include "base/string_util.h" 6 #include "base/string_util.h"
7 #include "base/stringprintf.h" 7 #include "base/stringprintf.h"
8 #include "base/utf_string_conversions.h" 8 #include "base/utf_string_conversions.h"
9 #include "chrome/browser/autocomplete/autocomplete_input.h" 9 #include "chrome/browser/autocomplete/autocomplete_input.h"
10 #include "chrome/browser/autocomplete/autocomplete_match.h" 10 #include "chrome/browser/autocomplete/autocomplete_match.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 } 52 }
53 53
54 AutocompleteController* GetAutocompleteController(Browser* browser) const { 54 AutocompleteController* GetAutocompleteController(Browser* browser) const {
55 return GetLocationBar(browser)->GetLocationEntry()->model()->popup_model()-> 55 return GetLocationBar(browser)->GetLocationEntry()->model()->popup_model()->
56 autocomplete_controller(); 56 autocomplete_controller();
57 } 57 }
58 58
59 // TODO(phajdan.jr): Get rid of this wait-in-a-loop pattern. 59 // TODO(phajdan.jr): Get rid of this wait-in-a-loop pattern.
60 void WaitForAutocompleteDone(AutocompleteController* controller) { 60 void WaitForAutocompleteDone(AutocompleteController* controller) {
61 while (!controller->done()) { 61 while (!controller->done()) {
62 ui_test_utils::WindowedNotificationObserver ready_observer( 62 content::WindowedNotificationObserver ready_observer(
63 chrome::NOTIFICATION_AUTOCOMPLETE_CONTROLLER_RESULT_READY, 63 chrome::NOTIFICATION_AUTOCOMPLETE_CONTROLLER_RESULT_READY,
64 content::Source<AutocompleteController>(controller)); 64 content::Source<AutocompleteController>(controller));
65 ready_observer.Wait(); 65 ready_observer.Wait();
66 } 66 }
67 } 67 }
68 }; 68 };
69 69
70 IN_PROC_BROWSER_TEST_F(OmniboxApiTest, DISABLED_Basic) { 70 IN_PROC_BROWSER_TEST_F(OmniboxApiTest, DISABLED_Basic) {
71 ASSERT_TRUE(RunExtensionTest("omnibox")) << message_; 71 ASSERT_TRUE(RunExtensionTest("omnibox")) << message_;
72 72
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
272 // incognito context. 272 // incognito context.
273 { 273 {
274 ResultCatcher catcher; 274 ResultCatcher catcher;
275 autocomplete_controller->Start( 275 autocomplete_controller->Start(
276 ASCIIToUTF16("keyword command incognito"), string16(), 276 ASCIIToUTF16("keyword command incognito"), string16(),
277 true, false, true, AutocompleteInput::ALL_MATCHES); 277 true, false, true, AutocompleteInput::ALL_MATCHES);
278 location_bar->AcceptInput(); 278 location_bar->AcceptInput();
279 EXPECT_TRUE(catcher.GetNextResult()) << catcher.message(); 279 EXPECT_TRUE(catcher.GetNextResult()) << catcher.message();
280 } 280 }
281 } 281 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698