Add an unresolved WebGLPolicy and an API to resolve it
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Feb 2014 23:11:52 +0000 (23:11 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 20 Feb 2014 23:11:52 +0000 (23:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=129109

Reviewed by Anders Carlsson.

Source/WebCore:

Add a third WebGLLoadPolicy which is "pending" allowing the page
to go ahead with creating the WebGLRenderingContext and resolve the policy
at a later time. Add a new API resolveWebGLLoadPolicy to do the resolution.

* html/HTMLCanvasElement.cpp:
(WebCore::HTMLCanvasElement::getContext): WebGLBlock -> WebGLBlockCreation
* loader/FrameLoaderClient.h:
(WebCore::FrameLoaderClient::webGLPolicyForURL): WebGLAllow -> WebGLAllowCreation.
(WebCore::FrameLoaderClient::resolveWebGLPolicyForURL): New method.
* loader/FrameLoaderTypes.h: Add WebGLPendingCreation.

Source/WebKit2:

Add a third WebGLLoadPolicy which is "pending" allowing the page
to go ahead with creating the WebGLRenderingContext and resolve the policy
at a later time. Add a new API resolveWebGLLoadPolicy to do the resolution.

* UIProcess/API/APILoaderClient.h:
(API::LoaderClient::webGLLoadPolicy): New return value.
(API::LoaderClient::resolveWebGLLoadPolicy): New method definition.
* UIProcess/API/C/WKAPICast.h:
(WebKit::toWebGLLoadPolicy): Change names of return types.
* UIProcess/API/C/WKPage.cpp:
(WKPageSetPageLoaderClient): New stubs.
* UIProcess/API/C/WKPageLoaderClient.h: New policy type kWKWebGLLoadPolicyPending.
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::resolveWebGLPolicyForURL): New method.
* UIProcess/WebPageProxy.h:
* UIProcess/WebPageProxy.messages.in: New message.
* WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::webGLPolicyForURL):
(WebKit::WebFrameLoaderClient::resolveWebGLPolicyForURL):
* WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::webGLPolicyForURL):
(WebKit::WebPage::resolveWebGLPolicyForURL):
* WebProcess/WebPage/WebPage.h:

Tools:

Add a new entry for resolveWebGLLoadPolicy.

* WebKitTestRunner/TestController.cpp:
(WTR::TestController::createWebViewWithOptions):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@164451 268f45cc-cd09-0410-ab3c-d52691b4dbfc

18 files changed:
Source/WebCore/ChangeLog
Source/WebCore/html/HTMLCanvasElement.cpp
Source/WebCore/loader/FrameLoaderClient.h
Source/WebCore/loader/FrameLoaderTypes.h
Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/APILoaderClient.h
Source/WebKit2/UIProcess/API/C/WKAPICast.h
Source/WebKit2/UIProcess/API/C/WKPage.cpp
Source/WebKit2/UIProcess/API/C/WKPageLoaderClient.h
Source/WebKit2/UIProcess/WebPageProxy.cpp
Source/WebKit2/UIProcess/WebPageProxy.h
Source/WebKit2/UIProcess/WebPageProxy.messages.in
Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp
Source/WebKit2/WebProcess/WebCoreSupport/WebFrameLoaderClient.h
Source/WebKit2/WebProcess/WebPage/WebPage.cpp
Source/WebKit2/WebProcess/WebPage/WebPage.h
Tools/ChangeLog
Tools/WebKitTestRunner/TestController.cpp

index 46afaf6..772f42e 100644 (file)
@@ -1,3 +1,21 @@
+2014-02-20  Dean Jackson  <dino@apple.com>
+
+        Add an unresolved WebGLPolicy and an API to resolve it
+        https://bugs.webkit.org/show_bug.cgi?id=129109
+
+        Reviewed by Anders Carlsson.
+
+        Add a third WebGLLoadPolicy which is "pending" allowing the page
+        to go ahead with creating the WebGLRenderingContext and resolve the policy
+        at a later time. Add a new API resolveWebGLLoadPolicy to do the resolution.
+
+        * html/HTMLCanvasElement.cpp:
+        (WebCore::HTMLCanvasElement::getContext): WebGLBlock -> WebGLBlockCreation
+        * loader/FrameLoaderClient.h:
+        (WebCore::FrameLoaderClient::webGLPolicyForURL): WebGLAllow -> WebGLAllowCreation.
+        (WebCore::FrameLoaderClient::resolveWebGLPolicyForURL): New method.
+        * loader/FrameLoaderTypes.h: Add WebGLPendingCreation.
+
 2014-02-20  Zalan Bujtas  <zalan@apple.com>
 
         Subpixel rendering: Enable compositing RenderLayer painting on device pixel position.
