[Mac] Fix regression with copying URLs from the Omnibox.

[Mac] Fix regression with copying URLs from the Omnibox.

The function ClipboardUtil::PasteboardItemFromUrl was incorrectly attempting to
create a public UTI using UTIFromPboardType. The resulting NSPasteboardItem was
not correctly writing to the NSPasteboard. Using the public UTIs directly fixes
this problem.

BUG=618771
Committed: https://crrev.com/23d52abd52313ed4431d33935d6a4d72394a350c
Cr-Commit-Position: refs/heads/master@{#401134}

[erikchen, 2016-06-21 17:10:54]

The CQ bit was checked by erikchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-21 17:11:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2080603007/1

[erikchen, 2016-06-21 17:11:26]

erikchen@chromium.org changed reviewers:
+ ccameron@chromium.org, shrike@chromium.org

[erikchen, 2016-06-21 17:11:26]

shrike, ccameron: PTAL

Note that I don't have Outlook, so I wasn't able to test the case listed in the
bug.

[shrike, 2016-06-21 17:22:20]

On 2016/06/21 17:11:26, erikchen wrote:
> shrike, ccameron: PTAL
> 
> Note that I don't have Outlook, so I wasn't able to test the case listed in
the
> bug.

Hi erikchen@,

Thank you for putting this together. Note that the problem exhibits itself in
the Notes app (/Applications/Notes.app), so you shouldn't need Outlook to test.
Your patch appears to work with Notes.

[erikchen, 2016-06-21 17:26:01]

On 2016/06/21 17:22:20, shrike wrote:
> On 2016/06/21 17:11:26, erikchen wrote:
> > shrike, ccameron: PTAL
> > 
> > Note that I don't have Outlook, so I wasn't able to test the case listed in
> the
> > bug.
> 
> Hi erikchen@,
> 
> Thank you for putting this together. Note that the problem exhibits itself in
> the Notes app (/Applications/Notes.app), so you shouldn't need Outlook to
test.
> Your patch appears to work with Notes.

Which version of macOS? copy-pasting from the omnibox into Notes works for me
with latest Chrome Canary.

[commit-bot: I haz the power, 2016-06-21 17:56:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-21 17:56:02]

Dry run: This issue passed the CQ dry run.

[shrike, 2016-06-21 22:19:03]

ccameron@ - do you have a spare moment  to review this cl? We're trying to get
the fix landed ASAP. Thank you.

[ccameron, 2016-06-21 22:23:52]

rs lgtm

[shrike, 2016-06-21 22:27:09]

On 2016/06/21 17:26:01, erikchen wrote:
> Which version of macOS? copy-pasting from the omnibox into Notes works for me
> with latest Chrome Canary.

I'm running 10.11.5, which I suspect you are as well.

Very strange - copy/paste from Chrome to Notes works for me on my desktop
machine, from Canary and M51. I am unable to do the same on my laptop. I confirm
that your cl fixes the problem on my laptop.

[erikchen, 2016-06-21 22:38:45]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-06-21 22:40:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2080603007/1

[commit-bot: I haz the power, 2016-06-21 22:51:41]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-21 22:53:18]

Description was changed from

==========
[Mac] Fix regression with copying URLs from the Omnibox.

The function ClipboardUtil::PasteboardItemFromUrl was incorrectly attempting to
create a public UTI using UTIFromPboardType. The resulting NSPasteboardItem was
not correctly writing to the NSPasteboard. Using the public UTIs directly fixes
this problem.

BUG=618771
==========

to

==========
[Mac] Fix regression with copying URLs from the Omnibox.

The function ClipboardUtil::PasteboardItemFromUrl was incorrectly attempting to
create a public UTI using UTIFromPboardType. The resulting NSPasteboardItem was
not correctly writing to the NSPasteboard. Using the public UTIs directly fixes
this problem.

BUG=618771

Committed: https://crrev.com/23d52abd52313ed4431d33935d6a4d72394a350c
Cr-Commit-Position: refs/heads/master@{#401134}
==========

[commit-bot: I haz the power, 2016-06-21 22:53:20]

Patchset 1 (id:??) landed as
https://crrev.com/23d52abd52313ed4431d33935d6a4d72394a350c
Cr-Commit-Position: refs/heads/master@{#401134}