Add explicit conversions methods for WebString <-> string16

Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131
Committed: https://crrev.com/d3054687cf8c2b90c25914065acaee0eb0a5ca28
Cr-Commit-Position: refs/heads/master@{#438801}

[kinuko, 2016-12-14 05:13:46]

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

[commit-bot: I haz the power, 2016-12-14 05:14:05]

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-12-14 07:13:17]

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

[commit-bot: I haz the power, 2016-12-14 07:13:18]

Dry run: This issue passed the CQ dry run.

[kinuko, 2016-12-15 06:37:47]

Description was changed from

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131
==========

to

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131
==========

[kinuko, 2016-12-15 06:37:48]

kinuko@chromium.org changed reviewers:
+ dcheng@chromium.org, esprehn@chromium.org, haraken@chromium.org

[kinuko, 2016-12-15 06:38:56]

Strip down the CL so that it only contains WebString changes (and one callsite
to exercise how it looks).

PTAL!!  Thanks.

[dcheng, 2016-12-15 06:52:35]

lgtm!

[haraken, 2016-12-15 07:28:37]

LGTM

[kinuko, 2016-12-15 08:16:28]

The CQ bit was checked by kinuko@chromium.org

[commit-bot: I haz the power, 2016-12-15 08:16: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-12-15 10:48:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481789788048620, "parent_rev":
"053b87af6502ee927e8fd99bda4840a3efa8d046", "commit_rev":
"748f23c066f62d429484a58eed0b32d219e7a58a"}

[commit-bot: I haz the power, 2016-12-15 10:49:09]

Description was changed from

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131
==========

to

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131

Review-Url: https://codereview.chromium.org/2571863003
==========

[commit-bot: I haz the power, 2016-12-15 10:49:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 10:51:11]

Description was changed from

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131

Review-Url: https://codereview.chromium.org/2571863003
==========

to

==========
Add explicit conversions methods for WebString <-> string16

Initial patch to remove implicit conversions.

This CL adds following utf16 conversion methods and use
them in blink_platform_impl.cc to show how it'd look like.

* static WebString WebString::fromUTF16(const string16&);
* static WebString WebString::fromUTF16(const NullableString16&);
* string16 WebString::utf16();
* static NullableString16 WebString::toNullableString16();

BUG=667131

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

[commit-bot: I haz the power, 2016-12-15 10:51:12]

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