Fix crash when referencing a registered non-inherited custom property

Fix crash when referencing a registered non-inherited custom property

This patch fixes a crash that occurs when a registered non-inherited
custom property is referenced on an element which doesn't explicitly
have any registered non-inherited custom properties set. The code was
missing a null-check for the non-inherited custom properties object.
Note that elsewhere in the file this is not needed as the object is
guaranteed to exist when we are resolving such properties.

BUG=671484
Committed: https://crrev.com/1bf1f4ca99b8ea3183c9aa358338e63d5de07b93
Cr-Commit-Position: refs/heads/master@{#436880}

[Timothy Loh, 2016-12-06 04:52:04]

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

[commit-bot: I haz the power, 2016-12-06 04:52:18]

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

[Timothy Loh, 2016-12-06 05:11:26]

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

[Timothy Loh, 2016-12-06 05:11:27]

https://codereview.chromium.org/2552163002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/custom-properties/registered-property-initial.html
(right):

https://codereview.chromium.org/2552163002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/custom-properties/registered-property-initial.html:17:
assert_equals(computedStyle.getPropertyValue('--inherited-color'), 'pink');
The discrepancy pink vs rgb().. is tracked in bug 671490

[alancutter (OOO until 2018), 2016-12-06 05:34:02]

lgtm

[commit-bot: I haz the power, 2016-12-06 05:41:28]

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

[commit-bot: I haz the power, 2016-12-06 05:41:29]

Dry run: 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, 2016-12-06 05:43:17]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2016-12-06 05:43:28]

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-12-06 06:22:59]

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

[commit-bot: I haz the power, 2016-12-06 06:23:00]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Timothy Loh, 2016-12-07 00:47:58]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2016-12-07 00:48:45]

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-12-07 03:52:46]

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

[commit-bot: I haz the power, 2016-12-07 03:52:47]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Timothy Loh, 2016-12-07 03:54:27]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2016-12-07 03:55:02]

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-12-07 06:45:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481082867731440, "parent_rev":
"5e05218f7678acbadc8a1469e6141ff593bd42fe", "commit_rev":
"9ae93810a3f76f4b4a9d3cde080d4242c6ffc2ff"}

[commit-bot: I haz the power, 2016-12-07 06:45:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 06:48:13]

Description was changed from

==========
Fix crash when referencing a registered non-inherited custom property

This patch fixes a crash that occurs when a registered non-inherited
custom property is referenced on an element which doesn't explicitly
have any registered non-inherited custom properties set. The code was
missing a null-check for the non-inherited custom properties object.
Note that elsewhere in the file this is not needed as the object is
guaranteed to exist when we are resolving such properties.

BUG=671484
==========

to

==========
Fix crash when referencing a registered non-inherited custom property

This patch fixes a crash that occurs when a registered non-inherited
custom property is referenced on an element which doesn't explicitly
have any registered non-inherited custom properties set. The code was
missing a null-check for the non-inherited custom properties object.
Note that elsewhere in the file this is not needed as the object is
guaranteed to exist when we are resolving such properties.

BUG=671484

Committed: https://crrev.com/1bf1f4ca99b8ea3183c9aa358338e63d5de07b93
Cr-Commit-Position: refs/heads/master@{#436880}
==========

[commit-bot: I haz the power, 2016-12-07 06:48:14]

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