Put all reasons in MainThreadScrollingReason into one enum

Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/48e95c8ed1c285048d172fdd00361673f808bfee
Cr-Commit-Position: refs/heads/master@{#429226}

[wychen, 2016-11-02 03:58:55]

Description was changed from

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary and behavior changes.

BUG=661401
==========

to

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary and behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[wychen, 2016-11-02 04:21:25]

wychen@chromium.org changed reviewers:
+ aelias@chromium.org

[wychen, 2016-11-02 04:21:25]

PTAL

[aelias_OOO_until_Jul13, 2016-11-02 06:23:37]

Seems legit, I don't see any reason why it was written the previous way (Dana
introduced it in https://codereview.chromium.org/1604053002 as a replacement for
a C++03-style enum.)  lgtm

[wychen, 2016-11-02 06:52:31]

Description was changed from

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary and behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[wychen, 2016-11-02 06:56:15]

The CQ bit was checked by wychen@chromium.org

[commit-bot: I haz the power, 2016-11-02 06:56:42]

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-11-02 07:11:31]

Description was changed from

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[commit-bot: I haz the power, 2016-11-02 07:11:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-02 07:13:39]

Description was changed from

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
Put all reasons in MainThreadScrollingReason into one enum

This is to avoid a false positive in enum type equality checking in
https://codereview.chromium.org/2469993002/.

No intended binary or behavior changes.

BUG=661401
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/48e95c8ed1c285048d172fdd00361673f808bfee
Cr-Commit-Position: refs/heads/master@{#429226}
==========

[commit-bot: I haz the power, 2016-11-02 07:13:40]

Patchset 1 (id:??) landed as
https://crrev.com/48e95c8ed1c285048d172fdd00361673f808bfee
Cr-Commit-Position: refs/heads/master@{#429226}

[wychen, 2016-11-02 21:29:50]

The CQ was way too fast. It turned out that none of the bots compiled the code,
because the GN file was incomplete. Filed a bug:
https://bugs.chromium.org/p/chromium/issues/detail?id=661774