views: Handle null DialogDelegate in DialogClientView layout [chromium/src : main]

0 views

Skip to first unread message

gwsq (Gerrit)

unread,

Aug 6, 2026, 10:20:22 PM (3 days ago) Aug 6

to Chunyang Huo, Chromium UI Views Reviews, Thomas Lukaszewicz, Chromium LUCI CQ, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, roblia...@chromium.org, sky+...@chromium.org

Attention needed from Thomas Lukaszewicz

Message from gwsq

Reviewer source(s):
tl...@chromium.org is from context(googleclient/chrome/chromium_gwsq/ui/views/config.gwsq)

Open in Gerrit

Related details

Attention is currently required from:

Thomas Lukaszewicz

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

Thomas Lukaszewicz (Gerrit)

unread,

4:33 PM (4 hours ago) 4:33 PM

to Chunyang Huo, David Yeung, Chromium UI Views Reviews, Chromium LUCI CQ, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, roblia...@chromium.org, sky+...@chromium.org

Attention needed from Chunyang Huo and David Yeung

Thomas Lukaszewicz voted and added 1 comment

Votes added by Thomas Lukaszewicz

Code-Review

1 comment

Patchset-level comments

File-level comment, Patchset 2 (Latest):

Thomas Lukaszewicz . resolved

lgtm! David ptal