CTTE: activeDOMWindow() and firstDOMWindow() should return references.
Created attachment 213883 [details] Patch
Comment on attachment 213883 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=213883&action=review > Source/WebCore/page/DOMWindow.cpp:1832 > +bool DOMWindow::isInsecureScriptAccess(DOMWindow& activeWindow, const String& urlString) Can this be const?
Comment on attachment 213883 [details] Patch Attachment 213883 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/3879029
Comment on attachment 213883 [details] Patch Attachment 213883 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/3867029
Comment on attachment 213883 [details] Patch Attachment 213883 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/3872027
Comment on attachment 213883 [details] Patch Attachment 213883 [details] did not pass win-ews (win): Output: http://webkit-queues.appspot.com/results/3869029
Comment on attachment 213883 [details] Patch Attachment 213883 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/3870036
Created attachment 213889 [details] Patch
Comment on attachment 213889 [details] Patch Clearing flags on attachment: 213889 Committed r157228: <http://trac.webkit.org/changeset/157228>
All reviewed patches have been landed. Closing bug.
(In reply to comment #9) > (From update of attachment 213889 [details]) > Clearing flags on attachment: 213889 > > Committed r157228: <http://trac.webkit.org/changeset/157228> FYI: This patch broke the Apple Windows build, as the EWS noticed it: 1>WebKit.exp : error LNK2001: unresolved external symbol "public: class WTF::PassRefPtr<class WebCore::DOMWindow> __thiscall WebCore::DOMWindow::open(class WTF::String const &,class WTF::AtomicString const &,class WTF::String const &,class WebCore::DOMWindow *,class WebCore::DOMWindow *)" (?open@DOMWindow@WebCore@@QAE?AV?$PassRefPtr@VDOMWindow@WebCore@@@WTF@@ABVString@4@ABVAtomicString@4@0PAV12@2@Z) 1>C:\cygwin\home\buildbot\slave\win-debug\build\WebKitBuild\Debug\bin32\WebKit.dll : fatal error LNK1120: 1 unresolved externals