Remove TransformOperation::None

Remove TransformOperation::None

This patch removes the enum value TransformOperation::None, which is
currently only used in an ASSERT_NOT_REACHED case.

Review-Url: https://codereview.chromium.org/2616863002
Cr-Commit-Position: refs/heads/master@{#443092}
Committed: https://chromium.googlesource.com/chromium/src/+/54aed49c7088d08159b8ed7485a070160cd98d40

[Timothy Loh, 2017-01-05 06:20:01]

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

[commit-bot: I haz the power, 2017-01-05 06:20:11]

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

[Timothy Loh, 2017-01-05 06:26:16]

timloh@chromium.org changed reviewers:
+ junov@chromium.org

[Timothy Loh, 2017-01-05 06:26:16]

Miscellaneous clean-up :)

[alancutter (OOO until 2018), 2017-01-05 06:46:39]

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

[alancutter (OOO until 2018), 2017-01-05 06:46:40]

lgtm

https://codereview.chromium.org/2616863002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp (right):

https://codereview.chromium.org/2616863002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/css/resolver/TransformBuilder.cpp:100: return
TransformOperation::Scale;
Optional nit: Consider putting the default case at the top and just falling
through to the first case statement.

[commit-bot: I haz the power, 2017-01-05 09:49:51]

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

[commit-bot: I haz the power, 2017-01-05 09:49:51]

Dry run: This issue passed the CQ dry run.

[Justin Novosad, 2017-01-10 16:40:03]

lgtm

https://codereview.chromium.org/2616863002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/animation/AnimationTranslationUtil.cpp
(left):

https://codereview.chromium.org/2616863002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/animation/AnimationTranslationUtil.cpp:123:
case TransformOperation::None:
Nit: perhaps a "default:" for this switch?
Your call.

[Timothy Loh, 2017-01-11 06:43:29]

The CQ bit was checked by timloh@chromium.org

[Timothy Loh, 2017-01-11 06:43:29]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org,
alancutter@chromium.org
Link to the patchset: https://codereview.chromium.org/2616863002/#ps20001
(title: "address comments")

[commit-bot: I haz the power, 2017-01-11 06:43:46]

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, 2017-01-11 08:03:37]

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

[commit-bot: I haz the power, 2017-01-11 08:03:39]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Timothy Loh, 2017-01-12 00:12:05]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2017-01-12 00:13:35]

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, 2017-01-12 00:55:26]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484179925097890,
"parent_rev": "2c9cbe19a0fcbf5386caec36010d201ec3430713", "commit_rev":
"54aed49c7088d08159b8ed7485a070160cd98d40"}

[commit-bot: I haz the power, 2017-01-12 00:55:54]

Description was changed from

==========
Remove TransformOperation::None

This patch removes the enum value TransformOperation::None, which is
currently only used in an ASSERT_NOT_REACHED case.
==========

to

==========
Remove TransformOperation::None

This patch removes the enum value TransformOperation::None, which is
currently only used in an ASSERT_NOT_REACHED case.

Review-Url: https://codereview.chromium.org/2616863002
Cr-Commit-Position: refs/heads/master@{#443092}
Committed:
https://chromium.googlesource.com/chromium/src/+/54aed49c7088d08159b8ed7485a0...
==========

[commit-bot: I haz the power, 2017-01-12 00:55:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/54aed49c7088d08159b8ed7485a0...