Last Comment Bug 785287 - Profiler: Add the document URL in DoReflow
: Profiler: Add the document URL in DoReflow
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Gecko Profiler (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: mozilla17
Assigned To: Nobody; OK to take it and work on it
:
:
Mentors:
Depends on: 707308
Blocks: 784920
  Show dependency treegraph
 
Reported: 2012-08-23 19:39 PDT by Benoit Girard (:BenWa)
Modified: 2012-08-24 20:01 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (940 bytes, patch)
2012-08-23 19:39 PDT, Benoit Girard (:BenWa)
roc: review+
Details | Diff | Splinter Review

Description Benoit Girard (:BenWa) 2012-08-23 19:39:43 PDT
Created attachment 654900 [details] [diff] [review]
patch

Here's an example profile in this patch:
http://people.mozilla.com/~bgirard/cleopatra/?report=830af9dcdc0c1f36df7222a0da8a63827d8b0de5

Notice how the top callstack is:
Timer::Fire
nsRefreshDriver::Notify
layout::DoReflow (chrome://browser/content/browser.xul)

This should help with bugs like bug 784920 where we're not entirely sure if the jank is coming from the twitter tab or another tab with a widget.
Comment 1 Benoit Girard (:BenWa) 2012-08-23 20:11:24 PDT
inbound is closed :(
Comment 3 Ryan VanderMeulen [:RyanVM] 2012-08-24 20:01:41 PDT
https://hg.mozilla.org/mozilla-central/rev/5870ef9c6afa

Note You need to log in before you can comment on or make changes to this bug.