Closed
Bug 573603
Opened 14 years ago
Closed 14 years ago
Speed up Page Info > Media
Categories
(Firefox :: Page Info Window, defect)
Firefox
Page Info Window
Tracking
()
RESOLVED
FIXED
Firefox 3.7a5
Tracking | Status | |
---|---|---|
status1.9.2 | --- | .9-fixed |
People
(Reporter: mozilla.bugs, Assigned: mozilla.bugs)
References
()
Details
(Keywords: perf)
Attachments
(1 file, 1 obsolete file)
1.05 KB,
patch
|
db48x
:
review+
beltzner
:
approval1.9.2.9+
|
Details | Diff | Splinter Review |
From the discussion in Bug 515549, let's do two--possibly three--things to speed up Page Info's media. 1. Lower the timeouts to the minimum of 10ms 2. Increase the number of elements processed at once 3. (Optional) Replace timeouts with postMessage as described here: http://dbaron.org/log/20100309-faster-timeouts This patch does the first two, and it can do the third if needed. http://commons.wikimedia.org/wiki/Category:Crater_Lake is a decent example of speed testing if you run "View Image Info" on the last image.
Attachment #452889 -
Flags: review?(db48x)
Comment 1•14 years ago
|
||
We need longer testcases. Page Info (trunk) has finished with every page I've tried so far before I could even click on the Media tab.
Assignee | ||
Comment 2•14 years ago
|
||
Setting to 500 elements per loop.
Attachment #452889 -
Attachment is obsolete: true
Attachment #452905 -
Flags: review?(db48x)
Attachment #452889 -
Flags: review?(db48x)
Comment 3•14 years ago
|
||
Comment on attachment 452905 [details] [diff] [review] Patch v. 1.1 r=db48x
Attachment #452905 -
Flags: review?(db48x) → review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/pushloghtml?changeset=a32ea115521e
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Assignee | ||
Updated•14 years ago
|
Target Milestone: Firefox 3.7a6 → Firefox 3.7a5
Assignee | ||
Comment 5•14 years ago
|
||
Comment on attachment 452905 [details] [diff] [review] Patch v. 1.1 From bug 579717 this appears to fix hangs and performance problems on Mac OSX--more than it does on Windows. I propose that we port the change to 1.9.2. I will quickly make sure that the patch doesn't conflict with anything, but it shouldn't touch code that's 2.0-branch specific.
Attachment #452905 -
Flags: approval1.9.2.8?
Comment 7•14 years ago
|
||
Comment on attachment 452905 [details] [diff] [review] Patch v. 1.1 Please renominate for branch landing once you've confirmed that it compiles on that branch, preferably with tryserver builds and test runs! :)
Attachment #452905 -
Flags: approval1.9.2.8? → approval1.9.2.8-
Assignee | ||
Comment 8•14 years ago
|
||
Comment on attachment 452905 [details] [diff] [review] Patch v. 1.1 This works on 1.9.2 like it does on Trunk. The patch doesn't need any branch-specific changes.
Attachment #452905 -
Flags: approval1.9.2.8- → approval1.9.2.8?
Comment 9•14 years ago
|
||
Comment on attachment 452905 [details] [diff] [review] Patch v. 1.1 a=beltzner for mozilla-1.9.2, simple safe perf improvement for non-critical code path
Attachment #452905 -
Flags: approval1.9.2.9? → approval1.9.2.9+
Comment 11•14 years ago
|
||
Can the same type of fix be applied for bug 513092?
Assignee | ||
Comment 12•14 years ago
|
||
(In reply to comment #11) > Can the same type of fix be applied for bug 513092? It's all a question of the code that populates that list. Either the code is using timeouts or it's waiting on something else. If it is the former, then yes, this same philosophy can be applied, but if it is the latter, I'm not sure. Since I don't know that code or what would be causing it, I couldn't say, but someone like Dao or Gavin would have better answers.
Assignee | ||
Updated•14 years ago
|
Whiteboard: [c-n 1.9.2]
You need to log in
before you can comment on or make changes to this bug.
Description
•