Change V8ObjectVar to store a pointer to the instance rather than an ID

Change V8ObjectVar to store a pointer to the instance rather than an ID

This changes the currently unused V8ObjectVar to store a pointer to the
instance rather than a PP_Instance ID. This is just more convenient for
the code which uses the V8ObjectVar. When the instance is deleted, the
pointer will be cleared so it should always be valid or NULL.

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

[Sam McNally, 2014-08-12 01:44:05]

LGTM

https://codereview.chromium.org/464653003/diff/1/content/renderer/pepper/v8ob...
File content/renderer/pepper/v8object_var.h (right):

https://codereview.chromium.org/464653003/diff/1/content/renderer/pepper/v8ob...
content/renderer/pepper/v8object_var.h:47: // Possibly 0 if the object has
outlived its instance.
s/0/NULL/

[raymes, 2014-08-12 03:09:09]

https://codereview.chromium.org/464653003/diff/1/content/renderer/pepper/v8ob...
File content/renderer/pepper/v8object_var.h (right):

https://codereview.chromium.org/464653003/diff/1/content/renderer/pepper/v8ob...
content/renderer/pepper/v8object_var.h:47: // Possibly 0 if the object has
outlived its instance.
On 2014/08/12 01:44:05, Sam McNally wrote:
> s/0/NULL/

Done.

[raymes, 2014-08-12 03:09:13]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-08-12 03:12:11]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/raymes@chromium.org/464653003/20001

[commit-bot: I haz the power, 2014-08-12 05:30:11]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-12 08:08:16]

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

[commit-bot: I haz the power, 2014-08-12 08:08:17]

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

[raymes, 2014-08-13 00:20:42]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-08-13 00:46:39]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/raymes@chromium.org/464653003/20001

[commit-bot: I haz the power, 2014-08-13 03:47:33]

Change committed as 289191