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

Issue 825863002: WIP: Add hardware jpeg decode on tegra platform

Created:
5 years, 12 months ago by rbhagwat
Modified:
5 years, 3 months ago
Reviewers:
scroggo, Pawel Osciak, reed1
CC:
chromium-reviews, posciak+watch_chromium.org, jam, mcasas+watch_chromium.org, feature-media-reviews_chromium.org, darin-cc_chromium.org, piman+watch_chromium.org, wjia+watch_chromium.org, kcwu, Pawel Osciak
Base URL:
https://chromium.googlesource.com/chromium/src.git@lkgr
Target Ref:
refs/pending/heads/lkgr
Project:
chromium
Visibility:
Public.

Description

WIP: Add hardware jpeg decode on tegra platform This change adds a jpeg hardware decode support on tegra platform. And implements generic jpeg codec interface which can load platform specific jpeg codec. BUG=none TEST=JpegDecoderTest

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+369 lines, -0 lines) Patch
A content/common/gpu/media/jpeg_codec.h View 1 chunk +48 lines, -0 lines 0 comments Download
A content/common/gpu/media/jpeg_codec.cc View 1 chunk +26 lines, -0 lines 0 comments Download
A content/common/gpu/media/jpeg_codec_unittest.cc View 1 chunk +119 lines, -0 lines 0 comments Download
A content/common/gpu/media/tegra_jpeg_codec_device.h View 1 chunk +41 lines, -0 lines 0 comments Download
A content/common/gpu/media/tegra_jpeg_codec_device.cc View 1 chunk +130 lines, -0 lines 0 comments Download
M content/content_common.gypi View 1 chunk +4 lines, -0 lines 0 comments Download
M content/content_tests.gypi View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
reed1
5 years, 11 months ago (2015-01-13 14:07:28 UTC) #3
reed1
content/common/gpu/media/jpeg_codec Skia also has wrappers for decoding and encoding images. Do we need both?
5 years, 11 months ago (2015-01-13 14:09:41 UTC) #4
scroggo
On 2015/01/13 14:09:41, reed1 wrote: > content/common/gpu/media/jpeg_codec > > Skia also has wrappers for decoding ...
5 years, 11 months ago (2015-01-13 22:17:33 UTC) #5
Pawel Osciak
not lgtm I am sorry, but we already have an interface for JPEG decoders - ...
5 years, 11 months ago (2015-01-14 00:02:22 UTC) #6
Kimmo Kinnunen
5 years, 3 months ago (2015-08-31 12:30:26 UTC) #8

          

Powered by Google App Engine
This is Rietveld 408576698