Fix crash when setting invalid URL on content

Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes serialization of invalid image CSS values after multiple
style recalcs where previously we should serialize as none instead of
the specified URL.

BUG=599705
Committed: https://crrev.com/37a62ed4183b7ff3c7711650f580ab3b7973f046
Cr-Commit-Position: refs/heads/master@{#385409}

[alancutter (OOO until 2018), 2016-04-06 06:52:51]

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

[alancutter (OOO until 2018), 2016-04-06 06:53:14]

Description was changed from

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes double serialization of invalid image CSS values where
previously we should serialize as none instead of the specified URL.

BUG=559705
==========

to

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes double serialization of invalid image CSS values where
previously we should serialize as none instead of the specified URL.

BUG=599705
==========

[alancutter (OOO until 2018), 2016-04-06 06:55:36]

Description was changed from

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes double serialization of invalid image CSS values where
previously we should serialize as none instead of the specified URL.

BUG=599705
==========

to

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes double recalc serialization of invalid image CSS values where
previously we should serialize as none instead of the specified URL.

BUG=599705
==========

[alancutter (OOO until 2018), 2016-04-06 06:56:09]

Description was changed from

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes double recalc serialization of invalid image CSS values where
previously we should serialize as none instead of the specified URL.

BUG=599705
==========

to

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes serialization of invalid image CSS values after multiple
style recalcs where previously we should serialize as none instead of
the specified URL.

BUG=599705
==========

[Timothy Loh, 2016-04-06 06:56:42]

On 2016/04/06 06:52:52, alancutter wrote:

lgtm

[alancutter (OOO until 2018), 2016-04-06 06:57:43]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2016-04-06 06:58:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1865603004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1865603004/20001

[commit-bot: I haz the power, 2016-04-06 08:58:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-06 09:00:30]

Description was changed from

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes serialization of invalid image CSS values after multiple
style recalcs where previously we should serialize as none instead of
the specified URL.

BUG=599705
==========

to

==========
Fix crash when setting invalid URL on content

This change fixes a crash caused by a recent refactor to content style
application.
This also fixes serialization of invalid image CSS values after multiple
style recalcs where previously we should serialize as none instead of
the specified URL.

BUG=599705

Committed: https://crrev.com/37a62ed4183b7ff3c7711650f580ab3b7973f046
Cr-Commit-Position: refs/heads/master@{#385409}
==========

[commit-bot: I haz the power, 2016-04-06 09:00:32]

Patchset 2 (id:??) landed as
https://crrev.com/37a62ed4183b7ff3c7711650f580ab3b7973f046
Cr-Commit-Position: refs/heads/master@{#385409}