[tracing] Introduce TraceStateObserver interface to the platform.

[tracing] Introduce TraceStateObserver interface to the platform.

Introduce TraceStateObserver interface along with add/remove
methods to the platform.

BUG=chromium:406277
Committed: https://crrev.com/ce037789264b33e3edf302fab74736537c04cf22
Cr-Commit-Position: refs/heads/master@{#39513}

[alph, 2016-09-15 23:18:52]

alph@chromium.org changed reviewers:
+ fmeawad@chromium.org, jochen@chromium.org

[alph, 2016-09-15 23:18:53]

Fadi, Jochen, could you please take a look.

I'm going to use it to start/stop emitting CPU profile events into the trace.

[alph, 2016-09-15 23:19:00]

The CQ bit was checked by alph@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-15 23:19:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fmeawad, 2016-09-15 23:43:48]

lgtm

[commit-bot: I haz the power, 2016-09-15 23:48:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-15 23:48:51]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-09-19 14:23:50]

lgtm

https://codereview.chromium.org/2344893005/diff/1/include/v8-platform.h
File include/v8-platform.h (right):

https://codereview.chromium.org/2344893005/diff/1/include/v8-platform.h#newco...
include/v8-platform.h:170: virtual ~TraceStateObserver() {}
nit. = default

[alph, 2016-09-19 19:20:27]

https://codereview.chromium.org/2344893005/diff/1/include/v8-platform.h
File include/v8-platform.h (right):

https://codereview.chromium.org/2344893005/diff/1/include/v8-platform.h#newco...
include/v8-platform.h:170: virtual ~TraceStateObserver() {}
On 2016/09/19 14:23:50, jochen wrote:
> nit. = default

Done.

[alph, 2016-09-19 19:20:42]

The CQ bit was checked by alph@chromium.org

[alph, 2016-09-19 19:20:43]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org,
fmeawad@chromium.org
Link to the patchset: https://codereview.chromium.org/2344893005/#ps20001
(title: "addressing nit")

[commit-bot: I haz the power, 2016-09-19 19:20:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 20:05:59]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-19 21:01:43]

Description was changed from

==========
[tracing] Introduce TraceStateObserver interface to the platform.

Introduce TraceStateObserver interface along with add/remove
methods to the platform.

BUG=chromium:406277
==========

to

==========
[tracing] Introduce TraceStateObserver interface to the platform.

Introduce TraceStateObserver interface along with add/remove
methods to the platform.

BUG=chromium:406277

Committed: https://crrev.com/ce037789264b33e3edf302fab74736537c04cf22
Cr-Commit-Position: refs/heads/master@{#39513}
==========

[commit-bot: I haz the power, 2016-09-19 21:01:44]

Patchset 2 (id:??) landed as
https://crrev.com/ce037789264b33e3edf302fab74736537c04cf22
Cr-Commit-Position: refs/heads/master@{#39513}