Fix inspector crash after inspecting a link tag

Fix inspector crash after inspecting a link tag

In r357842 text-decoration handling was converted to use
CSSParserTokens, however mistakenly m_currentShorthand was set
even in the case text-decoration falls back to non-shorthand
behavior, causing an ASSERT later. This patch resets
the m_currentShorthand when we fall-back to non-shorthand
behavior, fixing the crash.

BUG=551638

[rwlbuis, 2015-11-05 19:52:27]

Description was changed from

==========
Fix inspector crash after inspecting a link tag

In r357842 text-decoration handling was converted to use
CSSParserTokens, however mistakenly m_currentShorthand was set
even in the case text-decoration falls back to non-shorthand
behavior, causing an ASSERT later. This patch resets
the m_currentShorthand when we fall-back to non-shorthand
behavior.

BUG=551638
==========

to

==========
Fix inspector crash after inspecting a link tag

In r357842 text-decoration handling was converted to use
CSSParserTokens, however mistakenly m_currentShorthand was set
even in the case text-decoration falls back to non-shorthand
behavior, causing an ASSERT later. This patch resets
the m_currentShorthand when we fall-back to non-shorthand
behavior, fixing the crash.

BUG=551638
==========

[rwlbuis, 2015-11-05 19:54:02]

rob.buis@samsung.com changed reviewers:
+ leviw@chromium.org, timloh@chromium.org

[rwlbuis, 2015-11-05 19:54:03]

PTAL. Sadly I could not find a way to create a layout test for this, but it does
fix the crash, verified by testing manually.

[rwlbuis, 2015-11-05 21:32:50]

On 2015/11/05 19:54:03, rwlbuis wrote:
> PTAL. Sadly I could not find a way to create a layout test for this, but it
does
> fix the crash, verified by testing manually.

Will incorporate this fix in bringing the reverted patch back:
https://codereview.chromium.org/1405273005/.