Remove HiResEventTimeStamp runtime flag (status=stable)

Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501
Committed: https://crrev.com/cf2cd6b49ba5aed9c7229dda27c5c5d8db96fc41
Cr-Commit-Position: refs/heads/master@{#408087}

[rwlbuis, 2016-07-19 14:28:26]

Remove HiResEventTimeStamp runtime flag (status=stable)

BUG=402536

[rwlbuis, 2016-07-19 19:49:42]

Description was changed from

==========
hires WIP

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

to

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

BUG=402536
==========

[rwlbuis, 2016-07-19 19:52:11]

Description was changed from

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

BUG=402536
==========

to

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536
==========

[rwlbuis, 2016-07-19 19:52:31]

rob.buis@samsung.com changed reviewers:
+ majidvp@chromium.org

[rwlbuis, 2016-07-19 19:52:32]

PTAL.

[majidvp, 2016-07-22 17:53:02]

Thanks. Please also mention issue 605501 in the description.

lgtm with nit.

https://codereview.chromium.org/2161693003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/events/Event.h (left):

https://codereview.chromium.org/2161693003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/events/Event.h:144: // |m_createTime|). For more
info see http://crbug.com/160524
Please update this comment as well. Also relevant comment in Event.idl.

[rwlbuis, 2016-07-22 17:57:18]

Description was changed from

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536
==========

to

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501
==========

[rwlbuis, 2016-07-22 18:08:35]

On 2016/07/22 17:53:02, majidvp wrote:
> Thanks. Please also mention issue 605501 in the description.

Ah, did not know a bug existed for this! I added it.

> lgtm with nit.
> 
>
https://codereview.chromium.org/2161693003/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/events/Event.h (left):
> 
>
https://codereview.chromium.org/2161693003/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/events/Event.h:144: // |m_createTime|). For
more
> info see http://crbug.com/160524
> Please update this comment as well. Also relevant comment in Event.idl.

Done.

[rwlbuis, 2016-07-24 18:11:25]

rob.buis@samsung.com changed reviewers:
+ mkwst@chromium.org

[rwlbuis, 2016-07-24 18:11:26]

PTAL.

[majidvp, 2016-07-24 18:23:29]

Still lgtm

On Sun, Jul 24, 2016, 2:11 PM <rob.buis@samsung.com> wrote:

> PTAL.
>
> https://codereview.chromium.org/2161693003/
>

-- 
You received this message because you are subscribed to the Google Groups "Blink
Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[majidvp, 2016-07-24 18:23:29]

Still lgtm

On Sun, Jul 24, 2016, 2:11 PM <rob.buis@samsung.com> wrote:

> PTAL.
>
> https://codereview.chromium.org/2161693003/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[rwlbuis, 2016-07-26 19:35:59]

Patchset #5 (id:80001) has been deleted

[Mike West, 2016-07-27 08:40:38]

LGTM 2, for what it's worth.

[rwlbuis, 2016-07-27 09:12:00]

The CQ bit was checked by rob.buis@samsung.com

[rwlbuis, 2016-07-27 09:12:00]

The patchset sent to the CQ was uploaded after l-g-t-m from majidvp@chromium.org
Link to the patchset: https://codereview.chromium.org/2161693003/#ps60001
(title: "Change comment in Event.h")

[commit-bot: I haz the power, 2016-07-27 09:12:20]

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-07-27 10:28:09]

Description was changed from

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501
==========

to

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501
==========

[commit-bot: I haz the power, 2016-07-27 10:28:10]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-27 10:31:21]

Description was changed from

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501
==========

to

==========
Remove HiResEventTimeStamp runtime flag (status=stable)

This also removes the now unused m_createTime member variable.

BUG=402536, 605501

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

[commit-bot: I haz the power, 2016-07-27 10:31:22]

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