Timeline: disable start/stop button and properly indicate progress when stop is pending

Timeline: disable start/stop button and properly indicate progress when stop is pending

Now that we may fetch a lot of trace (or buffered timeline) events after
timeline is stopped, we need a better indication of that. The button remaining
red after it's pressed is very annoying. Let's turn it off and disable till
we get all the events, and if the progress pane is on, display the appropriate
message there.

BUG=361045
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176214

[yurys, 2014-06-16 07:18:08]

https://codereview.chromium.org/322403004/diff/20001/Source/devtools/front_en...
File Source/devtools/front_end/timeline/TimelinePanel.js (right):

https://codereview.chromium.org/322403004/diff/20001/Source/devtools/front_en...
Source/devtools/front_end/timeline/TimelinePanel.js:702:
this.toggleTimelineButton.title = WebInspector.UIString("Another profiler is
already active");
Hm, why do we show this message only when  toggled=false? It seems to me that we
need to to this when toggled=true and threre is already another profile running,
wdyt?

[caseq, 2014-06-16 08:39:30]

On 2014/06/16 07:18:08, yurys wrote:
>
https://codereview.chromium.org/322403004/diff/20001/Source/devtools/front_en...
> File Source/devtools/front_end/timeline/TimelinePanel.js (right):
> 
>
https://codereview.chromium.org/322403004/diff/20001/Source/devtools/front_en...
> Source/devtools/front_end/timeline/TimelinePanel.js:702:
> this.toggleTimelineButton.title = WebInspector.UIString("Another profiler is
> already active");
> Hm, why do we show this message only when  toggled=false? It seems to me that
we
> need to to this when toggled=true and threre is already another profile
running,
> wdyt?

I think when the button is toggled, this means we have timeline running and
(presumably, in the future?) the lock acquired.
The meaning of the button is "Stop timeline" in this case.

[yurys, 2014-06-16 09:47:18]

lgtm

[caseq, 2014-06-16 14:09:49]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2014-06-16 14:10:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/caseq@chromium.org/322403004/20001

[commit-bot: I haz the power, 2014-06-16 15:39:36]

Change committed as 176214