Apply text indent if the text direction matches the text alignment

Apply text indent if the text direction matches the text alignment

We text-align:left; our <li> elements so this shows up on live-sites that attempt
a text indent on their the list element's content.

BUG=568851
Committed: https://crrev.com/d7ea97a4c74a4f420f8a646109f15f32f61c419e
Cr-Commit-Position: refs/heads/master@{#371665}

[rhogan, 2016-01-26 20:07:37]

Description was changed from

==========
Apply text indent if the text direction matches the text alignment

BUG=568851
==========

to

==========
Apply text indent if the text direction matches the text alignment

We text-align:left; our <li> elements so this shows up on live-sites that
attempt
a text indent on their the list element's content.

BUG=568851
==========

[rhogan, 2016-01-26 20:07:37]

robhogan@gmail.com changed reviewers:
+ eae@chromium.org, jchaffraix@chromium.org

[eae, 2016-01-26 20:45:20]

LGTM

The test seems a bit over complicated but does cover the change so it's fine.

Next time try to keep the test simpler, easier to debug and reason about it that
way :)

Thanks for fixing this!

[eae, 2016-01-26 20:45:28]

The CQ bit was checked by eae@chromium.org

[commit-bot: I haz the power, 2016-01-26 20:46:45]

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

[commit-bot: I haz the power, 2016-01-26 21:28:25]

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

[commit-bot: I haz the power, 2016-01-26 21:28:26]

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

[rhogan, 2016-01-26 22:01:39]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2016-01-26 22:03:49]

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

[commit-bot: I haz the power, 2016-01-27 00:52:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 00:53:29]

Description was changed from

==========
Apply text indent if the text direction matches the text alignment

We text-align:left; our <li> elements so this shows up on live-sites that
attempt
a text indent on their the list element's content.

BUG=568851
==========

to

==========
Apply text indent if the text direction matches the text alignment

We text-align:left; our <li> elements so this shows up on live-sites that
attempt
a text indent on their the list element's content.

BUG=568851

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

[commit-bot: I haz the power, 2016-01-27 00:53:30]

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