Helmut Januschka (Gerrit)

unread,

Dec 25, 2025, 5:19:26 PM12/25/25

to Helmut Januschka, ccameron chromium, Vladimir Levin, Dave Tapuska, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Helmut Januschka added 1 comment

Patchset-level comments

File-level comment, Patchset 55:

ccameron chromium . resolved

I would also highly recommend breaking this into a couple CLs.
Like -- you can land a CL that just adds the flags and various enum values for JXL, while not actually adding the codec right now, cause that stuff is mostly mechanical.
Then a second CL could add the codec decoder itself (which would need more scrutiny). And also, if there is any build breakage, the revert would be less painful.

Helmut Januschka

thanks, did it now, feel free to tell me if the split you had in mind was different!

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Luca Versari
Philip Jägenstedt
ccameron chromium
mmenke

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Helmut Januschka (Gerrit)

unread,

Dec 26, 2025, 5:01:28 PM12/26/25

to Helmut Januschka, ccameron chromium, Vladimir Levin, Dave Tapuska, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Helmut Januschka added 1 comment

Patchset-level comments

File-level comment, Patchset 67 (Latest):

Helmut Januschka . resolved

ready for re-review 😊

satisfied_requirement

unsatisfied_requirement

open

diffy

Honey B (Gerrit)

unread,

Jan 4, 2026, 1:55:41 AM (8 days ago) Jan 4

to Helmut Januschka, ccameron chromium, Vladimir Levin, Dave Tapuska, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Helmut Januschka, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Honey B added 1 comment

File third_party/blink/renderer/platform/graphics/bitmap_image_metrics.h

Line 33, Patchset 70 (Latest): kMaxValue = kJXL,

Honey B . unresolved

