544452 - nsIGlobalHistory2::isVisited() should know the origin of the document

Status: RESOLVED WORKSFORME

(Core :: CSS Parsing and Computation, defect)

Description

[Giorgio Maone [:ma1]]

nsIGlobalHistory2::isVisited() should be changed to
take a second (optional) "referrer" URI parameter, and the call from
nsDocShell::GetLinkState() should pass it currentURI.

This way an extension overriding the global history service could easily
implement a SafeHistory-like policy, mitigating bug 147777 (per comment https://bugzilla.mozilla.org/show_bug.cgi?id=147777#c157 ) with limited usability impact.

Comment 1

[Boris Zbarsky [:bzbarsky]]

We could expose the principal of the Link on the Link if we wanted to in the IHistory world, right?

Comment 2

[Shawn Wilsher :sdwilsh]

(In reply to comment #1)
> We could expose the principal of the Link on the Link if we wanted to in the
> IHistory world, right?
Or on nsIContent (since everything that implements Link should be QIable to nsIContent).

Regardless, as requested, this bug is wontfix, right?

Comment 3

[Boris Zbarsky [:bzbarsky]]

I'm just as happy if extension history impls don't have to pull in nsIContent.

And I have no problem retitling this bug as needed.

Comment 4

[Andrew McCreight [:mccr8]]

nsIGlobalHistory doesn't exist.