Media: Add thumbnail extraction to AudioVideoMetadataExtractor.

Media: Add thumbnail extraction to AudioVideoMetadataExtractor.

Spinoff of patch https://codereview.chromium.org/250143002/

For trivial change in chrome/utility/media_galleries/media_metadata_parser.cc:
TBR=vandebo

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

[tommycli, 2014-05-08 20:50:03]

dalecurtis: PTAL

This is a spinoff of patch https://codereview.chromium.org/250143002/, which was
getting too gigantic.

[DaleCurtis, 2014-05-08 23:02:24]

lgtm

https://codereview.chromium.org/270873003/diff/20001/media/base/audio_video_m...
File media/base/audio_video_metadata_extractor_unittest.cc (right):

https://codereview.chromium.org/270873003/diff/20001/media/base/audio_video_m...
media/base/audio_video_metadata_extractor_unittest.cc:197:
EXPECT_EQ("\x89PNG\r\n\x1a\n",
You can use base::MD5 or base::Sha1 to hash the image for a more complete check
if you want.

[tommycli, 2014-05-08 23:20:42]

https://codereview.chromium.org/270873003/diff/20001/media/base/audio_video_m...
File media/base/audio_video_metadata_extractor_unittest.cc (right):

https://codereview.chromium.org/270873003/diff/20001/media/base/audio_video_m...
media/base/audio_video_metadata_extractor_unittest.cc:197:
EXPECT_EQ("\x89PNG\r\n\x1a\n",
On 2014/05/08 23:02:25, DaleCurtis wrote:
> You can use base::MD5 or base::Sha1 to hash the image for a more complete
check
> if you want.

Done.

[tommycli, 2014-05-08 23:22:28]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2014-05-08 23:27:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tommycli@chromium.org/270873003/40001

[tommycli, 2014-05-09 15:18:26]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2014-05-09 15:18:27]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2014-05-09 15:22:39]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tommycli@chromium.org/270873003/40001

[tommycli, 2014-05-09 22:19:12]

The CQ bit was unchecked by tommycli@chromium.org

[tommycli, 2014-05-09 22:19:12]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2014-05-09 22:53:53]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2014-05-09 22:56:57]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tommycli@chromium.org/270873003/40001

[commit-bot: I haz the power, 2014-05-10 18:14:25]

Change committed as 269588