Files.app: Disable the cloud import notification for active g+ photos app users.

Files.app: Disable the cloud import notification for active g+ photos app users.

If a user has auto-import enabled in the g+ photos app, don't enable cloud import, and don't pop up notifications for it yet.  We'll re-enable these notifications when the transition path has been smoothed a bit more.

BUG=462110
Committed: https://crrev.com/56c9dc7cb83ecb9fea6eeeefb2d7d401a3cda919
Cr-Commit-Position: refs/heads/master@{#318237}

[Ben Kwa, 2015-02-26 05:36:39]

kenobi@chromium.org changed reviewers:
+ mtomasz@chromium.org

[mtomasz, 2015-02-26 05:40:34]

lgtm

https://codereview.chromium.org/955313002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/device_handler.js (right):

https://codereview.chromium.org/955313002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/device_handler.js:489: // inserts a
removable device.  Only open files app if
nit: files -> Files
nit: photos -> Photos?

[Steve McKay, 2015-02-26 05:53:18]

smckay@chromium.org changed reviewers:
+ smckay@chromium.org

[Steve McKay, 2015-02-26 05:53:19]

lgtm

[Ben Kwa, 2015-02-26 15:20:17]

Done.  I had to remove the failing test because of the usual problem (no
reliable way to test that something *didn't* happen).  If you'd like a test I
can try to think of one and add it in a follow-up.

Thanks!

https://codereview.chromium.org/955313002/diff/1/ui/file_manager/file_manager...
File ui/file_manager/file_manager/background/js/device_handler.js (right):

https://codereview.chromium.org/955313002/diff/1/ui/file_manager/file_manager...
ui/file_manager/file_manager/background/js/device_handler.js:489: // inserts a
removable device.  Only open files app if
On 2015/02/26 05:40:33, mtomasz wrote:
> nit: files -> Files
> nit: photos -> Photos?

Done.

[Ben Kwa, 2015-02-26 15:20:22]

The CQ bit was checked by kenobi@chromium.org

[Ben Kwa, 2015-02-26 15:20:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
mtomasz@chromium.org, smckay@chromium.org
Link to the patchset: https://codereview.chromium.org/955313002/#ps20001 (title:
"Address feedback; fix tests.")

[commit-bot: I haz the power, 2015-02-26 15:21:19]

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

[commit-bot: I haz the power, 2015-02-26 15:53:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-26 15:54:20]

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