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

Side by Side Diff: components/omnibox/browser/autocomplete_controller.cc

Issue 2790993003: Add Generic Implementation of ClipboardRecentContent (Closed)
Patch Set: tested interactively; works. removed field trial force-enable code Created 3 years, 8 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
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 "components/omnibox/browser/autocomplete_controller.h" 5 #include "components/omnibox/browser/autocomplete_controller.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
11 11
12 #include "base/format_macros.h" 12 #include "base/format_macros.h"
13 #include "base/logging.h" 13 #include "base/logging.h"
14 #include "base/memory/ptr_util.h"
14 #include "base/metrics/histogram.h" 15 #include "base/metrics/histogram.h"
15 #include "base/strings/string_number_conversions.h" 16 #include "base/strings/string_number_conversions.h"
16 #include "base/strings/stringprintf.h" 17 #include "base/strings/stringprintf.h"
17 #include "base/strings/utf_string_conversions.h" 18 #include "base/strings/utf_string_conversions.h"
18 #include "base/time/time.h" 19 #include "base/time/time.h"
19 #include "base/trace_event/trace_event.h" 20 #include "base/trace_event/trace_event.h"
20 #include "build/build_config.h" 21 #include "build/build_config.h"
21 #include "components/omnibox/browser/autocomplete_controller_delegate.h" 22 #include "components/omnibox/browser/autocomplete_controller_delegate.h"
22 #include "components/omnibox/browser/bookmark_provider.h" 23 #include "components/omnibox/browser/bookmark_provider.h"
23 #include "components/omnibox/browser/builtin_provider.h" 24 #include "components/omnibox/browser/builtin_provider.h"
24 #include "components/omnibox/browser/clipboard_url_provider.h" 25 #include "components/omnibox/browser/clipboard_url_provider.h"
25 #include "components/omnibox/browser/history_quick_provider.h" 26 #include "components/omnibox/browser/history_quick_provider.h"
26 #include "components/omnibox/browser/history_url_provider.h" 27 #include "components/omnibox/browser/history_url_provider.h"
27 #include "components/omnibox/browser/keyword_provider.h" 28 #include "components/omnibox/browser/keyword_provider.h"
28 #include "components/omnibox/browser/omnibox_field_trial.h" 29 #include "components/omnibox/browser/omnibox_field_trial.h"
29 #include "components/omnibox/browser/physical_web_provider.h" 30 #include "components/omnibox/browser/physical_web_provider.h"
30 #include "components/omnibox/browser/search_provider.h" 31 #include "components/omnibox/browser/search_provider.h"
31 #include "components/omnibox/browser/shortcuts_provider.h" 32 #include "components/omnibox/browser/shortcuts_provider.h"
32 #include "components/omnibox/browser/zero_suggest_provider.h" 33 #include "components/omnibox/browser/zero_suggest_provider.h"
33 #include "components/open_from_clipboard/clipboard_recent_content.h" 34 #include "components/open_from_clipboard/clipboard_recent_content.h"
35 #if !defined(OS_IOS)
36 #include "components/open_from_clipboard/clipboard_recent_content_generic.h"
37 #endif
34 #include "components/search_engines/template_url.h" 38 #include "components/search_engines/template_url.h"
35 #include "components/search_engines/template_url_service.h" 39 #include "components/search_engines/template_url_service.h"
36 #include "components/strings/grit/components_strings.h" 40 #include "components/strings/grit/components_strings.h"
37 #include "ui/base/l10n/l10n_util.h" 41 #include "ui/base/l10n/l10n_util.h"
38 42
39 namespace { 43 namespace {
40 44
41 // Converts the given match to a type (and possibly subtype) based on the AQS 45 // Converts the given match to a type (and possibly subtype) based on the AQS
42 // specification. For more details, see 46 // specification. For more details, see
43 // http://goto.google.com/binary-clients-logging. 47 // http://goto.google.com/binary-clients-logging.
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 } 226 }
223 if (provider_types & AutocompleteProvider::TYPE_SHORTCUTS) 227 if (provider_types & AutocompleteProvider::TYPE_SHORTCUTS)
224 providers_.push_back(new ShortcutsProvider(provider_client_.get())); 228 providers_.push_back(new ShortcutsProvider(provider_client_.get()));
225 if (provider_types & AutocompleteProvider::TYPE_ZERO_SUGGEST) { 229 if (provider_types & AutocompleteProvider::TYPE_ZERO_SUGGEST) {
226 zero_suggest_provider_ = ZeroSuggestProvider::Create( 230 zero_suggest_provider_ = ZeroSuggestProvider::Create(
227 provider_client_.get(), history_url_provider_, this); 231 provider_client_.get(), history_url_provider_, this);
228 if (zero_suggest_provider_) 232 if (zero_suggest_provider_)
229 providers_.push_back(zero_suggest_provider_); 233 providers_.push_back(zero_suggest_provider_);
230 } 234 }
231 if (provider_types & AutocompleteProvider::TYPE_CLIPBOARD_URL) { 235 if (provider_types & AutocompleteProvider::TYPE_CLIPBOARD_URL) {
232 ClipboardRecentContent* clipboard_recent_content = 236 #if !defined(OS_IOS)
233 ClipboardRecentContent::GetInstance(); 237 // On iOS, a global ClipboardRecentContent should've been created by now
234 if (clipboard_recent_content) { 238 // (if enabled). If none has been created (e.g., we're on a different
235 providers_.push_back(new ClipboardURLProvider(provider_client_.get(), 239 // platform), use the generic implementation, which AutocompleteController
236 history_url_provider_, 240 // will own. Don't try to create a generic implementation on iOS because
237 clipboard_recent_content)); 241 // iOS doesn't want/need to link in the implementation and the libraries
242 // that would come with it.
243 if (!ClipboardRecentContent::GetInstance()) {
244 clipboard_recent_content_ =
245 base::MakeUnique<ClipboardRecentContentGeneric>();
246 ClipboardRecentContent::SetInstance(clipboard_recent_content_.get());
247 }
248 #endif
249 // ClipboardRecentContent can be null in iOS tests. For non-iOS, we
250 // create a ClipboardRecentContent as above (for both Chrome and tests).
251 if (ClipboardRecentContent::GetInstance()) {
252 providers_.push_back(new ClipboardURLProvider(
253 provider_client_.get(), history_url_provider_,
254 ClipboardRecentContent::GetInstance()));
238 } 255 }
239 } 256 }
240 if (provider_types & AutocompleteProvider::TYPE_PHYSICAL_WEB) { 257 if (provider_types & AutocompleteProvider::TYPE_PHYSICAL_WEB) {
241 PhysicalWebProvider* physical_web_provider = PhysicalWebProvider::Create( 258 PhysicalWebProvider* physical_web_provider = PhysicalWebProvider::Create(
242 provider_client_.get(), history_url_provider_); 259 provider_client_.get(), history_url_provider_);
243 if (physical_web_provider) 260 if (physical_web_provider)
244 providers_.push_back(physical_web_provider); 261 providers_.push_back(physical_web_provider);
245 } 262 }
246 } 263 }
247 264
(...skipping 434 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 expire_timer_.Stop(); 699 expire_timer_.Stop();
683 stop_timer_.Stop(); 700 stop_timer_.Stop();
684 done_ = true; 701 done_ = true;
685 if (clear_result && !result_.empty()) { 702 if (clear_result && !result_.empty()) {
686 result_.Reset(); 703 result_.Reset();
687 // NOTE: We pass in false since we're trying to only clear the popup, not 704 // NOTE: We pass in false since we're trying to only clear the popup, not
688 // touch the edit... this is all a mess and should be cleaned up :( 705 // touch the edit... this is all a mess and should be cleaned up :(
689 NotifyChanged(false); 706 NotifyChanged(false);
690 } 707 }
691 } 708 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698