Performance testing suggests that:
+ dense text this is 2x faster.
+ 'hello world' text this is 1.7x faster.
Change-Id: I4ff940663c44d0b22c9187deb4ee397a9d9953b0
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Simply calculate our loop variables from the iteration we're on.
Change-Id: I0bb73302fb09963b2a1f5b3d93ef302316ef1d4f
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Remove the special case for the first pixel, and instead have a
previous pixel run initialized to zero.
AVX2 has no effective shift for the while si256 so use permutation
to shift the last pixel of the previous run into the right place,
mask it and combine.
Saves a second un-aligned load of the same data, and branch.
Change-Id: I77c9cdead13d37aaf4d9f31d98cbd5c4a9c5ce24
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
just enough to get the same results as before
https://github.com/CollaboraOnline/online/issues/7165
Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I109c9b8f1e7935782c72e0179aa0ed48712eadb6
Split it out as a C file, to avoid accidental C++ header inclusion,
and C is a cross-platform assembler anyway so a good match.
Change-Id: I6c042781713aecaf143b9663af8377659a7deaf1
Signed-off-by: Michael Meeks <michael.meeks@collabora.com>
Autofilter dropdown has submenus which are created at the same
time as main menu. These submenus are hidden but send jsdialog
message with "popup" type. We ignored that message on desktop
so do the same for mobile to not show them instead of main dropdown.
Added also warning about unhandled messages.
Removed old style "autofilter" type handling which is not used anymore.
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I18d0c9078b88d6d24290e44179594a8eebd1c2e4
Will not create `<u>` element for mobile wizard
Signed-off-by: Darshan-upadhyay1110 <darshan.upadhyay@collabora.com>
Change-Id: Iaca88ac4be2c95dc4781180019ba8fede9cdc81a
The test now runs as many times as requested
and doesn't stop on first failure. Instead,
it reports the number of passing vs failing
runs.
Also simplifies the detection of the test
result by using the exit code instead
of grepping the output file.
Change-Id: Ie458b2963411632d566cd87d2dfb9137044d2b4b
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
We now exit immediately with error code when
we are given --unitlib argument but fail to
load the given library.
Change-Id: I4eb5f3a37b9838ecf83bdceef4d09f888afcca83
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
New unit-test that reproduces the corner-case
with disconnection. With graceful unloading
a previous unit-test verified that it worked
as expected.
This is restored from e6798e9062
after it was reverted (due to a PDF comment saving regression).
Since the issue is fixed separately in
0b27e85fc5, these tests can
now be restored.
Change-Id: Id84cda4f4599c559018247c32ea1205e154e4984
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
This disables the unreliable reload test.
On a slow box, sometimes the load takes
a little longer than 5 seconds. Waiting
a bit longer doesn't hurt in these cases.
Also, remove duplicate test and give each
test case its own testname.
Change-Id: I556f6da9e49a7fa4cd9917bc9964dd9561b93817
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
This test is currently disabled, as it is only
useful for stress-testing
Change-Id: I803d8d5818f004afa72bbf2dfe5c9d90b41e1133
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
Fixes a race-condition due to having multiple
views, we will get multiple onDocumentModified.
Change-Id: I806a3cc31f768c9f159e599c69fd1d31755e6cd1
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
- Added view option to all other apps on mobile view.
- previously only text contains the view option in mobile view. ( click on hamburger menu)
Signed-off-by: Darshan-upadhyay1110 <darshan.upadhyay@collabora.com>
Change-Id: I04e06e346d398d947e637bd71e4134bc3e901c4d
Currently [in docker it is possible to do configuration through
environment variables](https://col.la/dockercodeconfigviaenv), which
works using the start-collabora-online.sh start-collabora-online.pl
scripts. This commit lets COOLWSD listen to the same environment
variables directly
Change-Id: I75762ad620132037523fa82167a3ff17075c7027
Signed-off-by: Skyler Grey <skyler.grey@collabora.com>
- This patch only adds an option to enable this feature. It is off by
default
- This allows you to, say, have a docker container and --restart=always
to restart when you update the config
- This patch only listens for "/etc/coolwsd/", so if you specify a config
file that isn't there then you're out of luck... An improvement for a
followup patch will be make it listen to wherever your config files
actually are
- The current docker scripts only listen for modifications, so this
matches that behavior
Signed-off-by: Skyler Grey <skyler.grey@collabora.com>
Change-Id: I674756b0188893f5d192885bb436256aa827e7b5
And also allow automatic re-trying to decide if this is a reproducible
failure or an unstable test.
Signed-off-by: Miklos Vajna <vmiklos@collabora.com>
Change-Id: I254d83cc1acc6b80ec3978e26e5fd64e676ff2a9
This fixes a regression where PDF comments were lost when
the document closed immediately after adding a comment.
Change-Id: Iac78ec13fdbaa7d1ffe25067ea0f41704abb3312
Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
This reverts commit e6798e9062.
Reason: this commit caused a regression. PDF comments were not saved.
Signed-off-by: Andras Timar <andras.timar@collabora.com>
Change-Id: I180ecb9e298e37bd5a989a8cd6d23d02c25a6929
- Add or remove additional css class to the Tabbed view > File Tab >
Save button whenever the document is modified or saved.
- NotebookbarBuilder: onCommandStateChanged: Re-use existing state var
There seems to be no need to call again the object and its state when
we already have a variable for that.
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I3d0bee8903f4e7309d5d5e7389a824bf39e72651
This is **not** nonmodal dialog but it looks like. This is a modal:
user can’t interact with background content.
Signed-off-by: Pedro Pinto Silva <pedro.silva@collabora.com>
Change-Id: I9e68afadca3d6ab04311f18d38a9dd43b9679522
The problem affects no-a11y case since the editable area is empty.
Since no copy/cut event is emitted, Clipboard.copy/cut is never
invoked. So we need to emit it manually.
To be honest it seems a Firefox bug. We need to check if they fix it
in later version.
Signed-off-by: Marco Cecchetti <marco.cecchetti@collabora.com>
Change-Id: I036414b5ffb5b35ff1ef1d7de1044e890832c673
