Radha Kulkarni wrote:


> 3) With Global History we can provide an embeddor few options:
> Option #1:  The embeddor wants only link coloring and  is not interested 
> in our implementation of Global History. To achieve this, the embeddor 
> should maintain his own list of previously visited urls and we will ask 
> the embeddor thro' the following method in nsIBrowserChrome, if  it has 
> been previously visited and use the result to color the link.
> boolean IsURIVisited(nsIURI * aURI) ;

nsIWebBrowserChrome is per top-level-window and IsURIVisited() seems to 
be a global method to me. Did we just opt to stick it on 
nsIWebBrowserChrome so we didn't have to come up w/ a global iface w/ a 
single method on it that the embeddor provided?

Shouldn't IsURIVisited(nsIURI *aURI), just take a PRUnichar *? If it's a 
nsIURI, then the embeddor is required to create URI's and we should be 
minimizing their exposure to having to do that.

Jud



Reply via email to