[Binding] [Refactoring] Make SerializedScriptValueWriter protected

Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.

BUG=614919
Committed: https://crrev.com/6b136d3aa98f994b1d540d4dc95fdc2b3daee005
Cr-Commit-Position: refs/heads/master@{#396681}

[peria, 2016-05-26 08:58:20]

Description was changed from

==========
Remove a direct usage of SerializedScriptValueWriter

BUG=614919
==========

to

==========
Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.


BUG=614919
==========

[peria, 2016-05-26 09:03:08]

peria@chromium.org changed reviewers:
+ haraken@chromium.org, yukishiino@chromium.org

[peria, 2016-05-26 09:03:09]

PTL

[haraken, 2016-05-26 09:12:00]

LGTM

[peria, 2016-05-26 09:12:43]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-05-26 09:13:00]

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

[commit-bot: I haz the power, 2016-05-26 10:15:48]

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

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

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[peria, 2016-05-26 10:19:56]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-05-26 10:20:04]

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

[commit-bot: I haz the power, 2016-05-26 10:36:55]

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

[commit-bot: I haz the power, 2016-05-26 10:36:56]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[peria, 2016-05-26 10:37:34]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-05-26 10:37:43]

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

[commit-bot: I haz the power, 2016-05-26 10:42:02]

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

[commit-bot: I haz the power, 2016-05-26 10:42:03]

Try jobs failed on following builders:
  win8_chromium_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/...)

[peria, 2016-05-26 11:55:16]

PS1 was broken, because it uses an isolate with an empty context to create
ScriptValueSerializer.
PS2 is just a workaround. I'll redesign of creating SeraizedScriptValue
instances in another CL and update this CL.

[peria, 2016-05-30 04:02:04]

PTAL.

SerializedScriptValueFactory still uses writer.takeWireString(),
and removing it will be a bit complicated to do in one CL,
so let me do it in another CL.

[haraken, 2016-05-30 04:03:07]

LGTM

[peria, 2016-05-30 04:03:42]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2016-05-30 04:03:52]

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

[commit-bot: I haz the power, 2016-05-30 05:40:40]

Description was changed from

==========
Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.


BUG=614919
==========

to

==========
Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.


BUG=614919
==========

[commit-bot: I haz the power, 2016-05-30 05:40:41]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-30 05:41:35]

Description was changed from

==========
Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.


BUG=614919
==========

to

==========
Make most methods of SerializedScriptValueWriter protected.
We still have a usage of takeWireString(), and it will be
removed in another CL.

BUG=614919

Committed: https://crrev.com/6b136d3aa98f994b1d540d4dc95fdc2b3daee005
Cr-Commit-Position: refs/heads/master@{#396681}
==========

[commit-bot: I haz the power, 2016-05-30 05:41:36]

Patchset 4 (id:??) landed as
https://crrev.com/6b136d3aa98f994b1d540d4dc95fdc2b3daee005
Cr-Commit-Position: refs/heads/master@{#396681}