brave · bsclifton · Jun 29, 2020 · Jun 29, 2020
@@ -464,10 +464,6 @@ By installing this extension, you are agreeing to the Google Widevine Terms of U
       <message name="IDS_SETTINGS_PUSH_MESSAGING" desc="Select value">
         Use Google Services for Push Messaging
       </message>
-      <!-- Remote Debugging -->
-      <message name="IDS_SETTINGS_REMOTE_DEBUGGING_TITLE" desc="Select value">
-        Remote debugging
-      </message>
       <!-- Brave Sync Settings -->
       <message name="IDS_SETTINGS_BRAVE_SYNC_TITLE" desc="The title for Brave Sync in settings">
         Sync

diff --git a/browser/brave_local_state_prefs.cc b/browser/brave_local_state_prefs.cc
@@ -44,7 +44,6 @@ namespace brave {
 void RegisterLocalStatePrefs(PrefRegistrySimple* registry) {
   brave_shields::RegisterPrefsForAdBlockService(registry);
   RegisterPrefsForBraveStatsUpdater(registry);
-  registry->RegisterBooleanPref(kRemoteDebuggingEnabled, true);
   ntp_background_images::NTPBackgroundImagesService::RegisterLocalStatePrefs(
       registry);
 #if BUILDFLAG(ENABLE_BRAVE_REFERRALS)

diff --git a/browser/resources/settings/brave_privacy_page/brave_personalization_options.html b/browser/resources/settings/brave_privacy_page/brave_personalization_options.html
@@ -48,12 +48,6 @@
         </cr-button>
       </template>
     </settings-toggle-button>
-    <settings-toggle-button id="remoteDebuggingEnabled"
-        pref=""
-        checked="[[remoteDebuggingEnabled_]]"
-        label="$i18n{remoteDebuggingEnabledTitle}"
-        on-settings-boolean-control-change="onRemoteDebuggingEnabledChange_">
-    </settings-toggle-button>
   </template>
   <script src="brave_personalization_options.js"></script>
 </dom-module>
diff --git a/browser/resources/settings/brave_privacy_page/brave_personalization_options.js b/browser/resources/settings/brave_privacy_page/brave_personalization_options.js
@@ -27,8 +27,7 @@ Polymer({
     },
 
     webRTCPolicy_: String,
-    p3aEnabled_: Boolean,
-    remoteDebuggingEnabled_: Boolean
+    p3aEnabled_: Boolean
   },
 
   /** @private {?settings.BravePrivacyBrowserProxy} */
@@ -51,12 +50,6 @@ Polymer({
     this.browserProxy_.getP3AEnabled().then(enabled => {
       this.p3aEnabled_ = enabled;
     });
-    this.browserProxy_.getRemoteDebuggingEnabled().then(enabled => {
-      this.remoteDebuggingEnabled_ = enabled;
-    });
-    this.addWebUIListener('remote-debugging-enabled-changed', (enabled) => {
-      this.remoteDebuggingEnabled_ = enabled
-    })
     this.addWebUIListener('p3a-enabled-changed', (enabled) => {
       this.p3aEnabled_ = enabled
     })
@@ -80,10 +73,6 @@ Polymer({
     this.browserProxy_.setP3AEnabled(this.$.p3aEnabled.checked);
   },
 
-  onRemoteDebuggingEnabledChange_: function() {
-    this.browserProxy_.setRemoteDebuggingEnabled(this.$.remoteDebuggingEnabled.checked);
-  },
-
   shouldShowRestart_: function(enabled) {
     return enabled != this.browserProxy_.wasPushMessagingEnabledAtStartup();
   },

diff --git a/browser/resources/settings/brave_privacy_page/brave_privacy_page_browser_proxy.js b/browser/resources/settings/brave_privacy_page/brave_privacy_page_browser_proxy.js
@@ -21,14 +21,6 @@ cr.define('settings', function() {
      * @param {boolean} enabled (true/false).
      */
     setP3AEnabled(value) {}
-    /**
-     * @return {!Promise<string>}
-     */
-    getRemoteDebuggingEnabled() {}
-    /**
-     * @param {boolean} enabled (true/false).
-     */
-    setRemoteDebuggingEnabled(value) {}
     /**
      * @return {boolean}
      */
@@ -56,14 +48,6 @@ cr.define('settings', function() {
       chrome.send('setP3AEnabled', [value])
     }
 
-    getRemoteDebuggingEnabled() {
-      return cr.sendWithPromise('getRemoteDebuggingEnabled');
-    }
-
-    setRemoteDebuggingEnabled(value) {
-      chrome.send('setRemoteDebuggingEnabled', [value])
-    }
-
     wasPushMessagingEnabledAtStartup() {
       return loadTimeData.getBoolean('pushMessagingEnabledAtStartup');
     }

diff --git a/browser/ui/webui/settings/brave_privacy_handler.cc b/browser/ui/webui/settings/brave_privacy_handler.cc
@@ -29,11 +29,6 @@
 
 BravePrivacyHandler::BravePrivacyHandler() {
   local_state_change_registrar_.Init(g_browser_process->local_state());
-  local_state_change_registrar_.Add(
-      kRemoteDebuggingEnabled,
-      base::Bind(&BravePrivacyHandler::OnRemoteDebuggingEnabledChanged,
-                 base::Unretained(this)));
-
 #if BUILDFLAG(BRAVE_P3A_ENABLED)
   local_state_change_registrar_.Add(
       brave::kP3AEnabled,
@@ -66,15 +61,6 @@ void BravePrivacyHandler::RegisterMessages() {
       "getP3AEnabled", base::BindRepeating(&BravePrivacyHandler::GetP3AEnabled,
                                            base::Unretained(this)));
 #endif
-
-  web_ui()->RegisterMessageCallback(
-      "setRemoteDebuggingEnabled",
-      base::BindRepeating(&BravePrivacyHandler::SetRemoteDebuggingEnabled,
-                          base::Unretained(this)));
-  web_ui()->RegisterMessageCallback(
-      "getRemoteDebuggingEnabled",
-      base::BindRepeating(&BravePrivacyHandler::GetRemoteDebuggingEnabled,
-                          base::Unretained(this)));
 }
 
 // static
@@ -143,34 +129,3 @@ void BravePrivacyHandler::OnP3AEnabledChanged() {
   }
 }
 #endif
-
-void BravePrivacyHandler::SetRemoteDebuggingEnabled(
-    const base::ListValue* args) {
-  CHECK_EQ(args->GetSize(), 1U);
-
-  bool enabled;
-  args->GetBoolean(0, &enabled);
-
-  PrefService* local_state = g_browser_process->local_state();
-  local_state->SetBoolean(kRemoteDebuggingEnabled, enabled);
-}
-
-void BravePrivacyHandler::GetRemoteDebuggingEnabled(
-    const base::ListValue* args) {
-  CHECK_EQ(args->GetSize(), 1U);
-
-  PrefService* local_state = g_browser_process->local_state();
-  bool enabled = local_state->GetBoolean(kRemoteDebuggingEnabled);
-
-  AllowJavascript();
-  ResolveJavascriptCallback(args->GetList()[0].Clone(), base::Value(enabled));
-}
-
-void BravePrivacyHandler::OnRemoteDebuggingEnabledChanged() {
-  if (IsJavascriptAllowed()) {
-    PrefService* local_state = g_browser_process->local_state();
-    bool enabled = local_state->GetBoolean(kRemoteDebuggingEnabled);
-
-    FireWebUIListener("remote-debugging-enabled-changed", base::Value(enabled));
-  }
-}
diff --git a/browser/ui/webui/settings/brave_privacy_handler.h b/browser/ui/webui/settings/brave_privacy_handler.h
@@ -38,10 +38,6 @@ class BravePrivacyHandler : public settings::SettingsPageUIHandler {
   void OnP3AEnabledChanged();
 #endif
 
-  void SetRemoteDebuggingEnabled(const base::ListValue* args);
-  void GetRemoteDebuggingEnabled(const base::ListValue* args);
-  void OnRemoteDebuggingEnabledChanged();
-
   Profile* profile_ = nullptr;
   PrefChangeRegistrar local_state_change_registrar_;
 

@@ -276,8 +276,6 @@ void BraveAddCommonStrings(content::WebUIDataSource* html_source,
       IDS_BRAVE_P3A_ENABLE_SETTING},
     {"p3aEnabledDesc",
       IDS_BRAVE_P3A_ENABLE_SETTING_SUBITEM},
-    {"remoteDebuggingEnabledTitle",
-      IDS_SETTINGS_REMOTE_DEBUGGING_TITLE},
     {"siteSettings",
       IDS_SETTINGS_SITE_AND_SHIELDS_SETTINGS},
   };

diff --git a/common/pref_names.cc b/common/pref_names.cc
@@ -78,7 +78,6 @@ const char kBinanceAccessToken[] = "brave.binance.access_token";
 const char kBinanceRefreshToken[] = "brave.binance.refresh_token";
 const char kAlwaysShowBookmarkBarOnNTP[] =
     "brave.always_show_bookmark_bar_on_ntp";
-const char kRemoteDebuggingEnabled[] = "brave.remote_debugging_enabled";
 const char kAutocompleteEnabled[] = "brave.autocomplete_enabled";
 const char kTopSiteSuggestionsEnabled[] = "brave.top_site_suggestions_enabled";
 const char kBraveSuggestedSiteSuggestionsEnabled[] =

diff --git a/common/pref_names.h b/common/pref_names.h
@@ -67,7 +67,6 @@ extern const char kBraveWalletWeb3Provider[];
 extern const char kLoadCryptoWalletsOnStartup[];
 extern const char kOptedIntoCryptoWallets[];
 extern const char kAlwaysShowBookmarkBarOnNTP[];
-extern const char kRemoteDebuggingEnabled[];
 extern const char kAutocompleteEnabled[];
 extern const char kTopSiteSuggestionsEnabled[];
 extern const char kBraveSuggestedSiteSuggestionsEnabled[];