[DevTools] Migrate v8_inspector/public from String16 to String{View,Buffer}.

third_party/WebKit/Source/core/inspector/V8InspectorStringView.cpp

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "core/inspector/V8InspectorStringView.h"
+
+#include <cstring>
+
+namespace blink {
+
+v8_inspector::StringView toV8InspectorStringView(const String& string)
+{
+    if (string.isEmpty())
+        return v8_inspector::StringView();

[esprehn, 2016/08/19 18:53:12]

this is converting empty to null, those are two different things inside blink.

[dgozman, 2016/08/19 20:50:12]

Fixed.

+    if (string.is8Bit())
+        return v8_inspector::StringView(reinterpret_cast<const uint8_t*>(string.characters8()), string.length());
+    return v8_inspector::StringView(reinterpret_cast<const uint16_t*>(string.characters16()), string.length());
+}
+
+v8_inspector::StringView toV8InspectorStringView(const String16& string)
+{
+    if (string.isEmpty())
+        return v8_inspector::StringView();
+    return v8_inspector::StringView(reinterpret_cast<const uint16_t*>(string.characters16()), string.length());
+}
+
+v8_inspector::StringView toV8InspectorStringView(const char* string)
+{
+    return v8_inspector::StringView(reinterpret_cast<const uint8_t*>(string), strlen(string));

[esprehn, 2016/08/19 18:53:12]

You should either take a WTF::StringView or make this inline. Then the compiler
will figure out the length of the string for you.

[dgozman, 2016/08/19 20:50:12]

Done.

+}
+
+String toCoreString(const v8_inspector::StringView& string)
+{
+    if (!string.length())

[esprehn, 2016/08/19 18:53:12]

This converts zero length strings to empty strings. That means round tripping a
null string comes out empty not null.

toV8InspectorStringView(String()) => emptyString()

[dgozman, 2016/08/19 20:50:12]

Fixed.

+        return emptyString();
+    if (string.is8Bit())
+        return String(reinterpret_cast<const LChar*>(string.characters8()), string.length());
+    return String(reinterpret_cast<const UChar*>(string.characters16()), string.length());
+}
+
+} // namespace blink