Profiler: Add the document URL in DoReflow

RESOLVED FIXED in mozilla17

Status

()

Core
Gecko Profiler
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: BenWa, Unassigned)

Tracking

unspecified
mozilla17
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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.
Attachment #654900 - Flags: review?(roc)
Attachment #654900 - Flags: review?(roc) → review+
(Reporter)

Comment 1

5 years ago
inbound is closed :(
Keywords: checkin-needed
(Reporter)

Updated

5 years ago
Blocks: 784920
(Reporter)

Comment 2

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/5870ef9c6afa
Target Milestone: --- → mozilla17
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/5870ef9c6afa
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.