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

Unified Diff: components/browsing_data_ui/history_notice_utils.cc

Issue 2238453002: Merge //components/browsing_data_ui into //components/browsing_data/core (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase over https://codereview.chromium.org/2345843003/ Created 4 years, 3 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/browsing_data_ui/history_notice_utils.cc
diff --git a/components/browsing_data_ui/history_notice_utils.cc b/components/browsing_data_ui/history_notice_utils.cc
deleted file mode 100644
index f3556f3012fe73137a0971bf93e8833a02dbe65e..0000000000000000000000000000000000000000
--- a/components/browsing_data_ui/history_notice_utils.cc
+++ /dev/null
@@ -1,115 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "components/browsing_data_ui/history_notice_utils.h"
-
-#include "base/bind.h"
-#include "base/callback.h"
-#include "base/location.h"
-#include "base/single_thread_task_runner.h"
-#include "base/strings/stringprintf.h"
-#include "base/threading/thread_task_runner_handle.h"
-#include "components/history/core/browser/web_history_service.h"
-#include "components/sync/driver/sync_service.h"
-#include "components/version_info/version_info.h"
-
-namespace {
-
-// Merges several asynchronous boolean callbacks into one that returns a boolean
-// product of their responses. Deletes itself when done.
-class MergeBooleanCallbacks {
- public:
- // Constructor. Upon receiving |expected_call_count| calls to |RunCallback|,
- // |target_callback| will be run with the boolean product of their results.
- MergeBooleanCallbacks(
- int expected_call_count,
- const base::Callback<void(bool)>& target_callback)
- : expected_call_count_(expected_call_count),
- target_callback_(target_callback),
- final_response_(true),
- call_count_(0) {}
-
- // This method is to be bound to all asynchronous callbacks which we want
- // to merge.
- void RunCallback(bool response) {
- final_response_ &= response;
-
- if (++call_count_ < expected_call_count_)
- return;
-
- target_callback_.Run(final_response_);
- base::ThreadTaskRunnerHandle::Get()->DeleteSoon(FROM_HERE, this);
- }
-
- private:
- int expected_call_count_;
- base::Callback<void(bool)> target_callback_;
- bool final_response_;
- int call_count_;
-};
-
-} // namespace
-
-namespace browsing_data_ui {
-
-namespace testing {
-
-bool g_override_other_forms_of_browsing_history_query = false;
-
-} // namespace testing
-
-void ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(
- const sync_driver::SyncService* sync_service,
- history::WebHistoryService* history_service,
- base::Callback<void(bool)> callback) {
- if (!sync_service ||
- !sync_service->IsSyncActive() ||
- !sync_service->GetActiveDataTypes().Has(
- syncer::HISTORY_DELETE_DIRECTIVES) ||
- sync_service->IsUsingSecondaryPassphrase() ||
- !history_service) {
- callback.Run(false);
- return;
- }
-
- history_service->QueryWebAndAppActivity(callback);
-}
-
-void ShouldPopupDialogAboutOtherFormsOfBrowsingHistory(
- const sync_driver::SyncService* sync_service,
- history::WebHistoryService* history_service,
- version_info::Channel channel,
- base::Callback<void(bool)> callback) {
- // If the query for other forms of browsing history is overriden for testing,
- // the conditions are identical with
- // ShouldShowNoticeAboutOtherFormsOfBrowsingHistory.
- if (testing::g_override_other_forms_of_browsing_history_query) {
- ShouldShowNoticeAboutOtherFormsOfBrowsingHistory(
- sync_service, history_service, callback);
- return;
- }
-
- if (!sync_service ||
- !sync_service->IsSyncActive() ||
- !sync_service->GetActiveDataTypes().Has(
- syncer::HISTORY_DELETE_DIRECTIVES) ||
- sync_service->IsUsingSecondaryPassphrase() ||
- !history_service) {
- callback.Run(false);
- return;
- }
-
- // Return the boolean product of QueryWebAndAppActivity and
- // QueryOtherFormsOfBrowsingHistory. MergeBooleanCallbacks deletes itself
- // after processing both callbacks.
- MergeBooleanCallbacks* merger = new MergeBooleanCallbacks(2, callback);
- history_service->QueryWebAndAppActivity(base::Bind(
- &MergeBooleanCallbacks::RunCallback, base::Unretained(merger)));
- history_service->QueryOtherFormsOfBrowsingHistory(
- channel,
- base::Bind(
- &MergeBooleanCallbacks::RunCallback, base::Unretained(merger)));
-}
-
-} // namespace browsing_data_ui
« no previous file with comments | « components/browsing_data_ui/history_notice_utils.h ('k') | components/browsing_data_ui/history_notice_utils_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698