[WIP] make GURL::path() return a StringPiece

net/base/sdch_dictionary.cc

 // Copyright 2015 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 "net/base/sdch_dictionary.h"
 
+#include "base/strings/string_util.h"
 #include "base/time/clock.h"
 #include "base/time/default_clock.h"
 #include "base/values.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
 
 namespace {
 
 bool DomainMatch(const GURL& gurl, const std::string& restriction) {
   // TODO(jar): This is not precisely a domain match definition.
   return gurl.DomainIs(restriction);
@@ skipping 116 matching lines @@
 
   // TODO(jar): Remove overly restrictive failsafe test (added per security
   // review) when we have a need to be more general.
   if (!target_url.SchemeIsHTTPOrHTTPS())
     return SDCH_ATTEMPT_TO_DECODE_NON_HTTP_DATA;
 
   return SDCH_OK;
 }
 
 // static
-bool SdchDictionary::PathMatch(const std::string& path,
-                               const std::string& restriction) {
+bool SdchDictionary::PathMatch(const base::StringPiece& path,
+                               const base::StringPiece& restriction) {
   /*  Must be either:
    *  1. P2 is equal to P1
    *  2. P2 is a prefix of P1 and either the final character in P2 is "/"
    *     or the character following P2 in P1 is "/".
    */
   if (path == restriction)
     return true;
   size_t prefix_length = restriction.size();
   if (prefix_length > path.size())
     return false;  // Can't be a prefix.
-  if (0 != path.compare(0, prefix_length, restriction))
+  if (!base::StartsWith(path, restriction, base::CompareCase::SENSITIVE))
     return false;
   return restriction[prefix_length - 1] == '/' || path[prefix_length] == '/';
 }
 
 bool SdchDictionary::Expired() const {
   return base::Time::Now() > expiration_;
 }
 
 }  // namespace net