Re-enable photo importer in Files.app.

Re-enable photo importer in Files.app.

It has been disabled because of bugs.

TEST=Enter usb drive, right click on the usb drive item in the left nav in Files.app, choose Import media.
BUG=176203
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182422

[mtomasz, 2013-02-14 05:08:44]

@yoshiki: PTAL.

[yoshiki, 2013-02-14 06:18:08]

lgtm

[commit-bot: I haz the power, 2013-02-14 07:26:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/12255024/1

[commit-bot: I haz the power, 2013-02-14 07:26:15]

Presubmit check for 12255024-1 failed and returned exit status 1.

INFO:root:Found 2 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/chrome/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/chrome/browser/resources/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/chromeos/extensions

[mtomasz, 2013-02-14 08:23:36]

On 2013/02/14 07:26:15, I haz the power (commit-bot) wrote:
> Presubmit check for 12255024-1 failed and returned exit status 1.
> 
> INFO:root:Found 2 file(s).
> 
> Running presubmit commit checks ...
> Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
> Running /b/commit-queue/workdir/chromium/chrome/PRESUBMIT.py
> Running /b/commit-queue/workdir/chromium/chrome/browser/resources/PRESUBMIT.py
> 
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for files in these directories:
>     chrome/browser/chromeos/extensions

@hashimoto: PTAL at file_browser_event_router.cc. This is just as it was before:
http://src.chromium.org/viewvc/chrome/branches/1364/src/chrome/browser/chrome...

[hashimoto, 2013-02-14 08:30:18]

file_browser_event_router.cc lgtm

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
File chrome/browser/chromeos/extensions/file_browser_event_router.cc (right):

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_browser_event_router.cc:676:
file_manager_util::OpenActionChoiceDialog(FilePath(mount_path));
nit: It seems no one uses ViewRemovableDrive after this change, can we remove
it?

[mtomasz, 2013-02-14 08:43:39]

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
File chrome/browser/chromeos/extensions/file_browser_event_router.cc (right):

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_browser_event_router.cc:676:
file_manager_util::OpenActionChoiceDialog(FilePath(mount_path));
On 2013/02/14 08:30:18, hashimoto wrote:
> nit: It seems no one uses ViewRemovableDrive after this change, can we remove
> it?

I'm going to use it in m27. The loader will not be launched for any removable
drive other than sd cards.

[hashimoto, 2013-02-14 08:46:37]

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
File chrome/browser/chromeos/extensions/file_browser_event_router.cc (right):

https://codereview.chromium.org/12255024/diff/1/chrome/browser/chromeos/exten...
chrome/browser/chromeos/extensions/file_browser_event_router.cc:676:
file_manager_util::OpenActionChoiceDialog(FilePath(mount_path));
On 2013/02/14 08:43:39, mtomasz wrote:
> On 2013/02/14 08:30:18, hashimoto wrote:
> > nit: It seems no one uses ViewRemovableDrive after this change, can we
remove
> > it?
> 
> I'm going to use it in m27. The loader will not be launched for any removable
> drive other than sd cards.

OK, sgtm.