Changeset 80309 in webkit


Ignore:
Timestamp:
Mar 3, 2011 7:00:48 PM (13 years ago)
Author:
Adam Roben
Message:

Don't assume the web process has had a chance to paint before the view has to paint

Fixes <http://webkit.org/b/55739> REGRESSION (r80307): Lots of tests crashing in
BackingStore::paint

Reviewed by Anders Carlsson.

  • UIProcess/DrawingAreaProxyImpl.cpp:

(WebKit::DrawingAreaProxyImpl::paint): Bail if the web process hasn't had a chance to paint
yet.

Location:
trunk/Source/WebKit2
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebKit2/ChangeLog

    r80307 r80309  
     12011-03-03  Adam Roben  <aroben@apple.com>
     2
     3        Don't assume the web process has had a chance to paint before the view has to paint
     4
     5        Fixes <http://webkit.org/b/55739> REGRESSION (r80307): Lots of tests crashing in
     6        BackingStore::paint
     7
     8        Reviewed by Anders Carlsson.
     9
     10        * UIProcess/DrawingAreaProxyImpl.cpp:
     11        (WebKit::DrawingAreaProxyImpl::paint): Bail if the web process hasn't had a chance to paint
     12        yet.
     13
    1142011-03-03  Adam Roben  <aroben@apple.com>
    215
  • trunk/Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp

    r80307 r80309  
    9090    } else {
    9191        ASSERT(!m_isWaitingForDidUpdateBackingStoreState);
    92         ASSERT(m_backingStore);
     92        if (!m_backingStore) {
     93            // The view has asked us to paint before the web process has painted anything. There's
     94            // nothing we can do.
     95            return;
     96        }
    9397    }
    9498
Note: See TracChangeset for help on using the changeset viewer.