| commit | 0659c8afb9f57ec232cc25944449b78591b43b0e | |
| author | yoshiki@chromium.org <yoshiki@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Tue, 29 Apr 2014 09:09:53 +0000 (29 09:09 +0000) | ||
| committer | yoshiki@chromium.org <yoshiki@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Tue, 29 Apr 2014 09:09:53 +0000 (29 09:09 +0000) | ||
| tree | a6c63f4afc967ef7a36a31d94fda7ffe1163d7b7 | treesnapshot (tar.gz zip) |
| parent | fa66ba04b6f9acaf9a2f55afa5d72b01160752e9 | commitdiff |
Reland of r266437: [Files.app] Use getDriveEntryProperties() to retrieve metadata
This is a reland of r260688, which is reverted due to last-minute rebase mistake.
Previously, we used FileEntry.getMetadata() to retrieve filesystem metadata and getDriveEntryProperties() to get Drive metadata, so we need 2 calls for 1 file on Drive.
With this patch, getDriveEntryProperties() returns not only Drive metadata but also filesystem metadata. It's enough to call only getDriveEntryProperties() and we can reduce a number of calls by half.
BUG=345196
TEST=browser_test passes.
R=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
TBR=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
NOTRY=True
# NOTRYing for buildbots already passes (see. crrev.com/256023002)
Review URL: https://codereview.chromium.org/256023002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266795 0039d316-1c4b-4281-b951-d872f2087c98
This is a reland of r260688, which is reverted due to last-minute rebase mistake.
Previously, we used FileEntry.getMetadata() to retrieve filesystem metadata and getDriveEntryProperties() to get Drive metadata, so we need 2 calls for 1 file on Drive.
With this patch, getDriveEntryProperties() returns not only Drive metadata but also filesystem metadata. It's enough to call only getDriveEntryProperties() and we can reduce a number of calls by half.
BUG=345196
TEST=browser_test passes.
R=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
TBR=asargent@chromium.org, hashimoto@chromium.org, hirono@chromium.org
NOTRY=True
# NOTRYing for buildbots already passes (see. crrev.com/256023002)
Review URL: https://codereview.chromium.org/256023002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266795 0039d316-1c4b-4281-b951-d872f2087c98