Get rid of wrong assumption in reconcileTextDecorationProperties()

Get rid of wrong assumption in reconcileTextDecorationProperties()

This patch gets rid of |DCHECK()|, which is a wrong assumption, in
|reconcileTextDecorationProperties()| since we have a test case against the
assumption.

BUG=663613
TEST=LayoutTests/editing/execCommand/insert-list-and-strikethrough.html

Committed: https://crrev.com/a9814d2d1bbc0c1d77dad4f2619e21e28757b47f
Cr-Commit-Position: refs/heads/master@{#434857}

[yosin_UTC9, 2016-11-28 09:42:31]

Description was changed from

==========
2016-11-28T18:40:32

BUG=
==========

to

==========
Make reconcileTextDecorationProperties() to handle styles having both
text-decoration and webkit-text-decoration-in-effect

2016-11-28T18:40:32

BUG=
==========

[yosin_UTC9, 2016-11-28 09:42:58]

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

[commit-bot: I haz the power, 2016-11-28 09:43:10]

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

[yosin_UTC9, 2016-11-28 09:46:58]

Description was changed from

==========
Make reconcileTextDecorationProperties() to handle styles having both
text-decoration and webkit-text-decoration-in-effect

2016-11-28T18:40:32

BUG=
==========

to

==========
Get rid of wrong assumption in reconcileTextDecorationProperties()

This patch gets rid of |DCHECK()|, which is a wrong assumption, in
|reconcileTextDecorationProperties()| since we have a test case against the
assumption.

BUG=663613
TEST=LayoutTests/editing/execCommand/insert-list-and-strikethrough.html
==========

[commit-bot: I haz the power, 2016-11-28 10:59:37]

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

[commit-bot: I haz the power, 2016-11-28 10:59:38]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-11-29 01:07:55]

yosin@chromium.org changed reviewers:
+ xiaochengh@chromium.org, yoichio@chromium.org

[yosin_UTC9, 2016-11-29 01:07:56]

PTAL

[Xiaocheng, 2016-11-29 01:53:16]

lgtm

[yoichio, 2016-11-29 01:54:50]

lgtm

[yosin_UTC9, 2016-11-29 02:05:18]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-11-29 02:06:44]

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-11-29 03:05:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480385118296100, "parent_rev":
"1a80eeec4ae07eb154398412ce87157a207ec650", "commit_rev":
"542f9e5c882fa1ae213a6c6f4e2297af4ac09ab6"}

[commit-bot: I haz the power, 2016-11-29 03:05:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 03:11:09]

Description was changed from

==========
Get rid of wrong assumption in reconcileTextDecorationProperties()

This patch gets rid of |DCHECK()|, which is a wrong assumption, in
|reconcileTextDecorationProperties()| since we have a test case against the
assumption.

BUG=663613
TEST=LayoutTests/editing/execCommand/insert-list-and-strikethrough.html
==========

to

==========
Get rid of wrong assumption in reconcileTextDecorationProperties()

This patch gets rid of |DCHECK()|, which is a wrong assumption, in
|reconcileTextDecorationProperties()| since we have a test case against the
assumption.

BUG=663613
TEST=LayoutTests/editing/execCommand/insert-list-and-strikethrough.html

Committed: https://crrev.com/a9814d2d1bbc0c1d77dad4f2619e21e28757b47f
Cr-Commit-Position: refs/heads/master@{#434857}
==========

[commit-bot: I haz the power, 2016-11-29 03:11:10]

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