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

Side by Side Diff: chrome/browser/ui/views/constrained_web_dialog_delegate_views.cc

Issue 985133002: Remove NativeWebContentsModalDialog and NativePopup typedefs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@extension-popup
Patch Set: Created 5 years, 9 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/browser/ui/webui/constrained_web_dialog_delegate_base.h" 5 #include "chrome/browser/ui/webui/constrained_web_dialog_delegate_base.h"
6 6
7 #include "base/strings/utf_string_conversions.h" 7 #include "base/strings/utf_string_conversions.h"
8 #include "chrome/browser/ui/browser_finder.h" 8 #include "chrome/browser/ui/browser_finder.h"
9 #include "chrome/browser/ui/browser_window.h" 9 #include "chrome/browser/ui/browser_window.h"
10 #include "chrome/browser/ui/webui/chrome_web_contents_handler.h" 10 #include "chrome/browser/ui/webui/chrome_web_contents_handler.h"
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
117 117
118 // contents::WebContentsDelegate: 118 // contents::WebContentsDelegate:
119 void HandleKeyboardEvent( 119 void HandleKeyboardEvent(
120 content::WebContents* source, 120 content::WebContents* source,
121 const content::NativeWebKeyboardEvent& event) override { 121 const content::NativeWebKeyboardEvent& event) override {
122 unhandled_keyboard_event_handler_.HandleKeyboardEvent( 122 unhandled_keyboard_event_handler_.HandleKeyboardEvent(
123 event, view_->GetFocusManager()); 123 event, view_->GetFocusManager());
124 } 124 }
125 125
126 // ConstrainedWebDialogDelegate: 126 // ConstrainedWebDialogDelegate:
127 web_modal::NativeWebContentsModalDialog GetNativeDialog() override { 127 gfx::NativeWindow GetNativeDialog() override {
128 return view_->GetWidget()->GetNativeWindow(); 128 return view_->GetWidget()->GetNativeWindow();
129 } 129 }
130 130
131 private: 131 private:
132 // Converts keyboard events on the WebContents to accelerators. 132 // Converts keyboard events on the WebContents to accelerators.
133 views::UnhandledKeyboardEventHandler unhandled_keyboard_event_handler_; 133 views::UnhandledKeyboardEventHandler unhandled_keyboard_event_handler_;
134 134
135 views::WebView* view_; 135 views::WebView* view_;
136 136
137 DISALLOW_COPY_AND_ASSIGN(ConstrainedWebDialogDelegateViews); 137 DISALLOW_COPY_AND_ASSIGN(ConstrainedWebDialogDelegateViews);
(...skipping 28 matching lines...) Expand all
166 } 166 }
167 ui::WebDialogDelegate* GetWebDialogDelegate() override { 167 ui::WebDialogDelegate* GetWebDialogDelegate() override {
168 return impl_->GetWebDialogDelegate(); 168 return impl_->GetWebDialogDelegate();
169 } 169 }
170 void OnDialogCloseFromWebUI() override { 170 void OnDialogCloseFromWebUI() override {
171 return impl_->OnDialogCloseFromWebUI(); 171 return impl_->OnDialogCloseFromWebUI();
172 } 172 }
173 void ReleaseWebContentsOnDialogClose() override { 173 void ReleaseWebContentsOnDialogClose() override {
174 return impl_->ReleaseWebContentsOnDialogClose(); 174 return impl_->ReleaseWebContentsOnDialogClose();
175 } 175 }
176 web_modal::NativeWebContentsModalDialog GetNativeDialog() override { 176 gfx::NativeWindow GetNativeDialog() override {
177 return impl_->GetNativeDialog(); 177 return impl_->GetNativeDialog();
178 } 178 }
179 content::WebContents* GetWebContents() override { 179 content::WebContents* GetWebContents() override {
180 return impl_->GetWebContents(); 180 return impl_->GetWebContents();
181 } 181 }
182 182
183 // views::WidgetDelegate: 183 // views::WidgetDelegate:
184 views::View* GetInitiallyFocusedView() override { return this; } 184 views::View* GetInitiallyFocusedView() override { return this; }
185 void WindowClosing() override { 185 void WindowClosing() override {
186 if (!impl_->closed_via_webui()) 186 if (!impl_->closed_via_webui())
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
292 const gfx::Size& max_size) { 292 const gfx::Size& max_size) {
293 DCHECK(!min_size.IsEmpty()); 293 DCHECK(!min_size.IsEmpty());
294 DCHECK(!max_size.IsEmpty()); 294 DCHECK(!max_size.IsEmpty());
295 ConstrainedWebDialogDelegateViewViews* dialog = 295 ConstrainedWebDialogDelegateViewViews* dialog =
296 new ConstrainedWebDialogDelegateViewViews( 296 new ConstrainedWebDialogDelegateViewViews(
297 browser_context, delegate, web_contents, 297 browser_context, delegate, web_contents,
298 min_size, max_size); 298 min_size, max_size);
299 constrained_window::CreateWebModalDialogViews(dialog, web_contents); 299 constrained_window::CreateWebModalDialogViews(dialog, web_contents);
300 return dialog; 300 return dialog;
301 } 301 }
OLDNEW
« no previous file with comments | « chrome/browser/ui/tabs/tab_strip_model_unittest.cc ('k') | chrome/browser/ui/views/web_contents_modal_dialog_manager_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698