2010-08-02 Rajiv Makhijani <rajivmakhijani@chromium.org>
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2010 02:25:16 +0000 (02:25 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Aug 2010 02:25:16 +0000 (02:25 +0000)
commit6a7182d9b26816f2a6e3aa36b68d90821a730e6b
tree41dffd3999a40eff3ba188499a0e6cec42280bc3
parentab8426af2aee9dc8b55dac0650e007ce1ff3c244
2010-08-02  Rajiv Makhijani  <rajivmakhijani@chromium.org>

        Reviewed by Adam Barth.

        Committed by Dirk Pranke.

        Added missing "if (window.layoutTestController)" check.
        Removed expectation for test case that should now pass with this bugfix.

        https://bugs.webkit.org/show_bug.cgi?id=38705

        * http/tests/security/sandbox-inherit-to-initial-document-2.html:
        * platform/chromium/test_expectations.txt:
2010-08-02  Rajiv Makhijani  <rajivmakhijani@chromium.org>

        Reviewed by Adam Barth.

        Committed by Dirk Pranke.

        Bug 38705 - [v8] chromium fails http/tests/sandbox-inherit-to-initial-document-2
        https://bugs.webkit.org/show_bug.cgi?id=38705

        WebCore::V8Proxy::retrieve(Frame* frame) calls WebCore::ScriptController::canExecuteScripts
        and returns 0 if canExecuteScripts is false. It should return the proxy regardless
        of whether the frame can execute scripts.

        * bindings/v8/V8Proxy.cpp:
        (WebCore::V8Proxy::retrieve): Removed canExecuteScripts check.

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@64525 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/http/tests/security/sandbox-inherit-to-initial-document-2.html
LayoutTests/platform/chromium/test_expectations.txt
WebCore/ChangeLog
WebCore/bindings/v8/V8Proxy.cpp