index 7fc11e6..5fb5106 100644 (file)
@@ -226,7 +226,7 @@ CanvasRenderingContext* HTMLCanvasElement::getContext(const String& type, Canvas
                 if (page && !topDocument.url().isLocalFile()) {
                     WebGLLoadPolicy policy = page->mainFrame().loader().client().webGLPolicyForURL(topDocument.url());
 
-                    if (policy == WebGLBlock)
+                    if (policy == WebGLBlockCreation)
                         return nullptr;
                 }
                 m_context = WebGLRenderingContext::create(this, static_cast<WebGLContextAttributes*>(attrs));
index 020a3a9..933a379 100644 (file)
@@ -325,7 +325,8 @@ namespace WebCore {
         // Informs the embedder that a WebGL canvas inside this frame received a lost context
         // notification with the given GL_ARB_robustness guilt/innocence code (see Extensions3D.h).
         virtual void didLoseWebGLContext(int) { }
-        virtual WebGLLoadPolicy webGLPolicyForURL(const String&) const { return WebGLAllow; }
+        virtual WebGLLoadPolicy webGLPolicyForURL(const String&) const { return WebGLAllowCreation; }
+        virtual WebGLLoadPolicy resolveWebGLPolicyForURL(const String&) const { return WebGLAllowCreation; }
 #endif
 
         virtual void forcePageTransitionIfNeeded() { }
index 6385f6d..39a6b14 100644 (file)
@@ -107,8 +107,9 @@ namespace WebCore {
     };
     
     enum WebGLLoadPolicy {
-        WebGLBlock = 0,
-        WebGLAllow
+        WebGLBlockCreation,
+        WebGLAllowCreation,
+        WebGLPendingCreation
     };
 
 }
index d24d01c..98ec935 100644 (file)
@@ -1,3 +1,35 @@
+2014-02-20  Dean Jackson  <dino@apple.com>
+
+        Add an unresolved WebGLPolicy and an API to resolve it
+        https://bugs.webkit.org/show_bug.cgi?id=129109
+
+        Reviewed by Anders Carlsson.
+
+        Add a third WebGLLoadPolicy which is "pending" allowing the page
+        to go ahead with creating the WebGLRenderingContext and resolve the policy
+        at a later time. Add a new API resolveWebGLLoadPolicy to do the resolution.
+
+        * UIProcess/API/APILoaderClient.h:
+        (API::LoaderClient::webGLLoadPolicy): New return value.
+        (API::LoaderClient::resolveWebGLLoadPolicy): New method definition.
+        * UIProcess/API/C/WKAPICast.h:
+        (WebKit::toWebGLLoadPolicy): Change names of return types.
+        * UIProcess/API/C/WKPage.cpp:
+        (WKPageSetPageLoaderClient): New stubs.
+        * UIProcess/API/C/WKPageLoaderClient.h: New policy type kWKWebGLLoadPolicyPending.
+        * UIProcess/WebPageProxy.cpp:
+        (WebKit::WebPageProxy::resolveWebGLPolicyForURL): New method.
+        * UIProcess/WebPageProxy.h:
+        * UIProcess/WebPageProxy.messages.in: New message.
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
+        (WebKit::WebFrameLoaderClient::webGLPolicyForURL):
+        (WebKit::WebFrameLoaderClient::resolveWebGLPolicyForURL):
+        * WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::webGLPolicyForURL):
+        (WebKit::WebPage::resolveWebGLPolicyForURL):
+        * WebProcess/WebPage/WebPage.h:
+
 2014-02-20  Anders Carlsson  <andersca@apple.com>
 
         Make it possible to reply asynchronously to JavaScript panels and alerts
index d4b40c0..0a72efb 100644 (file)
@@ -97,7 +97,8 @@ public:
 #endif // ENABLE(NETSCAPE_PLUGIN_API)
 
 #if ENABLE(WEBGL)
-    virtual WebCore::WebGLLoadPolicy webGLLoadPolicy(WebKit::WebPageProxy*, const WTF::String&) const { return WebCore::WebGLLoadPolicy::WebGLAllow; }
+    virtual WebCore::WebGLLoadPolicy webGLLoadPolicy(WebKit::WebPageProxy*, const WTF::String&) const { return WebCore::WebGLLoadPolicy::WebGLAllowCreation; }
+    virtual WebCore::WebGLLoadPolicy resolveWebGLLoadPolicy(WebKit::WebPageProxy*, const WTF::String&) const { return WebCore::WebGLLoadPolicy::WebGLAllowCreation; }
 #endif // ENABLE(WEBGL)
 };
 
