Judson Valeski wrote:

> 
> 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?

DocShell is the ultimate client of this method and since that is going 
to be per-window, we thought it could go in to nsIWebBrowserChrome.  
Either way it doesn't matter.

> 
> 
> 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.

I think PRUnichar should suffice.

> 


Reply via email to