Allow storing multiple replacements on SpellCheckResult

components/spellcheck/common/spellcheck_result.h

Index: components/spellcheck/common/spellcheck_result.h
diff --git a/components/spellcheck/common/spellcheck_result.h b/components/spellcheck/common/spellcheck_result.h
index 0c4fb45eff7752edc88a9b2ba0ea7ca20150a9d7..181bbe5687d1dad53b032708b7c18778afa681aa 100644
--- a/components/spellcheck/common/spellcheck_result.h
+++ b/components/spellcheck/common/spellcheck_result.h
@@ -6,6 +6,7 @@
 #define COMPONENTS_SPELLCHECK_COMMON_SPELLCHECK_RESULT_H_
 
 #include <stdint.h>
+#include <vector>
 
 #include "base/strings/string16.h"
 
@@ -22,16 +23,18 @@ struct SpellCheckResult {
     GRAMMAR = 1 << 2,
   };
 
-  explicit SpellCheckResult(Decoration d = SPELLING,
-                            int loc = 0,
-                            int len = 0,
-                            const base::string16& rep = base::string16())
-      : decoration(d), location(loc), length(len), replacement(rep) {}
+  explicit SpellCheckResult(
+      Decoration d = SPELLING,

[groby-ooo-7-16, 2017/04/28 23:45:55]

Can we just kill the defaults here?

[rlanday, 2017/05/01 23:25:07]

Seems that these are needed for IPC::ReadParam() to compile properly

+      int loc = 0,
+      int len = 0,
+      const std::vector<base::string16>& rep = std::vector<base::string16>());
+  ~SpellCheckResult();
+  SpellCheckResult(const SpellCheckResult&);
 
   Decoration decoration;
   int location;
   int length;
-  base::string16 replacement;
+  std::vector<base::string16> replacements;
 };
 
 #endif  // COMPONENTS_SPELLCHECK_COMMON_SPELLCHECK_RESULT_H_