Add null check to animations for registered custom property initial values

Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234, 685472
Review-Url: https://codereview.chromium.org/2649863006
Cr-Original-Commit-Position: refs/heads/master@{#445969}
Committed: https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf5f23c0d1b3ff
Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#447095}
Committed: https://chromium.googlesource.com/chromium/src/+/1f1de2e98a13bae871ef18de230ec8ff45965eaf

[alancutter (OOO until 2018), 2017-01-24 02:38:26]

Description was changed from

==========
Add null check for registered custom property initial value to animations

BUG=dogs
==========

to

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234
==========

[alancutter (OOO until 2018), 2017-01-24 02:38:26]

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

[Eric Willigers, 2017-01-25 04:04:01]

lgtm

[alancutter (OOO until 2018), 2017-01-25 04:31:24]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2017-01-25 04:32:19]

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-25 06:21:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485318684401630,
"parent_rev": "99d6b7f8f877750caa5654d2c62594b3761fd1ad", "commit_rev":
"f2ec8922cbe5f632a937cf242faf5f23c0d1b3ff"}

[commit-bot: I haz the power, 2017-01-25 06:22:59]

Description was changed from

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234
==========

to

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234

Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#445969}
Committed:
https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf...
==========

[commit-bot: I haz the power, 2017-01-25 06:22:59]

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

[sof, 2017-01-25 11:11:56]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2650403002/ by sigbjornf@opera.com.

The reason for reverting is: This goes together with the revert
https://codereview.chromium.org/2649103008/ ; looks like the two CLs didn't
match up, 


https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.10/build...

(Hope the double revert doesn't cause too much work.).

[alancutter (OOO until 2018), 2017-01-30 20:13:14]

Description was changed from

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234

Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#445969}
Committed:
https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf...
==========

to

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234,685472

Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#445969}
Committed:
https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf...
==========

[alancutter (OOO until 2018), 2017-01-30 20:56:54]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2017-01-30 20:56:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
ericwilligers@chromium.org
Link to the patchset: https://codereview.chromium.org/2649863006/#ps40001
(title: "Fix shadowed variable")

[commit-bot: I haz the power, 2017-01-30 20:57:50]

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-30 22:47:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1485809814838710,
"parent_rev": "1e8b5cfd4f7a74990a9b3bb7e0a0f095ca91ef3b", "commit_rev":
"1f1de2e98a13bae871ef18de230ec8ff45965eaf"}

[commit-bot: I haz the power, 2017-01-30 22:48:09]

Description was changed from

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234,685472

Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#445969}
Committed:
https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf...
==========

to

==========
Add null check to animations for registered custom property initial values

After a recent change to CSSInterpolationType for registered custom properties
it was assumed that all registered custom properties had initial CSSValues.
This is not the case and null derefs were reachable. This patch fixes up
the false assumption.

BUG=684234,685472

Review-Url: https://codereview.chromium.org/2649863006
Cr-Original-Commit-Position: refs/heads/master@{#445969}
Committed:
https://chromium.googlesource.com/chromium/src/+/f2ec8922cbe5f632a937cf242faf...
Review-Url: https://codereview.chromium.org/2649863006
Cr-Commit-Position: refs/heads/master@{#447095}
Committed:
https://chromium.googlesource.com/chromium/src/+/1f1de2e98a13bae871ef18de230e...
==========

[commit-bot: I haz the power, 2017-01-30 22:48:10]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/1f1de2e98a13bae871ef18de230e...