Include thread CPU time in Timeline

Include thread CPU time in Timeline.

- [x] Refactor old OSThread interface for querying thread CPU time into OS::GetCurrentThreadCPUMicros.
- [x] Add data to TimelineEvent to track thread CPU time.
- [x] Include "tts" and "tdur" properties when converting the timeline to JSON

Fixes #26396

BUG=
R=rmacnak@google.com

Committed: https://github.com/dart-lang/sdk/commit/f6b570aa2a8849a1b44831836eb115759170283e

[Cutch, 2016-05-05 23:03:19]

Description was changed from

==========
Include thread CPU time in Timeline

- [x] Refactor old OSThread interface for querying thread CPU time into
OS::GetCurrentThreadCPUMicros.
- [x] Add data to TimelineEvent to track thread CPU time.
- [x] Include "tts" and "tdur" properties when converting the timeline to JSON

Fixes #26396

BUG=
==========

to

==========
Include thread CPU time in Timeline.

- [x] Refactor old OSThread interface for querying thread CPU time into
OS::GetCurrentThreadCPUMicros.
- [x] Add data to TimelineEvent to track thread CPU time.
- [x] Include "tts" and "tdur" properties when converting the timeline to JSON

Fixes #26396

BUG=
==========

[Cutch, 2016-05-05 23:03:19]

johnmccutchan@google.com changed reviewers:
+ rmacnak@google.com

[rmacnak, 2016-05-05 23:26:07]

lgtm

[Cutch, 2016-05-05 23:34:13]

Description was changed from

==========
Include thread CPU time in Timeline.

- [x] Refactor old OSThread interface for querying thread CPU time into
OS::GetCurrentThreadCPUMicros.
- [x] Add data to TimelineEvent to track thread CPU time.
- [x] Include "tts" and "tdur" properties when converting the timeline to JSON

Fixes #26396

BUG=
==========

to

==========
Include thread CPU time in Timeline.

- [x] Refactor old OSThread interface for querying thread CPU time into
OS::GetCurrentThreadCPUMicros.
- [x] Add data to TimelineEvent to track thread CPU time.
- [x] Include "tts" and "tdur" properties when converting the timeline to JSON

Fixes #26396

BUG=
R=rmacnak@google.com

Committed:
https://github.com/dart-lang/sdk/commit/f6b570aa2a8849a1b44831836eb115759170283e
==========

[Cutch, 2016-05-05 23:34:14]

Committed patchset #2 (id:20001) manually as
f6b570aa2a8849a1b44831836eb115759170283e (presubmit successful).

[jamesr, 2016-05-09 19:52:21]

jamesr@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2016-05-09 19:52:22]

https://codereview.chromium.org/1960483002/diff/20001/runtime/vm/os_macos.cc
File runtime/vm/os_macos.cc (right):

https://codereview.chromium.org/1960483002/diff/20001/runtime/vm/os_macos.cc#...
runtime/vm/os_macos.cc:144: // TODO(johnmccutchan): Implement this. No
implementation exists in base.
FYI Chinmay investigated this function for iOS for Flutter and found that the
Mac version works fine on modern iOS:

https://github.com/domokit/base/blob/master/time/time_mac.cc#L71

[Cutch, 2016-05-09 20:38:41]

https://codereview.chromium.org/1960483002/diff/20001/runtime/vm/os_macos.cc
File runtime/vm/os_macos.cc (right):

https://codereview.chromium.org/1960483002/diff/20001/runtime/vm/os_macos.cc#...
runtime/vm/os_macos.cc:144: // TODO(johnmccutchan): Implement this. No
implementation exists in base.
On 2016/05/09 19:52:22, jamesr wrote:
> FYI Chinmay investigated this function for iOS for Flutter and found that the
> Mac version works fine on modern iOS:
> 
> https://github.com/domokit/base/blob/master/time/time_mac.cc#L71

Acknowledged.