Merge lp:~townsend/unity/fix-lp1243529 into lp:unity

[Christopher Townsend]

Support libxpathselect1.4.

Support libxpathselect1.4. This comes from https://code.launchpad.net/~3v1n0/unity/xpathselect-1.4/+merge/186092 and is basically just rev. 3507 of his branch to make things simple for now.

[Brandon Schaefer (brandontschaefer)]

Cool, LGTM

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:3581
http://jenkins.qa.ubuntu.com/job/unity-ci/473/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-trusty-amd64-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-trusty-armhf-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-trusty-i386-ci/4

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/unity-ci/473/rebuild

review: Approve (continuous-integration)

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2013-10-14 11:16:33 +0000
+++ CMakeLists.txt	2013-10-23 17:56:46 +0000
@@ -233,7 +233,7 @@
     nux-4.0>=4.0.1
     sigc++-2.0
     unity-misc>=0.4.0
-    xpathselect>=1.2
+    xpathselect=1.4
     zeitgeist-2.0
 )
 

=== modified file 'debian/control'
--- debian/control	2013-09-27 12:26:20 +0000
+++ debian/control	2013-10-23 17:56:46 +0000
@@ -48,7 +48,7 @@
                libxcb-icccm4-dev,
                libxfixes-dev (>= 1:5.0.1-1),
                libxi-dev (>= 2:1.7.1.901),
-               libxpathselect-dev (>=1.3),
+               libxpathselect-dev (>=1.4),
                libzeitgeist-2.0-dev,
                pkg-config,
                python (>= 2.7),
@@ -178,7 +178,7 @@
          python-testtools,
          python-xdg,
          python-xlib,
-         libxpathselect1.3 (>= 1.3),
+         libxpathselect1.4 (>= 1.4),
 Description: Autopiloted tests for Unity
  We test Unity automatically through autopilot, a framework which enables us
  to trigger keyboard and mouse events on the fly as well as introspecting

=== modified file 'unity-shared/DebugDBusInterface.cpp'
--- unity-shared/DebugDBusInterface.cpp	2013-05-14 02:25:47 +0000
+++ unity-shared/DebugDBusInterface.cpp	2013-10-23 17:56:46 +0000
@@ -17,18 +17,11 @@
  * Authored by: Alex Launi <[e-mail]>
  */
 
-#include <queue>
-#include <iostream>
 #include <fstream>
 #include <sstream>
-#include <iostream>
 #include <boost/algorithm/string.hpp>
-#include <boost/algorithm/string/split.hpp>
-#include <boost/algorithm/string/classification.hpp>
-#include <boost/bind.hpp>
 #include <NuxCore/Logger.h>
 #include <NuxCore/LoggingWriter.h>
-#include <xpathselect/node.h>
 #include <xpathselect/xpathselect.h>
 #include <dlfcn.h>
 
