Change the behavior of TSFTextStore::QueryInsert() to support partial commit with Metro MS-IME.

ui/base/ime/win/tsf_text_store_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/base/ime/win/tsf_text_store.h"
 
 #include <initguid.h>  // for GUID_NULL and GUID_PROP_INPUTSCOPE
 #include <InputScope.h>
 #include <OleCtl.h>
 
@@ skipping 305 matching lines @@
   EXPECT_EQ(E_INVALIDARG,
             text_store_->QueryInsert(0, 0, 0, NULL, &result_end));
   EXPECT_EQ(E_INVALIDARG,
             text_store_->QueryInsert(0, 0, 0, &result_start, NULL));
   EXPECT_EQ(S_OK,
             text_store_->QueryInsert(0, 0, 0, &result_start, &result_end));
   EXPECT_EQ(0, result_start);
   EXPECT_EQ(0, result_end);
   *string_buffer() = L"1234";
   *committed_size() = 1;
-  EXPECT_EQ(E_INVALIDARG,
+  EXPECT_EQ(S_OK,
             text_store_->QueryInsert(0, 1, 0, &result_start, &result_end));
+  EXPECT_EQ(1, result_start);
+  EXPECT_EQ(1, result_end);
   EXPECT_EQ(E_INVALIDARG,
             text_store_->QueryInsert(1, 0, 0, &result_start, &result_end));
   EXPECT_EQ(S_OK,
             text_store_->QueryInsert(2, 2, 0, &result_start, &result_end));
   EXPECT_EQ(2, result_start);
   EXPECT_EQ(2, result_end);
   EXPECT_EQ(S_OK,
             text_store_->QueryInsert(2, 3, 0, &result_start, &result_end));
   EXPECT_EQ(2, result_start);
   EXPECT_EQ(3, result_end);
   EXPECT_EQ(E_INVALIDARG,
             text_store_->QueryInsert(3, 2, 0, &result_start, &result_end));
   EXPECT_EQ(S_OK,
             text_store_->QueryInsert(3, 4, 0, &result_start, &result_end));
   EXPECT_EQ(3, result_start);
   EXPECT_EQ(4, result_end);
-  EXPECT_EQ(E_INVALIDARG,
+  EXPECT_EQ(S_OK,
             text_store_->QueryInsert(3, 5, 0, &result_start, &result_end));
+  EXPECT_EQ(3, result_start);
+  EXPECT_EQ(4, result_end);
 }
 
 class SyncRequestLockTestCallback : public TSFTextStoreTestCallback {
  public:
   explicit SyncRequestLockTestCallback(TSFTextStore* text_store)
       : TSFTextStoreTestCallback(text_store) {}
 
   HRESULT LockGranted1(DWORD flags) {
     EXPECT_TRUE(HasReadLock());
     EXPECT_FALSE(HasReadWriteLock());
@@ skipping 930 matching lines @@
     text_store_->SetFocusedTextInputClient(NULL, NULL);
     num_copied = 0xfffffff;
     TS_ATTRVAL buffer[2] = {};
     EXPECT_HRESULT_FAILED(text_store_->RetrieveRequestedAttrs(
         arraysize(buffer), buffer, &num_copied));
   }
 }
 
 }  // namespace
 }  // namespace ui