Changeset 259750 in webkit

Timestamp:

Apr 8, 2020, 1:05:06 PM (6 years ago)

Author:

ddkilzer@apple.com

Message:

Follow-up: [iOS] WebPageProxy::didCommitLoadForFrame should not crash with null or empty mimeType parameter
<​https://webkit.org/b/209994>
<rdar://problem/60068700>

• UIProcess/mac/PageClientImplMac.mm:

(WebKit::PageClientImpl::didCommitLoadForMainFrame):

• Remove UNUSED_PARAM() in lieu of deleting parameter names.

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/mac/PageClientImplMac.mm (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2020-04-08  David Kilzer  <ddkilzer@apple.com>
+
+        Follow-up: [iOS] WebPageProxy::didCommitLoadForFrame should not crash with null or empty `mimeType` parameter
+        <https://webkit.org/b/209994>
+        <rdar://problem/60068700>
+
+        * UIProcess/mac/PageClientImplMac.mm:
+        (WebKit::PageClientImpl::didCommitLoadForMainFrame):
+        - Remove UNUSED_PARAM() in lieu of deleting parameter names.
+
 2020-04-08  Michael Catanzaro  <mcatanzaro@gnome.org>
 

trunk/Source/WebKit/UIProcess/mac/PageClientImplMac.mm

@@ -278 +278 @@
 }
 
-void PageClientImpl::didCommitLoadForMainFrame(const String& mimeType, bool useCustomContentProvider)
-{
-    UNUSED_PARAM(mimeType);
-    UNUSED_PARAM(useCustomContentProvider);
+void PageClientImpl::didCommitLoadForMainFrame(const String&, bool)
+{
     m_impl->updateSupportsArbitraryLayoutModes();
     m_impl->dismissContentRelativeChildWindowsWithAnimation(true);