From 5c7c2a3ff333e79d8010a3fa1218274e6de5ef0b Mon Sep 17 00:00:00 2001
From: Jan Holesovsky <kendy@collabora.com>
Date: Thu, 23 Jul 2020 14:06:57 +0200
Subject: [PATCH] Revert "android: avoid expensive load of un-used sidebar
 icons on mobile."

Unfortunately we need this on Android for the tablets, otherwise the
sidebar lacks the icons.

Also it is not easily possible to use the isLOKMobilePhone() check here,
because that is per-view in general, and handled in sfx in particular,
making it hard to use here in vcl.

This reverts commit 9a38b194eec5bcf6cb8d073e5d35173c5856f28f.

Change-Id: I2b599e884ad4d00b7c246743c180a5324c9a143d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99329
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Andras Timar <andras.timar@collabora.com>
---
 vcl/source/window/toolbox2.cxx | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/vcl/source/window/toolbox2.cxx b/vcl/source/window/toolbox2.cxx
index b3313518ab69..95813d97e4e8 100644
--- a/vcl/source/window/toolbox2.cxx
+++ b/vcl/source/window/toolbox2.cxx
@@ -20,7 +20,6 @@
 #include <sal/config.h>
 #include <sal/log.hxx>
 
-#include <comphelper/lok.hxx>
 #include <comphelper/processfactory.hxx>
 #include <boost/property_tree/ptree.hpp>
 
@@ -426,12 +425,7 @@ void ToolBox::InsertItem(const OUString& rCommand, const css::uno::Reference<css
     auto aProperties = vcl::CommandInfoProvider::GetCommandProperties(rCommand, aModuleName);
     OUString aLabel(vcl::CommandInfoProvider::GetLabelForCommand(aProperties));
     OUString aTooltip(vcl::CommandInfoProvider::GetTooltipForCommand(rCommand, aProperties, rFrame));
-
-#ifdef ANDROID
-    Image aImage; // Loading redundant icons for sidebars shows in profiles.
-#else
     Image aImage(CommandInfoProvider::GetImageForCommand(rCommand, rFrame, GetImageSize()));
-#endif
 
     sal_uInt16 nItemId = GetItemCount() + 1;
         //TODO: ImplToolItems::size_type -> sal_uInt16!
-- 
2.11.4.GIT