Unconditionally include <string> and <iostream> in json_reader.cpp

Unconditionally include <string> and <iostream> in json_reader.cpp
Non-PNaCl builds that use libc++ also require these headers.

BUG=None
R=mallinath@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271429

[Alexander Potapenko, 2014-05-19 09:00:05]

PTAL

[pbos, 2014-05-19 09:24:43]

On 2014/05/19 09:00:05, Alexander Potapenko wrote:
> PTAL

drive-by lgtm. mallinath@ for context this un-breaks Debug tsanv2 builds of
peerconnection_client in webrtc which it looks like no one has been building.

[Mallinath (Gone from Chromium), 2014-05-19 15:13:29]

lgtm

[Alexander Potapenko, 2014-05-19 15:46:48]

The CQ bit was checked by glider@chromium.org

[commit-bot: I haz the power, 2014-05-19 15:46:54]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/glider@chromium.org/291803003/1

[commit-bot: I haz the power, 2014-05-19 18:24:22]

Change committed as 271429

[Nico, 2014-05-19 19:05:10]

A revert of this CL has been created in
https://codereview.chromium.org/290923003/ by thakis@chromium.org.

The reason for reverting is: This adds a static initializer..

[Nico, 2014-05-19 19:23:04]

https://codereview.chromium.org/291803003/diff/1/third_party/jsoncpp/override...
File third_party/jsoncpp/overrides/src/lib_json/json_reader.cpp (right):

https://codereview.chromium.org/291803003/diff/1/third_party/jsoncpp/override...
third_party/jsoncpp/overrides/src/lib_json/json_reader.cpp:21: // but defines it
in <iostream>. The code therefore fails linking, which
As iostream causes static initializers, that seems like a bug in libc++. It
should be defined in some other stream header probably.