diff options
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/console/02_console.js | 57 | ||||
-rw-r--r-- | extensions/console/internal.d.ts | 16 | ||||
-rw-r--r-- | extensions/fetch/23_request.js | 20 | ||||
-rw-r--r-- | extensions/fetch/23_response.js | 26 | ||||
-rw-r--r-- | extensions/timers/02_performance.js | 42 | ||||
-rw-r--r-- | extensions/web/01_dom_exception.js | 17 | ||||
-rw-r--r-- | extensions/web/02_event.js | 127 | ||||
-rw-r--r-- | extensions/web/06_streams.js | 69 | ||||
-rw-r--r-- | extensions/web/09_file.js | 10 |
9 files changed, 264 insertions, 120 deletions
diff --git a/extensions/console/02_console.js b/extensions/console/02_console.js index 6f2ed0c0e..b4b29e1ea 100644 --- a/extensions/console/02_console.js +++ b/extensions/console/02_console.js @@ -87,6 +87,10 @@ MathFloor, Number, NumberPrototypeToString, + Proxy, + ReflectGet, + ReflectGetOwnPropertyDescriptor, + ReflectGetPrototypeOf, WeakMap, WeakSet, } = window.__bootstrap.primordials; @@ -1953,6 +1957,58 @@ }); } + /** Creates a proxy that represents a subset of the properties + * of the original object optionally without evaluating the properties + * in order to get the values. */ + function createFilteredInspectProxy({ object, keys, evaluate }) { + return new Proxy({}, { + get(_target, key) { + if (key === SymbolToStringTag) { + return object.constructor?.name; + } else if (ArrayPrototypeIncludes(keys, key)) { + return ReflectGet(object, key); + } else { + return undefined; + } + }, + getOwnPropertyDescriptor(_target, key) { + if (!ArrayPrototypeIncludes(keys, key)) { + return undefined; + } else if (evaluate) { + return getEvaluatedDescriptor(object, key); + } else { + return getDescendantPropertyDescriptor(object, key) ?? + getEvaluatedDescriptor(object, key); + } + }, + has(_target, key) { + return ArrayPrototypeIncludes(keys, key); + }, + ownKeys() { + return keys; + }, + }); + + function getDescendantPropertyDescriptor(object, key) { + let propertyDescriptor = ReflectGetOwnPropertyDescriptor(object, key); + if (!propertyDescriptor) { + const prototype = ReflectGetPrototypeOf(object); + if (prototype) { + propertyDescriptor = getDescendantPropertyDescriptor(prototype, key); + } + } + return propertyDescriptor; + } + + function getEvaluatedDescriptor(object, key) { + return { + configurable: true, + enumerable: true, + value: object[key], + }; + } + } + // A helper function that will bind our own console implementation // with default implementation of Console from V8. This will cause // console messages to be piped to inspector console. @@ -1997,5 +2053,6 @@ customInspect, inspect, wrapConsole, + createFilteredInspectProxy, }; })(this); diff --git a/extensions/console/internal.d.ts b/extensions/console/internal.d.ts new file mode 100644 index 000000000..ef7834ba6 --- /dev/null +++ b/extensions/console/internal.d.ts @@ -0,0 +1,16 @@ +// Copyright 2018-2021 the Deno authors. All rights reserved. MIT license. + +/// <reference no-default-lib="true" /> +/// <reference lib="esnext" /> + +declare namespace globalThis { + declare namespace __bootstrap { + declare namespace console { + declare function createFilteredInspectProxy<TObject>(params: { + object: TObject; + keys: (keyof TObject)[]; + evaluate: boolean; + }): Record<string, unknown>; + } + } +} diff --git a/extensions/fetch/23_request.js b/extensions/fetch/23_request.js index 829f7e6dc..1372125c1 100644 --- a/extensions/fetch/23_request.js +++ b/extensions/fetch/23_request.js @@ -12,6 +12,7 @@ ((window) => { const webidl = window.__bootstrap.webidl; + const consoleInternal = window.__bootstrap.console; const { HTTP_TOKEN_CODE_POINT_RE, byteUpperCase } = window.__bootstrap.infra; const { URL } = window.__bootstrap.url; const { guardFromHeaders } = window.__bootstrap.headers; @@ -393,14 +394,17 @@ } [SymbolFor("Deno.customInspect")](inspect) { - const inner = { - bodyUsed: this.bodyUsed, - headers: this.headers, - method: this.method, - redirect: this.redirect, - url: this.url, - }; - return `Request ${inspect(inner)}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof Request, + keys: [ + "bodyUsed", + "headers", + "method", + "redirect", + "url", + ], + })); } } diff --git a/extensions/fetch/23_response.js b/extensions/fetch/23_response.js index 11eb13570..0db20e90e 100644 --- a/extensions/fetch/23_response.js +++ b/extensions/fetch/23_response.js @@ -13,6 +13,7 @@ ((window) => { const webidl = window.__bootstrap.webidl; + const consoleInternal = window.__bootstrap.console; const { HTTP_TAB_OR_SPACE, regexMatcher } = window.__bootstrap.infra; const { extractBody, mixinBody } = window.__bootstrap.fetchBody; const { getLocationHref } = window.__bootstrap.location; @@ -377,17 +378,20 @@ } [SymbolFor("Deno.customInspect")](inspect) { - const inner = { - body: this.body, - bodyUsed: this.bodyUsed, - headers: this.headers, - ok: this.ok, - redirected: this.redirected, - status: this.status, - statusText: this.statusText, - url: this.url, - }; - return `Response ${inspect(inner)}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof Response, + keys: [ + "body", + "bodyUsed", + "headers", + "ok", + "redirected", + "status", + "statusText", + "url", + ], + })); } } diff --git a/extensions/timers/02_performance.js b/extensions/timers/02_performance.js index b4cf5760b..f752ba933 100644 --- a/extensions/timers/02_performance.js +++ b/extensions/timers/02_performance.js @@ -16,6 +16,7 @@ } = window.__bootstrap.primordials; const { webidl, structuredClone } = window.__bootstrap; + const consoleInternal = window.__bootstrap.console; const { opNow } = window.__bootstrap.timers; const { DOMException } = window.__bootstrap.domException; @@ -175,7 +176,16 @@ } [customInspect](inspect) { - return `${this.constructor.name} ${inspect(this.toJSON())}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof PerformanceEntry, + keys: [ + "name", + "entryType", + "startTime", + "duration", + ], + })); } } webidl.configurePrototype(PerformanceEntry); @@ -235,7 +245,17 @@ } [customInspect](inspect) { - return `${this.constructor.name} ${inspect(this.toJSON())}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof PerformanceMark, + keys: [ + "name", + "entryType", + "startTime", + "duration", + "detail", + ], + })); } } webidl.configurePrototype(PerformanceMark); @@ -283,7 +303,17 @@ } [customInspect](inspect) { - return `${this.constructor.name} ${inspect(this.toJSON())}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof PerformanceMeasure, + keys: [ + "name", + "entryType", + "startTime", + "duration", + "detail", + ], + })); } } webidl.configurePrototype(PerformanceMeasure); @@ -516,7 +546,11 @@ } [customInspect](inspect) { - return `${this.constructor.name} ${inspect(this.toJSON())}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof Performance, + keys: [], + })); } get [SymbolToStringTag]() { diff --git a/extensions/web/01_dom_exception.js b/extensions/web/01_dom_exception.js index 90ddb267b..3e282d969 100644 --- a/extensions/web/01_dom_exception.js +++ b/extensions/web/01_dom_exception.js @@ -17,6 +17,7 @@ ObjectSetPrototypeOf, } = window.__bootstrap.primordials; const webidl = window.__bootstrap.webidl; + const consoleInternal = window.__bootstrap.console; // Defined in WebIDL 4.3. // https://heycam.github.io/webidl/#idl-DOMException @@ -109,8 +110,20 @@ return "DOMException"; } - [Symbol.for("Deno.customInspect")]() { - return `DOMException: ${this.#message}`; + [Symbol.for("Deno.customInspect")](inspect) { + if (this instanceof DOMException) { + return `DOMException: ${this.#message}`; + } else { + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: false, + keys: [ + "message", + "name", + "code", + ], + })); + } } } diff --git a/extensions/web/02_event.js b/extensions/web/02_event.js index 4ae8b50f2..6b2cc2c04 100644 --- a/extensions/web/02_event.js +++ b/extensions/web/02_event.js @@ -9,6 +9,7 @@ ((window) => { const webidl = window.__bootstrap.webidl; const { DOMException } = window.__bootstrap.domException; + const consoleInternal = window.__bootstrap.console; const { ArrayPrototypeFilter, ArrayPrototypeIncludes, @@ -28,10 +29,7 @@ ObjectCreate, ObjectDefineProperty, ObjectGetOwnPropertyDescriptor, - Proxy, ReflectDefineProperty, - ReflectGet, - ReflectGetOwnPropertyDescriptor, Symbol, SymbolFor, SymbolToStringTag, @@ -175,7 +173,11 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, EVENT_PROPS)); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof Event, + keys: EVENT_PROPS, + })); } get type() { @@ -397,43 +399,6 @@ } } - function buildFilteredPropertyInspectObject(object, keys) { - // forward the subset of properties from `object` without evaluating - // as evaluation could lead to an error, which is better handled - // in the inspect code - return new Proxy({}, { - get(_target, key) { - if (key === SymbolToStringTag) { - return object.constructor?.name; - } else if (ArrayPrototypeIncludes(keys, key)) { - return ReflectGet(object, key); - } else { - return undefined; - } - }, - getOwnPropertyDescriptor(_target, key) { - if (!ArrayPrototypeIncludes(keys, key)) { - return undefined; - } - - return ReflectGetOwnPropertyDescriptor(object, key) ?? - (object.prototype && - ReflectGetOwnPropertyDescriptor(object.prototype, key)) ?? - { - configurable: true, - enumerable: true, - value: object[key], - }; - }, - has(_target, key) { - return ArrayPrototypeIncludes(keys, key); - }, - ownKeys() { - return keys; - }, - }); - } - function defineEnumerableProps( Ctor, props, @@ -1097,14 +1062,18 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, [ - ...EVENT_PROPS, - "message", - "filename", - "lineno", - "colno", - "error", - ])); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof ErrorEvent, + keys: [ + ...EVENT_PROPS, + "message", + "filename", + "lineno", + "colno", + "error", + ], + })); } } @@ -1151,12 +1120,16 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, [ - ...EVENT_PROPS, - "wasClean", - "code", - "reason", - ])); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof CloseEvent, + keys: [ + ...EVENT_PROPS, + "wasClean", + "code", + "reason", + ], + })); } } @@ -1179,12 +1152,16 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, [ - ...EVENT_PROPS, - "data", - "origin", - "lastEventId", - ])); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof MessageEvent, + keys: [ + ...EVENT_PROPS, + "data", + "origin", + "lastEventId", + ], + })); } } @@ -1209,10 +1186,14 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, [ - ...EVENT_PROPS, - "detail", - ])); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof CustomEvent, + keys: [ + ...EVENT_PROPS, + "detail", + ], + })); } } @@ -1232,12 +1213,16 @@ } [SymbolFor("Deno.privateCustomInspect")](inspect) { - return inspect(buildFilteredPropertyInspectObject(this, [ - ...EVENT_PROPS, - "lengthComputable", - "loaded", - "total", - ])); + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof ProgressEvent, + keys: [ + ...EVENT_PROPS, + "lengthComputable", + "loaded", + "total", + ], + })); } } diff --git a/extensions/web/06_streams.js b/extensions/web/06_streams.js index 388b7b13c..ff6c9d7d8 100644 --- a/extensions/web/06_streams.js +++ b/extensions/web/06_streams.js @@ -36,6 +36,7 @@ WeakMapPrototypeHas, WeakMapPrototypeSet, } = globalThis.__bootstrap.primordials; + const consoleInternal = window.__bootstrap.console; const { DOMException } = window.__bootstrap.domException; class AssertionError extends Error { @@ -3018,9 +3019,14 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ highWaterMark: this.highWaterMark, size: this.size }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof ByteLengthQueuingStrategy, + keys: [ + "highWaterMark", + "size", + ], + })); } get [Symbol.toStringTag]() { @@ -3069,9 +3075,14 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ highWaterMark: this.highWaterMark, size: this.size }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof CountQueuingStrategy, + keys: [ + "highWaterMark", + "size", + ], + })); } get [Symbol.toStringTag]() { @@ -3561,9 +3572,11 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ desiredSize: this.desiredSize }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof ReadableByteStreamController, + keys: ["desiredSize"], + })); } get [Symbol.toStringTag]() { @@ -3684,9 +3697,11 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ desiredSize: this.desiredSize }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof ReadableStreamDefaultController, + keys: ["desiredSize"], + })); } get [Symbol.toStringTag]() { @@ -3905,9 +3920,11 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ desiredSize: this.desiredSize }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof TransformStreamDefaultController, + keys: ["desiredSize"], + })); } get [Symbol.toStringTag]() { @@ -4182,13 +4199,15 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${ - inspect({ - closed: this.closed, - desiredSize: this.desiredSize, - ready: this.ready, - }) - }`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof WritableStreamDefaultWriter, + keys: [ + "closed", + "desiredSize", + "ready", + ], + })); } get [Symbol.toStringTag]() { @@ -4240,7 +4259,11 @@ } [Symbol.for("Deno.customInspect")](inspect) { - return `${this.constructor.name} ${inspect({})}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof WritableStreamDefaultController, + keys: [], + })); } get [Symbol.toStringTag]() { diff --git a/extensions/web/09_file.js b/extensions/web/09_file.js index a39b380a6..516e80adf 100644 --- a/extensions/web/09_file.js +++ b/extensions/web/09_file.js @@ -34,6 +34,7 @@ TypeError, Uint8Array, } = window.__bootstrap.primordials; + const consoleInternal = window.__bootstrap.console; // TODO(lucacasonato): this needs to not be hardcoded and instead depend on // host os. @@ -362,7 +363,14 @@ } [SymbolFor("Deno.customInspect")](inspect) { - return `Blob ${inspect({ size: this.size, type: this.#type })}`; + return inspect(consoleInternal.createFilteredInspectProxy({ + object: this, + evaluate: this instanceof Blob, + keys: [ + "size", + "type", + ], + })); } } |