[Webkit-unassigned] [Bug 73351] New: ASSERT fails in updateState ACTION_DRAW case

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=73351

           Summary: ASSERT fails in updateState ACTION_DRAW case
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Android
        OS/Version: Android
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: CSS
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: armand.navabi at gmail.com
                CC: jamesr at chromium.org, nduca at chromium.org


The ASSERT in case ACTION_DRAW of CCSchedulerStateMachine::updateState(Action action) fails.

A potential fix is to change the assert to ASSERT(m_needsCommit || !m_visible).  The issue arises because updateState is being called right after m_needsCommit has been set to true and m_visible has been set to false.  This will set m_commitState = COMMIT_STATE_WAITING_FOR_FIRST_DRAW in the ACTION_COMMIT case.  Then on the draw action the assert is triggered and fails because it asserts only that m_needsCommit is true.

Other solutions may be that m_needsCommit should not be set to true, m_visible may be incorrectly being set to false or m_needsForcedRedraw should not be set.  The main question is what should the ASSERT statement be asserting in the ACTION_DRAW case of updateState.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.