Bug 77906 - Review page takes forever to load when the patch is big
Summary: Review page takes forever to load when the patch is big
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-06 15:28 PST by Ryosuke Niwa
Modified: 2023-07-03 09:04 PDT (History)
8 users (show)

See Also:

Attachments
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 Ryosuke Niwa 2012-02-06 15:28:58 PST 
Both ToT WebKit and Chromium appears to hang when I try to open:
https://bugs.webkit.org/attachment.cgi?id=125284&action=review

It's only 2.14MB! WebKit shouldn't be hanging on this page.
Comment 1 Ryosuke Niwa 2012-02-06 15:32:47 PST 
Note the patch is for https://bugs.webkit.org/show_bug.cgi?id=77504.
Comment 2 Antonio Gomes 2012-02-07 11:10:35 PST 
on my blackerry device it rebooted the phone, on iphone it killed the browser...
Comment 3 Ryosuke Niwa 2012-02-07 11:12:16 PST 
Just note that this bug is about fixing WebKit to load it fast, not about "fixing" our Bugzilla.
Comment 4 Ahmad Saleem 2023-07-02 15:53:49 PDT 
@ap - Now that we have moved to GitHub for review, do you think it is OK to keep it open. Also devices has evolved a lot and also I am not able to open link from Comment 0 but this also work (another 1.4 MB one):

https://bugs.webkit.org/attachment.cgi?id=125233&action=review

It takes time to load in Safari & Chrome but should we keep it as open for performance profiling?
Comment 5 Alexey Proskuryakov 2023-07-03 09:04:04 PDT 
There were a lot of line layout performance improvements lately, and indeed, large patches appears to work well now.

At this point, patches in Bugzilla remain a supported way to contribute to WebKit.