| OLD | NEW |
| 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 "base/format_macros.h" |
| 5 #include "base/string_util.h" | 6 #include "base/string_util.h" |
| 6 #include "base/utf_string_conversions.h" | 7 #include "base/utf_string_conversions.h" |
| 7 #include "chrome/browser/autocomplete/autocomplete.h" | 8 #include "chrome/browser/autocomplete/autocomplete.h" |
| 8 #include "chrome/browser/autocomplete/autocomplete_edit.h" | 9 #include "chrome/browser/autocomplete/autocomplete_edit.h" |
| 9 #include "chrome/browser/autocomplete/autocomplete_edit_view.h" | 10 #include "chrome/browser/autocomplete/autocomplete_edit_view.h" |
| 10 #include "chrome/browser/autocomplete/autocomplete_match.h" | 11 #include "chrome/browser/autocomplete/autocomplete_match.h" |
| 11 #include "chrome/browser/autocomplete/autocomplete_popup_model.h" | 12 #include "chrome/browser/autocomplete/autocomplete_popup_model.h" |
| 12 #include "chrome/browser/browser_window.h" | 13 #include "chrome/browser/browser_window.h" |
| 13 #include "chrome/browser/history/history.h" | 14 #include "chrome/browser/history/history.h" |
| 14 #include "chrome/browser/tab_contents/tab_contents.h" | 15 #include "chrome/browser/tab_contents/tab_contents.h" |
| (...skipping 10 matching lines...) Expand all Loading... |
| 25 // http://crbug.com/52928 | 26 // http://crbug.com/52928 |
| 26 #if defined(OS_CHROMEOS) | 27 #if defined(OS_CHROMEOS) |
| 27 #define MAYBE_Autocomplete FLAKY_Autocomplete | 28 #define MAYBE_Autocomplete FLAKY_Autocomplete |
| 28 #else | 29 #else |
| 29 #define MAYBE_Autocomplete Autocomplete | 30 #define MAYBE_Autocomplete Autocomplete |
| 30 #endif | 31 #endif |
| 31 | 32 |
| 32 | 33 |
| 33 namespace { | 34 namespace { |
| 34 | 35 |
| 35 std::wstring AutocompleteResultAsString(const AutocompleteResult& result) { | 36 string16 AutocompleteResultAsString(const AutocompleteResult& result) { |
| 36 std::wstring output(StringPrintf(L"{%d} ", result.size())); | 37 std::string output(base::StringPrintf("{%" PRIuS "} ", result.size())); |
| 37 for (size_t i = 0; i < result.size(); ++i) { | 38 for (size_t i = 0; i < result.size(); ++i) { |
| 38 AutocompleteMatch match = result.match_at(i); | 39 AutocompleteMatch match = result.match_at(i); |
| 39 std::wstring provider_name(ASCIIToWide(match.provider->name())); | 40 std::string provider_name = match.provider->name(); |
| 40 output.append(StringPrintf(L"[\"%ls\" by \"%ls\"] ", | 41 output.append(base::StringPrintf("[\"%s\" by \"%s\"] ", |
| 41 match.contents.c_str(), | 42 UTF16ToUTF8(match.contents).c_str(), |
| 42 provider_name.c_str())); | 43 provider_name.c_str())); |
| 43 } | 44 } |
| 44 return output; | 45 return UTF8ToUTF16(output); |
| 45 } | 46 } |
| 46 | 47 |
| 47 } // namespace | 48 } // namespace |
| 48 | 49 |
| 49 class AutocompleteBrowserTest : public InProcessBrowserTest { | 50 class AutocompleteBrowserTest : public InProcessBrowserTest { |
| 50 protected: | 51 protected: |
| 51 LocationBar* GetLocationBar() const { | 52 LocationBar* GetLocationBar() const { |
| 52 return browser()->window()->GetLocationBar(); | 53 return browser()->window()->GetLocationBar(); |
| 53 } | 54 } |
| 54 | 55 |
| 55 AutocompleteController* GetAutocompleteController() const { | 56 AutocompleteController* GetAutocompleteController() const { |
| 56 return GetLocationBar()->location_entry()->model()->popup_model()-> | 57 return GetLocationBar()->location_entry()->model()->popup_model()-> |
| 57 autocomplete_controller(); | 58 autocomplete_controller(); |
| 58 } | 59 } |
| 59 }; | 60 }; |
| 60 | 61 |
| 61 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, Basic) { | 62 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, Basic) { |
| 62 LocationBar* location_bar = GetLocationBar(); | 63 LocationBar* location_bar = GetLocationBar(); |
| 63 | 64 |
| 64 EXPECT_TRUE(location_bar->GetInputString().empty()); | 65 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 65 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 66 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 66 location_bar->location_entry()->GetText()); | 67 location_bar->location_entry()->GetText()); |
| 67 // TODO(phajdan.jr): check state of IsSelectAll when it's consistent across | 68 // TODO(phajdan.jr): check state of IsSelectAll when it's consistent across |
| 68 // platforms. | 69 // platforms. |
| 69 | 70 |
| 70 location_bar->FocusLocation(true); | 71 location_bar->FocusLocation(true); |
| 71 | 72 |
| 72 EXPECT_TRUE(location_bar->GetInputString().empty()); | 73 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 73 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 74 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 74 location_bar->location_entry()->GetText()); | 75 location_bar->location_entry()->GetText()); |
| 75 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); | 76 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); |
| 76 | 77 |
| 77 location_bar->location_entry()->SetUserText(L"chrome"); | 78 location_bar->location_entry()->SetUserText(ASCIIToUTF16("chrome")); |
| 78 | 79 |
| 79 EXPECT_TRUE(location_bar->GetInputString().empty()); | 80 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 80 EXPECT_EQ(L"chrome", location_bar->location_entry()->GetText()); | 81 EXPECT_EQ(ASCIIToUTF16("chrome"), location_bar->location_entry()->GetText()); |
| 81 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); | 82 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); |
| 82 | 83 |
| 83 location_bar->location_entry()->RevertAll(); | 84 location_bar->location_entry()->RevertAll(); |
| 84 | 85 |
| 85 EXPECT_TRUE(location_bar->GetInputString().empty()); | 86 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 86 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 87 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 87 location_bar->location_entry()->GetText()); | 88 location_bar->location_entry()->GetText()); |
| 88 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); | 89 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); |
| 89 | 90 |
| 90 location_bar->location_entry()->SetUserText(L"chrome"); | 91 location_bar->location_entry()->SetUserText(ASCIIToUTF16("chrome")); |
| 91 location_bar->Revert(); | 92 location_bar->Revert(); |
| 92 | 93 |
| 93 EXPECT_TRUE(location_bar->GetInputString().empty()); | 94 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 94 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 95 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 95 location_bar->location_entry()->GetText()); | 96 location_bar->location_entry()->GetText()); |
| 96 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); | 97 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); |
| 97 } | 98 } |
| 98 | 99 |
| 99 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, MAYBE_Autocomplete) { | 100 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, MAYBE_Autocomplete) { |
| 100 // The results depend on the history backend being loaded. Make sure it is | 101 // The results depend on the history backend being loaded. Make sure it is |
| 101 // loaded so that the autocomplete results are consistent. | 102 // loaded so that the autocomplete results are consistent. |
| 102 ui_test_utils::WaitForHistoryToLoad(browser()); | 103 ui_test_utils::WaitForHistoryToLoad(browser()); |
| 103 | 104 |
| 104 LocationBar* location_bar = GetLocationBar(); | 105 LocationBar* location_bar = GetLocationBar(); |
| 105 AutocompleteController* autocomplete_controller = GetAutocompleteController(); | 106 AutocompleteController* autocomplete_controller = GetAutocompleteController(); |
| 106 | 107 |
| 107 { | 108 { |
| 108 autocomplete_controller->Start(L"chrome", std::wstring(), | 109 autocomplete_controller->Start(ASCIIToUTF16("chrome"), string16(), |
| 109 true, false, true, true); | 110 true, false, true, true); |
| 110 | 111 |
| 111 EXPECT_TRUE(autocomplete_controller->done()); | 112 EXPECT_TRUE(autocomplete_controller->done()); |
| 112 EXPECT_TRUE(location_bar->GetInputString().empty()); | 113 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 113 EXPECT_TRUE(location_bar->location_entry()->GetText().empty()); | 114 EXPECT_TRUE(location_bar->location_entry()->GetText().empty()); |
| 114 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); | 115 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); |
| 115 const AutocompleteResult& result = autocomplete_controller->result(); | 116 const AutocompleteResult& result = autocomplete_controller->result(); |
| 116 ASSERT_EQ(1U, result.size()) << AutocompleteResultAsString(result); | 117 ASSERT_EQ(1U, result.size()) << AutocompleteResultAsString(result); |
| 117 AutocompleteMatch match = result.match_at(0); | 118 AutocompleteMatch match = result.match_at(0); |
| 118 EXPECT_EQ(AutocompleteMatch::SEARCH_WHAT_YOU_TYPED, match.type); | 119 EXPECT_EQ(AutocompleteMatch::SEARCH_WHAT_YOU_TYPED, match.type); |
| 119 EXPECT_FALSE(match.deletable); | 120 EXPECT_FALSE(match.deletable); |
| 120 } | 121 } |
| 121 | 122 |
| 122 { | 123 { |
| 123 location_bar->Revert(); | 124 location_bar->Revert(); |
| 124 | 125 |
| 125 EXPECT_TRUE(location_bar->GetInputString().empty()); | 126 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 126 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 127 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 127 location_bar->location_entry()->GetText()); | 128 location_bar->location_entry()->GetText()); |
| 128 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); | 129 EXPECT_FALSE(location_bar->location_entry()->IsSelectAll()); |
| 129 const AutocompleteResult& result = autocomplete_controller->result(); | 130 const AutocompleteResult& result = autocomplete_controller->result(); |
| 130 EXPECT_TRUE(result.empty()) << AutocompleteResultAsString(result); | 131 EXPECT_TRUE(result.empty()) << AutocompleteResultAsString(result); |
| 131 } | 132 } |
| 132 } | 133 } |
| 133 | 134 |
| 134 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, TabAwayRevertSelect) { | 135 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, TabAwayRevertSelect) { |
| 135 // http://code.google.com/p/chromium/issues/detail?id=38385 | 136 // http://code.google.com/p/chromium/issues/detail?id=38385 |
| 136 // Make sure that tabbing away from an empty omnibar causes a revert | 137 // Make sure that tabbing away from an empty omnibar causes a revert |
| 137 // and select all. | 138 // and select all. |
| 138 LocationBar* location_bar = GetLocationBar(); | 139 LocationBar* location_bar = GetLocationBar(); |
| 139 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 140 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 140 location_bar->location_entry()->GetText()); | 141 location_bar->location_entry()->GetText()); |
| 141 location_bar->location_entry()->SetUserText(L""); | 142 location_bar->location_entry()->SetUserText(string16()); |
| 142 browser()->AddSelectedTabWithURL(GURL(chrome::kAboutBlankURL), | 143 browser()->AddSelectedTabWithURL(GURL(chrome::kAboutBlankURL), |
| 143 PageTransition::START_PAGE); | 144 PageTransition::START_PAGE); |
| 144 ui_test_utils::WaitForNavigation( | 145 ui_test_utils::WaitForNavigation( |
| 145 &browser()->GetSelectedTabContents()->controller()); | 146 &browser()->GetSelectedTabContents()->controller()); |
| 146 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 147 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 147 location_bar->location_entry()->GetText()); | 148 location_bar->location_entry()->GetText()); |
| 148 browser()->CloseTab(); | 149 browser()->CloseTab(); |
| 149 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 150 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 150 location_bar->location_entry()->GetText()); | 151 location_bar->location_entry()->GetText()); |
| 151 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); | 152 EXPECT_TRUE(location_bar->location_entry()->IsSelectAll()); |
| 152 } | 153 } |
| 153 | 154 |
| 154 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, FocusSearch) { | 155 IN_PROC_BROWSER_TEST_F(AutocompleteBrowserTest, FocusSearch) { |
| 155 LocationBar* location_bar = GetLocationBar(); | 156 LocationBar* location_bar = GetLocationBar(); |
| 156 | 157 |
| 157 // Focus search when omnibox is blank | 158 // Focus search when omnibox is blank |
| 158 { | 159 { |
| 159 EXPECT_TRUE(location_bar->GetInputString().empty()); | 160 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 160 EXPECT_EQ(UTF8ToWide(chrome::kAboutBlankURL), | 161 EXPECT_EQ(UTF8ToUTF16(chrome::kAboutBlankURL), |
| 161 location_bar->location_entry()->GetText()); | 162 location_bar->location_entry()->GetText()); |
| 162 | 163 |
| 163 location_bar->FocusSearch(); | 164 location_bar->FocusSearch(); |
| 164 EXPECT_TRUE(location_bar->GetInputString().empty()); | 165 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 165 EXPECT_EQ(L"?", location_bar->location_entry()->GetText()); | 166 EXPECT_EQ(ASCIIToUTF16("?"), location_bar->location_entry()->GetText()); |
| 166 | 167 |
| 167 size_t selection_start, selection_end; | 168 size_t selection_start, selection_end; |
| 168 location_bar->location_entry()->GetSelectionBounds(&selection_start, | 169 location_bar->location_entry()->GetSelectionBounds(&selection_start, |
| 169 &selection_end); | 170 &selection_end); |
| 170 EXPECT_EQ(1U, selection_start); | 171 EXPECT_EQ(1U, selection_start); |
| 171 EXPECT_EQ(1U, selection_end); | 172 EXPECT_EQ(1U, selection_end); |
| 172 } | 173 } |
| 173 | 174 |
| 174 // Focus search when omnibox is _not_ alread in forced query mode. | 175 // Focus search when omnibox is _not_ alread in forced query mode. |
| 175 { | 176 { |
| 176 location_bar->location_entry()->SetUserText(L"foo"); | 177 location_bar->location_entry()->SetUserText(ASCIIToUTF16("foo")); |
| 177 EXPECT_TRUE(location_bar->GetInputString().empty()); | 178 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 178 EXPECT_EQ(L"foo", location_bar->location_entry()->GetText()); | 179 EXPECT_EQ(ASCIIToUTF16("foo"), location_bar->location_entry()->GetText()); |
| 179 | 180 |
| 180 location_bar->FocusSearch(); | 181 location_bar->FocusSearch(); |
| 181 EXPECT_TRUE(location_bar->GetInputString().empty()); | 182 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 182 EXPECT_EQ(L"?", location_bar->location_entry()->GetText()); | 183 EXPECT_EQ(ASCIIToUTF16("?"), location_bar->location_entry()->GetText()); |
| 183 | 184 |
| 184 size_t selection_start, selection_end; | 185 size_t selection_start, selection_end; |
| 185 location_bar->location_entry()->GetSelectionBounds(&selection_start, | 186 location_bar->location_entry()->GetSelectionBounds(&selection_start, |
| 186 &selection_end); | 187 &selection_end); |
| 187 EXPECT_EQ(1U, selection_start); | 188 EXPECT_EQ(1U, selection_start); |
| 188 EXPECT_EQ(1U, selection_end); | 189 EXPECT_EQ(1U, selection_end); |
| 189 } | 190 } |
| 190 | 191 |
| 191 // Focus search when omnibox _is_ already in forced query mode, but no query | 192 // Focus search when omnibox _is_ already in forced query mode, but no query |
| 192 // has been typed. | 193 // has been typed. |
| 193 { | 194 { |
| 194 location_bar->location_entry()->SetUserText(L"?"); | 195 location_bar->location_entry()->SetUserText(ASCIIToUTF16("?")); |
| 195 EXPECT_TRUE(location_bar->GetInputString().empty()); | 196 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 196 EXPECT_EQ(L"?", location_bar->location_entry()->GetText()); | 197 EXPECT_EQ(ASCIIToUTF16("?"), location_bar->location_entry()->GetText()); |
| 197 | 198 |
| 198 location_bar->FocusSearch(); | 199 location_bar->FocusSearch(); |
| 199 EXPECT_TRUE(location_bar->GetInputString().empty()); | 200 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 200 EXPECT_EQ(L"?", location_bar->location_entry()->GetText()); | 201 EXPECT_EQ(ASCIIToUTF16("?"), location_bar->location_entry()->GetText()); |
| 201 | 202 |
| 202 size_t selection_start, selection_end; | 203 size_t selection_start, selection_end; |
| 203 location_bar->location_entry()->GetSelectionBounds(&selection_start, | 204 location_bar->location_entry()->GetSelectionBounds(&selection_start, |
| 204 &selection_end); | 205 &selection_end); |
| 205 EXPECT_EQ(1U, selection_start); | 206 EXPECT_EQ(1U, selection_start); |
| 206 EXPECT_EQ(1U, selection_end); | 207 EXPECT_EQ(1U, selection_end); |
| 207 } | 208 } |
| 208 | 209 |
| 209 // Focus search when omnibox _is_ already in forced query mode, and some query | 210 // Focus search when omnibox _is_ already in forced query mode, and some query |
| 210 // has been typed. | 211 // has been typed. |
| 211 { | 212 { |
| 212 location_bar->location_entry()->SetUserText(L"?foo"); | 213 location_bar->location_entry()->SetUserText(ASCIIToUTF16("?foo")); |
| 213 EXPECT_TRUE(location_bar->GetInputString().empty()); | 214 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 214 EXPECT_EQ(L"?foo", location_bar->location_entry()->GetText()); | 215 EXPECT_EQ(ASCIIToUTF16("?foo"), location_bar->location_entry()->GetText()); |
| 215 | 216 |
| 216 location_bar->FocusSearch(); | 217 location_bar->FocusSearch(); |
| 217 EXPECT_TRUE(location_bar->GetInputString().empty()); | 218 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 218 EXPECT_EQ(L"?foo", location_bar->location_entry()->GetText()); | 219 EXPECT_EQ(ASCIIToUTF16("?foo"), location_bar->location_entry()->GetText()); |
| 219 | 220 |
| 220 size_t selection_start, selection_end; | 221 size_t selection_start, selection_end; |
| 221 location_bar->location_entry()->GetSelectionBounds(&selection_start, | 222 location_bar->location_entry()->GetSelectionBounds(&selection_start, |
| 222 &selection_end); | 223 &selection_end); |
| 223 EXPECT_EQ(1U, std::min(selection_start, selection_end)); | 224 EXPECT_EQ(1U, std::min(selection_start, selection_end)); |
| 224 EXPECT_EQ(4U, std::max(selection_start, selection_end)); | 225 EXPECT_EQ(4U, std::max(selection_start, selection_end)); |
| 225 } | 226 } |
| 226 | 227 |
| 227 // Focus search when omnibox is in forced query mode with leading whitespace. | 228 // Focus search when omnibox is in forced query mode with leading whitespace. |
| 228 { | 229 { |
| 229 location_bar->location_entry()->SetUserText(L" ?foo"); | 230 location_bar->location_entry()->SetUserText(ASCIIToUTF16(" ?foo")); |
| 230 EXPECT_TRUE(location_bar->GetInputString().empty()); | 231 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 231 EXPECT_EQ(L" ?foo", location_bar->location_entry()->GetText()); | 232 EXPECT_EQ(ASCIIToUTF16(" ?foo"), |
| 233 location_bar->location_entry()->GetText()); |
| 232 | 234 |
| 233 location_bar->FocusSearch(); | 235 location_bar->FocusSearch(); |
| 234 EXPECT_TRUE(location_bar->GetInputString().empty()); | 236 EXPECT_TRUE(location_bar->GetInputString().empty()); |
| 235 EXPECT_EQ(L" ?foo", location_bar->location_entry()->GetText()); | 237 EXPECT_EQ(ASCIIToUTF16(" ?foo"), |
| 238 location_bar->location_entry()->GetText()); |
| 236 | 239 |
| 237 size_t selection_start, selection_end; | 240 size_t selection_start, selection_end; |
| 238 location_bar->location_entry()->GetSelectionBounds(&selection_start, | 241 location_bar->location_entry()->GetSelectionBounds(&selection_start, |
| 239 &selection_end); | 242 &selection_end); |
| 240 EXPECT_EQ(4U, std::min(selection_start, selection_end)); | 243 EXPECT_EQ(4U, std::min(selection_start, selection_end)); |
| 241 EXPECT_EQ(7U, std::max(selection_start, selection_end)); | 244 EXPECT_EQ(7U, std::max(selection_start, selection_end)); |
| 242 } | 245 } |
| 243 } | 246 } |
| OLD | NEW |