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

Unified Diff: chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm

Issue 2036503002: Correctly move permission bubbles after entering / exiting fullscreen. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tapted feedback Created 4 years, 7 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 | « chrome/browser/ui/cocoa/info_bubble_view.mm ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm b/chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm
index 72bfbf8f9d7e5f95d1308577dac9c489ffcbfc11..6067243821fc2967ef9fe83b42e3dc6187a56763 100644
--- a/chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/website_settings/permission_bubble_controller.mm
@@ -301,7 +301,7 @@ const NSSize kPermissionIconSize = {18, 18};
- (void)parentWindowDidResize:(NSNotification*)notification {
DCHECK(bridge_);
- [self setAnchorPoint:[self getExpectedAnchorPoint]];
+ [self updateAnchorPosition];
}
- (void)parentWindowDidMove:(NSNotification*)notification {
@@ -471,6 +471,7 @@ const NSSize kPermissionIconSize = {18, 18};
- (void)updateAnchorPosition {
[self setParentWindow:[self getExpectedParentWindow]];
[self setAnchorPoint:[self getExpectedAnchorPoint]];
+ [[self bubble] setArrowLocation:[self getExpectedArrowLocation]];
}
- (NSPoint)getExpectedAnchorPoint {
« no previous file with comments | « chrome/browser/ui/cocoa/info_bubble_view.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698