Diff
Modified: branches/safari-534.59-branch/Source/WebKit2/ChangeLog (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/ChangeLog 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/ChangeLog 2013-03-03 07:31:07 UTC (rev 144560)
@@ -1,5 +1,36 @@
2013-03-02 Brady Eidson <[email protected]>
+ Merge r143790
+
+ 2013-02-22 Anders Carlsson <[email protected]>
+
+ Move pluginLoadPolicy to the page loader client
+ https://bugs.webkit.org/show_bug.cgi?id=110635
+ <rdar://problem/13265303>
+
+ Reviewed by Sam Weinig.
+
+ Move the pluginLoadPolicy callback to the page loader client where it belongs.
+
+ * UIProcess/API/C/WKPage.h:
+ * UIProcess/WebLoaderClient.cpp:
+ (WebKit::toWKPluginLoadPolicy):
+ (WebKit):
+ (WebKit::toPluginModuleLoadPolicy):
+ (WebKit::WebLoaderClient::pluginLoadPolicy):
+ * UIProcess/WebLoaderClient.h:
+ (WebLoaderClient):
+ * UIProcess/WebPageProxy.cpp:
+ (WebKit::WebPageProxy::getPluginPath):
+ * UIProcess/WebUIClient.cpp:
+ (WebKit):
+ * UIProcess/WebUIClient.h:
+ (WebUIClient):
+ * UIProcess/mac/WebInspectorProxyMac.mm:
+ (WebKit::WebInspectorProxy::platformCreateInspectorPage):
+
+2013-03-02 Brady Eidson <[email protected]>
+
Merge r142919
2013-02-14 Anders Carlsson <[email protected]>
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/API/C/WKPage.h (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/API/C/WKPage.h 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/API/C/WKPage.h 2013-03-03 07:31:07 UTC (rev 144560)
@@ -48,6 +48,13 @@
};
typedef uint32_t WKFocusDirection;
+enum {
+ kWKPluginLoadPolicyLoadNormally = 0,
+ kWKPluginLoadPolicyBlocked,
+ kWKPluginLoadPolicyInactive,
+};
+typedef uint32_t WKPluginLoadPolicy;
+
typedef void (*WKPageCallback)(WKPageRef page, const void* clientInfo);
// FrameLoad Client
@@ -70,6 +77,7 @@
typedef void (*WKPageDidChangeBackForwardListCallback)(WKPageRef page, WKBackForwardListItemRef addedItem, WKArrayRef removedItems, const void *clientInfo);
typedef bool (*WKPageShouldGoToBackForwardListItemCallback)(WKPageRef page, WKBackForwardListItemRef item, const void *clientInfo);
typedef void (*WKPagePluginDidFailCallback)(WKPageRef page, WKErrorCode errorCode, WKStringRef mimeType, WKStringRef pluginIdentifier, WKStringRef pluginVersion, const void *clientInfo);
+typedef WKPluginLoadPolicy (*WKPagePluginLoadPolicyCallback)(WKPageRef page, WKStringRef identifier, WKStringRef displayName, WKURLRef documentURL, WKPluginLoadPolicy currentPluginLoadPolicy, const void* clientInfo);
// Deprecated
typedef void (*WKPageDidFailToInitializePluginCallback_deprecatedForUseWithV0)(WKPageRef page, WKStringRef mimeType, const void* clientInfo);
@@ -107,6 +115,7 @@
WKPageShouldGoToBackForwardListItemCallback shouldGoToBackForwardListItem;
WKPageDidFailToInitializePluginCallback_deprecatedForUseWithV0 didFailToInitializePlugin_deprecatedForUseWithV0;
WKPagePluginDidFailCallback pluginDidFail;
+ WKPagePluginLoadPolicyCallback pluginLoadPolicy;
};
typedef struct WKPageLoaderClient WKPageLoaderClient;
@@ -163,13 +172,6 @@
};
typedef uint32_t WKPluginUnavailabilityReason;
-enum {
- kWKPluginLoadPolicyLoadNormally = 0,
- kWKPluginLoadPolicyBlocked,
- kWKPluginLoadPolicyInactive,
-};
-typedef uint32_t WKPluginLoadPolicy;
-
// UI Client
typedef WKPageRef (*WKPageCreateNewPageCallback)(WKPageRef page, WKDictionaryRef features, WKEventModifiers modifiers, WKEventMouseButton mouseButton, const void *clientInfo);
typedef void (*WKPageRunJavaScriptAlertCallback)(WKPageRef page, WKStringRef alertText, WKFrameRef frame, const void *clientInfo);
@@ -205,7 +207,6 @@
typedef void (*WKPageSaveDataToFileInDownloadsFolderCallback)(WKPageRef page, WKStringRef suggestedFilename, WKStringRef mimeType, WKURLRef originatingURL, WKDataRef data, const void* clientInfo);
typedef bool (*WKPageShouldInterruptJavaScriptCallback)(WKPageRef page, const void *clientInfo);
typedef void (*WKPageUnavailablePluginButtonClickedCallback)(WKPageRef page, WKPluginUnavailabilityReason pluginUnavailabilityReason, WKStringRef mimeType, WKStringRef url, WKStringRef pluginsPageURL, const void* clientInfo);
-typedef WKPluginLoadPolicy (*WKPagePluginLoadPolicyCallback)(WKPageRef page, WKStringRef identifier, WKStringRef displayName, WKURLRef documentURL, WKPluginLoadPolicy currentPluginLoadPolicy, const void* clientInfo);
// Deprecated
typedef void (*WKPageMissingPluginButtonClickedCallback_deprecatedForUseWithV0)(WKPageRef page, WKStringRef mimeType, WKStringRef url, WKStringRef pluginsPageURL, const void* clientInfo);
@@ -253,7 +254,6 @@
WKPageSaveDataToFileInDownloadsFolderCallback saveDataToFileInDownloadsFolder;
WKPageShouldInterruptJavaScriptCallback shouldInterruptJavaScript;
WKPageUnavailablePluginButtonClickedCallback unavailablePluginButtonClicked;
- WKPagePluginLoadPolicyCallback pluginLoadPolicy;
};
typedef struct WKPageUIClient WKPageUIClient;
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.cpp (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.cpp 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.cpp 2013-03-03 07:31:07 UTC (rev 144560)
@@ -250,4 +250,43 @@
m_client.pluginDidFail(toAPI(page), kWKErrorCodeInsecurePlugInVersion, toAPI(mimeType.impl()), toAPI(pluginIdentifier.impl()), toAPI(pluginVersion.impl()), m_client.clientInfo);
}
+static inline WKPluginLoadPolicy toWKPluginLoadPolicy(PluginModuleLoadPolicy pluginModuleLoadPolicy)
+{
+ switch (pluginModuleLoadPolicy) {
+ case PluginModuleLoadNormally:
+ return kWKPluginLoadPolicyLoadNormally;
+ case PluginModuleBlocked:
+ return kWKPluginLoadPolicyBlocked;
+ case PluginModuleInactive:
+ return kWKPluginLoadPolicyInactive;
+ }
+
+ ASSERT_NOT_REACHED();
+ return kWKPluginLoadPolicyBlocked;
+}
+
+static inline PluginModuleLoadPolicy toPluginModuleLoadPolicy(WKPluginLoadPolicy pluginLoadPolicy)
+{
+ switch (pluginLoadPolicy) {
+ case kWKPluginLoadPolicyLoadNormally:
+ return PluginModuleLoadNormally;
+ case kWKPluginLoadPolicyBlocked:
+ return PluginModuleBlocked;
+ case kWKPluginLoadPolicyInactive:
+ return PluginModuleInactive;
+ }
+
+ ASSERT_NOT_REACHED();
+ return PluginModuleBlocked;
+}
+
+PluginModuleLoadPolicy WebLoaderClient::pluginLoadPolicy(WebPageProxy* page, const String& identifier, const String& displayName, const String& documentURLString, PluginModuleLoadPolicy currentPluginLoadPolicy)
+{
+ if (!m_client.pluginLoadPolicy)
+ return currentPluginLoadPolicy;
+
+ return toPluginModuleLoadPolicy(m_client.pluginLoadPolicy(toAPI(page), toAPI(identifier.impl()), toAPI(displayName.impl()), toURLRef(documentURLString.impl()), toWKPluginLoadPolicy(currentPluginLoadPolicy), m_client.clientInfo));
+}
+
+
} // namespace WebKit
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.h (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.h 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebLoaderClient.h 2013-03-03 07:31:07 UTC (rev 144560)
@@ -27,6 +27,7 @@
#define WebLoaderClient_h
#include "APIClient.h"
+#include "PluginModuleInfo.h"
#include "SameDocumentNavigationType.h"
#include "WKPage.h"
#include <wtf/Forward.h>
@@ -79,8 +80,10 @@
void didChangeBackForwardList(WebPageProxy*, WebBackForwardListItem* addedItem, Vector<RefPtr<APIObject> >* removedItems);
bool shouldGoToBackForwardListItem(WebPageProxy*, WebBackForwardListItem*);
+ PluginModuleLoadPolicy pluginLoadPolicy(WebPageProxy*, const String& identifier, const String& displayName, const String& documentURLString, PluginModuleLoadPolicy currentPluginLoadPolicy);
void didFailToInitializePlugin(WebPageProxy*, const String& mimeType);
void didBlockInsecurePluginVersion(WebPageProxy*, const String& mimeType, const String& pluginIdentifier, const String& pluginVersion);
+
};
} // namespace WebKit
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebPageProxy.cpp 2013-03-03 07:31:07 UTC (rev 144560)
@@ -934,7 +934,7 @@
pluginLoadPolicy = PluginInfoStore::policyForPlugin(plugin);
PluginModuleLoadPolicy currentPluginLoadPolicy = static_cast<PluginModuleLoadPolicy>(pluginLoadPolicy);
- pluginLoadPolicy = m_uiClient.pluginLoadPolicy(this, plugin.bundleIdentifier, plugin.info.name, documentURLString, currentPluginLoadPolicy);
+ pluginLoadPolicy = m_loaderClient.pluginLoadPolicy(this, plugin.bundleIdentifier, plugin.info.name, documentURLString, currentPluginLoadPolicy);
if (pluginLoadPolicy != PluginModuleLoadNormally)
return;
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.cpp (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.cpp 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.cpp 2013-03-03 07:31:07 UTC (rev 144560)
@@ -399,42 +399,4 @@
return m_client.shouldInterruptJavaScript(toAPI(page), m_client.clientInfo);
}
-static inline WKPluginLoadPolicy toWKPluginLoadPolicy(PluginModuleLoadPolicy pluginModuleLoadPolicy)
-{
- switch (pluginModuleLoadPolicy) {
- case PluginModuleLoadNormally:
- return kWKPluginLoadPolicyLoadNormally;
- case PluginModuleBlocked:
- return kWKPluginLoadPolicyBlocked;
- case PluginModuleInactive:
- return kWKPluginLoadPolicyInactive;
- }
-
- ASSERT_NOT_REACHED();
- return kWKPluginLoadPolicyBlocked;
-}
-
-static inline PluginModuleLoadPolicy toPluginModuleLoadPolicy(WKPluginLoadPolicy pluginLoadPolicy)
-{
- switch (pluginLoadPolicy) {
- case kWKPluginLoadPolicyLoadNormally:
- return PluginModuleLoadNormally;
- case kWKPluginLoadPolicyBlocked:
- return PluginModuleBlocked;
- case kWKPluginLoadPolicyInactive:
- return PluginModuleInactive;
- }
-
- ASSERT_NOT_REACHED();
- return PluginModuleBlocked;
-}
-
-PluginModuleLoadPolicy WebUIClient::pluginLoadPolicy(WebPageProxy* page, const String& identifier, const String& displayName, const String& documentURLString, PluginModuleLoadPolicy currentPluginLoadPolicy)
-{
- if (!m_client.pluginLoadPolicy)
- return currentPluginLoadPolicy;
-
- return toPluginModuleLoadPolicy(m_client.pluginLoadPolicy(toAPI(page), toAPI(identifier.impl()), toAPI(displayName.impl()), toURLRef(documentURLString.impl()), toWKPluginLoadPolicy(currentPluginLoadPolicy), m_client.clientInfo));
-}
-
} // namespace WebKit
Modified: branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.h (144559 => 144560)
--- branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.h 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Source/WebKit2/UIProcess/WebUIClient.h 2013-03-03 07:31:07 UTC (rev 144560)
@@ -27,7 +27,6 @@
#define WebUIClient_h
#include "APIClient.h"
-#include "PluginModuleInfo.h"
#include "WKPage.h"
#include "WebEvent.h"
#include "WebOpenPanelParameters.h"
@@ -114,8 +113,6 @@
void saveDataToFileInDownloadsFolder(WebPageProxy*, const String& suggestedFilename, const String& mimeType, const String& originatingURLString, WebData*);
bool shouldInterruptJavaScript(WebPageProxy*);
-
- PluginModuleLoadPolicy pluginLoadPolicy(WebPageProxy*, const String& identifier, const String& displayName, const String& documentURLString, PluginModuleLoadPolicy currentPluginLoadPolicy);
};
} // namespace WebKit
Modified: branches/safari-534.59-branch/Tools/ChangeLog (144559 => 144560)
--- branches/safari-534.59-branch/Tools/ChangeLog 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Tools/ChangeLog 2013-03-03 07:31:07 UTC (rev 144560)
@@ -1,5 +1,22 @@
2013-03-02 Brady Eidson <[email protected]>
+ Merge r143790
+
+ 2013-02-22 Anders Carlsson <[email protected]>
+
+ Move pluginLoadPolicy to the page loader client
+ https://bugs.webkit.org/show_bug.cgi?id=110635
+
+ Reviewed by Sam Weinig.
+
+ * MiniBrowser/mac/WK2BrowserWindowController.m:
+ (-[WK2BrowserWindowController awakeFromNib]):
+ * WebKitTestRunner/TestController.cpp:
+ (WTR::TestController::createOtherPage):
+ (WTR::TestController::createWebViewWithOptions):
+
+2013-03-02 Brady Eidson <[email protected]>
+
Merge r141486
2013-01-31 Brian Weinstein <[email protected]>
Modified: branches/safari-534.59-branch/Tools/MiniBrowser/mac/BrowserWindowController.m (144559 => 144560)
--- branches/safari-534.59-branch/Tools/MiniBrowser/mac/BrowserWindowController.m 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Tools/MiniBrowser/mac/BrowserWindowController.m 2013-03-03 07:31:07 UTC (rev 144560)
@@ -599,8 +599,9 @@
processDidExit,
didChangeBackForwardList,
0, // shouldGoToBackForwardItem
- 0, // didFailToInitializePlugin
+ 0, // didFailToInitializePlugin
0, // pluginDidFail
+ 0, // pluginLoadPolicy
};
WKPageSetPageLoaderClient(_webView.pageRef, &loadClient);
@@ -657,7 +658,6 @@
0, // saveDataToFileInDownloadsFolder
0, // shouldInterruptJavaScript
0, // unavailablePluginButtonClicked
- 0, // shouldInstantiatePlugin
};
WKPageSetPageUIClient(_webView.pageRef, &uiClient);
}
Modified: branches/safari-534.59-branch/Tools/WebKitTestRunner/TestController.cpp (144559 => 144560)
--- branches/safari-534.59-branch/Tools/WebKitTestRunner/TestController.cpp 2013-03-03 07:13:29 UTC (rev 144559)
+++ branches/safari-534.59-branch/Tools/WebKitTestRunner/TestController.cpp 2013-03-03 07:31:07 UTC (rev 144560)
@@ -180,7 +180,6 @@
0, // saveDataToFileInDownloadsFolder
0, // shouldInterruptJavaScript
0, // unavailablePluginButtonClicked
- 0, // shouldInstantiatePlugin
};
WKPageSetPageUIClient(newPage, &otherPageUIClient);
@@ -325,7 +324,6 @@
0, // saveDataToFileInDownloadsFolder
0, // shouldInterruptJavaScript
0, // unavailablePluginButtonClicked
- 0, // shouldInstantiatePlugin
};
WKPageSetPageUIClient(m_mainWebView->page(), &pageUIClient);
@@ -358,6 +356,7 @@
0, // shouldGoToBackForwardListItem
0, // didFailToInitializePlugin
0, // pluginDidFail
+ 0, // pluginLoadPolicy
};
WKPageSetPageLoaderClient(m_mainWebView->page(), &pageLoaderClient);
}
