Client variations catch error if parse fails

Request headers will show empty decoded data
https://i.imgur.com/dwRxcMK.png

Bug: 1160346
Change-Id: Idd7fef49f8f3ead4736f253c9dead61c5ff2d1ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2605396
Commit-Queue: Chait Pinnamaneni <cpinnamaneni@microsoft.com>
Reviewed-by: Mathias Bynens <mathias@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#840789}

components/variations/proto/devtools/client_variations_uncompiled.js

@@ -31,8 +31,15 @@ export function parse(data) {
     bytes.push(decoded.charCodeAt(i));
   }
 
-
-  const parsed = ClientVariations.deserializeBinary(bytes);
+  let parsed = null;
+  try {
+    parsed = ClientVariations.deserializeBinary(bytes);
+  } catch (e) {
+    // Deserialization is never expected to fail in Chromium,
+    // but might fail in downstream repositories such as Edgium or
+    // if any website uses the same header name 'x-client-data'
+    parsed = ClientVariations.deserializeBinary([]);
+  }
   return {
     'variationIds': parsed.getVariationIdList(),
     'triggerVariationIds': parsed.getTriggerVariationIdList(),