It's a mistake to use DocumentOrderedMap in Document. Rename the class to make this clear.
Created attachment 339517 [details] Cleanup
Created attachment 339595 [details] Patch for landing
Comment on attachment 339595 [details] Patch for landing Wait for EWS.
Attachment 339595 [details] did not pass style-queue: ERROR: Source/WebCore/dom/TreeScopeOrderedMap.cpp:53: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/dom/TreeScopeOrderedMap.cpp:63: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/dom/TreeScopeOrderedMap.cpp:84: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/dom/TreeScopeOrderedMap.cpp:111: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] ERROR: Source/WebCore/dom/TreeScopeOrderedMap.cpp:121: Please replace ASSERT_WITH_SECURITY_IMPLICATION() with RELEASE_ASSERT_WITH_SECURITY_IMPLICATION(). [security/assertion] [5] Total errors found: 5 in 9 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r231390: <https://trac.webkit.org/changeset/231390>
<rdar://problem/39994362>