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

Side by Side Diff: chrome/browser/sync/glue/extensions_activity_monitor.h

Issue 645043003: Cleanup: Prevent usage of various extension headers when extensions support is not compiled in. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_ 5 #ifndef CHROME_BROWSER_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_
6 #define CHROME_BROWSER_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_ 6 #define CHROME_BROWSER_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_
7 7
8 #include "base/compiler_specific.h" 8 #include "base/macros.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "content/public/browser/notification_observer.h" 10 #include "content/public/browser/notification_observer.h"
11 #include "content/public/browser/notification_registrar.h" 11 #include "content/public/browser/notification_registrar.h"
12 12
13 namespace syncer { 13 namespace syncer {
14 class ExtensionsActivity; 14 class ExtensionsActivity;
15 } 15 }
16 16
17 namespace browser_sync { 17 namespace browser_sync {
18 18
19 // Observe and record usage of extension bookmark API. 19 // Observe and record usage of extension bookmark API.
20 class ExtensionsActivityMonitor : public content::NotificationObserver { 20 class ExtensionsActivityMonitor : public content::NotificationObserver {
21 public: 21 public:
22 ExtensionsActivityMonitor(); 22 ExtensionsActivityMonitor();
23 virtual ~ExtensionsActivityMonitor(); 23 virtual ~ExtensionsActivityMonitor();
24 24
25 // content::NotificationObserver implementation. 25 // content::NotificationObserver implementation.
26 virtual void Observe(int type, 26 virtual void Observe(int type,
27 const content::NotificationSource& source, 27 const content::NotificationSource& source,
28 const content::NotificationDetails& details) override; 28 const content::NotificationDetails& details) override;
29 29
30 const scoped_refptr<syncer::ExtensionsActivity>& GetExtensionsActivity(); 30 const scoped_refptr<syncer::ExtensionsActivity>& GetExtensionsActivity();
31 31
32 private: 32 private:
33 scoped_refptr<syncer::ExtensionsActivity> extensions_activity_; 33 scoped_refptr<syncer::ExtensionsActivity> extensions_activity_;
34 34
35 #if defined(ENABLE_EXTENSIONS)
35 // Used only on UI loop. 36 // Used only on UI loop.
36 content::NotificationRegistrar registrar_; 37 content::NotificationRegistrar registrar_;
38 #endif
39
40 DISALLOW_COPY_AND_ASSIGN(ExtensionsActivityMonitor);
37 }; 41 };
38 42
39 } // namespace browser_sync 43 } // namespace browser_sync
40 44
41 #endif // CHROME_BROWSER_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_ 45 #endif // CHROME_BROWSER_SYNC_GLUE_EXTENSIONS_ACTIVITY_MONITOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698