Wrong filename on file upload

url/url_parse_unittest.cc

 // Copyright 2013 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 "url/url_parse.h"
 
 #include "base/basictypes.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/url_parse.h"
 
@@ skipping 489 matching lines @@
     {"http://www.google.com", NULL},
     {"http://www.google.com/", ""},
     {"http://www.google.com/search", "search"},
     {"http://www.google.com/search/", ""},
     {"http://www.google.com/foo/bar.html?baz=22", "bar.html"},
     {"http://www.google.com/foo/bar.html#ref", "bar.html"},
     {"http://www.google.com/search/;param", ""},
     {"http://www.google.com/foo/bar.html;param#ref", "bar.html"},
     {"http://www.google.com/foo/bar.html;foo;param#ref", "bar.html;foo"},
     {"http://www.google.com/foo/bar.html?query#ref", "bar.html"},
+    // Test cases added to verify Issue - 323156

[brettw, 2014/09/16 22:37:10]

I don't think you need this comment. Can you add some tests with multiple
semicolons also?

[arun87.kumar, 2014/09/17 11:39:12]

Done.

+    {"http://www.google.com/foo;/bar.html", "bar.html"},
+    {"http://www.google.com/foo;/", ""},
+    {"http://www.google.com/foo;", "foo"},
+    {"http://www.google.com/;", ""},
   };
 
   for (size_t i = 0; i < ARRAYSIZE(file_cases); i++) {
     const char* url = file_cases[i].input;
     int len = static_cast<int>(strlen(url));
 
     Parsed parsed;
     ParseStandardURL(url, len, &parsed);
 
     Component file_name;
@@ skipping 161 matching lines @@
     // The remaining components are never used for filesystem urls.
     ExpectInvalidComponent(parsed.username);
     ExpectInvalidComponent(parsed.password);
     ExpectInvalidComponent(parsed.host);
     ExpectInvalidComponent(parsed.port);
   }
 }
 
 }  // namespace
 }  // namespace url