Bug 942054 - bump range and number of buckets for content reflow timings and add interruptible boolean. r=bz

This commit is contained in:
John Daggett 2013-11-26 13:34:31 +09:00
parent fa697d2df2
commit 56467ffc38
2 changed files with 23 additions and 9 deletions

View File

@ -8203,6 +8203,9 @@ PresShell::DoVerifyReflow()
}
#endif
// used with Telemetry metrics
#define NS_LONG_REFLOW_TIME_MS 5000
bool
PresShell::ProcessReflowCommands(bool aInterruptible)
{
@ -8295,6 +8298,9 @@ PresShell::ProcessReflowCommands(bool aInterruptible)
}
if (mDocument->GetRootElement()) {
TimeDuration elapsed = TimeStamp::Now() - timerStart;
int32_t intElapsed = int32_t(elapsed.ToMilliseconds());
Telemetry::ID id;
if (mDocument->GetRootElement()->IsXUL()) {
id = mIsActive
@ -8302,10 +8308,14 @@ PresShell::ProcessReflowCommands(bool aInterruptible)
: Telemetry::XUL_BACKGROUND_REFLOW_MS;
} else {
id = mIsActive
? Telemetry::HTML_FOREGROUND_REFLOW_MS
: Telemetry::HTML_BACKGROUND_REFLOW_MS;
? Telemetry::HTML_FOREGROUND_REFLOW_MS_2
: Telemetry::HTML_BACKGROUND_REFLOW_MS_2;
}
Telemetry::Accumulate(id, intElapsed);
if (intElapsed > NS_LONG_REFLOW_TIME_MS) {
Telemetry::Accumulate(Telemetry::LONG_REFLOW_INTERRUPTIBLE,
aInterruptible ? 1 : 0);
}
Telemetry::AccumulateTimeDelta(id, timerStart);
}
return !interrupted;

View File

@ -3002,18 +3002,22 @@
"n_buckets": 10,
"description": "XUL reflows in background windows (ms)"
},
"HTML_FOREGROUND_REFLOW_MS": {
"HTML_FOREGROUND_REFLOW_MS_2": {
"kind": "exponential",
"high": "3000",
"n_buckets": 10,
"high": "10000",
"n_buckets": 20,
"description": "HTML reflows in foreground windows (ms)"
},
"HTML_BACKGROUND_REFLOW_MS": {
"HTML_BACKGROUND_REFLOW_MS_2": {
"kind": "exponential",
"high": "3000",
"n_buckets": 10,
"high": "10000",
"n_buckets": 20,
"description": "HTML reflows in background windows (ms)"
},
"LONG_REFLOW_INTERRUPTIBLE": {
"kind": "boolean",
"description": "Long running reflow, interruptible or not"
},
"XUL_INITIAL_FRAME_CONSTRUCTION": {
"kind": "exponential",
"high": "3000",