Use browser::ShowWebDialog() directly in MobileSetup rather than using BrowserList.

chrome/browser/ui/webui/chromeos/mobile_setup_dialog.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/chromeos/mobile_setup_dialog.h"
 
 #include "base/bind.h"
 #include "base/memory/singleton.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/platform_util.h"
-#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/browser_dialogs.h"
-#include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/webui/web_dialog_delegate.h"
 #include "chrome/common/url_constants.h"
+#include "content/public/browser/browser_thread.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using content::BrowserThread;
 using content::WebContents;
 using content::WebUIMessageHandler;
 
 class MobileSetupDialogDelegate : public WebDialogDelegate {
  public:
   static MobileSetupDialogDelegate* GetInstance();
@@ skipping 39 matching lines @@
   return Singleton<MobileSetupDialogDelegate>::get();
 }
 
 MobileSetupDialogDelegate::MobileSetupDialogDelegate() {
 }
 
 MobileSetupDialogDelegate::~MobileSetupDialogDelegate() {
 }
 
 void MobileSetupDialogDelegate::ShowDialog() {
-  Browser* browser = BrowserList::GetLastActive();
-  if (!browser)
-    return;
-  browser->BrowserShowWebDialog(this, NULL);
+  browser::ShowWebDialog(NULL,
+                         ProfileManager::GetDefaultProfileOrOffTheRecord(),

[Ben Goodger (Google), 2012/05/22 17:58:30]

Is this how you get the active profile on cros?

[zel, 2012/05/22 18:02:32]

I think so, even though GetDefaultProfileOrOffTheRecord() method seems new to
me. ProfileManager::GetDefaultProfile() used to give you default or offline
profile depending on login state.

[Nikita (slow), 2012/05/23 19:01:20]

This is an ugly hack that we're planning to get rid of:
http://code.google.com/p/chromium/issues/detail?id=128520

+                         NULL,
+                         this);
 }
 
 ui::ModalType MobileSetupDialogDelegate::GetDialogModalType() const {
   return ui::MODAL_TYPE_SYSTEM;
 }
 
 string16 MobileSetupDialogDelegate::GetDialogTitle() const {
   return l10n_util::GetStringUTF16(IDS_MOBILE_SETUP_TITLE);
 }
 
@@ skipping 26 matching lines @@
 }
 
 bool MobileSetupDialogDelegate::ShouldShowDialogTitle() const {
   return true;
 }
 
 bool MobileSetupDialogDelegate::HandleContextMenu(
     const content::ContextMenuParams& params) {
   return true;
 }