Bug 59776 – Sometimes WebKit2 pauses during launch due to waiting for the first render

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit2

[Stephanie Lewis]

Since the WebProcess is just going to send us a blank page until they have rendered the first time we shouldn't pause and wait for their update.  Instead we should continue to paint a blank window until the WebProcess has sent us the new bits.

[Stephanie Lewis]

Created attachment 91635 [details]
first patch

[WebKit Review Bot]

Attachment 91635 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit..." exit_code: 1

Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp:80:  Missing space before ( in if(  [whitespace/parens] [5]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Darin Adler]

Comment on attachment 91635 [details]
first patch

View in context: https://bugs.webkit.org/attachment.cgi?id=91635&action=review

>> Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp:80
>> +        if(!m_hasReceivedFirstUpdate)
> 
> Missing space before ( in if(  [whitespace/parens] [5]

I agree with the style bot.

> Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp:188
> +    else if (!m_hasReceivedFirstUpdate)
> +        m_hasReceivedFirstUpdate = true;

No need for the if statement here. You don’t have to check a boolean is false before setting it to true.

> Source/WebKit2/UIProcess/DrawingAreaProxyImpl.h:102
> +    // For the first draw, don't draw anything until the WebProcess has sent over the bits.

This comment says “for the first draw”, but the code affects more than just the first draw, right?

[Stephanie Lewis]

> Source/WebKit2/UIProcess/DrawingAreaProxyImpl.h:102
> +    // For the first draw, don't draw anything until the WebProcess has sent over the bits.

This comment says “for the first draw”, but the code affects more than just the first draw, right?

I should have said the first substantial draw or the first draw with content.

[Darin Adler]

http://trac.webkit.org/changeset/85359