Implementing context menu for the new views textfield.

views/controls/textfield/native_textfield_views_unittest.cc

 // Copyright (c) 2010 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 "app/keyboard_codes.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
 #include "testing/gtest/include/gtest/gtest.h"
+#include "ui/base/clipboard/clipboard.h"
+#include "views/controls/menu/menu_2.h"
 #include "views/controls/textfield/native_textfield_views.h"
 #include "views/controls/textfield/textfield.h"
 #include "views/controls/textfield/textfield_views_model.h"
 #include "views/event.h"
 #include "views/focus/focus_manager.h"
+#include "views/test/test_views_delegate.h"
 #include "views/widget/widget.h"
+#include "views/views_delegate.h"
+
+#if !defined(OS_WIN)
+#if defined(TOUCH_UI)
+#include "views/controls/menu/native_menu_x.h"
+#else
+#include "views/controls/menu/native_menu_gtk.h"
+#endif
+#endif
 
 namespace views {
 
 #define EXPECT_STR_EQ(ascii, utf16) \
   EXPECT_EQ(ASCIIToWide(ascii), UTF16ToWide(utf16))
 
 // TODO(oshima): Move tests that are independent of TextfieldViews to
 // textfield_unittests.cc once we move the test utility functions
 // from chrome/browser/automation/ to app/test/.
 class NativeTextfieldViewsTest : public ::testing::Test,
@@ skipping 56 matching lines @@
     for (int i = 1; i < count; i++) {
       Textfield* textfield = new Textfield(style);
       container->AddChildView(textfield);
       textfield->SetID(i + 1);
     }
 
     DCHECK(textfield_view_);
     model_ = textfield_view_->model_.get();
   }
 
+  views::Menu2* GetContextMenu() {
+    if (!textfield_view_->context_menu_menu_.get())
+      textfield_view_->CreateContextMenu();
+    return textfield_view_->context_menu_menu_.get();
+  }
+
  protected:
   bool SendKeyEventToTextfieldViews(app::KeyboardCode key_code,
                                     bool shift,
                                     bool control,
                                     bool capslock) {
     int flags = (shift ? KeyEvent::EF_SHIFT_DOWN : 0) |
         (control ? KeyEvent::EF_CONTROL_DOWN : 0) |
         (capslock ? KeyEvent::EF_CAPS_LOCK_DOWN : 0);
     KeyEvent event(KeyEvent::ET_KEY_PRESSED, key_code, flags, 1, 0);
     return textfield_->OnKeyPressed(event);
@@ skipping 272 matching lines @@
   EXPECT_EQ(1, GetFocusedView()->GetID());
   // Cycle back to the last textfield.
   widget_->GetFocusManager()->AdvanceFocus(true);
   EXPECT_EQ(3, GetFocusedView()->GetID());
 
   // Request focus should still work.
   textfield_->RequestFocus();
   EXPECT_EQ(1, GetFocusedView()->GetID());
 }
 
+void VerifyTextfieldContextMenuContents(bool textfield_has_selection,
+    menus::MenuModel* menu_model) {
+  EXPECT_TRUE(menu_model->IsEnabledAt(4 /* Separator */));
+  EXPECT_TRUE(menu_model->IsEnabledAt(5 /* SELECT ALL */));
+  EXPECT_EQ(textfield_has_selection, menu_model->IsEnabledAt(0 /* CUT */));
+  EXPECT_EQ(textfield_has_selection, menu_model->IsEnabledAt(1 /* COPY */));
+  EXPECT_EQ(textfield_has_selection, menu_model->IsEnabledAt(3 /* DELETE */));
+  string16 str;
+  views::ViewsDelegate::views_delegate->GetClipboard()
+      ->ReadText(ui::Clipboard::BUFFER_STANDARD, &str);
+  EXPECT_NE(str.empty(), menu_model->IsEnabledAt(2 /* PASTE */));
+}
+
+TEST_F(NativeTextfieldViewsTest, ContextMenuDisplayTest) {
+  views::ViewsDelegate::views_delegate = new TestViewsDelegate();

[oshima, 2011/01/12 18:46:25]

don't you have to delete this?

[tfarina, 2011/01/12 19:19:19]

It seems, refer to this CL for a fix?
http://codereview.chromium.org/6132009/diff/1/views/controls/textfield/textfi...

[varunjain, 2011/01/12 19:59:05]

Fixed... I cannot believe I did this again (especially because I reviewed that
CL this morning!!)... need coffee :(

+  InitTextfield(Textfield::STYLE_DEFAULT);
+  textfield_->SetText(ASCIIToUTF16("hello world"));
+  EXPECT_TRUE(GetContextMenu());
+  VerifyTextfieldContextMenuContents(false, GetContextMenu()->model());
+
+  textfield_->SelectAll();
+  VerifyTextfieldContextMenuContents(true, GetContextMenu()->model());
+}
+
+#if !defined(OS_WIN)
+MenuWrapper* MenuWrapper::CreateWrapper(Menu2* menu) {

[oshima, 2011/01/12 18:46:25]

why do we need this here?

[varunjain, 2011/01/12 19:59:05]

for touchui=1 && chromeos=1, we do not get an implementation of CreateWrapper.
However, this is wrong in its current form. Fixed.

[oshima, 2011/01/12 21:01:47]

That sounds like general issue with touchui&chromeos=1 build. Shouldn't we fix
it in native_menu_x.cc? What would happen to menu in chrome if I build with
touchui=1 and chromeos=1?

[varunjain, 2011/01/12 21:28:11]

In the general case, when touchui=1&&chromeos=1, the CreateWrapper
implementation comes from chromeos/views/native_menu_domui.cc. However, this is
a views testcase and cannot depend on the file from chromeos tree. Hence I
specified the implementation here.

+#if defined(TOUCH_UI)
+  return new NativeMenuX(menu);
+#else
+  return new NativeMenuGtk(menu);
+#endif
+}
+#endif
+
 }  // namespace views