Issue #4694 - Add mutex protection to JS event handling

src/JsEngineInternal.h

 
 #if !defined(ADBLOCK_PLUS_JS_ENGINE_INTERNAL_H)
 #define ADBLOCK_PLUS_JS_ENGINE_INTERNAL_H
 
 #include <AdblockPlus/JsEngine.h>
 #include <v8.h>
 #include <array>
 #include "AllocatedArray.h"
+#include "Event.h"
 #include "V8Upgrade.h"
 
 class PersistentValueArray
   : public AllocatedArray<V8PersistentNG<v8::Value>>
 {
   typedef AllocatedArray<V8PersistentNG<v8::Value>> Base;
   /**
    * Copy constructor deleted. Allocation is unique to single object.
    */
   PersistentValueArray(const PersistentValueArray&); // = delete;
@@ skipping 41 matching lines @@
   }
 };
 
 /**
  * \par Implementation Notes
  */
 class JsEngineInternal
   : public AdblockPlus::JsEngine
 {
   /**
+   * Allows implementations of public API functions to access internal members.
+   */
+  friend class AdblockPlus::JsEngine;
+
+  /**
    * Unique context associated with this isolate.
    */
   v8::Persistent<v8::Context> context;
 
+  /**
+   * Event manager provides implementation of event set, remove, and trigger.
+   */
+  EventManager eventMan;

[sergei, 2017/03/15 12:17:21]

what about "ager"?

+
 public:
   JsEngineInternal(const AdblockPlus::ScopedV8IsolatePtr& isolate);
 
   /**
    * Retrieve our persistent v8 context as a local handle.
    */
   v8::Local<v8::Context> GetContextAsLocal() const;
 
   /**
    * Retrieve the global object of our context.
@@ skipping 78 matching lines @@
    */
   const v8::HandleScope handleScope;
   const v8::Context::Scope contextScope;
 public:
   V8ExecutionScope(JsEngineInternal* engine);
 };
 
 
 
 #endif