WebView unit tests for Defect 248426

Source/web/tests/WebViewTest.cpp

Index: Source/web/tests/WebViewTest.cpp
diff --git a/Source/web/tests/WebViewTest.cpp b/Source/web/tests/WebViewTest.cpp
index 502c90288406941958972a9872270334c6d5ce22..bf92cf0cddb5de4c903dc53ed9485f43191886a6 100644
--- a/Source/web/tests/WebViewTest.cpp
+++ b/Source/web/tests/WebViewTest.cpp
@@ -66,6 +66,7 @@
 #include "public/web/WebFrameClient.h"
 #include "public/web/WebHitTestResult.h"
 #include "public/web/WebInputEvent.h"
+#include "public/web/WebScriptSource.h"
 #include "public/web/WebSettings.h"
 #include "public/web/WebViewClient.h"
 #include "public/web/WebWidget.h"
@@ -2137,4 +2138,28 @@ TEST_F(WebViewTest, FirstUserGestureObservedGestureTap)
     webView->setAutofillClient(0);
 }
 
+TEST_F(WebViewTest, OnTTSUtteranceForNoSelection)

[jamesr, 2014/08/12 23:34:15]

this test doesn't have anything to do with TTS. it appears to be testing the
behavior of execCommand('SelectAll') and selectionAsText(). please update the
name accordingly

+{
+    // Note: This is a test to somewhat simulate a CL submitted for
+    // crbug.com/248426. Follow https://codereview.chromium.org/342143004/.
+    // As of now there is no good way to test a TTS session. Since the patch
+    // uses contentAsText() to fetch fully rendered text from a web content
+    // it is okay to check if text returned from SelectAll is same as
+    // contentAsText().

[jamesr, 2014/08/12 23:34:15]

none of this comment has anything to do with what the test actually does, it's
just transient information. this belongs in the commit message, not the code

+    // TODO(a.sarkar.arun@gmail.com): When Chromium has a better support for
+    // testing TTS extensively, this test should be improved and moved to a
+    // more appropriate place.
+    URLTestHelpers::registerMockedURLFromBaseURL(WebString::fromUTF8(m_baseURL.c_str()), WebString::fromUTF8("longpress_selection.html"));
+    WebView* webView = m_webViewHelper.initializeAndLoad(m_baseURL + "longpress_selection.html", true);
+    WebLocalFrameImpl* frame = toWebLocalFrameImpl(webView->mainFrame());
+    frame->executeScript(WebScriptSource(WebString::fromUTF8("document.execCommand('SelectAll', false, null)")));
+
+    std::string output1 = frame->selectionAsText().utf8();

[jamesr, 2014/08/12 23:34:15]

output1 isn't a very useful variable name. what does this represent? maybe
'actual' vs 'expected' ?

+
+    const int kMaxOutputCharacters = 1024;
+    std::string output = frame->contentAsText(kMaxOutputCharacters).utf8();
+    EXPECT_EQ(output1, output);
+
+}
+
 } // namespace