predictors: Remove unknown fields from PrecacheManifest before store it.

chrome/browser/predictors/resource_prefetch_predictor_unittest.cc

 // Copyright 2014 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 "chrome/browser/predictors/resource_prefetch_predictor.h"
 
 #include <iostream>
 #include <memory>
 #include <utility>
 
@@ skipping 1023 matching lines @@
 
   EXPECT_CALL(*mock_tables_.get(),
               DeleteManifestData(std::vector<std::string>({"google.com"})));
 
   EXPECT_CALL(*mock_tables_.get(),
               UpdateManifestData("en.wikipedia.org", manifest));
 
   predictor_->OnManifestFetched("en.wikipedia.org", manifest);
 }
 
+TEST_F(ResourcePrefetchPredictorTest, ManifestUnknownFieldsRemoved) {
+  precache::PrecacheManifest manifest = CreateManifestData(1);
+  InitializePrecacheResource(manifest.add_resource(),
+                             "http://cdn.google.com/script.js", 0.9);
+  InitializePrecacheResource(manifest.add_resource(),
+                             "http://cdn.google.com/style.css", 0.75);
+
+  precache::PrecacheManifest manifest_with_unknown_fields(manifest);
+  manifest_with_unknown_fields.mutable_id()->mutable_unknown_fields()->append(
+      "DATA");
+  manifest_with_unknown_fields.mutable_unknown_fields()->append("DATA");
+  for (auto& resource : *manifest_with_unknown_fields.mutable_resource()) {
+    resource.mutable_unknown_fields()->append("DATA");
+  }
+
+  int manifest_size = manifest.ByteSize();
+  auto match_size = [manifest_size](const precache::PrecacheManifest& m) {
+    return m.ByteSize() == manifest_size;
+  };
+  EXPECT_CALL(
+      *mock_tables_.get(),
+      UpdateManifestData("google.com",
+                         testing::AllOf(manifest, testing::Truly(match_size))));
+
+  predictor_->OnManifestFetched("google.com", manifest_with_unknown_fields);
+}
+
 TEST_F(ResourcePrefetchPredictorTest, DeleteUrls) {
   // Add some dummy entries to cache.
   predictor_->url_table_cache_->insert(
       std::make_pair("http://www.google.com/page1.html",
                      CreatePrefetchData("http://www.google.com/page1.html")));
   predictor_->url_table_cache_->insert(
       std::make_pair("http://www.google.com/page2.html",
                      CreatePrefetchData("http://www.google.com/page2.html")));
   predictor_->url_table_cache_->insert(std::make_pair(
       "http://www.yahoo.com/", CreatePrefetchData("http://www.yahoo.com/")));
@@ skipping 780 matching lines @@
   predictor_->host_table_cache_->insert(
       std::make_pair(google.primary_key(), google));
 
   predictor_->StartPrefetching(GURL(main_frame_url), PrefetchOrigin::EXTERNAL);
   predictor_->StopPrefetching(GURL(main_frame_url));
   histogram_tester_->ExpectTotalCount(
       internal::kResourcePrefetchPredictorPrefetchingDurationHistogram, 1);
 }
 
 }  // namespace predictors