Commit graph

4202 commits

Author SHA1 Message Date
Ashod Nakashian
800e711321 loolwsd: proper ChildProcess cleanup
Change-Id: If9be827aa50471b7d1d922402414d028ccdcff8b
Reviewed-on: https://gerrit.libreoffice.org/30629
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:32:10 +00:00
Ashod Nakashian
3993757ee8 loolwsd: fix race while creating new documents
This fixes the race rather than trying to patch it.
It still minimizes the locking necessary to a minimum
to maximize parallelism.

The approach is to have at least the DocBrokers lock
or the DocumentBroker lock (if we already have a doc)
while creating new document views.

Change-Id: I96b4f17b3be3d03cd5e6f4d17d39e2165fe008a7
Reviewed-on: https://gerrit.libreoffice.org/30628
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:31:26 +00:00
Ashod Nakashian
8b9372d517 loolwsd: improved ChildProcess message handling logs
Change-Id: I28a1de58f4c886d7c6f8c9f723a84e2a8726edc2
Reviewed-on: https://gerrit.libreoffice.org/30625
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:29:40 +00:00
Ashod Nakashian
52b8a11554 loolwsd: lock -> docBrokersLock
Change-Id: Ia1e1e65003967edf092508211be5f3953239e0f6
Reviewed-on: https://gerrit.libreoffice.org/30627
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:29:24 +00:00
Ashod Nakashian
0dec678b16 loolwsd: PrisonerSession logs updated
Change-Id: I3cae4d77be1c556abef64e0ced05747ef41ddb23
Reviewed-on: https://gerrit.libreoffice.org/30626
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:29:11 +00:00
Ashod Nakashian
f63fbef560 loolwsd: ChildSession logs updated
Change-Id: I5675c5b79fc96ab34605b68b0fe05c5fe90e92db
Reviewed-on: https://gerrit.libreoffice.org/30624
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:28:44 +00:00
Ashod Nakashian
a537acf7d6 loolwsd: add genRandomString and use it to test SaveOnDisconnect
Change-Id: I3e60579032882b921ee7c833a78e8d87e79e1360
Reviewed-on: https://gerrit.libreoffice.org/30623
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:28:32 +00:00
Ashod Nakashian
f4d72d66e3 loolwsd: move genRandomData to helpers
Change-Id: I1edafb41fefc07e64fa8625d2b88617079a327fa
Reviewed-on: https://gerrit.libreoffice.org/30622
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:26:22 +00:00
Ashod Nakashian
0305cf4819 loolwsd: more efficient child message forwarding
Change-Id: I60ab308b2f7d108ee9e4f2892fe3dda68f7f3201
Reviewed-on: https://gerrit.libreoffice.org/30621
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:26:05 +00:00
Ashod Nakashian
fd8b426860 loolwsd: use correct log level and not typo
Change-Id: I71a04714c1ad406d454c3312ffa72930293a94c1
Reviewed-on: https://gerrit.libreoffice.org/30620
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-07 06:25:44 +00:00
Pranav Kant
216d1b6a52 Undefined variable
Change-Id: I0c5a07fa383523053ca7de319fcc9d1acdca63cd
2016-11-07 10:26:32 +05:30
Miklos Vajna
4034006338 getNewChild: avoid multiplication at int precision, then compare with long
Change-Id: I23c26ef9014150fda7e400e19526244ddaa411cf
2016-11-04 14:09:09 +01:00
Ashod Nakashian
871a67bd98 loolwsd: avoid side bookkeeping
Change-Id: I7ad2cbf45c07173447c4e80dcf06f0155de904e9
Reviewed-on: https://gerrit.libreoffice.org/30554
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:22:07 +00:00
Ashod Nakashian
d17d148880 loolwsd: logging improvements
Change-Id: If291bf20ae59085f2120d4c92a26de6ccddbc98a
Reviewed-on: https://gerrit.libreoffice.org/30553
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:17:57 +00:00
Ashod Nakashian
b6fae38f17 loolwsd: simplified createSession
Change-Id: I2ebb37cae61d9625731b48ea6b9ce31646ad3e42
Reviewed-on: https://gerrit.libreoffice.org/30552
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:13:49 +00:00
Ashod Nakashian
61488cce78 loolwsd: move child communication logging
...and simplify the message (new line not needed).

