Make sure the test holds a reference of secondDataChannel.

Make sure the test holds a reference of secondDataChannel.

BUG=467097
Committed: https://crrev.com/194c77793f7158729c91cbbefd6f813ba1ccf203
Cr-Commit-Position: refs/heads/master@{#321155}

[jiayl, 2015-03-13 22:21:45]

jiayl@chromium.org changed reviewers:
+ tommi@chromium.org

[jiayl, 2015-03-13 22:21:45]

PTAL

[rmcilroy, 2015-03-18 10:53:01]

On 2015/03/13 22:21:45, jiayl wrote:
> PTAL

This works for me and fixes the issues I was seeing with crbug.com/467655 when
landing long idle times.  Any chance this could be landed soon? I would really
like to land https://codereview.chromium.org/1011103004/ which is currently
blocked on this.

lgtm.

[jiayl, 2015-03-18 16:10:53]

The CQ bit was checked by jiayl@chromium.org

[commit-bot: I haz the power, 2015-03-18 16:11:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1005243002/1

[commit-bot: I haz the power, 2015-03-18 17:14:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-18 17:16:13]

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

[tommi (sloooow) - chröme, 2015-03-20 13:10:49]

tommi@chromium.org changed reviewers:
+ rmcilroy@chromium.org

[tommi (sloooow) - chröme, 2015-03-20 13:10:49]

lgtm (sorry about slowness).

Isn't there still though an issue in blink wrt gc-ing data channels?

[jiayl, 2015-03-20 16:11:31]

The blink issue is tracked separately.

On Fri, Mar 20, 2015 at 6:10 AM, <tommi@chromium.org> wrote:

> lgtm (sorry about slowness).
>
> Isn't there still though an issue in blink wrt gc-ing data channels?
>
> https://codereview.chromium.org/1005243002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.