Add levels to media::VideoEncoder pathway. Use for H.264. [chromium/src : main]

0 views
Skip to first unread message

Dale Curtis (Gerrit)

unread,
May 1, 2025, 6:10:41 PMMay 1
to Chromium LUCI CQ, AyeAye, Hirokazu Honda, chromium...@chromium.org, blink-...@chromium.org, emircan+watch...@chromium.org, feature-me...@chromium.org, mcasas+med...@chromium.org
Attention needed from Hirokazu Honda

Dale Curtis added 1 comment

Patchset-level comments
File-level comment, Patchset 4 (Latest):
Dale Curtis . unresolved

Hiro, can you test to see if this resolve the issue on the device? Alternatively, if it's easier we can just land this and then have the test team verify.

Open in Gerrit

Related details

Attention is currently required from:
  • Hirokazu Honda
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I42712c96261b89f187b7418faa4687a3ca6ea81a
Gerrit-Change-Number: 6507220
Gerrit-PatchSet: 4
Gerrit-Owner: Dale Curtis <dalec...@chromium.org>
Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
Gerrit-Reviewer: Hirokazu Honda <hi...@chromium.org>
Gerrit-Attention: Hirokazu Honda <hi...@chromium.org>
Gerrit-Comment-Date: Thu, 01 May 2025 22:10:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy

Dale Curtis (Gerrit)

unread,
Oct 10, 2025, 6:16:08 PM (7 hours ago) Oct 10
to Code Review Nudger, Chromium LUCI CQ, AyeAye, Hirokazu Honda, chromium...@chromium.org, blink-...@chromium.org, emircan+watch...@chromium.org, feature-me...@chromium.org, mcasas+med...@chromium.org

Dale Curtis abandoned this change

Related details

Attention set is empty
Submit Requirements:
    • requirement satisfiedCode-Coverage
    • requirement is not satisfiedCode-Review
    • requirement is not satisfiedNo-Unresolved-Comments
    • requirement is not satisfiedReview-Enforcement
    Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
    Gerrit-MessageType: abandon
    Gerrit-Project: chromium/src
    Gerrit-Branch: main
    Gerrit-Change-Id: I42712c96261b89f187b7418faa4687a3ca6ea81a
    Gerrit-Change-Number: 6507220
    Gerrit-PatchSet: 4
    Gerrit-Owner: Dale Curtis <dalec...@chromium.org>
    Gerrit-Reviewer: Dale Curtis <dalec...@chromium.org>
    Gerrit-Reviewer: Hirokazu Honda <hi...@chromium.org>
    Gerrit-CC: Code Review Nudger <android-build...@prod.google.com>
    satisfied_requirement
    unsatisfied_requirement
    open
    diffy
    Reply all
    Reply to author
    Forward
    0 new messages