[parser] Refactor AstTraversalVisitor

src/debug/liveedit.h

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef V8_DEBUG_LIVEEDIT_H_
 #define V8_DEBUG_LIVEEDIT_H_
 
 
 // Live Edit feature implementation.
 // User should be able to change script on already running VM. This feature
 // matches hot swap features in other frameworks.
 //
 // The basic use-case is when user spots some mistake in function body
 // from debugger and wishes to change the algorithm without restart.
 //
 // A single change always has a form of a simple replacement (in pseudo-code):
 //   script.source[positions, positions+length] = new_string;
 // Implementation first determines, which function's body includes this
 // change area. Then both old and new versions of script are fully compiled
 // in order to analyze, whether the function changed its outer scope
 // expectations (or number of parameters). If it didn't, function's code is
 // patched with a newly compiled code. If it did change, enclosing function
 // gets patched. All inner functions are left untouched, whatever happened
 // to them in a new script version. However, new version of code will
 // instantiate newly compiled functions.
 
 
 #include "src/allocation.h"
+#include "src/ast/ast-traversal-visitor.h"
 #include "src/compiler.h"
 
 namespace v8 {
 namespace internal {
 
 // This class collects some specific information on structure of functions
 // in a particular script.
 //
 // The primary interest of the Tracker is to record function scope structures
 // in order to analyze whether function code may be safely patched (with new
 // code successfully reading existing data from function scopes). The Tracker
 // also collects compiled function codes.
-class LiveEditFunctionTracker : public AstTraversalVisitor {
+class LiveEditFunctionTracker
+    : public AstTraversalVisitor<LiveEditFunctionTracker> {
  public:
   // Traverses the entire AST, and records information about all
   // FunctionLiterals for further use by LiveEdit code patching. The collected
   // information is returned as a serialized array.
   static Handle<JSArray> Collect(FunctionLiteral* node, Handle<Script> script,
                                  Zone* zone, Isolate* isolate);
 
-  virtual ~LiveEditFunctionTracker() {}
-  void VisitFunctionLiteral(FunctionLiteral* node) override;
+ protected:
+  friend AstTraversalVisitor<LiveEditFunctionTracker>;
+  void VisitFunctionLiteral(FunctionLiteral* node);
 
  private:
   LiveEditFunctionTracker(Handle<Script> script, Zone* zone, Isolate* isolate);
 
   void FunctionStarted(FunctionLiteral* fun);
   void FunctionDone(Handle<SharedFunctionInfo> shared, Scope* scope);
   Handle<Object> SerializeFunctionScope(Scope* scope);
 
   Handle<Script> script_;
   Zone* zone_;
@@ skipping 309 matching lines @@
   static const int kSharedInfoOffset_ = 3;
   static const int kSize_ = 4;
 
   friend class JSArrayBasedStruct<SharedInfoWrapper>;
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif /* V8_DEBUG_LIVEEDIT_H_ */