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

extensions/renderer/script_context.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 "extensions/renderer/script_context.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/strings/string_split.h"
@@ skipping 114 matching lines @@
 }
 
 // static
 bool ScriptContext::IsSandboxedPage(const GURL& url) {
   // TODO(kalman): This is checking the wrong thing. See comment in
   // HasAccessOrThrowError.
   if (url.SchemeIs(kExtensionScheme)) {
     const Extension* extension =
         RendererExtensionRegistry::Get()->GetByID(url.host());
     if (extension) {
-      return SandboxedPageInfo::IsSandboxedPage(extension, url.path());
+      return SandboxedPageInfo::IsSandboxedPage(extension,
+                                                url.path().as_string());
     }
   }
   return false;
 }
 
 void ScriptContext::Invalidate() {
   DCHECK(thread_checker_.CalledOnValidThread());
   CHECK(is_valid_);
   is_valid_ = false;
 
@@ skipping 249 matching lines @@
   // sandboxed frames, for example content scripts. Don't let them execute API
   // functions.
   //
   // In any case, this check is silly. The frame's document's security origin
   // already tells us if it's sandboxed. The only problem is that until
   // crbug.com/466373 is fixed, we don't know the security origin up-front and
   // may not know it here, either.
   //
   // [1] citation needed. This ScriptContext should already be in a state that
   // doesn't allow this, from ScriptContextSet::ClassifyJavaScriptContext.
-  if (extension() &&
-      SandboxedPageInfo::IsSandboxedPage(extension(), url_.path())) {
+  if (extension() && SandboxedPageInfo::IsSandboxedPage(
+                         extension(), url_.path().as_string())) {
     static const char kMessage[] =
         "%s cannot be used within a sandboxed frame.";
     std::string error_msg = base::StringPrintf(kMessage, name.c_str());
     isolate()->ThrowException(v8::Exception::Error(
         v8::String::NewFromUtf8(isolate(), error_msg.c_str())));
     return false;
   }
 
   Feature::Availability availability = GetAvailability(name);
   if (!availability.is_available()) {
@@ skipping 95 matching lines @@
     v8::Local<v8::Value> argv[]) {
   return context_->CallFunction(function, argc, argv);
 }
 
 gin::ContextHolder* ScriptContext::Runner::GetContextHolder() {
   v8::HandleScope handle_scope(context_->isolate());
   return gin::PerContextData::From(context_->v8_context())->context_holder();
 }
 
 }  // namespace extensions