Brad Lassey [:blassey] (use needinfo?) Reporter
	Description • 16 years ago

error is:
Error: win.getWebNavigation is not a function
Source File: http://localhost:8888/redirect.js
Line: 51


the js that is failing is:
  var windowMediator = Cc['@mozilla.org/appshell/window-mediator;1'].
  getService(Ci.nsIWindowMediator);
  var win = windowMediator.getMostRecentWindow("navigator:browser");
  win.getWebNavigation().loadURI(aURL + location.search,
                                 null, null, null, null);

	:Gavin Sharp [email: gavin@gavinsharp.com] Assignee
	Comment 1 • 16 years ago

The easy fix would be to add the getWebNavigation() helper to fennec chrome (we'll probably end up doing that anyways for Firefox extension compat), but perhaps we should fix that mochitest code to use something more generic. I think the following would work:

let webNav = window.QueryInterface(Ci.nsIInterfaceRequestor)
                   .getInterface(Ci.nsIWebNavigation)
                   .QueryInterface(Ci.nsIDocShellTreeItem)
                   .treeOwner.primaryContentShell
                   .QueryInterface(Ci.nsIWebNavigation);

	Brad Lassey [:blassey] (use needinfo?) Reporter
	Comment 2 • 16 years ago

for some reason using let doesn't work.

	Boris Zbarsky [:bzbarsky]
	Comment 3 • 16 years ago

Comment on attachment 338635 [details] [diff] [review]
gavin's code as a patch

Shouldn't that be |win| instead of |window|?  Or do you really want to run it againstd the current window as opposed to the most recent navigator window?

let probably didn't work because this stuff is not being loaded right; that's worth filing a bug on.

	:Gavin Sharp [email: gavin@gavinsharp.com] Assignee
	Comment 4 • 16 years ago

Yeah, it should be win.

	:Gavin Sharp [email: gavin@gavinsharp.com] Assignee
	Comment 5 • 16 years ago

Actually, this file is loaded in the content docshell anyways, so we can just use window.QueryInterface(Ci.nsIInterfaceRequestor).getInterface(Ci.nsIWebNavigation) directly. Or maybe even just set document.location? I guess that might run into security checks.

	:Gavin Sharp [email: gavin@gavinsharp.com] Assignee
	Comment 7 • 16 years ago

I'll add this comment:
  // Can't just set window.location because of security restrictions
  // that don't care about our UniversalXPConnectness

	:Gavin Sharp [email: gavin@gavinsharp.com] Assignee
	Comment 8 • 16 years ago

http://hg.mozilla.org/mozilla-central/rev/5486e202bbc6