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

Unified Diff: chrome/browser/ui/views/certificate_viewer_mac_views.mm

Issue 1779383002: MacViews: Remove constrained window dependencies for certificate viewer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: msw@ & rsesek@ review, move sheetDidEnd::: into separate SSLCertificateViewerMac declaration. Created 4 years, 4 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/certificate_viewer_mac_cocoa.mm ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/certificate_viewer_mac_views.mm
diff --git a/chrome/browser/ui/views/certificate_viewer_mac_views.mm b/chrome/browser/ui/views/certificate_viewer_mac_views.mm
new file mode 100644
index 0000000000000000000000000000000000000000..334205c92d55a7060b22d55154808d641deec4a9
--- /dev/null
+++ b/chrome/browser/ui/views/certificate_viewer_mac_views.mm
@@ -0,0 +1,70 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#import "chrome/browser/ui/certificate_viewer_mac.h"
+
+#import "base/mac/scoped_nsobject.h"
+#include "chrome/browser/certificate_viewer.h"
+#include "components/constrained_window/constrained_window_views.h"
+#include "components/web_modal/web_contents_modal_dialog_manager.h"
+#include "ui/views/widget/widget.h"
+#include "ui/views/widget/widget_delegate.h"
+
+// Certificate viewer class for MacViews which handles displaying and closing
+// the Cocoa certificate viewer.
+@interface SSLCertificateViewerMacViews : SSLCertificateViewerMac {
+ // Invisible overlay window used to block interaction with the tab underneath.
+ views::Widget* overlayWindow_;
+}
+
+- (void)setOverlayWindow:(views::Widget*)overlayWindow;
+@end
+
+// A fully transparent, borderless web-modal dialog used to display the
+// OS-provided window-modal sheet that displays certificate information.
+class CertificateAnchorWidgetDelegate : public views::WidgetDelegateView {
+ public:
+ CertificateAnchorWidgetDelegate(content::WebContents* web_contents,
+ net::X509Certificate* cert)
+ : certificate_viewer_([[SSLCertificateViewerMacViews alloc]
+ initWithCertificate:cert
+ forWebContents:web_contents]) {
+ views::Widget* overlayWindow =
+ constrained_window::ShowWebModalDialogWithOverlayViews(this,
+ web_contents);
+ [certificate_viewer_ showCertificateSheet:overlayWindow->GetNativeWindow()];
+ [certificate_viewer_ setOverlayWindow:overlayWindow];
+ }
+
+ // WidgetDelegate:
+ ui::ModalType GetModalType() const override { return ui::MODAL_TYPE_CHILD; }
+
+ private:
+ base::scoped_nsobject<SSLCertificateViewerMacViews> certificate_viewer_;
+
+ DISALLOW_COPY_AND_ASSIGN(CertificateAnchorWidgetDelegate);
+};
+
+@implementation SSLCertificateViewerMacViews
+
+- (void)sheetDidEnd:(NSWindow*)parent
+ returnCode:(NSInteger)returnCode
+ context:(void*)context {
+ [self closeCertificateSheet];
+ overlayWindow_->Close(); // Asynchronously releases |self|.
+ [self releaseSheetWindow];
+}
+
+- (void)setOverlayWindow:(views::Widget*)overlayWindow {
+ overlayWindow_ = overlayWindow;
+}
+
+@end
+
+void ShowCertificateViewer(content::WebContents* web_contents,
+ gfx::NativeWindow parent,
+ net::X509Certificate* cert) {
+ // Shows a new widget, which owns the delegate.
+ new CertificateAnchorWidgetDelegate(web_contents, cert);
+}
« no previous file with comments | « chrome/browser/ui/cocoa/certificate_viewer_mac_cocoa.mm ('k') | chrome/chrome_browser_ui.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698