Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Making the Oilpan references weak but the V8 wrapper references strong
is causing confusion at least, and bugs at worst.

Not clear this will fix the referenced bug, but should do no harm.

BUG=671791
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/aa5cbb1a6f3500536feec0c97acdd72e1c56dbb4
Cr-Commit-Position: refs/heads/master@{#437183}

[Ken Russell (switch to Gerrit), 2016-12-08 01:49:58]

Description was changed from

==========
Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Making the Oilpan references weak but the V8 wrapper references strong
is causing confusion at least, and bugs at worst.

Not clear this will fix the referenced bug, but should do no harm.

BUG=671791
==========

to

==========
Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Making the Oilpan references weak but the V8 wrapper references strong
is causing confusion at least, and bugs at worst.

Not clear this will fix the referenced bug, but should do no harm.

BUG=671791
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Ken Russell (switch to Gerrit), 2016-12-08 01:51:10]

kbr@chromium.org changed reviewers:
+ haraken@chromium.org, mlippautz@chromium.org

[Ken Russell (switch to Gerrit), 2016-12-08 01:51:11]

haraken or mlippautz: please review this and CQ it if it looks good. I'd like to
eliminate as many chances for errors as possible, as I don't know where the real
bug is here.

zmo: FYI.

[Ken Russell (switch to Gerrit), 2016-12-08 01:51:17]

The CQ bit was checked by kbr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-08 01:51:49]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2016-12-08 01:52:58]

LGTM

[Ken Russell (switch to Gerrit), 2016-12-08 01:54:34]

The CQ bit was unchecked by kbr@chromium.org

[Ken Russell (switch to Gerrit), 2016-12-08 01:54:37]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2016-12-08 01:54:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-08 05:00:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481162077278420, "parent_rev":
"fae9e19fd0e78da3afd2dcba3c194dd2cabc0f2b", "commit_rev":
"3deb9d90f84d1f17764068e2d541fecda98b7774"}

[commit-bot: I haz the power, 2016-12-08 05:01:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-08 05:04:30]

Description was changed from

==========
Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Making the Oilpan references weak but the V8 wrapper references strong
is causing confusion at least, and bugs at worst.

Not clear this will fix the referenced bug, but should do no harm.

BUG=671791
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Use strong references from WebGLContextGroup to WebGLRenderingContextBase.

Making the Oilpan references weak but the V8 wrapper references strong
is causing confusion at least, and bugs at worst.

Not clear this will fix the referenced bug, but should do no harm.

BUG=671791
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Committed: https://crrev.com/aa5cbb1a6f3500536feec0c97acdd72e1c56dbb4
Cr-Commit-Position: refs/heads/master@{#437183}
==========

[commit-bot: I haz the power, 2016-12-08 05:04:31]

Patchset 1 (id:??) landed as
https://crrev.com/aa5cbb1a6f3500536feec0c97acdd72e1c56dbb4
Cr-Commit-Position: refs/heads/master@{#437183}