Skip to content

Commit

Permalink
http2: elevate v8 namespaces of repeated references
Browse files Browse the repository at this point in the history
PR-URL: #24453
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
  • Loading branch information
codegagan authored and Trott committed Nov 18, 2018
1 parent a19a268 commit 38e5946
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions src/node_http2.cc
Expand Up @@ -12,11 +12,13 @@
namespace node { namespace node {


using v8::ArrayBuffer; using v8::ArrayBuffer;
using v8::ArrayBufferCreationMode;
using v8::Boolean; using v8::Boolean;
using v8::Context; using v8::Context;
using v8::Float64Array; using v8::Float64Array;
using v8::Function; using v8::Function;
using v8::Integer; using v8::Integer;
using v8::NewStringType;
using v8::Number; using v8::Number;
using v8::ObjectTemplate; using v8::ObjectTemplate;
using v8::String; using v8::String;
Expand Down Expand Up @@ -1401,11 +1403,11 @@ void Http2Session::HandleAltSvcFrame(const nghttp2_frame* frame) {
Integer::New(isolate, id), Integer::New(isolate, id),
String::NewFromOneByte(isolate, String::NewFromOneByte(isolate,
altsvc->origin, altsvc->origin,
v8::NewStringType::kNormal, NewStringType::kNormal,
altsvc->origin_len).ToLocalChecked(), altsvc->origin_len).ToLocalChecked(),
String::NewFromOneByte(isolate, String::NewFromOneByte(isolate,
altsvc->field_value, altsvc->field_value,
v8::NewStringType::kNormal, NewStringType::kNormal,
altsvc->field_value_len).ToLocalChecked(), altsvc->field_value_len).ToLocalChecked(),
}; };


Expand All @@ -1430,7 +1432,7 @@ void Http2Session::HandleOriginFrame(const nghttp2_frame* frame) {
const nghttp2_origin_entry& entry = origin->ov[i]; const nghttp2_origin_entry& entry = origin->ov[i];
origin_v[i] = origin_v[i] =
String::NewFromOneByte( String::NewFromOneByte(
isolate, entry.origin, v8::NewStringType::kNormal, entry.origin_len) isolate, entry.origin, NewStringType::kNormal, entry.origin_len)
.ToLocalChecked(); .ToLocalChecked();
} }
Local<Value> holder = Array::New(isolate, origin_v.data(), origin_v.size()); Local<Value> holder = Array::New(isolate, origin_v.data(), origin_v.size());
Expand Down Expand Up @@ -1813,7 +1815,7 @@ void Http2Session::OnStreamRead(ssize_t nread, const uv_buf_t& buf) {
ArrayBuffer::New(isolate, ArrayBuffer::New(isolate,
buf.base, buf.base,
nread, nread,
v8::ArrayBufferCreationMode::kInternalized); ArrayBufferCreationMode::kInternalized);


statistics_.data_received += nread; statistics_.data_received += nread;
ssize_t ret = Write(&stream_buf_, 1); ssize_t ret = Write(&stream_buf_, 1);
Expand Down Expand Up @@ -2316,7 +2318,7 @@ void HttpErrorString(const FunctionCallbackInfo<Value>& args) {
String::NewFromOneByte( String::NewFromOneByte(
env->isolate(), env->isolate(),
reinterpret_cast<const uint8_t*>(nghttp2_strerror(val)), reinterpret_cast<const uint8_t*>(nghttp2_strerror(val)),
v8::NewStringType::kInternalized).ToLocalChecked()); NewStringType::kInternalized).ToLocalChecked());
} }




Expand Down

0 comments on commit 38e5946

Please sign in to comment.