Attachment 347713 Details for Bug 188764 – Patch

[patch] Patch

bug-188764-20180821153323.patch (text/plain), 19.66 KB, created by Jeff Miller on 2018-08-21 15:33:23 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jeff Miller

Created: 2018-08-21 15:33:23 PDT

Size: 19.66 KB

patch

obsolete

>Subversion Revision: 235116
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 47d23dabb8a34aa932a5dd6319bcffbba1a7838d..e1aec8a0bba56886471c7e754888eebfbbc860d5 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,52 @@
>+2018-08-21  Jeff Miller  <jeffm@apple.com>
>+
>+        WKNavigationDelegate needs to allow clients to specify a custom blocked plug-in message
>+        https://bugs.webkit.org/show_bug.cgi?id=188764
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add a new private WKNavigationDelegate method that takes a completion handler so clients can return
>+        a custom unavailability description in addition to the policy. Change WebPageProxy::findPlugin() to
>+        send a delayed reply and API::NavigationClient::decidePolicyForPluginLoad() to take a completion
>+        handler to support this.
>+
>+        * UIProcess/API/APINavigationClient.h:
>+        (API::NavigationClient::decidePolicyForPluginLoad):
>+        Changed to take a completion handler. The default implementation doesn't return a custom
>+        unavailability description.
>+
>+        * UIProcess/API/C/WKPage.cpp:
>+        (WKPageSetPageNavigationClient):
>+        Changed decidePolicyForPluginLoad() to take a completion handler.
>+
>+        * UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h:
>+        Add alternate version of -decidePolicyForPluginLoadWithCurrentPolicy: that allows clients to return
>+        the unavailibility description.
>+
>+        * UIProcess/Cocoa/NavigationState.h:
>+        Add flag for new delegate method.
>+        Changed decidePolicyForPluginLoad() to take a completion handler.
>+
>+        * UIProcess/Cocoa/NavigationState.mm:
>+        (WebKit::NavigationState::setNavigationDelegate):
>+        Calculate flag for new delegate method.
>+
>+        (WebKit::NavigationState::NavigationClient::decidePolicyForPluginLoad):
>+        Changed to take a completion handler.
>+        Prefer new delegate method, if available.
>+
>+        * UIProcess/WebPageProxy.cpp:
>+        (WebKit::WebPageProxy::findPlugin):
>+        Changed to send a delayed reply.
>+        Refactored some code into a lambda so it can be shared when using m_navigationClient->decidePolicyForPluginLoad().
>+        Removed unneeded UNUSED_PARAM() macros.
>+
>+        * UIProcess/WebPageProxy.h:
>+        Changed findPlugin() to send a delayed reply.
>+
>+        * UIProcess/WebPageProxy.messages.in:
>+        Ditto.
>+
> 2018-08-21  Commit Queue  <commit-queue@webkit.org>
> 
>         Unreviewed, rolling out r234981.
>diff --git a/Source/WebKit/UIProcess/API/APINavigationClient.h b/Source/WebKit/UIProcess/API/APINavigationClient.h
>index 45c19e6b0628e3b556ea3b1e06adaf24c24f257a..0fcee0de4be6891411ad6c8c5d013e0d83cd63a0 100644
>--- a/Source/WebKit/UIProcess/API/APINavigationClient.h
>+++ b/Source/WebKit/UIProcess/API/APINavigationClient.h
>@@ -117,9 +117,9 @@ public:
> #if ENABLE(NETSCAPE_PLUGIN_API)
>     virtual bool didFailToInitializePlugIn(WebKit::WebPageProxy&, API::Dictionary&) { return false; }
>     virtual bool didBlockInsecurePluginVersion(WebKit::WebPageProxy&, API::Dictionary&) { return false; }
>-    virtual WebKit::PluginModuleLoadPolicy decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy currentPluginLoadPolicy, Dictionary&, WTF::String&)
>+    virtual void decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy currentPluginLoadPolicy, Dictionary&, WTF::Function<void(WebKit::PluginModuleLoadPolicy, const WTF::String&)>&& completionHandler)
>     {
>-        return currentPluginLoadPolicy;
>+        completionHandler(currentPluginLoadPolicy, WTF::String());
>     }
> #endif
> 
>diff --git a/Source/WebKit/UIProcess/API/C/WKPage.cpp b/Source/WebKit/UIProcess/API/C/WKPage.cpp
>index 5a2b9ff48b90a3ee023d3b56a7a52c8e190d4dc2..e96e0327c2433b0ea1f82e12ce5fdd995b4a0cda 100644
>--- a/Source/WebKit/UIProcess/API/C/WKPage.cpp
>+++ b/Source/WebKit/UIProcess/API/C/WKPage.cpp
>@@ -2405,7 +2405,7 @@ void WKPageSetPageNavigationClient(WKPageRef pageRef, const WKPageNavigationClie
>             m_client.contentRuleListNotification(toAPI(&page), toURLRef(url.string().impl()), toAPI(API::Array::create(WTFMove(apiListIdentifiers)).ptr()), toAPI(API::Array::create(WTFMove(apiNotifications)).ptr()), m_client.base.clientInfo);
>         }
> #if ENABLE(NETSCAPE_PLUGIN_API)
>-        PluginModuleLoadPolicy decidePolicyForPluginLoad(WebPageProxy& page, PluginModuleLoadPolicy currentPluginLoadPolicy, API::Dictionary& pluginInformation, String& unavailabilityDescription) override
>+        void decidePolicyForPluginLoad(WebPageProxy& page, PluginModuleLoadPolicy currentPluginLoadPolicy, API::Dictionary& pluginInformation, WTF::Function<void(PluginModuleLoadPolicy, const String&)>&& completionHandler) override
>         {
>             WKStringRef unavailabilityDescriptionOut = 0;
>             PluginModuleLoadPolicy loadPolicy = currentPluginLoadPolicy;
>@@ -2413,12 +2413,13 @@ void WKPageSetPageNavigationClient(WKPageRef pageRef, const WKPageNavigationClie
>             if (m_client.decidePolicyForPluginLoad)
>                 loadPolicy = toPluginModuleLoadPolicy(m_client.decidePolicyForPluginLoad(toAPI(&page), toWKPluginLoadPolicy(currentPluginLoadPolicy), toAPI(&pluginInformation), &unavailabilityDescriptionOut, m_client.base.clientInfo));
>             
>+            String unavailabilityDescription;
>             if (unavailabilityDescriptionOut) {
>                 RefPtr<API::String> webUnavailabilityDescription = adoptRef(toImpl(unavailabilityDescriptionOut));
>                 unavailabilityDescription = webUnavailabilityDescription->string();
>             }
>             
>-            return loadPolicy;
>+            completionHandler(loadPolicy, unavailabilityDescription);
>         }
> #endif
>     };
>diff --git a/Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h b/Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h
>index fea2ae751af595b2b401ed7a280b7b94b8c1bf5c..edaf84061e240cf5bb2f8f889a634e99b4876e2b 100644
>--- a/Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h
>+++ b/Source/WebKit/UIProcess/API/Cocoa/WKNavigationDelegatePrivate.h
>@@ -107,6 +107,7 @@ static const WKNavigationResponsePolicy _WKNavigationResponsePolicyBecomeDownloa
> - (void)_webView:(WKWebView *)webView didFailToInitializePlugInWithInfo:(NSDictionary *)info WK_API_AVAILABLE(macosx(10.13.4));
> - (void)_webView:(WKWebView *)webView didBlockInsecurePluginVersionWithInfo:(NSDictionary *)info WK_API_AVAILABLE(macosx(WK_MAC_TBA));
> - (_WKPluginModuleLoadPolicy)_webView:(WKWebView *)webView decidePolicyForPluginLoadWithCurrentPolicy:(_WKPluginModuleLoadPolicy)policy pluginInfo:(NSDictionary *)info unavailabilityDescription:(NSString *)unavailabilityDescription WK_API_AVAILABLE(macosx(WK_MAC_TBA));
>+- (void)_webView:(WKWebView *)webView decidePolicyForPluginLoadWithCurrentPolicy:(_WKPluginModuleLoadPolicy)policy pluginInfo:(NSDictionary *)info completionHandler:(void (^)(_WKPluginModuleLoadPolicy policy, NSString * unavailabilityDescription))completionHandler WK_API_AVAILABLE(macosx(WK_MAC_TBA));
> - (void)_webView:(WKWebView *)webView backForwardListItemAdded:(WKBackForwardListItem *)itemAdded removed:(NSArray<WKBackForwardListItem *> *)itemsRemoved WK_API_AVAILABLE(macosx(10.13.4));
> #endif
> 
>diff --git a/Source/WebKit/UIProcess/Cocoa/NavigationState.h b/Source/WebKit/UIProcess/Cocoa/NavigationState.h
>index d16d2a9f9bc215bd7055749316f90a80ca0657de..33581fa7cb2350d7b11d5d5c957abfd7bd84ee95 100644
>--- a/Source/WebKit/UIProcess/Cocoa/NavigationState.h
>+++ b/Source/WebKit/UIProcess/Cocoa/NavigationState.h
>@@ -128,7 +128,7 @@ private:
>         bool willGoToBackForwardListItem(WebPageProxy&, WebBackForwardListItem&, bool inPageCache, API::Object*) final;
>         bool didFailToInitializePlugIn(WebPageProxy&, API::Dictionary&) final;
>         bool didBlockInsecurePluginVersion(WebPageProxy&, API::Dictionary&) final;
>-        WebKit::PluginModuleLoadPolicy decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy, API::Dictionary&, WTF::String&) final;
>+        void decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy, API::Dictionary&, WTF::Function<void(WebKit::PluginModuleLoadPolicy, const WTF::String&)>&& completionHandler) final;
>         bool didChangeBackForwardList(WebPageProxy&, WebBackForwardListItem*, const Vector<Ref<WebBackForwardListItem>>&) final;
> #endif
> 
>@@ -230,6 +230,7 @@ private:
>         bool webViewDidBlockInsecurePluginVersionWithInfo : 1;
>         bool webViewWillGoToBackForwardListItemInPageCache : 1;
>         bool webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoUnavailabilityDescription : 1;
>+        bool webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoCompletionHandler : 1;
> #endif
>     } m_navigationDelegateMethods;
> 
>diff --git a/Source/WebKit/UIProcess/Cocoa/NavigationState.mm b/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>index 8f60c3697b1a3dda4aff4fa3480107bb73b92323..30ab758458f919e619ce1c999bf2258011860c6c 100644
>--- a/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>+++ b/Source/WebKit/UIProcess/Cocoa/NavigationState.mm
>@@ -187,6 +187,7 @@ void NavigationState::setNavigationDelegate(id <WKNavigationDelegate> delegate)
>     m_navigationDelegateMethods.webViewDidBlockInsecurePluginVersionWithInfo = [delegate respondsToSelector:@selector(_webView:didBlockInsecurePluginVersionWithInfo:)];
>     m_navigationDelegateMethods.webViewBackForwardListItemAddedRemoved = [delegate respondsToSelector:@selector(_webView:backForwardListItemAdded:removed:)];
>     m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoUnavailabilityDescription = [delegate respondsToSelector:@selector(_webView:decidePolicyForPluginLoadWithCurrentPolicy:pluginInfo:unavailabilityDescription:)];
>+    m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoCompletionHandler = [delegate respondsToSelector:@selector(_webView:decidePolicyForPluginLoadWithCurrentPolicy:pluginInfo:completionHandler:)];
> #endif
> }
> 
>@@ -359,16 +360,23 @@ static _WKPluginModuleLoadPolicy wkPluginModuleLoadPolicy(WebKit::PluginModuleLo
>     return _WKPluginModuleLoadPolicyLoadNormally;
> }
> 
>-WebKit::PluginModuleLoadPolicy NavigationState::NavigationClient::decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy currentPluginLoadPolicy, API::Dictionary& pluginInformation, WTF::String& unavailabilityDescription)
>+void NavigationState::NavigationClient::decidePolicyForPluginLoad(WebKit::WebPageProxy&, WebKit::PluginModuleLoadPolicy currentPluginLoadPolicy, API::Dictionary& pluginInformation, WTF::Function<void(WebKit::PluginModuleLoadPolicy, const String&)>&& completionHandler)
> {
>-    if (!m_navigationState.m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoUnavailabilityDescription)
>-        return currentPluginLoadPolicy;
>+    if (!m_navigationState.m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoUnavailabilityDescription && !m_navigationState.m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoCompletionHandler)
>+        completionHandler(currentPluginLoadPolicy, String());
>     
>     auto navigationDelegate = m_navigationState.m_navigationDelegate.get();
>     if (!navigationDelegate)
>-        return currentPluginLoadPolicy;
>+        completionHandler(currentPluginLoadPolicy, String());
> 
>-    return pluginModuleLoadPolicy([(id <WKNavigationDelegatePrivate>)navigationDelegate _webView:m_navigationState.m_webView decidePolicyForPluginLoadWithCurrentPolicy:wkPluginModuleLoadPolicy(currentPluginLoadPolicy) pluginInfo:wrapper(pluginInformation) unavailabilityDescription:unavailabilityDescription]);
>+    if (m_navigationState.m_navigationDelegateMethods.webViewDecidePolicyForPluginLoadWithCurrentPolicyPluginInfoCompletionHandler) {
>+        [(id <WKNavigationDelegatePrivate>)navigationDelegate _webView:m_navigationState.m_webView decidePolicyForPluginLoadWithCurrentPolicy:wkPluginModuleLoadPolicy(currentPluginLoadPolicy) pluginInfo:wrapper(pluginInformation) completionHandler:^(_WKPluginModuleLoadPolicy policy, NSString *unavailabilityDescription) {
>+            completionHandler(pluginModuleLoadPolicy(policy), String(unavailabilityDescription));
>+        }];
>+        return;
>+    }
>+
>+    completionHandler(pluginModuleLoadPolicy([(id <WKNavigationDelegatePrivate>)navigationDelegate _webView:m_navigationState.m_webView decidePolicyForPluginLoadWithCurrentPolicy:wkPluginModuleLoadPolicy(currentPluginLoadPolicy) pluginInfo:wrapper(pluginInformation) unavailabilityDescription:nil]), String());
> }
> 
> inline WebCore::WebGLLoadPolicy toWebCoreWebGLLoadPolicy(_WKWebGLLoadPolicy policy)
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.cpp b/Source/WebKit/UIProcess/WebPageProxy.cpp
>index 8acd1b2574f210e69d2353476fad4aef422e5d1a..2050b2b3e8d7bedf368c28fd8033410f6cc09947 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.cpp
>+++ b/Source/WebKit/UIProcess/WebPageProxy.cpp
>@@ -2140,62 +2140,64 @@ WebPreferencesStore WebPageProxy::preferencesStore() const
> }
> 
> #if ENABLE(NETSCAPE_PLUGIN_API)
>-void WebPageProxy::findPlugin(const String& mimeType, uint32_t processType, const String& urlString, const String& frameURLString, const String& pageURLString, bool allowOnlyApplicationPlugins, uint64_t& pluginProcessToken, String& newMimeType, uint32_t& pluginLoadPolicy, String& unavailabilityDescription, bool& isUnsupported)
>+void WebPageProxy::findPlugin(const String& mimeType, uint32_t processType, const String& urlString, const String& frameURLString, const String& pageURLString, bool allowOnlyApplicationPlugins, Messages::WebPageProxy::FindPlugin::DelayedReply&& reply)
> {
>     PageClientProtector protector(m_pageClient);
> 
>     MESSAGE_CHECK_URL(urlString);
> 
>     URL pluginURL = URL { URL(), urlString };
>-    newMimeType = mimeType.convertToASCIILowercase();
>+    String newMimeType = mimeType.convertToASCIILowercase();
> 
>     PluginData::AllowedPluginTypes allowedPluginTypes = allowOnlyApplicationPlugins ? PluginData::OnlyApplicationPlugins : PluginData::AllPlugins;
> 
>     URL pageURL = URL { URL(), pageURLString };
>     if (!m_process->processPool().pluginInfoStore().isSupportedPlugin(mimeType, pluginURL, frameURLString, pageURL)) {
>-        isUnsupported = true;
>-        pluginProcessToken = 0;
>+        reply(0, newMimeType, PluginModuleLoadNormally, String(), true);
>         return;
>     }
> 
>-    isUnsupported = false;
>-    pluginLoadPolicy = PluginModuleLoadNormally;
>     PluginModuleInfo plugin = m_process->processPool().pluginInfoStore().findPlugin(newMimeType, pluginURL, allowedPluginTypes);
>     if (!plugin.path) {
>-        pluginProcessToken = 0;
>+        reply(0, newMimeType, PluginModuleLoadNormally, String(), false);
>         return;
>     }
> 
>-    pluginLoadPolicy = PluginInfoStore::defaultLoadPolicyForPlugin(plugin);
>+    uint32_t pluginLoadPolicy = PluginInfoStore::defaultLoadPolicyForPlugin(plugin);
>+
>+    auto findPluginCompletion = [processType, &reply, &newMimeType, &plugin] (uint32_t pluginLoadPolicy, const String& unavailabilityDescription) {
>+        PluginProcessSandboxPolicy pluginProcessSandboxPolicy = PluginProcessSandboxPolicyNormal;
>+        switch (pluginLoadPolicy) {
>+        case PluginModuleLoadNormally:
>+            pluginProcessSandboxPolicy = PluginProcessSandboxPolicyNormal;
>+            break;
>+        case PluginModuleLoadUnsandboxed:
>+            pluginProcessSandboxPolicy = PluginProcessSandboxPolicyUnsandboxed;
>+            break;
>+
>+        case PluginModuleBlockedForSecurity:
>+        case PluginModuleBlockedForCompatibility:
>+            reply(0, newMimeType, pluginLoadPolicy, unavailabilityDescription, false);
>+            return;
>+        }
>+
>+        reply(PluginProcessManager::singleton().pluginProcessToken(plugin, static_cast<PluginProcessType>(processType), pluginProcessSandboxPolicy), newMimeType, pluginLoadPolicy, unavailabilityDescription, false);
>+    };
> 
> #if PLATFORM(COCOA)
>     auto pluginInformation = createPluginInformationDictionary(plugin, frameURLString, String(), pageURLString, String(), String());
>-    if (m_navigationClient)
>-        pluginLoadPolicy = m_navigationClient->decidePolicyForPluginLoad(*this, static_cast<PluginModuleLoadPolicy>(pluginLoadPolicy), pluginInformation.get(), unavailabilityDescription);
>-    else
>+    if (m_navigationClient) {
>+        m_navigationClient->decidePolicyForPluginLoad(*this, static_cast<PluginModuleLoadPolicy>(pluginLoadPolicy), pluginInformation.get(), [findPluginCompletion = WTFMove(findPluginCompletion)] (PluginModuleLoadPolicy pluginLoadPolicy, const String& unavailabilityDescription) mutable {
>+            findPluginCompletion(pluginLoadPolicy, unavailabilityDescription);
>+        });
>+    } else {
>+        String unavailabilityDescription;
>         pluginLoadPolicy = m_loaderClient->pluginLoadPolicy(*this, static_cast<PluginModuleLoadPolicy>(pluginLoadPolicy), pluginInformation.get(), unavailabilityDescription);
>+        findPluginCompletion(pluginLoadPolicy, unavailabilityDescription);
>+    }
> #else
>-    UNUSED_PARAM(frameURLString);
>-    UNUSED_PARAM(pageURLString);
>-    UNUSED_PARAM(unavailabilityDescription);
>+    findPluginCompletion(pluginLoadPolicy, String());
> #endif
>-
>-    PluginProcessSandboxPolicy pluginProcessSandboxPolicy = PluginProcessSandboxPolicyNormal;
>-    switch (pluginLoadPolicy) {
>-    case PluginModuleLoadNormally:
>-        pluginProcessSandboxPolicy = PluginProcessSandboxPolicyNormal;
>-        break;
>-    case PluginModuleLoadUnsandboxed:
>-        pluginProcessSandboxPolicy = PluginProcessSandboxPolicyUnsandboxed;
>-        break;
>-
>-    case PluginModuleBlockedForSecurity:
>-    case PluginModuleBlockedForCompatibility:
>-        pluginProcessToken = 0;
>-        return;
>-    }
>-
>-    pluginProcessToken = PluginProcessManager::singleton().pluginProcessToken(plugin, static_cast<PluginProcessType>(processType), pluginProcessSandboxPolicy);
> }
> 
> #endif // ENABLE(NETSCAPE_PLUGIN_API)
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.h b/Source/WebKit/UIProcess/WebPageProxy.h
>index 9f625ddfb90c915723db35b17a0a6155b670b14c..724beafc8f9e98737724d1c1d21f9d5717d979fb 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.h
>+++ b/Source/WebKit/UIProcess/WebPageProxy.h
>@@ -1753,7 +1753,7 @@ private:
> #endif
> 
> #if ENABLE(NETSCAPE_PLUGIN_API)
>-    void findPlugin(const String& mimeType, uint32_t processType, const String& urlString, const String& frameURLString, const String& pageURLString, bool allowOnlyApplicationPlugins, uint64_t& pluginProcessToken, String& newMIMEType, uint32_t& pluginLoadPolicy, String& unavailabilityDescription, bool& isUnsupported);
>+    void findPlugin(const String& mimeType, uint32_t processType, const String& urlString, const String& frameURLString, const String& pageURLString, bool allowOnlyApplicationPlugins, Messages::WebPageProxy::FindPlugin::DelayedReply&&);
> #endif
> 
> #if USE(QUICK_LOOK)
>diff --git a/Source/WebKit/UIProcess/WebPageProxy.messages.in b/Source/WebKit/UIProcess/WebPageProxy.messages.in
>index 89cafaea5f586d8397eeb8e2c43bd4fc85abf930..a6613252ef56045d056f8ae3aaf94bf9053a2671 100644
>--- a/Source/WebKit/UIProcess/WebPageProxy.messages.in
>+++ b/Source/WebKit/UIProcess/WebPageProxy.messages.in
>@@ -435,7 +435,7 @@ messages -> WebPageProxy {
> #endif
> 
> #if ENABLE(NETSCAPE_PLUGIN_API)
>-    FindPlugin(String mimeType, uint32_t processType, String urlString, String frameURLString, String pageURLString, bool allowOnlyApplicationPlugins) -> (uint64_t pluginProcessToken, String newMIMEType, uint32_t pluginLoadPolicy, String unavailabilityDescription, bool isUnsupported)
>+    FindPlugin(String mimeType, uint32_t processType, String urlString, String frameURLString, String pageURLString, bool allowOnlyApplicationPlugins) -> (uint64_t pluginProcessToken, String newMIMEType, uint32_t pluginLoadPolicy, String unavailabilityDescription, bool isUnsupported) Delayed
> #endif
> 
>     DidUpdateActivityState()

Actions: View | Formatted Diff | Diff

Attachments on bug 188764: 347586 | 347651 | 347713 | 347726 | 347816