summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoryangguo <yangguo@chromium.org>2014-10-15 00:23:26 -0700
committerCommit bot <commit-bot@chromium.org>2014-10-15 07:23:45 +0000
commit0d0c476f5dffd58b9f93be01db23c3847dbf0865 (patch)
tree159267f7d66eeb45f10bc568b7dcd1de163321c0
parent755aaca0859cf9db13fb17d99e987ade0553af6a (diff)
downloadchromium_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.gyp4
-rw-r--r--extensions/renderer/BUILD.gn4
-rw-r--r--extensions/renderer/resource_bundle_source_map.cc4
-rw-r--r--extensions/renderer/resource_bundle_source_map.h2
-rw-r--r--extensions/renderer/static_v8_external_ascii_string_resource.cc25
-rw-r--r--extensions/renderer/static_v8_external_one_byte_string_resource.cc26
-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.cc3
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.