DocumentOrderedMap::add should assert that the tree scope of the element matches that of the passed value instead of simply asserting that the element is in a tree scope.
Created attachment 324637 [details] Tighens the assertion
Created attachment 324638 [details] Tighens the assertion
<rdar://problem/35009035>
Attachment 324638 [details] did not pass style-queue: ERROR: Source/WebCore/dom/DocumentOrderedMap.cpp:110: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 324638 [details] Tighens the assertion Attachment 324638 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4966599 New failing tests: http/tests/appcache/decide-navigation-policy-after-delay.html
Created attachment 324643 [details] Archive of layout-test-results from ews106 for mac-elcapitan-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
I don't think http/tests/appcache/decide-navigation-policy-after-delay.html failure is related to this change since it only modifies the release assertions.
Comment on attachment 324638 [details] Tighens the assertion Oops, I meant to set cq+ on another patch.
Comment on attachment 324638 [details] Tighens the assertion Clearing flags on attachment: 324638 Committed r223886: <https://trac.webkit.org/changeset/223886>
All reviewed patches have been landed. Closing bug.