Apply first-line transform-text style

Apply first-line transform-text style

A second go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

BUG=129669, 644733
Committed: https://crrev.com/25c2cadce2d4a92f4f0f8b669fed2d097a6b7afa
Cr-Commit-Position: refs/heads/master@{#418032}

[rhogan, 2016-09-11 15:17:35]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-11 15:17:43]

Dry run: 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-09-11 19:15:16]

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

[commit-bot: I haz the power, 2016-09-11 19:15:17]

Dry run: This issue passed the CQ dry run.

[rhogan, 2016-09-12 18:43:32]

Description was changed from

==========
Apply first-line transform-text style

A second go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

BUG=129669
==========

to

==========
Apply first-line transform-text style

A second go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

BUG=129669,644733
==========

[rhogan, 2016-09-12 18:43:32]

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

[rhogan, 2016-09-12 18:57:13]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-12 18:57:43]

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

[rhogan, 2016-09-12 18:58:25]

https://codereview.chromium.org/2331793002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutText.cpp (right):

https://codereview.chromium.org/2331793002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutText.cpp:1429: m_text.replace(from,
len, textToCapitalize);
A couple of things may have been the source of the crash in 644733. However I
haven't been able to recreate the crash so have just added a couple of defensive
measures here.

[rhogan, 2016-09-12 18:59:17]

The CQ bit was unchecked by robhogan@gmail.com

[rhogan, 2016-09-12 19:00:47]

The CQ bit was checked by robhogan@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2016-09-12 19:01:44]

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

[eae, 2016-09-12 19:22:30]

OK, LGTM but let's keep an eye on the crash rate.

Thank you so much for sticking with this!

[commit-bot: I haz the power, 2016-09-12 20:35:46]

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

[commit-bot: I haz the power, 2016-09-12 20:35:47]

Dry run: This issue passed the CQ dry run.

[rhogan, 2016-09-12 20:37:20]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2016-09-12 20:37:57]

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-09-12 20:42:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-12 20:44:39]

Description was changed from

==========
Apply first-line transform-text style

A second go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

BUG=129669,644733
==========

to

==========
Apply first-line transform-text style

A second go at https://crrev.com/3c64df1fc98aa06eabfc18d1f5c2f2b0aec1a658

BUG=129669,644733

Committed: https://crrev.com/25c2cadce2d4a92f4f0f8b669fed2d097a6b7afa
Cr-Commit-Position: refs/heads/master@{#418032}
==========

[commit-bot: I haz the power, 2016-09-12 20:44:40]

Patchset 3 (id:??) landed as
https://crrev.com/25c2cadce2d4a92f4f0f8b669fed2d097a6b7afa
Cr-Commit-Position: refs/heads/master@{#418032}

[rhogan, 2016-09-13 21:27:07]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2337133004/ by robhogan@gmail.com.

The reason for reverting is: Still hitting clusterfuzz..