Bug 164098

Summary: Web Inspector: Replace sublists inside DOM-related model objects with WI.Collection
Product: WebKit Reporter: Devin Rousso <hi>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, buildbot, commit-queue, joepeck, mattbaker, nvasilyev, rniwa, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 145906    
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews105 for mac-yosemite-wk2
none
Archive of layout-test-results from ews112 for mac-yosemite
none
Archive of layout-test-results from ews101 for mac-yosemite
none
Patch
none
Archive of layout-test-results from ews103 for mac-yosemite
none
Archive of layout-test-results from ews105 for mac-yosemite-wk2
none
Archive of layout-test-results from ews114 for mac-yosemite
none
Patch none

Description Devin Rousso 2016-10-27 17:21:37 PDT
Instead of using an array/dictionary for holding related model objects (such as how a WI.DOMTree has a list of WI.ContentFlow), we can make use of the new WI.Collection model object for more extensibility and control.
Comment 1 Devin Rousso 2016-10-27 17:33:52 PDT
Created attachment 293088 [details]
Patch
Comment 2 Build Bot 2016-10-27 18:36:29 PDT
Comment on attachment 293088 [details]
Patch

Attachment 293088 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/2389439

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 3 Build Bot 2016-10-27 18:36:32 PDT
Created attachment 293095 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 4 Build Bot 2016-10-27 18:46:25 PDT
Comment on attachment 293088 [details]
Patch

Attachment 293088 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/2389440

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 5 Build Bot 2016-10-27 18:46:28 PDT
Created attachment 293096 [details]
Archive of layout-test-results from ews112 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 6 Build Bot 2016-10-27 18:49:06 PDT
Comment on attachment 293088 [details]
Patch

Attachment 293088 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/2389493

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 7 Build Bot 2016-10-27 18:49:09 PDT
Created attachment 293098 [details]
Archive of layout-test-results from ews101 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 8 Devin Rousso 2016-10-27 19:25:14 PDT
Created attachment 293100 [details]
Patch
Comment 9 Build Bot 2016-10-27 20:24:21 PDT
Comment on attachment 293100 [details]
Patch

Attachment 293100 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/2389915

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 10 Build Bot 2016-10-27 20:24:24 PDT
Created attachment 293104 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 11 Build Bot 2016-10-27 20:27:00 PDT
Comment on attachment 293100 [details]
Patch

Attachment 293100 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/2389917

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 12 Build Bot 2016-10-27 20:27:03 PDT
Created attachment 293105 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 13 Build Bot 2016-10-27 20:41:57 PDT
Comment on attachment 293100 [details]
Patch

Attachment 293100 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/2389954

New failing tests:
inspector/dom/highlightNode.html
inspector/dom/highlightSelector.html
inspector/dom/highlightFrame.html
http/tests/inspector/console/cross-domain-inspected-node-access.html
Comment 14 Build Bot 2016-10-27 20:42:00 PDT
Created attachment 293107 [details]
Archive of layout-test-results from ews114 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 15 Devin Rousso 2016-10-27 21:20:17 PDT
Created attachment 293110 [details]
Patch
Comment 16 Timothy Hatcher 2016-11-01 14:27:13 PDT
Comment on attachment 293110 [details]
Patch

Where is Collection.js? I don't see that in this patch.
Comment 17 Timothy Hatcher 2016-11-01 14:28:41 PDT
Nevermind, my checkout was old. I see Collection.js now.
Comment 18 WebKit Commit Bot 2016-11-01 14:51:28 PDT
Comment on attachment 293110 [details]
Patch

Clearing flags on attachment: 293110

Committed r208246: <http://trac.webkit.org/changeset/208246>
Comment 19 WebKit Commit Bot 2016-11-01 14:51:33 PDT
All reviewed patches have been landed.  Closing bug.
Comment 20 Joseph Pecoraro 2016-11-01 14:53:14 PDT
Comment on attachment 293110 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=293110&action=review

> Source/WebInspectorUI/UserInterface/Models/Collection.js:115
> +    ContentFlow: (object) => object instanceof WebInspector.ContentFlow,

I don't think this deserves a global. Just inline it in DOMTree.

> LayoutTests/inspector/dom/highlightNode-expected.txt:4
> +PASS: Page should have a subframe.

It is weird to have a PASS outside of a running test case.
Comment 21 Radar WebKit Bug Importer 2016-11-01 16:21:30 PDT
<rdar://problem/29054784>