src: avoid copy when creating Blob

PR-URL: #44616
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Daeyeon Jeong <[email protected]>

Author: tniessen

src/node_blob.cc

@@ -71,11 +71,9 @@ bool Blob::HasInstance(Environment* env, v8::Local<v8::Value> object) {
   return GetConstructorTemplate(env)->HasInstance(object);
 }
 
-BaseObjectPtr<Blob> Blob::Create(
-    Environment* env,
-    const std::vector<BlobEntry> store,
-    size_t length) {
-
+BaseObjectPtr<Blob> Blob::Create(Environment* env,
+                                 const std::vector<BlobEntry>& store,
+                                 size_t length) {
   HandleScope scope(env->isolate());
 
   Local<Function> ctor;

src/node_blob.h

@@ -45,16 +45,13 @@ class Blob : public BaseObject {
   static v8::Local<v8::FunctionTemplate> GetConstructorTemplate(
       Environment* env);
 
-  static BaseObjectPtr<Blob> Create(
-      Environment* env,
-      const std::vector<BlobEntry> store,
-      size_t length);
+  static BaseObjectPtr<Blob> Create(Environment* env,
+                                    const std::vector<BlobEntry>& store,
+                                    size_t length);
 
   static bool HasInstance(Environment* env, v8::Local<v8::Value> object);
 
-  const std::vector<BlobEntry> entries() const {
-    return store_;
-  }
+  const std::vector<BlobEntry>& entries() const { return store_; }
 
   void MemoryInfo(MemoryTracker* tracker) const override;
   SET_MEMORY_INFO_NAME(Blob)