Allow dragging text/url content out of the browser. Dragging back in not yet ...

Allow dragging text/url content out of the browser. Dragging back in not yet implemented.
BUG=13155
TEST=dragging urls and selected text to other apps, the desktop, or the Dock should work.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=20619

[stuartmorgan, 2009-07-13 21:43:32]

LGTM, just small nits.

http://codereview.chromium.org/149565/diff/1003/29
File chrome/browser/tab_contents/tab_contents_view_mac.mm (right):

http://codereview.chromium.org/149565/diff/1003/29#newcode85
Line 85: // populated in decending order of richness.
Isn't HTML richer than a URL?

http://codereview.chromium.org/149565/diff/1003/29#newcode143
Line 143: slideBack:YES];
Isn't our style either one line, or one argument per line aligned at the :?

http://codereview.chromium.org/149565/diff/1003/29#newcode373
Line 373: 
Why is there an empty and TODO-less method?

http://codereview.chromium.org/149565/diff/1003/29#newcode381
Line 381: // TODO(pinkerton): Force drops to be rejected until that's hooked up.
Seems like this is describing the current behavior, so it's confusing as a TODO;
replace with a description of what should be done eventually.

[pink (ping after 24hrs), 2009-07-13 21:57:56]

http://codereview.chromium.org/149565/diff/1003/29
File chrome/browser/tab_contents/tab_contents_view_mac.mm (right):

http://codereview.chromium.org/149565/diff/1003/29#newcode85
Line 85: // populated in decending order of richness.
On 2009/07/13 21:43:32, stuartmorgan wrote:
> Isn't HTML richer than a URL?

Done.

http://codereview.chromium.org/149565/diff/1003/29#newcode143
Line 143: slideBack:YES];
On 2009/07/13 21:43:32, stuartmorgan wrote:
> Isn't our style either one line, or one argument per line aligned at the :?

Done.

http://codereview.chromium.org/149565/diff/1003/29#newcode373
Line 373: 
On 2009/07/13 21:43:32, stuartmorgan wrote:
> Why is there an empty and TODO-less method?

Done.

http://codereview.chromium.org/149565/diff/1003/29#newcode381
Line 381: // TODO(pinkerton): Force drops to be rejected until that's hooked up.
On 2009/07/13 21:43:32, stuartmorgan wrote:
> Seems like this is describing the current behavior, so it's confusing as a
TODO;
> replace with a description of what should be done eventually.

Done.