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

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

Issue 1717453003: Introduce BubbleDialogDelegateView, which extends DialogDelegateView (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 4 years, 10 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/simple_message_box.h" 5 #include "chrome/browser/ui/simple_message_box.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
(...skipping 232 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 ); 243 );
244 constrained_window::CreateBrowserModalDialogViews(dialog, parent)->Show(); 244 constrained_window::CreateBrowserModalDialogViews(dialog, parent)->Show();
245 245
246 // NOTE: |dialog| may have been deleted by the time |RunDialogAndGetResult()| 246 // NOTE: |dialog| may have been deleted by the time |RunDialogAndGetResult()|
247 // returns. 247 // returns.
248 return dialog->RunDialogAndGetResult(); 248 return dialog->RunDialogAndGetResult();
249 } 249 }
250 250
251 } // namespace 251 } // namespace
252 252
253 MessageBoxResult ShowMessageBox(gfx::NativeWindow parent, 253 void ShowWarningMessageBox(gfx::NativeWindow parent,
msw 2016/02/22 21:43:56 Did you intend to change this file? If so, how is
Evan Stade 2016/02/22 22:58:22 nope, sorry, this is tangential. I broke it off in
254 const base::string16& title, 254 const base::string16& title,
255 const base::string16& message, 255 const base::string16& message) {
256 MessageBoxType type) { 256 ShowMessageBoxImpl(parent, title, message, MESSAGE_BOX_TYPE_WARNING,
257 return ShowMessageBoxImpl( 257 base::string16(), base::string16());
258 parent, title, message, type, base::string16(), base::string16()); 258 }
259
260 MessageBoxResult ShowQuestionMessageBox(gfx::NativeWindow parent,
261 const base::string16& title,
262 const base::string16& message) {
263 return ShowMessageBoxImpl(parent, title, message, MESSAGE_BOX_TYPE_QUESTION,
264 base::string16(), base::string16());
259 } 265 }
260 266
261 MessageBoxResult ShowMessageBoxWithButtonText(gfx::NativeWindow parent, 267 MessageBoxResult ShowMessageBoxWithButtonText(gfx::NativeWindow parent,
262 const base::string16& title, 268 const base::string16& title,
263 const base::string16& message, 269 const base::string16& message,
264 const base::string16& yes_text, 270 const base::string16& yes_text,
265 const base::string16& no_text) { 271 const base::string16& no_text) {
266 return ShowMessageBoxImpl( 272 return ShowMessageBoxImpl(
267 parent, title, message, MESSAGE_BOX_TYPE_QUESTION, yes_text, no_text); 273 parent, title, message, MESSAGE_BOX_TYPE_QUESTION, yes_text, no_text);
268 } 274 }
269 275
270 } // namespace chrome 276 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698