Migrate fullPaths to URLs in appState.

Migrate fullPaths to URLs in appState.

This patch replaces currentDirectoryPath with currentDirectoryURL, and same for the selectionPath. As a result, the entire appState does not depend on paths anymore.

TEST=browser_tests, also tested manually.
BUG=333168
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=247855

[mtomasz, 2014-01-27 05:16:16]

@hirono: PTAL. Thanks!

[hirono, 2014-01-27 05:39:56]

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
File chrome/browser/resources/file_manager/background/js/background.js (right):

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/background/js/background.js:536:
contentWindow.appState.currentDirectoryPath) {
currentDirectoryURL?

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/background/js/background.js:784:
currentDirectoryPath: background.appWindows[key].contentWindow.
ditto.

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
File
chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js
(right):

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js:465:
fileManager.getCurrentDirectoryEntry().toURL
toURL()

[mtomasz, 2014-01-27 06:12:52]

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
File chrome/browser/resources/file_manager/background/js/background.js (right):

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/background/js/background.js:536:
contentWindow.appState.currentDirectoryPath) {
On 2014/01/27 05:39:56, hirono wrote:
> currentDirectoryURL?

Done.

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/background/js/background.js:784:
currentDirectoryPath: background.appWindows[key].contentWindow.
On 2014/01/27 05:39:56, hirono wrote:
> ditto.

Done.

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
File
chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js
(right):

https://codereview.chromium.org/132453007/diff/1/chrome/browser/resources/fil...
chrome/browser/resources/file_manager/foreground/js/file_manager_commands.js:465:
fileManager.getCurrentDirectoryEntry().toURL
On 2014/01/27 05:39:56, hirono wrote:
> toURL()

Done.

[hirono, 2014-01-27 06:31:33]

lgtm! Thanks!

[mtomasz, 2014-01-29 02:57:01]

On 2014/01/27 06:31:33, hirono wrote:
> lgtm! Thanks!

I've rebased the CL.

@sky: PTAL at select_file_dialog_extension.cc
@hirono: PTAL one more time.

Thanks.

[hirono, 2014-01-29 06:02:59]

On 2014/01/29 02:57:01, mtomasz wrote:
> On 2014/01/27 06:31:33, hirono wrote:
> > lgtm! Thanks!
> 
> I've rebased the CL.
> 
> @sky: PTAL at select_file_dialog_extension.cc
> @hirono: PTAL one more time.
> 
> Thanks.

still lgtm!

[sky, 2014-01-29 17:27:54]

LGTM

[commit-bot: I haz the power, 2014-01-29 22:59:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/132453007/320001

[commit-bot: I haz the power, 2014-01-30 00:17:33]

Retried try job too often on android_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2014-01-30 00:28:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mtomasz@chromium.org/132453007/320001

[commit-bot: I haz the power, 2014-01-30 06:41:14]

Change committed as 247855