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

Unified Diff: components/omnibox/browser/autocomplete_controller.cc

Issue 2801813003: Omnibox - ClipboardRecentContent - Make Proper Singleton (Closed)
Patch Set: restore field trial state 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 side-by-side diff with in-line comments
Download patch
Index: components/omnibox/browser/autocomplete_controller.cc
diff --git a/components/omnibox/browser/autocomplete_controller.cc b/components/omnibox/browser/autocomplete_controller.cc
index ccbdfd3d0d1059d4ff3af027e8001ba2f80abb75..eca556529a60f20f07c2fbf5eccc58002f723b6b 100644
--- a/components/omnibox/browser/autocomplete_controller.cc
+++ b/components/omnibox/browser/autocomplete_controller.cc
@@ -32,14 +32,15 @@
#include "components/omnibox/browser/shortcuts_provider.h"
#include "components/omnibox/browser/zero_suggest_provider.h"
#include "components/open_from_clipboard/clipboard_recent_content.h"
-#if !defined(OS_IOS)
-#include "components/open_from_clipboard/clipboard_recent_content_generic.h"
-#endif
#include "components/search_engines/template_url.h"
#include "components/search_engines/template_url_service.h"
#include "components/strings/grit/components_strings.h"
#include "ui/base/l10n/l10n_util.h"
+#if !defined(OS_IOS)
+#include "components/open_from_clipboard/clipboard_recent_content_generic.h"
+#endif
+
namespace {
// Converts the given match to a type (and possibly subtype) based on the AQS
@@ -241,9 +242,7 @@ AutocompleteController::AutocompleteController(
// iOS doesn't want/need to link in the implementation and the libraries
// that would come with it.
if (!ClipboardRecentContent::GetInstance()) {
- clipboard_recent_content_ =
- base::MakeUnique<ClipboardRecentContentGeneric>();
- ClipboardRecentContent::SetInstance(clipboard_recent_content_.get());
+ ClipboardRecentContent::SetInstance(new ClipboardRecentContentGeneric());
Peter Kasting 2017/04/05 23:23:51 Nit: Might want a comment like "This pointer is in
Mark P 2017/04/06 17:58:00 Done.
}
#endif
// ClipboardRecentContent can be null in iOS tests. For non-iOS, we

Powered by Google App Engine
This is Rietveld 408576698