Double variations crash key size

components/crash/core/common/crash_keys.cc

Index: components/crash/core/common/crash_keys.cc
diff --git a/components/crash/core/common/crash_keys.cc b/components/crash/core/common/crash_keys.cc
index 6c2d3828deee3ad3c6096a0f31431d026c476275..9e3346bfd988e827665d6a69932ea7515403fbbe 100644
--- a/components/crash/core/common/crash_keys.cc
+++ b/components/crash/core/common/crash_keys.cc
@@ -93,12 +93,12 @@ void SetVariationsList(const std::vector<std::string>& variations) {
       base::StringPrintf("%" PRIuS, variations.size()));
 
   std::string variations_string;
-  variations_string.reserve(kLargeSize);
+  variations_string.reserve(kHugeSize);

[Sigurður Ásgeirsson, 2017/02/06 15:44:56]

is it possible to test that this actually works? There's an associated unittest
I haven't looked at...

[manzagop (departed), 2017/02/06 16:44:18]

Ack.

 
   for (size_t i = 0; i < variations.size(); ++i) {
     const std::string& variation = variations[i];
     // Do not truncate an individual experiment.
-    if (variations_string.size() + variation.size() >= kLargeSize)
+    if (variations_string.size() + variation.size() >= kHugeSize)
       break;
     variations_string += variation;
     variations_string += ",";