Convert RTF on the pasteboard to HTML and all it to substitute if present and...

Convert RTF on the pasteboard to HTML and all it to substitute if present and no HTML is on a paste or drop. Safari does not put HTML on the pasteboard, only RTF.
BUG=44661
TEST=dropping/pasting html should retain basic structure, but fonts/colors don't seem to propagate.

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

[Robert Sesek, 2010-06-04 17:12:59]

http://codereview.chromium.org/2674002/diff/1/2
File app/clipboard/clipboard_mac.mm (right):

http://codereview.chromium.org/2674002/diff/1/2#newcode1
app/clipboard/clipboard_mac.mm:1: // Copyright (c) 2006-2008 The Chromium
Authors. All rights reserved.
Make this just 2010

http://codereview.chromium.org/2674002/diff/1/2#newcode17
app/clipboard/clipboard_mac.mm:17: #import
"third_party/mozilla/NSPasteboard+Utils.h"
Will this pass checkdeps?

http://codereview.chromium.org/2674002/diff/1/3
File chrome/browser/cocoa/web_drop_target.mm (right):

http://codereview.chromium.org/2674002/diff/1/3#newcode1
chrome/browser/cocoa/web_drop_target.mm:1: // Copyright (c) 2009 The Chromium
Authors. All rights reserved.
2010

http://codereview.chromium.org/2674002/diff/1/3#newcode7
chrome/browser/cocoa/web_drop_target.mm:7: #include "base/scoped_nsobject.h"
Why was this added?

http://codereview.chromium.org/2674002/diff/1/5
File third_party/mozilla/NSPasteboard+Utils.mm (right):

http://codereview.chromium.org/2674002/diff/1/5#newcode291
third_party/mozilla/NSPasteboard+Utils.mm:291: [[NSAttributedString alloc]
This is leaked

[pink (ping after 24hrs), 2010-06-04 17:20:19]

all issues addressed. take another look.

http://codereview.chromium.org/2674002/diff/1/2
File app/clipboard/clipboard_mac.mm (right):

http://codereview.chromium.org/2674002/diff/1/2#newcode1
app/clipboard/clipboard_mac.mm:1: // Copyright (c) 2006-2008 The Chromium
Authors. All rights reserved.
On 2010/06/04 17:12:59, rsesek wrote:
> Make this just 2010

Done.

http://codereview.chromium.org/2674002/diff/1/2#newcode17
app/clipboard/clipboard_mac.mm:17: #import
"third_party/mozilla/NSPasteboard+Utils.h"
On 2010/06/04 17:12:59, rsesek wrote:
> Will this pass checkdeps?

It should. third_party isn't off-limits.

http://codereview.chromium.org/2674002/diff/1/3
File chrome/browser/cocoa/web_drop_target.mm (right):

http://codereview.chromium.org/2674002/diff/1/3#newcode1
chrome/browser/cocoa/web_drop_target.mm:1: // Copyright (c) 2009 The Chromium
Authors. All rights reserved.
On 2010/06/04 17:12:59, rsesek wrote:
> 2010

Done.

http://codereview.chromium.org/2674002/diff/1/3#newcode7
chrome/browser/cocoa/web_drop_target.mm:7: #include "base/scoped_nsobject.h"
On 2010/06/04 17:12:59, rsesek wrote:
> Why was this added?

for code that i moved to the category. removed.

http://codereview.chromium.org/2674002/diff/1/5
File third_party/mozilla/NSPasteboard+Utils.mm (right):

http://codereview.chromium.org/2674002/diff/1/5#newcode291
third_party/mozilla/NSPasteboard+Utils.mm:291: [[NSAttributedString alloc]
On 2010/06/04 17:12:59, rsesek wrote:
> This is leaked

Done.

[Robert Sesek, 2010-06-04 17:23:49]

LGTM