API cleanup, get rid of JsObject and GetProperty() methods that don`t return JsValue

src/JsValue.cpp

 #include <AdblockPlus.h>
 
 namespace
 {
   std::string fromV8String(v8::Handle<v8::Value> value)
   {
     v8::String::Utf8Value stringValue(value);
     if (stringValue.length())
       return std::string(*stringValue, stringValue.length());
     else
       return std::string();
   }
 
   v8::Local<v8::String> toV8String(const std::string& str)
   {
     return v8::String::New(str.c_str(), str.length());
   }
 }
 
 AdblockPlus::JsValue::JsValue(AdblockPlus::JsEnginePtr jsEngine,
       v8::Handle<v8::Value> value)
     : jsEngine(jsEngine),
       value(v8::Persistent<v8::Value>::New(jsEngine->isolate, value))
 {
 }
 
+AdblockPlus::JsValue::JsValue(AdblockPlus::JsValuePtr value)
+    : jsEngine(value->jsEngine),
+      value(v8::Persistent<v8::Value>::New(jsEngine->isolate, value->value))
+{
+}
+
 AdblockPlus::JsValue::~JsValue()
 {
   value.Dispose(jsEngine->isolate);
 }
 
 bool AdblockPlus::JsValue::IsUndefined() const
 {
   const JsEngine::Context context(jsEngine);
   return value->IsUndefined();
 }
@@ skipping 168 matching lines @@
   const v8::TryCatch tryCatch;
   v8::Persistent<v8::Function> func = v8::Persistent<v8::Function>::Cast(value);
   v8::Local<v8::Value> result = func->Call(thisObj, argc, argv);
   delete argv;
 
   if (tryCatch.HasCaught())
     throw AdblockPlus::JsError(tryCatch.Exception(), tryCatch.Message());
 
   return JsValuePtr(new JsValue(jsEngine, result));
 }