Issue 5013 - Use const JsValue and pass reference where applicable

include/AdblockPlus/JsValue.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 120 matching lines @@
      * Invokes the value as a function (see `IsFunction()`) with single
      * parameter.
      * @param arg A single required parameter.
      * @return Value returned by the function.
      */
     JsValue Call(const JsValue& arg) const;
 
     v8::Local<v8::Value> UnwrapValue() const;
 
     /**
-     * Clone this value wrapper to new one. I still hold the same internal handle

[sergei, 2017/04/13 13:55:44]

What about
"Creates a new `JsValue` wrapper for the same JavaScript value."
?

[hub, 2017/04/13 15:46:00]

Acknowledged.

-     * @return Value wrapper clone
+     * Creates a new `JsValue` wrapper for the same JavaScript value.
+     * @return Value new `JsValue`
      */
     JsValue Clone() const;
   protected:
     JsEnginePtr jsEngine;
   private:
     JsValue(JsEnginePtr jsEngine, v8::Handle<v8::Value> value);
     void SetProperty(const std::string& name, v8::Handle<v8::Value> val);
     // Parameter args is not const because a pointer to its internal arrays is
     // passed to v8::Function::Call but the latter does not expect a const pointer.
     JsValue Call(std::vector<v8::Handle<v8::Value>>& args, v8::Local<v8::Object> thisObj) const;
 
     std::unique_ptr<v8::Persistent<v8::Value>> value;
   };
 }
 
 #endif