Store feedback for spellcheck results from spelling service

chrome/browser/spellchecker/feedback_sender_unittest.cc

Index: chrome/browser/spellchecker/feedback_sender_unittest.cc
diff --git a/chrome/browser/spellchecker/feedback_sender_unittest.cc b/chrome/browser/spellchecker/feedback_sender_unittest.cc
index 23eaf5af88e0ba71b42770a6466da1ba9c1067a5..69e91b96d12cd87d5a4b4ff2a520b12c36a8e611 100644
--- a/chrome/browser/spellchecker/feedback_sender_unittest.cc
+++ b/chrome/browser/spellchecker/feedback_sender_unittest.cc
@@ -433,4 +433,19 @@ TEST_F(FeedbackSenderTest, FeedbackAPI) {
       << "\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