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

Side by Side Diff: chrome/browser/ui/webui/settings/settings_default_browser_handler.h

Issue 1657933003: Fixes the interactive default browser UX for policy setting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix merge issue Created 4 years, 10 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_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_ 5 #ifndef CHROME_BROWSER_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_
6 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_ 6 #define CHROME_BROWSER_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "chrome/browser/profiles/profile.h" 9 #include "chrome/browser/profiles/profile.h"
10 #include "chrome/browser/shell_integration.h" 10 #include "chrome/browser/shell_integration.h"
(...skipping 19 matching lines...) Expand all
30 public: 30 public:
31 explicit DefaultBrowserHandler(content::WebUI* webui); 31 explicit DefaultBrowserHandler(content::WebUI* webui);
32 ~DefaultBrowserHandler() override; 32 ~DefaultBrowserHandler() override;
33 33
34 // SettingsPageUIHandler implementation. 34 // SettingsPageUIHandler implementation.
35 void RegisterMessages() override; 35 void RegisterMessages() override;
36 36
37 // shell_integration::DefaultWebClientObserver implementation. 37 // shell_integration::DefaultWebClientObserver implementation.
38 void SetDefaultWebClientUIState( 38 void SetDefaultWebClientUIState(
39 shell_integration::DefaultWebClientUIState state) override; 39 shell_integration::DefaultWebClientUIState state) override;
40 bool IsInteractiveSetDefaultPermitted() override;
41 void OnSetAsDefaultConcluded(bool succeeded) override; 40 void OnSetAsDefaultConcluded(bool succeeded) override;
42 41
43 private: 42 private:
44 // Called from WebUI to request the current state. 43 // Called from WebUI to request the current state.
45 void RequestDefaultBrowserState(const base::ListValue* args); 44 void RequestDefaultBrowserState(const base::ListValue* args);
46 45
47 // Makes this the default browser. Called from WebUI. 46 // Makes this the default browser. Called from WebUI.
48 void SetAsDefaultBrowser(const base::ListValue* args); 47 void SetAsDefaultBrowser(const base::ListValue* args);
49 48
50 // Reference to a background worker that handles default browser settings. 49 // Reference to a background worker that handles default browser settings.
51 scoped_refptr<shell_integration::DefaultBrowserWorker> 50 scoped_refptr<shell_integration::DefaultBrowserWorker>
52 default_browser_worker_; 51 default_browser_worker_;
53 52
54 // Policy setting to determine if default browser setting is managed. 53 // Policy setting to determine if default browser setting is managed.
55 BooleanPrefMember default_browser_policy_; 54 BooleanPrefMember default_browser_policy_;
56 55
57 DISALLOW_COPY_AND_ASSIGN(DefaultBrowserHandler); 56 DISALLOW_COPY_AND_ASSIGN(DefaultBrowserHandler);
58 }; 57 };
59 58
60 } // namespace settings 59 } // namespace settings
61 60
62 #endif // CHROME_BROWSER_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_ 61 #endif // CHROME_BROWSER_UI_WEBUI_SETTINGS_SETTINGS_DEFAULT_BROWSER_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698