Allow data urls to have a fragment part.

net/base/data_url.cc

Index: net/base/data_url.cc
diff --git a/net/base/data_url.cc b/net/base/data_url.cc
index 9699136c85fd5cff3f62270b9336e35bd4d51f74..f22d5e490053dfd2f315c363e83e516f81a8f812 100644
--- a/net/base/data_url.cc
+++ b/net/base/data_url.cc
@@ -23,20 +23,18 @@ bool DataURL::Parse(const GURL& url, std::string* mime_type,
                     std::string* charset, std::string* data) {
   DCHECK(mime_type->empty());
   DCHECK(charset->empty());
-  std::string::const_iterator begin = url.spec().begin();
-  std::string::const_iterator end = url.spec().end();
+  DCHECK(url.SchemeIs("data"));
 
-  std::string::const_iterator after_colon = std::find(begin, end, ':');
-  if (after_colon == end)
-    return false;
-  ++after_colon;
+  const std::string path = url.path();

[asanka, 2014/08/13 15:41:41]

Shall we try to avoid constructing strings? I believe that's why the previous
code was using the iterators over the spec.

i.e. begin = url.possibly_invalid_spec().begin() +
url.parsed_for_possibly_invalid_spec().path.begin;
   end = ....path.end();

You'd also want to DCHECK(url.is_valid()).

+  std::string::const_iterator begin = path.begin();
+  std::string::const_iterator end = path.end();
 
-  std::string::const_iterator comma = std::find(after_colon, end, ',');
+  std::string::const_iterator comma = std::find(begin, end, ',');
   if (comma == end)
     return false;
 
   std::vector<std::string> meta_data;
-  std::string unparsed_meta_data(after_colon, comma);
+  std::string unparsed_meta_data(begin, comma);
   base::SplitString(unparsed_meta_data, ';', &meta_data);
 
   std::vector<std::string>::iterator iter = meta_data.begin();