Title: [211887] trunk/Source/WebKit2
Revision
211887
Author
[email protected]
Date
2017-02-08 10:55:30 -0800 (Wed, 08 Feb 2017)

Log Message

GUM is crashing in Debug mode when access is granted/denied for a page that is gone
https://bugs.webkit.org/show_bug.cgi?id=168003

Patch by Youenn Fablet <[email protected]> on 2017-02-08
Reviewed by Eric Carlson.

Removing the assertion since manager can be null and case is coevered by the if statement behind it.

* UIProcess/UserMediaPermissionRequestProxy.cpp:
(WebKit::UserMediaPermissionRequestProxy::deny):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (211886 => 211887)


--- trunk/Source/WebKit2/ChangeLog	2017-02-08 18:54:55 UTC (rev 211886)
+++ trunk/Source/WebKit2/ChangeLog	2017-02-08 18:55:30 UTC (rev 211887)
@@ -1,3 +1,15 @@
+2017-02-08  Youenn Fablet  <[email protected]>
+
+        GUM is crashing in Debug mode when access is granted/denied for a page that is gone
+        https://bugs.webkit.org/show_bug.cgi?id=168003
+
+        Reviewed by Eric Carlson.
+
+        Removing the assertion since manager can be null and case is coevered by the if statement behind it.
+
+        * UIProcess/UserMediaPermissionRequestProxy.cpp:
+        (WebKit::UserMediaPermissionRequestProxy::deny):
+
 2017-02-08  Ryan Haddad  <[email protected]>
 
         Unreviewed, rolling out r211869.

Modified: trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestProxy.cpp (211886 => 211887)


--- trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestProxy.cpp	2017-02-08 18:54:55 UTC (rev 211886)
+++ trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestProxy.cpp	2017-02-08 18:55:30 UTC (rev 211887)
@@ -53,7 +53,6 @@
 
 void UserMediaPermissionRequestProxy::deny(UserMediaAccessDenialReason reason)
 {
-    ASSERT(m_manager);
     if (!m_manager)
         return;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to