views::Label: Implement context menu, keyboard shortcuts for copy/select all.

ui/views/controls/label_unittest.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 "ui/views/controls/label.h"
 
 #include <stddef.h>
 
 #include "base/command_line.h"
 #include "base/i18n/rtl.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/accessibility/ax_node_data.h"
 #include "ui/base/clipboard/clipboard.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/compositor/canvas_painter.h"
 #include "ui/events/base_event_utils.h"
+#include "ui/events/test/event_generator.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/render_text.h"
 #include "ui/gfx/switches.h"
+#include "ui/strings/grit/ui_strings.h"
 #include "ui/views/border.h"
 #include "ui/views/test/focus_manager_test.h"
 #include "ui/views/test/views_test_base.h"
 #include "ui/views/widget/widget.h"
 
 using base::ASCIIToUTF16;
 
 #define EXPECT_STR_EQ(ascii, utf16) EXPECT_EQ(ASCIIToUTF16(ascii), utf16)
 
 namespace views {
 
+namespace {
+
+#if defined(OS_MACOSX)
+const int kControlCommandModifier = ui::EF_COMMAND_DOWN;
+#else
+const int kControlCommandModifier = ui::EF_CONTROL_DOWN;
+#endif
+
+// All text sizing measurements (width and height) should be greater than this.
+const int kMinTextDimension = 4;
+
+class TestLabel : public Label {
+ public:
+  TestLabel() : Label(ASCIIToUTF16("TestLabel")) { SizeToPreferredSize(); }
+
+  int schedule_paint_count() const { return schedule_paint_count_; }
+
+  void SimulatePaint() {
+    gfx::Canvas canvas(bounds().size(), 1.0, false /* is_opaque */);
+    Paint(ui::CanvasPainter(&canvas, 1.f).context());
+  }
+
+  // View:
+  void SchedulePaintInRect(const gfx::Rect& r) override {
+    ++schedule_paint_count_;
+    Label::SchedulePaintInRect(r);
+  }
+
+ private:
+  int schedule_paint_count_ = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(TestLabel);
+};
+
+// A test utility function to set the application default text direction.
+void SetRTL(bool rtl) {
+  // Override the current locale/direction.
+  base::i18n::SetICUDefaultLocale(rtl ? "he" : "en");
+  EXPECT_EQ(rtl, base::i18n::IsRTL());
+}
+
+// Returns true if |current| is bigger than |last|. Sets |last| to |current|.
+bool Increased(int current, int* last) {
+  bool increased = current > *last;
+  *last = current;
+  return increased;
+}
+
+base::string16 GetClipboardText(ui::ClipboardType clipboard_type) {
+  base::string16 clipboard_text;
+  ui::Clipboard::GetForCurrentThread()->ReadText(clipboard_type,
+                                                 &clipboard_text);
+  return clipboard_text;
+}
+
+}  // namespace
+
 class LabelTest : public ViewsTestBase {
  public:
   LabelTest() {}
 
   // ViewsTestBase overrides:
   void SetUp() override {
     ViewsTestBase::SetUp();
 
     Widget::InitParams params =
         CreateParams(Widget::InitParams::TYPE_WINDOW_FRAMELESS);
@@ skipping 35 matching lines @@
   void SetUp() override {
 #if defined(OS_MACOSX)
     // On Mac, by default RenderTextMac is used for labels which does not
     // support text selection. Instead use RenderTextHarfBuzz for selection
     // related tests. TODO(crbug.com/661394): Remove this once Mac also uses
     // RenderTextHarfBuzz for Labels.
     base::CommandLine::ForCurrentProcess()->AppendSwitch(
         switches::kEnableHarfBuzzRenderText);
 #endif
     LabelTest::SetUp();
+    event_generator_ =
+        base::MakeUnique<ui::test::EventGenerator>(widget()->GetNativeWindow());
   }
 
  protected:
   View* GetFocusedView() {
     return widget()->GetFocusManager()->GetFocusedView();
   }
 
   void PerformMousePress(const gfx::Point& point, int extra_flags = 0) {
     ui::MouseEvent pressed_event = ui::MouseEvent(
         ui::ET_MOUSE_PRESSED, point, point, ui::EventTimeForNow(),
@@ skipping 20 matching lines @@
   }
 
   gfx::Point GetCursorPoint(int cursor_pos) {
     return label()
         ->GetRenderTextForSelectionController()
         ->GetCursorBounds(gfx::SelectionModel(cursor_pos, gfx::CURSOR_FORWARD),
                           false)
         .origin();
   }
 
-  base::string16 GetSelectedText() {
-    const gfx::RenderText* render_text =
-        label()->GetRenderTextForSelectionController();
-    return render_text->GetTextFromRange(render_text->selection());
+  base::string16 GetSelectedText() { return label()->GetSelectedText(); }
+
+  ui::test::EventGenerator* event_generator() { return event_generator_.get(); }
+
+  bool IsMenuCommandEnabled(int command_id) {
+    return label()->IsCommandIdEnabled(command_id);
   }
 
  private:
+  std::unique_ptr<ui::test::EventGenerator> event_generator_;
+
   DISALLOW_COPY_AND_ASSIGN(LabelSelectionTest);
 };
 
-namespace {
-
-// All text sizing measurements (width and height) should be greater than this.
-const int kMinTextDimension = 4;
-
-class TestLabel : public Label {
- public:
-  TestLabel() : Label(ASCIIToUTF16("TestLabel")) { SizeToPreferredSize(); }
-
-  int schedule_paint_count() const { return schedule_paint_count_; }
-
-  void SimulatePaint() {
-    gfx::Canvas canvas(bounds().size(), 1.0, false /* is_opaque */);
-    Paint(ui::CanvasPainter(&canvas, 1.f).context());
-  }
-
-  // View:
-  void SchedulePaintInRect(const gfx::Rect& r) override {
-    ++schedule_paint_count_;
-    Label::SchedulePaintInRect(r);
-  }
-
- private:
-  int schedule_paint_count_ = 0;
-
-  DISALLOW_COPY_AND_ASSIGN(TestLabel);
-};
-
-// A test utility function to set the application default text direction.
-void SetRTL(bool rtl) {
-  // Override the current locale/direction.
-  base::i18n::SetICUDefaultLocale(rtl ? "he" : "en");
-  EXPECT_EQ(rtl, base::i18n::IsRTL());
-}
-
-// Returns true if |current| is bigger than |last|. Sets |last| to |current|.
-bool Increased(int current, int* last) {
-  bool increased = current > *last;
-  *last = current;
-  return increased;
-}
-
-base::string16 GetSelectionClipboardText() {
-  base::string16 selection_clipboard_text;
-  ui::Clipboard::GetForCurrentThread()->ReadText(ui::CLIPBOARD_TYPE_SELECTION,
-                                                 &selection_clipboard_text);
-  return selection_clipboard_text;
-}
-
-}  // namespace
-
 // Crashes on Linux only. http://crbug.com/612406
 #if defined(OS_LINUX)
 #define MAYBE_FontPropertySymbol DISABLED_FontPropertySymbol
 #else
 #define MAYBE_FontPropertySymbol FontPropertySymbol
 #endif
 TEST_F(LabelTest, MAYBE_FontPropertySymbol) {
   std::string font_name("symbol");
   gfx::Font font(font_name, 26);
   label()->SetFontList(gfx::FontList(font));
@@ skipping 603 matching lines @@
   // By default, labels don't support text selection.
   EXPECT_FALSE(label()->selectable());
 
   ASSERT_TRUE(label()->SetSelectable(true));
   EXPECT_TRUE(label()->selectable());
 
   // Verify that making a label multiline causes the label to not support text
   // selection.
   label()->SetMultiLine(true);
   EXPECT_FALSE(label()->selectable());
+
+  label()->SetMultiLine(false);
+  ASSERT_TRUE(label()->SetSelectable(true));
+  EXPECT_TRUE(label()->selectable());
+
+  // Verify that obscuring the label text causes the label to not support text
+  // selection.
+  label()->SetObscured(true);
+  EXPECT_FALSE(label()->selectable());
 }
 
 // Verify that labels supporting text selection get focus on clicks.
 TEST_F(LabelSelectionTest, FocusOnClick) {
   label()->SetText(ASCIIToUTF16("text"));
   label()->SizeToPreferredSize();
 
   // By default, labels don't get focus on click.
   PerformClick(gfx::Point());
   EXPECT_NE(label(), GetFocusedView());
@@ skipping 71 matching lines @@
   PerformMousePress(GetCursorPoint(5));
   PerformMouseDragTo(gfx::Point());
   EXPECT_STR_EQ("Label", GetSelectedText());
 
   PerformMouseDragTo(GetCursorPoint(8));
   EXPECT_STR_EQ(" mo", GetSelectedText());
 
   PerformMouseDragTo(gfx::Point(200, 0));
   PerformMouseRelease(gfx::Point(200, 0));
   EXPECT_STR_EQ(" mouse drag", GetSelectedText());
+
+  event_generator()->PressKey(ui::VKEY_C, kControlCommandModifier);
+  EXPECT_STR_EQ(" mouse drag", GetClipboardText(ui::CLIPBOARD_TYPE_COPY_PASTE));
 }
 
 // Verify the initially selected word on a double click, remains selected on
 // mouse dragging.
 TEST_F(LabelSelectionTest, MouseDragWord) {
   label()->SetText(ASCIIToUTF16("Label drag word"));
   label()->SizeToPreferredSize();
   ASSERT_TRUE(label()->SetSelectable(true));
 
   PerformClick(GetCursorPoint(8));
@@ skipping 12 matching lines @@
 // Verify selection clipboard behavior on text selection.
 TEST_F(LabelSelectionTest, SelectionClipboard) {
   label()->SetText(ASCIIToUTF16("Label selection clipboard"));
   label()->SizeToPreferredSize();
   ASSERT_TRUE(label()->SetSelectable(true));
 
   // Verify programmatic modification of selection, does not modify the
   // selection clipboard.
   label()->SelectRange(gfx::Range(2, 5));
   EXPECT_STR_EQ("bel", GetSelectedText());
-  EXPECT_TRUE(GetSelectionClipboardText().empty());
+  EXPECT_TRUE(GetClipboardText(ui::CLIPBOARD_TYPE_SELECTION).empty());
 
   // Verify text selection using the mouse updates the selection clipboard.
   PerformMousePress(GetCursorPoint(5));
   PerformMouseDragTo(gfx::Point());
   PerformMouseRelease(gfx::Point());
   EXPECT_STR_EQ("Label", GetSelectedText());
-  EXPECT_STR_EQ("Label", GetSelectionClipboardText());
+  EXPECT_STR_EQ("Label", GetClipboardText(ui::CLIPBOARD_TYPE_SELECTION));
 }
 #endif
 
+// Verify that keyboard shortcuts for Copy and Select All work when a selectable
+// label is focused.
+TEST_F(LabelSelectionTest, KeyboardActions) {
+  const base::string16 initial_text = ASCIIToUTF16("Label keyboard actions");
+  label()->SetText(initial_text);
+  label()->SizeToPreferredSize();
+  ASSERT_TRUE(label()->SetSelectable(true));
+
+  PerformClick(gfx::Point());
+  EXPECT_EQ(label(), GetFocusedView());
+
+  event_generator()->PressKey(ui::VKEY_A, kControlCommandModifier);
+  EXPECT_EQ(initial_text, GetSelectedText());
+
+  event_generator()->PressKey(ui::VKEY_C, kControlCommandModifier);
+  EXPECT_EQ(initial_text, GetClipboardText(ui::CLIPBOARD_TYPE_COPY_PASTE));
+
+  // The selection should get cleared on changing the text, but focus should not
+  // be affected.
+  const base::string16 new_text = ASCIIToUTF16("Label obscured text");
+  label()->SetText(new_text);
+  EXPECT_FALSE(label()->HasSelection());
+  EXPECT_EQ(label(), GetFocusedView());
+
+  // Obscured labels do not support text selection.
+  label()->SetObscured(true);
+  EXPECT_FALSE(label()->selectable());
+  event_generator()->PressKey(ui::VKEY_A, kControlCommandModifier);
+  EXPECT_EQ(base::string16(), GetSelectedText());
+}
+
+// Verify the context menu options are enabled and disabled appropriately.
+TEST_F(LabelSelectionTest, ContextMenuContents) {
+  label()->SetText(ASCIIToUTF16("Label context menu"));
+  label()->SizeToPreferredSize();
+
+  // A non-selectable label would not show a context menu and both COPY and
+  // SELECT_ALL context menu items should be disabled for it.
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_COPY));
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_SELECT_ALL));
+
+  // For a selectable label with no selection, only SELECT_ALL should be
+  // enabled.
+  ASSERT_TRUE(label()->SetSelectable(true));
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_COPY));
+  EXPECT_TRUE(IsMenuCommandEnabled(IDS_APP_SELECT_ALL));
+
+  // For a selectable label with a selection, both COPY and SELECT_ALL should be
+  // enabled.
+  label()->SelectRange(gfx::Range(0, 4));
+  EXPECT_TRUE(IsMenuCommandEnabled(IDS_APP_COPY));
+  EXPECT_TRUE(IsMenuCommandEnabled(IDS_APP_SELECT_ALL));
+  // Ensure unsupported commands like PASTE are not enabled.
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_PASTE));
+
+  // An obscured label would not show a context menu and both COPY and
+  // SELECT_ALL should be disabled for it.
+  label()->SetObscured(true);
+  EXPECT_FALSE(label()->selectable());
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_COPY));
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_SELECT_ALL));
+  label()->SetObscured(false);
+
+  // For an empty label, both COPY and SELECT_ALL should be disabled.
+  label()->SetText(base::string16());
+  ASSERT_TRUE(label()->SetSelectable(true));
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_COPY));
+  EXPECT_FALSE(IsMenuCommandEnabled(IDS_APP_SELECT_ALL));
+}
+
 }  // namespace views