perf: we don't need to 'really' insert a preview when providing json

if we have intercepting 'image' when generating the json, then we
don't need to really insert the image into the server side TreeView
at all, we just need to provide it in the client json payload.

so we generate each bitmap once, and its base64 png representation
once.

Change-Id: I1b6916b036a0b84ef4346ebf2141240c4ae5b706
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152675
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolan.mcnamara@collabora.com>

1 files changed, 4 insertions, 1 deletions

diff --git a/vcl/inc/salvtables.hxx b/vcl/inc/salvtables.hxx
index 65d9b4cb1a47..7ac28808ce3d 100644
--- a/vcl/inc/salvtables.hxx
+++ b/vcl/inc/salvtables.hxx
@@ -1921,6 +1921,7 @@ private:
     DECL_LINK(CommandHdl, const CommandEvent&, bool);
     DECL_LINK(TooltipHdl, SvTreeListEntry*, OUString);
     DECL_LINK(EntryAccessibleDescriptionHdl, SvTreeListEntry*, OUString);
+    DECL_LINK(DumpElemToPropertyTreeHdl, const ::IconView::json_prop_query&, bool);
 
 public:
     SalInstanceIconView(::IconView* pIconView, SalInstanceBuilder* pBuilder, bool bTakeOwnership);
@@ -1943,7 +1944,7 @@ public:
     virtual void connect_query_tooltip(const Link<const weld::TreeIter&, OUString>& rLink) override;
 
     virtual void
-    connect_get_property_tree_elem(const Link<const tools::json_prop_query&, bool>& rLink) override;
+    connect_get_property_tree_elem(const Link<const weld::json_prop_query&, bool>& rLink) override;
 
     virtual OUString get_selected_id() const override;
 
@@ -1974,6 +1975,8 @@ public:
 
     virtual OUString get_id(const weld::TreeIter& rIter) const override;
 
+    virtual OUString get_text(const weld::TreeIter& rIter) const override;
+
     virtual void clear() override;
 
     virtual ~SalInstanceIconView() override;