Peter Van der Beken [:peterv] Assignee
	Description • 11 years ago

If we stop using a permanent window.document property by instead relying on WebIDL for that we need to stop nulling out mDoc in nsGlobalWindow::FreeInnerObjects because otherwise window.document becomes null. Turns out we rely on mDoc being null to avoid firing certain events in nsGlobalWindow. I've fixed that by doing a check for "is current inner window" instead. I though about doing the check in a more central place like DispatchEvent, but then the callers would still be doing a bunch of work (creating events etc) for all non-current inner windows, just to throw it all away again. So I've opted to assert in DispatchEvent instead. Let me know how you feel about that.

	Peter Van der Beken [:peterv] Assignee
	Comment 1 • 11 years ago

The GamepadServiceTest fix is just to avoid a crash when running some of the gamepad tests on their own, which happened while I was debugging this patch.

	Peter Van der Beken [:peterv] Assignee
	Comment 2 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/a6878c4ee086

	Ed Morley [:emorley]
	Comment 3 • 11 years ago

Backed out for assertions, eg:
https://tbpl.mozilla.org/php/getParsedLog.php?id=29961392&tree=Mozilla-Inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/2137bf09cdc1

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 5 • 11 years ago

Comment on attachment 825986 [details] [diff] [review]
Additional fixes v1


> #ifdef MOZ_B2G
>   if (!nsCRT::strcmp(aTopic, NS_NETWORK_ACTIVITY_BLIP_UPLOAD_TOPIC) ||
>       !nsCRT::strcmp(aTopic, NS_NETWORK_ACTIVITY_BLIP_DOWNLOAD_TOPIC)) {
>+    if (!IsInnerWindow() || !IsCurrentInnerWindow()) {
>+      return NS_OK;
>+    }

MOZ_ASSERT(IsInnerWindow());
if (!IsCurrentInnerWindow()) {
  return NS_OK;
}

	Peter Van der Beken [:peterv] Assignee
	Comment 6 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/0e3bcb78784e
https://hg.mozilla.org/integration/mozilla-inbound/rev/2b99a0f6ff1b

	Ryan VanderMeulen [:RyanVM]
	Comment 7 • 11 years ago

Backed out for making Linux32 opt mochitest-bc perma-orange.
https://hg.mozilla.org/integration/mozilla-inbound/rev/08d4847dfd5c

https://tbpl.mozilla.org/php/getParsedLog.php?id=30087228&tree=Mozilla-Inbound

	Peter Van der Beken [:peterv] Assignee
	Comment 8 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/e30f630191b9
https://hg.mozilla.org/integration/mozilla-inbound/rev/d4fdcf5f3c04

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 11 years ago

https://hg.mozilla.org/mozilla-central/rev/e30f630191b9
https://hg.mozilla.org/mozilla-central/rev/d4fdcf5f3c04

	Gregor Wagner [:gwagner]
	Comment 11 • 11 years ago

Comment on attachment 828313 [details] [diff] [review]
better debug message

Review of attachment 828313 [details] [diff] [review]:
-----------------------------------------------------------------

I don't know why, but this gives us a stack as well.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 12 • 11 years ago

I need to null check event.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 13 • 11 years ago

The method throws if null is passed.

	Boris Zbarsky [:bzbarsky]
	Comment 14 • 11 years ago

Peter, how reasonable is it to backport this to beta 27?  The fix for bug 936056 basically depends on this patch...

	Peter Van der Beken [:peterv] Assignee
	Comment 15 • 11 years ago

Bz is going to try to backport bug 936056 to branch without this fix, because it's a bit scary to backport this one.