CONTINUED Make FrameSelection to hold non-canonicalized positions

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 5101 matching lines @@
   WebRect startWebRect;
   WebRect endWebRect;
 
   registerMockedHttpURLLoad("select_range_basic.html");
   registerMockedHttpURLLoad("select_range_scroll.html");
 
   FrameTestHelpers::WebViewHelper webViewHelper;
   initializeTextSelectionWebView(m_baseURL + "select_range_basic.html",
                                  &webViewHelper);
   frame = webViewHelper.webView()->mainFrameImpl();
-  EXPECT_EQ("Some test text for testing.", selectionAsString(frame));
+  // TODO(yosin): Since selection is canonicalized with size(0, 0) window,
+  // we should hold original range.
+  EXPECT_EQ(" Some test text for testing.", selectionAsString(frame));
   webViewHelper.webView()->selectionBounds(startWebRect, endWebRect);
   frame->executeCommand(WebString::fromUTF8("Unselect"));
   EXPECT_EQ("", selectionAsString(frame));
   frame->selectRange(topLeft(startWebRect), bottomRightMinusOne(endWebRect));
   // On some devices, the above bottomRightMinusOne() causes the ending '.' not
   // selected.
   std::string selectionString = selectionAsString(frame);
-  EXPECT_TRUE(selectionString == "Some test text for testing." ||
-              selectionString == "Some test text for testing");
+  EXPECT_TRUE(selectionString == " Some test text for testing." ||
+              selectionString == " Some test text for testing");
 
   initializeTextSelectionWebView(m_baseURL + "select_range_scroll.html",
                                  &webViewHelper);
   frame = webViewHelper.webView()->mainFrameImpl();
   EXPECT_EQ("Some offscreen test text for testing.", selectionAsString(frame));
   webViewHelper.webView()->selectionBounds(startWebRect, endWebRect);
   frame->executeCommand(WebString::fromUTF8("Unselect"));
   EXPECT_EQ("", selectionAsString(frame));
   frame->selectRange(topLeft(startWebRect), bottomRightMinusOne(endWebRect));
   // On some devices, the above bottomRightMinusOne() causes the ending '.' not
@@ skipping 6091 matching lines @@
 
   EXPECT_TRUE(mainFrameClient.childClient().didCallFrameDetached());
   EXPECT_TRUE(mainFrameClient.childClient().didCallDidStopLoading());
   EXPECT_TRUE(mainFrameClient.childClient().didCallDidFinishDocumentLoad());
   EXPECT_TRUE(mainFrameClient.childClient().didCallDidHandleOnloadEvents());
 
   webViewHelper.reset();
 }
 
 }  // namespace blink