Title: [158799] trunk/Source/WebKit2
Revision
158799
Author
m...@apple.com
Date
2013-11-06 15:53:02 -0800 (Wed, 06 Nov 2013)

Log Message

Fixed multiple API test failures after r158764.

Reviewed by Sam Weinig.

* UIProcess/API/mac/WKBrowsingContextController.mm:
(setUpPagePolicyClient): If the delegate isn’t set or doesn’t respond to a selector, respond
with WKFramePolicyListenerUse().

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (158798 => 158799)


--- trunk/Source/WebKit2/ChangeLog	2013-11-06 23:08:56 UTC (rev 158798)
+++ trunk/Source/WebKit2/ChangeLog	2013-11-06 23:53:02 UTC (rev 158799)
@@ -1,3 +1,13 @@
+2013-11-06  Dan Bernstein  <m...@apple.com>
+
+        Fixed multiple API test failures after r158764.
+
+        Reviewed by Sam Weinig.
+
+        * UIProcess/API/mac/WKBrowsingContextController.mm:
+        (setUpPagePolicyClient): If the delegate isn’t set or doesn’t respond to a selector, respond
+        with WKFramePolicyListenerUse().
+
 2013-11-06  Anders Carlsson  <ander...@apple.com>
 
         Move allowed classes handling to decodeObject

Modified: trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm (158798 => 158799)


--- trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm	2013-11-06 23:08:56 UTC (rev 158798)
+++ trunk/Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm	2013-11-06 23:53:02 UTC (rev 158799)
@@ -630,7 +630,8 @@
             };
             
             [browsingContext.policyDelegate browsingContextController:browsingContext decidePolicyForNavigationAction:actionDictionary decisionHandler:makePolicyDecisionBlock(listener)];
-        }
+        } else
+            WKFramePolicyListenerUse(listener);
     };
 
     policyClient.decidePolicyForNewWindowAction = [](WKPageRef page, WKFrameRef frame, WKFrameNavigationType navigationType, WKEventModifiers modifiers, WKEventMouseButton mouseButton, WKURLRequestRef request, WKStringRef frameName, WKFramePolicyListenerRef listener, WKTypeRef userData, const void* clientInfo)
@@ -647,8 +648,8 @@
             };
             
             [browsingContext.policyDelegate browsingContextController:browsingContext decidePolicyForNewWindowAction:actionDictionary decisionHandler:makePolicyDecisionBlock(listener)];
-        }
-
+        } else
+            WKFramePolicyListenerUse(listener);
     };
 
     policyClient.decidePolicyForResponse = [](WKPageRef page, WKFrameRef frame, WKURLResponseRef response, WKURLRequestRef request, WKFramePolicyListenerRef listener, WKTypeRef userData, const void* clientInfo)
@@ -662,7 +663,8 @@
             };
 
             [browsingContext.policyDelegate browsingContextController:browsingContext decidePolicyForResponseAction:actionDictionary decisionHandler:makePolicyDecisionBlock(listener)];
-        }
+        } else
+            WKFramePolicyListenerUse(listener);
     };
 
     WKPageSetPagePolicyClient(pageRef, &policyClient);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to