Fix use-after-scope issues with incorrect usage of ManifestParser.

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 53 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(ManifestParserTest);
 };
 
 const GURL ManifestParserTest::default_document_url(
     "http://foo.com/index.html");
 const GURL ManifestParserTest::default_manifest_url(
     "http://foo.com/manifest.json");
 
 TEST_F(ManifestParserTest, CrashTest) {
   // Passing temporary variables should not crash.
-  ManifestParser parser("{\"start_url\": \"/\"}",
-                        GURL("http://example.com"),
-                        GURL("http://example.com"));
+  const base::StringPiece json = "{\"start_url\": \"/\"}";
+  GURL url("http://example.com");
+  ManifestParser parser(json, url, url);
   parser.Parse();
   std::vector<ManifestDebugInfo::Error> errors;
   parser.TakeErrors(&errors);
 
   // .Parse() should have been call without crashing and succeeded.
   EXPECT_EQ(0u, errors.size());
   EXPECT_FALSE(parser.manifest().IsEmpty());
 }
 
 TEST_F(ManifestParserTest, EmptyStringNull) {
@@ skipping 1327 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