Fix serialization references to Cell objects.

Fix serialization references to Cell objects.

R=yangguo@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=23484

[Slava Chigrin, 2014-08-27 17:46:05]

vchigrin@yandex-team.ru changed reviewers:
+ yangguo@chromium.org

[Slava Chigrin, 2014-08-27 17:47:13]

https://codereview.chromium.org/514783002/diff/1/src/serialize.cc
File src/serialize.cc (right):

https://codereview.chromium.org/514783002/diff/1/src/serialize.cc#newcode1640
src/serialize.cc:1640: }
Seems, this increment was missed, so pointers go out-of-sync during restorinng
process.

[Yang, 2014-08-28 09:54:06]

On 2014/08/27 17:47:13, Slava Chigrin wrote:
> https://codereview.chromium.org/514783002/diff/1/src/serialize.cc
> File src/serialize.cc (right):
> 
> https://codereview.chromium.org/514783002/diff/1/src/serialize.cc#newcode1640
> src/serialize.cc:1640: }
> Seems, this increment was missed, so pointers go out-of-sync during restorinng
> process.

lgtm.

[Slava Chigrin, 2014-08-28 09:57:08]

The CQ bit was checked by vchigrin@yandex-team.ru

[commit-bot: I haz the power, 2014-08-28 09:57:26]

CQ is trying da patch. Follow status at
 https://v8-status.appspot.com/cq/vchigrin@yandex-team.ru/514783002/1

[commit-bot: I haz the power, 2014-08-28 09:57:27]

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

[commit-bot: I haz the power, 2014-08-28 09:57:28]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Yang, 2014-08-28 10:30:17]

On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer or
> a provisional committer, _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

I'll have to land this for you. I'm curious, is there any test case that would
expose this bug?

[Slava Chigrin, 2014-08-28 10:38:09]

On 2014/08/28 10:30:17, Yang wrote:
> On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
> > No LGTM from a valid reviewer yet. Only full committers are accepted.
> > Even if an LGTM may have been provided, it was from a non-committer or
> > a provisional committer, _not_ a full super star committer.
> > See http://www.chromium.org/getting-involved/become-a-committer
> > Note that this has nothing to do with OWNERS files.
> 
> I'll have to land this for you. I'm curious, is there any test case that would
> expose this bug?

Thank you so much! Sorry, I can not quickly compose test case - it related to
some non-typical serializer usage (unfortunatelly, I can not disclose details
due to NDA).

[Yang, 2014-08-28 10:39:12]

On 2014/08/28 10:38:09, Slava Chigrin wrote:
> On 2014/08/28 10:30:17, Yang wrote:
> > On 2014/08/28 09:57:28, I haz the power (commit-bot) wrote:
> > > No LGTM from a valid reviewer yet. Only full committers are accepted.
> > > Even if an LGTM may have been provided, it was from a non-committer or
> > > a provisional committer, _not_ a full super star committer.
> > > See http://www.chromium.org/getting-involved/become-a-committer
> > > Note that this has nothing to do with OWNERS files.
> > 
> > I'll have to land this for you. I'm curious, is there any test case that
would
> > expose this bug?
> 
> Thank you so much! Sorry, I can not quickly compose test case - it related to
> some non-typical serializer usage (unfortunatelly, I can not disclose details
> due to NDA).

I see.

[Yang, 2014-08-28 10:51:51]

Committed patchset #1 (id:1) manually as 23484 (presubmit successful).