Summary: | Web Inspector: ES6: Improved Console Support for Promise Objects | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||||||||
Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | buildbot, burg, graouts, joepeck, jonowells, mattbaker, nvasilyev, ossy, rniwa, timothy, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | 141746 | ||||||||||||
Bug Blocks: | |||||||||||||
Attachments: |
|
Description
Joseph Pecoraro
2015-02-16 00:01:49 PST
Created attachment 246637 [details]
[PATCH] Work In Progress
Work in progress patch:
- First time we make sure of InjectedScriptHost.getInternalProperties
- Add a way to distinguish Internal property previews from regular properties
- Trivially supports [[PromiseState]] and [[PromiseResult]].
Whats next:
- show the chain of reactions next (the .thens, and .catches)
- discuss UI for Internal Properties in ObjectPreviews / ObjectTrees
> [[PromiseFulfillReactions]] > - A List of PromiseReaction records to be processed when/if the promise > transitions from the "pending" state to the "fulfilled" state. > > [[PromiseRejectReactions]] > - A List of PromiseReaction records to be processed when/if the promise > transitions from the "pending" state to the "rejected" state. I will handle these states in: <https://webkit.org/b/141664> Web Inspector: ES6: Improved Support for Promises - Promise Reactions For now, lets just do state / value. Created attachment 246675 [details]
[PATCH] Proposed Fix
Comment on attachment 246675 [details] [PATCH] Proposed Fix Attachment 246675 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/5867389241524224 New failing tests: inspector-protocol/runtime/getProperties.html Created attachment 246682 [details]
Archive of layout-test-results from ews102 for mac-mavericks
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102 Port: mac-mavericks Platform: Mac OS X 10.9.5
Comment on attachment 246675 [details] [PATCH] Proposed Fix Attachment 246675 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/4929187995451392 New failing tests: inspector-protocol/runtime/getProperties.html Created attachment 246683 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105 Port: mac-mavericks-wk2 Platform: Mac OS X 10.9.5
Comment on attachment 246675 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=246675&action=review > Source/JavaScriptCore/inspector/InjectedScript.cpp:139 > - auto array = BindingTraits<Array<Inspector::Protocol::Runtime::InternalPropertyDescriptor>>::runtimeCast(WTF::move(result)); > - *properties = array->length() > 0 ? array : nullptr; > + *properties = BindingTraits<Array<Inspector::Protocol::Runtime::InternalPropertyDescriptor>>::runtimeCast(WTF::move(result)); I should revert this part for: spector-protocol/runtime/getProperties.html I wanted to always include the array, but not including it if the array is empty will simplify the response. Comment on attachment 246675 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=246675&action=review > Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:200 > + result->putDirect(exec->vm(), Identifier(exec, "name"), jsString(exec, name)); jsNontrivialString and ASCIILiteral for name? Comment on attachment 246675 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=246675&action=review >> Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:200 >> + result->putDirect(exec->vm(), Identifier(exec, "name"), jsString(exec, name)); > > jsNontrivialString and ASCIILiteral for name? jsNontrivialString - I debated it, and said no in case we ever have a single character string. (it asserts length > 1, not >= 1). ASCIILiteral - I thought there was a jsString for char* but there isn't. I'll change the param to const String& and use ASCIILiteral above. (In reply to comment #12) > http://trac.webkit.org/changeset/180235 It broke the !ENABLE(PROMISES) build, see bug141746 for details. |