Fully relax AVC/HEVC analysis classes. Used only for logs/keyframes [chromium/src : main]
0 views
Skip to first unread message
Dale Curtis (Gerrit)
Jun 18, 2026, 6:34:50 PM (10 days ago) Jun 18
to Eugene Zemtsov, chromium...@chromium.org, feature-me...@chromium.org
Attention needed from Eugene Zemtsov
Dale Curtis voted![]( "Open in Gerrit")
| Auto-Submit | +1 |
| Commit-Queue | +1 |
Related details
Attention is currently required from:
- Eugene Zemtsov
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Eugene Zemtsov (Gerrit)
Jun 26, 2026, 4:24:56 PM (2 days ago) Jun 26
to Dale Curtis, Eugene Zemtsov, Code Review Nudger, Chromium LUCI CQ, chromium...@chromium.org, feature-me...@chromium.org
Attention needed from Dale Curtis
Eugene Zemtsov voted![]( "Open in Gerrit")
Attention is currently required from:
- Dale Curtis
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Jun 26, 2026, 5:34:44 PM (2 days ago) Jun 26
to Dale Curtis, Eugene Zemtsov, Code Review Nudger, chromium...@chromium.org, feature-me...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Fully relax AVC/HEVC analysis classes. Used only for logs/keyframes
The analysis portions of the AVC/HEVC classes are only used for
generating non-conformant logs and key-frame detection. As such it
doesn't really make sense to bail out early upon non-conformance when
the packet may still be a key-frame. This change fully relaxes the
parsers so that both is_conformant and is_keyframe will be populated.
R=eugene
Fixed: 519012430
Change-Id: I1e2265c6a8b3e26ed3680b7e191a0c87e8942a0f
Auto-Submit: Dale Curtis <dalec...@chromium.org>
Commit-Queue: Eugene Zemtsov <eug...@chromium.org>
Reviewed-by: Eugene Zemtsov <eug...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1653458}
Files:
- M media/formats/mp4/avc.cc
- M media/formats/mp4/avc.h
- M media/formats/mp4/avc_unittest.cc
- M media/formats/mp4/hevc.cc
- M media/formats/mp4/hevc.h
- M media/formats/mp4/hevc_unittest.cc
Change size: M
Delta: 6 files changed, 90 insertions(+), 59 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Eugene Zemtsov
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages