Created attachment 409570 [details] [diff] [review] patch Rather than having everybody access browser.mIconURL, this passes the URL to onLinkIconAvailable and also adds a getIcon method, which complements setIcon.
Attachment #409570 - Flags: review?(gavin.sharp)
Comment on attachment 409570 [details] [diff] [review] patch s/var/let/ in new code. r=mano.
Attachment #409570 - Flags: review?(gavin.sharp) → review+
Why does getIcon take a tab? Don't most tabbrowser accessors take <browser>s? It's also easier to get from tab->browser than vice-versa, IIRC.
Some take a tab, some take a browser. setIcon specifically takes a tab.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
Verified fixed based on check-in and testing with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a1pre) Gecko/20091220 Minefield/3.7a1pre ID:20091220030635. Adding in-testsuite+ because we have a test here.
Status: RESOLVED → VERIFIED
Documented browser.getIcon here: https://developer.mozilla.org/en/XUL/Method/getIcon And while I was at it, noticed that setIcon() wasn't documented, so wrote it up too: https://developer.mozilla.org/en/XUL/Method/setIcon Also added to Firefox 4 for developers.
Keywords: dev-doc-needed → dev-doc-complete
We'll also need the extra argument for onLinkIconAvailable.
You need to log in before you can comment on or make changes to this bug.