Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3237)

Unified Diff: chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h

Issue 2752623003: [Mac] Fix for the security chip (Closed)
Patch Set: Rebased Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h
diff --git a/chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h b/chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h
index 5dec145b6d2e940aae39e07d530c174f7f1741ba..18bc0ef4ab165e7c9a27507e1e72033d5bc3db6b 100644
--- a/chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h
+++ b/chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.h
@@ -8,17 +8,18 @@
#import <Cocoa/Cocoa.h>
#include "base/mac/scoped_nsobject.h"
-#import "chrome/browser/ui/cocoa/omnibox_decoration_bubble_controller.h"
+#import "chrome/browser/ui/cocoa/base_bubble_controller.h"
#include "chrome/browser/ui/permission_bubble/permission_prompt.h"
#include "ui/base/models/simple_menu_model.h"
class Browser;
+class LocationBarDecoration;
@class MenuController;
class PermissionBubbleCocoa;
class PermissionRequest;
@interface PermissionBubbleController
- : OmniboxDecorationBubbleController<NSTextViewDelegate> {
+ : BaseBubbleController<NSTextViewDelegate> {
@private
// Array of views that are the checkboxes for every requested permission.
// Only populated if multiple requests are shown at once.
@@ -35,6 +36,12 @@ class PermissionRequest;
// Bridge to the C++ class that created this object.
PermissionBubbleCocoa* bridge_; // Weak.
+
+ // The omnibox icon the bubble is anchored to. The icon is set as active
+ // when the bubble is opened, and inactive when the bubble is closed.
+ // Usually we would override OmniboxDecorationBubbleController but the page
+ // info icon has a special case where it might cause a race condition.
+ LocationBarDecoration* decoration_; // Weak.
}
// Designated initializer. |browser| and |bridge| must both be non-nil.
« no previous file with comments | « no previous file | chrome/browser/ui/cocoa/permission_bubble/permission_bubble_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698