mediaview: Register ARC documents provider file system.

mediaview: Register ARC documents provider file system.

ArcContentFileSystemService is renamed to ArcFileSystemService, and it
will register two ARC file systems.

Also constants in arc_content_file_system_url_util.h are renamed
now that we have two ARC file systems.

BUG=chromium:671511
TEST=unit_tests --gtest_filter='Arc*'
TEST=trybot

Review-Url: https://codereview.chromium.org/2589643002
Cr-Commit-Position: refs/heads/master@{#441905}
Committed: https://chromium.googlesource.com/chromium/src/+/3022c4f86fd02985e11e524e951512c85c4f4268

[Shuhei Takahashi, 2016-12-19 05:18:45]

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

[commit-bot: I haz the power, 2016-12-19 05:18:54]

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, 2016-12-19 07:45:59]

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

[commit-bot: I haz the power, 2016-12-19 07:45:59]

Dry run: This issue passed the CQ dry run.

[Shuhei Takahashi, 2016-12-19 11:34:16]

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

[commit-bot: I haz the power, 2016-12-19 11:34:31]

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

[Shuhei Takahashi, 2016-12-19 11:35:03]

nya@chromium.org changed reviewers:
+ hashimoto@chromium.org, lhchavez@chromium.org

[Shuhei Takahashi, 2016-12-19 11:35:03]

lhchavez, hashimoto: PTAL

[commit-bot: I haz the power, 2016-12-19 12:54:28]

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

[commit-bot: I haz the power, 2016-12-19 12:54:28]

Dry run: This issue passed the CQ dry run.

[Luis Héctor Chávez, 2016-12-19 22:37:27]

lgtm with a nit.

https://codereview.chromium.org/2589643002/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/arc/fileapi/arc_file_system_service.h (right):

https://codereview.chromium.org/2589643002/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/arc/fileapi/arc_file_system_service.h:20: public
InstanceHolder<mojom::FileSystemInstance>::Observer {
You don't seem to need this.

[Shuhei Takahashi, 2017-01-05 08:49:43]

hashimoto: PTAL

https://codereview.chromium.org/2589643002/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/arc/fileapi/arc_file_system_service.h (right):

https://codereview.chromium.org/2589643002/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/arc/fileapi/arc_file_system_service.h:20: public
InstanceHolder<mojom::FileSystemInstance>::Observer {
On 2016/12/19 22:37:27, Luis Héctor Chávez wrote:
> You don't seem to need this.

Oops, good catch.

[Shuhei Takahashi, 2017-01-05 09:15:19]

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

[commit-bot: I haz the power, 2017-01-05 09:15:28]

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

[hashimoto, 2017-01-05 09:24:13]

lgtm

[commit-bot: I haz the power, 2017-01-05 10:16:11]

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

[commit-bot: I haz the power, 2017-01-05 10:16:12]

Dry run: This issue passed the CQ dry run.

[Shuhei Takahashi, 2017-01-06 05:39:29]

The CQ bit was checked by nya@chromium.org

[Shuhei Takahashi, 2017-01-06 05:39:29]

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

[commit-bot: I haz the power, 2017-01-06 05:39:45]

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

[commit-bot: I haz the power, 2017-01-06 05:39:46]

This CL has an open dependency (Issue 2580713004 Patch 200001). 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-01-06 08:02:09]

The CQ bit was checked by nya@chromium.org

[commit-bot: I haz the power, 2017-01-06 08:02:21]

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-01-06 09:04:22]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1483689729005670,
"parent_rev": "8e9a8aabbde296324fa081995177dd5aac5622e4", "commit_rev":
"3022c4f86fd02985e11e524e951512c85c4f4268"}

[commit-bot: I haz the power, 2017-01-06 09:04:49]

Description was changed from

==========
mediaview: Register ARC documents provider file system.

ArcContentFileSystemService is renamed to ArcFileSystemService, and it
will register two ARC file systems.

Also constants in arc_content_file_system_url_util.h are renamed
now that we have two ARC file systems.

BUG=chromium:671511
TEST=unit_tests --gtest_filter='Arc*'
TEST=trybot
==========

to

==========
mediaview: Register ARC documents provider file system.

ArcContentFileSystemService is renamed to ArcFileSystemService, and it
will register two ARC file systems.

Also constants in arc_content_file_system_url_util.h are renamed
now that we have two ARC file systems.

BUG=chromium:671511
TEST=unit_tests --gtest_filter='Arc*'
TEST=trybot

Review-Url: https://codereview.chromium.org/2589643002
Cr-Commit-Position: refs/heads/master@{#441905}
Committed:
https://chromium.googlesource.com/chromium/src/+/3022c4f86fd02985e11e524e9515...
==========

[commit-bot: I haz the power, 2017-01-06 09:04:51]

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