Ivailo Monev
ea9ae0d25a
libs: remove unused header inclusions
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-07 21:22:55 +03:00
Ivailo Monev
a0a2761746
libs: favicons KDED module documentation review
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-07 21:21:24 +03:00
Ivailo Monev
15838c25d7
libs: replace FavIconsModule::startDownload() with its body
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-07 21:15:39 +03:00
Ivailo Monev
d811882f97
libs: favicons KDED module rewrite
...
saves a lot of I/O and network trafic if there are multiple bookmarks to
the same host and multiple queries are made (which essentially will end up
being many jobs for the same icon). also implemented fallback to
alternative http://foo.bar/favicon.png and added tests for more cases,
bonus points?
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-07 20:57:44 +03:00
Ivailo Monev
409d5e4225
libs: remove compatibility bits
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2023-06-02 14:53:47 +03:00
Ivailo Monev
9ecb90f238
libs: adjust to KIO::Job changes
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-12-23 01:40:17 +02:00
Ivailo Monev
8b2eba7361
generic: prepare for Katie changes
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-11-13 01:41:49 +02:00
Ivailo Monev
1212fc79f7
generic: adjust to authorization changes
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-09-25 19:12:57 +03:00
Ivailo Monev
7bb060f0cf
libs: deal with TODOs related to KonqOperations
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-09-23 14:58:11 +03:00
Ivailo Monev
45fbbf2582
generic: compiler warnings fixes
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-06-05 22:06:42 +03:00
Ivailo Monev
0fac8d3580
libs: remove NFS template
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-06-03 11:37:44 +03:00
Ivailo Monev
38cdd9c0f4
generic: cast pointers with QObject as base class via qobject_cast<T>()
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-06-01 21:54:59 +03:00
Ivailo Monev
f4ec94e4e0
libs: do not down-cast the job to KIO::Job and up-cast it via qobject_cast<T>()
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-29 07:53:49 +03:00
Ivailo Monev
de8c7f6af1
libs: do not down-scale favicons images to 16x16
...
https://www.ibm.com/favicon.ico is 32x32 for example
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-27 08:26:43 +03:00
Ivailo Monev
509439c8b3
libs: skip favicon tests if images are not readable
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-27 07:52:04 +03:00
Ivailo Monev
b1c155416e
libs: call setIconForUrl() in the non-QEventLoop version of the favicons module test
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-26 04:06:29 +03:00
Ivailo Monev
c7429922cf
libs: comment correction
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 12:44:33 +03:00
Ivailo Monev
b4629ae603
libs: manually delete favicons module jobs
...
fixes possible crash in case the job is automatically deleted and
KJob::errorString() is called to get the error
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 10:22:48 +03:00
Ivailo Monev
038d9286b0
libs: workaround for QSignalSpy in favicon test
...
TODO: reimplement QSignalSpy to use connections or just write a new class
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 09:39:49 +03:00
Ivailo Monev
3bcc2da57b
libs: convert FavIconsModule::isIconOld() to static function
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 09:32:44 +03:00
Ivailo Monev
ec06deac86
libs: declare download info struct as primitive and rename it
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 09:19:23 +03:00
Ivailo Monev
c2f38b1a3b
libs: connect to all favicons module signals in its test
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 08:39:43 +03:00
Ivailo Monev
aa43bf3820
libs: test favicons module with IP addresses
...
requires:
38079e572c
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-24 08:06:19 +03:00
Ivailo Monev
02d8a78de8
libs: remove redundant favicons memory cache
...
the cache is supposed to speed-up non-host icon lookups but it does so only
for the life-time of the module, it becomes cache miss as soon as the
module is unloaded since the cache is not restored by the module from the
FavIconsModule constructor
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 09:04:20 +03:00
Ivailo Monev
11c62954d8
libs: favicons KDED module test re-write
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 06:54:52 +03:00
Ivailo Monev
6009accdc6
libs: do not store the favicons cache directory
...
KStandardDirs::saveLocation() will re-create it if it is deleted
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 05:25:14 +03:00
Ivailo Monev
e806e8a4c7
libs: warn when the image reader claims the data is readable but returns null image from favicon KDED module
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 04:35:12 +03:00
Ivailo Monev
055a8ca5f1
libs: warn when the image reader cannot read the favicon data
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 04:32:23 +03:00
Ivailo Monev
a55034d4f2
libs: fix possible favicons KDED module race-conditions
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 04:04:45 +03:00
Ivailo Monev
f437e42045
libs: kill the job immediately if the favicon is too big and remove it from the downloads queue
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 03:58:22 +03:00
Ivailo Monev
9790faa5d7
libs: double the wait time for favicon signals
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 01:35:46 +03:00
Ivailo Monev
3ed404a6be
libs: convert FavIconsModulePrivate::makeIconName() to static function
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 01:16:42 +03:00
Ivailo Monev
83a8cd1cf1
libs: de-virtualize FavIconsModulePrivate destructor
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 01:14:45 +03:00
Ivailo Monev
5f6c75eee2
libs: attach the signal spy after D-Bus initialization stuff happens in favicon test
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 00:09:32 +03:00
Ivailo Monev
8acb29cd9f
libs: remove unused system ctime header inclusion
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-23 00:08:32 +03:00
Ivailo Monev
6153a04df0
libs: if the favicon modification time is not valid consider it as old
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-22 23:43:16 +03:00
Ivailo Monev
c5c1ecc7fc
libs: skip only the tests relying on permissions when they are unexpected
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-22 22:38:08 +03:00
Ivailo Monev
731cdb39d5
libs: skip some tests when the permissions do not match the expectations
...
this is the case when the tests are run as root
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-22 20:46:47 +03:00
Ivailo Monev
61965cf96f
libs: quit event loops if signals are not emitted after 10sec in favicons test case
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-22 08:28:43 +03:00
Ivailo Monev
6a5265b9da
libs: bootstrap testing build fix
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-20 21:39:26 +03:00
Ivailo Monev
3892372c80
libs: always enable share action for local directories
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-16 23:51:50 +03:00
Ivailo Monev
f68295ea28
generic: move sub-projects from kde-baseapps [ci reset]
...
Signed-off-by: Ivailo Monev <xakepa10@gmail.com>
2022-05-14 21:56:54 +03:00