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

Side by Side Diff: chrome/browser/ui/cocoa/certificate_viewer_mac_cocoa.mm

Issue 1779383002: MacViews: Remove constrained window dependencies for certificate viewer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase & address review comments. Created 4 years, 6 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
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #import "chrome/browser/ui/cocoa/certificate_viewer_mac_cocoa.h"
6
7 #include "base/logging.h"
8 #import "base/mac/foundation_util.h"
9 #include "chrome/browser/certificate_viewer.h"
10 #import "chrome/browser/ui/cocoa/constrained_window/constrained_window_mac.h"
11 #import "chrome/browser/ui/cocoa/constrained_window/constrained_window_sheet_con troller.h"
12
13 @interface SSLCertificateViewerCocoa ()
14 - (void)displayForWebContents:(content::WebContents*)webContents;
15 - (void)onConstrainedWindowClosed;
16 @end
17
18 namespace {
19
20 class SSLCertificateViewerCocoaBridge : public ConstrainedWindowMacDelegate {
21 public:
22 explicit SSLCertificateViewerCocoaBridge(
23 SSLCertificateViewerCocoa* controller)
24 : controller_(controller) {}
25
26 // ConstrainedWindowMacDelegate:
27 void OnConstrainedWindowClosed(ConstrainedWindowMac* window) override {
28 // |onConstrainedWindowClosed| will delete the sheet which might still be
29 // in use higher up the call stack. Wait for the next cycle of the event
30 // loop to call this function.
31 [controller_ performSelector:@selector(onConstrainedWindowClosed)
32 withObject:nil
33 afterDelay:0];
34 }
35
36 private:
37 SSLCertificateViewerCocoa* controller_; // Weak. Owns this.
38
39 DISALLOW_COPY_AND_ASSIGN(SSLCertificateViewerCocoaBridge);
40 };
41
42 } // namespace
43
44 @implementation SSLCertificateViewerCocoa {
45 std::unique_ptr<SSLCertificateViewerCocoaBridge> observer_;
46 std::unique_ptr<ConstrainedWindowMac> constrainedWindow_;
47 base::scoped_nsobject<NSWindow> overlayWindow_;
48 // A copy of the sheet's frame. Used to restore on show.
49 NSRect oldSheetFrame_;
50 BOOL closePending_;
51 // A copy of the sheet's |autoresizesSubviews| flag to restore on show.
52 BOOL oldResizesSubviews_;
53 }
54
55 - (NSWindow*)overlayWindow {
56 return overlayWindow_;
57 }
58
59 - (void)displayForWebContents:(content::WebContents*)webContents {
60 [super displayForWebContents:webContents];
61
62 observer_.reset(new SSLCertificateViewerCocoaBridge(self));
63 constrainedWindow_ =
64 CreateAndShowWebModalDialogMac(observer_.get(), webContents, self);
65 }
66
67 - (void)onConstrainedWindowClosed {
68 [self releaseSheetWindow];
69 constrainedWindow_.reset();
70 [self release];
71 }
72
73 // SSLCertificateViewerMac overrides:
74
75 - (void)sheetDidEnd:(NSWindow*)parent
76 returnCode:(NSInteger)returnCode
77 context:(void*)context {
78 if (!closePending_)
79 constrainedWindow_->CloseWebContentsModalDialog();
80 }
81
82 // ConstrainedWindowSheet protocol implementation.
83
84 - (void)showSheetForWindow:(NSWindow*)window {
85 overlayWindow_.reset([window retain]);
86 [self showCertificateSheet:window];
87 }
88
89 - (void)closeSheetWithAnimation:(BOOL)withAnimation {
90 closePending_ = YES;
91 overlayWindow_.reset();
92 [self closeCertificateSheet];
93 }
94
95 - (void)hideSheet {
96 NSWindow* sheetWindow = [overlayWindow_ attachedSheet];
97 [sheetWindow setAlphaValue:0.0];
98 [sheetWindow setIgnoresMouseEvents:YES];
99
100 oldResizesSubviews_ = [[sheetWindow contentView] autoresizesSubviews];
101 [[sheetWindow contentView] setAutoresizesSubviews:NO];
102 }
103
104 - (void)unhideSheet {
105 NSWindow* sheetWindow = [overlayWindow_ attachedSheet];
106 [sheetWindow setIgnoresMouseEvents:NO];
107
108 [[sheetWindow contentView] setAutoresizesSubviews:oldResizesSubviews_];
109 [[overlayWindow_ attachedSheet] setAlphaValue:1.0];
110 }
111
112 - (void)pulseSheet {
113 // NOOP
114 }
115
116 - (void)makeSheetKeyAndOrderFront {
117 [[overlayWindow_ attachedSheet] makeKeyAndOrderFront:nil];
118 }
119
120 - (void)updateSheetPosition {
121 // NOOP
122 }
123
124 - (void)resizeWithNewSize:(NSSize)preferredSize {
125 // NOOP
126 }
127
128 - (NSWindow*)sheetWindow {
129 return [self certificatePanel];
130 }
131
132 @end
133
134 void ShowCertificateViewer(content::WebContents* web_contents,
135 gfx::NativeWindow parent,
136 net::X509Certificate* cert) {
137 // SSLCertificateViewerCocoa will manage its own lifetime and will release
138 // itself when the dialog is closed.
139 // See -[SSLCertificateViewerCocoa onConstrainedWindowClosed].
140 SSLCertificateViewerCocoa* viewer =
141 [[SSLCertificateViewerCocoa alloc] initWithCertificate:cert];
142 [viewer displayForWebContents:web_contents];
143 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698