Gallery: Update metadata in GalleryItem when saving items.

Gallery: Obtain metadata from MetadataCache when saving items.

Previously, when saving items, the new metadata was generated by
GalleryDataModel. But the generated metadata was broken and it contained old
thumbnail data that causes memory shortage.

The metadata for new items can be otained by using normal code path
(MetadataCache), so the CL removes special geneartion of metadata for new items
and let Gallery obtain the metadata by using MeatadataCache.

BUG=439898
TEST=GalleryJsTest.GalleryItemTest

Committed: https://crrev.com/699c53ebb13827f09e6cb5d4564f69da03d58692
Cr-Commit-Position: refs/heads/master@{#310205}

[hirono, 2015-01-06 08:02:02]

Patchset #1 (id:1) has been deleted

[hirono, 2015-01-06 08:02:09]

Patchset #1 (id:20001) has been deleted

[hirono, 2015-01-06 08:11:10]

hirono@chromium.org changed reviewers:
+ yawano@chromium.org

[hirono, 2015-01-06 08:11:11]

PTAL the CL? Thanks!

[yawano, 2015-01-06 08:40:02]

Thank you.

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
File ui/file_manager/gallery/js/gallery_item.js (right):

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/gallery_item.js:251:
this.metadataCache_.clear([this.entry_], 'fetchedMedia');
Since the file is changed by this operation, other types of metadata value seem
to be also invalid. How about to clear all types of metadata values by setting
type as *?

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/gallery_item.js:258: if (this.metadata_.filesystem)
Just for confirmation, event after the file write is ended, does the metadata
parser return old modification time or wrong present value?

[hirono, 2015-01-06 10:33:57]

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
File ui/file_manager/gallery/js/gallery_item.js (right):

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/gallery_item.js:251:
this.metadataCache_.clear([this.entry_], 'fetchedMedia');
On 2015/01/06 08:40:02, yawano wrote:
> Since the file is changed by this operation, other types of metadata value
seem
> to be also invalid. How about to clear all types of metadata values by setting
> type as *?

Done.

https://codereview.chromium.org/802283005/diff/40001/ui/file_manager/gallery/...
ui/file_manager/gallery/js/gallery_item.js:258: if (this.metadata_.filesystem)
On 2015/01/06 08:40:02, yawano wrote:
> Just for confirmation, event after the file write is ended, does the metadata
> parser return old modification time or wrong present value?

It looks the values are property updated. Removed the lines.

[yawano, 2015-01-06 10:37:45]

lgtm. Thank you!

[hirono, 2015-01-07 01:01:21]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-01-07 01:02:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/802283005/60001

[commit-bot: I haz the power, 2015-01-07 01:17:22]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2015-01-07 01:18:22]

Patchset 2 (id:??) landed as
https://crrev.com/699c53ebb13827f09e6cb5d4564f69da03d58692
Cr-Commit-Position: refs/heads/master@{#310205}