Abstract codec and format for local image test cases [chromium/src : main]

0 views

Skip to first unread message

Michael Cui (Gerrit)

unread,

Jul 4, 2024, 1:46:40 AM (yesterday) Jul 4

to Michael Cui, Darren Shen, AyeAye, feature-me...@chromium.org

Attention needed from Darren Shen

Michael Cui voted Commit-Queue+1

Commit-Queue

Open in Gerrit

Related details

Attention is currently required from:

Darren Shen

Submit Requirements:

Code-Coverage
Code-Review

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

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Darren Shen (Gerrit)

unread,

Jul 4, 2024, 6:48:26 PM (10 hours ago) Jul 4

to Michael Cui, Chromium LUCI CQ, AyeAye, feature-me...@chromium.org

Attention needed from Michael Cui

Darren Shen voted and added 1 comment

Votes added by Darren Shen

Code-Review

1 comment

File ash/picker/picker_insert_media_request_unittest.cc

Line 116, Patchset 3 (Latest): temp_dir_.GetPath().Append(base::StrCat({"test_image.", format_}));

Darren Shen . unresolved

Q: is there some proper base::Path thing to add an extension to a file name?