I'm going to be refactoring some of our window startup code while I implement the home tab and some corresponding new startup logic, so I figured it would be a good time to clean up some of the cruft in the startup code. This pref hasn't been accessible from the UI for a long time (I don't know when it was), and lastPageVisited is a synchronous API only used by this pref, so I think we'd be okay killing it.

I'm fine with killing the API, but just for information, are there many add-ons using it? If it ends up being needed we may want to add an asynchronous replacement.

As discussed on IRC, only one add-on [1] uses this API. It only supports up to 3.6a1pre, so I don't think we need to worry about breaking it. [1] https://addons.mozilla.org/en-US/firefox/addon/firetray-10702/

Oops, forgot to save some changes before exporting patch.

Comment on attachment 564375 [details] [diff] [review] patch Review of attachment 564375 [details] [diff] [review]: ----------------------------------------------------------------- This looks good to me. It still needs a SR for the API change before landing, gavin is already aware of this so going through him.

Docs updated: https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIBrowserHistory Firefox 10 for developers updated.