Change-Id: Ic304d39e444b13c140e16dbe232c2cb03659ff99
Reviewed-on: https://gerrit.libreoffice.org/30551
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:11:43 +00:00
Ashod Nakashian
84fb2d43fb loolwsd: warn if sessions remain by ~DocumentBroker
Change-Id: I03be11b02af7560f934b138e86f8857fd9870a60
Reviewed-on: https://gerrit.libreoffice.org/30550
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:04:47 +00:00
Ashod Nakashian
159f055a89 loolwsd: Storage logs updated
Change-Id: I1e78371d03c6d5734e60fb843e9f0e9d15db6fa7
Reviewed-on: https://gerrit.libreoffice.org/30549
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:04:20 +00:00
Ashod Nakashian
b9b69e9534 loolwsd: rename LOG_XXX internal variable to minimize conflicts
Change-Id: I82e98b38a0ac6a5e821148f213aa1b15827e23df
Reviewed-on: https://gerrit.libreoffice.org/30548
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:02:05 +00:00
Ashod Nakashian
4a74d49780 loolwsd: logs updated
Change-Id: I87ad4979c789ec1933ecdf39bd81dabce95ef9bb
Reviewed-on: https://gerrit.libreoffice.org/30547
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-04 05:01:32 +00:00
Andras Timar
f96ef2c4f3 loleaflet: page orientation in Writer's Format menu 2016-11-03 19:40:27 +01:00
Michael Meeks
59d28fdeba Help->About can point to the log more helpfully than the commit. 2016-11-02 14:38:58 +00:00
Ashod Nakashian
f02b98b432 loolwsd: use new log macros in LOOLSession
Change-Id: Ia971d4ef3b67fe289b58c6d2b1f27d6e6454a56f
Reviewed-on: https://gerrit.libreoffice.org/30495
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:23:17 +00:00
Ashod Nakashian
0402256390 Log file and line number using the LOG_XXX macro
Change-Id: Ie157daab8b1f52aba22e87ef07b14e571cc59771
Reviewed-on: https://gerrit.libreoffice.org/30494
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:22:25 +00:00
Ashod Nakashian
74ad4fb529 loolwsd: log large message transmission
Change-Id: I68d7cb7cf69c45c8050d8202f51f6160bb050587
Reviewed-on: https://gerrit.libreoffice.org/30493
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:22:09 +00:00
Ashod Nakashian
05d4234208 loolwsd: more efficient client message forwarding
Change-Id: I10f880059380599cf0187064e1009bef5fa693b1
Reviewed-on: https://gerrit.libreoffice.org/30492
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:21:44 +00:00
Ashod Nakashian
254005af74 loolwsd: log only abbreviated messages
Change-Id: Ic5780b6d3c00445888568ee43c6f5fb453e5b0df
Reviewed-on: https://gerrit.libreoffice.org/30491
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:21:18 +00:00
Ashod Nakashian
5656c12713 loolwsd: cap abbraviated message length
Change-Id: Idddfd55ae9a9fe8c2ae0c78e416d541529adaa66
Reviewed-on: https://gerrit.libreoffice.org/30490
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-02 13:20:49 +00:00
Andras Timar
469dc3e8ea %debug_package is only needed on SUSE, on CentOS it's defined already 2016-11-02 11:01:00 +01:00
Michael Meeks
a28b3d0d7e Disable port re-use for internal unit-tests, so we find a free socket. 2016-11-02 07:59:59 +00:00
Michael Meeks
e48f9d7557 loolwsd: remove assertion that sessions are cleaned up nicely. 2016-11-02 01:23:00 +00:00
Michael Meeks
5e9b582ed4 loolwsd: avoid ordering issues with atexit handlers.
Otherwise Admin::instance() has been destroyed by the time we're
cleaning up the DocumentBrokers resulting in FMRs etc.
2016-11-02 00:46:17 +00:00
Michael Meeks
da76cffc0d loolwsd: Better fix for 'Address already in use' vs. 86383d40db 2016-11-02 00:42:56 +00:00
Michael Meeks
2b9ae4b263 DocumentBroker - hold mutex while iterating over _sessions. 2016-11-01 23:38:25 +00:00
Michael Meeks
4b0ebe4948 Slide thumbnailing / canceltiles fix.
Avoid canceltiles on in-flight message queue in loolkit potentially
overlooked in 97adaf935b
2016-11-01 22:50:48 +00:00
László Németh
f7f55dede5 loolwsd: set debuginfo packaging 2016-11-01 22:47:45 +01:00
Andras Timar
ac86cdf764 typo fix 2016-11-01 22:35:18 +01:00
Henry Castro
09cb79affd fix typo 2016-11-01 09:37:50 -04:00
Ashod Nakashian
e08503e275 loolwsd: save and replay doc state changes
Document state changes come in with the same
event ID. We need to store them in a separate
map (also renamed the old 'states' map to 'events').

