Issue 1280 - Update v8

include/AdblockPlus/JsValue.h

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 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 37 matching lines @@
    */
   typedef std::vector<AdblockPlus::JsValuePtr> JsValueList;
 
   /**
    * Wrapper for JavaScript values.
    * See `JsEngine` for creating `JsValue` objects.
    */
   class JsValue
   {
     friend class JsEngine;
-  protected:
-    JsValue(const JsValuePtr& value);
   public:
     virtual ~JsValue();
 
     bool IsUndefined() const;
     bool IsNull() const;
     bool IsString() const;
     bool IsNumber() const;
     bool IsBool() const;
     bool IsObject() const;
     bool IsArray() const;
@@ skipping 49 matching lines @@
     /**
      * Invokes the value as a function (see `IsFunction()`).
      * @param params Optional list of parameters.
      * @param thisPtr Optional `this` value.
      * @return Value returned by the function.
      */
     JsValuePtr Call(const JsValueList& params = JsValueList(),
         AdblockPlus::JsValuePtr thisPtr = AdblockPlus::JsValuePtr()) const;
 
     v8::Local<v8::Value> UnwrapValue() const;
-  private:
-    JsValue(const JsEnginePtr& jsEngine, v8::Handle<v8::Value> value);
-    void SetProperty(const std::string& name, v8::Handle<v8::Value> val);
   protected:
+    JsValue(JsValuePtr value);
     JsEnginePtr jsEngine;
   private:

[Wladimir Palant, 2014/11/01 22:30:25]

Nit: Why not merge the two private sections?

[sergei, 2014/11/03 12:58:19]

merged

+    JsValue(JsEnginePtr jsEngine, v8::Handle<v8::Value> value);
+    void SetProperty(const std::string& name, v8::Handle<v8::Value> val);
     V8ValueHolder<v8::Value> value;
   };
 }
 
 #endif