Bug 59276

Summary: [Gtk] No need to erase background before loading a new page
Product: WebKit Reporter: Joone Hur <joone.hur>
Component: WebKitGTKAssignee: Joone Hur <joone.hur>
Status: RESOLVED FIXED    
Severity: Normal CC: gustavo, mrobinson, xan.lopez
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Bug Depends on: 59821    
Bug Blocks:    
Attachments:
Description Flags
Propsed Patch mrobinson: review-

Joone Hur
Reported 2011-04-23 08:25:04 PDT
WebKitGtk+ erases background before loading a new page. So, if it takes some time to load the page, users have to see the empty page for a second. This makes the user think that the page loading is very slow. However, in the case of other WebKit based browser, they leave the current page before preparing the layout of a new page, and then just replace the current page with the new one.
Attachments
Propsed Patch (2.11 KB, patch)
2011-04-25 15:45 PDT, Joone Hur 		mrobinson: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Joone Hur
Comment 1 2011-04-25 15:45:34 PDT
Created attachment 90968 [details] Propsed Patch
Xan Lopez
Comment 2 2011-04-26 15:45:46 PDT
Comment on attachment 90968 [details] Propsed Patch Isn't finishedLoad extremely late for us to attach the adjusments?
Xan Lopez
Comment 3 2011-04-26 15:52:18 PDT
OK, this is the load finished for the DOM Document, not for the whole thing, I think. I believe Martin is working to remove the need to attach adjustments entirely in any case, so perhaps we want to wait for that.
Martin Robinson
Comment 4 2011-04-26 15:57:04 PDT
Comment on attachment 90968 [details] Propsed Patch Going to r- this one now, as hopefully in a few days we won't need to attach adjustments at all. If that approach fails though, please ping me later and we can re-address this one.
Joone Hur
Comment 5 2011-05-09 17:03:52 PDT
#59821 fixed this issue.
Note You need to log in before you can comment on or make changes to this bug.