Add an ostream operator to Value (based on JSONWriter) so that assertions

Add an ostream operator to Value (based on JSONWriter) so that assertions
involving Value::Equals are much easier to understand. For example,

ListValue list;
DictionaryValue dict;
ASSERT_TRUE(Value::Equals(&list, &dict)) << list << "does not equal\n" << dict;

will now print

Value of: Value::Equals(&list, &dict)
  Actual: false
Expected: true
[  ]
does not equal
{

}


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=156226

[willchan no longer on Chromium, 2012-09-12 01:20:13]

lgtm

https://chromiumcodereview.appspot.com/10915194/diff/2001/base/values.cc
File base/values.cc (right):

https://chromiumcodereview.appspot.com/10915194/diff/2001/base/values.cc#newc...
base/values.cc:1064: base::JSONWriter::WriteWithOptions(&value,
is the base:: here necessary? We're in the base namespace already...

[not at google - send to devlin, 2012-09-12 01:21:48]

https://chromiumcodereview.appspot.com/10915194/diff/2001/base/values.cc
File base/values.cc (right):

https://chromiumcodereview.appspot.com/10915194/diff/2001/base/values.cc#newc...
base/values.cc:1064: base::JSONWriter::WriteWithOptions(&value,
On 2012/09/12 01:20:13, willchan wrote:
> is the base:: here necessary? We're in the base namespace already...

Done.

[commit-bot: I haz the power, 2012-09-12 01:22:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kalman@chromium.org/10915194/5001

[commit-bot: I haz the power, 2012-09-12 03:39:37]

Change committed as 156226