fix(video): fix leaky abstraction in ogl luma shader

[ns6089]

Description

The shader ignored part of color conversion matrix. Should hopefully fix #4403

Screenshot

Issues Fixed or Closed

Roadmap Issues

Type of Change

• feat: New feature (non-breaking change which adds functionality)
• fix: Bug fix (non-breaking change which fixes an issue)
• docs: Documentation only changes
• style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
• refactor: Code change that neither fixes a bug nor adds a feature
• perf: Code change that improves performance
• test: Adding missing tests or correcting existing tests
• build: Changes that affect the build system or external dependencies
• ci: Changes to CI configuration files and scripts
• chore: Other changes that don't modify src or test files
• revert: Reverts a previous commit
• BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

• Code follows the style guidelines of this project
• Code has been self-reviewed
• Code has been commented, particularly in hard-to-understand areas
• Code docstring/documentation-blocks for new or existing methods/components have been added or updated
• Unit tests have been added or updated for any new or modified functionality

AI Usage

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Bundle Report

Bundle size has no change ✅

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 12.08%. Comparing base (8836db5) to head (13f9ebf).
⚠️ Report is 102 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff            @@
##           master    #4404    +/-   ##
========================================
  Coverage   12.08%   12.08%            
========================================
  Files          87       87            
  Lines       17595    17595            
  Branches     8096     8096            
========================================
  Hits         2126     2126            
- Misses      14566    14748   +182     
+ Partials      903      721   -182     

Flag	Coverage Δ
Linux-AppImage	11.57% <ø> (ø)
Windows-AMD64	13.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 21 files with indirect coverage changes

[ns6089]

@ReenigneArcher Could you please disable newline linter until it stops triggering on existing code? If I add newline to the modified file, we end up with inconsistent files. If I modify other files too, we end with unrelated git log message.

[ReenigneArcher]

@ns6089 this linter only runs on modified files. The linter is not intelligent enough to know if the issue existed previously or not.

I can ignore the rule for this PR though. Ideally these get fixed over time.