Fixed IAccessibleText::TextAtOffset with IA2_TEXT_BOUNDARY_WORD to return text that spans from the …

content/browser/accessibility/browser_accessibility_win_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 "base/memory/scoped_ptr.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/win/scoped_bstr.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/scoped_variant.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
@@ skipping 356 matching lines @@
   long end;
   ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
       1, IA2_TEXT_BOUNDARY_CHAR, &start, &end, text.Receive()));
   ASSERT_EQ(1, start);
   ASSERT_EQ(2, end);
   ASSERT_STREQ(L"n", text);
   text.Reset();
 
   ASSERT_EQ(S_FALSE, text1_obj->get_textAtOffset(
       text1_len, IA2_TEXT_BOUNDARY_CHAR, &start, &end, text.Receive()));
-  ASSERT_EQ(text1_len, start);
-  ASSERT_EQ(text1_len, end);
+  ASSERT_EQ(0, start);

[dmazzoni, 2015/02/20 18:16:21]

Why does this return 0 for start and end? We asked for offset test1_len.

+  ASSERT_EQ(0, end);
   text.Reset();
 
   ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
       1, IA2_TEXT_BOUNDARY_WORD, &start, &end, text.Receive()));
   ASSERT_EQ(0, start);
-  ASSERT_EQ(3, end);
-  ASSERT_STREQ(L"One", text);
+  ASSERT_EQ(4, end);
+  ASSERT_STREQ(L"One ", text);
   text.Reset();
 
   ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
       6, IA2_TEXT_BOUNDARY_WORD, &start, &end, text.Receive()));
   ASSERT_EQ(4, start);
-  ASSERT_EQ(7, end);
-  ASSERT_STREQ(L"two", text);
+  ASSERT_EQ(8, end);
+  ASSERT_STREQ(L"two\n", text);
   text.Reset();
 
   ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
       text1_len, IA2_TEXT_BOUNDARY_WORD, &start, &end, text.Receive()));
   ASSERT_EQ(25, start);
   ASSERT_EQ(29, end);
   ASSERT_STREQ(L"six.", text);
   text.Reset();
 
   ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
       1, IA2_TEXT_BOUNDARY_LINE, &start, &end, text.Receive()));
   ASSERT_EQ(0, start);
   ASSERT_EQ(15, end);
   ASSERT_STREQ(L"One two three.\n", text);
   text.Reset();
 
+  ASSERT_EQ(S_OK, text1_obj->get_textAtOffset(
+      text1_len, IA2_TEXT_BOUNDARY_LINE, &start, &end, text.Receive()));
+  ASSERT_EQ(15, start);
+  ASSERT_EQ(text1_len, end);
+  ASSERT_STREQ(L"Four five six.", text);
+  text.Reset();
+
   ASSERT_EQ(S_OK,
             text1_obj->get_text(0, IA2_TEXT_OFFSET_LENGTH, text.Receive()));
+  ASSERT_EQ(0, start);
+  ASSERT_EQ(text1_len, end);
   ASSERT_STREQ(L"One two three.\nFour five six.", text);
 
   // Delete the manager and test that all BrowserAccessibility instances are
   // deleted.
   manager.reset();
   ASSERT_EQ(0, CountedBrowserAccessibility::num_instances());
 }
 
 TEST_F(BrowserAccessibilityTest, TestSimpleHypertext) {
   const std::string text1_name = "One two three.";
@@ skipping 319 matching lines @@
   EXPECT_EQ(S_OK, hr);
   EXPECT_NE(nullptr, static_cast<BSTR>(attributes));
   std::wstring attributes_str(attributes, attributes.Length());
   EXPECT_EQ(L"checkable:true;", attributes_str);
 
   manager.reset();
   ASSERT_EQ(0, CountedBrowserAccessibility::num_instances());
 }
 
 }  // namespace content