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

Side by Side Diff: chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.h

Issue 2335623002: Use WebContents instead of Browser to construct PermissionPrompt (Closed)
Patch Set: Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import <Cocoa/Cocoa.h> 5 #import <Cocoa/Cocoa.h>
6 6
7 #include "base/mac/scoped_nsobject.h" 7 #include "base/mac/scoped_nsobject.h"
8 #include "chrome/browser/ui/browser.h"
raymes 2016/09/15 02:06:46 nit: I think we can forward declare this rather th
lshang 2016/09/15 09:45:06 Done.
8 #import "chrome/browser/ui/cocoa/base_bubble_controller.h" 9 #import "chrome/browser/ui/cocoa/base_bubble_controller.h"
9 #include "chrome/browser/ui/website_settings/permission_prompt.h" 10 #include "chrome/browser/ui/website_settings/permission_prompt.h"
10 #include "ui/base/models/simple_menu_model.h" 11 #include "ui/base/models/simple_menu_model.h"
11 12
12 @class MenuController; 13 @class MenuController;
13 class PermissionBubbleCocoa; 14 class PermissionBubbleCocoa;
14 class PermissionRequest; 15 class PermissionRequest;
15 16
16 @interface PermissionBubbleController : 17 @interface PermissionBubbleController :
17 BaseBubbleController<NSTextViewDelegate> { 18 BaseBubbleController<NSTextViewDelegate> {
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
53 54
54 // Will calculate the expected anchor point for this bubble. 55 // Will calculate the expected anchor point for this bubble.
55 // Should only be used outside this class for tests. 56 // Should only be used outside this class for tests.
56 - (NSPoint)getExpectedAnchorPoint; 57 - (NSPoint)getExpectedAnchorPoint;
57 58
58 // Returns true if the browser has a visible location bar. 59 // Returns true if the browser has a visible location bar.
59 // Should only be used outside this class for tests. 60 // Should only be used outside this class for tests.
60 - (bool)hasVisibleLocationBar; 61 - (bool)hasVisibleLocationBar;
61 62
62 @end 63 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698