Merge lp:~mikemc/unity-scope-click/remove-dummy-progress-actions into lp:unity-scope-click

[Mike McCracken]

- Remove extra buttons that are not needed in new scopes API.

- Remove extra buttons that are not needed in new scopes API.

These specially named buttons were used in the old api as a hackaround to send hidden info.

[Alejandro J. Cura (alecu)]

Looks good

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:158
http://jenkins.qa.ubuntu.com/job/unity-scope-click-ci/322/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-scope-click-trusty-amd64-ci/223
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-scope-click-trusty-armhf-ci/220
        deb: http://jenkins.qa.ubuntu.com/job/unity-scope-click-trusty-armhf-ci/220/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-scope-click-ci/322/rebuild

review: Approve (continuous-integration)

=== modified file 'scope/click/preview.cpp'
--- scope/click/preview.cpp	2014-02-18 15:45:09 +0000
+++ scope/click/preview.cpp	2014-02-18 23:30:08 +0000
@@ -231,21 +231,6 @@
         tuple["dbus-object"] = object_path;
         progress.add_attribute("source", scopes::Variant(tuple));
         widgets.push_back(progress);
-
-        scopes::PreviewWidget buttons("buttons", "actions");
-        scopes::VariantBuilder builder;
-        builder.add_tuple(
-        {
-            {"id", scopes::Variant(click::Preview::Actions::DOWNLOAD_COMPLETED)},
-            {"label", scopes::Variant("*** download_completed")}
-        });
-        builder.add_tuple(
-        {
-            {"id", scopes::Variant(click::Preview::Actions::DOWNLOAD_FAILED)},
-            {"label", scopes::Variant("*** download_failed")}
-        });
-        buttons.add_attribute("actions", builder.end());
-        widgets.push_back(buttons);
     }
 
     buildDescriptionAndReviews(reply, widgets, details);