[oilpan]: Don't assert the context is destroyed _after_ the NewWebSocketChannelImpl in oilpan.

[oilpan]: Don't assert the context is destroyed _after_ the NewWebSocketChannelImpl in oilpan.

This should fix a couple of the crashes we are seeing in the debug oilpan builds for websockets.

R=ager@chromium.org, erik.corry@gmail.com, haraken@chromium.org, oilpan-reviews@chromium.org, tkent@chromium.org, tyoshino@chromium.org, vegorov@chromium.org, yhirano@chromium.org, zerny@chromium.org
BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=178379

[Mads Ager (chromium), 2014-07-17 13:36:47]

LGTM, but Haraken should have a look as well.

[sof, 2014-07-17 13:39:30]

https://codereview.chromium.org/405453002/diff/1/Source/modules/websockets/Ne...
File Source/modules/websockets/NewWebSocketChannelImpl.h (left):

https://codereview.chromium.org/405453002/diff/1/Source/modules/websockets/Ne...
Source/modules/websockets/NewWebSocketChannelImpl.h:152: // This object must be
destroyed before the context.
surface nit: move this comment down into the #else part?

[wibling-chromium, 2014-07-17 14:16:49]

Thanks for the reviews! I have updated the diff.

https://codereview.chromium.org/405453002/diff/1/Source/modules/websockets/Ne...
File Source/modules/websockets/NewWebSocketChannelImpl.h (left):

https://codereview.chromium.org/405453002/diff/1/Source/modules/websockets/Ne...
Source/modules/websockets/NewWebSocketChannelImpl.h:152: // This object must be
destroyed before the context.
On 2014/07/17 13:39:30, sof wrote:
> surface nit: move this comment down into the #else part?

Done.

[haraken, 2014-07-17 14:35:45]

Thanks for the fix, LGTM.

[wibling-chromium, 2014-07-17 14:46:30]

The CQ bit was checked by wibling@chromium.org

[commit-bot: I haz the power, 2014-07-17 14:47:02]

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

[commit-bot: I haz the power, 2014-07-17 15:50:47]

Change committed as 178379

[tyoshino (SeeGerritForStatus), 2014-07-18 12:00:08]

lgtm