@@ -46,16 +39,21 @@
 namespace local
 {
   std::ofstream output_file;
-  void* xpathselect_driver_ = NULL;
+  void* xpathselect_driver_ = nullptr;
 
-  class IntrospectableAdapter: public xpathselect::Node
+  class IntrospectableAdapter : public std::enable_shared_from_this<IntrospectableAdapter>, public xpathselect::Node
   {
   public:
     typedef std::shared_ptr<IntrospectableAdapter> Ptr;
-    IntrospectableAdapter(Introspectable* node, std::string const& parent_path)
-    : node_(node)
+    IntrospectableAdapter(Introspectable* node, IntrospectableAdapter::Ptr const& parent = nullptr)
+      : node_(node)
+      , parent_(parent)
+      , full_path_((parent_ ? parent_->GetPath() : "") + "/" + GetName())
+    {}
+
+    int32_t GetId() const
     {
-      full_path_ = parent_path + "/" + GetName();
+      return GPOINTER_TO_INT(node_);
     }
 
     std::string GetName() const
@@ -68,6 +66,27 @@
       return full_path_;
     }
 
+    Node::Ptr GetParent() const
+    {
+      return parent_;
+    }
+
+    virtual bool MatchBooleanProperty(const std::string& name, bool value) const
+    {
+      return false;
+    }
+
+    virtual bool MatchIntegerProperty(const std::string& name, int32_t value) const
+    {
+      return false;
+    }
+
+    virtual bool MatchStringProperty(const std::string& name, const std::string& value) const
+    {
+      return false;
+    }
+
+
     bool MatchProperty(const std::string& name, const std::string& value) const
     {
       bool matches = false;
@@ -77,7 +96,7 @@
       g_variant_builder_add(&child_builder, "{sv}", "id", g_variant_new_uint64(node_->GetIntrospectionId()));
       node_->AddProperties(&child_builder);
       GVariant* prop_dict = g_variant_builder_end(&child_builder);
-      GVariant *prop_value = g_variant_lookup_value(prop_dict, name.c_str(), NULL);
+      GVariant* prop_value = g_variant_lookup_value(prop_dict, name.c_str(), NULL);
 
       if (prop_value != NULL)
       {
@@ -100,10 +119,7 @@
           case G_VARIANT_CLASS_BOOLEAN:
           {
             std::string value = boost::to_upper_copy(value);
-            bool p = value == "TRUE" ||
-                      value == "ON" ||
-                      value == "YES" ||
-                      value == "1";
+            bool p = value == "TRUE" || value == "ON" || value == "YES" || value == "1";
             matches = (g_variant_get_boolean(prop_value) == p);
           }
           break;
@@ -183,30 +199,34 @@
     std::vector<xpathselect::Node::Ptr> Children() const
     {
       std::vector<xpathselect::Node::Ptr> children;
-      for(auto child: node_->GetIntrospectableChildren())
-      {
-        children.push_back(std::make_shared<IntrospectableAdapter>(child, GetPath() ));
-      }
+      auto const& this_ptr = std::const_pointer_cast<IntrospectableAdapter>(shared_from_this());
+      for(auto const& child: node_->GetIntrospectableChildren())
+        children.push_back(std::make_shared<IntrospectableAdapter>(child, this_ptr));
+
       return children;
-
-    }
-
+    }
+
+    Introspectable* Node() const
+    {
+      return node_;
+    }
+
+  private:
     Introspectable* node_;
-  private:
+    IntrospectableAdapter::Ptr parent_;
     std::string full_path_;
   };
 
-  xpathselect::NodeList select_nodes(local::IntrospectableAdapter::Ptr root,
-                                     std::string const& query)
+  xpathselect::NodeVector select_nodes(IntrospectableAdapter::Ptr const& root, std::string const& query)
   {
-    if (xpathselect_driver_ == NULL)
-      xpathselect_driver_ = dlopen("libxpathselect.so.1.3", RTLD_LAZY);
+    if (!xpathselect_driver_)
+      xpathselect_driver_ = dlopen("libxpathselect.so.1.4", RTLD_LAZY);
 
     if (xpathselect_driver_)
     {
-      typedef decltype(&xpathselect::SelectNodes) entry_t;
+      typedef decltype(&xpathselect::SelectNodes) select_nodes_t;
       dlerror();
-      entry_t entry_point = (entry_t) dlsym(xpathselect_driver_, "SelectNodes");
+      auto SelectNodes = reinterpret_cast<select_nodes_t>(dlsym(xpathselect_driver_, "SelectNodes"));
       const char* err = dlerror();
       if (err)
       {
@@ -214,7 +234,7 @@
       }
       else
       {
-        return entry_point(root, query);
+        return SelectNodes(root, query);
       }
     }
     else
@@ -223,7 +243,7 @@
     }
 
     // Fallen through here as we've hit an error
-    return xpathselect::NodeList();
+    return xpathselect::NodeVector();
   }
 
   // This needs to be called at destruction to cleanup the dlopen
@@ -355,13 +375,12 @@
   GVariantBuilder  builder;
   g_variant_builder_init(&builder, G_VARIANT_TYPE("a(sv)"));
 
-  local::IntrospectableAdapter::Ptr root_node = std::make_shared<local::IntrospectableAdapter>(_parent_introspectable, std::string());
-  auto nodes = local::select_nodes(root_node, query);
-  for (auto n : nodes)
+  auto root_node = std::make_shared<local::IntrospectableAdapter>(_parent_introspectable);
+  for (auto const& n : local::select_nodes(root_node, query))
   {
-    auto p = std::static_pointer_cast<local::IntrospectableAdapter>(n);
+    auto p = std::static_pointer_cast<local::IntrospectableAdapter const>(n);
     if (p)
-      g_variant_builder_add(&builder, "(sv)", p->GetPath().c_str(), p->node_->Introspect());
+      g_variant_builder_add(&builder, "(sv)", p->GetPath().c_str(), p->Node()->Introspect());
   }
 
   return g_variant_new("(a(sv))", &builder);