Skip to content

Commit

Permalink
fix: some more console spec compliance (#10983)
Browse files Browse the repository at this point in the history
  • Loading branch information
lucacasonato committed Jun 16, 2021
1 parent 8e4a70c commit 77ab2be
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
4 changes: 2 additions & 2 deletions extensions/console/02_console.js
Original file line number Diff line number Diff line change
Expand Up @@ -1521,7 +1521,7 @@
);
};

dir = (obj, options = {}) => {
dir = (obj = undefined, options = {}) => {
this.#printFunc(
inspectArgs([obj], { ...getConsoleInspectOptions(), ...options }) +
"\n",
Expand Down Expand Up @@ -1594,7 +1594,7 @@
}
};

table = (data, properties) => {
table = (data = undefined, properties = undefined) => {
if (properties !== undefined && !Array.isArray(properties)) {
throw new Error(
"The 'properties' argument must be of type Array. " +
Expand Down
6 changes: 1 addition & 5 deletions tools/wpt/expectation.json
Original file line number Diff line number Diff line change
Expand Up @@ -144,11 +144,7 @@
"console-label-conversion.any.html": true,
"console-namespace-object-class-string.any.html": true,
"console-tests-historical.any.html": true,
"idlharness.any.html": [
"console namespace: operation table(optional any, optional sequence<DOMString>)",
"console namespace: operation dir(optional any, optional object?)",
"console namespace: operation dirxml(any...)"
]
"idlharness.any.html": true
},
"dom": {
"abort": {
Expand Down

0 comments on commit 77ab2be

Please sign in to comment.