nodejs
diff --git a/‎src/encoding_binding.cc‎
Lines changed: 5 additions & 3 deletions b/‎src/encoding_binding.cc‎
Lines changed: 5 additions & 3 deletions
diff --git a/‎test/cctest/test_encoding_binding.cc‎
Lines changed: 22 additions & 1 deletion b/‎test/cctest/test_encoding_binding.cc‎
Lines changed: 22 additions & 1 deletion
diff --git a/‎test/parallel/test-util-text-decoder.js‎
Lines changed: 17 additions & 0 deletions b/‎test/parallel/test-util-text-decoder.js‎
Lines changed: 17 additions & 0 deletions
@@ -286,9 +286,11 @@ void BindingData::DecodeLatin1(const FunctionCallbackInfo<Value>& args) {
         env->isolate(), "The encoded data was not valid for encoding latin1");
   }
 
-  Local<Object> buffer_result =
-      node::Buffer::Copy(env, result.c_str(), written).ToLocalChecked();
-  args.GetReturnValue().Set(buffer_result);
+  Local<String> output =
+      String::NewFromUtf8(
+          env->isolate(), result.c_str(), v8::NewStringType::kNormal, written)
+          .ToLocalChecked();
+  args.GetReturnValue().Set(output);
 }
 
 }  // namespace encoding_binding
 
@@ -26,7 +26,7 @@ bool RunDecodeLatin1(Environment* env,
     return false;
   }
 
-  *result = try_catch.Exception();
+  *result = args[0];
   return true;
 }
 
@@ -151,5 +151,26 @@ TEST_F(EncodingBindingTest, DecodeLatin1_BOMPresent) {
   EXPECT_STREQ(*utf8_result, "Áéó");
 }
 
+TEST_F(EncodingBindingTest, DecodeLatin1_ReturnsString) {
+  Environment* env = CreateEnvironment();
+  Isolate* isolate = env->isolate();
+  HandleScope handle_scope(isolate);
+
+  const uint8_t latin1_data[] = {0xC1, 0xE9, 0xF3};
+  Local<ArrayBuffer> ab = ArrayBuffer::New(isolate, sizeof(latin1_data));
+  memcpy(ab->GetBackingStore()->Data(), latin1_data, sizeof(latin1_data));
+
+  Local<Uint8Array> array = Uint8Array::New(ab, 0, sizeof(latin1_data));
+  Local<Value> args[] = {array};
+
+  Local<Value> result;
+  ASSERT_TRUE(RunDecodeLatin1(env, args, false, false, &result));
+
+  ASSERT_TRUE(result->IsString());
+
+  String::Utf8Value utf8_result(isolate, result);
+  EXPECT_STREQ(*utf8_result, "Áéó");
+}
+
 }  // namespace encoding_binding
 }  // namespace node
@@ -0,0 +1,17 @@
+'use strict';
+
+const common = require('../common');
+
+const test = require('node:test');
+const assert = require('node:assert');
+
+test('TextDecoder correctly decodes windows-1252 encoded data', { skip: !common.hasIntl }, () => {
+  const latin1Bytes = new Uint8Array([0xc1, 0xe9, 0xf3]);
+
+  const expectedString = 'Áéó';
+
+  const decoder = new TextDecoder('windows-1252');
+  const decodedString = decoder.decode(latin1Bytes);
+
+  assert.strictEqual(decodedString, expectedString);
+});
Original file line number	Diff line number	Diff line change
@@ -286,9 +286,11 @@ void BindingData::DecodeLatin1(const FunctionCallbackInfo<Value>& args) {
286	286	env->isolate(), "The encoded data was not valid for encoding latin1");
287	287	}
288	288
289		- Local<Object> buffer_result =
290		- node::Buffer::Copy(env, result.c_str(), written).ToLocalChecked();
291		- args.GetReturnValue().Set(buffer_result);
	289	+ Local<String> output =
	290	+ String::NewFromUtf8(
	291	+ env->isolate(), result.c_str(), v8::NewStringType::kNormal, written)
	292	+ .ToLocalChecked();
	293	+ args.GetReturnValue().Set(output);
292	294	}
293	295
294	296	} // namespace encoding_binding