Noissue - generalize storing of `JsValue`s in `JsEngine`

include/AdblockPlus/JsEngine.h

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 213 matching lines @@
      * Stores `JsValue`s in a way they don't keep a strong reference to
      * `JsEngine` and which are destroyed when `JsEngine` is destroyed. These
      * methods should be used when one needs to carry a JsValue in a callback
      * directly or indirectly passed to `JsEngine`.
      * The method is thread-safe.
      * @param `JsValueList` to store.
      * @return `JsWeakValuesID` of stored values which allows to restore them
      * later.
      */
     JsWeakValuesID StoreJsValues(const JsValueList& values);
+
     /**
      * Extracts and removes from `JsEngine` earlier stored `JsValue`s.
      * The method is thread-safe.
      * @param id `JsWeakValuesID` of values.
      * @return `JsValueList` of stored values.
      */
     JsValueList TakeJsValues(const JsWeakValuesID& id);
 
     /*
      * Private functionality required to implement timers.
@@ skipping 97 matching lines @@
     std::mutex eventCallbacksMutex;
     mutable std::mutex isConnectionAllowedMutex;
     IsConnectionAllowedCallback isConnectionAllowed;
     JsWeakValuesLists jsWeakValuesLists;
     std::mutex jsWeakValuesListsMutex;
     TimerPtr timer;
   };
 }
 
 #endif