Chromium Code Reviews| Index: Source/devtools/front_end/timeline/TimelineFrameOverview.js |
| diff --git a/Source/devtools/front_end/timeline/TimelineFrameOverview.js b/Source/devtools/front_end/timeline/TimelineFrameOverview.js |
| index 0672b6b281ecc0060dad63f3b04e00b94f8636a3..399a2fb5c412fe38ba0a05f9dad4be0b45d62ffb 100644 |
| --- a/Source/devtools/front_end/timeline/TimelineFrameOverview.js |
| +++ b/Source/devtools/front_end/timeline/TimelineFrameOverview.js |
| @@ -136,6 +136,11 @@ WebInspector.TimelineFrameOverview.prototype = { |
| */ |
| _computeTargetFrameLength: function(frames) |
| { |
| + const targetFPS = 20; |
|
apavlov
2014/07/25 09:10:10
They say "const" prevents function optimizations.
|
| + var result = 1000.0 / targetFPS; |
| + if (!frames.length) |
| + return result; |
| + |
| var durations = []; |
| for (var i = 0; i < frames.length; ++i) { |
| if (frames[i]) |
| @@ -145,8 +150,6 @@ WebInspector.TimelineFrameOverview.prototype = { |
| // Optimize appearance for 30fps, but leave some space so it's evident when a frame overflows. |
| // However, if at least half frames won't fit at this scale, fall back to using autoscale. |
| - const targetFPS = 20; |
| - var result = 1000.0 / targetFPS; |
| if (result >= medianFrameLength) |
| return result; |