From https://bugs.webkit.org/show_bug.cgi?id=137839#c4 View in context: https://bugs.webkit.org/attachment.cgi?id=240045&action=review > Source/WebCore/bindings/js/JSHTMLDocumentCustom.cpp:65 > + if (propertyName == Identifier(exec, "open")) { I believe this is less efficient than: if (propertyName == "open") { > Source/WebCore/bindings/js/JSHTMLDocumentCustom.cpp:76 > + const HashTableValue* entry = JSHTMLDocument::info()->staticPropHashTable->entry(propertyName); > + if (entry) { Would be nice to put this definition inside the if.
Created attachment 240147 [details] Patch
Comment on attachment 240147 [details] Patch r=me
Committed r174913: <http://trac.webkit.org/changeset/174913>
(In reply to comment #3) > Committed r174913: <http://trac.webkit.org/changeset/174913> Changes didn't land with this revision due to webkit-patch issue.
Committed r174918: <http://trac.webkit.org/changeset/174918>