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

Side by Side Diff: chrome/browser/android/webapps/webapp_registry.h

Issue 2384413003: Revert of Refactor WebappRegistry into a singleton instance. (Closed)
Patch Set: Created 4 years, 2 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_ 5 #ifndef CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_
6 #define CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_ 6 #define CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_
7 7
8 #include "base/android/jni_android.h"
9 #include "base/android/scoped_java_ref.h"
8 #include "base/callback_forward.h" 10 #include "base/callback_forward.h"
9 #include "base/macros.h" 11 #include "base/macros.h"
10 12
11 class GURL; 13 class GURL;
12 14
13 // WebappRegistry is the C++ counterpart of 15 // WebappRegistry is the C++ counterpart of
14 // org.chromium.chrome.browser.webapp's WebappRegistry in Java. 16 // org.chromium.chrome.browser.webapp's WebappRegistry in Java.
15 // All methods in this class which make JNI calls should be declared virtual and 17 // All methods in this class which make JNI calls should be declared virtual and
16 // mocked out in C++ unit tests. The JNI call cannot be made in this environment 18 // mocked out in C++ unit tests. The JNI call cannot be made in this environment
17 // as the Java side will not be initialised. 19 // as the Java side will not be initialised.
18 class WebappRegistry { 20 class WebappRegistry {
19 public: 21 public:
20 WebappRegistry() { } 22 WebappRegistry() { }
21 virtual ~WebappRegistry() { } 23 virtual ~WebappRegistry() { }
22 24
25 // Registers JNI hooks.
26 static bool RegisterWebappRegistry(JNIEnv* env);
27
23 // Cleans up data stored by web apps on URLs matching |url_filter|. 28 // Cleans up data stored by web apps on URLs matching |url_filter|.
24 virtual void UnregisterWebappsForUrls( 29 virtual void UnregisterWebappsForUrls(
25 const base::Callback<bool(const GURL&)>& url_filter); 30 const base::Callback<bool(const GURL&)>& url_filter,
31 const base::Closure& callback);
26 32
27 // Removes history data (last used time and URLs) stored by web apps with 33 // Removes history data (last used time and URLs) stored by web apps with
28 // URLs matching |url_filter|, whilst leaving other data intact. 34 // URLs matching |url_filter|, whilst leaving other data intact.
29 virtual void ClearWebappHistoryForUrls( 35 virtual void ClearWebappHistoryForUrls(
30 const base::Callback<bool(const GURL&)>& url_filter); 36 const base::Callback<bool(const GURL&)>& url_filter,
37 const base::Closure& callback);
31 38
32 private: 39 private:
33 DISALLOW_COPY_AND_ASSIGN(WebappRegistry); 40 DISALLOW_COPY_AND_ASSIGN(WebappRegistry);
34 }; 41 };
35 42
36 #endif // CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_ 43 #endif // CHROME_BROWSER_ANDROID_WEBAPPS_WEBAPP_REGISTRY_H_
OLDNEW
« no previous file with comments | « chrome/browser/android/chrome_jni_registrar.cc ('k') | chrome/browser/android/webapps/webapp_registry.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698