All important events are now replayed correctly and traced.

Change-Id: I52346b0d252730ba5cedb091b14898479c2a52c5
Reviewed-on: https://gerrit.libreoffice.org/30466
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-01 13:26:49 +00:00
Ashod Nakashian
56b1f22498 loolwsd: replay state change events when re-activating a client
Change-Id: Id2181c8426712320a9fffacd7d8d350f1476580a
Reviewed-on: https://gerrit.libreoffice.org/30465
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-11-01 13:00:30 +00:00
Andras Timar
e7355553cc loleaflet: update translations from Pootle 2016-11-01 12:39:54 +01:00
Pranav Kant
86383d40db loolwsd: Tentative fix for 'Address already in use' 2016-11-01 10:25:29 +01:00
Henry Castro
2c5cbe6555 loleaflet: row/column highlighting to reflect the cell cursor 2016-10-31 21:37:27 -04:00
Andras Timar
760c1e732a loleaflet: hide zoom controls for spreadsheets 2016-10-31 18:39:32 +01:00
Marco Cecchetti
8c522c44d1 loleaflet: print dialog: it seems that this magic hack is no more needed
Change-Id: I5dc3af5d8c70b381638f82de14ea47b1627237b9
2016-10-31 17:45:22 +01:00
Andras Timar
cbff583462 loleaflet: add link to git hash 2016-10-31 17:24:31 +01:00
Tor Lillqvist
be22779c21 Assertion failures are for developers to read
'_sessions.empty()' does not need a verbal explanation. We don't do
those in other assertions in the same file anyway.

(Also, I dislike the use of exclamation marks, especially in contexts
that by themselves already are alerts, like in assertion failure
messages. Exposure to LibreOffice source code with its style (in some
neighbourhoods) of multiple exclamation marks, even, in comments,
makes you like that. Exclamation marks makes your comments or messages
look like check-out tabloid headlines.)
2016-10-31 18:06:40 +02:00
Andras Timar
d589618bc1 disable zoom for Calc 2016-10-31 14:45:04 +01:00
Andras Timar
513d963b57 loleaflet: add Format - Page size to Writer menu 2016-10-31 13:27:02 +01:00
Ashod Nakashian
4ba5bedf6a loolwsd: explicitly scope DocumentBroker mutex
Change-Id: I64629774ec745f47e7376231a3ca569bcbfffaac
Reviewed-on: https://gerrit.libreoffice.org/30424
Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
Tested-by: Ashod Nakashian <ashnakash@gmail.com>
2016-10-31 06:10:57 +00:00