filemanager: Pass the entire Drive path to DirectoryModel for setupCurrentDirectory.

filemanager: Pass the entire Drive path to DirectoryModel for setupCurrentDirectory.

This can restore the original behavior.
Note that Directory contents do not resolve the sub-directories in /drive/root
and the UI cannot open a deep subdirectory in Drive (http://crbug.com/227530).


BUG=227488
TEST=Open Google Drive in Files App, and reload the app and verify the directory contents properly opened.

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

[Haruki Sato, 2013-04-08 09:59:55]

I think this works and we can investigate further improvement 
later.
Do you think this is OK?

I also filed 
https://code.google.com/p/chromium/issues/detail?id=227530&thanks=227530&ts=1...

[yoshiki, 2013-04-08 10:05:31]

Thanks! LGTM.

https://codereview.chromium.org/13496017/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/file_manager.js (right):

https://codereview.chromium.org/13496017/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/file_manager.js:1363: //
this.directoryModel_.setupPath(RootDirectory.DRIVE);
nit: remove this line.

[Haruki Sato, 2013-04-08 10:16:30]

https://codereview.chromium.org/13496017/diff/1/chrome/browser/resources/file...
File chrome/browser/resources/file_manager/js/file_manager.js (right):

https://codereview.chromium.org/13496017/diff/1/chrome/browser/resources/file...
chrome/browser/resources/file_manager/js/file_manager.js:1363: //
this.directoryModel_.setupPath(RootDirectory.DRIVE);
On 2013/04/08 10:05:31, yoshiki wrote:
> nit: remove this line.

oops. Thanks. Done.

[commit-bot: I haz the power, 2013-04-08 10:16:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/haruki@chromium.org/13496017/3

[commit-bot: I haz the power, 2013-04-08 10:23:48]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[Haruki Sato, 2013-04-08 11:26:55]

Committed patchset #2 manually as r192804 (presubmit successful).