Summary: | Web Inspector: remove RemoteObject.prototype.getPropertyDescriptorsAsObject | ||
---|---|---|---|
Product: | WebKit | Reporter: | Devin Rousso <hi> |
Component: | Web Inspector | Assignee: | Devin Rousso <hi> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | commit-queue, ews-watchlist, hi, inspector-bugzilla-changes, mattbaker, rniwa, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | All | ||
OS: | All | ||
Attachments: |
Description
Devin Rousso
2019-05-30 15:55:40 PDT
Created attachment 371022 [details]
Patch
Comment on attachment 371022 [details] Patch Attachment 371022 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/12335099 New failing tests: inspector/audit/data-domNodes.html inspector/audit/data-domAttributes.html inspector/audit/data-errors.html Created attachment 371025 [details]
Archive of layout-test-results from ews102 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102 Port: mac-highsierra Platform: Mac OS X 10.13.6
Comment on attachment 371022 [details] Patch Attachment 371022 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/12335267 New failing tests: inspector/audit/data-domNodes.html inspector/audit/data-domAttributes.html inspector/audit/data-errors.html Created attachment 371026 [details]
Archive of layout-test-results from ews105 for mac-highsierra-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105 Port: mac-highsierra-wk2 Platform: Mac OS X 10.13.6
Comment on attachment 371022 [details] Patch Attachment 371022 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/12335345 New failing tests: inspector/audit/data-domNodes.html inspector/audit/data-domAttributes.html inspector/audit/data-errors.html Created attachment 371030 [details]
Archive of layout-test-results from ews112 for mac-highsierra
The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112 Port: mac-highsierra Platform: Mac OS X 10.13.6
Created attachment 371033 [details]
Patch
Integer vs String comparison -.-
Comment on attachment 371033 [details]
Patch
r=me, nice fix.
Since ErrorObjectView.prototype.update is only called internally, and isn't an override, let's make it private or even inline it inside ErrorObjectView.prototype.expand, which is the only call site.
(In reply to Matt Baker from comment #9) > Since ErrorObjectView.prototype.update is only called internally, and isn't an override, let's make it private or even inline it inside ErrorObjectView.prototype.expand, which is the only call site. I'd rather not make a change like this as it's frankly unnecessary for this patch, and could be more prone to issues than what I'd tested. Furthermore, I'm actually a fan of the idea of modularizing functionality, even if it's only used once, as it makes code easier to read and more future-capable. Comment on attachment 371033 [details] Patch Clearing flags on attachment: 371033 Committed r246046: <https://trac.webkit.org/changeset/246046> All reviewed patches have been landed. Closing bug. |