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

Unified Diff: ios/chrome/browser/browsing_data/browsing_data_counter_wrapper.h

Issue 2350773002: Add a browsing data counter factory on iOS (Closed)
Patch Set: Addressed comments 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: ios/chrome/browser/browsing_data/browsing_data_counter_wrapper.h
diff --git a/ios/chrome/browser/browsing_data/browsing_data_counter_wrapper.h b/ios/chrome/browser/browsing_data/browsing_data_counter_wrapper.h
new file mode 100644
index 0000000000000000000000000000000000000000..8361f231dd7b474616f63d55ae1b3f9b8419e435
--- /dev/null
+++ b/ios/chrome/browser/browsing_data/browsing_data_counter_wrapper.h
@@ -0,0 +1,60 @@
+// 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.
+
+#ifndef IOS_CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_WRAPPER_H_
+#define IOS_CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_WRAPPER_H_
+
+#include <memory>
+#include <string>
+
+#include "base/callback_forward.h"
+#include "components/browsing_data/core/counters/browsing_data_counter.h"
+
+class PrefService;
+
+namespace ios {
+class ChromeBrowserState;
+}
+
+// Wrapper around a browsing data volume counter that bridges the update counter
+// UI callback to the UI.
+class BrowsingDataCounterWrapper {
+ public:
+ using UpdateUICallback =
+ base::Callback<void(const browsing_data::BrowsingDataCounter::Result&)>;
+
+ // This method returns a nullptr if there is no corresponding counter for the
sdefresne 2016/09/20 11:50:46 nit: We usually do not use "nullptr" in comments,
ioanap 2016/09/20 12:43:29 Done.
+ // data type specified by the |pref_name|.
+ static std::unique_ptr<BrowsingDataCounterWrapper> CreateCounterWrapper(
+ const char* pref_name,
+ ios::ChromeBrowserState* browser_state,
+ PrefService* pref_service,
+ const UpdateUICallback& update_ui_callback);
+
+ ~BrowsingDataCounterWrapper();
+
+ void RestartCounter();
+
+ private:
+ BrowsingDataCounterWrapper(
+ std::unique_ptr<browsing_data::BrowsingDataCounter> counter,
+ PrefService* pref_service,
+ const UpdateUICallback& update_ui_callback);
+
+ // Method to be passed as callback to the counter. This will be invoked when
+ // the result is ready.
+ void UpdateWithResult(
+ std::unique_ptr<browsing_data::BrowsingDataCounter::Result> result);
+
+ // The counter for which the interaction is managed by this wrapper.
+ std::unique_ptr<browsing_data::BrowsingDataCounter> counter_;
+
+ // Callback that updates the UI once the counter result is ready. This is
+ // invoked by UpdateWithResult.
+ UpdateUICallback update_ui_callback_;
+
+ DISALLOW_COPY_AND_ASSIGN(BrowsingDataCounterWrapper);
+};
+
+#endif // IOS_CHROME_BROWSER_BROWSING_DATA_BROWSING_DATA_COUNTER_WRAPPER_

Powered by Google App Engine
This is Rietveld 408576698