nodejs
diff --git a/Collapse file
‎src/encoding_binding.cc‎
Copy file name to clipboardExpand all lines: src/encoding_binding.cc
+5-3Lines changed: 5 additions & 3 deletions b/Collapse file
‎src/encoding_binding.cc‎
Copy file name to clipboardExpand all lines: src/encoding_binding.cc
+5-3Lines changed: 5 additions & 3 deletions
diff --git a/Collapse file
‎test/cctest/test_encoding_binding.cc‎
Copy file name to clipboardExpand all lines: test/cctest/test_encoding_binding.cc
+22-1Lines changed: 22 additions & 1 deletion b/Collapse file
‎test/cctest/test_encoding_binding.cc‎
Copy file name to clipboardExpand all lines: test/cctest/test_encoding_binding.cc
+22-1Lines changed: 22 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-util-text-decoder.js‎
Copy file name to clipboard
+17Lines changed: 17 additions & 0 deletions b/Collapse file
‎test/parallel/test-util-text-decoder.js‎
Copy file name to clipboard
+17Lines 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`