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

Unified Diff: chrome/browser/ui/ash/system_tray_delegate_chromeos.cc

Issue 2441133002: mash: Introduce ShowWebDialogWithContainer for webui dialogs (Closed)
Patch Set: 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/ui/ash/system_tray_delegate_chromeos.cc
diff --git a/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc b/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
index 77d9c12076246f7febc53007826713be2a0466c1..8dfda9900227ec4d2f6c367d63cd892ab2fad5b3 100644
--- a/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
+++ b/chrome/browser/ui/ash/system_tray_delegate_chromeos.cc
@@ -349,8 +349,9 @@ bool SystemTrayDelegateChromeOS::ShouldShowSettings() {
}
void SystemTrayDelegateChromeOS::ShowSetTimeDialog() {
- // TODO(mash): Refactor out GetNativeWindow and move to SystemTrayClient.
- SetTimeDialog::ShowDialog(GetNativeWindow());
+ // TODO(mash): Move to SystemTrayClient.
+ SetTimeDialog::ShowDialogInContainer(
+ SystemTrayClient::GetDialogParentContainerId());
James Cook 2016/10/21 21:21:49 This is the core reason for this change. "GetNativ
}
void SystemTrayDelegateChromeOS::ShowEnterpriseInfo() {

Powered by Google App Engine
This is Rietveld 408576698