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

Unified Diff: chrome/browser/ui/webui/chromeos/mobile_setup_dialog.cc

Issue 8341037: Added logic that will retry to reload+reconnect the mobile payment portal page 5 times before it ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 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/webui/chromeos/mobile_setup_dialog.cc
===================================================================
--- chrome/browser/ui/webui/chromeos/mobile_setup_dialog.cc (revision 107314)
+++ chrome/browser/ui/webui/chromeos/mobile_setup_dialog.cc (working copy)
@@ -16,49 +16,80 @@
#include "grit/generated_resources.h"
#include "ui/base/l10n/l10n_util.h"
+class MobileSetupDialogDelegate : public HtmlDialogUIDelegate {
+ public:
+ static MobileSetupDialogDelegate* GetInstance();
+ void ShowDialog();
+
+ protected:
+ friend struct DefaultSingletonTraits<MobileSetupDialogDelegate>;
+
+ MobileSetupDialogDelegate();
+ virtual ~MobileSetupDialogDelegate();
+
+ void OnCloseDialog();
+
+ // HtmlDialogUIDelegate overrides.
+ virtual bool IsDialogModal() const OVERRIDE;
+ virtual string16 GetDialogTitle() const OVERRIDE;
+ virtual GURL GetDialogContentURL() const OVERRIDE;
+ virtual void GetWebUIMessageHandlers(
+ std::vector<WebUIMessageHandler*>* handlers) const OVERRIDE;
+ virtual void GetDialogSize(gfx::Size* size) const OVERRIDE;
+ virtual std::string GetDialogArgs() const OVERRIDE;
+ virtual void OnDialogClosed(const std::string& json_retval) OVERRIDE;
+ virtual void OnCloseContents(TabContents* source,
+ bool* out_close_dialog) OVERRIDE;
+ virtual bool ShouldShowDialogTitle() const OVERRIDE;
+ virtual bool HandleContextMenu(const ContextMenuParams& params) OVERRIDE;
+
+ private:
+
+ DISALLOW_COPY_AND_ASSIGN(MobileSetupDialogDelegate);
+};
+
// static
-MobileSetupDialog* MobileSetupDialog::GetInstance() {
+void MobileSetupDialog::Show() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- return Singleton<MobileSetupDialog>::get();
+ MobileSetupDialogDelegate::GetInstance()->ShowDialog();
}
-MobileSetupDialog::MobileSetupDialog() {
+// static
+MobileSetupDialogDelegate* MobileSetupDialogDelegate::GetInstance() {
+ DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+ return Singleton<MobileSetupDialogDelegate>::get();
}
-MobileSetupDialog::~MobileSetupDialog() {
+MobileSetupDialogDelegate::MobileSetupDialogDelegate() {
}
-// static
-void MobileSetupDialog::Show() {
- DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- MobileSetupDialog* dialog = MobileSetupDialog::GetInstance();
- dialog->ShowDialog();
+MobileSetupDialogDelegate::~MobileSetupDialogDelegate() {
}
-void MobileSetupDialog::ShowDialog() {
+void MobileSetupDialogDelegate::ShowDialog() {
Browser* browser = BrowserList::GetLastActive();
if (!browser)
return;
browser->BrowserShowHtmlDialog(this, NULL);
}
-bool MobileSetupDialog::IsDialogModal() const {
+bool MobileSetupDialogDelegate::IsDialogModal() const {
return true;
}
-string16 MobileSetupDialog::GetDialogTitle() const {
+string16 MobileSetupDialogDelegate::GetDialogTitle() const {
return l10n_util::GetStringUTF16(IDS_MOBILE_SETUP_TITLE);
}
-GURL MobileSetupDialog::GetDialogContentURL() const {
+GURL MobileSetupDialogDelegate::GetDialogContentURL() const {
return GURL(chrome::kChromeUIMobileSetupURL);
}
-void MobileSetupDialog::GetWebUIMessageHandlers(
+void MobileSetupDialogDelegate::GetWebUIMessageHandlers(
std::vector<WebUIMessageHandler*>* handlers) const{
}
-void MobileSetupDialog::GetDialogSize(gfx::Size* size) const {
+void MobileSetupDialogDelegate::GetDialogSize(gfx::Size* size) const {
#if defined(POST_PORTAL)
size->SetSize(850, 650);
#else
@@ -66,22 +97,22 @@
#endif
}
-std::string MobileSetupDialog::GetDialogArgs() const {
+std::string MobileSetupDialogDelegate::GetDialogArgs() const {
return std::string();
}
-void MobileSetupDialog::OnDialogClosed(const std::string& json_retval) {
+void MobileSetupDialogDelegate::OnDialogClosed(const std::string& json_retval) {
}
-void MobileSetupDialog::OnCloseContents(TabContents* source,
+void MobileSetupDialogDelegate::OnCloseContents(TabContents* source,
bool* out_close_dialog) {
*out_close_dialog = true;
}
-bool MobileSetupDialog::ShouldShowDialogTitle() const {
+bool MobileSetupDialogDelegate::ShouldShowDialogTitle() const {
return true;
}
-bool MobileSetupDialog::HandleContextMenu(const ContextMenuParams& params) {
+bool MobileSetupDialogDelegate::HandleContextMenu(const ContextMenuParams& params) {
return true;
}
« no previous file with comments | « chrome/browser/ui/webui/chromeos/mobile_setup_dialog.h ('k') | chrome/browser/ui/webui/chromeos/mobile_setup_ui.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698