Store feedback for spellcheck results from spelling service

chrome/browser/spellchecker/feedback_sender_unittest.cc

 // Copyright (c) 2013 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 <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/json/json_reader.h"
 #include "base/message_loop.h"
@@ skipping 415 matching lines @@
       "\"suggestionId\":\"42\","
       "\"suggestions\":[\"Hello\"],"
       "\"timestamp\":\"9001\","
       "\"userActions\":[{\"actionType\":\"NO_ACTION\"}]}]}}";
   scoped_ptr<base::Value> expected(base::JSONReader::Read(expected_data));
   EXPECT_TRUE(expected->Equals(actual.get()))
       << "Expected data: " << expected_data
       << "\nActual data:   " << actual_data;
 }
 
+// Duplicate spellcheck results should be matched to the existing markers.
+TEST_F(FeedbackSenderTest, MatchDupliateResultsWithExistingMarkers) {

[groby-ooo-7-16, 2013/05/23 19:10:50]

Why a new test - was there new functionality?

[please use gerrit instead, 2013/05/23 19:50:47]

No new functionality. I was increasing test coverage.

+  uint32 hash = AddPendingFeedback();
+  std::vector<SpellCheckResult> results(1, SpellCheckResult(
+      SpellCheckResult::SPELLING,
+      kMisspellingStart + 10,
+      kMisspellingLength,
+      ASCIIToUTF16("Hello")));
+  std::vector<SpellCheckMarker> markers(1, SpellCheckMarker(
+      hash, results[0].location));
+  EXPECT_EQ(static_cast<uint32>(0), results[0].hash);
+  feedback_.OnSpellcheckResults(&results, kRendererProcessId, kText, markers);
+  EXPECT_EQ(hash, results[0].hash);
+}
+
 }  // namespace spellcheck