Delete unused extensions::Bloсklist::GetMalwareIds [chromium/src : main]

0 views

Skip to first unread message

Alexander Kalugin (Gerrit)

unread,

6:01 AM (11 hours ago) 6:01 AM

to Finnur Thorarinsson, Devlin Cronin, Chromium LUCI CQ, chromium...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org

Attention needed from Devlin Cronin

Alexander Kalugin added 1 comment

Commit Message

Line 7, Patchset 1:Delete unused extensions::Blobklist::GetMalwareIds

Finnur Thorarinsson . resolved

Typo nit: s/Blobklist/Blocklist/

Alexander Kalugin

Thanks! Fixed.

Open in Gerrit

Related details

Attention is currently required from:

Devlin Cronin

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

Devlin Cronin (Gerrit)

unread,

1:57 PM (3 hours ago) 1:57 PM

to Alexander Kalugin, Devlin Cronin, Finnur Thorarinsson, Chromium LUCI CQ, chromium...@chromium.org, chromium-a...@chromium.org, extension...@chromium.org

Attention needed from Alexander Kalugin

Devlin Cronin voted and added 1 comment

Votes added by Devlin Cronin

Code-Review

1 comment

Patchset-level comments

File-level comment, Patchset 2 (Latest):

Devlin Cronin . resolved

LGTM! Thank you for the cleanup, Alexander!

Open in Gerrit

Related details

Attention is currently required from:

Alexander Kalugin

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

open

diffy

Reply all

Reply to author

Forward

0 new messages