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

Side by Side Diff: chrome/browser/ui/cocoa/single_web_contents_dialog_manager_cocoa.h

Issue 1779383002: MacViews: Remove constrained window dependencies for certificate viewer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Keep a reference to the sheet window in single_web_contents_dialog_manager_cocoa.mm. 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
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_ 5 #ifndef CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_
6 #define CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_ 6 #define CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_
7 7
8 #import "base/mac/scoped_nsobject.h" 8 #import "base/mac/scoped_nsobject.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "components/web_modal/single_web_contents_dialog_manager.h" 10 #include "components/web_modal/single_web_contents_dialog_manager.h"
(...skipping 18 matching lines...) Expand all
29 void Close() override; 29 void Close() override;
30 void Focus() override; 30 void Focus() override;
31 void Pulse() override; 31 void Pulse() override;
32 void HostChanged(web_modal::WebContentsModalDialogHost* new_host) override; 32 void HostChanged(web_modal::WebContentsModalDialogHost* new_host) override;
33 gfx::NativeWindow dialog() override; 33 gfx::NativeWindow dialog() override;
34 34
35 private: 35 private:
36 // Weak. Legacy DialogManager-style delegate interface that Cocoa uses. 36 // Weak. Legacy DialogManager-style delegate interface that Cocoa uses.
37 ConstrainedWindowMac* client_; 37 ConstrainedWindowMac* client_;
38 base::scoped_nsprotocol<id<ConstrainedWindowSheet>> sheet_; 38 base::scoped_nsprotocol<id<ConstrainedWindowSheet>> sheet_;
39 // Reference to the sheet window hosted by |sheet_|.
40 NSWindow* dialog_;
39 // Weak. Owns this. 41 // Weak. Owns this.
40 web_modal::SingleWebContentsDialogManagerDelegate* delegate_; 42 web_modal::SingleWebContentsDialogManagerDelegate* delegate_;
41 // Weak. Owned by parent window. 43 // Weak. Owned by parent window.
42 web_modal::WebContentsModalDialogHost* host_; 44 web_modal::WebContentsModalDialogHost* host_;
43 45
44 DISALLOW_COPY_AND_ASSIGN(SingleWebContentsDialogManagerCocoa); 46 DISALLOW_COPY_AND_ASSIGN(SingleWebContentsDialogManagerCocoa);
45 }; 47 };
46 48
47 #endif // CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_ 49 #endif // CHROME_BROWSER_UI_COCOA_SINGLE_WEB_CONTENTS_DIALOG_MANAGER_COCOA_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698