Select All present even all selectable text has been selected

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 100 matching lines @@
 #include "public/platform/WebClipboard.h"
 #include "public/platform/WebFloatRect.h"
 #include "public/platform/WebMockClipboard.h"
 #include "public/platform/WebSecurityOrigin.h"
 #include "public/platform/WebThread.h"
 #include "public/platform/WebURL.h"
 #include "public/platform/WebURLLoaderClient.h"
 #include "public/platform/WebURLLoaderMockFactory.h"
 #include "public/platform/WebURLResponse.h"
 #include "public/web/WebConsoleMessage.h"
+#include "public/web/WebContextMenuData.h"
 #include "public/web/WebDataSource.h"
 #include "public/web/WebDeviceEmulationParams.h"
 #include "public/web/WebDocument.h"
 #include "public/web/WebFindOptions.h"
 #include "public/web/WebFormElement.h"
 #include "public/web/WebFrameClient.h"
 #include "public/web/WebFrameContentDumper.h"
 #include "public/web/WebFrameWidget.h"
 #include "public/web/WebHistoryItem.h"
 #include "public/web/WebPrintParams.h"
@@ skipping 11310 matching lines @@
       WebScriptSource("window.focus();"
                       "document.querySelector('input').focus();"));
 
   // Verify that the right WebWidgetClient has been notified.
   EXPECT_TRUE(webWidgetClient.didShowVirtualKeyboard());
 
   remoteFrame->close();
   webViewHelper.reset();
 }
 
+class ContextMenuWebFrameClient : public FrameTestHelpers::TestWebFrameClient {
+ public:
+  // WebFrameClient methods
+  void showContextMenu(const WebContextMenuData& data) override {
+    m_menuData = data;
+  }
+
+  WebContextMenuData getMenuData() { return m_menuData; }
+
+ private:
+  WebContextMenuData m_menuData;
+};

[yosin_UTC9, 2017/03/16 01:21:35]

nit: Could you add |DISALLOW_COPY_AND_ASSIGN(ContextMenuWebFrameClient)|?

+
+bool testSelectAll(const std::string& html) {
+  ContextMenuWebFrameClient frame;
+  FrameTestHelpers::WebViewHelper webViewHelper;
+  WebViewImpl* webView = webViewHelper.initialize(true, &frame);
+  FrameTestHelpers::loadHTMLString(webView->mainFrame(), html,
+                                   toKURL("about:blank"));
+  webView->resize(WebSize(500, 300));
+  webView->updateAllLifecyclePhases();
+  runPendingTasks();
+  webView->setInitialFocus(false);
+  runPendingTasks();
+
+  WebMouseEvent mouseEvent(WebInputEvent::MouseDown, WebInputEvent::NoModifiers,
+                           WebInputEvent::TimeStampForTesting);
+
+  mouseEvent.button = WebMouseEvent::Button::Right;
+  mouseEvent.x = 10;
+  mouseEvent.y = 10;
+  mouseEvent.clickCount = 1;
+  webView->handleInputEvent(WebCoalescedInputEvent(mouseEvent));
+  runPendingTasks();
+  webViewHelper.reset();
+  return frame.getMenuData().editFlags & WebContextMenuData::CanSelectAll;
+}
+
+TEST_F(WebFrameTest, ContextMenuData) {
+  EXPECT_FALSE(testSelectAll("<textarea></textarea>"));
+  EXPECT_TRUE(testSelectAll("<textarea>nonempty</textarea>"));
+  EXPECT_FALSE(testSelectAll("<input>"));
+  EXPECT_TRUE(testSelectAll("<input value='nonempty'>"));
+  EXPECT_TRUE(testSelectAll("<div contenteditable></div>"));

[yosin_UTC9, 2017/03/16 01:21:35]

nit: Could you add a TODO comment that we actually expect this is false.

+  EXPECT_TRUE(testSelectAll("<div contenteditable>nonempty</div>"));
+}
+
 }  // namespace blink