[Webkit-unassigned] [Bug 195391] New: Remove an unneeded assert that was added with r242113

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 6 17:38:32 PST 2019


            Bug ID: 195391
           Summary: Remove an unneeded assert that was added with r242113
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: New Bugs
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ryanhaddad at apple.com

This assertion failure is seen with multiple layout tests after it was added in  https://trac.webkit.org/changeset/242113/webkit

ASSERTION FAILED: m_mediaBufferingIsSuspended
./page/Page.cpp(1784) : void WebCore::Page::resumeAllMediaBuffering()
1   0x493aeaee9 WTFCrash
2   0x497bb9c5b WTFCrashWithInfo(int, char const*, char const*, int)
3   0x49a78750b WebCore::Page::resumeAllMediaBuffering()
4   0x102bea184 WebKit::WebPage::resumeAllMediaBuffering()
5   0x1028a83a6 WebKit::WebProcess::resumeAllMediaBuffering()
6   0x1028a8e2a WebKit::WebProcess::processDidResume()


It doesn't seem necessary since m_mediaBufferingIsSuspended is protected by an 'if' statement right below the ASSERT:

void Page::resumeAllMediaBuffering()
    if (!m_mediaBufferingIsSuspended)

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20190307/f6211f92/attachment.html>

More information about the webkit-unassigned mailing list