Bug 54480 - REGRESSION (WebKit2): A web page remains unresponsive for a long time after printing
Summary: REGRESSION (WebKit2): A web page remains unresponsive for a long time after p...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Printing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.6
: P2 Normal
Assignee: Alexey Proskuryakov
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2011-02-15 11:38 PST by Alexey Proskuryakov
Modified: 2011-02-15 12:37 PST (History)
3 users (show)

See Also:

Attachments
naive fix (1.57 KB, patch)
2011-02-15 11:43 PST, Alexey Proskuryakov 		darin: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2011-02-15 11:38:48 PST 
I can't select, scroll etc for several seconds after dismissing print dialog, even on a small page.

Steps to reproduce:
1. Open http://www.apple.com/hotnews/
2. Cmd+P, Cancel.
3. Try to select text or drag the scroller.

Results: the page remains unresponsive for several seconds

<rdar://problem/8975229>
Comment 1 Alexey Proskuryakov 2011-02-15 11:43:57 PST 
Created attachment 82492 [details]
naive fix

I don't understand how painting works between AppKit and WebKit2, and it's quite possible that the root cause lies elsewhere. This seems safe though.
Comment 2 WebKit Review Bot 2011-02-15 11:45:40 PST 
Attachment 82492 [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/ChangeLog:9:  Line contains tab character.  [whitespace/tab] [5]
Source/WebKit2/ChangeLog:10:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 2 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Darin Adler 2011-02-15 12:06:53 PST 
Comment on attachment 82492 [details]
naive fix

Ideally we’d want to trigger the display timer, not immediate display. So this is a workaround for a bug not a real fix.
Comment 4 Alexey Proskuryakov 2011-02-15 12:18:37 PST 
One thing that's good about displaying immediately is that we start relayout when going back to screen mode, so drawing the latest bits from before printing is better than drawing new ones after a short period of time.

What's surprising to me is that actions like selecting text on page or even scrolling don't make WebCore call invalidateContentsAndWindow().
Comment 5 Alexey Proskuryakov 2011-02-15 12:37:38 PST 
Committed <http://trac.webkit.org/changeset/78603>.