[mac bug triage] Assigning to lgrey@ to investigate

What's going on is that in Cocoa, the chrome/browser/ui/cocoa/tabs/tab_strip_view.mm was an URLDropTarget, which called PopulateURLAndTitleFromPasteboard, which called out to the code that read .webloc files.

In Views, the OSExchangeDataProvider is used, and the Mac version doesn't call PopulateURLAndTitleFromPasteboard. I tried a simple fix (https://chromium-review.googlesource.com/c/chromium/src/+/1148797) but it makes all kinds of things break.

Based on the CQ failures, we could patch third_party/mozilla/NSPasteboard+Utils.mm to do what Views wants assuming we're OK with slightly changing the Cocoa behavior. Or did other stuff break too?

I have a fix.

(And looking at the pasteboard code and having filed bug 867553, I'll probably take ownership of the pasteboard code in general.)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f70278fe826d0c212852ca2c0c23ca23952b0a8d

commit f70278fe826d0c212852ca2c0c23ca23952b0a8d
Author: Avi Drissman <avi@chromium.org>
Date: Wed Jul 25 20:45:15 2018

Load .webloc files correctly.

If a user drops a .webloc file onto the tabstrip, load the URL
inside the .webloc file, not the contents of the .webloc file
itself.

BUG= 856469 

Change-Id: Id581936a712180b188ff0f4f92ae10bb4353e5e6
Reviewed-on: https://chromium-review.googlesource.com/1148797
Reviewed-by: Leonard Grey <lgrey@chromium.org>
Commit-Queue: Avi Drissman <avi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578047}
[modify] https://crrev.com/f70278fe826d0c212852ca2c0c23ca23952b0a8d/ui/base/dragdrop/os_exchange_data_provider_mac.mm

@avi@chromium.org : It would be really helpful if a sample Test file/URL is provided, so that we can verifying the fix.

Thanks...!

Repro:
1. Go to a webpage with a link on the page.
2. Drag the link to the desktop. Confirm that the file created is a .webloc file.
3. Drag that .webloc file to the titlebar, confirm that the link that was the source from step 2 is loaded.