Only recurse to depth 10 in v8_value_converter_impl.cc.

Recurse to a maximum depth of 10 in v8_value_converter_impl.cc. There are 
objects that get passed in (canonically <input> elements apparently) which
recurse infinitely (as opposed to having a self-referential loop).

The previous solution to this problem (r150035) was to disable getters, which
apparently were the main cause, but this is no longer appropriate - we now use 
this mechanism for all extension messaging, and this has become a problem (see
bug 246213).

TBR=jamesr@chromium.org,
R=mpcomplete@chromium.org
BUG=246213, 139933
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=204057

[Matt Perry, 2013-06-03 23:15:38]

lgtm

https://codereview.chromium.org/16295013/diff/3010/content/renderer/v8_value_...
File content/renderer/v8_value_converter_impl.cc (right):

https://codereview.chromium.org/16295013/diff/3010/content/renderer/v8_value_...
content/renderer/v8_value_converter_impl.cc:64:
unique_map_.insert(std::pair<int, v8::Handle<v8::Object> >(hash, handle));
why not unique_map_[hash] = handle?
or make_pair instead of std::pair<>

[not at google - send to devlin, 2013-06-04 00:15:05]

https://codereview.chromium.org/16295013/diff/3010/content/renderer/v8_value_...
File content/renderer/v8_value_converter_impl.cc (right):

https://codereview.chromium.org/16295013/diff/3010/content/renderer/v8_value_...
content/renderer/v8_value_converter_impl.cc:64:
unique_map_.insert(std::pair<int, v8::Handle<v8::Object> >(hash, handle));
On 2013/06/03 23:15:38, Matt Perry wrote:
> why not unique_map_[hash] = handle?
> or make_pair instead of std::pair<>

Changed to make_pair, nice cleanup, I don't want to think too hard about making
much more than a super trivial change here since I didn't write the code.

[not at google - send to devlin, 2013-06-04 18:53:30]

ping - I know it hasn't been that long, but this is really blocking me.

[commit-bot: I haz the power, 2013-06-04 19:24:22]

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

[jamesr, 2013-06-04 19:27:18]

I don't really understand this code, but if Matt's happy then lgtm

[not at google - send to devlin, 2013-06-04 22:01:39]

Committed patchset #11 manually as r204057 (presubmit successful).