[BAM] Introducing OpenLink ContentContext Commands [chromium/src : main]
0 views
Skip to first unread message
Darryl James (Gerrit)
4:00 PM (3 hours ago) 4:00 PM
to Kunal Daftari, Chromium LUCI CQ, chromium...@chromium.org
Attention needed from Kunal Daftari
Darryl James voted and added 1 comment![]( "Open in Gerrit")
Votes added by Darryl James
| Code-Review | +1 |
1 comment
Patchset-level comments
Related details
Attention is currently required from:
- Kunal Daftari
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Kunal Daftari (Gerrit)
4:01 PM (3 hours ago) 4:01 PM
to Darryl James, Chromium LUCI CQ, chromium...@chromium.org
Kunal Daftari voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
4:06 PM (3 hours ago) 4:06 PM
to Kunal Daftari, Darryl James, chromium...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[BAM] Introducing OpenLink ContentContext Commands
kActionContentContextOpenLinkNewTab, kActionContentContextOpenLinkNewWindow, kActionContentContextOpenLinkOffTheRecord.
Bug: 525118424
Change-Id: I6f9fa0d78cefb38c9b1e5d4c330d9b16bfa33822
Commit-Queue: Kunal Daftari <kunald...@google.com>
Reviewed-by: Darryl James <dlj...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1659015}
Files:
- M chrome/browser/ui/actions/chrome_action_properties.cc
- M chrome/browser/ui/actions/chrome_action_properties.h
- M chrome/browser/ui/browser_actions.cc
Change size: M
Delta: 3 files changed, 145 insertions(+), 0 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Darryl James
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages