chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc - Issue 2790993003: Add Generic Implementation of ClipboardRecentContent

Unified Diff: chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

Issue 2790993003: Add Generic Implementation of ClipboardRecentContent (Closed)

Patch Set: use ui/base/test/test_clipboard Created 3 years, 9 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

« chrome/browser/android/omnibox/autocomplete_controller_android.cc ('K') | « chrome/browser/android/omnibox/autocomplete_controller_android.cc ('k') | components/omnibox/browser/autocomplete_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

diff --git a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

index e585e9efca2504d16bb8a26658bb08b885a76e58..940075468a2b3ac84d34f57f6102c56d8014af02 100644

--- a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

+++ b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate.cc

@@ -60,6 +60,7 @@

#include "components/ntp_snippets/bookmarks/bookmark_last_visit_utils.h"

#include "components/ntp_snippets/content_suggestions_service.h"

#include "components/omnibox/browser/omnibox_pref_names.h"

+#include "components/open_from_clipboard/clipboard_recent_content.h"

#include "components/password_manager/core/browser/password_store.h"

#include "components/prefs/pref_service.h"

#include "components/previews/core/previews_ui_service.h"

@@ -419,6 +420,10 @@ void ChromeBrowsingDataRemoverDelegate::RemoveEmbedderData(

delete_begin_, delete_end_,

clear_history_.GetCompletionCallback(),

&history_task_tracker_);

+ // TODO(mpearson): do I want to / need to gate this on "not incognito"

+ // the way iOS does? Or does requiring history_service be valid already

+ // do that for me?

+ ClipboardRecentContent::GetInstance()->SuppressClipboardContent();

Mark P 2017/04/02 05:33:56 For context, I'm trying to make a call analogous t

msramek 2017/04/03 12:37:55 I'd prefer not putting this into "if (history_serv

Mark P 2017/04/03 16:29:53 Yes, I believe so. I think the source of my quest

}

// Currently, ContentSuggestionService instance exists only on Android.