index 71569bb..75314ed 100644 (file)
@@ -481,13 +481,15 @@ inline WebCore::WebGLLoadPolicy toWebGLLoadPolicy(WKWebGLLoadPolicy webGLLoadPol
 {
     switch (webGLLoadPolicy) {
     case kWKWebGLLoadPolicyLoadNormally:
-        return WebCore::WebGLAllow;
+        return WebCore::WebGLAllowCreation;
     case kWKWebGLLoadPolicyBlocked:
-        return WebCore::WebGLBlock;
+        return WebCore::WebGLBlockCreation;
+    case kWKWebGLLoadPolicyPending:
+        return WebCore::WebGLPendingCreation;
     }
     
     ASSERT_NOT_REACHED();
-    return WebCore::WebGLAllow;
+    return WebCore::WebGLAllowCreation;
 }
 
 inline ProxyingRefPtr<WebGrammarDetail> toAPI(const WebCore::GrammarDetail& grammarDetail)
index 47f0547..39ca974 100644 (file)
@@ -967,13 +967,24 @@ void WKPageSetPageLoaderClient(WKPageRef pageRef, const WKPageLoaderClientBase*
 #if ENABLE(WEBGL)
         virtual WebCore::WebGLLoadPolicy webGLLoadPolicy(WebPageProxy* page, const String& url) const override
         {
-            WebCore::WebGLLoadPolicy loadPolicy = WebGLAllow;
+            WebCore::WebGLLoadPolicy loadPolicy = WebGLAllowCreation;
 
             if (m_client.webGLLoadPolicy)
                 loadPolicy = toWebGLLoadPolicy(m_client.webGLLoadPolicy(toAPI(page), toAPI(url.impl()), m_client.base.clientInfo));
 
             return loadPolicy;
         }
+
+        virtual WebCore::WebGLLoadPolicy resolveWebGLLoadPolicy(WebPageProxy* page, const String& url) const override
+        {
+            WebCore::WebGLLoadPolicy loadPolicy = WebGLAllowCreation;
+
+            if (m_client.resolveWebGLLoadPolicy)
+                loadPolicy = toWebGLLoadPolicy(m_client.resolveWebGLLoadPolicy(toAPI(page), toAPI(url.impl()), m_client.base.clientInfo));
+
+            return loadPolicy;
+        }
+
 #endif // ENABLE(WEBGL)
     };
 
index 579d8c2..3cea23e 100644 (file)
@@ -45,6 +45,7 @@ typedef uint32_t WKPluginLoadPolicy;
 enum {
     kWKWebGLLoadPolicyBlocked = 0,
     kWKWebGLLoadPolicyLoadNormally,
+    kWKWebGLLoadPolicyPending
 };
 typedef uint32_t WKWebGLLoadPolicy;
 
@@ -330,6 +331,7 @@ typedef struct WKPageLoaderClientV4 {
     
     // Version 4
     WKPageWebGLLoadPolicyCallback                                       webGLLoadPolicy;
+    WKPageWebGLLoadPolicyCallback                                       resolveWebGLLoadPolicy;
 } WKPageLoaderClientV4;
 
 // FIXME: These should be deprecated.
index b9e8b7c..c9e4e98 100644 (file)
@@ -2751,6 +2751,11 @@ void WebPageProxy::webGLPolicyForURL(const String& url, uint32_t& loadPolicy)
 {
     loadPolicy = static_cast<uint32_t>(m_loaderClient->webGLLoadPolicy(this, url));
 }
+
+void WebPageProxy::resolveWebGLPolicyForURL(const String& url, uint32_t& loadPolicy)
+{
+    loadPolicy = static_cast<uint32_t>(m_loaderClient->resolveWebGLLoadPolicy(this, url));
+}
 #endif // ENABLE(WEBGL)
 
 void WebPageProxy::setToolbarsAreVisible(bool toolbarsAreVisible)
index 174f944..b850802 100644 (file)
@@ -967,6 +967,7 @@ private:
 #endif // ENABLE(NETSCAPE_PLUGIN_API)
 #if ENABLE(WEBGL)
     void webGLPolicyForURL(const String& url, uint32_t& loadPolicy);
+    void resolveWebGLPolicyForURL(const String& url, uint32_t& loadPolicy);
 #endif // ENABLE(WEBGL)
     void setToolbarsAreVisible(bool toolbarsAreVisible);
     void getToolbarsAreVisible(bool& toolbarsAreVisible);
index 22b4d13..7315a89 100644 (file)
@@ -35,6 +35,7 @@ messages -> WebPageProxy {
 #endif // ENABLE(NETSCAPE_PLUGIN_API)
 #if ENABLE(WEBGL)
     WebGLPolicyForURL(String url) -> (uint32_t loadPolicy)
+    ResolveWebGLPolicyForURL(String url) -> (uint32_t loadPolicy)
 #endif // ENABLE(WEBGL)
     DidChangeViewportProperties(WebCore::ViewportAttributes attributes)
     DidReceiveEvent(uint32_t type, bool handled)
index 5090b1f..ecc771e 100644 (file)
@@ -1348,7 +1348,15 @@ WebCore::WebGLLoadPolicy WebFrameLoaderClient::webGLPolicyForURL(const String& u
     if (WebPage* webPage = m_frame->page())
         return webPage->webGLPolicyForURL(m_frame, url);
 
-    return WebGLAllow;
+    return WebGLAllowCreation;
+}
+
+WebCore::WebGLLoadPolicy WebFrameLoaderClient::resolveWebGLPolicyForURL(const String& url) const
+{
+    if (WebPage* webPage = m_frame->page())
+        return webPage->resolveWebGLPolicyForURL(m_frame, url);
+
+    return WebGLAllowCreation;
 }
 #endif // ENABLE(WEBGL)
 
index d0e3f8f..ba1e496 100644 (file)
@@ -187,6 +187,7 @@ private:
     
 #if ENABLE(WEBGL)
     virtual WebCore::WebGLLoadPolicy webGLPolicyForURL(const String&) const override;
+    virtual WebCore::WebGLLoadPolicy resolveWebGLPolicyForURL(const String&) const override;
 #endif // ENABLE(WEBGL)
 
     virtual PassRefPtr<WebCore::Widget> createJavaAppletWidget(const WebCore::IntSize&, WebCore::HTMLAppletElement*, const WebCore::URL& baseURL, const Vector<String>& paramNames, const Vector<String>& paramValues) override;
index 0eea2bb..b253c07 100644 (file)
@@ -620,7 +620,17 @@ WebCore::WebGLLoadPolicy WebPage::webGLPolicyForURL(WebFrame* frame, const Strin
     if (sendSync(Messages::WebPageProxy::WebGLPolicyForURL(url), Messages::WebPageProxy::WebGLPolicyForURL::Reply(policyResult)))
         return static_cast<WebGLLoadPolicy>(policyResult);
 
-    return WebGLAllow;
+    return WebGLAllowCreation;
+}
+
+WebCore::WebGLLoadPolicy WebPage::resolveWebGLPolicyForURL(WebFrame* frame, const String& url)
+{
+    uint32_t policyResult = 0;
+
+    if (sendSync(Messages::WebPageProxy::ResolveWebGLPolicyForURL(url), Messages::WebPageProxy::ResolveWebGLPolicyForURL::Reply(policyResult)))
+        return static_cast<WebGLLoadPolicy>(policyResult);
+
+    return WebGLAllowCreation;
 }
 #endif // ENABLE(WEBGL)
 
index 513124c..aa4ac2c 100644 (file)
@@ -306,6 +306,7 @@ public:
 
 #if ENABLE(WEBGL)
     WebCore::WebGLLoadPolicy webGLPolicyForURL(WebFrame*, const String&);
+    WebCore::WebGLLoadPolicy resolveWebGLPolicyForURL(WebFrame*, const String&);
 #endif // ENABLE(WEBGL)
     
     EditorState editorState() const;
index 8a8e1cd..7871ad4 100644 (file)
@@ -1,3 +1,15 @@
+2014-02-20  Dean Jackson  <dino@apple.com>
+
+        Add an unresolved WebGLPolicy and an API to resolve it
+        https://bugs.webkit.org/show_bug.cgi?id=129109
+
+        Reviewed by Anders Carlsson.
+
+        Add a new entry for resolveWebGLLoadPolicy.
+
+        * WebKitTestRunner/TestController.cpp:
+        (WTR::TestController::createWebViewWithOptions):
+
 2014-02-19  Gustavo Noronha Silva  <gustavo.noronha@collabora.com>
 
         [GTK] Add new ARM build slave for GTK+
index c66f015..3560b8c 100644 (file)
@@ -476,6 +476,7 @@ void TestController::createWebViewWithOptions(WKDictionaryRef options)
         0, // pluginDidFail
         pluginLoadPolicy, // pluginLoadPolicy
         0, // webGLLoadPolicy
+        0, // resolveWebGLLoadPolicy
     };
     WKPageSetPageLoaderClient(m_mainWebView->page(), &pageLoaderClient.base);