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

Side by Side Diff: chrome/browser/chromeos/arc/arc_settings_bridge_impl.h

Issue 1523643002: arc-bridge: Move most methods to Mojo interfaces (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Rebased to ToT Created 5 years 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
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_settings_bridge_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 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 #ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_ 4 #ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_
5 #define CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_ 5 #define CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/gtest_prod_util.h" 9 #include "base/gtest_prod_util.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 // Starts listening to state changes of the ArcBridgeService. 49 // Starts listening to state changes of the ArcBridgeService.
50 // This must be called before the bridge service starts bootstrapping. 50 // This must be called before the bridge service starts bootstrapping.
51 void StartObservingBridgeServiceChanges() override; 51 void StartObservingBridgeServiceChanges() override;
52 52
53 // Called when a Chrome pref we have registered an observer for has changed. 53 // Called when a Chrome pref we have registered an observer for has changed.
54 // Obtains the new pref value and sends it to Android. 54 // Obtains the new pref value and sends it to Android.
55 void OnPrefChanged(const std::string& pref_name) const; 55 void OnPrefChanged(const std::string& pref_name) const;
56 56
57 // ArcBridgeService::Observer 57 // ArcBridgeService::Observer
58 void OnStateChanged(ArcBridgeService::State state) override; 58 void OnStateChanged(ArcBridgeService::State state) override;
59 59 void OnSettingsInstanceReady() override;
60 // ArcBridgeService::Observer
61 void OnInstanceBootPhase(InstanceBootPhase phase) override;
62 60
63 private: 61 private:
64 // Registers to observe changes for Chrome settings we care about. 62 // Registers to observe changes for Chrome settings we care about.
65 void StartObservingPrefChanges(); 63 void StartObservingPrefChanges();
66 64
67 // Stops listening for Chrome settings changes. 65 // Stops listening for Chrome settings changes.
68 void StopObservingPrefChanges(); 66 void StopObservingPrefChanges();
69 67
70 // Retrives Chrome's state for the settings and send it to Android. 68 // Retrives Chrome's state for the settings and send it to Android.
71 void SyncAllPrefs() const; 69 void SyncAllPrefs() const;
(...skipping 12 matching lines...) Expand all
84 82
85 // Manages pref observation registration. 83 // Manages pref observation registration.
86 PrefChangeRegistrar registrar_; 84 PrefChangeRegistrar registrar_;
87 85
88 DISALLOW_COPY_AND_ASSIGN(ArcSettingsBridgeImpl); 86 DISALLOW_COPY_AND_ASSIGN(ArcSettingsBridgeImpl);
89 }; 87 };
90 88
91 } // namespace arc 89 } // namespace arc
92 90
93 #endif // CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_ 91 #endif // CHROME_BROWSER_CHROMEOS_ARC_ARC_SETTINGS_BRIDGE_IMPL_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_settings_bridge_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698