Fix order of manifest_url & document_url in ParseManifestUrls()

content/renderer/manifest/manifest_parser_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 "content/renderer/manifest/manifest_parser.h"
 
 #include <stdint.h>
 
 #include <memory>
 
@@ skipping 11 matching lines @@
 }
 
 } // anonymous namespace
 
 class ManifestParserTest : public testing::Test  {
  protected:
   ManifestParserTest() {}
   ~ManifestParserTest() override {}
 
   Manifest ParseManifestWithURLs(const base::StringPiece& data,
-                                 const GURL& document_url,
-                                 const GURL& manifest_url) {
-    ManifestParser parser(data, document_url, manifest_url);
+                                 const GURL& manifest_url,
+                                 const GURL& document_url) {
+    ManifestParser parser(data, manifest_url, document_url);
     parser.Parse();
     std::vector<ManifestDebugInfo::Error> errors;
     parser.TakeErrors(&errors);
 
     errors_.clear();
     for (const auto& error : errors)
       errors_.push_back(error.message);
     return parser.manifest();
   }
 
   Manifest ParseManifest(const base::StringPiece& data) {
     return ParseManifestWithURLs(
-        data, default_document_url, default_manifest_url);
+        data, default_manifest_url, default_document_url);
   }
 
   const std::vector<std::string>& errors() const {
     return errors_;
   }
 
   unsigned int GetErrorCount() const {
     return errors_.size();
   }
 
@@ skipping 447 matching lines @@
     Manifest manifest = ParseManifest("{ \"icons\": [ { \"icons\": [] } ] }");
     EXPECT_EQ(manifest.icons.size(), 0u);
     EXPECT_TRUE(manifest.IsEmpty());
     EXPECT_EQ(0u, GetErrorCount());
   }
 
   // Smoke test: if icon with empty src, it will be present in the list.
   {
     Manifest manifest = ParseManifest("{ \"icons\": [ { \"src\": \"\" } ] }");
     EXPECT_EQ(manifest.icons.size(), 1u);
-    EXPECT_EQ(manifest.icons[0].src.spec(), "http://foo.com/index.html");
+    EXPECT_EQ(manifest.icons[0].src.spec(), "http://foo.com/manifest.json");
     EXPECT_FALSE(manifest.IsEmpty());
     EXPECT_EQ(0u, GetErrorCount());
   }
 
   // Smoke test: if one icons with valid src, it will be present in the list.
   {
     Manifest manifest =
         ParseManifest("{ \"icons\": [{ \"src\": \"foo.jpg\" }] }");
     EXPECT_EQ(manifest.icons.size(), 1u);
     EXPECT_EQ(manifest.icons[0].src.spec(), "http://foo.com/foo.jpg");
@@ skipping 734 matching lines @@
   {
     Manifest manifest = ParseManifest("{ \"gcm_sender_id\": 42 }");
     EXPECT_TRUE(manifest.gcm_sender_id.is_null());
     EXPECT_EQ(1u, GetErrorCount());
     EXPECT_EQ("property 'gcm_sender_id' ignored, type string expected.",
               errors()[0]);
   }
 }
 
 } // namespace content