User Agent: Mozilla/5.0 (Windows NT 6.1; rv:8.0a1) Gecko/20110729 Firefox/8.0a1
Build ID: 20110729030824
Steps to reproduce:
1. Open http://www.mozilla.com/en-US/firefox/all.html in a tab.
2. Press and hold CTRL, then drag this tab anywhere into bookmarks.
A new bookmark is created with a title "all.html" (from webpage file name).
New bookmark's title should be "Firefox web browser…etc" (from webpage title), as it was before new CTRL+drag bookmarking was implemented.
Created attachment 549470 [details] [diff] [review]
Comment on attachment 549470 [details] [diff] [review]
Better to use browser.contentTitle, since tab.label can contain other cruft.
Also, can you do whatever it takes to fix creating a link on the desktop or in a file manager? I guess the text/html flavor does this. See also gIdentityHandler.onDragStart.
Created attachment 549501 [details] [diff] [review]
(In reply to comment #2)
> Also, can you do whatever it takes to fix creating a link on the desktop or
> in a file manager? I guess the text/html flavor does this.
No, text/x-moz-url does this too. I tested dragging to the desktop and Windows Explorer on Windows and to the desktop and Finder on OS X.
Pinning a titled shortcut to the start menu on Windows 7 only works in IE9 though, because they use their proprietary .website-extension-associated type. I'm not sure what the mime type for that is.
Is there a reason why one would use this.tabbrowser.getBrowserForTab(tab) instead of tab.linkedBrowser? AFAICT, the former simply returns the latter.
Oh, we currently can't handle those IE9 shortcuts anyway, so until we fix bug 605222 and bug 624070, it doesn't make sense to include that type.
Pushed to fx-team. Will flip status when merged to m-c.
Verified using Mozilla/5.0 (Windows NT 6.1; rv:8.0) Gecko/20100101 Firefox/8.0