Cleanup: Add fileapi::GetIsolatedFileSystemRootURIString

Cleanup: Add fileapi::GetIsolatedFileSystemRootURIString

to hide the details how we create a root URI string for IsolatedFileSystems
from each extension code.

BUG=none
TEST=existing tests

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176036

[kinuko, 2013-01-09 17:02:06]

Can you take a look? Should be an innocent cleanup.

benwells@: chrome/renderer/extensions/file_system_natives.cc
thestig@: chrome/renderer/extensions/media_galleries_custom_bindings.cc

Thx,

[Lei Zhang, 2013-01-09 21:29:03]

lgtm

https://codereview.chromium.org/11833002/diff/1/webkit/fileapi/file_system_ut...
File webkit/fileapi/file_system_util.h (right):

https://codereview.chromium.org/11833002/diff/1/webkit/fileapi/file_system_ut...
webkit/fileapi/file_system_util.h:127: // for |filesystem_id|. If the
|optional_root_name| is given the resulting
nit: for -> and

[benwells, 2013-01-10 03:26:54]

lgtm

[kinuko, 2013-01-10 04:03:14]

tzik: can you give a quick look at the fileapi changes? Thx,

https://codereview.chromium.org/11833002/diff/1/webkit/fileapi/file_system_ut...
File webkit/fileapi/file_system_util.h (right):

https://codereview.chromium.org/11833002/diff/1/webkit/fileapi/file_system_ut...
webkit/fileapi/file_system_util.h:127: // for |filesystem_id|. If the
|optional_root_name| is given the resulting
On 2013/01/09 21:29:03, Lei Zhang wrote:
> nit: for -> and

Done.

[tzik, 2013-01-10 05:24:19]

lgtm

[commit-bot: I haz the power, 2013-01-10 05:26:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kinuko@chromium.org/11833002/9002