Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't raise error in HTMLMediaElement.h5vccAudioConnectors #2687

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

zhongqiliang
Copy link
Contributor

b/330929788

@datadog-cobalt-youtube
Copy link

datadog-cobalt-youtube bot commented Mar 23, 2024

Datadog Report

Branch report: b-330929788_HTMLMediaElement_idl
Commit report: 544efc5
Test service: cobalt

✅ 0 Failed, 29880 Passed, 1 Skipped, 10m 21.81s Wall Time

Copy link

codecov bot commented Mar 23, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 58.88%. Comparing base (62f49e9) to head (544efc5).
Report is 23 commits behind head on main.

Files Patch % Lines
cobalt/dom/html_media_element.cc 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2687      +/-   ##
==========================================
+ Coverage   58.24%   58.88%   +0.64%     
==========================================
  Files        1794     1811      +17     
  Lines       84889    87501    +2612     
==========================================
+ Hits        49443    51526    +2083     
- Misses      35446    35975     +529     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@xiaomings xiaomings left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also consider refining the commit message to be more specific.

cobalt/dom/html_media_element.idl Outdated Show resolved Hide resolved
cobalt/dom/html_media_element.cc Outdated Show resolved Hide resolved
@zhongqiliang zhongqiliang force-pushed the b-330929788_HTMLMediaElement_idl branch from bf17afe to 544efc5 Compare March 25, 2024 17:16
@zhongqiliang zhongqiliang changed the title Change h5vccAudioConnectors attribute in html_media_element.idl Don't raise error in HTMLMediaElement.h5vccAudioConnectors Mar 25, 2024
@zhongqiliang zhongqiliang merged commit cb7bc13 into main Mar 25, 2024
349 of 352 checks passed
@zhongqiliang zhongqiliang deleted the b-330929788_HTMLMediaElement_idl branch March 25, 2024 23:50
@zhongqiliang zhongqiliang added cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch labels Mar 25, 2024
cobalt-github-releaser-bot pushed a commit that referenced this pull request Mar 25, 2024
b/330929788

Co-authored-by: Colin Liang <[email protected]>
(cherry picked from commit cb7bc13)
cobalt-github-releaser-bot pushed a commit that referenced this pull request Mar 25, 2024
b/330929788

Co-authored-by: Colin Liang <[email protected]>
(cherry picked from commit cb7bc13)
zhongqiliang added a commit that referenced this pull request Mar 26, 2024
b/330929788

Co-authored-by: Colin Liang <[email protected]>
(cherry picked from commit cb7bc13)
zhongqiliang added a commit that referenced this pull request Apr 6, 2024
b/330929788

Co-authored-by: Colin Liang <[email protected]>
(cherry picked from commit cb7bc13)
zhongqiliang added a commit that referenced this pull request Apr 6, 2024
…oConnectors (#2715)

Refer to the original PR: #2687

b/330929788

Co-authored-by: Colin Liang <[email protected]>
zhongqiliang added a commit that referenced this pull request Apr 8, 2024
…oConnectors (#2714)

Refer to the original PR: #2687

b/330929788

Co-authored-by: Colin Liang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-23.lts.1+ Cherry Pick to the 23.lts.1+ branch cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants