Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Issue 8771037: File Browser Exif parser: support more exotic format variants (Closed)

Created:
9 years ago by Vladislav Kaznacheev
Modified:
9 years ago
Reviewers:
dgozman
CC:
chromium-reviews, rginda+watch_chromium.org, arv (Not doing code reviews), achuith+watch_chromium.org
Visibility:
Public.

Description

File Browser Exif parser: support more exotic format variants BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112972

Patch Set 1 #

Patch Set 2 : EOF check for corrupt files #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -1 line) Patch
M chrome/browser/resources/file_manager/js/exif_parser.js View 1 1 chunk +7 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Vladislav Kaznacheev
Please review
9 years ago (2011-12-02 15:32:15 UTC) #1
dgozman
LGTM
9 years ago (2011-12-05 09:08:39 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kaznacheev@chromium.org/8771037/4
9 years ago (2011-12-05 09:11:14 UTC) #3
commit-bot: I haz the power
9 years ago (2011-12-05 11:15:08 UTC) #4
Change committed as 112972

Powered by Google App Engine
This is Rietveld 408576698