diff options
author | yangguo <yangguo@chromium.org> | 2014-10-15 00:23:26 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-10-15 07:23:45 +0000 |
commit | 0d0c476f5dffd58b9f93be01db23c3847dbf0865 (patch) | |
tree | 159267f7d66eeb45f10bc568b7dcd1de163321c0 | |
parent | 755aaca0859cf9db13fb17d99e987ade0553af6a (diff) | |
download | chromium_src-0d0c476f5dffd58b9f93be01db23c3847dbf0865.zip chromium_src-0d0c476f5dffd58b9f93be01db23c3847dbf0865.tar.gz chromium_src-0d0c476f5dffd58b9f93be01db23c3847dbf0865.tar.bz2 |
Rename v8::ExternalAsciiStringResource to v8::ExternalOneByteStringResource.
The former is a deprecated alias for the latter and will be removed soon.
TBR=eroman@chromium.org
BUG=none
Review URL: https://codereview.chromium.org/640413003
Cr-Commit-Position: refs/heads/master@{#299661}
-rw-r--r-- | extensions/extensions.gyp | 4 | ||||
-rw-r--r-- | extensions/renderer/BUILD.gn | 4 | ||||
-rw-r--r-- | extensions/renderer/resource_bundle_source_map.cc | 4 | ||||
-rw-r--r-- | extensions/renderer/resource_bundle_source_map.h | 2 | ||||
-rw-r--r-- | extensions/renderer/static_v8_external_ascii_string_resource.cc | 25 | ||||
-rw-r--r-- | extensions/renderer/static_v8_external_one_byte_string_resource.cc | 26 | ||||
-rw-r--r-- | extensions/renderer/static_v8_external_one_byte_string_resource.h (renamed from extensions/renderer/static_v8_external_ascii_string_resource.h) | 15 | ||||
-rw-r--r-- | net/proxy/proxy_resolver_v8.cc | 3 |
8 files changed, 43 insertions, 40 deletions
diff --git a/extensions/extensions.gyp b/extensions/extensions.gyp index 1e51f2d..ee0a40dd 100644 --- a/extensions/extensions.gyp +++ b/extensions/extensions.gyp @@ -930,8 +930,8 @@ 'renderer/send_request_natives.h', 'renderer/set_icon_natives.cc', 'renderer/set_icon_natives.h', - 'renderer/static_v8_external_ascii_string_resource.cc', - 'renderer/static_v8_external_ascii_string_resource.h', + 'renderer/static_v8_external_one_byte_string_resource.cc', + 'renderer/static_v8_external_one_byte_string_resource.h', 'renderer/test_features_native_handler.cc', 'renderer/test_features_native_handler.h', 'renderer/user_gestures_native_handler.cc', diff --git a/extensions/renderer/BUILD.gn b/extensions/renderer/BUILD.gn index 01577ea2..ded00cf 100644 --- a/extensions/renderer/BUILD.gn +++ b/extensions/renderer/BUILD.gn @@ -132,8 +132,8 @@ source_set("renderer") { "send_request_natives.h", "set_icon_natives.cc", "set_icon_natives.h", - "static_v8_external_ascii_string_resource.cc", - "static_v8_external_ascii_string_resource.h", + "static_v8_external_one_byte_string_resource.cc", + "static_v8_external_one_byte_string_resource.h", "test_features_native_handler.cc", "test_features_native_handler.h", "user_gestures_native_handler.cc", diff --git a/extensions/renderer/resource_bundle_source_map.cc b/extensions/renderer/resource_bundle_source_map.cc index 954a6d7..9b0a85a 100644 --- a/extensions/renderer/resource_bundle_source_map.cc +++ b/extensions/renderer/resource_bundle_source_map.cc @@ -38,10 +38,10 @@ bool ResourceBundleSourceMap::Contains(const std::string& name) { v8::Handle<v8::String> ResourceBundleSourceMap::ConvertString( v8::Isolate* isolate, const base::StringPiece& string) { - // v8 takes ownership of the StaticV8ExternalAsciiStringResource (see + // v8 takes ownership of the StaticV8ExternalOneByteStringResource (see // v8::String::NewExternal()). return v8::String::NewExternal( - isolate, new StaticV8ExternalAsciiStringResource(string)); + isolate, new StaticV8ExternalOneByteStringResource(string)); } } // namespace extensions diff --git a/extensions/renderer/resource_bundle_source_map.h b/extensions/renderer/resource_bundle_source_map.h index 1e088cd..84a5411 100644 --- a/extensions/renderer/resource_bundle_source_map.h +++ b/extensions/renderer/resource_bundle_source_map.h @@ -12,7 +12,7 @@ #include "base/memory/linked_ptr.h" #include "base/strings/string_piece.h" #include "extensions/renderer/module_system.h" -#include "extensions/renderer/static_v8_external_ascii_string_resource.h" +#include "extensions/renderer/static_v8_external_one_byte_string_resource.h" #include "v8/include/v8.h" namespace ui { diff --git a/extensions/renderer/static_v8_external_ascii_string_resource.cc b/extensions/renderer/static_v8_external_ascii_string_resource.cc deleted file mode 100644 index 5d64d2e..0000000 --- a/extensions/renderer/static_v8_external_ascii_string_resource.cc +++ /dev/null @@ -1,25 +0,0 @@ -// 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/static_v8_external_ascii_string_resource.h" - -namespace extensions { - -StaticV8ExternalAsciiStringResource::StaticV8ExternalAsciiStringResource( - const base::StringPiece& buffer) - : buffer_(buffer) { -} - -StaticV8ExternalAsciiStringResource::~StaticV8ExternalAsciiStringResource() { -} - -const char* StaticV8ExternalAsciiStringResource::data() const { - return buffer_.data(); -} - -size_t StaticV8ExternalAsciiStringResource::length() const { - return buffer_.length(); -} - -} // namespace extensions diff --git a/extensions/renderer/static_v8_external_one_byte_string_resource.cc b/extensions/renderer/static_v8_external_one_byte_string_resource.cc new file mode 100644 index 0000000..3f54b5d --- /dev/null +++ b/extensions/renderer/static_v8_external_one_byte_string_resource.cc @@ -0,0 +1,26 @@ +// 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/static_v8_external_one_byte_string_resource.h" + +namespace extensions { + +StaticV8ExternalOneByteStringResource::StaticV8ExternalOneByteStringResource( + const base::StringPiece& buffer) + : buffer_(buffer) { +} + +StaticV8ExternalOneByteStringResource:: + ~StaticV8ExternalOneByteStringResource() { +} + +const char* StaticV8ExternalOneByteStringResource::data() const { + return buffer_.data(); +} + +size_t StaticV8ExternalOneByteStringResource::length() const { + return buffer_.length(); +} + +} // namespace extensions diff --git a/extensions/renderer/static_v8_external_ascii_string_resource.h b/extensions/renderer/static_v8_external_one_byte_string_resource.h index 850f36d..97923b8 100644 --- a/extensions/renderer/static_v8_external_ascii_string_resource.h +++ b/extensions/renderer/static_v8_external_one_byte_string_resource.h @@ -2,8 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. -#ifndef EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ASCII_STRING_RESOURCE_H_ -#define EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ASCII_STRING_RESOURCE_H_ +#ifndef EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ONE_BYTE_STRING_RESOURCE_H_ +#define EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ONE_BYTE_STRING_RESOURCE_H_ #include "base/compiler_specific.h" #include "base/strings/string_piece.h" @@ -14,11 +14,12 @@ namespace extensions { // A very simple implementation of v8::ExternalAsciiStringResource that just // wraps a buffer. The buffer must outlive the v8 runtime instance this resource // is used in. -class StaticV8ExternalAsciiStringResource - : public v8::String::ExternalAsciiStringResource { +class StaticV8ExternalOneByteStringResource + : public v8::String::ExternalOneByteStringResource { public: - explicit StaticV8ExternalAsciiStringResource(const base::StringPiece& buffer); - virtual ~StaticV8ExternalAsciiStringResource(); + explicit StaticV8ExternalOneByteStringResource( + const base::StringPiece& buffer); + virtual ~StaticV8ExternalOneByteStringResource(); virtual const char* data() const override; virtual size_t length() const override; @@ -29,4 +30,4 @@ class StaticV8ExternalAsciiStringResource } // namespace extensions -#endif // EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ASCII_STRING_RESOURCE_H_ +#endif // EXTENSIONS_RENDERER_STATIC_V8_EXTERNAL_ONE_BYTE_STRING_RESOURCE_H_ diff --git a/net/proxy/proxy_resolver_v8.cc b/net/proxy/proxy_resolver_v8.cc index 26f3e71..dc224cd 100644 --- a/net/proxy/proxy_resolver_v8.cc +++ b/net/proxy/proxy_resolver_v8.cc @@ -107,7 +107,8 @@ class V8ExternalStringFromScriptData }; // External string wrapper so V8 can access a string literal. -class V8ExternalASCIILiteral : public v8::String::ExternalAsciiStringResource { +class V8ExternalASCIILiteral + : public v8::String::ExternalOneByteStringResource { public: // |ascii| must be a NULL-terminated C string, and must remain valid // throughout this object's lifetime. |