Improved function entry hook coverage

src/snapshot-common.cc

Index: src/snapshot-common.cc
diff --git a/src/snapshot-common.cc b/src/snapshot-common.cc
old mode 100644
new mode 100755
index a8806f053fe13b2f796c9009eb05c48843217394..8dab3671cef9f16451e6ea26a86f113b43d1e3b2
--- a/src/snapshot-common.cc
+++ b/src/snapshot-common.cc
@@ -82,6 +82,12 @@ void Snapshot::ReserveSpaceForLinkedInSnapshot(Deserializer* deserializer) {
 
 
 bool Snapshot::Initialize(const char* snapshot_file) {

[danno, 2013/06/13 15:33:44]

If you need the isolate, you should pass it in as a parameter.

[Sigurður Ásgeirsson, 2013/06/19 20:51:51]

Moved this to the sole non-test user of this function, as I'm not sure how to
otherwise untangle this.

+  // If the isolate has a function entry hook, it needs to re-build all its
+  // code stubs with entry hooks embedded.
+  Isolate* isolate = Isolate::UncheckedCurrent();
+  if (isolate && isolate->HasFunctionEntryHook())

[danno, 2013/06/13 15:33:44]

nit: isolate != NULL

[Sigurður Ásgeirsson, 2013/06/19 20:51:51]

Done.

+    return false;
+
   if (snapshot_file) {
     int len;
     byte* str = ReadBytes(snapshot_file, &len);