[cleanup] Use Delegate method to control context menu and eliminate HtmlDialogWithoutContextMenuView

chrome/browser/chromeos/sim_dialog_delegate.cc

 // Copyright (c) 2011 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/chromeos/sim_dialog_delegate.h"
 
 #include "base/stringprintf.h"
 #include "chrome/browser/chromeos/frame/bubble_window.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/profiles/profile_manager.h"
@@ skipping 18 matching lines @@
 // set-lock-on  - enable RequirePin restriction.
 // set-lock-off - disable RequirePin restriction.
 // In general SIM unlock case sim-unlock URL is loaded w/o parameters.
 const char kSimDialogSpecialModeURL[] = "chrome://sim-unlock/?mode=%s";
 
 // Dialog mode constants.
 const char kSimDialogChangePinMode[]  = "change-pin";
 const char kSimDialogSetLockOnMode[]  = "set-lock-on";
 const char kSimDialogSetLockOffMode[] = "set-lock-off";
 
-// Custom HtmlDialogView with disabled context menu.
-class HtmlDialogWithoutContextMenuView : public HtmlDialogView {
- public:
-  HtmlDialogWithoutContextMenuView(Profile* profile,
-                                   HtmlDialogUIDelegate* delegate)
-      : HtmlDialogView(profile, delegate) {}
-
-  // TabContentsDelegate implementation.
-  bool HandleContextMenu(const ContextMenuParams& params) {
-    // Disable context menu.
-    return true;
-  }
-};
-
 }  // namespace
 
 namespace chromeos {
 
 // static
 void SimDialogDelegate::ShowDialog(gfx::NativeWindow owning_window,
-                                         SimDialogMode mode) {
+                                   SimDialogMode mode) {
   Profile* profile;
   if (UserManager::Get()->user_is_logged_in()) {
     Browser* browser = BrowserList::GetLastActive();
     DCHECK(browser);
     profile = browser->profile();
   } else {
     profile = ProfileManager::GetDefaultProfile();
   }
-  HtmlDialogView* html_view = new HtmlDialogWithoutContextMenuView(
-      profile, new SimDialogDelegate(mode));
+  HtmlDialogView* html_view =
+      new HtmlDialogView(profile, new SimDialogDelegate(mode));
   html_view->InitDialog();
   chromeos::BubbleWindow::Create(owning_window,
                                  gfx::Rect(),
                                  chromeos::BubbleWindow::STYLE_GENERIC,
                                  html_view);
   html_view->window()->Show();
 }
 
 SimDialogDelegate::SimDialogDelegate(SimDialogMode dialog_mode)
     : dialog_mode_(dialog_mode) {
@@ skipping 49 matching lines @@
 void SimDialogDelegate::OnCloseContents(TabContents* source,
                                               bool* out_close_dialog) {
   if (out_close_dialog)
     *out_close_dialog = true;
 }
 
 bool SimDialogDelegate::ShouldShowDialogTitle() const {
   return false;
 }
 
+bool SimDialogDelegate::HandleContextMenu(const ContextMenuParams& params) {
+  // Disable context menu.
+  return true;
+}
+
 }  // namespace chromeos