Bug 77906

Summary:	Review page takes forever to load when the patch is big
Product:	WebKit	Reporter:	Ryosuke Niwa <rniwa>
Component:	Layout and Rendering	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED CONFIGURATION CHANGED
Severity:	Normal	CC:	ahmad.saleem792, ap, jchaffraix, kling, koivisto, morrita, tonikitoo, zalan
Priority:	P2
Version:	528+ (Nightly build)
Hardware:	Unspecified
OS:	Unspecified

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.