Open cros filemanager when "Open Downloads Folder" is clicked.

Open cros filemanager when "Open Downloads Folder" is clicked.

BUG=chromium-os:21585
TEST=On ChromeOS, ctrl-j for the downloads page, click on Open Downloads Folder to have a new tab open with the filemanager extension positioned in the Downloads folder.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108374

[achuithb, 2011-11-01 22:51:28]

There is some urgency with this CL as it fixes a Stable-block bug for M15 on
ChromeOS. Please take a look at your earliest convenience.

[benjhayden, 2011-11-02 13:22:51]

LGTM assuming it was tested manually. Please wait for Randy and Asanka.

[Randy Smith (Not in Mondays), 2011-11-02 13:25:31]

LGTM.

[asanka, 2011-11-02 14:14:47]

Ideally shouldn't this fixed in FileManagerUtil::ViewItem()?  Or perhaps in
platform_util?  That would move platform specific code into the platform
specific implementations.

LGTM given the urgency, but long term I think we need to fix
platform_util::OpenItem() to do the right thing when given a folder.

[achuithb, 2011-11-02 20:45:49]

On 2011/11/02 14:14:47, asanka wrote:
> Ideally shouldn't this fixed in FileManagerUtil::ViewItem()?  Or perhaps in
> platform_util?  That would move platform specific code into the platform
> specific implementations.
> 
> LGTM given the urgency, but long term I think we need to fix
> platform_util::OpenItem() to do the right thing when given a folder.

I'll fix OpenItem for all platforms in a separate CL. I need minimal changes for
M15. 

Thanks for the reviews, Randy, Ben and Asanka!