Remove SerializedScriptValue::serialize and ScriptValueSerializer::serializeWTFString.

Remove SerializedScriptValue::serialize and ScriptValueSerializer::serializeWTFString.

To do so, make WebFrameTest serialize inefficiently (but it's just a unit test)
to call postMessage, and have EventSource construct a MessageEvent with a string,
rather than serialized script value, as its data.

BUG=690132
Review-Url: https://codereview.chromium.org/2686503005
Cr-Commit-Position: refs/heads/master@{#449241}
Committed: https://chromium.googlesource.com/chromium/src/+/4f94559a234a15ed12c9244ffee47279f5dca8a5

[jbroman, 2017-02-08 20:14:22]

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

[commit-bot: I haz the power, 2017-02-08 20:14:58]

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

[jbroman, 2017-02-08 20:49:46]

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

[commit-bot: I haz the power, 2017-02-08 20:50:02]

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

[jbroman, 2017-02-08 22:15:48]

jbroman@chromium.org changed reviewers:
+ haraken@chromium.org

[commit-bot: I haz the power, 2017-02-08 23:20:03]

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

[commit-bot: I haz the power, 2017-02-08 23:20:05]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-02-09 01:04:20]

LGTM

[jbroman, 2017-02-09 01:07:02]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2017-02-09 01:07:28]

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, 2017-02-09 01:14:04]

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

[commit-bot: I haz the power, 2017-02-09 01:14:05]

Failed to apply patch for
third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp:
While running git apply --index -p1;
  error: patch failed:
third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp:802
  error: third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp:
patch does not apply

Patch:      
third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
Index: third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
diff --git
a/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
b/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
index
38145b011abe4555b194bcd767aeec34dd05e7ca..95d8a18bfcb67a9d0d6f088e117a33acd76dd50c
100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
@@ -802,13 +802,6 @@ void ScriptValueSerializer::transferData(
 }
 
 // static
-String ScriptValueSerializer::serializeWTFString(const String& data) {
-  SerializedScriptValueWriter valueWriter;
-  valueWriter.writeWebCoreString(data);
-  return valueWriter.takeWireString();
-}
-
-// static
 String ScriptValueSerializer::serializeNullValue() {
   SerializedScriptValueWriter valueWriter;
   valueWriter.writeNull();

[jbroman, 2017-02-09 02:39:42]

The CQ bit was checked by jbroman@chromium.org

[jbroman, 2017-02-09 02:39:42]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2686503005/#ps40001
(title: "Merge branch 'master' into delete-ssv")

[commit-bot: I haz the power, 2017-02-09 02:40:25]

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, 2017-02-09 08:43:50]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486607982240730,
"parent_rev": "9785b29a0942cee4da9f079fd36312efad171167", "commit_rev":
"4f94559a234a15ed12c9244ffee47279f5dca8a5"}

[commit-bot: I haz the power, 2017-02-09 08:44:29]

Description was changed from

==========
Remove SerializedScriptValue::serialize and
ScriptValueSerializer::serializeWTFString.

To do so, make WebFrameTest serialize inefficiently (but it's just a unit test)
to call postMessage, and have EventSource construct a MessageEvent with a
string,
rather than serialized script value, as its data.

BUG=690132
==========

to

==========
Remove SerializedScriptValue::serialize and
ScriptValueSerializer::serializeWTFString.

To do so, make WebFrameTest serialize inefficiently (but it's just a unit test)
to call postMessage, and have EventSource construct a MessageEvent with a
string,
rather than serialized script value, as its data.

BUG=690132

Review-Url: https://codereview.chromium.org/2686503005
Cr-Commit-Position: refs/heads/master@{#449241}
Committed:
https://chromium.googlesource.com/chromium/src/+/4f94559a234a15ed12c9244ffee4...
==========

[commit-bot: I haz the power, 2017-02-09 08:44:30]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4f94559a234a15ed12c9244ffee4...