mediaview: Implement ArcDocumentsProviderWatcherManager.

mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
TEST=Media view refreshes on media provider changes.

Review-Url: https://codereview.chromium.org/2709013003
Cr-Commit-Position: refs/heads/master@{#454598}
Committed: https://chromium.googlesource.com/chromium/src/+/7b44260f27f6b52b3a1b81e81cdedeb8164b4497

[Shuhei Takahashi, 2017-02-22 12:47:47]

Description was changed from

==========
wip
==========

to

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

BUG=chromium:684233
TEST=trybot
==========

[Shuhei Takahashi, 2017-02-22 12:47:56]

The CQ bit was checked by nya@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 12:48:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Shuhei Takahashi, 2017-02-22 12:49:37]

Description was changed from

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

BUG=chromium:684233
TEST=trybot
==========

to

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

TODO: Write CL description

BUG=chromium:684233
TEST=trybot
==========

[commit-bot: I haz the power, 2017-02-22 13:53:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-22 13:53:36]

Dry run: This issue passed the CQ dry run.

[Shuhei Takahashi, 2017-02-23 04:52:52]

Description was changed from

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

TODO: Write CL description

BUG=chromium:684233
TEST=trybot
==========

to

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
==========

[Shuhei Takahashi, 2017-02-24 07:17:41]

Description was changed from

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
==========

to

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
TEST=Media view refreshes on media provider changes.
==========

[Shuhei Takahashi, 2017-03-02 07:07:45]

The CQ bit was checked by nya@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-02 07:08:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-02 08:28:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-02 08:28:11]

Dry run: This issue passed the CQ dry run.

[Shuhei Takahashi, 2017-03-02 08:40:24]

nya@chromium.org changed reviewers:
+ hidehiko@chromium.org

[Shuhei Takahashi, 2017-03-02 08:40:25]

hidehiko: PTAL

[Shuhei Takahashi, 2017-03-02 08:41:22]

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

[Shuhei Takahashi, 2017-03-02 08:41:23]

mtomasz: PTAL for chrome/browser/chromeos/fileapi

[mtomasz, 2017-03-02 08:50:18]

lgtm with a question

https://codereview.chromium.org/2709013003/diff/60001/chrome/browser/chromeos...
File chrome/browser/chromeos/fileapi/file_system_backend.cc (right):

https://codereview.chromium.org/2709013003/diff/60001/chrome/browser/chromeos...
chrome/browser/chromeos/fileapi/file_system_backend.cc:10: #include <utility>
nit: Is this include needed?

[Shuhei Takahashi, 2017-03-02 08:51:31]

https://codereview.chromium.org/2709013003/diff/60001/chrome/browser/chromeos...
File chrome/browser/chromeos/fileapi/file_system_backend.cc (right):

https://codereview.chromium.org/2709013003/diff/60001/chrome/browser/chromeos...
chrome/browser/chromeos/fileapi/file_system_backend.cc:10: #include <utility>
On 2017/03/02 08:50:18, mtomasz wrote:
> nit: Is this include needed?

Yes, for std::move(). Actually lint complains for the current code :)

[hidehiko, 2017-03-02 14:51:23]

c/b/c/arc LGTM.

[Shuhei Takahashi, 2017-03-03 15:36:06]

The CQ bit was checked by nya@chromium.org

[commit-bot: I haz the power, 2017-03-03 15:36:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-03 15:36:21]

This CL has an open dependency (Issue 2715473003 Patch 120001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[Shuhei Takahashi, 2017-03-03 15:49:16]

The CQ bit was checked by nya@chromium.org

[commit-bot: I haz the power, 2017-03-03 15:49:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-03 15:51:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-03 15:51:39]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Shuhei Takahashi, 2017-03-03 16:01:13]

The CQ bit was checked by nya@chromium.org

[Shuhei Takahashi, 2017-03-03 16:01:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
hidehiko@chromium.org, mtomasz@chromium.org
Link to the patchset: https://codereview.chromium.org/2709013003/#ps80001
(title: "Rebased to ToT.")

[commit-bot: I haz the power, 2017-03-03 16:01:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-03 16:56:15]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1488556873645390,
"parent_rev": "b41df0b205caeb396659371b5874936cee6c0c6f", "commit_rev":
"7b44260f27f6b52b3a1b81e81cdedeb8164b4497"}

[commit-bot: I haz the power, 2017-03-03 16:56:53]

Description was changed from

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
TEST=Media view refreshes on media provider changes.
==========

to

==========
mediaview: Implement ArcDocumentsProviderWatcherManager.

ArcDocumentsProviderWatcherManager simply calls ArcDocumentsProviderRoot.

BUG=chromium:684233
TEST=trybot
TEST=Media view refreshes on media provider changes.

Review-Url: https://codereview.chromium.org/2709013003
Cr-Commit-Position: refs/heads/master@{#454598}
Committed:
https://chromium.googlesource.com/chromium/src/+/7b44260f27f6b52b3a1b81e81cde...
==========

[commit-bot: I haz the power, 2017-03-03 16:56:54]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/7b44260f27f6b52b3a1b81e81cde...