temporarily remove persistent destructor to allow trunk push

include/v8.h

 // Copyright 2012 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 531 matching lines @@
   template <class S, class M2>
   V8_INLINE(Persistent& operator=(const Persistent<S, M2>& that)) { // NOLINT
     Copy(that);
     return *this;
   }
   /**
    * The destructor will dispose the Persistent based on the
    * kResetInDestructor flags in the traits class.  Since not calling dispose
    * can result in a memory leak, it is recommended to always set this flag.
    */
-  V8_INLINE(~Persistent()) {
-    if (M::kResetInDestructor) Reset();
-  }
+//  V8_INLINE(~Persistent()) {

[Dmitry Lomov (no reviews), 2013/09/09 11:38:49]

Since it is a public API, please remove completely instead of commenting out
(just for the push)

+//    if (M::kResetInDestructor) Reset();
+//  }
 
   /**
    * If non-empty, destroy the underlying storage cell
    * IsEmpty() will return true after this call.
    */
   V8_INLINE(void Reset());
   template <class S>
   /**
    * If non-empty, destroy the underlying storage cell
    * and create a new one with the contents of other if other is non empty
@@ skipping 5865 matching lines @@
  */
 
 
 }  // namespace v8
 
 
 #undef TYPE_CHECK
 
 
 #endif  // V8_H_