Skip to content

Commit

Permalink
dom: Hide debugging extension methods behind a pref.
Browse files Browse the repository at this point in the history
  • Loading branch information
jdm committed Jul 15, 2020
1 parent 9fe92ec commit e6cdfdd
Show file tree
Hide file tree
Showing 14 changed files with 19 additions and 4 deletions.
3 changes: 3 additions & 0 deletions components/config/prefs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -187,6 +187,9 @@ mod gen {
enabled: bool,
timeout_seconds: i64,
},
servo_helpers: {
enabled: bool,
},
servoparser: {
async_html_tokenizer: {
enabled: bool,
Expand Down
3 changes: 3 additions & 0 deletions components/script/dom/webidls/Window.webidl
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,11 @@ partial interface Window {

// Proprietary extensions.
partial interface Window {
[Pref="dom.servo_helpers.enabled"]
void debug(DOMString arg);
[Pref="dom.servo_helpers.enabled"]
void gc();
[Pref="dom.servo_helpers.enabled"]
void trap();
};

Expand Down
1 change: 1 addition & 0 deletions resources/prefs.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
"dom.permissions.testing.allowed_in_nonsecure_contexts": false,
"dom.serviceworker.enabled": false,
"dom.serviceworker.timeout_seconds": 60,
"dom.servo_helpers.enabled": false,
"dom.servoparser.async_html_tokenizer.enabled": false,
"dom.shadowdom.enabled": false,
"dom.svg.enabled": false,
Expand Down
1 change: 1 addition & 0 deletions resources/user-agent-js/00.example.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@
// Files are sorted alphabetically. When committing polyfills
// order them with numbers, e.g. `01.innerhtml.js` will be executed before
// `05.jquery.js`
onunhandledrejection = (e) => console.error("xxxjdm error: " + JSON.stringify(e.reason));
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
5 changes: 2 additions & 3 deletions tests/wpt/mozilla/meta-layout-2020/mozilla/weakref.html.ini
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
[weakref.html]
[Weak references work]
expected: FAIL

type: testharness
prefs: ["dom.testbinding.enabled:true", "dom.servo_helpers.enabled:true"]
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
1 change: 1 addition & 0 deletions tests/wpt/mozilla/meta/mozilla/prototypes.html.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
1 change: 1 addition & 0 deletions tests/wpt/mozilla/meta/mozilla/trace_null.html.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
prefs: ["dom.servo_helpers.enabled:true"]
2 changes: 1 addition & 1 deletion tests/wpt/mozilla/meta/mozilla/weakref.html.ini
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
[weakref.html]
type: testharness
prefs: [dom.testbinding.enabled:true]
prefs: ["dom.testbinding.enabled:true", "dom.servo_helpers.enabled:true"]

0 comments on commit e6cdfdd

Please sign in to comment.