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

Issue 792123006: Revert of Generalize V4L2 HW video codec device names (Closed)

Created:
6 years ago by Pawel Osciak
Modified:
6 years ago
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, jln+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Revert of Generalize V4L2 HW video codec device names (patchset #3 id:40001 of https://codereview.chromium.org/786993002/) Reason for revert: Still hitting crbug.com/439175 on Pit. Original issue's description: > Generalize V4L2 HW video codec device names > > Switch to more generalized V4L2 video accelerator device names and update > sandbox rules to reflect that. > > After video_ChromeHWDecodeUsed autotest is fixed, this cl is submitted again > due to crrev.com/730693005 reverted in crrev.com/784553004. > > BUG=405861, 437598, 439175 > TEST=passed video_ChromeHWDecodeUsed with h264 and vp8 on peach_pit. > Media.GpuVideoDecoderInitializeStatus value is 0 when playing a video > from youtube. DUT only has /dev/video-enc and /dev/video-dec device nodes. > > Committed: https://crrev.com/b777ad3f16655a3493a0ecd9975c58a7b84c740d > Cr-Commit-Position: refs/heads/master@{#307674} TBR=jorgelo@chromium.org,wuchengli@chromium.org,scherkus@chromium.org,henryhsu@chromium.org NOTREECHECKS=true NOTRY=true BUG=405861, 437598, 439175 Committed: https://crrev.com/157c05e80bf3cdbd10a0c472e242745ca917fa8b Cr-Commit-Position: refs/heads/master@{#308257}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+281 lines, -283 lines) Patch
A content/common/gpu/media/exynos_v4l2_video_device.h View 1 chunk +58 lines, -0 lines 0 comments Download
A content/common/gpu/media/exynos_v4l2_video_device.cc View 1 chunk +211 lines, -0 lines 0 comments Download
D content/common/gpu/media/generic_v4l2_video_device.h View 1 chunk +0 lines, -58 lines 0 comments Download
D content/common/gpu/media/generic_v4l2_video_device.cc View 1 chunk +0 lines, -211 lines 0 comments Download
M content/common/gpu/media/v4l2_video_device.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M content/common/sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc View 1 chunk +6 lines, -8 lines 0 comments Download
M content/content_common.gypi View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Pawel Osciak
Created Revert of Generalize V4L2 HW video codec device names
6 years ago (2014-12-13 07:03:50 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/792123006/1
6 years ago (2014-12-13 07:04:48 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years ago (2014-12-13 07:05:34 UTC) #3
commit-bot: I haz the power
6 years ago (2014-12-13 07:06:18 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/157c05e80bf3cdbd10a0c472e242745ca917fa8b
Cr-Commit-Position: refs/heads/master@{#308257}

Powered by Google App Engine
This is Rietveld 408576698