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

      No description provided.

	Boris Zbarsky [:bzbarsky]
	Comment 2 • 10 years ago

Comment on attachment 8402817 [details] [diff] [review]
v1.1

>@@ -2335,66 +2320,65 @@ nsGlobalWindow::SetNewDocument(nsIDocume
>     if (aState) {
>-      mInnerWindowHolder = wsh->GetInnerWindowHolder();

Why is that line being removed?

The rest looks fine, though the crash-on-touch bits seem like dead code?  Maybe a followup to nix it?

r=me with the above removal fixed or explained.

	Peter Van der Beken [:peterv] Assignee
	Comment 3 • 10 years ago

(In reply to Boris Zbarsky [:bz] from comment #2)
> >-      mInnerWindowHolder = wsh->GetInnerWindowHolder();
> 
> Why is that line being removed?

Yeah, looks iffy. It didn't seem to cause problems but I've moved it into the patch that actually removes mInnerWindowHolder.

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

	Ryan VanderMeulen [:RyanVM]
	Comment 4 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/f1e4f0f82a9a

	Carsten Book [:Tomcat]
	Comment 5 • 10 years ago

https://hg.mozilla.org/mozilla-central/rev/874d97918b82