Add notification and observer to copy the file between other users.

Add notification and observer to copy the file between other users.

* NotificationRegister and NotificationObserver is need to collect
  file information.

* This CL is related to https://codereview.chromium.org/530553006

BUG=412691
Committed: https://crrev.com/0407ff6ae1510a622f422959af156f35860c9e30
Cr-Commit-Position: refs/heads/master@{#294346}

[iseki, 2014-09-11 03:24:18]

iseki@chromium.org changed reviewers:
+ fukino@chromium.org, yoshiki@chromium.org

[iseki, 2014-09-11 03:24:19]

Please take a look.

[yoshiki, 2014-09-11 04:43:25]

lgtm

[fukino, 2014-09-11 05:00:16]

https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
File chrome/browser/chromeos/extensions/file_manager/event_router.cc (right):

https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
chrome/browser/chromeos/extensions/file_manager/event_router.cc:968:
file_browser_private::OnProfileAdded::kEventName,
I think there is no listener for onProfileAdded event, so only
GrantAccessForAddedProfileToRunningInstance(added_profile, profile_) looks
necessary.
Could you confirm this, and remove this BroadcastEvent() and onProfileAdded in
idl file if possible?

[iseki, 2014-09-11 05:15:52]

Thank you for your review.

https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
File chrome/browser/chromeos/extensions/file_manager/event_router.cc (right):

https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
chrome/browser/chromeos/extensions/file_manager/event_router.cc:968:
file_browser_private::OnProfileAdded::kEventName,
On 2014/09/11 05:00:16, fukino wrote:
> I think there is no listener for onProfileAdded event, so only
> GrantAccessForAddedProfileToRunningInstance(added_profile, profile_) looks
> necessary.
> Could you confirm this, and remove this BroadcastEvent() and onProfileAdded in
> idl file if possible?

It can be removed.

[fukino, 2014-09-11 06:13:07]

On 2014/09/11 05:15:52, iseki wrote:
> Thank you for your review.
> 
>
https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
> File chrome/browser/chromeos/extensions/file_manager/event_router.cc (right):
> 
>
https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
> chrome/browser/chromeos/extensions/file_manager/event_router.cc:968:
> file_browser_private::OnProfileAdded::kEventName,
> On 2014/09/11 05:00:16, fukino wrote:
> > I think there is no listener for onProfileAdded event, so only
> > GrantAccessForAddedProfileToRunningInstance(added_profile, profile_) looks
> > necessary.
> > Could you confirm this, and remove this BroadcastEvent() and onProfileAdded
in
> > idl file if possible?
> 
> It can be removed.

lgtm.

[fukino, 2014-09-11 06:13:08]

On 2014/09/11 05:15:52, iseki wrote:
> Thank you for your review.
> 
>
https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
> File chrome/browser/chromeos/extensions/file_manager/event_router.cc (right):
> 
>
https://codereview.chromium.org/566433004/diff/1/chrome/browser/chromeos/exte...
> chrome/browser/chromeos/extensions/file_manager/event_router.cc:968:
> file_browser_private::OnProfileAdded::kEventName,
> On 2014/09/11 05:00:16, fukino wrote:
> > I think there is no listener for onProfileAdded event, so only
> > GrantAccessForAddedProfileToRunningInstance(added_profile, profile_) looks
> > necessary.
> > Could you confirm this, and remove this BroadcastEvent() and onProfileAdded
in
> > idl file if possible?
> 
> It can be removed.

lgtm.

[iseki, 2014-09-11 06:13:44]

The CQ bit was checked by iseki@chromium.org

[commit-bot: I haz the power, 2014-09-11 06:24:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/566433004/20001

[commit-bot: I haz the power, 2014-09-11 07:30:29]

Committed patchset #2 (id:20001) as 68c0136eacf331607ae023af902ad428a10cf39d

[commit-bot: I haz the power, 2014-09-11 07:40:13]

Patchset 2 (id:??) landed as
https://crrev.com/0407ff6ae1510a622f422959af156f35860c9e30
Cr-Commit-Position: refs/heads/master@{#294346}