[Mac] On Mavericks or later, use NSWorkspace to open download items.

[Mac] On Mavericks or later, use NSWorkspace to open download items.

With the NSWorkspaceLaunchWithErrorPresentation option, Finder no longer
silently fails opening the file.

BUG=32921
R=avi@chromium.org
TEST=Download a DMG file. Click the download shelf button to open/mount it. Finder activates the new disk image window as the topmost.
TEST=Download a file with an unassociated file extension type (e.g.  "test.foobar"). Click the download shelf button to open it. Finder presents the unknown file type dialog.

Committed: https://chromium.googlesource.com/chromium/src/+/b83907e0384a25d8d10e5e15c564db4ec18f21bc

[Avi (use Gerrit), 2014-09-29 15:36:00]

https://codereview.chromium.org/613673003/diff/1/chrome/browser/platform_util...
File chrome/browser/platform_util_mac.mm (right):

https://codereview.chromium.org/613673003/diff/1/chrome/browser/platform_util...
chrome/browser/platform_util_mac.mm:43: if (base::mac::IsOSMavericksOrLater()) {
Fix the comment at the top of the function ("This function opens a file...") or
move that function to below here.

[Robert Sesek, 2014-09-29 15:41:16]

https://codereview.chromium.org/613673003/diff/1/chrome/browser/platform_util...
File chrome/browser/platform_util_mac.mm (right):

https://codereview.chromium.org/613673003/diff/1/chrome/browser/platform_util...
chrome/browser/platform_util_mac.mm:43: if (base::mac::IsOSMavericksOrLater()) {
On 2014/09/29 15:36:00, Avi wrote:
> Fix the comment at the top of the function ("This function opens a file...")
or
> move that function to below here.

Done.

[Avi (use Gerrit), 2014-09-29 15:48:38]

LGTM!

[commit-bot: I haz the power, 2014-09-29 19:48:03]

Patchset 3 (id:??) landed as
https://crrev.com/b83907e0384a25d8d10e5e15c564db4ec18f21bc
Cr-Commit-Position: refs/heads/master@{#297239}

[Robert Sesek, 2014-09-29 19:48:18]

Committed patchset #3 (id:40001) manually as b83907e.