summaryrefslogtreecommitdiffstats
path: root/devtools
diff options
context:
space:
mode:
authorAscrod <32915892+Ascrod@users.noreply.github.com>2019-04-08 20:05:53 -0400
committerAscrod <32915892+Ascrod@users.noreply.github.com>2019-04-13 11:37:45 -0400
commit609a7215e6ef705fd098fd4adb4f922795ff1119 (patch)
treecba7742899754d237f2eab6d74634f4f23fd7667 /devtools
parent7591326915a100b54ca17ad6fedb391645bac6b5 (diff)
downloadUXP-609a7215e6ef705fd098fd4adb4f922795ff1119.tar
UXP-609a7215e6ef705fd098fd4adb4f922795ff1119.tar.gz
UXP-609a7215e6ef705fd098fd4adb4f922795ff1119.tar.lz
UXP-609a7215e6ef705fd098fd4adb4f922795ff1119.tar.xz
UXP-609a7215e6ef705fd098fd4adb4f922795ff1119.zip
Issue #991 Part 3: Devtools
Diffstat (limited to 'devtools')
-rw-r--r--devtools/client/framework/devtools.js7
-rw-r--r--devtools/client/framework/toolbox-process-window.js17
-rw-r--r--devtools/client/framework/toolbox.js7
-rw-r--r--devtools/client/inspector/markup/markup.js7
-rw-r--r--devtools/shared/touch/simulator-core.js15
5 files changed, 12 insertions, 41 deletions
diff --git a/devtools/client/framework/devtools.js b/devtools/client/framework/devtools.js
index 90f88023b..976a4b56d 100644
--- a/devtools/client/framework/devtools.js
+++ b/devtools/client/framework/devtools.js
@@ -195,12 +195,7 @@ DevTools.prototype = {
return tool;
}
- let enabled;
- try {
- enabled = Services.prefs.getBoolPref(tool.visibilityswitch);
- } catch (e) {
- enabled = true;
- }
+ let enabled = Services.prefs.getBoolPref(tool.visibilityswitch, true);
return enabled ? tool : null;
},
diff --git a/devtools/client/framework/toolbox-process-window.js b/devtools/client/framework/toolbox-process-window.js
index 8ead718b3..c34fe3355 100644
--- a/devtools/client/framework/toolbox-process-window.js
+++ b/devtools/client/framework/toolbox-process-window.js
@@ -100,17 +100,12 @@ function openToolbox({ form, chrome, isTabActor }) {
};
TargetFactory.forRemoteTab(options).then(target => {
let frame = document.getElementById("toolbox-iframe");
- let selectedTool = "jsdebugger";
-
- try {
- // Remember the last panel that was used inside of this profile.
- selectedTool = Services.prefs.getCharPref("devtools.toolbox.selectedTool");
- } catch(e) {}
-
- try {
- // But if we are testing, then it should always open the debugger panel.
- selectedTool = Services.prefs.getCharPref("devtools.browsertoolbox.panel");
- } catch(e) {}
+ // Remember the last panel that was used inside of this profile.
+ // But if we are testing, then it should always open the debugger panel.
+ let selectedTool =
+ Services.prefs.getCharPref("devtools.toolbox.selectedTool",
+ Services.prefs.getCharPref("devtools.browsertoolbox.panel",
+ "jsdebugger"));
let options = { customIframe: frame };
gDevTools.showToolbox(target,
diff --git a/devtools/client/framework/toolbox.js b/devtools/client/framework/toolbox.js
index 926e30647..cde7de0e4 100644
--- a/devtools/client/framework/toolbox.js
+++ b/devtools/client/framework/toolbox.js
@@ -1131,12 +1131,7 @@ Toolbox.prototype = {
setToolboxButtonsVisibility: function () {
this.toolboxButtons.forEach(buttonSpec => {
let { visibilityswitch, button, isTargetSupported } = buttonSpec;
- let on = true;
- try {
- on = Services.prefs.getBoolPref(visibilityswitch);
- } catch (ex) {
- // Do nothing.
- }
+ let on = Services.prefs.getBoolPref(visibilityswitch, true);
on = on && isTargetSupported(this.target);
diff --git a/devtools/client/inspector/markup/markup.js b/devtools/client/inspector/markup/markup.js
index d6e9f8c11..a4b65c899 100644
--- a/devtools/client/inspector/markup/markup.js
+++ b/devtools/client/inspector/markup/markup.js
@@ -73,11 +73,8 @@ function MarkupView(inspector, frame, controllerWindow) {
this._elt = this.doc.querySelector("#root");
this.htmlEditor = new HTMLEditor(this.doc);
- try {
- this.maxChildren = Services.prefs.getIntPref("devtools.markup.pagesize");
- } catch (ex) {
- this.maxChildren = DEFAULT_MAX_CHILDREN;
- }
+ this.maxChildren = Services.prefs.getIntPref("devtools.markup.pagesize",
+ DEFAULT_MAX_CHILDREN);
this.collapseAttributes =
Services.prefs.getBoolPref(ATTR_COLLAPSE_ENABLED_PREF);
diff --git a/devtools/shared/touch/simulator-core.js b/devtools/shared/touch/simulator-core.js
index 6933f9207..fff7d2e78 100644
--- a/devtools/shared/touch/simulator-core.js
+++ b/devtools/shared/touch/simulator-core.js
@@ -18,19 +18,8 @@ var systemAppOrigin = (function () {
return systemOrigin;
})();
-var threshold = 25;
-try {
- threshold = Services.prefs.getIntPref("ui.dragThresholdX");
-} catch (e) {
- // Fall back to default value
-}
-
-var delay = 500;
-try {
- delay = Services.prefs.getIntPref("ui.click_hold_context_menus.delay");
-} catch (e) {
- // Fall back to default value
-}
+var threshold = Services.prefs.getIntPref("ui.dragThresholdX", 25);
+var delay = Services.prefs.getIntPref("ui.click_hold_context_menus.delay", 500);
function SimulatorCore(simulatorTarget) {
this.simulatorTarget = simulatorTarget;