New issue
Advanced search Search tips

Issue 855982 link

Starred by 1 user
Status: Fixed
Owner:
fukino@chromium.org
Closed: Jun 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Files in "Play Files" can't be opened by Android apps.

Project Member Reported by fukino@chromium.org, Jun 25 2018 
With a flag #show-android-files-in-files-app enabled, Files app shows Android files inside "Play Files" root on left-side nav.
However, files inside "Play Files" can't be opened by Android apps.
Chrome should handle files under "Play Files" to generate appropriate content URL, and ArcIntentHelper should handle request with such URLs.

This is a counterpart of b/110639857.
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 25 2018 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a956143e09b3083ea7bc0c142f8a11bc24493540

commit a956143e09b3083ea7bc0c142f8a11bc24493540
Author: Naoki Fukino <fukino@chromium.org>
Date: Mon Jun 25 10:37:47 2018

Convert URLs of Android files to ArcUrls.

Android container now supports content urls with /external_files to access all
files on emulated external storage.
We need to convert URLs under /run/arc/sdcard/write/emulated/0 on Chrome OS side
to the content URLs.
To share the path to Android files, the definition of kAndroidFilesPath is moved
from volume_manager to path_util.

Bug:  855982 
Test: Manually tested on Kevin and Eve with a patch ag/4411504 on Android.
Change-Id: Idb6fbd9aa697c306d953b0bcb7cf332ec9a26c19
Reviewed-on: https://chromium-review.googlesource.com/1113169
Commit-Queue: Naoki Fukino <fukino@chromium.org>
Reviewed-by: Shuhei Takahashi <nya@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570001}
[modify] https://crrev.com/a956143e09b3083ea7bc0c142f8a11bc24493540/chrome/browser/chromeos/file_manager/path_util.cc
[modify] https://crrev.com/a956143e09b3083ea7bc0c142f8a11bc24493540/chrome/browser/chromeos/file_manager/path_util.h
[modify] https://crrev.com/a956143e09b3083ea7bc0c142f8a11bc24493540/chrome/browser/chromeos/file_manager/path_util_unittest.cc
[modify] https://crrev.com/a956143e09b3083ea7bc0c142f8a11bc24493540/chrome/browser/chromeos/file_manager/volume_manager.cc

Comment 2 by fukino@chromium.org, Jun 25 2018

Status: Fixed (was: Started)
►

Sign in to add a comment