Should also update [this](https://source.chromium.org/chromium/chromium/src/+/main:tools/metrics/histograms/metadata/blink/enums.xml;l=214;drc=2b547e7e232bbad5d46a04e466757dfc96de93f8)?

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Helmut Januschka

Luca Versari
Philip Jägenstedt
ccameron chromium
mmenke

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review

No-Unresolved-Comments

Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

mmenke (Gerrit)

unread,

Jan 5, 2026, 11:03:21 AM (6 days ago) Jan 5

to Helmut Januschka, Honey B, ccameron chromium, Vladimir Levin, Dave Tapuska, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Helmut Januschka, Luca Versari, Philip Jägenstedt and ccameron chromium

mmenke added 1 comment

File third_party/blink/common/loader/network_utils.cc

Line 37, Patchset 70 (Latest):

#if BUILDFLAG(ENABLE_JXL_DECODER) && BUILDFLAG(ENABLE_AV1_DECODER)
  return "image/jxl,image/avif,image/webp,image/apng,image/svg+xml,image/*,*/"
         "*;q=0.8";
#elif BUILDFLAG(ENABLE_JXL_DECODER)
  return "image/jxl,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8";

mmenke . unresolved

What if the feature is disabled? We should probably not include jxl, no? That does mean we need runtime checks here in addition to the build-time checks, unfortunately, but think they're needed. We probably also need unit test - admittedly, not very exciting ones. Fine to just hard code these strings there as well.

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Helmut Januschka
Luca Versari
Philip Jägenstedt
ccameron chromium

satisfied_requirement

unsatisfied_requirement

open

diffy

Dave Tapuska (Gerrit)

unread,

Jan 5, 2026, 1:46:20 PM (6 days ago) Jan 5

to Helmut Januschka, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Hans Wennborg, Helmut Januschka, Luca Versari, Philip Jägenstedt and ccameron chromium

Dave Tapuska voted and added 1 comment

Votes added by Dave Tapuska

Code-Review

+1

1 comment

File cc/base/devtools_instrumentation.cc

Line 109, Patchset 70 (Latest): histogram_name = "Renderer4.ImageDecodeTaskDurationUs.Jxl";

Dave Tapuska . unresolved

Update the variants here: https://source.chromium.org/chromium/chromium/src/+/main:tools/metrics/histograms/metadata/renderer4/histograms.xml;l=153?q=ImageDecodeTaskDurationUs&ss=chromium

Open in Gerrit

Related details

Attention is currently required from:

Hans Wennborg
Helmut Januschka
Luca Versari
Philip Jägenstedt
ccameron chromium

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review

No-Unresolved-Comments

Review-Enforcement

satisfied_requirement

unsatisfied_requirement

open

diffy

Helmut Januschka (Gerrit)

unread,

Jan 9, 2026, 2:20:13 AM (3 days ago) Jan 9

to Helmut Januschka, Dave Tapuska, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Honey B, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Helmut Januschka added 4 comments

Patchset-level comments

File-level comment, Patchset 78 (Latest):

Helmut Januschka . resolved

bots are all green now, please let me know if you want me to address anything!

File cc/base/devtools_instrumentation.cc

Line 109, Patchset 70: histogram_name = "Renderer4.ImageDecodeTaskDurationUs.Jxl";

Dave Tapuska . resolved

Update the variants here: https://source.chromium.org/chromium/chromium/src/+/main:tools/metrics/histograms/metadata/renderer4/histograms.xml;l=153?q=ImageDecodeTaskDurationUs&ss=chromium

Helmut Januschka

Done

File third_party/blink/common/loader/network_utils.cc

Line 37, Patchset 70:#if BUILDFLAG(ENABLE_JXL_DECODER) && BUILDFLAG(ENABLE_AV1_DECODER)


  return "image/jxl,image/avif,image/webp,image/apng,image/svg+xml,image/*,*/"
         "*;q=0.8";
#elif BUILDFLAG(ENABLE_JXL_DECODER)
  return "image/jxl,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8";

mmenke . resolved

What if the feature is disabled? We should probably not include jxl, no? That does mean we need runtime checks here in addition to the build-time checks, unfortunately, but think they're needed. We probably also need unit test - admittedly, not very exciting ones. Fine to just hard code these strings there as well.

Helmut Januschka

Done. Added runtime checks for `features::kJXLImageFormat` in both `ImageAcceptHeader()` and `FrameAcceptHeaderValue()`.

File third_party/blink/renderer/platform/graphics/bitmap_image_metrics.h

Line 33, Patchset 70: kMaxValue = kJXL,

Honey B . resolved

Should also update [this](https://source.chromium.org/chromium/chromium/src/+/main:tools/metrics/histograms/metadata/blink/enums.xml;l=214;drc=2b547e7e232bbad5d46a04e466757dfc96de93f8)?

Helmut Januschka

Done

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Honey B

Luca Versari
Philip Jägenstedt
ccameron chromium

mmenke

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Moritz Firsching (Gerrit)

unread,

Jan 9, 2026, 4:29:02 AM (3 days ago) Jan 9

to Helmut Januschka, Chrome Metrics Logs, Dave Tapuska, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Helmut Januschka, Honey B, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Moritz Firsching added 1 comment

Patchset-level comments

File-level comment, Patchset 78 (Latest):

Moritz Firsching . resolved

LGTM

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Helmut Januschka

Honey B
Luca Versari
Philip Jägenstedt
ccameron chromium
mmenke

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Dave Tapuska (Gerrit)

unread,

Jan 9, 2026, 10:37:07 AM (2 days ago) Jan 9

to Helmut Januschka, Moritz Firsching, Chrome Metrics Logs, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Hans Wennborg, Helmut Januschka, Honey B, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Dave Tapuska voted Code-Review+1

Code-Review

+1

Open in Gerrit

Related details

Attention is currently required from:

Hans Wennborg
Helmut Januschka
Honey B
Luca Versari
Philip Jägenstedt
ccameron chromium
mmenke

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review
Review-Enforcement

satisfied_requirement

unsatisfied_requirement

open

diffy

mmenke (Gerrit)

unread,

Jan 9, 2026, 11:00:09 AM (2 days ago) Jan 9

to Helmut Januschka, Dave Tapuska, Moritz Firsching, Chrome Metrics Logs, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Hans Wennborg, Helmut Januschka, Honey B, Luca Versari, Philip Jägenstedt and ccameron chromium

mmenke voted and added 1 comment

Votes added by mmenke

Code-Review

+1

1 comment

Patchset-level comments

File-level comment, Patchset 78 (Latest):

mmenke . resolved

Still LGTM

Open in Gerrit

Related details

Attention is currently required from:

Hans Wennborg
Helmut Januschka
Honey B
Luca Versari
Philip Jägenstedt
ccameron chromium

satisfied_requirement

unsatisfied_requirement

open

diffy

Honey B (Gerrit)

unread,

12:21 AM (20 hours ago) 12:21 AM

to Helmut Januschka, Dave Tapuska, Moritz Firsching, Chrome Metrics Logs, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Helmut Januschka, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Honey B added 1 comment

File third_party/blink/renderer/platform/blink_platform_unittests_bundle_data.filelist

Line 149, Patchset 79 (Latest):../../web_tests/images/resources/5_frames_numbered.jxl

Honey B . unresolved

Should keep the list sorted. You can use `build/ios/update_bundle_filelist.py` to update this file.

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Helmut Januschka

Luca Versari
Philip Jägenstedt
ccameron chromium

mmenke

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review
No-Unresolved-Comments
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Helmut Januschka (Gerrit)

unread,

6:10 AM (14 hours ago) 6:10 AM

to Helmut Januschka, Dave Tapuska, Moritz Firsching, Chrome Metrics Logs, Honey B, ccameron chromium, Vladimir Levin, Luca Versari, Hans Wennborg, Łukasz Anforowicz, Dale Curtis, Chromium Metrics Reviews, Stephen Chenney, Dirk Schulze, Philip Jägenstedt, AyeAye, Chromium LUCI CQ, chromium...@chromium.org, devtools...@chromium.org, kinuko+...@chromium.org, jmedle...@chromium.org, asvitki...@chromium.org, jshin...@chromium.org, fserb...@chromium.org, asvitkine...@chromium.org, drott+bl...@chromium.org, csharris...@chromium.org, blink-reviews-p...@chromium.org, blink-re...@chromium.org, fmalit...@chromium.org, bmcquad...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, bnc+...@chromium.org, cblume+im...@chromium.org, cc-...@chromium.org, devtools-re...@chromium.org, feature-me...@chromium.org, fuzzin...@chromium.org, kinuko...@chromium.org, mbarowsky+watc...@chromium.org, net-r...@chromium.org

Attention needed from Dave Tapuska, Hans Wennborg, Honey B, Luca Versari, Philip Jägenstedt, ccameron chromium and mmenke

Helmut Januschka added 1 comment

File third_party/blink/renderer/platform/blink_platform_unittests_bundle_data.filelist

Line 149, Patchset 79:../../web_tests/images/resources/5_frames_numbered.jxl

Honey B . resolved

Should keep the list sorted. You can use `build/ios/update_bundle_filelist.py` to update this file.

Helmut Januschka

Done

Open in Gerrit

Related details

Attention is currently required from:

Dave Tapuska
Hans Wennborg
Honey B

Luca Versari
Philip Jägenstedt
ccameron chromium
mmenke

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review

Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Reply all

Reply to author

Forward

Wire up JXL decoder. [chromium/src : main]

Helmut Januschka (Gerrit)

Helmut Januschka added 1 comment

Related details

Helmut Januschka (Gerrit)

Helmut Januschka added 1 comment

Honey B (Gerrit)

Honey B added 1 comment

Related details

mmenke (Gerrit)

mmenke added 1 comment

Related details

Dave Tapuska (Gerrit)

Dave Tapuska voted and added 1 comment

Votes added by Dave Tapuska

1 comment

Related details

Helmut Januschka (Gerrit)

Helmut Januschka added 4 comments

Related details

Moritz Firsching (Gerrit)

Moritz Firsching added 1 comment

Related details

Dave Tapuska (Gerrit)

Dave Tapuska voted Code-Review+1

Related details

mmenke (Gerrit)

mmenke voted and added 1 comment

Votes added by mmenke

1 comment

Related details

Honey B (Gerrit)

Honey B added 1 comment

Related details

Helmut Januschka (Gerrit)

Helmut Januschka added 1 comment

Related details