From cdae1c1af5bf030c7588d959f99b5ab6847a1be9 Mon Sep 17 00:00:00 2001
From: "christian.plesner.hansen@gmail.com"
 <christian.plesner.hansen@gmail.com@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Date: Tue, 9 Sep 2008 07:57:43 +0000
Subject: [PATCH] Fixed issue 54, under some circumstances internal field count
 set on object templates did not take effect.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@226 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
---
 src/api.cc              |  6 ++++++
 test/cctest/test-api.cc | 18 ++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/src/api.cc b/src/api.cc
index 6ff2bd69eb..f8bd63f482 100644
--- a/src/api.cc
+++ b/src/api.cc
@@ -979,6 +979,12 @@ void ObjectTemplate::SetInternalFieldCount(int value) {
                 "Invalid internal field count")) {
     return;
   }
+  if (value > 0) {
+    // The internal field count is set by the constructor function's
+    // construct code, so we ensure that there is a constructor
+    // function to do the setting.
+    EnsureConstructor(this);
+  }
   Utils::OpenHandle(this)->set_internal_field_count(i::Smi::FromInt(value));
 }
 
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index d70fd8e8e0..a550e5ea36 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -4825,3 +4825,21 @@ THREADED_TEST(DisposeEnteredContext) {
     inner->Exit();
   }
 }
+
+
+// Regression test for issue 54, object templates with internal fields
+// but no accessors or interceptors did not get their internal field
+// count set on instances.
+THREADED_TEST(Regress54) {
+  v8::HandleScope outer;
+  LocalContext context;
+  static v8::Persistent<v8::ObjectTemplate> templ;
+  if (templ.IsEmpty()) {
+    v8::HandleScope inner;
+    v8::Handle<v8::ObjectTemplate> local = v8::ObjectTemplate::New();
+    local->SetInternalFieldCount(1);
+    templ = v8::Persistent<v8::ObjectTemplate>::New(inner.Close(local));
+  }
+  v8::Handle<v8::Object> result = templ->NewInstance();
+  CHECK_EQ(1, result->InternalFieldCount());
+}