[inspector] Split V8DebuggerScript implementation for wasm

src/inspector/v8-debugger-script.h

 /*
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 22 matching lines @@
 #include "src/base/macros.h"
 #include "src/inspector/string-16.h"
 
 #include "include/v8.h"
 #include "src/debug/debug-interface.h"
 
 namespace v8_inspector {
 
 class V8DebuggerScript {
  public:
-  V8DebuggerScript(v8::Isolate* isolate,
-                   v8::Local<v8::DebugInterface::Script> script,
-                   bool isLiveEdit);
-  V8DebuggerScript(String16 id, String16 url, String16 source);
-  ~V8DebuggerScript();
+  static std::unique_ptr<V8DebuggerScript> Create(
+      v8::Isolate* isolate, v8::Local<v8::DebugInterface::Script> script,
+      bool isLiveEdit);
+  static std::unique_ptr<V8DebuggerScript> CreateWasm(
+      v8::Local<v8::DebugInterface::Script> script, String16 id, String16 url,
+      String16 source);
+
+  virtual ~V8DebuggerScript();
 
   const String16& scriptId() const { return m_id; }
   const String16& url() const { return m_url; }
   bool hasSourceURL() const { return !m_sourceURL.isEmpty(); }
   const String16& sourceURL() const;
-  const String16& sourceMappingURL() const { return m_sourceMappingURL; }
-  String16 source(v8::Isolate*) const;
+  virtual const String16& sourceMappingURL() const = 0;
+  virtual String16 source(v8::Isolate*) const { return m_source; }
   const String16& hash(v8::Isolate*) const;
   int startLine() const { return m_startLine; }
   int startColumn() const { return m_startColumn; }
   int endLine() const { return m_endLine; }
   int endColumn() const { return m_endColumn; }
   int executionContextId() const { return m_executionContextId; }
-  const String16& executionContextAuxData() const {
-    return m_executionContextAuxData;
-  }
-  bool isLiveEdit() const { return m_isLiveEdit; }
+  virtual const String16& executionContextAuxData() const = 0;
+  virtual bool isLiveEdit() const = 0;
 
   void setSourceURL(const String16&);
-  void setSourceMappingURL(const String16&);
-  void setSource(v8::Local<v8::String>);
+  virtual void setSourceMappingURL(const String16&) {}

[kozy, 2016/11/30 18:34:37]

If sourceMappingUrl getter is abstract, lets make this method abstract too.

[Clemens Hammacher, 2016/12/02 17:44:13]

Done.

+  virtual void setSource(v8::Local<v8::String>) {}

[kozy, 2016/11/30 18:34:37]

Can we assign new source to m_source here?

[Clemens Hammacher, 2016/12/02 17:44:13]

Done. But note that this requires to include string-util.h in
v8-debugger-script.h.
Also, I am not sure in which situations setSource will be called on wasm
scripts. It will definitely not impact the code being executed.

 
-  bool getPossibleBreakpoints(
+  virtual bool getPossibleBreakpoints(
       const v8::DebugInterface::Location& start,
       const v8::DebugInterface::Location& end,
-      std::vector<v8::DebugInterface::Location>* locations);
+      std::vector<v8::DebugInterface::Location>* locations) = 0;
 
- private:
+ protected:
+  V8DebuggerScript(v8::Isolate*, String16 id, String16 url);
+
   String16 m_id;
   String16 m_url;
   String16 m_sourceURL;
-  String16 m_sourceMappingURL;
-  v8::Global<v8::String> m_sourceObj;
   String16 m_source;
   mutable String16 m_hash;
   int m_startLine = 0;
   int m_startColumn = 0;
   int m_endLine = 0;
   int m_endColumn = 0;
   int m_executionContextId = 0;
-  String16 m_executionContextAuxData;
-  bool m_isLiveEdit = false;
 
   v8::Isolate* m_isolate;
-  v8::Global<v8::DebugInterface::Script> m_script;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(V8DebuggerScript);
 };
 
 }  // namespace v8_inspector
 
 #endif  // V8_INSPECTOR_V8DEBUGGERSCRIPT_H_