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

Unified Diff: chrome/browser/chromeos/ui/choose_mobile_network_dialog.h

Issue 2446573003: chromeos: Make "mobile network" and "set time" dialogs work with mash (Closed)
Patch Set: cleanup Created 4 years, 2 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
Index: chrome/browser/chromeos/ui/choose_mobile_network_dialog.h
diff --git a/chrome/browser/chromeos/ui/choose_mobile_network_dialog.h b/chrome/browser/chromeos/ui/choose_mobile_network_dialog.h
index 0d656306db39fcef21da1f37489de4d6f8cc697a..7cef16812f76bd678438b5465c40d1f6e810108b 100644
--- a/chrome/browser/chromeos/ui/choose_mobile_network_dialog.h
+++ b/chrome/browser/chromeos/ui/choose_mobile_network_dialog.h
@@ -5,7 +5,6 @@
#ifndef CHROME_BROWSER_CHROMEOS_UI_CHOOSE_MOBILE_NETWORK_DIALOG_H_
#define CHROME_BROWSER_CHROMEOS_UI_CHOOSE_MOBILE_NETWORK_DIALOG_H_
-#include "base/compiler_specific.h"
#include "base/macros.h"
#include "ui/gfx/native_widget_types.h"
#include "ui/web_dialogs/web_dialog_delegate.h"
@@ -15,9 +14,12 @@ namespace chromeos {
// Dialog for manual selection of cellular network.
class ChooseMobileNetworkDialog : public ui::WebDialogDelegate {
public:
- // Shows the dialog box.
+ // Shows the dialog box. |owning_window| may be null.
msw 2016/10/24 22:18:09 nit: Maybe explain the significance of |owning_win
James Cook 2016/10/25 00:29:04 Done (and renamed to parent).
static void ShowDialog(gfx::NativeWindow owning_window);
+ // Shows the dialog in an ash window container.
+ static void ShowDialogInContainer(int container_id);
+
private:
ChooseMobileNetworkDialog();

Powered by Google App Engine
This is Rietveld 408576698