[sync] DCHECK from UTF16ToASCII from TemplateURLService

[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
Committed: https://crrev.com/122369ab8e771215d1afca236a20d16002c5000b
Cr-Commit-Position: refs/heads/master@{#396876}

[Patrick Noland, 2016-05-25 23:53:26]

Description was changed from

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
==========

to

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
==========

[Peter Kasting, 2016-05-27 00:04:57]

LGTM

[Patrick Noland, 2016-05-31 17:11:58]

The CQ bit was checked by pnoland@chromium.org

[commit-bot: I haz the power, 2016-05-31 17:12:21]

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

[commit-bot: I haz the power, 2016-05-31 18:43:41]

Description was changed from

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
==========

to

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
==========

[commit-bot: I haz the power, 2016-05-31 18:43:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-31 18:45:17]

Description was changed from

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082
==========

to

==========
[sync] DCHECK from UTF16ToASCII from TemplateURLService
LogDuplicatesHistogram erroneously assumes that all keywords are ASCII.
This CL removes that assumption and adds a test for the crash.

R=pkasting@chromium.org

BUG=613082

Committed: https://crrev.com/122369ab8e771215d1afca236a20d16002c5000b
Cr-Commit-Position: refs/heads/master@{#396876}
==========

[commit-bot: I haz the power, 2016-05-31 18:45:19]

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