[Isolates] Fix the crash in AssertNoContextChange ctor when current context i...

src/isolate.h

 // Copyright 2010 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 1162 matching lines @@
 #endif
   SaveContext* prev_;
   Address js_sp_;  // The top JS frame's sp when saving context.
 };
 
 
 class AssertNoContextChange BASE_EMBEDDED {
 #ifdef DEBUG
  public:
   AssertNoContextChange() :
-      context_(Isolate::Current()->context()) {
+          isolate_(Isolate::Current()),

[Vitaly Repeshko, 2011/03/17 18:13:43]

Convert tabs to spaces.

+          scope_(isolate_),
+      context_(isolate_->context(), isolate_) {

[Vitaly Repeshko, 2011/03/17 18:13:43]

Just pass Isolate::Current(). No need to save it in a field.

   }
 
   ~AssertNoContextChange() {
     ASSERT(Isolate::Current()->context() == *context_);
   }
 
  private:
+  Isolate* isolate_;
   HandleScope scope_;
   Handle<Context> context_;
 #else
  public:
   AssertNoContextChange() { }
 #endif
 };
 
 
 class ExecutionAccess BASE_EMBEDDED {
@@ skipping 93 matching lines @@
 
 } }  // namespace v8::internal
 
 // TODO(isolates): Get rid of these -inl.h includes and place them only where
 //                 they're needed.
 #include "allocation-inl.h"
 #include "zone-inl.h"
 #include "frames-inl.h"
 
 #endif  // V8_ISOLATE_H_