Fix ImageAnimation constant names after Blink renaming

Fix ImageAnimation constant names after Blink renaming

The ImageAnimation constants such as AnimationNone used to be named like
cAnimationNone. This didn't match other constants within the image
decoders.

After the recent Blink style update, these were renamed like
kCAnimationNone, making them even weirder.

They should be like kAnimationNone.

BUG=717669
Review-Url: https://codereview.chromium.org/2857753002
Cr-Commit-Position: refs/heads/master@{#471648}
Committed: https://chromium.googlesource.com/chromium/src/+/83bcead92eb1c2aec8bcb5e50deb6fc2440240f4

[cblume, 2017-05-02 20:02:40]

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

[commit-bot: I haz the power, 2017-05-02 20:03:02]

Dry run: CQ is trying da patch.

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

[cblume, 2017-05-02 20:41:08]

cblume@chromium.org changed reviewers:
+ noel@chromium.org, pkasting@chromium.org, scroggo@chromium.org

[cblume, 2017-05-02 20:41:08]

PTAL

[commit-bot: I haz the power, 2017-05-02 21:46:14]

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

[commit-bot: I haz the power, 2017-05-02 21:46:15]

Dry run: This issue passed the CQ dry run.

[Noel Gordon, 2017-05-03 00:14:34]

LGTM.

[Peter Kasting, 2017-05-05 00:26:19]

RS LGTM

[cblume, 2017-05-08 20:12:31]

On 2017/05/05 00:26:19, Peter Kasting wrote:
> RS LGTM

What is the "RS" in "RS LGTM"?

[pdr., 2017-05-08 20:14:55]

On 2017/05/08 at 20:12:31, cblume wrote:
> On 2017/05/05 00:26:19, Peter Kasting wrote:
> > RS LGTM
> 
> What is the "RS" in "RS LGTM"?

Rubber Stamp. I think folks typically use it when they don't look closely at the
patch and are relying on other reviewers.

[cblume, 2017-05-08 20:15:31]

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

[commit-bot: I haz the power, 2017-05-08 20:16:41]

Dry run: 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-05-09 01:28:27]

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

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

Dry run: This issue passed the CQ dry run.

[cblume, 2017-05-14 23:13:40]

The CQ bit was checked by cblume@chromium.org

[cblume, 2017-05-14 23:13:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org, noel@chromium.org
Link to the patchset: https://codereview.chromium.org/2857753002/#ps40001
(title: "Rebasing")

[commit-bot: I haz the power, 2017-05-14 23:13:47]

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

[commit-bot: I haz the power, 2017-05-14 23:13:48]

This CL has an open dependency (Issue 2880533002 Patch 20001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[cblume, 2017-05-14 23:44:13]

The CQ bit was checked by cblume@chromium.org

[cblume, 2017-05-14 23:44:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org, noel@chromium.org
Link to the patchset: https://codereview.chromium.org/2857753002/#ps60001
(title: "Fix incorrect dependent CL")

[commit-bot: I haz the power, 2017-05-14 23:44:25]

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-05-15 01:41:16]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1494805453723620,
"parent_rev": "ff308e070f61164178200816832dc00a974f8430", "commit_rev":
"83bcead92eb1c2aec8bcb5e50deb6fc2440240f4"}

[commit-bot: I haz the power, 2017-05-15 01:41:27]

Description was changed from

==========
Fix ImageAnimation constant names after Blink renaming

The ImageAnimation constants such as AnimationNone used to be named like
cAnimationNone. This didn't match other constants within the image
decoders.

After the recent Blink style update, these were renamed like
kCAnimationNone, making them even weirder.

They should be like kAnimationNone.

BUG=717669
==========

to

==========
Fix ImageAnimation constant names after Blink renaming

The ImageAnimation constants such as AnimationNone used to be named like
cAnimationNone. This didn't match other constants within the image
decoders.

After the recent Blink style update, these were renamed like
kCAnimationNone, making them even weirder.

They should be like kAnimationNone.

BUG=717669

Review-Url: https://codereview.chromium.org/2857753002
Cr-Commit-Position: refs/heads/master@{#471648}
Committed:
https://chromium.googlesource.com/chromium/src/+/83bcead92eb1c2aec8bcb5e50deb...
==========

[commit-bot: I haz the power, 2017-05-15 01:41:28]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/83bcead92eb1c2aec8bcb5e50deb...

[scroggo_chromium, 2017-05-15 12:32:19]

Hey, you didn't properly fix the dependency. You pulled your dependency into
this one, and landed the separate change (which is currently under review at
crrev.com/2880533002) along with this one...

[cblume, 2017-05-15 17:07:11]

On 2017/05/15 12:32:19, scroggo_chromium wrote:
> Hey, you didn't properly fix the dependency. You pulled your dependency into
> this one, and landed the separate change (which is currently under review at
> crrev.com/2880533002) along with this one...

Oh sorry. I'll revert.

That is weird. The latest patch set shows it doesn't depend on it.
Oh, I bet I know what happened. I bet when I accidentally made this branch
dependent I also did a git pull. And I forgot to undo that.

[cblume, 2017-05-15 17:08:18]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/2878363003/ by cblume@chromium.org.

The reason for reverting is: Accidentally included false dependency..