Fix issue 1417: check for script source availability when enumerating optimized functions.

src/log.cc

Index: src/log.cc
diff --git a/src/log.cc b/src/log.cc
index f9c3b8830babd31a06ff9eefe67c276d18e34d4a..377dcb57cdb1ddfd824d6057b812de745205dc9b 100644
--- a/src/log.cc
+++ b/src/log.cc
@@ -1545,8 +1545,11 @@ class EnumerateOptimizedFunctionsVisitor: public OptimizedFunctionVisitor {
   virtual void LeaveContext(Context* context) {}
 
   virtual void VisitFunction(JSFunction* function) {
+    SharedFunctionInfo* sfi = SharedFunctionInfo::cast(function->shared());
+    if (sfi->script()->IsScript()

[Kasper Lund, 2011/06/17 05:20:21]

Maybe cache sfi->script() in a new local? That will make it obvious that
script() returns a non Script* and help explain why the Script::cast is required
without adding more comments.

[mnaganov (inactive), 2011/06/17 08:37:41]

Done.

+        && !Script::cast(sfi->script())->HasValidSource()) return;
     if (sfis_ != NULL) {
-      sfis_[*count_] = Handle<SharedFunctionInfo>(function->shared());
+      sfis_[*count_] = Handle<SharedFunctionInfo>(sfi);
     }
     if (code_objects_ != NULL) {
       ASSERT(function->code()->kind() == Code::OPTIMIZED_FUNCTION);