[Libreoffice-commits] core.git: Branch 'libreoffice-5-3' - wizards/source

2017-03-26 Thread Andreas Säger
 wizards/source/depot/Currency.xba|4 ++--
 wizards/source/depot/Internet.xba|4 ++--
 wizards/source/formwizard/DBMeta.xba |6 +++---
 wizards/source/formwizard/FormWizard.xba |6 +++---
 wizards/source/formwizard/Language.xba   |   14 +++---
 5 files changed, 17 insertions(+), 17 deletions(-)

New commits:
commit 7fb7b091e963d8c45abf1db1202a88f2e188e9b0
Author: Andreas Säger 
Date:   Sun Mar 26 23:34:35 2017 +0300

tdf#106529: fix closing parentheses of bundled macros

Change-Id: I43b6d4f97cb3b1cd1426e6aa7e13550d94cb7c51
Signed-off-by: Mike Kaganski 
Reviewed-on: https://gerrit.libreoffice.org/35726
Tested-by: Jenkins 
(cherry picked from commit ad4857211bcc39ccd4c84c670bfab1b937f65089)
Reviewed-on: https://gerrit.libreoffice.org/35741

diff --git a/wizards/source/depot/Currency.xba 
b/wizards/source/depot/Currency.xba
index 5a428d757246..d728424d3214 100644
--- a/wizards/source/depot/Currency.xba
+++ b/wizards/source/depot/Currency.xba
@@ -89,7 +89,7 @@ Dim bIsDocCountry as Boolean
End If
sCurStockIDLabel = sMarket(Index,5)
sCurExtension = sMarket(Index,8)
-   iValueCol = Val(sMarket(Index,10)
+   iValueCol = Val(sMarket(Index,10))
If Instr(sCurExtension,";") <> 0 Then
' Take the german extension as the stock place is Frankfurt
sCurExtension = "407"
@@ -192,4 +192,4 @@ Function CheckFormatType(oStyle as Object)
 Dim oFormatofObject as Object
oFormatofObject = oDocFormats.getByKey(oStyle.NumberFormat)
CheckFormatType = INT(oFormatOfObject.Type) AND 
com.sun.star.util.NumberFormat.CURRENCY
-End Function
\ No newline at end of file
+End Function
diff --git a/wizards/source/depot/Internet.xba 
b/wizards/source/depot/Internet.xba
index b4bcf579b808..333bcf98af89 100644
--- a/wizards/source/depot/Internet.xba
+++ b/wizards/source/depot/Internet.xba
@@ -223,7 +223,7 @@ Dim iCellValue as Long
If iCellValue > 0 Then
oCell.SetValue(oLinkSheet.GetCellByPosition(0,i).Value)
Else
-   
oCell.SetValue(StringToDate(oLinkSheet.GetCellByPosition(0,i).String)
+   
oCell.SetValue(StringToDate(oLinkSheet.GetCellByPosition(0,i).String))
End If
oCell = oSheet.GetCellbyPosition(SBVALUECOLUMN,iCurRow)
oCell.SetValue(oLinkSheet.GetCellByPosition(4,i).Value)
@@ -353,4 +353,4 @@ Dim bLeaveLoop as Boolean
oDateCell.Annotation.SetString(NoteText)
End If
 End Sub
-
\ No newline at end of file
+
diff --git a/wizards/source/formwizard/DBMeta.xba 
b/wizards/source/formwizard/DBMeta.xba
index 68759966daf9..b0fa20b7a901 100644
--- a/wizards/source/formwizard/DBMeta.xba
+++ b/wizards/source/formwizard/DBMeta.xba
@@ -305,8 +305,8 @@ Dim MaxQueryIndex as Integer
 Dim MaxIndex as Integer
 Dim i as Integer
 Dim a as Integer
-   MaxTableIndex = Ubound(TableNames()
-   MaxQueryIndex = Ubound(QueryNames()
+   MaxTableIndex = Ubound(TableNames())
+   MaxQueryIndex = Ubound(QueryNames())
MaxIndex = MaxTableIndex + MaxQueryIndex + 1
If MaxIndex > -1 Then
Dim LocCommandTypes(MaxIndex) as Integer
@@ -344,4 +344,4 @@ Function AssignFieldLength(FieldLength as Long) as Integer
AssignFieldLength() = FieldLength
End If
 End Function
-
\ No newline at end of file
+
diff --git a/wizards/source/formwizard/FormWizard.xba 
b/wizards/source/formwizard/FormWizard.xba
index ffc2c4267e52..873ce4b22dea 100644
--- a/wizards/source/formwizard/FormWizard.xba
+++ b/wizards/source/formwizard/FormWizard.xba
@@ -153,11 +153,11 @@ Dim LocList() as String

DlgFormDB.GetControl("lstTables").SelectItem(sContent, True)
Else
If CommandType = 
com.sun.star.sdb.CommandType.QUERY Then
-   SelIndex = 
IndexInArray(sContent, QueryNames()
+   SelIndex = 
IndexInArray(sContent, QueryNames())

DlgFormDB.GetControl("lstTables").SelectItemPos(SelIndex, True)
ElseIf CommandType = 
com.sun.star.sdb.CommandType.TABLE Then
-   SelIndex = 
IndexInArray(sContent, TableNames()
-   
DlgFormDB.GetControl("lstTables").SelectItemPos(Ubound(QueryNames()+1 
+ SelIndex, True)
+   SelIndex = 
IndexInArray(sContent, TableNames())
+   
DlgFormDB.GetControl("lstTables").SelectItemPos(Ubound(QueryNames()+1 
+ SelIndex, True))
   

[Libreoffice-commits] core.git: include/svx sc/inc sc/sdi svx/sdi

2017-03-26 Thread Noel Grandin
 include/svx/svxids.hrc |1 
 sc/inc/sc.hrc  |   11 -
 sc/sdi/cellsh.sdi  |1 
 sc/sdi/formatsh.sdi|4 ---
 sc/sdi/scalc.sdi   |   54 -
 sc/sdi/tabvwsh.sdi |   26 ---
 svx/sdi/svx.sdi|   19 -
 7 files changed, 116 deletions(-)

New commits:
commit c6adcf85eb1c68ac05794f115d62e25667ec3f6a
Author: Noel Grandin 
Date:   Fri Mar 24 14:59:02 2017 +0200

remove unhandled SID commands in sc

checked to make sure none of the related .uno commands are present
in any .xcu files.

Change-Id: I026eec90351f525be7417f697debe2444756193b
Reviewed-on: https://gerrit.libreoffice.org/35622
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/include/svx/svxids.hrc b/include/svx/svxids.hrc
index d95368c5ed3f..2ba5f668067d 100644
--- a/include/svx/svxids.hrc
+++ b/include/svx/svxids.hrc
@@ -448,7 +448,6 @@
 
 //#define SID_SIDEBAR   ( SID_SVX_START + 336 
)   -> sfxsids.hrc
 
-#define SID_BORDER_OBJECT   ( SID_SVX_START + 340 )
 #define SID_SB_CONNECTIONPOOLING( SID_SVX_START + 348 )
 #define SID_SB_DBREGISTEROPTIONS( SID_SVX_START + 349 )
 #define SID_BMPMASK ( SID_SVX_START + 350 )
diff --git a/sc/inc/sc.hrc b/sc/inc/sc.hrc
index 768ed088c2fa..193f99de6763 100644
--- a/sc/inc/sc.hrc
+++ b/sc/inc/sc.hrc
@@ -131,7 +131,6 @@
 #define SID_LINKS   (SC_VIEW_START + 60)
 #define SID_INSERT_SIMAGE   (SC_VIEW_START + 61)
 #define SID_INSERT_SMATH(SC_VIEW_START + 63)
-#define SID_STANDARD_FONTS  (SC_VIEW_START + 64)
 #define SID_MIRROR_VERTICAL (SC_VIEW_START + 65)
 #define SID_MIRROR_HORIZONTAL   (SC_VIEW_START + 66)
 #define SID_CELL_FORMAT_RESET   (SC_VIEW_START + 67)
@@ -278,7 +277,6 @@
 #define FID_FILL_TAB(EDIT_MENU_START + 8)
 #define FID_FILL_SERIES (EDIT_MENU_START + 9)
 #define FID_FILL_SINGLE_EDIT(EDIT_MENU_START + 10)
-#define SID_SC_EDIT_OBJECT  (EDIT_MENU_START + 14)
 #define SID_HFEDIT  (EDIT_MENU_START + 15)
 
 #define SID_DEL_ROWS(EDIT_MENU_START + 16)//JN
@@ -358,7 +356,6 @@
 #define SID_SCOPTIONS   (EXTRA_MENU_START + 9)
 #define SID_SHARE_DOC   (EXTRA_MENU_START + 11)
 #define SID_SCENARIOS   (EXTRA_MENU_START + 12)
-#define SID_ADD_IN_MANAGER  (EXTRA_MENU_START + 13)
 #define SID_PIVOT_RECALC(EXTRA_MENU_START + 14)
 #define SID_PIVOT_KILL  (EXTRA_MENU_START + 15)
 
@@ -545,8 +542,6 @@
 #define SID_EXPORT_AS_GRAPHIC   (SID_NEW_SLOTS+2)
 
 #define FID_MERGE_TOGGLE(SID_NEW_SLOTS+10)
-#define SID_ATTR_ROWHEADERS (SID_NEW_SLOTS+11)
-#define SID_ATTR_COLHEADERS (SID_NEW_SLOTS+12)
 
 #define SID_SC_CELLS(SID_NEW_SLOTS+13)
 #define SID_SC_RANGE(SID_NEW_SLOTS+14)
@@ -563,8 +558,6 @@
 #define SID_RANGE_TEXTVALUE (SID_NEW_SLOTS+26)
 #define SID_TABLE_ACTIVATE  (SID_NEW_SLOTS+27)
 #define SID_TABLES_GET  (SID_NEW_SLOTS+28)
-#define SID_PROP_INTERIOR   (SID_NEW_SLOTS+29)
-#define SID_PROP_FONT   (SID_NEW_SLOTS+30)
 
 #define SID_DEFINE_PRINTAREA(SID_NEW_SLOTS+31)
 #define SID_DELETE_PRINTAREA(SID_NEW_SLOTS+32)
@@ -645,10 +638,6 @@
 #define SID_OBJECT_HEIGHT   (SID_NEW_SLOTS+88)
 
 #define SID_PIVOT_CREATE(SID_NEW_SLOTS+89)
-#define SID_PIVOT_FIELDS(SID_NEW_SLOTS+90)
-#define SID_PFIELD_NAME (SID_NEW_SLOTS+91)
-#define SID_PFIELD_ORIENTATION  (SID_NEW_SLOTS+92)
-#define SID_PFIELD_FUNCTION (SID_NEW_SLOTS+93)
 #define SID_PIVOT_GET   (SID_NEW_SLOTS+94)
 
 #define SID_DELETE_SCENARIO (SID_NEW_SLOTS+95)
diff --git a/sc/sdi/cellsh.sdi b/sc/sdi/cellsh.sdi
index 9f6f53da7a8a..c4c484cdc559 100644
--- a/sc/sdi/cellsh.sdi
+++ b/sc/sdi/cellsh.sdi
@@ -63,7 +63,6 @@ interface CellSelection
 // } Datenbank-Operationen
 
 
-SID_BORDER_OBJECT   [ StateMethod = GetState; ]
 SID_RANGE_ADDRESS   [ StateMethod = GetState; ]
 
 SID_HYPERLINK_SETLINK   [ ExecMethod = ExecuteEdit; ]
diff --git a/sc/sdi/formatsh.sdi b/sc/sdi/formatsh.sdi
index 0acabdc56999..1aecdb48484c 100644
--- a/sc/sdi/formatsh.sdi
+++ b/sc/sdi/formatsh.sdi
@@ -38,10 +38,6 @@ interface TableFont
 
 interface FormatForSelection
 {
-//Auch das Basic muss wieder laufen
-SbxObject Font SID_PROP_FONT ()
-SbxObject Interior SID_PROP_INTERIOR () // status(Final)
-
 // Slot's die in der DrawShell disabled werden. {
 SID_STYLE_FAMILY2   [ ExecMethod = ExecuteStyle; StateMethod = 
GetStyleState; ]
 SID_STYLE_APPLY [ ExecMethod = ExecuteStyle; StateMethod = 
GetStyleState; ]
diff --git a/sc/sdi/scalc.sdi b/sc/sdi/scalc.sdi
index 10300522474d..732ee8d89c6a 100644
--- a/sc/sdi/scalc.sdi
+++ b/sc/sdi/scalc.sdi
@@ -139,24 +139,6 @@ SfxVoidItem AddChartData SID_CHART_ADD

[Libreoffice-commits] core.git: include/sfx2 sd/inc sd/sdi sfx2/sdi

2017-03-26 Thread Noel Grandin
 include/sfx2/sfxsids.hrc |1 
 sd/inc/app.hrc   |9 +--
 sd/sdi/_drvwsh.sdi   |   20 
 sd/sdi/sdraw.sdi |  109 ---
 sd/sdi/tables.sdi|5 --
 sfx2/sdi/sfx.sdi |   18 ---
 6 files changed, 3 insertions(+), 159 deletions(-)

New commits:
commit ee045805e640058d798029ce20b7d9f75d138b38
Author: Noel Grandin 
Date:   Fri Mar 24 13:06:46 2017 +0200

remove unhandled SID commands in sd

checked to make sure none of the related .uno commands are present
in any .xcu files.

Change-Id: Iec555836a0a926e070b7632dc1205decf49d331f
Reviewed-on: https://gerrit.libreoffice.org/35615
Tested-by: Jenkins 
Reviewed-by: Noel Grandin 

diff --git a/include/sfx2/sfxsids.hrc b/include/sfx2/sfxsids.hrc
index f497bea5a00b..81b113b58160 100644
--- a/include/sfx2/sfxsids.hrc
+++ b/include/sfx2/sfxsids.hrc
@@ -541,7 +541,6 @@
 #define SID_SHOW_BROWSER(SID_SVX_START + 163)
 #define SID_GALLERY_BG_BRUSH(SID_SVX_START + 279)
 #define SID_SEARCH_OPTIONS  (SID_SVX_START + 281)
-#define SID_ZOOM(SID_SVX_START + 289)
 #define SID_SEARCH_ITEM (SID_SVX_START + 291)
 #define SID_SIDEBAR (SID_SVX_START + 336)
 #define SID_NOTEBOOKBAR (SID_SVX_START + 338)
diff --git a/sd/inc/app.hrc b/sd/inc/app.hrc
index 2bf78397a1b3..0897e6c38fbe 100644
--- a/sd/inc/app.hrc
+++ b/sd/inc/app.hrc
@@ -84,7 +84,7 @@
 #define SID_COMBINE (SID_SD_START+26)
 #define SID_NAME_GROUP  (SID_SD_START+27)
 #define SID_DRAWTBX_CONNECTORS  (SID_SD_START+28)
-#define SID_TEXTALIGNMENT   (SID_SD_START+29)
+// FREE
 // In svxids.hrc: SID_BEZIER_ELIMINATE_POINTS   (SID_SD_START+30)
 #define SID_MOREFRONT   (SID_SD_START+31)
 #define SID_MOREBACK(SID_SD_START+32)
@@ -104,7 +104,7 @@
 #define SID_MODIFYLAYER (SID_SD_START+45)
 #define SID_PAGEMODE(SID_SD_START+46)
 #define SID_LAYERMODE   (SID_SD_START+47)
-#define SID_STARTAPP(SID_SD_START+49)
+// FREE
 #define SID_MASTERPAGE  (SID_SD_START+50)
 // Navigation between slides
 #define SID_GO_TO_NEXT_PAGE (SID_SD_START+51)
@@ -128,7 +128,6 @@
 #define SID_HANDOUT_MASTER_MODE (SID_SD_START+70)
 #define SID_LINEEND_POLYGON (SID_SD_START+71)
 // FREE
-#define SID_GRAPHIC_EXPORT  (SID_SD_START+73)
 #define SID_SWITCH_POINTEDIT(SID_SD_START+74)
 // FREE
 // In svxids.hrc: SID_TEXTEDIT  (SID_SD_START+76)
@@ -278,7 +277,7 @@
 // In svxids.hrc: SID_BEZIERTO  (SID_SD_START+252)
 #define SID_PAGESIZE(SID_SD_START+253)
 #define SID_PAGEMARGIN  (SID_SD_START+254)
-#define SID_SELECTAT(SID_SD_START+256)
+// FREE
 #define SID_COLORVIEW   (SID_SD_START+257)
 // FREE
 #define SID_GETRED  (SID_SD_START+259)
@@ -421,7 +420,6 @@
 #define SID_LEFT_PANE_DRAW  (SID_SD_START+415)
 // FREE
 #define SID_NOTES_WINDOW(SID_SD_START+417)
-#define SID_SWITCH_SHELL_PANE   (SID_SD_START+418)
 // FREE
 #define SID_NORMAL_MULTI_PANE_GUI   (SID_SD_START+420)
 #define SID_SLIDE_SORTER_MULTI_PANE_GUI (SID_SD_START+421)
@@ -450,7 +448,6 @@
 // Add companion for the SID_HIDE_SLIDE (that is defined in svx)
 #define SID_SHOW_SLIDE  (SID_SD_START+441)
 
-#define SID_TABLE_TOOLBOX   (SID_SD_START+443)
 // FREE
 #define SID_ZOOM_MODE   (SID_SD_START+447)
 #define SID_HIDE_LAST_LEVEL (SID_SD_START+448)
diff --git a/sd/sdi/_drvwsh.sdi b/sd/sdi/_drvwsh.sdi
index b5f40fff9084..e2206f7ca94a 100644
--- a/sd/sdi/_drvwsh.sdi
+++ b/sd/sdi/_drvwsh.sdi
@@ -432,11 +432,6 @@ interface DrawView
 ExecMethod = FuTemporary ;
 StateMethod = GetMenuState ;
 ]
-SID_SELECTAT // ole : no, status : play rec
-[
-ExecMethod = FuTemporary ;
-StateMethod = GetMenuState ;
-]
 SID_PRESENTATION_LAYOUT // ole : no, status : ?
 [
 ExecMethod = FuTemporary ;
@@ -669,11 +664,6 @@ interface DrawView
 ExecMethod = FuSupport ;
 StateMethod = GetMenuState ;
 ]
-SID_ZOOM // ole : no, status : play rec
-[
-ExecMethod = FuSupport ;
-StateMethod = GetMenuState ;
-]
 SID_DELETE // ole : no, status : ?
 [
 ExecMethod = FuSupport ;
@@ -1633,11 +1623,6 @@ interface DrawView
 ExecMethod = FuTemporary ;
 StateMethod = GetMenuState ;
 ]
-SID_GRAPHIC_EXPORT // ole : no, status : ?
-[
-

Re: Han Zhu license statement

2017-03-26 Thread jan i.
Thanks for your license statement.


I have added you to our wiki:

https://wiki.documentfoundation.org/Development/Developers


If you want help to get started or have any questions, then please contact
me. I am here to help you (and others) in getting their first patch
submitted.


LibreOffice is a very big program and getting it built, setting up gerrit,
and getting the first patch right can be a bit challenging, therefore do
not hesitate to email me if you want help.


We have made a step by step guide to help you get started:

https://wiki.documentfoundation.org/Development/GetInvolved


rgds

Jan Iversen.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: Andreas Säger license statement

2017-03-26 Thread jan i.
Thanks for your license statement.


I have added you to our wiki:

https://wiki.documentfoundation.org/Development/Developers


If you want help to get started or have any questions, then please contact
me. I am here to help you (and others) in getting their first patch
submitted.


LibreOffice is a very big program and getting it built, setting up gerrit,
and getting the first patch right can be a bit challenging, therefore do
not hesitate to email me if you want help.


We have made a step by step guide to help you get started:

https://wiki.documentfoundation.org/Development/GetInvolved


rgds

Jan Iversen.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: José Peixoto license statement

2017-03-26 Thread jan i.
Thanks for your license statement.


I have added you to our wiki:

https://wiki.documentfoundation.org/Development/Developers


If you want help to get started or have any questions, then please contact
me. I am here to help you (and others) in getting their first patch
submitted.


LibreOffice is a very big program and getting it built, setting up gerrit,
and getting the first patch right can be a bit challenging, therefore do
not hesitate to email me if you want help.


We have made a step by step guide to help you get started:

https://wiki.documentfoundation.org/Development/GetInvolved


rgds

Jan Iversen.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: ios/experimental

2017-03-26 Thread jan Iversen
 dev/null   
  |binary
 ios/experimental/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj   
  |8 
 ios/experimental/LibreOfficeLight/LibreOfficeLight/AppDelegate.swift   
  |   10 -
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/AppIcon.appiconset/Contents.json
  |   91 +++--
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/pdf.imageset/Contents.json
|   23 --
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/print.imageset/Contents.json
  |   23 --
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/properties.imageset/Contents.json
 |   23 --
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/save.imageset/Contents.json
   |   23 --
 
ios/experimental/LibreOfficeLight/LibreOfficeLight/Assets.xcassets/saveas.imageset/Contents.json
 |   23 --
 ios/experimental/LibreOfficeLight/LibreOfficeLight/DocumentController.swift
  |   48 
 ios/experimental/LibreOfficeLight/LibreOfficeLight/Info.plist  
  |4 
 ios/experimental/LibreOfficeLight/LibreOfficeLight/LO.swift
  |   40 
 ios/experimental/LibreOfficeLight/LibreOfficeLight/SideMenu.swift  
  |   30 +--
 ios/experimental/LibreOfficeLight/LibreOfficeLight/ViewController.swift
  |   99 --
 ios/experimental/LibreOfficeLight/LibreOfficeLight/en.lproj/Main.storyboard
  |   67 +++---
 15 files changed, 178 insertions(+), 334 deletions(-)

New commits:
commit 595e3971b130addbc6e5b749f53fc681fa6bc031
Author: jan Iversen 
Date:   Sun Mar 19 19:22:30 2017 +0100

iOS LibreOfficeLight reactivate sidemenu

Reactivate side menu in DocumentController
removed unused png files

Change-Id: Ib7da718f53370d6a8a3c82d6850817ea7bc12abc
Reviewed-on: https://gerrit.libreoffice.org/35675
Tested-by: Jenkins 
Reviewed-by: jan iversen 

diff --git 
a/ios/experimental/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj 
b/ios/experimental/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
index 6fe0b9801a43..cf301c36063e 100644
--- 
a/ios/experimental/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
+++ 
b/ios/experimental/LibreOfficeLight/LibreOfficeLight.xcodeproj/project.pbxproj
@@ -16,12 +16,10 @@
397E09081E597BD8001374E0 /* Assets.xcassets in Resources */ = 
{isa = PBXBuildFile; fileRef = 397E09071E597BD8001374E0 /* Assets.xcassets */; 
};
397E09161E597BD8001374E0 /* LibreOfficeLightTests.swift in 
Sources */ = {isa = PBXBuildFile; fileRef = 397E09151E597BD8001374E0 /* 
LibreOfficeLightTests.swift */; };
397E09211E597BD8001374E0 /* LibreOfficeLightUITests.swift in 
Sources */ = {isa = PBXBuildFile; fileRef = 397E09201E597BD8001374E0 /* 
LibreOfficeLightUITests.swift */; };
-   3992D8581E5B761700BEA987 /* ViewController.swift in Sources */ 
= {isa = PBXBuildFile; fileRef = 3992D8571E5B761700BEA987 /* 
ViewController.swift */; };
3992D85A1E5B762A00BEA987 /* DocumentController.swift in Sources 
*/ = {isa = PBXBuildFile; fileRef = 3992D8591E5B762A00BEA987 /* 
DocumentController.swift */; };
3992D85D1E5B764A00BEA987 /* SideMenu.swift in Sources */ = {isa 
= PBXBuildFile; fileRef = 3992D85B1E5B764A00BEA987 /* SideMenu.swift */; };
3992D85E1E5B764A00BEA987 /* FileManagerController.swift in 
Sources */ = {isa = PBXBuildFile; fileRef = 3992D85C1E5B764A00BEA987 /* 
FileManagerController.swift */; };
399648471E5B87DC00E73E83 /* ViewProperties.swift in Sources */ 
= {isa = PBXBuildFile; fileRef = 399648461E5B87DC00E73E83 /* 
ViewProperties.swift */; };
-   399648491E5C4E5500E73E83 /* LO.swift in Sources */ = {isa = 
PBXBuildFile; fileRef = 399648481E5C4E5500E73E83 /* LO.swift */; };
39A83A8F1E5F471D00D0C683 /* lokit.c in Sources */ = {isa = 
PBXBuildFile; fileRef = 39A83A8E1E5F471D00D0C683 /* lokit.c */; };
39B084E31E5F0A9600682A59 /* lo.xcconfig in Resources */ = {isa 
= PBXBuildFile; fileRef = 39B084E21E5F0A9600682A59 /* lo.xcconfig */; };
39B08B9D1E5F0BB600682A59 /* fundamentalrc in Resources */ = 
{isa = PBXBuildFile; fileRef = 39B084E51E5F0BB400682A59 /* fundamentalrc */; };
@@ -69,12 +67,10 @@
397E091C1E597BD8001374E0 /* LibreOfficeLightUITests.xctest */ = 
{isa = PBXFileReference; explicitFileType = wrapper.cfbundle; includeInIndex = 
0; path = LibreOfficeLightUITests.xctest; sourceTree = BUILT_PRODUCTS_DIR; };
397E09201E597BD8001374E0 /* LibreOfficeLightUITests.swift */ = 
{isa = PBXFileReference; lastKnownFileType = sourcecode.swift; name = 
LibreO

[Libreoffice-commits] core.git: wizards/source

2017-03-26 Thread Andreas Säger
 wizards/source/depot/Currency.xba|4 ++--
 wizards/source/depot/Internet.xba|4 ++--
 wizards/source/formwizard/DBMeta.xba |6 +++---
 wizards/source/formwizard/FormWizard.xba |6 +++---
 wizards/source/formwizard/Language.xba   |   14 +++---
 5 files changed, 17 insertions(+), 17 deletions(-)

New commits:
commit ad4857211bcc39ccd4c84c670bfab1b937f65089
Author: Andreas Säger 
Date:   Sun Mar 26 23:34:35 2017 +0300

tdf#106529: fix closing parentheses of bundled macros

Change-Id: I43b6d4f97cb3b1cd1426e6aa7e13550d94cb7c51
Signed-off-by: Mike Kaganski 
Reviewed-on: https://gerrit.libreoffice.org/35726
Tested-by: Jenkins 

diff --git a/wizards/source/depot/Currency.xba 
b/wizards/source/depot/Currency.xba
index 5a428d757246..d728424d3214 100644
--- a/wizards/source/depot/Currency.xba
+++ b/wizards/source/depot/Currency.xba
@@ -89,7 +89,7 @@ Dim bIsDocCountry as Boolean
End If
sCurStockIDLabel = sMarket(Index,5)
sCurExtension = sMarket(Index,8)
-   iValueCol = Val(sMarket(Index,10)
+   iValueCol = Val(sMarket(Index,10))
If Instr(sCurExtension,";") <> 0 Then
' Take the german extension as the stock place is Frankfurt
sCurExtension = "407"
@@ -192,4 +192,4 @@ Function CheckFormatType(oStyle as Object)
 Dim oFormatofObject as Object
oFormatofObject = oDocFormats.getByKey(oStyle.NumberFormat)
CheckFormatType = INT(oFormatOfObject.Type) AND 
com.sun.star.util.NumberFormat.CURRENCY
-End Function
\ No newline at end of file
+End Function
diff --git a/wizards/source/depot/Internet.xba 
b/wizards/source/depot/Internet.xba
index b4bcf579b808..333bcf98af89 100644
--- a/wizards/source/depot/Internet.xba
+++ b/wizards/source/depot/Internet.xba
@@ -223,7 +223,7 @@ Dim iCellValue as Long
If iCellValue > 0 Then
oCell.SetValue(oLinkSheet.GetCellByPosition(0,i).Value)
Else
-   
oCell.SetValue(StringToDate(oLinkSheet.GetCellByPosition(0,i).String)
+   
oCell.SetValue(StringToDate(oLinkSheet.GetCellByPosition(0,i).String))
End If
oCell = oSheet.GetCellbyPosition(SBVALUECOLUMN,iCurRow)
oCell.SetValue(oLinkSheet.GetCellByPosition(4,i).Value)
@@ -353,4 +353,4 @@ Dim bLeaveLoop as Boolean
oDateCell.Annotation.SetString(NoteText)
End If
 End Sub
-
\ No newline at end of file
+
diff --git a/wizards/source/formwizard/DBMeta.xba 
b/wizards/source/formwizard/DBMeta.xba
index 68759966daf9..b0fa20b7a901 100644
--- a/wizards/source/formwizard/DBMeta.xba
+++ b/wizards/source/formwizard/DBMeta.xba
@@ -305,8 +305,8 @@ Dim MaxQueryIndex as Integer
 Dim MaxIndex as Integer
 Dim i as Integer
 Dim a as Integer
-   MaxTableIndex = Ubound(TableNames()
-   MaxQueryIndex = Ubound(QueryNames()
+   MaxTableIndex = Ubound(TableNames())
+   MaxQueryIndex = Ubound(QueryNames())
MaxIndex = MaxTableIndex + MaxQueryIndex + 1
If MaxIndex > -1 Then
Dim LocCommandTypes(MaxIndex) as Integer
@@ -344,4 +344,4 @@ Function AssignFieldLength(FieldLength as Long) as Integer
AssignFieldLength() = FieldLength
End If
 End Function
-
\ No newline at end of file
+
diff --git a/wizards/source/formwizard/FormWizard.xba 
b/wizards/source/formwizard/FormWizard.xba
index aa2dc15bbd35..f32bba10edcd 100644
--- a/wizards/source/formwizard/FormWizard.xba
+++ b/wizards/source/formwizard/FormWizard.xba
@@ -151,11 +151,11 @@ Dim LocList() as String

DlgFormDB.GetControl("lstTables").SelectItem(sContent, True)
Else
If CommandType = 
com.sun.star.sdb.CommandType.QUERY Then
-   SelIndex = 
IndexInArray(sContent, QueryNames()
+   SelIndex = 
IndexInArray(sContent, QueryNames())

DlgFormDB.GetControl("lstTables").SelectItemPos(SelIndex, True)
ElseIf CommandType = 
com.sun.star.sdb.CommandType.TABLE Then
-   SelIndex = 
IndexInArray(sContent, TableNames()
-   
DlgFormDB.GetControl("lstTables").SelectItemPos(Ubound(QueryNames()+1 
+ SelIndex, True)
+   SelIndex = 
IndexInArray(sContent, TableNames())
+   
DlgFormDB.GetControl("lstTables").SelectItemPos(Ubound(QueryNames()+1 
+ SelIndex, True))
End If
End If
CurCommandType = CommandType
dif

[Libreoffice-commits] online.git: net/Socket.hpp wsd/DocumentBroker.cpp

2017-03-26 Thread Ashod Nakashian
 net/Socket.hpp |6 ++
 wsd/DocumentBroker.cpp |4 
 2 files changed, 10 insertions(+)

New commits:
commit 6d7f39356c26aed41aef5042b13e52aa63e9d812
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:10:24 2017 -0400

wsd: flush sockets before exiting DocBroker poll thread

Change-Id: Id5384c113bd761b5b8c25286f1bd5a28051c63d3
Reviewed-on: https://gerrit.libreoffice.org/35740
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/net/Socket.hpp b/net/Socket.hpp
index 9ff5a85a..95384397 100644
--- a/net/Socket.hpp
+++ b/net/Socket.hpp
@@ -435,6 +435,12 @@ public:
 " leaving " << _pollSockets.size());
 }
 
+size_t getSocketCount() const
+{
+assert(isCorrectThread());
+return _pollSockets.size();
+}
+
 const std::string& name() const { return _name; }
 
 /// Start the polling thread (if desired)
diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index 26452a9b..97b81fdd 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -247,6 +247,10 @@ void DocumentBroker::pollThread()
 }
 }
 
+// Flush socket data.
+for (int i = 0; i < 7 && _poll->getSocketCount() > 0; ++i)
+_poll->poll(POLL_TIMEOUT_MS / 5);
+
 // Terminate properly while we can.
 auto lock = getLock();
 terminateChild(lock, "", false);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: wsd/DocumentBroker.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/DocumentBroker.cpp |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit fddb86744e72ef6d2a0427126d56c154217d4df3
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:09:59 2017 -0400

wsd: stop DocBroker poll thread when poll is flagged to stop

Change-Id: I1bfc3a4e0a0cc7753d330a009bb295c0fce9efca
Reviewed-on: https://gerrit.libreoffice.org/35739
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index 47e86ecb..26452a9b 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -209,7 +209,7 @@ void DocumentBroker::pollThread()
 auto last30SecCheckTime = std::chrono::steady_clock::now();
 
 // Main polling loop goodness.
-while (!_stop && !TerminationFlag && !ShutdownRequestFlag)
+while (!_stop && _poll->continuePolling() && !TerminationFlag && 
!ShutdownRequestFlag)
 {
 // First, load new sessions.
 for (const auto& pair : _sessions)
@@ -1290,6 +1290,7 @@ void 
DocumentBroker::terminateChild(std::unique_lock& lock, const st
 
 // Stop the polling thread.
 _poll->stop();
+_stop = true;
 }
 
 void DocumentBroker::closeDocument(const std::string& reason)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: net/Socket.hpp wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 net/Socket.hpp  |2 +-
 wsd/LOOLWSD.cpp |1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

New commits:
commit f2f00402755e765019c9a38a84500f243156a1bd
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:08:10 2017 -0400

wsd: use isCorrectThread

Change-Id: Id63c30b19489d590b995e0ea66ef2f44e8cddfb7
Reviewed-on: https://gerrit.libreoffice.org/35738
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/net/Socket.hpp b/net/Socket.hpp
index 6a511744..9ff5a85a 100644
--- a/net/Socket.hpp
+++ b/net/Socket.hpp
@@ -288,7 +288,7 @@ public:
 }
 
 /// Are we running in either shutdown, or the polling thread.
-bool isCorrectThread()
+bool isCorrectThread() const
 {
 return _stop || std::this_thread::get_id() == _owner;
 }
diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 05abba1d..ad1f4ece 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -1416,6 +1416,7 @@ private:
 if (docBroker)
 {
 auto lock = docBroker->getLock();
+assert(docBroker->isCorrectThread());
 docBroker->terminateChild(lock, "Service unavailable", false);
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: net/Socket.hpp net/WebSocketHandler.hpp wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 net/Socket.hpp   |2 +-
 net/WebSocketHandler.hpp |9 -
 wsd/LOOLWSD.cpp  |   20 +++-
 3 files changed, 24 insertions(+), 7 deletions(-)

New commits:
commit 5ae94cc7ecc023122dfa4b67fa22bb2cf75fb088
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:06:44 2017 -0400

wsd: onConnect takes shared_ptr and better logging

Change-Id: I175dfa1f28444b9bb1f612242ee44b6d0507272c
Reviewed-on: https://gerrit.libreoffice.org/35737
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/net/Socket.hpp b/net/Socket.hpp
index 56a33de6..6a511744 100644
--- a/net/Socket.hpp
+++ b/net/Socket.hpp
@@ -516,7 +516,7 @@ public:
 /// Called when the socket is newly created to
 /// set the socket associated with this ResponseClient.
 /// Will be called exactly once.
-virtual void onConnect(const std::weak_ptr& socket) = 0;
+virtual void onConnect(const std::shared_ptr& socket) = 0;
 
 enum class SocketOwnership
 {
diff --git a/net/WebSocketHandler.hpp b/net/WebSocketHandler.hpp
index 6983fbae..ce0d13e9 100644
--- a/net/WebSocketHandler.hpp
+++ b/net/WebSocketHandler.hpp
@@ -62,9 +62,10 @@ public:
 }
 
 /// Implementation of the SocketHandlerInterface.
-void onConnect(const std::weak_ptr& socket) override
+void onConnect(const std::shared_ptr& socket) override
 {
 _socket = socket;
+LOG_TRC("#" << socket->getFD() << " Connected to WS Handler 0x" << 
std::hex << this << std::dec);
 }
 
 enum WSOpCode {
@@ -105,7 +106,10 @@ public:
 {
 auto socket = _socket.lock();
 if (socket == nullptr)
+{
+LOG_ERR("No socket associated with WebSocketHandler 0x" << 
std::hex << this << std::dec);
 return;
+}
 
 LOG_TRC("#" << socket->getFD() << ": Shutdown websocket, code: " <<
 static_cast(statusCode) << ", message: " << 
statusMessage);
@@ -128,7 +132,10 @@ public:
 {
 auto socket = _socket.lock();
 if (socket == nullptr)
+{
+LOG_ERR("No socket associated with WebSocketHandler 0x" << 
std::hex << this << std::dec);
 return false;
+}
 
 // websocket fun !
 const size_t len = socket->_inBuffer.size();
diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index d9b1db56..05abba1d 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -1396,15 +1396,19 @@ public:
 
 private:
 /// Keep our socket around ...
-void onConnect(const std::weak_ptr& socket) override
+void onConnect(const std::shared_ptr& socket) override
 {
-LOG_TRC("Prisoner - new socket");
 _socket = socket;
+LOG_TRC("#" << socket->getFD() << " Prisoner connected.");
 }
 
 void onDisconnect() override
 {
-LOG_TRC("Prisoner connection disconnected");
+auto socket = _socket.lock();
+if (socket)
+LOG_TRC("#" << socket->getFD() << " Prisoner connection 
disconnected.");
+else
+LOG_WRN("Prisoner connection disconnected but without valid 
socket.");
 
 // Notify the broker that we're done.
 auto child = _childProcess.lock();
@@ -1525,7 +1529,11 @@ private:
 if (UnitWSD::get().filterChildMessage(data))
 return;
 
-LOG_TRC("Prisoner message [" << getAbbreviatedMessage(&data[0], 
data.size()) << "].");
+auto socket = _socket.lock();
+if (socket)
+LOG_TRC("#" << socket->getFD() << " Prisoner message [" << 
getAbbreviatedMessage(&data[0], data.size()) << "].");
+else
+LOG_WRN("Message handler called but without valid socket.");
 
 auto child = _childProcess.lock();
 auto docBroker = child ? child->getDocumentBroker() : nullptr;
@@ -1567,10 +1575,11 @@ public:
 private:
 
 /// Set the socket associated with this ResponseClient.
-void onConnect(const std::weak_ptr& socket) override
+void onConnect(const std::shared_ptr& socket) override
 {
 _id = LOOLWSD::GenSessionId();
 _socket = socket;
+LOG_TRC("#" << socket->getFD() << " Connected to 
ClientRequestDispatcher.");
 }
 
 /// Called after successful socket reads.
@@ -2108,6 +2117,7 @@ private:
 {
 // Set the ClientSession to handle Socket events.
 socket->setHandler(clientSession);
+LOG_DBG("Socket #" << socket->getFD() << " handler is " << 
clientSession->getName());
 
 // Move the socket into DocBroker.
 docBroker->addSocketToPoll(socket);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |   46 --
 1 file changed, 46 deletions(-)

New commits:
commit 2a049cf60f6ca29dc6af570e7a1da2e012faa404
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:05:08 2017 -0400

wsd: kill unused code

Change-Id: I09fb52f0455e45e7217b290d5355019e14b9dbc1
Reviewed-on: https://gerrit.libreoffice.org/35736
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index df3382a0..d9b1db56 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -1556,52 +1556,6 @@ private:
 }
 };
 
-/// Handles dispatching socket traffic to the ClientSession.
-class ConvertToHandler : public SocketHandlerInterface
-{
-public:
-ConvertToHandler(const std::shared_ptr& clientSession) :
-_clientSession(clientSession)
-{
-}
-
-private:
-
-/// Set the socket associated with this ResponseClient.
-void onConnect(const std::weak_ptr& socket) override
-{
-LOG_ERR("onConnect");
-_socket = socket;
-}
-
-void onDisconnect() override
-{
-LOG_ERR("onDisconnect");
-}
-
-SocketHandlerInterface::SocketOwnership handleIncomingMessage() override
-{
-LOG_ERR("handleIncomingMessage");
-return SocketHandlerInterface::SocketOwnership::UNCHANGED;
-}
-
-int getPollEvents(std::chrono::steady_clock::time_point /* now */,
-  int & /* timeoutMaxMs */) override
-{
-return POLLIN;
-}
-
-void performWrites() override
-{
-LOG_ERR("performWrites");
-}
-
-private:
-// The socket that owns us (we can't own it).
-std::weak_ptr _socket;
-std::shared_ptr _clientSession;
-};
-
 /// Handles incoming connections and dispatches to the appropriate handler.
 class ClientRequestDispatcher : public SocketHandlerInterface
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: common/Session.hpp wsd/ClientSession.hpp wsd/DocumentBroker.cpp

2017-03-26 Thread Ashod Nakashian
 common/Session.hpp |7 ---
 wsd/ClientSession.hpp  |   13 ++---
 wsd/DocumentBroker.cpp |8 +---
 3 files changed, 3 insertions(+), 25 deletions(-)

New commits:
commit f1c371555a686611ba0b6bd324c57e7c6e7cd353
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:04:27 2017 -0400

wsd: kill unused headless stuff

Change-Id: Ic4ee7603ecd7acd10e4499457de181342f2d7483
Reviewed-on: https://gerrit.libreoffice.org/35735
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/common/Session.hpp b/common/Session.hpp
index c8f2e63b..7c337fe2 100644
--- a/common/Session.hpp
+++ b/common/Session.hpp
@@ -68,13 +68,6 @@ public:
 void closeFrame() { _isCloseFrame = true; };
 bool isCloseFrame() const { return _isCloseFrame; }
 
-bool isHeadless() const
-{
-// TODO loolnb here we should return true when there is no
-// client socket (i.e. when converting documents).
-return false;
-}
-
 protected:
 Session(const std::string& name, const std::string& id);
 virtual ~Session();
diff --git a/wsd/ClientSession.hpp b/wsd/ClientSession.hpp
index c027f4b3..5c505a43 100644
--- a/wsd/ClientSession.hpp
+++ b/wsd/ClientSession.hpp
@@ -70,17 +70,8 @@ public:
 // If in the correct thread - no need for wakeups.
 assert (!docBroker || docBroker->isCorrectThread());
 
-if (isHeadless())
-{
-// Fail silently and return as there is no actual websocket
-// connection in this case.
-LOG_INF(getName() << ": Headless peer, not forwarding message [" 
<< data->abbr() << "].");
-}
-else
-{
-LOG_TRC(getName() << " enqueueing client message " << data->id());
-_senderQueue.enqueue(data);
-}
+LOG_TRC(getName() << " enqueueing client message " << data->id());
+_senderQueue.enqueue(data);
 }
 
 bool stopping() const { return _stop || _senderQueue.stopping(); }
diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index ba3a7595..47e86ecb 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -1200,13 +1200,7 @@ bool DocumentBroker::forwardToClient(const 
std::shared_ptr& payload)
 // Broadcast to all.
 for (const auto& pair : _sessions)
 {
-#if 0 // loolnb
-if (!pair.second->isHeadless() && !pair.second->isCloseFrame())
-#endif
-if (!pair.second->isHeadless())
-{
-pair.second->handleKitToClientMessage(data, size);
-}
+pair.second->handleKitToClientMessage(data, size);
 }
 }
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: common/Message.hpp

2017-03-26 Thread Ashod Nakashian
 common/Message.hpp |4 
 1 file changed, 4 insertions(+)

New commits:
commit b69d5683c3f1718a796bd71f35b350df7e040f30
Author: Ashod Nakashian 
Date:   Sun Mar 26 23:03:15 2017 -0400

wsd: log messages upon construction

Change-Id: Id8f2eeaa1fabf538af0c327a971d35cc17cb3596
Reviewed-on: https://gerrit.libreoffice.org/35734
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 

diff --git a/common/Message.hpp b/common/Message.hpp
index d75b1e9e..99ccd77a 100644
--- a/common/Message.hpp
+++ b/common/Message.hpp
@@ -15,6 +15,7 @@
 #include 
 
 #include "Protocol.hpp"
+#include "Log.hpp"
 
 /// The payload type used to send/receive data.
 class Message
@@ -36,6 +37,7 @@ public:
 _abbr(_id + ' ' + LOOLProtocol::getAbbreviatedMessage(_data.data(), 
_data.size())),
 _type(detectType())
 {
+LOG_TRC("Message " << _abbr);
 }
 
 /// Construct a message from a string with type and
@@ -55,6 +57,7 @@ public:
 _data.resize(message.size());
 const auto offset = skipWhitespace(message.data() + 
_forwardToken.size());
 std::memcpy(_data.data(), offset, message.size() - (offset - 
message.data()));
+LOG_TRC("Message " << _abbr);
 }
 
 /// Construct a message from a character array with type.
@@ -70,6 +73,7 @@ public:
 _abbr(_id + ' ' + LOOLProtocol::getAbbreviatedMessage(_data.data(), 
_data.size())),
 _type(detectType())
 {
+LOG_TRC("Message " << _abbr);
 }
 
 size_t size() const { return _data.size(); }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |   26 +-
 1 file changed, 17 insertions(+), 9 deletions(-)

New commits:
commit d32178652df26b9607101319eae0d9d9bf5e7b9e
Author: Ashod Nakashian 
Date:   Sun Jan 1 15:42:22 2017 -0500

wsd: flag for shutdown when we fail to create forkit

And say 'forkit' in the logs where we recognize it
instead of the generic 'child'.

Change-Id: I7628b064bb6330db145a948640e48b727def3270
Reviewed-on: https://gerrit.libreoffice.org/35578
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 9ea9d497..866ad79f 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -431,7 +431,7 @@ static std::shared_ptr getNewChild()
 }
 }
 
-LOG_DBG("getNewChild: No live child, forking more.");
+LOG_WRN("getNewChild: No available child. Sending spawn request to 
forkit and failing.");
 }
 while 
(chrono::duration_cast(chrono::steady_clock::now() - 
startTime).count() <
CHILD_TIMEOUT_MS * 4);
@@ -2038,12 +2038,12 @@ int LOOLWSD::main(const std::vector& 
/*args*/)
 {
 if (WIFEXITED(status))
 {
-LOG_INF("Child process [" << pid << "] exited with 
code: " <<
+LOG_INF("Forkit process [" << pid << "] exited with 
code: " <<
 WEXITSTATUS(status) << ".");
 }
 else
 {
-LOG_ERR("Child process [" << pid << "] " <<
+LOG_ERR("Forkit process [" << pid << "] " <<
 (WCOREDUMP(status) ? "core-dumped" : "died") <<
 " with " << 
SigUtil::signalName(WTERMSIG(status)));
 }
@@ -2054,17 +2054,18 @@ int LOOLWSD::main(const std::vector& 
/*args*/)
 if (forKitPid < 0)
 {
 LOG_FTL("Failed to spawn forkit instance. Shutting 
down.");
+SigUtil::requestShutdown();
 break;
 }
 }
 else if (WIFSTOPPED(status) == true)
 {
-LOG_INF("Child process [" << pid << "] stopped with " <<
+LOG_INF("Forkit process [" << pid << "] stopped with " <<
 SigUtil::signalName(WSTOPSIG(status)));
 }
 else if (WIFCONTINUED(status) == true)
 {
-LOG_INF("Child process [" << pid << "] resumed with 
SIGCONT.");
+LOG_INF("Forkit process [" << pid << "] resumed with 
SIGCONT.");
 }
 else
 {
@@ -2073,17 +2074,24 @@ int LOOLWSD::main(const std::vector& 
/*args*/)
 }
 else
 {
-LOG_ERR("An unknown child process died, pid: " << pid);
+LOG_ERR("An unknown child process [" << pid << "] died.");
 }
 }
 else if (pid < 0)
 {
-LOG_SYS("waitpid failed.");
+LOG_SYS("Forkit waitpid failed.");
 if (errno == ECHILD)
 {
 // No child processes.
-LOG_FTL("No Forkit instance. Terminating.");
-break;
+// Spawn a new forkit and try to dust it off and resume.
+close(ForKitWritePipe);
+forKitPid = createForKit();
+if (forKitPid < 0)
+{
+LOG_FTL("Failed to spawn forkit instance. Shutting down.");
+SigUtil::requestShutdown();
+break;
+}
 }
 }
 else // pid == 0, no children have died
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 082678adb24003dc1128923a470cec1f9203c713
Author: Ashod Nakashian 
Date:   Mon Jan 2 15:40:20 2017 -0500

wsd: correctly count outstanding spawned children

Change-Id: I6e2d5c7d8fdfbcd268ef97cb1272bce76e8b69de
Reviewed-on: https://gerrit.libreoffice.org/35577
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 39538828..9ea9d497 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -296,7 +296,7 @@ static void forkChildren(const int number)
 const std::string aMessage = "spawn " + std::to_string(number) + "\n";
 LOG_DBG("MasterToForKit: " << aMessage.substr(0, aMessage.length() - 
1));
 
-++OutstandingForks;
+OutstandingForks += number;
 IoUtil::writeToPipe(LOOLWSD::ForKitWritePipe, aMessage);
 LastForkRequestTime = std::chrono::steady_clock::now();
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0783eb343a035d3ddd5c76f853e4a286e8534288
Author: Ashod Nakashian 
Date:   Fri Feb 24 21:27:13 2017 -0500

wsd: allow more time for initial kit instance

Change-Id: I141d30e0d1089521593c614be29ef5c33fd3428a
Reviewed-on: https://gerrit.libreoffice.org/35576
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index a696d8fb..39538828 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -335,7 +335,7 @@ static void preForkChildren()
 forkChildren(numPreSpawn);
 
 // Wait until we have at least one child.
-const auto timeout = std::chrono::milliseconds(CHILD_TIMEOUT_MS);
+const auto timeout = std::chrono::milliseconds(CHILD_TIMEOUT_MS * 3);
 NewChildrenCV.wait_for(lock, timeout, []() { return !NewChildren.empty(); 
});
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Ashod Nakashian
 loleaflet/src/core/Socket.js |7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 75c17fdb3c173d9cd16fbe9744ffcaaabb232052
Author: Ashod Nakashian 
Date:   Sun Feb 12 22:59:48 2017 -0500

loleaflet: reconnect silently when disconnected

Change-Id: I7761dcfd283e9b24fd70d255899d438209103a22
Reviewed-on: https://gerrit.libreoffice.org/35575
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/core/Socket.js b/loleaflet/src/core/Socket.js
index a7c18ebc..cc962d30 100644
--- a/loleaflet/src/core/Socket.js
+++ b/loleaflet/src/core/Socket.js
@@ -413,7 +413,7 @@ L.Socket = L.Class.extend({
this._map.fire('doclayerinit');
} else if (textMsg.startsWith('status:') && this._reconnecting) 
{
// we are reconnecting ...
-   this._reconecting = false;
+   this._reconnecting = false;
this._map._docLayer._onMessage('invalidatetiles: 
EMPTY', null);
this._map.fire('statusindicator', {statusType: 
'reconnected'});
this._map.setPermission(this._map.options.permission);
@@ -460,6 +460,11 @@ L.Socket = L.Class.extend({
 
// Reset wopi's app loaded so that reconnecting again informs 
outerframe about initialization again
this._map['wopi'].resetAppLoaded();
+
+   if (!this._reconnecting) {
+   this._reconnecting = true;
+   this._map._activate();
+   }
},
 
parseServerCmd: function (msg) {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Tomaž Vajngerl
 loleaflet/src/map/Map.js  |5 ++-
 loleaflet/src/map/handler/Map.Keyboard.js |   47 --
 2 files changed, 43 insertions(+), 9 deletions(-)

New commits:
commit bc1cd6bbce159a6707311a7e5404ad73dd892629
Author: Tomaž Vajngerl 
Date:   Sun Jan 15 23:53:00 2017 +0100

fix Chrome Android and Firefox Android text input

- Change from textarea to input (type="text") to prevent Firefox
to auto-capitalize.

- Clean the text input content after each word so that the soft
keyboard spellcheck suggestions are correct and that backspace
doesn't delete long IME suggestions before it has effect.

- Workaround on Chrome Android 'space' and applying spell-check
corrections.

Change-Id: I635789bc8f3b47069b7ad8b8e88bc36feb90b2b1
Reviewed-on: https://gerrit.libreoffice.org/35571
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js
index cadd3d52..fb395694 100644
--- a/loleaflet/src/map/Map.js
+++ b/loleaflet/src/map/Map.js
@@ -594,7 +594,10 @@ L.Map = L.Evented.extend({
}
 
var textAreaContainer = L.DomUtil.create('div', 
'clipboard-container', container.parentElement);
-   this._textArea = L.DomUtil.create('textarea', 'clipboard', 
textAreaContainer);
+   this._textArea = L.DomUtil.create('input', 'clipboard', 
textAreaContainer);
+   this._textArea.setAttribute('type', 'text');
+   this._textArea.setAttribute('autocorrect', 'off');
+   this._textArea.setAttribute('autocapitalize', 'off');
this._resizeDetector = L.DomUtil.create('iframe', 
'resize-detector', container);
this._fileDownloader = L.DomUtil.create('iframe', '', 
container);
L.DomUtil.setStyle(this._fileDownloader, 'display', 'none');
diff --git a/loleaflet/src/map/handler/Map.Keyboard.js 
b/loleaflet/src/map/handler/Map.Keyboard.js
index 8aff7550..752e74b0 100644
--- a/loleaflet/src/map/handler/Map.Keyboard.js
+++ b/loleaflet/src/map/handler/Map.Keyboard.js
@@ -284,12 +284,28 @@ L.Map.Keyboard = L.Handler.extend({
var charCode = e.originalEvent.charCode;
var keyCode = e.originalEvent.keyCode;
 
+   if (e.type === 'compositionstart' || e.type === 
'compositionupdate') {
+   this._isComposing = true; // we are starting composing 
with IME
+   }
+
if (e.type === 'compositionend') {
+   this._isComposing = false; // stop of composing with IME
+   // get the composited char codes
var compCharCodes = [];
for (var i = 0; i < e.originalEvent.data.length; i++) {

compCharCodes.push(e.originalEvent.data[i].charCodeAt());
}
+   // clear the input now - best to do this ASAP so the 
input
+   // is clear for the next word
+   this._map._textArea.value = '';
+   }
+
+   if (!this._isComposing && e.type === 'keyup') {
+   // not compositing and keyup, clear the input so it is 
ready
+   // for next word (or char only)
+   this._map._textArea.value = '';
}
+
var unoKeyCode = this._toUNOKeyCode(keyCode);
 
if (this.modifier) {
@@ -338,15 +354,30 @@ L.Map.Keyboard = L.Handler.extend({
this._bufferedTextInputEvent = e;
}
else if (e.type === 'keyup') {
-   // Hack for making space work in chrome when 
IME is enabled
-   // Chrome doesn't fire compositionend event or 
keypress when
-   // IME is enabled *and* user presses .
-   // However, it sends 'textInput' event in such 
a case.
-   // Use the buffered textInput event if its the 
space key and has not been
-   // handled already by 'keypress' or 
'compositionend' events above
-   if (!this._keyHandled && 
this._bufferedTextInputEvent && e.originalEvent.key === 
this._bufferedTextInputEvent.originalEvent.data) {
+   // Hack for making space and spell-check text 
insert work
+   // in Chrome (on Andorid) or Chrome with IME.
+   //
+   // Chrome (Android) IME triggers keyup/keydown 
input with
+   // code 229 when hitting space (as with all 
composiiton events)
+   // with addition to 'textinput' event, in which 
we only see that
+ 

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - kit/Kit.cpp

2017-03-26 Thread Ashod Nakashian
 kit/Kit.cpp |   17 ++---
 1 file changed, 14 insertions(+), 3 deletions(-)

New commits:
commit 164ce37afd1ee329fc0d60ac89389e9dcf5d3ba3
Author: Ashod Nakashian 
Date:   Sun Jan 15 23:54:08 2017 -0500

wsd: copy jail files when symlinking fails

Change-Id: I1f56d1489820a0689495e09151101d41a2322b82
Reviewed-on: https://gerrit.libreoffice.org/35570
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/kit/Kit.cpp b/kit/Kit.cpp
index 15fd559d..a9f7417e 100644
--- a/kit/Kit.cpp
+++ b/kit/Kit.cpp
@@ -148,9 +148,18 @@ namespace
 File(newPath.parent()).createDirectories();
 if (link(fpath, newPath.toString().c_str()) == -1)
 {
-LOG_SYS("link(\"" << std::string(fpath) << "\",\"" <<
-newPath.toString() << "\") failed. Exiting.");
-std::_Exit(Application::EXIT_SOFTWARE);
+LOG_SYS("link(\"" << fpath << "\", \"" <<
+newPath.toString() << "\") failed. Will copy.");
+try
+{
+File(fpath).copyTo(newPath.toString());
+}
+catch (const std::exception& exc)
+{
+LOG_ERR("Copying of '" << fpath << "' to " << 
newPath.toString() <<
+" failed: " << exc.what() << ". Exiting.");
+std::_Exit(Application::EXIT_SOFTWARE);
+}
 }
 break;
 case FTW_D:
@@ -201,7 +210,9 @@ namespace
 sourceForLinkOrCopy.pop_back();
 destinationForLinkOrCopy = destination;
 if (nftw(source.c_str(), linkOrCopyFunction, 10, FTW_ACTIONRETVAL) == 
-1)
+{
 LOG_ERR("linkOrCopy: nftw() failed for '" << source << "'");
+}
 }
 
 void dropCapability(cap_value_t capability)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 003e1ee5204e4191b2404ca674bf223797c1d2e0
Author: Pranav Kant 
Date:   Tue Dec 20 15:56:42 2016 +0530

loleaflet: Disable bullet/numbering in calc

Change-Id: I7509fbf8cfb593fc23ace3ca02c71342d1112def
Reviewed-on: https://gerrit.libreoffice.org/35567
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index b2ac4432..e4b7a0f3 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -848,8 +848,8 @@ map.on('doclayerinit', function () {
 
switch (docType) {
case 'spreadsheet':
-   toolbarUp.remove('inserttable', 'styles', 'alignblock', 
'bullet', 'numbering', 'break-numbering');
-   toolbarUpMore.remove('inserttable', 'styles', 'alignblock', 
'bullet', 'numbering', 'break-numbering');
+   toolbarUp.remove('inserttable', 'styles', 'alignblock', 
'defaultbullet', 'defaultnumbering', 'break-numbering');
+   toolbarUpMore.remove('inserttable', 'styles', 'alignblock', 
'defaultbullet', 'defaultnumbering', 'break-numbering');
statusbar.disable('zoomreset', 'zoomout', 'zoomin', 
'zoomlevel');
statusbar.insert('left', [
{type: 'break', id:'break1'},
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 27c2163e6d1a58be1fcf39b63c696d85a6a125c6
Author: Pranav Kant 
Date:   Thu Jan 12 16:44:36 2017 +0530

loleaflet: Move this logic to updateCommandValues()

Change-Id: I64e96f87e56c1b14dd2085d552eebd40977cd7e9
Reviewed-on: https://gerrit.libreoffice.org/35565
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 5ffa88b9..b2ac4432 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -498,10 +498,6 @@ $(function () {
}
 
updateCommandValues();
-   if (fontsSelectValue){
-   
$('.fontsizes-select').val(fontsizesSelectValue);
-   updateFontSizeList(fontsSelectValue);
-   }
 
insertTable();
}
@@ -1236,6 +1232,10 @@ function updateCommandValues() {
});
 
$('.fontsizes-select').on('select2:select', onFontSizeSelect);
+   if (fontsSelectValue) {
+   updateFontSizeList(fontsSelectValue);
+   }
+   
$('.fontsizes-select').val(fontsizesSelectValue).trigger('change');
}
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   55 +-
 1 file changed, 42 insertions(+), 13 deletions(-)

New commits:
commit 07065bbb16a2e525f1a509725ef99362f175b76c
Author: Pranav Kant 
Date:   Thu Jan 12 16:33:35 2017 +0530

loleaflet: Update dropdowns on refresh everytime

Change-Id: I39071f1c6249e6ecf71684d3b93cc88f1f859510
Reviewed-on: https://gerrit.libreoffice.org/35564
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index cc1c0786..5ffa88b9 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -332,6 +332,10 @@ function onColorPick(e, color) {
 // edit/view mode, state from this object is read and then applied on 
corresponding buttons
 var formatButtons = {};
 
+var stylesSelectValue;
+var fontsSelectValue;
+var fontsizesSelectValue;
+
 $(function () {
$('#toolbar-up-more').w2toolbar({
name: 'toolbar-up-more',
@@ -493,6 +497,12 @@ $(function () {
}
}
 
+   updateCommandValues();
+   if (fontsSelectValue){
+   
$('.fontsizes-select').val(fontsizesSelectValue);
+   updateFontSizeList(fontsSelectValue);
+   }
+
insertTable();
}
});
@@ -952,6 +962,8 @@ map.on('commandstatechanged', function (e) {
.append($('')
.text(state));
}
+
+   stylesSelectValue = state;
$('.styles-select').val(state).trigger('change');
}
else if (commandName === '.uno:CharFontName') {
@@ -969,6 +981,7 @@ map.on('commandstatechanged', function (e) {
.append($('')
.text(state));
}
+   fontsSelectValue = state;
$('.fonts-select').val(state).trigger('change');
}
else if (commandName === '.uno:FontHeight') {
@@ -987,6 +1000,7 @@ map.on('commandstatechanged', function (e) {
.append($('')
.text(state).val(state));
}
+   fontsizesSelectValue = state;
$('.fontsizes-select').val(state).trigger('change');
sortFontSizes();
}
@@ -1129,13 +1143,17 @@ map.on('search', function (e) {
}
 });
 
-map.on('updatetoolbarcommandvalues', function (e) {
-   // we need an empty option for the place holder to work
+function updateCommandValues() {
var data = [];
-   var styles = [];
-   var topStyles = [];
-   if (e.commandName === '.uno:StyleApply') {
-   var commands = e.commandValues.Commands;
+   // 1) For .uno:StyleApply
+   // we need an empty option for the place holder to work
+   if ($('.styles-select option').length === 0) {
+   var styles = [];
+   var topStyles = [];
+   var commandValues = 
map.getToolbarCommandValues('.uno:StyleApply');
+   if (typeof commandValues === 'undefined')
+   return;
+   var commands = commandValues.Commands;
if (commands && commands.length > 0) {
// Inserts a separator element
data = data.concat({text: 
'\u2500\u2500\u2500\u2500\u2500\u2500', disabled: true});
@@ -1146,11 +1164,11 @@ map.on('updatetoolbarcommandvalues', function (e) {
}
 
if (map.getDocType() === 'text') {
-   styles = e.commandValues.ParagraphStyles.slice(7, 19);
-   topStyles = e.commandValues.ParagraphStyles.slice(0, 7);
+   styles = commandValues.ParagraphStyles.slice(7, 19);
+   topStyles = commandValues.ParagraphStyles.slice(0, 7);
}
else if (map.getDocType() === 'spreadsheet') {
-   styles = e.commandValues.CellStyles;
+   styles = commandValues.CellStyles;
}
else if (map.getDocType() === 'presentation') {
// styles are not applied for presentation
@@ -1188,27 +1206,38 @@ map.on('updatetoolbarcommandvalues', function (e) {
data: data,
placeholder: _('Style')
});
+   $('.styles-select').val(stylesSelectValue).trigger('change');
$('.styles-select').on('select2:select', onStyleSelect);
}
-   else if (e.commandName === '.uno:CharFontName') {
+
+   if ($('.fonts-select option').length === 0) {
+   // 2) For .uno:CharFontName
+   commandValues = 
map.getToolbarCommandValues('.uno:CharFontName

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - discovery.xml

2017-03-26 Thread Andras Timar
 discovery.xml |6 ++
 1 file changed, 6 insertions(+)

New commits:
commit acd766a5638494c3a94e434e91fde484292a7249
Author: Andras Timar 
Date:   Sat Jan 7 21:26:00 2017 +0100

add text/rtf and text/plain as supported MIME types

Change-Id: Ic46e8ccaeecc24ef2c6b6fce6b342c976c3ddc60
Reviewed-on: https://gerrit.libreoffice.org/35562
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/discovery.xml b/discovery.xml
index 0edd631c..c8b87c5b 100644
--- a/discovery.xml
+++ b/discovery.xml
@@ -254,6 +254,12 @@
 
 
 
+
+
+
+
+
+
 
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - 2 commits - loleaflet/dist loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/dist/toolbar/toolbar.js |   44 ++
 loleaflet/src/control/Toolbar.js  |9 +++
 2 files changed, 53 insertions(+)

New commits:
commit 2371a82381ef1e6ab6826dd81e26afc5f39e8b0d
Author: Henry Castro 
Date:   Thu Dec 22 13:41:26 2016 -0400

tdf#103779: Help menu accessibility

(cherry picked from commit 1bede62b132e61fa847aaf066447c2670d14dbab)

Change-Id: Ib2b7dc3222c064d7a3f3d90d2182daf970b5e6a5
Reviewed-on: https://gerrit.libreoffice.org/35480
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/control/Toolbar.js b/loleaflet/src/control/Toolbar.js
index 2e87ff65..f95b7b7b 100644
--- a/loleaflet/src/control/Toolbar.js
+++ b/loleaflet/src/control/Toolbar.js
@@ -212,6 +212,8 @@ L.Map.include({

translatableContent[i].firstChild.nodeValue = 
translatableContent[i].firstChild.nodeValue.toLocaleString();
}
 
+   $('.vex-content').attr('tabindex', -1);
+   $('.vex-content').focus();
// workaround for 
https://github.com/HubSpot/vex/issues/43
$('.vex-overlay').css({ 
'pointer-events': 'none'});
$('.vex').click(function() {
@@ -220,6 +222,9 @@ L.Map.include({
$('.vex-content').click(function(e) {
e.stopPropagation();
});
+   },
+   beforeClose: function () {
+   map.focus();
}
});
});
@@ -242,6 +247,7 @@ L.Map.include({
contentCSS: { width: w + 'px'},
buttons: {},
afterOpen: function($vexContent) {
+   map.enable(false);
// workaround for 
https://github.com/HubSpot/vex/issues/43
$('.vex-overlay').css({ 'pointer-events': 
'none'});
$('.vex').click(function() {
@@ -250,6 +256,9 @@ L.Map.include({
$('.vex-content').click(function(e) {
e.stopPropagation();
});
+   },
+   beforeClose: function () {
+   map.enable(true);
}
});
}
commit c29369231cdda89e008881690f20cdb2940b0908
Author: Henry Castro 
Date:   Wed Dec 21 19:50:04 2016 -0400

tdf#103775: Font color/Highlighting button issue

(cherry picked from commit 65bfc63ace567501912e4b6399ff8f81cc62516e)

Change-Id: Ie52972574cc07ac5826da261aab6671e9fd0272c
Reviewed-on: https://gerrit.libreoffice.org/35479
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 819404f5..cc1c0786 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -408,6 +408,28 @@ $(function () {
if (!L.DomUtil.get('fontcolorindicator')) {
var fontColorIndicator = 
L.DomUtil.create('div', 'font-color-indicator', 
L.DomUtil.get('tb_toolbar-up_item_fontcolor'));
fontColorIndicator.id = 'fontcolorindicator';
+   L.DomEvent.on(fontColorIndicator, 'mouseover', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).addClass('over');
+   });
+   L.DomEvent.on(fontColorIndicator, 'mouseout', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).removeClass('over');
+   });
+   L.DomEvent.on(fontColorIndicator, 'mousedown', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).addClass('down');
+   });
+   L.DomEvent.on(fontColorIndicator, 'mouseup', 
function () {
+   var button = 
fontColorIndicator.parentNode.firstChild;
+   $(button).removeClass('down');
+   });
+   fontColorIndicator.addEventListener('click', 
function () {
+

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/main.js

2017-03-26 Thread Henry Castro
 loleaflet/main.js |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 674ba96da0351f3a216d7974e35358d3a7c0b72e
Author: Henry Castro 
Date:   Wed Dec 28 13:44:49 2016 -0400

tdf#103642 Page refresh causes unwanted error message

(cherry picked from commit 9c64db3ad1d60c4de659d779a4fba17a298ed648)

Change-Id: Ia276b00a4a1638cdb878c052886c01a9dc47db58
Reviewed-on: https://gerrit.libreoffice.org/35481
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/main.js b/loleaflet/main.js
index 93e85115..29399a92 100644
--- a/loleaflet/main.js
+++ b/loleaflet/main.js
@@ -114,4 +114,9 @@ map.addControl(L.control.rowHeader());
 map.addControl(L.control.contextMenu());
 map.addControl(L.control.menubar());
 
+window.addEventListener('beforeunload', function () {
+   if (global.map && global.map._socket) {
+   global.map._socket.close();
+   }
+});
 //require('./dist/toolbar/toolbar');
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - 2 commits - loleaflet/dist

2017-03-26 Thread Henry Castro
 loleaflet/dist/toolbar/toolbar.js |   24 +++-
 1 file changed, 19 insertions(+), 5 deletions(-)

New commits:
commit 9a9b6e05c115a8bc766f6330080f5c30db00736c
Author: Henry Castro 
Date:   Tue Dec 20 14:41:37 2016 -0400

tdf#103776: Hide/Deactivate "Document repair" button

(cherry picked from commit 9450f3938c16a35a7d261c31d830781782697797)

Change-Id: I1a4077534327aea8032fe9d48d951a824cb81113
Reviewed-on: https://gerrit.libreoffice.org/35478
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 1cefc8ae..819404f5 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -1060,6 +1060,14 @@ map.on('commandstatechanged', function (e) {
}
}
}
+
+   if (id === 'undo' || id === 'redo') {
+   if (toolbar.get('undo').disabled && 
toolbar.get('redo').disabled) {
+   toolbar.disable('repair');
+   } else {
+   toolbar.enable('repair');
+   }
+   }
 });
 
 map.on('search', function (e) {
commit 2c32ca0ecc361e1b38a91f5c855ca6594cad4057
Author: Pranav Kant 
Date:   Fri Jan 6 14:42:27 2017 +0530

loleaflet: Make id of these toolbar items consistent with others

We assume that id of the toolbar item is lowercase(unocommand),
so lets keep it consistent too for alignment buttons.

This also makes alignment buttons toggle except for spreadsheet
where alignment of cell text happens via a different uno command,
HorizontalAlignment.

Change-Id: Ibacf06bd99435cd24b27c7870173a2519b167af1
Reviewed-on: https://gerrit.libreoffice.org/35477
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 23c62410..1cefc8ae 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -368,10 +368,10 @@ $(function () {
{type: 'html',  id: 'backcolor-html', html: ''},
{type: 'button',  id: 'backcolor', img: 'backcolor', 
hint: _('Highlighting')},
{type: 'break'},
-   {type: 'button',  id: 'alignleft',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
-   {type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
-   {type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
-   {type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'button',  id: 'leftpara',  img: 'alignleft', 
hint: _('Align left'), uno: 'LeftPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"1"}}'},
+   {type: 'button',  id: 'centerpara',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
+   {type: 'button',  id: 'rightpara',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
+   {type: 'button',  id: 'justifypara',  img: 
'alignblock', hint: _('Justified'), uno: 'JustifyPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"4"}}'},
{type: 'break',  id: 'wraptextseparator'},
{type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
{type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
@@ -588,6 +588,11 @@ function toLocalePattern (pattern, regex, text, sub1, 
sub2) {
return text;
 }
 
+function unoCmdToToolbarId(commandname)
+{
+   return commandname.toLowerCase().substr(5);
+}
+
 function selectItem(item, func)
 {
var index = -1;
@@ -873,6 +878,7 @@ map.on('commandstatechanged', function (e) {
var found = false;
var value, color, div;
var matches;
+
if (commandName === '.uno:AssignLayout') {
$('.styles-select').val(state).trigger('change');
} els

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   13 +++--
 1 file changed, 7 insertions(+), 6 deletions(-)

New commits:
commit 4d13636ab7ffa5437acf64c47b3ff64bec0afe0a
Author: Pranav Kant 
Date:   Tue Dec 20 15:53:20 2016 +0530

loleaflet: Separator before 'wrap text' in calc

Change-Id: If1f4536dec24238ff1476e9e7d9ab58207d86495
Reviewed-on: https://gerrit.libreoffice.org/35476
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 85a987f9..23c62410 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -372,6 +372,7 @@ $(function () {
{type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
{type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
{type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'break',  id: 'wraptextseparator'},
{type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
{type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
{type: 'break',   id: 'break-toggle'},
@@ -824,8 +825,8 @@ map.on('doclayerinit', function () {
]);
break;
case 'text':
-   toolbarUp.remove('wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
-   toolbarUpMore.remove('wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
+   toolbarUp.remove('wraptextseparator', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('wraptextseparator', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'StatePageNumber',
@@ -844,8 +845,8 @@ map.on('doclayerinit', function () {
case 'presentation':
var presentationToolbar = w2ui['presentation-toolbar'];
presentationToolbar.show('presentation', 'presentationbreak', 
'insertpage', 'duplicatepage', 'deletepage');
-   toolbarUp.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
-   toolbarUpMore.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUp.remove('insertannotation', 'wraptextseparator', 
'wraptext', 'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('insertannotation', 'wraptextseparator', 
'wraptext', 'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'PageStatus',
@@ -853,8 +854,8 @@ map.on('doclayerinit', function () {

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 8d3815c32aa55a9ab5b17f98c08ce8c2359257bd
Author: Ashod Nakashian 
Date:   Thu Dec 22 19:33:46 2016 -0500

wsd: log failure in else and not unconditionally

Change-Id: I24f1ac57f350aa67cc14682d695e1e4a07453c8d
Reviewed-on: https://gerrit.libreoffice.org/35475
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index b069fcb6..a696d8fb 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -834,9 +834,12 @@ private:
 }
 }
 }
+else
+{
+LOG_DBG("No DocumentBroker with docKey [" << docKey << "] found. 
New Child and Document.");
+}
 
 Util::assertIsLocked(docBrokersLock);
-LOG_DBG("No DocumentBroker with docKey [" << docKey << "] found. New 
Child and Document.");
 
 if (TerminationFlag)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/DocumentBroker.hpp wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/DocumentBroker.hpp |6 ++
 wsd/LOOLWSD.cpp|   18 --
 2 files changed, 18 insertions(+), 6 deletions(-)

New commits:
commit a9a21eb232411a00afe350ebc61f20d75253b7ff
Author: Ashod Nakashian 
Date:   Tue Dec 20 23:06:59 2016 -0500

wsd: don't poll WS for write to check child alive

The WS could be chocked on write, but we shouldn't
assume the child is dead because of that. We are
only trying to test if the child process is alive
in that helper.

Also, log DocBroker activity better.

Change-Id: I108a297e43f923cab0dfa30204837dc4df15d3a1
Reviewed-on: https://gerrit.libreoffice.org/35474
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/DocumentBroker.hpp b/wsd/DocumentBroker.hpp
index f31471aa..feb96979 100644
--- a/wsd/DocumentBroker.hpp
+++ b/wsd/DocumentBroker.hpp
@@ -140,17 +140,15 @@ public:
 return false;
 }
 
-/// Check whether this child is alive and able to respond.
+/// Check whether this child is alive and socket not in error.
 /// Note: zombies will show as alive, and sockets have waiting
 /// time after the other end-point closes. So this isn't accurate.
 bool isAlive() const
 {
 try
 {
-using namespace Poco::Net;
 return (_pid > 1 && _ws && kill(_pid, 0) == 0 &&
-_ws->poll(Poco::Timespan(0), 
Socket::SelectMode::SELECT_WRITE) &&
-!_ws->poll(Poco::Timespan(0), 
Socket::SelectMode::SELECT_ERROR));
+!_ws->poll(Poco::Timespan(0), 
Poco::Net::Socket::SelectMode::SELECT_ERROR));
 }
 catch (const std::exception& exc)
 {
diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 11b37694..b069fcb6 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -265,7 +265,18 @@ bool cleanupDocBrokers()
 }
 }
 
-return (count != DocBrokers.size());
+if (count != DocBrokers.size())
+{
+LOG_TRC("Have " << DocBrokers.size() << " DocBrokers after cleanup.");
+for (auto& pair : DocBrokers)
+{
+LOG_TRC("DocumentBroker [" << pair.first << "].");
+}
+
+return true;
+}
+
+return false;
 }
 
 static void forkChildren(const int number)
@@ -596,6 +607,7 @@ private:
 LOG_DBG("Removing DocumentBroker for docKey [" << 
docKey << "].");
 DocBrokers.erase(docKey);
 docBroker->terminateChild(docLock);
+LOG_TRC("Have " << DocBrokers.size() << " DocBrokers 
after removing.");
 }
 else
 {
@@ -760,7 +772,7 @@ private:
 if (it != DocBrokers.end() && it->first == docKey)
 {
 // Get the DocumentBroker from the Cache.
-LOG_DBG("Found DocumentBroker for docKey [" << docKey << "].");
+LOG_DBG("Found DocumentBroker with docKey [" << docKey << "].");
 docBroker = it->second;
 assert(docBroker);
 
@@ -824,6 +836,7 @@ private:
 }
 
 Util::assertIsLocked(docBrokersLock);
+LOG_DBG("No DocumentBroker with docKey [" << docKey << "] found. New 
Child and Document.");
 
 if (TerminationFlag)
 {
@@ -976,6 +989,7 @@ private:
 LOG_INF("Removing DocumentBroker for docKey [" << 
docKey << "].");
 DocBrokers.erase(docKey);
 docBroker->terminateChild(lock);
+LOG_TRC("Have " << DocBrokers.size() << " DocBrokers 
after removing.");
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 22ab7a3ad37ef347b5d164453b8d66bacfc03adb
Author: Pranav Kant 
Date:   Sun Dec 18 09:56:03 2016 +0530

loleaflet: Fix remaining incorrect key name, insertannotation

Fallback from e2151b30b270ef47faf403a412ee6e858febdeee

Change-Id: I26130bd75e26f2ae4f8fba864115d2b50f938f30
Reviewed-on: https://gerrit.libreoffice.org/35473
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 87b0bc4f..85a987f9 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -844,8 +844,8 @@ map.on('doclayerinit', function () {
case 'presentation':
var presentationToolbar = w2ui['presentation-toolbar'];
presentationToolbar.show('presentation', 'presentationbreak', 
'insertpage', 'duplicatepage', 'deletepage');
-   toolbarUp.remove('annotation', 'wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
-   toolbarUpMore.remove('annotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUp.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'PageStatus',
@@ -853,8 +853,8 @@ map.on('doclayerinit', function () {
]);
break;
case 'drawing':
-   toolbarUp.remove('annotation', 'wraptext', 'togglemergecells', 
'break-toggle', 'numberformatcurrency', 'numberformatpercent', 
'numberformatdecimal', 'numberformatdate', 'numberformatincdecimals', 
'numberformatdecdecimals', 'break-number', 'sortascending', 'sortdescending');
-   toolbarUpMore.remove('annotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUp.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
+   toolbarUpMore.remove('insertannotation', 'wraptext', 
'togglemergecells', 'break-toggle', 'numberformatcurrency', 
'numberformatpercent', 'numberformatdecimal', 'numberformatdate', 
'numberformatincdecimals', 'numberformatdecdecimals', 'break-number', 
'sortascending', 'sortdescending');
break;
}
toolbarUp.refresh();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   33 -
 1 file changed, 16 insertions(+), 17 deletions(-)

New commits:
commit e4c0e3706712e188e520d0e06fc0bbc87bf74bf7
Author: Pranav Kant 
Date:   Fri Dec 16 22:08:17 2016 +0530

loleaflet: Cleanup & fix default numbering/bullet state

Automatically initialize formatButtons internal object to
preclude the possibility of one forgetting to enter the id of the
button in formatButtons object.

Fix 'id' of default numbering/bullet button after which it is
correctly showing the state now.

Change-Id: I57247e1b0e6d462153a47d31a883ceb04ecff74b
Reviewed-on: https://gerrit.libreoffice.org/35471
Reviewed-by: pranavk 
Tested-by: pranavk 
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 7d5ab254..87b0bc4f 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -327,6 +327,11 @@ function onColorPick(e, color) {
map.focus();
 }
 
+// This object is used to store enabled/disabled state of each and every 
eligible item
+// (i.e the one having UNO command) of toolbar-up. When the permission is 
changed to/from
+// edit/view mode, state from this object is read and then applied on 
corresponding buttons
+var formatButtons = {};
+
 $(function () {
$('#toolbar-up-more').w2toolbar({
name: 'toolbar-up-more',
@@ -380,8 +385,8 @@ $(function () {
{type: 'button',  id: 'sortascending',  img: 
'sortascending', hint: _('Sort Ascending'), uno: 'SortAscending'},
{type: 'button',  id: 'sortdescending',  img: 
'sortdescending', hint: _('Sort Descending'), uno: 'SortDescending'},
{type: 'break',   id: 'break-align'},
-   {type: 'button',  id: 'bullet',  img: 'bullet', hint: 
_('Bullets on/off'), uno: 'DefaultBullet'},
-   {type: 'button',  id: 'numbering',  img: 'numbering', 
hint: _('Numbering on/off'), uno: 'DefaultNumbering'},
+   {type: 'button',  id: 'defaultbullet',  img: 'bullet', 
hint: _('Bullets on/off'), uno: 'DefaultBullet'},
+   {type: 'button',  id: 'defaultnumbering',  img: 
'numbering', hint: _('Numbering on/off'), uno: 'DefaultNumbering'},
{type: 'break',   id: 'break-numbering'},
{type: 'button',  id: 'incrementindent',  img: 
'incrementindent', hint: _('Increase indent'), uno: 'IncrementIndent'},
{type: 'button',  id: 'decrementindent',  img: 
'decrementindent', hint: _('Decrease indent'), uno: 'DecrementIndent'},
@@ -434,6 +439,15 @@ $(function () {
}
}
 
+   // Intialize the formatButtons object
+   if (Object.keys(formatButtons).length === 0) {
+   for (var itemIdx in w2ui['toolbar-up'].items) {
+   if 
(w2ui['toolbar-up'].items[itemIdx].uno) {
+   
formatButtons[w2ui['toolbar-up'].items[itemIdx].id] = true;
+   }
+   }
+   }
+
insertTable();
}
});
@@ -520,21 +534,6 @@ $(function () {
});
 });
 
-// This object is used to track enabled/disabled state when one is in view mode
-var formatButtons = {
-   'undo': true, 'redo': true, 'save': true,
-   'bold': true, 'italic': true, 'underline': true, 'strikeout': true,
-   'insertannotation': true, 'inserttable': true,
-   'fontcolor': true, 'backcolor': true, 'bullet': true, 'numbering': true,
-   'alignleft': true, 'alignhorizontal': true, 'alignright': true, 
'alignblock': true,
-   'incrementindent': true, 'decrementindent': true, 'insertgraphic': true,
-   'insertfootnote': true, 'repair': true, 'specialcharacter': true,
-   'wraptext': true, 'togglemergecells': true, 'numberformatcurrency': 
true,
-   'numberformatpercent': true, 'numberformatdecimal': true, 
'numberformatdate': true,
-   'numberformatincdecimals': true, 'numberformatdecdecimals': true,
-   'sortascending': true, 'sortdescending': true
-};
-
 var userJoinedPopupMessage = '' + _('%user has joined') + '';
 var userLeftPopupMessage = '' + _('%user has left') + '';
 var userPopupTimeout = null;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Pranav Kant
 loleaflet/src/map/Map.js |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit e2383db17942ba7324dc4e1a397362bddbf25be4
Author: Pranav Kant 
Date:   Fri Dec 16 23:08:44 2016 +0530

tdf#106655 loleaflet: Fix cursor dissappearing during document load

Change-Id: I9e301d7d2b6e7d2626de91c242b7617fbc823572
Reviewed-on: https://gerrit.libreoffice.org/35465
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js
index bc1b2790..cadd3d52 100644
--- a/loleaflet/src/map/Map.js
+++ b/loleaflet/src/map/Map.js
@@ -823,6 +823,7 @@ L.Map = L.Evented.extend({
_onLostFocus: function () {
if (!this._loaded) { return; }
 
+   console.debug('_onLostFocus: ');
var doclayer = this._docLayer;
if (!doclayer) { return; }
 
@@ -843,7 +844,9 @@ L.Map = L.Evented.extend({
if (!this._loaded) { return; }
 
var doclayer = this._docLayer;
-   if (doclayer) {
+   if (doclayer &&
+   typeof doclayer._isCursorOverlayVisibleOnLostFocus !== 
'undefined' &&
+   typeof doclayer._isCursorVisibleOnLostFocus !== 
'undefined') {
// we restore the old cursor position by a small delay, 
so that if the user clicks
// inside the document we skip to restore it, so that 
the user does not see the cursor
// jumping from the old position to the new one
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Andras Timar
 loleaflet/src/control/Control.CharacterMap.js |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 62b02cac63c00d06563ae3f323914fa3d2371514
Author: Andras Timar 
Date:   Mon Dec 12 16:20:25 2016 +0100

loleaflet: typos

Change-Id: I53c3ed3ce21c29fb13917bf92a8512a2aec5a90c
Reviewed-on: https://gerrit.libreoffice.org/35461
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/control/Control.CharacterMap.js 
b/loleaflet/src/control/Control.CharacterMap.js
index c6885a39..60703b52 100644
--- a/loleaflet/src/control/Control.CharacterMap.js
+++ b/loleaflet/src/control/Control.CharacterMap.js
@@ -73,7 +73,7 @@ L.Control.CharacterMap = L.Control.extend({
{ name: _('Hiragana'),  start: 
0x3040, end: 0x309F }, /*UBLOCK_HIRAGANA=62*/
{ name: _('Katakana'),  start: 
0x30A0, end: 0x30FF }, /*UBLOCK_KATAKANA=63*/
{ name: _('Bopomofo'),  start: 
0x3100, end: 0x312F }, /*UBLOCK_BOPOMOFO=64*/
-   { name: _('Hangul Compatability Jamo'), start: 
0x3130, end: 0x318F }, /*UBLOCK_HANGUL_COMPATIBILITY_JAMO=65*/
+   { name: _('Hangul Compatibility Jamo'), start: 
0x3130, end: 0x318F }, /*UBLOCK_HANGUL_COMPATIBILITY_JAMO=65*/
{ name: _('Kanbun'),start: 
0x3190, end: 0x319F }, /*UBLOCK_KANBUN=66*/
{ name: _('Bopomofo Extended'), start: 
0x31A0, end: 0x31BF }, /*UBLOCK_BOPOMOFO_EXTENDED=67*/
{ name: _('Enclosed CJK Letters and Months'),   start: 
0x3200, end: 0x32FF }, /*UBLOCK_ENCLOSED_CJK_LETTERS_AND_MONTHS=68*/
@@ -147,7 +147,7 @@ L.Control.CharacterMap = L.Control.extend({
{ name: _('Glagolitic'),start: 
0x2C00, end: 0x2C5F }, /*UBLOCK_GLAGOLITIC=136*/
{ name: _('Kharoshthi'),start: 
0x10A00, end: 0x10A5F }, /*UBLOCK_KHAROSHTHI=137*/
{ name: _('Modifier Tone Letters'), start: 
0xA700, end: 0xA71F }, /*UBLOCK_MODIFIER_TONE_LETTERS=138*/
-   { name: _('"New Tai Lue'),  start: 
0x1980, end: 0x19DF }, /*UBLOCK_NEW_TAI_LUE=139*/
+   { name: _('New Tai Lue'),   start: 
0x1980, end: 0x19DF }, /*UBLOCK_NEW_TAI_LUE=139*/
{ name: _('Old Persian'),   start: 
0x103A0, end: 0x103DF }, /*UBLOCK_OLD_PERSIAN=140*/
{ name: _('Phonetic Extensions Supplement'),start: 
0x1D80, end: 0x1DBF }, /*UBLOCK_PHONETIC_EXTENSIONS_SUPPLEMENT=141*/
{ name: _('Supplemental Punctuation'),  start: 
0x2E00, end: 0x2E7F }, /*UBLOCK_SUPPLEMENTAL_PUNCTUATION=142*/
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread László Németh
 loleaflet/dist/toolbar/toolbar.js |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 9a1544041a000c5c8169db77581638b52cba2784
Author: László Németh 
Date:   Thu Dec 8 23:21:22 2016 +0100

tdf#106654 loleaflet: fix footnote insertion

Ctrl-Alt-F (insert footnote) activated the Search bar (Ctrl-F), too.

Change-Id: I681156f703bc044417c9b47aad8b3ebce5f259f6
Reviewed-on: https://gerrit.libreoffice.org/35457
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 7df233c0..7d5ab254 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -1320,7 +1320,7 @@ map.on('updatepermission', function (e) {
 });
 
 map.on('keydown', function (e) {
-   if (e.originalEvent.ctrlKey &&
+   if (e.originalEvent.ctrlKey && !e.originalEvent.altKey &&
   (e.originalEvent.key === 'f' || e.originalEvent.key === 'F')) {
L.DomUtil.get('search-input').focus();
e.originalEvent.preventDefault();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |   20 +---
 1 file changed, 5 insertions(+), 15 deletions(-)

New commits:
commit eedf7a8e004ce4bb08328fc782891e88bf49b185
Author: Pranav Kant 
Date:   Mon Dec 5 22:04:06 2016 +0530

tdf#106601 loleaflet: Disable calc toolbar items in readonly/view mode

Change-Id: I2989de5f201eb7f52f5f47d98f9172c640e38fad
Reviewed-on: https://gerrit.libreoffice.org/35361
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 159bcf58..7df233c0 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -528,7 +528,11 @@ var formatButtons = {
'fontcolor': true, 'backcolor': true, 'bullet': true, 'numbering': true,
'alignleft': true, 'alignhorizontal': true, 'alignright': true, 
'alignblock': true,
'incrementindent': true, 'decrementindent': true, 'insertgraphic': true,
-   'insertfootnote': true, 'repair': true, 'specialcharacter': true
+   'insertfootnote': true, 'repair': true, 'specialcharacter': true,
+   'wraptext': true, 'togglemergecells': true, 'numberformatcurrency': 
true,
+   'numberformatpercent': true, 'numberformatdecimal': true, 
'numberformatdate': true,
+   'numberformatincdecimals': true, 'numberformatdecdecimals': true,
+   'sortascending': true, 'sortdescending': true
 };
 
 var userJoinedPopupMessage = '' + _('%user has joined') + '';
@@ -1011,20 +1015,6 @@ map.on('commandstatechanged', function (e) {
state = toLocalePattern('Slide %1 of %2', 'Slide (\\d+) of 
(\\d+)', state, '%1', '%2');
$('#PageStatus').html(state ? state : 
' ');
}
-   else if (commandName === '.uno:WrapText' ||
-commandName === '.uno:ToggleMergeCells' ||
-commandName === '.uno:NumberFormatCurrency' ||
-commandName === '.uno:NumberFormatDecimal' ||
-commandName === '.uno:NumberFormatPercent' ||
-commandName === '.uno:NumberFormatDate') {
-   toggleButton(toolbar, state, commandName);
-   toggleButton(toolbarUpMore, state, commandName);
-   }
-   else if (commandName === '.uno:SortAscending' ||
-commandName === '.uno:SortDescending') {
-   disableButton(toolbar, state, commandName);
-   disableButton(toolbarUpMore, state, commandName);
-   }
 
var id = commandName.toLowerCase().substr(5);
if (typeof formatButtons[id] !== 'undefined') {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread László Németh
 loleaflet/src/layer/Popup.js  |   10 +++---
 loleaflet/src/layer/tile/TileLayer.js |   12 ++--
 2 files changed, 13 insertions(+), 9 deletions(-)

New commits:
commit 058836c16cbadeb818e8178dc3f735b4d4ff90d3
Author: László Németh 
Date:   Thu Dec 8 00:34:17 2016 +0100

loleaflet: better user tooltip layout in Calc and Impress

Now it is similar to Writer user tooltip coloring.

Change-Id: I68ae24145a199957c506491d38ff50472abcf1bb
Reviewed-on: https://gerrit.libreoffice.org/35366
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/layer/Popup.js b/loleaflet/src/layer/Popup.js
index 9e8f0cf7..5acc2962 100644
--- a/loleaflet/src/layer/Popup.js
+++ b/loleaflet/src/layer/Popup.js
@@ -168,10 +168,14 @@ L.Popup = L.Layer.extend({
this._tipContainer = L.DomUtil.create('div', prefix + 
'-tip-container', container);
this._tip = L.DomUtil.create('div', prefix + '-tip', 
this._tipContainer);
 
-   if (this.options.borderColor) {
-   this._tip.style['border'] = 
this._wrapper.style['border'] = '2px solid ' + this.options.borderColor;
-   this._closeButton.style['color'] = 
this.options.borderColor;
+   if (this.options.backgroundColor) {
+   this._tip.style['background-color'] = 
this._wrapper.style['background-color'] = this.options.backgroundColor;
}
+
+   if (this.options.color) {
+   this._wrapper.style['color'] = this.options.color;
+   }
+
},
 
_updateContent: function () {
diff --git a/loleaflet/src/layer/tile/TileLayer.js 
b/loleaflet/src/layer/tile/TileLayer.js
index c4618fb5..13d00342 100644
--- a/loleaflet/src/layer/tile/TileLayer.js
+++ b/loleaflet/src/layer/tile/TileLayer.js
@@ -718,10 +718,10 @@ L.TileLayer = L.GridLayer.extend({
 
if 
(!this._isEmptyRectangle(this._cellViewCursors[viewId].bounds) && 
this._selectedPart === viewPart) {
if (!cellViewCursorMarker) {
-   var borderColor = 
L.LOUtil.rgbToHex(this._map.getViewColor(viewId));
-   cellViewCursorMarker = 
L.rectangle(this._cellViewCursors[viewId].bounds, {fill: false, color: 
borderColor, weight: 2});
+   var backgroundColor = 
L.LOUtil.rgbToHex(this._map.getViewColor(viewId));
+   cellViewCursorMarker = 
L.rectangle(this._cellViewCursors[viewId].bounds, {fill: false, color: 
backgroundColor, weight: 2});
this._cellViewCursors[viewId].marker = 
cellViewCursorMarker;
-   
cellViewCursorMarker.bindPopup(this._map.getViewName(viewId), {autoClose: 
false, autoPan: false, borderColor: borderColor});
+   
cellViewCursorMarker.bindPopup(this._map.getViewName(viewId), {autoClose: 
false, autoPan: false, backgroundColor: backgroundColor, color: 'white', 
closeButton: false});
}
else {

cellViewCursorMarker.setBounds(this._cellViewCursors[viewId].bounds);
@@ -765,8 +765,8 @@ L.TileLayer = L.GridLayer.extend({
}
}
 
-   var borderColor = 
L.LOUtil.rgbToHex(this._map.getViewColor(viewId));
-   
this._cellViewCursors[viewId].marker.bindPopup(this._map.getViewName(viewId), 
{autoClose: false, autoPan: false, borderColor: borderColor});
+   var backgroundColor = 
L.LOUtil.rgbToHex(this._map.getViewColor(viewId));
+   
this._cellViewCursors[viewId].marker.bindPopup(this._map.getViewName(viewId), 
{autoClose: false, autoPan: false, backgroundColor: backgroundColor, color: 
'white', closeButton: false});
}
},
 
@@ -1430,7 +1430,7 @@ L.TileLayer = L.GridLayer.extend({
color: color
});
// Disable autoPan, so the graphic view 
selection doesn't make the view jump to the popup.
-   
viewMarker.bindPopup(this._map.getViewName(viewId), {autoClose: false, autoPan: 
false, borderColor: color});
+   
viewMarker.bindPopup(this._map.getViewName(viewId), {autoClose: false, autoPan: 
false, backgroundColor: color, color: 'white', closeButton: false});
this._graphicViewMarkers[viewId].marker = 
viewMarker;
}
else {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - kit/ChildSession.cpp

2017-03-26 Thread Ashod Nakashian
 kit/ChildSession.cpp |   22 --
 1 file changed, 20 insertions(+), 2 deletions(-)

New commits:
commit d559fce02bba5f2b96bb95778e99de1b396c8bce
Author: Ashod Nakashian 
Date:   Sun Dec 4 23:45:26 2016 -0500

tdf#106600 loolwsd: invalidate all parts when document size changes

This is a stop-gap until Core sends these invalidations
with more precision. Otherwise, we may be wasting valuable
cache data here.

This fixes showing incorrect tiles when slides are
inserted/removed from a doc.

Change-Id: I7a02c6fb96ff27223afe71d891db639ab5b2cfae
Reviewed-on: https://gerrit.libreoffice.org/35358
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/kit/ChildSession.cpp b/kit/ChildSession.cpp
index 56a63b3e..c34bead8 100644
--- a/kit/ChildSession.cpp
+++ b/kit/ChildSession.cpp
@@ -1084,8 +1084,26 @@ void ChildSession::loKitCallback(const int nType, const 
std::string& rPayload)
 sendTextFrame("searchresultselection: " + rPayload);
 break;
 case LOK_CALLBACK_DOCUMENT_SIZE_CHANGED:
-getStatus("", 0);
-getPartPageRectangles("", 0);
+{
+//TODO: clenaup and merge.
+
+std::unique_lock lock(_docManager.getDocumentMutex());
+const int parts = getLOKitDocument()->getParts();
+for (int i = 0; i < parts; ++i)
+{
+sendTextFrame("invalidatetiles:"
+  " part=" + std::to_string(i) +
+  " x=0" +
+  " y=0" +
+  " width=" + std::to_string(INT_MAX) +
+  " height=" + std::to_string(INT_MAX));
+}
+
+lock.unlock();
+
+getStatus("", 0);
+getPartPageRectangles("", 0);
+}
 break;
 case LOK_CALLBACK_SET_PART:
 sendTextFrame("setpart: " + rPayload);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - common/IoUtil.cpp

2017-03-26 Thread Tor Lillqvist
 common/IoUtil.cpp |7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 412c505430e9f61cfba11d4e4129198fccd13801
Author: Tor Lillqvist 
Date:   Fri Dec 2 16:51:11 2016 +0200

Handle return value -1 from our LOOLWebSocket::receiveFrame()

It doesn't mean anything is wrong or that the connection would be in
some invalid or closed state, but just that we didn't actually receive
any "interesting" frame that could be handled in the caller.

Change-Id: Iba98f054390ad8e04cdf72a342cf3d8ac08f78b7
Reviewed-on: https://gerrit.libreoffice.org/35352
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/common/IoUtil.cpp b/common/IoUtil.cpp
index 1250066f..04eca76a 100644
--- a/common/IoUtil.cpp
+++ b/common/IoUtil.cpp
@@ -90,7 +90,12 @@ void SocketProcessor(const std::shared_ptr& 
ws,
 continue;
 }
 
-if (n <= 0 || ((flags & WebSocket::FRAME_OP_BITMASK) == 
WebSocket::FRAME_OP_CLOSE))
+if (n == -1)
+{
+LOG_DBG("SocketProcessor [" << name << "]: was not an 
interesting frame, nothing to do here");
+continue;
+}
+else if (n == 0 || ((flags & WebSocket::FRAME_OP_BITMASK) == 
WebSocket::FRAME_OP_CLOSE))
 {
 LOG_WRN("SocketProcessor [" << name << "]: Connection 
closed.");
 closeFrame();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - 2 commits - loleaflet/src

2017-03-26 Thread Tomaž Vajngerl
 loleaflet/src/control/Control.Header.js |   23 +++--
 loleaflet/src/layer/tile/TileLayer.js   |   35 +---
 loleaflet/src/map/Map.js|   22 ++--
 3 files changed, 57 insertions(+), 23 deletions(-)

New commits:
commit dd3b68e0f6c66d3091ec13546274fbb72f9c5bce
Author: Tomaž Vajngerl 
Date:   Mon Dec 5 21:58:07 2016 +0100

tdf#106595 don't scroll to the cursor position after focus lose/get

Change-Id: Id017c5ea8c162a71a8b4d8ec47a6d9deb7357fca
Reviewed-on: https://gerrit.libreoffice.org/35354
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/layer/tile/TileLayer.js 
b/loleaflet/src/layer/tile/TileLayer.js
index d196f046..c4618fb5 100644
--- a/loleaflet/src/layer/tile/TileLayer.js
+++ b/loleaflet/src/layer/tile/TileLayer.js
@@ -1289,11 +1289,31 @@ L.TileLayer = L.GridLayer.extend({
}
}
 
-   if (this._map._permission === 'edit' && this._isCursorVisible 
&& this._isCursorOverlayVisible
-   && 
!this._isEmptyRectangle(this._visibleCursor)) {
+   this._updateCursorAndOverlay();
+
+   this.eachView(this._viewCursors, function (item) {
+   var viewCursorMarker = item.marker;
+   if (viewCursorMarker) {
+   
viewCursorMarker.setOpacity(this._map.hasLayer(this._cursorMarker) && 
this._cursorMarker.getLatLng().equals(viewCursorMarker.getLatLng()) ? 0 : 1);
+   }
+   }, this, true);
+   },
+
+   // enable or disable blinking cursor and  the cursor overlay depending 
on
+   // the state of the document (if the falgs are set)
+   _updateCursorAndOverlay: function (update) {
+   if (this._map._permission === 'edit'
+   && this._isCursorVisible
+   && this._isCursorOverlayVisible
+   && !this._isEmptyRectangle(this._visibleCursor)) {
+
+   var pixBounds = 
L.bounds(this._map.latLngToLayerPoint(this._visibleCursor.getSouthWest()),
+
this._map.latLngToLayerPoint(this._visibleCursor.getNorthEast()));
+
+   var cursorPos = this._visibleCursor.getNorthWest();
+
if (!this._cursorMarker) {
-   this._cursorMarker = L.cursor(cursorPos, 
pixBounds.getSize().multiplyBy(this._map.getZoomScale(this._map.getZoom())),
-   {blink: true});
+   this._cursorMarker = L.cursor(cursorPos, 
pixBounds.getSize().multiplyBy(this._map.getZoomScale(this._map.getZoom())), 
{blink: true});
}
else {
this._cursorMarker.setLatLng(cursorPos, 
pixBounds.getSize().multiplyBy(this._map.getZoomScale(this._map.getZoom(;
@@ -1304,13 +1324,6 @@ L.TileLayer = L.GridLayer.extend({
this._map.removeLayer(this._cursorMarker);
this._isCursorOverlayVisible = false;
}
-
-   this.eachView(this._viewCursors, function (item) {
-   var viewCursorMarker = item.marker;
-   if (viewCursorMarker) {
-   
viewCursorMarker.setOpacity(this._map.hasLayer(this._cursorMarker) && 
this._cursorMarker.getLatLng().equals(viewCursorMarker.getLatLng()) ? 0 : 1);
-   }
-   }, this, true);
},
 
// Update colored non-blinking view cursor
diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js
index 7e103244..bc1b2790 100644
--- a/loleaflet/src/map/Map.js
+++ b/loleaflet/src/map/Map.js
@@ -824,14 +824,16 @@ L.Map = L.Evented.extend({
if (!this._loaded) { return; }
 
var doclayer = this._docLayer;
-   if (!doclayer) {
-   return;
-   }
-   doclayer._isCursorVisibleOnLostFocus = 
doclayer._isCursorOverlayVisible;
+   if (!doclayer) { return; }
+
+   // save state of cursor (blinking marker) and the cursor overlay
+   doclayer._isCursorVisibleOnLostFocus = 
doclayer._isCursorVisible;
doclayer._isCursorOverlayVisibleOnLostFocus = 
doclayer._isCursorOverlayVisible;
+
+   // if the blinking cursor is visible, disable the overlay when 
we go out of focus
if (doclayer._isCursorVisible && 
doclayer._isCursorOverlayVisible) {
doclayer._isCursorOverlayVisible = false;
-   doclayer._onUpdateCursor();
+   doclayer._updateCursorAndOverlay();
}
 
this._deactivate();
@@ -841,15 +843,15 @@ L.Map = L.Evented.extend({
if (!this._loaded) { 

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Pranav Kant
 loleaflet/dist/toolbar/toolbar.js |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d5807b5536f94e8f367cf5671f90bc91cfd6a9c4
Author: Pranav Kant 
Date:   Mon Dec 5 15:07:27 2016 +0530

loleaflet: Disable zoom controls in spreadsheets, don't remove

Due to a bug in w2ui library, w2overlay doesn't popup when it is
near the edge of the screen (reproduceable only when no. of users
are greater than 3). Lets disable zoom controls in case of
spreadsheets so that userlist is not near the edge of the screen,
and hence pops up.

Change-Id: I6f2647d124484235afe6c92da18be38ed8e2b9ce
Reviewed-on: https://gerrit.libreoffice.org/35351
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 19b33ff6..159bcf58 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -791,7 +791,7 @@ map.on('doclayerinit', function () {
case 'spreadsheet':
toolbarUp.remove('inserttable', 'styles', 'alignblock', 
'bullet', 'numbering', 'break-numbering');
toolbarUpMore.remove('inserttable', 'styles', 'alignblock', 
'bullet', 'numbering', 'break-numbering');
-   statusbar.remove('zoomreset', 'zoomout', 'zoomin', 'zoomlevel');
+   statusbar.disable('zoomreset', 'zoomout', 'zoomin', 
'zoomlevel');
statusbar.insert('left', [
{type: 'break', id:'break1'},
{type: 'html',  id: 'StatusDocPos',
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/dist

2017-03-26 Thread Andras Timar
 loleaflet/dist/toolbar/toolbar.js |   87 --
 1 file changed, 38 insertions(+), 49 deletions(-)

New commits:
commit 215944457da123b55a386032b27e017783017147
Author: Andras Timar 
Date:   Sun Dec 4 22:40:44 2016 +0100

fix mangled calc toolbar

Change-Id: I59826d82d1096ad5b9383611421cd4fd81416a85
Reviewed-on: https://gerrit.libreoffice.org/35350
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index 7d76b8d7..19b33ff6 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -367,6 +367,18 @@ $(function () {
{type: 'button',  id: 'alignhorizontal',  img: 
'alignhorizontal', hint: _('Center horizontally'), uno: 'CenterPara', unosheet: 
'HorizontalAlignment {"HorizontalAlignment":{"type":"unsigned short", 
"value":"2"}}'},
{type: 'button',  id: 'alignright',  img: 'alignright', 
hint: _('Align right'), uno: 'RightPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"3"}}'},
{type: 'button',  id: 'alignblock',  img: 'alignblock', 
hint: _('Justified'), uno: 'JustifyPara', unosheet: 'HorizontalAlignment 
{"HorizontalAlignment":{"type":"unsigned short", "value":"4"}}'},
+   {type: 'button',  id: 'wraptext',  img: 'wraptext', 
hint: _('Wrap Text'), uno: 'WrapText'},
+   {type: 'button',  id: 'togglemergecells',  img: 
'togglemergecells', hint: _('Merge and Center Cells'), uno: 'ToggleMergeCells'},
+   {type: 'break',   id: 'break-toggle'},
+   {type: 'button',  id: 'numberformatcurrency',  img: 
'numberformatcurrency', hint: _('Format as Currency'), uno: 
'NumberFormatCurrency'},
+   {type: 'button',  id: 'numberformatpercent',  img: 
'numberformatpercent', hint: _('Format as Percent'), uno: 
'NumberFormatPercent'},
+   {type: 'button',  id: 'numberformatdecimal',  img: 
'numberformatdecimal', hint: _('Format as Number'), uno: 'NumberFormatDecimal'},
+   {type: 'button',  id: 'numberformatdate',  img: 
'numberformatdate', hint: _('Format as Date'), uno: 'NumberFormatDate'},
+   {type: 'button',  id: 'numberformatincdecimals',  img: 
'numberformatincdecimals', hint: _('Add Decimal Place'), uno: 
'NumberFormatIncDecimals'},
+   {type: 'button',  id: 'numberformatdecdecimals',  img: 
'numberformatdecdecimals', hint: _('Delete Decimal Place'), uno: 
'NumberFormatDecDecimals'},
+   {type: 'break',   id: 'break-number'},
+   {type: 'button',  id: 'sortascending',  img: 
'sortascending', hint: _('Sort Ascending'), uno: 'SortAscending'},
+   {type: 'button',  id: 'sortdescending',  img: 
'sortdescending', hint: _('Sort Descending'), uno: 'SortDescending'},
{type: 'break',   id: 'break-align'},
{type: 'button',  id: 'bullet',  img: 'bullet', hint: 
_('Bullets on/off'), uno: 'DefaultBullet'},
{type: 'button',  id: 'numbering',  img: 'numbering', 
hint: _('Numbering on/off'), uno: 'DefaultNumbering'},
@@ -526,7 +538,7 @@ var userPopupTimeout = null;
 function toggleButton(toolbar, state, command)
 {
var checked;
-   command = command.replace('.uno:', '');
+   command = command.replace('.uno:', '').toLowerCase();
var item = toolbar.get(command);
if (!item) {
return;
@@ -550,7 +562,7 @@ function toggleButton(toolbar, state, command)
 function disableButton(toolbar, state, command)
 {
var disabled;
-   command = command.replace('.uno:', '');
+   command = command.replace('.uno:', '').toLowerCase();
var item = toolbar.get(command);
if (!item) {
return;
@@ -770,37 +782,16 @@ map.on('wopiprops', function(e) {
 });
 
 map.on('doclayerinit', function () {
-   var toolbar = w2ui['toolbar-up'];
+   var toolbarUp = w2ui['toolbar-up'];
+   var toolbarUpMore = w2ui['toolbar-up-more'];
+   var statusbar = w2ui['toolbar-down'];
var docType = map.getDocType();
-   if (docType !== 'text') {
-   if (docType === 'presentation') {
-   toolbar.hide('annotation');
-
-   var presentationToolbar = w2ui['presentation-toolbar'];
-   presentationToolbar.show('presentation');
-   presentationToolbar.show('presentationbreak');
-   presentationToolbar.show('insertpage');
-   presentationToolbar.show('duplicatepage');
-   presentationToolbar.show('deletepage');
-   }
-   else if (docType === 'drawing') {
-   toolbar.hide('annotation');

[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Marco Cecchetti
 loleaflet/src/map/Map.js |9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

New commits:
commit 4b5c4a8f292cc2e6aee7bffd8848575daf2b3497
Author: Marco Cecchetti 
Date:   Wed Nov 30 16:40:01 2016 +0100

tdf#106594 loleaflet - calc: bad handling of text cursor on lost/got focus

This patch fixes the following bug:

Create blank sheet:
enter "." ie. just a period into A1
enter a "." into A2
now switch virtual desktop away and back again.
hit enter in A2
now switch virtual desktop away and back again.

You should have a blinking edit cursor in A2 - but the focus in A3.

Change-Id: Icda0123f35ecda04559fec859b6bbe6ff3ac33e8
Reviewed-on: https://gerrit.libreoffice.org/35349
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js
index 7ef14af7..7e103244 100644
--- a/loleaflet/src/map/Map.js
+++ b/loleaflet/src/map/Map.js
@@ -824,9 +824,12 @@ L.Map = L.Evented.extend({
if (!this._loaded) { return; }
 
var doclayer = this._docLayer;
-   if (doclayer && doclayer._isCursorVisible && 
doclayer._isCursorOverlayVisible) {
-   doclayer._visibleCursorOnLostFocus = 
doclayer._visibleCursor;
-   doclayer._isCursorOverlayVisibleOnLostFocus = 
doclayer._isCursorVisibleOnLostFocus = true;
+   if (!doclayer) {
+   return;
+   }
+   doclayer._isCursorVisibleOnLostFocus = 
doclayer._isCursorOverlayVisible;
+   doclayer._isCursorOverlayVisibleOnLostFocus = 
doclayer._isCursorOverlayVisible;
+   if (doclayer._isCursorVisible && 
doclayer._isCursorOverlayVisible) {
doclayer._isCursorOverlayVisible = false;
doclayer._onUpdateCursor();
}
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Marco Cecchetti
 loleaflet/src/core/Util.js |9 +
 loleaflet/src/map/Map.js   |8 +++-
 2 files changed, 16 insertions(+), 1 deletion(-)

New commits:
commit bbff6b0045e05507e73d8d3566fa0ff95e04a0cb
Author: Marco Cecchetti 
Date:   Wed Nov 30 23:04:48 2016 +0100

loleaflet - calc: fixed one pixel horizontal auto-scrolling issue

This patch fixes two problems for spreadsheets:

- one pixel alignment offset btw grid and column header

- a one pixel horizontal auto-scrolling issue

Change-Id: Ifd6a3b47863d345656d0dcf3fba2d253c43ba9b1
Reviewed-on: https://gerrit.libreoffice.org/35348
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/core/Util.js b/loleaflet/src/core/Util.js
index 3637688a..5cb3ade1 100644
--- a/loleaflet/src/core/Util.js
+++ b/loleaflet/src/core/Util.js
@@ -126,6 +126,14 @@ L.Util = {
return ((!existingUrl || existingUrl.indexOf('?') === -1) ? '?' 
: '&') + params.join('&');
},
 
+   round: function(x, e) {
+   if (!e) {
+   return Math.round(x);
+   }
+   var f = 1.0/e;
+   return Math.round(x * f) * e;
+   },
+
// super-simple templating facility, used for TileLayer URLs
template: function (str, data) {
return str.replace(L.Util.templateRe, function (str, key) {
@@ -194,3 +202,4 @@ L.extend = L.Util.extend;
 L.bind = L.Util.bind;
 L.stamp = L.Util.stamp;
 L.setOptions = L.Util.setOptions;
+L.round = L.Util.round;
diff --git a/loleaflet/src/map/Map.js b/loleaflet/src/map/Map.js
index 28455165..7ef14af7 100644
--- a/loleaflet/src/map/Map.js
+++ b/loleaflet/src/map/Map.js
@@ -507,7 +507,8 @@ L.Map = L.Evented.extend({
 
project: function (latlng, zoom) { // (LatLng[, Number]) -> Point
zoom = zoom === undefined ? this._zoom : zoom;
-   return this.options.crs.latLngToPoint(L.latLng(latlng), zoom);
+   var projectedPoint = 
this.options.crs.latLngToPoint(L.latLng(latlng), zoom);
+   return new L.Point(L.round(projectedPoint.x, 1e-6), 
L.round(projectedPoint.y, 1e-6));
},
 
unproject: function (point, zoom) { // (Point[, Number]) -> LatLng
@@ -1054,6 +1055,11 @@ L.Map = L.Evented.extend({
return left + right > 0 ?
Math.round(left - right) / 2 :
Math.max(0, Math.ceil(left)) - Math.max(0, 
Math.floor(right));
+   // TODO: do we really need ceil and floor ?
+   // for spreadsheets it can cause one pixel alignment offset btw 
grid and row/column header
+   // and a one pixel horizontal auto-scrolling issue;
+   // both issues have been fixed by rounding the projection: see 
Map.project above;
+   // anyway in case of similar problems, this code needs to be 
checked
},
 
_limitZoom: function (zoom) {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Marco Cecchetti
 loleaflet/src/control/Control.Scroll.js |   15 +--
 1 file changed, 13 insertions(+), 2 deletions(-)

New commits:
commit 458045bb43a31b02443459e053c18ed95d134d7a
Author: Marco Cecchetti 
Date:   Thu Dec 1 21:00:37 2016 +0100

tdf#106597 loleaflet - calc: horizontal scrollbar not updated on doc width 
change

Change-Id: I268ab65d0bf7a6794028e23e51012c11534d6f2f
Reviewed-on: https://gerrit.libreoffice.org/35347
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/control/Control.Scroll.js 
b/loleaflet/src/control/Control.Scroll.js
index 31314408..40089a99 100644
--- a/loleaflet/src/control/Control.Scroll.js
+++ b/loleaflet/src/control/Control.Scroll.js
@@ -118,14 +118,25 @@ L.Control.Scroll = L.Control.extend({
},
 
_onUpdateSize: function (e) {
+   // we need to avoid precision issues in comparison (in the end 
values are pixels)
+   var prevDocWidth = 
Math.ceil(parseFloat(L.DomUtil.getStyle(this._mockDoc, 'width')));
+   var prevDocHeight = 
Math.ceil(parseFloat(L.DomUtil.getStyle(this._mockDoc, 'height')));
+   var newDocWidth = Math.ceil(e.x);
+   var newDocHeight = Math.ceil(e.y);
// for writer documents, ignore scroll while document size is 
being reduced
-   var prevDocY = parseFloat(L.DomUtil.getStyle(this._mockDoc, 
'height'));
-   if (this._map.getDocType() === 'text' && e.y < prevDocY) {
+   if (this._map.getDocType() === 'text' && newDocHeight < 
prevDocHeight) {
this._ignoreScroll = true;
}
 
L.DomUtil.setStyle(this._mockDoc, 'width', e.x + 'px');
L.DomUtil.setStyle(this._mockDoc, 'height', e.y + 'px');
+
+   // custom scrollbar plugin checks automatically for content 
height changes but not for content width changes
+   // so we need to update scrollbars explicitly; moreover we want 
to avoid to have 'update' invoked twice
+   // in case prevDocHeight !== newDocHeight
+   if (prevDocWidth !== newDocWidth && prevDocHeight === 
newDocHeight) {
+   $('.scroll-container').mCustomScrollbar('update');
+   }
},
 
_onUpdateScrollOffset: function (e) {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/node_shrinkwrap

2017-03-26 Thread László Németh
 loleaflet/node_shrinkwrap/l10n-for-node-0.0.1.tgz |binary
 1 file changed

New commits:
commit 5bc5fedb20909bcdfed1fe44904fc392404b3bdb
Author: László Németh 
Date:   Fri Dec 2 09:17:36 2016 +0100

tdf#106593 loleaflet: patch l10n-for-node for older Chrome versions

Online didn't work in Chrome 39 (2014-11). Fixed in
the l10n-for-node shrinkwrap archive (see the patch file in it).

Change-Id: Ia801a8e3decca760cfcabb46aeb3b117f4005337
Reviewed-on: https://gerrit.libreoffice.org/35346
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/node_shrinkwrap/l10n-for-node-0.0.1.tgz 
b/loleaflet/node_shrinkwrap/l10n-for-node-0.0.1.tgz
index 6834ccbc..476f6044 100644
Binary files a/loleaflet/node_shrinkwrap/l10n-for-node-0.0.1.tgz and 
b/loleaflet/node_shrinkwrap/l10n-for-node-0.0.1.tgz differ
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/LOOLWSD.cpp

2017-03-26 Thread Ashod Nakashian
 wsd/LOOLWSD.cpp |   23 +++
 1 file changed, 11 insertions(+), 12 deletions(-)

New commits:
commit 94f0663a4513f74421d87a16edd783f9ac9f55c1
Author: Ashod Nakashian 
Date:   Thu Dec 1 22:04:43 2016 -0500

loolwsd: guarantee DocBrokersMutex locked when alerting

We now guarantee that forkChildren will be invoked
under DocBrokersMutex lock.

This eliminates the case when alertAllUsersInternal
is invoked when this mutex isn't locked.

Change-Id: Ibb259bbb4f380300a90ad2fc7affe6013dd71fef
Reviewed-on: https://gerrit.libreoffice.org/35345
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 673758cc..11b37694 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -270,6 +270,7 @@ bool cleanupDocBrokers()
 
 static void forkChildren(const int number)
 {
+Util::assertIsLocked(DocBrokersMutex);
 Util::assertIsLocked(NewChildrenMutex);
 
 if (number > 0)
@@ -332,12 +333,8 @@ static void preForkChildren()
 static void prespawnChildren()
 {
 // First remove dead DocBrokers, if possible.
-std::unique_lock docBrokersLock(DocBrokersMutex, 
std::defer_lock);
-if (docBrokersLock.try_lock())
-{
-cleanupDocBrokers();
-docBrokersLock.unlock();
-}
+std::unique_lock docBrokersLock(DocBrokersMutex);
+cleanupDocBrokers();
 
 std::unique_lock lock(NewChildrenMutex, std::defer_lock);
 if (!lock.try_lock())
@@ -384,6 +381,7 @@ static size_t addNewChild(const 
std::shared_ptr& child)
 
 static std::shared_ptr getNewChild()
 {
+Util::assertIsLocked(DocBrokersMutex);
 std::unique_lock lock(NewChildrenMutex);
 
 namespace chrono = std::chrono;
@@ -518,6 +516,13 @@ private:
 {
 LOG_INF("Conversion request for URI [" << fromPath << 
"].");
 
+auto uriPublic = DocumentBroker::sanitizeURI(fromPath);
+const auto docKey = DocumentBroker::getDocKey(uriPublic);
+
+// This lock could become a bottleneck.
+// In that case, we can use a pool and index by publicPath.
+std::unique_lock 
docBrokersLock(DocBrokersMutex);
+
 // Request a kit process for this doc.
 auto child = getNewChild();
 if (!child)
@@ -526,16 +531,10 @@ private:
 throw std::runtime_error("Failed to spawn lokit 
child.");
 }
 
-auto uriPublic = DocumentBroker::sanitizeURI(fromPath);
-const auto docKey = DocumentBroker::getDocKey(uriPublic);
 LOG_DBG("New DocumentBroker for docKey [" << docKey << 
"].");
 auto docBroker = 
std::make_shared(uriPublic, docKey, LOOLWSD::ChildRoot, child);
 child->setDocumentBroker(docBroker);
 
-// This lock could become a bottleneck.
-// In that case, we can use a pool and index by publicPath.
-std::unique_lock 
docBrokersLock(DocBrokersMutex);
-
 cleanupDocBrokers();
 
 // FIXME: What if the same document is already open? Need 
a fake dockey here?
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/src/control/Control.CharacterMap.js |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit a949769c2e4cc3062ffd254e9548bdcbcc0444e6
Author: Henry Castro 
Date:   Thu Dec 1 14:29:58 2016 -0400

tdf#106592 loleaflet: remove invalid fonts

Change-Id: Ie0383d01f6040d00c630c0aab6b9a6d079d3
Reviewed-on: https://gerrit.libreoffice.org/35344
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/loleaflet/src/control/Control.CharacterMap.js 
b/loleaflet/src/control/Control.CharacterMap.js
index 550fff64..c6885a39 100644
--- a/loleaflet/src/control/Control.CharacterMap.js
+++ b/loleaflet/src/control/Control.CharacterMap.js
@@ -422,6 +422,9 @@ L.Control.CharacterMap = L.Control.extend({
option.innerHTML = 
this.unicodeBlocks[list[iterator]].name;
}, this);
this._onUnicodeSubsetChange();
+   } else {
+   
L.DomUtil.remove(this._fontNames.options[this._fontNames.selectedIndex]);
+   this._onFontNamesChange();
}
},
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'libreoffice-5-3' - wsd/PrisonerSession.cpp

2017-03-26 Thread Henry Castro
 wsd/PrisonerSession.cpp |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 994fbbfc9669973015b1e0409df982b86a54cd8c
Author: Henry Castro 
Date:   Sun Nov 27 22:39:50 2016 -0400

tdf#106274 wsd: fix sending wrong UTF-8 string to the client

Client side console error "ws stopped cannot read
utf8 string", the cause it is sending special character '{',
'}',

Conflicts:
wsd/PrisonerSession.cpp

Change-Id: Ide757ed5206a7674b0004f80d1f28847d770209a
Reviewed-on: https://gerrit.libreoffice.org/34789
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/wsd/PrisonerSession.cpp b/wsd/PrisonerSession.cpp
index faf8232b..93611f7e 100644
--- a/wsd/PrisonerSession.cpp
+++ b/wsd/PrisonerSession.cpp
@@ -243,6 +243,8 @@ bool PrisonerSession::_handleInput(const char *buffer, int 
length)
 Poco::URI::encode(text, "", encodedChar);
 assert(firstLine.size() < 
static_cast(length));
 _docBroker->tileCache().saveRendering(font+encodedChar, "font", 
buffer + firstLine.size() + 1, length - firstLine.size() - 1);
+forwardToPeer(_peer, buffer, length, true);
+return true;
 }
 }
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sc/qa

2017-03-26 Thread Markus Mohrhard
 sc/qa/uitest/autofilter/autofilter.py |   22 ++
 1 file changed, 22 insertions(+)

New commits:
commit 23282b476f094c735ecfae4f82798e36ac9cbbc9
Author: Markus Mohrhard 
Date:   Sun Mar 26 23:10:45 2017 +0200

add test for multiple row filtering

Change-Id: I55c898a7bf6796802ee6d04071801fec3fcf218d
Reviewed-on: https://gerrit.libreoffice.org/35732
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/qa/uitest/autofilter/autofilter.py 
b/sc/qa/uitest/autofilter/autofilter.py
index ece49657d0c9..57504e66e8e9 100644
--- a/sc/qa/uitest/autofilter/autofilter.py
+++ b/sc/qa/uitest/autofilter/autofilter.py
@@ -48,4 +48,26 @@ class AutofilterTest(UITestCase):
 
 self.ui_test.close_doc()
 
+def test_filter_multiple_rows(self):
+doc = self.ui_test.load_file(get_url_for_data_file("autofilter.ods"))
+
+xGridWin = self.xUITest.getTopFocusWindow().getChild("grid_window")
+xGridWin.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", 
"COL": "2", "ROW": "0"}))
+xFloatWindow = self.xUITest.getFloatWindow()
+xCheckListMenu = xFloatWindow.getChild("check_list_menu")
+
+xTreeList = xCheckListMenu.getChild("check_list_box")
+xFirstEntry = xTreeList.getChild("1")
+
+xFirstEntry.executeAction("CLICK", tuple())
+
+xOkBtn = xFloatWindow.getChild("ok")
+xOkBtn.executeAction("CLICK", tuple())
+
+self.assertFalse(is_row_hidden(doc, 0))
+self.assertFalse(is_row_hidden(doc, 1))
+self.assertTrue(is_row_hidden(doc, 2))
+self.assertTrue(is_row_hidden(doc, 3))
+self.assertFalse(is_row_hidden(doc, 4))
+
 # vim: set shiftwidth=4 softtabstop=4 expandtab:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 2 commits - sc/Module_sc.mk sc/qa sc/source sc/UITest_autofilter.mk

2017-03-26 Thread Markus Mohrhard
 sc/Module_sc.mk |1 
 sc/UITest_autofilter.mk |   15 
 sc/qa/uitest/autofilter/autofilter.py   |   51 
 sc/qa/uitest/autofilter/data/autofilter.ods |binary
 sc/source/ui/cctrl/checklistmenu.cxx|   10 +++--
 sc/source/ui/inc/checklistmenu.hxx  |   13 ++-
 sc/source/ui/view/gridwin.cxx   |   24 +
 sc/source/ui/view/gridwin2.cxx  |   14 +++
 8 files changed, 102 insertions(+), 26 deletions(-)

New commits:
commit 5d420856d432ff178560b2be37d33c6ddc439397
Author: Markus Mohrhard 
Date:   Sun Mar 26 23:00:24 2017 +0200

add test for tdf#106214

Change-Id: I9998aa40c3831a6033d4d61a6eab90d639ec5aa9
Reviewed-on: https://gerrit.libreoffice.org/35731
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/Module_sc.mk b/sc/Module_sc.mk
index 31c1a58b7d81..4659f55ebfeb 100644
--- a/sc/Module_sc.mk
+++ b/sc/Module_sc.mk
@@ -141,6 +141,7 @@ $(eval $(call gb_Module_add_uicheck_targets,sc,\
UITest_conditional_format \
UITest_range_name \
UITest_hide_cols \
+   UITest_autofilter \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/sc/UITest_autofilter.mk b/sc/UITest_autofilter.mk
new file mode 100644
index ..93bf17200bb1
--- /dev/null
+++ b/sc/UITest_autofilter.mk
@@ -0,0 +1,15 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call gb_UITest_UITest,autofilter))
+
+$(eval $(call gb_UITest_add_modules,autofilter,$(SRCDIR)/sc/qa/uitest,\
+   autofilter/ \
+))
+# vim: set noet sw=4 ts=4:
diff --git a/sc/qa/uitest/autofilter/autofilter.py 
b/sc/qa/uitest/autofilter/autofilter.py
new file mode 100644
index ..ece49657d0c9
--- /dev/null
+++ b/sc/qa/uitest/autofilter/autofilter.py
@@ -0,0 +1,51 @@
+# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-offset: 4 -*-
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+from uitest.framework import UITestCase
+from uitest.path import get_srcdir_url
+
+from libreoffice.uno.propertyvalue import mkPropertyValues
+from libreoffice.calc.document import get_row
+
+import time
+
+def get_url_for_data_file(file_name):
+return get_srcdir_url() + "/sc/qa/uitest/autofilter/data/" + file_name
+
+def is_row_hidden(doc, index):
+row = get_row(doc, index)
+val = row.getPropertyValue("IsVisible")
+return not val
+
+class AutofilterTest(UITestCase):
+
+def test_tdf106214(self):
+doc = self.ui_test.load_file(get_url_for_data_file("autofilter.ods"))
+
+xGridWin = self.xUITest.getTopFocusWindow().getChild("grid_window")
+xGridWin.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", 
"COL": "0", "ROW": "0"}))
+
+xFloatWindow = self.xUITest.getFloatWindow()
+xCheckListMenu = xFloatWindow.getChild("check_list_menu")
+
+xTreeList = xCheckListMenu.getChild("check_list_box")
+xFirstEntry = xTreeList.getChild("0")
+
+xFirstEntry.executeAction("CLICK", tuple())
+
+xOkBtn = xFloatWindow.getChild("ok")
+xOkBtn.executeAction("CLICK", tuple())
+
+self.assertFalse(is_row_hidden(doc, 0))
+self.assertFalse(is_row_hidden(doc, 1))
+self.assertFalse(is_row_hidden(doc, 2))
+self.assertTrue(is_row_hidden(doc, 3))
+self.assertFalse(is_row_hidden(doc, 4))
+
+self.ui_test.close_doc()
+
+# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/qa/uitest/autofilter/data/autofilter.ods 
b/sc/qa/uitest/autofilter/data/autofilter.ods
new file mode 100644
index ..49285933cec3
Binary files /dev/null and b/sc/qa/uitest/autofilter/data/autofilter.ods differ
commit b3d498755238cb0d9a7a3e33b6070c1e4c0e3482
Author: Markus Mohrhard 
Date:   Sun Mar 26 22:50:18 2017 +0200

handle date autofilter entries correctly, tdf#106214

Change-Id: I8fd2896c8998e79794a0ccaae1c2442caf8b89ac
Reviewed-on: https://gerrit.libreoffice.org/35730
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/sc/source/ui/cctrl/checklistmenu.cxx 
b/sc/source/ui/cctrl/checklistmenu.cxx
index ce637818191c..dba72366b4aa 100644
--- a/sc/source/ui/cctrl/checklistmenu.cxx
+++ b/sc/source/ui/cctrl/checklistmenu.cxx
@@ -1916,12 +1916,14 @@ void ScCheckListMenuWindow::getResult(ResultType& 
rResult)
 if (aLabel.isEmpty())
 aLabel = ScGlobal::GetRscString(STR_EMPTYDATA);
 bool bState =  maChecks->IsChecked( aLabel,  maMembers[i].mpParent 
);
-OUStri

[Libreoffice-commits] core.git: uitest/libreoffice

2017-03-26 Thread Markus Mohrhard
 uitest/libreoffice/calc/document.py |   11 +++
 1 file changed, 11 insertions(+)

New commits:
commit d6861b8f38db9dc465f344474943b9477dd22a0c
Author: Markus Mohrhard 
Date:   Sun Mar 26 22:48:26 2017 +0200

uitest: add method to get the row object from a calc document

Change-Id: If24a995fe1e174d3ffc531021b290a455ae28512
Reviewed-on: https://gerrit.libreoffice.org/35729
Tested-by: Jenkins 
Reviewed-by: Markus Mohrhard 

diff --git a/uitest/libreoffice/calc/document.py 
b/uitest/libreoffice/calc/document.py
index a251f29b2490..9f722839d43e 100644
--- a/uitest/libreoffice/calc/document.py
+++ b/uitest/libreoffice/calc/document.py
@@ -37,4 +37,15 @@ def get_column(document, column, tab = 0):
 sheet = get_sheet_from_doc(document, tab)
 return sheet.getColumns().getByIndex(column)
 
+def get_row(document, row, tab = 0):
+""" Get the row object through the row index
+
+Keyword arguments:
+document -- The document that should be used
+tab -- The 0-based sheet number
+column -- The 0-based row number
+"""
+sheet = get_sheet_from_doc(document, tab)
+return sheet.getRows().getByIndex(row)
+
 # vim: set shiftwidth=4 softtabstop=4 expandtab:
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Andreas Säger license statement

2017-03-26 Thread Andreas Säger
All of my past & future contributions to LibreOffice may be licensed
under the MPLv2/LGPLv3+ dual license.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: sc/qa

2017-03-26 Thread Markus Mohrhard
 sc/qa/uitest/conditional_format/tdf105466.py |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit ff387f848d1695535ad33de6eef2577bcd91c9d0
Author: Markus Mohrhard 
Date:   Mon Mar 27 00:08:04 2017 +0200

improve the test by introducing sub tests

Change-Id: I414da64f55348973a8f60c7518d26a835385125d

diff --git a/sc/qa/uitest/conditional_format/tdf105466.py 
b/sc/qa/uitest/conditional_format/tdf105466.py
index e1cfd2d0a27e..fb0a872702b9 100644
--- a/sc/qa/uitest/conditional_format/tdf105466.py
+++ b/sc/qa/uitest/conditional_format/tdf105466.py
@@ -17,9 +17,10 @@ class tdf105466(UITestCase):
 
self.ui_test.execute_modeless_dialog_through_command(".uno:ConditionalFormatDialog")
 
 for i in range(0,4):
-xCondFormatDlg = self.xUITest.getTopFocusWindow()
-xTypeLstBox = xCondFormatDlg.getChild("type")
-xTypeLstBox.executeAction("SELECT", mkPropertyValues({"POS": 
str(i)}))
+with self.subTest(i = i):
+xCondFormatDlg = self.xUITest.getTopFocusWindow()
+xTypeLstBox = xCondFormatDlg.getChild("type")
+xTypeLstBox.executeAction("SELECT", mkPropertyValues({"POS": 
str(i)}))
 
 xCondFormatDlg = self.xUITest.getTopFocusWindow()
 xOkBtn = xCondFormatDlg.getChild("ok")
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: System theme or branded theme

2017-03-26 Thread Heiko Tietze
Thanks for your reply. I forwarded it to the ticket.

On 03/26/2017 10:52 PM, Caolán McNamara wrote:
> On Fri, 2017-03-24 at 22:15 +0100, Heiko Tietze wrote:
>> In tdf#106756 [1] the fundamental idea to blend LibreOffice well into
>> the system theme is challenged. While we may be able to solve most of
>> the issues the idea is tempting to have a unique look and feel with
>> the possibility to customize the default according the system. 
> 
> I very much disagree, I want the gtk3 LibreOffice to blend in with the
> system gtk3 theme. And in some places we are using the native gtk3
> stuff directly so it's out of our hands already, e.g. menubar, menus,
> file picker, tooltips, popovers, whether modal dialogs are locked in
> position or not. I'd like to go further there and use more native
> widgets rather than the current scheme of using our own widgets and
> using the gtk theming apis to render them.
> 
> I think the specific bug should be split up into the pieces that are
> causing problems and see if they are currently been taken from the
> wrong parts of the system theme and can be individually improved.
> ___
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice
> 

-- 
Dr. Heiko Tietze
UX Designer
Tel. +49 (0)179/1268509



signature.asc
Description: OpenPGP digital signature
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: System theme or branded theme

2017-03-26 Thread Caolán McNamara
On Fri, 2017-03-24 at 22:15 +0100, Heiko Tietze wrote:
> In tdf#106756 [1] the fundamental idea to blend LibreOffice well into
> the system theme is challenged. While we may be able to solve most of
> the issues the idea is tempting to have a unique look and feel with
> the possibility to customize the default according the system. 

I very much disagree, I want the gtk3 LibreOffice to blend in with the
system gtk3 theme. And in some places we are using the native gtk3
stuff directly so it's out of our hands already, e.g. menubar, menus,
file picker, tooltips, popovers, whether modal dialogs are locked in
position or not. I'd like to go further there and use more native
widgets rather than the current scheme of using our own widgets and
using the gtk theming apis to render them.

I think the specific bug should be split up into the pieces that are
causing problems and see if they are currently been taken from the
wrong parts of the system theme and can be individually improved.
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: android/source apple_remote/source avmedia/source basctl/source basegfx/source basic/source bean/com bin/parse-perfcheck.py bridges/source

2017-03-26 Thread Andrea Gelmini
 
android/source/src/java/org/libreoffice/storage/external/DirectoryBrowserActivity.java
 |2 +-
 apple_remote/source/RemoteControl.m
|2 +-
 avmedia/source/gstreamer/gstplayer.cxx 
|2 +-
 basctl/source/basicide/baside2b.cxx
|2 +-
 basctl/source/basicide/localizationmgr.cxx 
|2 +-
 basegfx/source/matrix/b3dhommatrix.cxx 
|2 +-
 basic/source/classes/eventatt.cxx  
|2 +-
 basic/source/comp/exprtree.cxx 
|2 +-
 basic/source/runtime/methods.cxx   
|2 +-
 basic/source/sbx/sbxexec.cxx   
|2 +-
 bean/com/sun/star/comp/beans/LocalOfficeConnection.java
|2 +-
 bean/com/sun/star/comp/beans/OOoBean.java  
|2 +-
 bin/parse-perfcheck.py 
|2 +-
 bridges/source/cpp_uno/msvc_win32_x86-64/mscx.hxx  
|2 +-
 14 files changed, 14 insertions(+), 14 deletions(-)

New commits:
commit 5ad636acc61eabd8123bea5b203e2c49a80f9ce3
Author: Andrea Gelmini 
Date:   Fri Mar 24 08:43:21 2017 +0100

Fix typos

Change-Id: Idbf0aa261b829e83d4c08a2e989a4ced871dc6de
Reviewed-on: https://gerrit.libreoffice.org/35616
Tested-by: Jenkins 
Reviewed-by: Julien Nabet 
Tested-by: Julien Nabet 

diff --git 
a/android/source/src/java/org/libreoffice/storage/external/DirectoryBrowserActivity.java
 
b/android/source/src/java/org/libreoffice/storage/external/DirectoryBrowserActivity.java
index 224526adb17b..1cf9f52fa7c0 100644
--- 
a/android/source/src/java/org/libreoffice/storage/external/DirectoryBrowserActivity.java
+++ 
b/android/source/src/java/org/libreoffice/storage/external/DirectoryBrowserActivity.java
@@ -14,7 +14,7 @@ import org.libreoffice.R;
  * Container for DirectoryBrowserFragment
  */
 public class DirectoryBrowserActivity extends AppCompatActivity {
-public static final String DIRECTORY_PATH_EXTRA = 
"org.libreoffie.directory_path_extra";
+public static final String DIRECTORY_PATH_EXTRA = 
"org.libreoffice.directory_path_extra";
 
 @Override
 protected void onCreate(@Nullable Bundle savedInstanceState) {
diff --git a/apple_remote/source/RemoteControl.m 
b/apple_remote/source/RemoteControl.m
index d43597d96c7d..1f1b7f179350 100644
--- a/apple_remote/source/RemoteControl.m
+++ b/apple_remote/source/RemoteControl.m
@@ -31,7 +31,7 @@
 
 #import 
 
-// notifaction names that are being used to signal that an application wants to
+// notification names that are being used to signal that an application wants 
to
 // have access to the remote control device or if the application has finished
 // using the remote control device
 NSString* REQUEST_FOR_REMOTE_CONTROL_NOTIFCATION = 
@"mac.remotecontrols.RequestForRemoteControl";
diff --git a/avmedia/source/gstreamer/gstplayer.cxx 
b/avmedia/source/gstreamer/gstplayer.cxx
index 50fa9b4cb1dc..26b712441692 100644
--- a/avmedia/source/gstreamer/gstplayer.cxx
+++ b/avmedia/source/gstreamer/gstplayer.cxx
@@ -614,7 +614,7 @@ void Player::preparePlaybin( const OUString& rURL, 
GstElement *pSink )
 mpPlaybin = gst_element_factory_make( "playbin", nullptr );
 
 //tdf#96989 on systems with flat-volumes setting the volume directly on the
-//playbin to 100% results in setting the global volums to 100% of the
+//playbin to 100% results in setting the global volume to 100% of the
 //maximum. We expect to set as % of the current volume.
 mpVolumeControl = gst_element_factory_make( "volume", nullptr );
 GstElement *pAudioSink = gst_element_factory_make( "autoaudiosink", 
nullptr );
diff --git a/basctl/source/basicide/baside2b.cxx 
b/basctl/source/basicide/baside2b.cxx
index c02bac26d0f2..d817a61c8eae 100644
--- a/basctl/source/basicide/baside2b.cxx
+++ b/basctl/source/basicide/baside2b.cxx
@@ -2413,7 +2413,7 @@ void WatchTreeListBox::UpdateWatches( bool bBasicStopped )
 SbxDataType eType = pVar->GetType();
 if ( eType & SbxARRAY )
 {
-// consider multidimensinal arrays!
+// consider multidimensional arrays!
 if (SbxDimArray* pNewArray = 
dynamic_cast(pVar->GetObject()))
 {
 SbxDimArray* pOldArray = pItem->mpArray.get();
diff --git a/basctl/source/basicide/localizationmgr.cxx 
b/basctl/source/basicide/localizationmgr.cxx
index b03412614b71..bfca283dceb3 100644
--- a/basctl/source/basicide/localizationmgr.cxx
+++ b/basctl/

Re: master builds, starts but no UI

2017-03-26 Thread Caolán McNamara
On Sun, 2017-03-26 at 19:14 +0200, Arnaud Versini wrote:
> The patch is here : https://gerrit.libreoffice.org/#/c/35720/

yeah, resolved now
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: 2 commits - lotuswordpro/source

2017-03-26 Thread Caolán McNamara
 lotuswordpro/source/filter/lwpframelayout.cxx |7 ++-
 lotuswordpro/source/filter/lwpframelayout.hxx |1 +
 lotuswordpro/source/filter/lwpuidoc.cxx   |   12 +---
 3 files changed, 12 insertions(+), 8 deletions(-)

New commits:
commit 19882e0c7daeaa16cbdbca2838064d4ad8762649
Author: Caolán McNamara 
Date:   Sun Mar 26 21:23:44 2017 +0100

lwp: convert hopeful asserts to exceptions

Change-Id: I3111929ea84aa8280607811c851d033156c65731

diff --git a/lotuswordpro/source/filter/lwpuidoc.cxx 
b/lotuswordpro/source/filter/lwpuidoc.cxx
index 1f45c1da0544..8e26b0daea37 100644
--- a/lotuswordpro/source/filter/lwpuidoc.cxx
+++ b/lotuswordpro/source/filter/lwpuidoc.cxx
@@ -92,11 +92,9 @@ void LwpNamedProperties::Read(LwpObjectStream *pStrm)
 {
 sal_uInt16 numEntries = pStrm->QuickReaduInt16();
 
-for (sal_uInt16 k = 0 ; k < numEntries; k++)
-{
-assert(false);
-// TODO: Read each NamedProperties
-}
+if (numEntries)
+throw std::runtime_error("TODO: Read each NamedProperties");
+
 pStrm->SkipExtra();
 }
 /**
@@ -121,8 +119,8 @@ void LwpMergeOptions::Read(LwpObjectStream *pStrm)
 m_nType = pStrm->QuickReaduInt16();
 
 //Does not process m_nType here. Assume m_nType is 0.
-// TODO: Read the CMergeDataFile
-assert(m_nType==0);
+if (m_nType != 0)
+throw std::runtime_error("TODO: Read the CMergeDataFile");
 
 m_nLastActionFlag = pStrm->QuickReaduInt16();
 pStrm->SkipExtra();
commit 7eb92eea21e2af47b17c9901107f6e4833567928
Author: Caolán McNamara 
Date:   Sun Mar 26 21:21:22 2017 +0100

ofz#944 avoid recurse to death

Change-Id: I27ed8cf5e8296a1ad91c33614913fc7af4a56d45

diff --git a/lotuswordpro/source/filter/lwpframelayout.cxx 
b/lotuswordpro/source/filter/lwpframelayout.cxx
index c4de8630c4be..c26d61f75378 100644
--- a/lotuswordpro/source/filter/lwpframelayout.cxx
+++ b/lotuswordpro/source/filter/lwpframelayout.cxx
@@ -761,7 +761,7 @@ void LwpFrameLink::Read(LwpObjectStream* pStrm)
 }
 
 LwpFrameLayout::LwpFrameLayout(LwpObjectHeader &objHdr, LwpSvStream* pStrm)
-: LwpPlacableLayout(objHdr, pStrm), m_pFrame(nullptr)
+: LwpPlacableLayout(objHdr, pStrm), m_pFrame(nullptr), 
m_bGettingMaxWidth(false)
 {
 }
 
@@ -945,6 +945,10 @@ double LwpFrameLayout::GetWidth()
  */
 double LwpFrameLayout::GetMaxWidth()
 {
+if (m_bGettingMaxWidth)
+throw std::runtime_error("recursive GetMaxWidth");
+
+m_bGettingMaxWidth = true;
 double fActualWidth = 0;
 rtl::Reference xLayout(GetContainerLayout());
 LwpMiddleLayout* pParent = dynamic_cast(xLayout.get());
@@ -973,6 +977,7 @@ double LwpFrameLayout::GetMaxWidth()
 fActualWidth = fParentWidth - fXOffset - fParentMarginRight - 
fWrapRight;
 }
 
+m_bGettingMaxWidth = false;
 return fActualWidth;
 }
 
diff --git a/lotuswordpro/source/filter/lwpframelayout.hxx 
b/lotuswordpro/source/filter/lwpframelayout.hxx
index 527e5fbd50b5..734fe88ae35f 100644
--- a/lotuswordpro/source/filter/lwpframelayout.hxx
+++ b/lotuswordpro/source/filter/lwpframelayout.hxx
@@ -141,6 +141,7 @@ private:
 private:
 LwpFrameLink m_Link;
 std::unique_ptr m_pFrame;
+bool m_bGettingMaxWidth;
 };
 
 /**
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/unx

2017-03-26 Thread Arnaud Versini
 vcl/unx/gtk3/gtk3gtkframe.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit 80c18d5dd33d0681c58816d7536fe3e7d1297b9e
Author: Arnaud Versini 
Date:   Sun Mar 26 15:48:49 2017 +0200

Fixes tdf#106772 show the window in X11 in VCL gtk3 plugin.

The window is shown only in Wayland.
Thanks to Jean-Baptiste Faure for the bisect.

Change-Id: Ibeffc5ae62d4c341e30b15021eb695859fc1025a
Reviewed-on: https://gerrit.libreoffice.org/35720
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index 847675f082c9..67d79c1e0c51 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -1461,6 +1461,10 @@ void GtkSalFrame::Show( bool bVisible, bool 
/*bNoActivate*/ )
 gtk_widget_show(m_pWindow);
 g_set_prgname(sOrigName.getStr());
 }
+else
+{
+gtk_widget_show(m_pWindow);
+}
 #else
 gtk_widget_show(m_pWindow);
 #endif
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3' - wizards/source

2017-03-26 Thread Jean-Pierre Ledure
 wizards/source/access2base/Application.xba |2 +-
 wizards/source/access2base/Database.xba|2 +-
 wizards/source/access2base/Event.xba   |2 +-
 wizards/source/access2base/Recordset.xba   |1 +
 wizards/source/access2base/Trace.xba   |2 +-
 wizards/source/access2base/Utils.xba   |2 +-
 6 files changed, 6 insertions(+), 5 deletions(-)

New commits:
commit 6cbe9c583720e8e38112dbda2335f86e9cf25742
Author: Jean-Pierre Ledure 
Date:   Sat Mar 25 11:54:27 2017 +0100

Access2Base Missing parentheses - tdf#106710

Due to more severe Basic interpreter on final parentheses
missing parentheses were revealed

Change-Id: I84ead9ba4957ae20ed094d7ae42df436fd1f88f2
Reviewed-on: https://gerrit.libreoffice.org/35683
Reviewed-by: Mike Kaganski 
Tested-by: Mike Kaganski 
(cherry picked from commit a403671af80be1e1563eea013d155e6a0635f0f0)

diff --git a/wizards/source/access2base/Application.xba 
b/wizards/source/access2base/Application.xba
index 19a872007f9f..783fca73541e 100644
--- a/wizards/source/access2base/Application.xba
+++ b/wizards/source/access2base/Application.xba
@@ -915,7 +915,7 @@ Dim sOutput As String, l As Long, lLength As Long
If Len(pvString) > 0 Then
For l = 1 To Len(pvString)
If lLength > 0 And Len(sOutput) > lLength Then 
Exit For
-   sOutput = sOutput & Utils._UTF8Encode(Mid(pvString, 
l, 1)
+   sOutput = sOutput & Utils._UTF8Encode(Mid(pvString, 
l, 1))
Next l
End If
 
diff --git a/wizards/source/access2base/Database.xba 
b/wizards/source/access2base/Database.xba
index 72445e0f3407..63ad4e5a3431 100644
--- a/wizards/source/access2base/Database.xba
+++ b/wizards/source/access2base/Database.xba
@@ -1442,7 +1442,7 @@ Dim i As Integer, l As Long
lNextPattern = InStr(lCurrentChar, psString, 
vPatterns(i), 1)   '  Text (not case-sensitive) string comparison
If lNextPattern > 0 And lNextPattern < lPattern 
Then
lPattern = lNextPattern
-   sPattern = Mid(psString, lPattern, 
Len(vPatterns(i))
+   sPattern = Mid(psString, lPattern, 
Len(vPatterns(i)))
End If
Next i
'  Up to the next pattern or to the end of the string, 
UTF8-encode each character
diff --git a/wizards/source/access2base/Event.xba 
b/wizards/source/access2base/Event.xba
index de6aa2a6e4c6..7688668b2afd 100644
--- a/wizards/source/access2base/Event.xba
+++ b/wizards/source/access2base/Event.xba
@@ -254,7 +254,7 @@ Const cstDatabaseForm = 
"com.sun.star.comp.forms.ODatabaseForm"
Set oObject = poEvent.Source
_EventSource = oObject
sArray = Split(Utils._getUNOTypeName(poEvent), ".")
-   _EventType = UCase(sArray(UBound(sArray))
+   _EventType = UCase(sArray(UBound(sArray)))
If Utils._hasUNOProperty(poEvent, "EventName") Then 
_EventName = poEvent.EventName
 
Select Case _EventType
diff --git a/wizards/source/access2base/Recordset.xba 
b/wizards/source/access2base/Recordset.xba
index 0f7be5b01827..c4ba6ce77f82 100644
--- a/wizards/source/access2base/Recordset.xba
+++ b/wizards/source/access2base/Recordset.xba
@@ -1120,6 +1120,7 @@ Private Function _PropertiesList() As Variant
 
_PropertiesList = Array("AbsolutePosition", "BOF", 
"Bookmarkable", "Bookmark", "EditMode" _
, "EOF", "Filter", 
"LastModified", "Name", "ObjectType" , 
"RecordCount" _
+   )
 
 End Function   '  _PropertiesList
 
diff --git a/wizards/source/access2base/Trace.xba 
b/wizards/source/access2base/Trace.xba
index fdc08d4d623c..601b711fdf33 100644
--- a/wizards/source/access2base/Trace.xba
+++ b/wizards/source/access2base/Trace.xba
@@ -182,7 +182,7 @@ Public Sub TraceLevel(ByVal Optional psTraceLevel As String)
Case psTraceLevel = ""
:   psTraceLevel = "ERROR"
Case Utils._InList(UCase(psTraceLevel), Array( _
TRACEDEBUG, TRACEINFO, TRACEWARNING, TRACEERRORS, 
TRACEFATAL, TRACEABORT _
-   )
+   ))
Case Else   
:   Goto Exit_Sub
End Select
_A2B_.MinimalTraceLevel = _TraceLevel(psTraceLevel)
diff --git a/wizards/source/access2base/Utils.xba 
b/wizards/source/access2base/Utils.xba
index c2b6172b2484..da9e57f7ab73 100644
--- a/wizards/source/access2base/Utils.xba
+++ b/wizards/source/access2base/Utils.xba
@@ -831,7 +831,7 @@ Const cstDot = "."
 Dim sName As String
 
If InStr(psName, ".") > 0 Then
-   sName = Join(Split(psName, cstDot), cstSquareClose & cstDot 
& cstSquareOpen
+   sName = Join(Split(psName, cstDot), cstSquareClose & cstDot 
& cstSqu

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.3' - 10 commits - sc/qa sc/source

2017-03-26 Thread Tor Lillqvist
 sc/qa/unit/data/ods/comment.ods   |binary
 sc/qa/unit/subsequent_export-test.cxx |   21 +
 sc/source/filter/excel/excdoc.cxx |8 +---
 sc/source/filter/excel/xecontent.cxx  |   18 ++
 sc/source/filter/excel/xeescher.cxx   |9 +
 sc/source/filter/excel/xestyle.cxx|2 ++
 sc/source/filter/inc/biffhelper.hxx   |   12 +---
 sc/source/filter/inc/xecontent.hxx|3 ++-
 sc/source/filter/inc/xestream.hxx |4 ++--
 sc/source/filter/inc/xltable.hxx  |2 +-
 sc/source/filter/xcl97/xcl97rec.cxx   |   21 -
 11 files changed, 81 insertions(+), 19 deletions(-)

New commits:
commit 37482cba41c49e9168387acfb470d4bb0a0eef7d
Author: Tor Lillqvist 
Date:   Wed Mar 8 09:50:46 2017 +0200

Re-introduce a check I dropped in 7ceda09f6780c954fedc49764d5457aa2616b39a

Without fully understanding the code, best to let the condition be
there. Use the same function to check for validity as before.
(IsValid() which was renamed to IsValidForXml().)

Change-Id: If50df539737352f12cb0178706cbc38512a99366
(cherry picked from commit a77908a2d82986ad9d2d9597536d1e22ce2cafd2)

diff --git a/sc/source/filter/excel/xecontent.cxx 
b/sc/source/filter/excel/xecontent.cxx
index 8d8fb874ca70..fa5f336329e2 100644
--- a/sc/source/filter/excel/xecontent.cxx
+++ b/sc/source/filter/excel/xecontent.cxx
@@ -1485,7 +1485,8 @@ XclExpCondFormatBuffer::XclExpCondFormatBuffer( const 
XclExpRoot& rRoot, XclExtL
 itr != pCondFmtList->end(); ++itr)
 {
 XclExpCondfmtList::RecordRefType xCondfmtRec( new XclExpCondfmt( 
GetRoot(), **itr, xExtLst, nIndex ));
-maCondfmtList.AppendRecord( xCondfmtRec );
+if( xCondfmtRec->IsValidForXml() )
+maCondfmtList.AppendRecord( xCondfmtRec );
 }
 }
 }
commit 0dd6c9160f49faf352eff9abf5b64fb57a44ac1c
Author: Tor Lillqvist 
Date:   Tue Mar 7 22:40:55 2017 +0200

A 'CondFmt' record can have a maximum of three CF records following (eek)

What an odd restriction. Oh well. Don't export the conditional formats
for the cell(s) in that case then.

See 
https://msdn.microsoft.com/en-us/library/03AE6098-BDC2-475B-BA2C-B8AEF7882174

Change-Id: I4eeec8d33f9fbc572a02f727f38564d6c43b4f10
(cherry picked from commit 7ceda09f6780c954fedc49764d5457aa2616b39a)

diff --git a/sc/source/filter/excel/xecontent.cxx 
b/sc/source/filter/excel/xecontent.cxx
index 7771e92203c6..8d8fb874ca70 100644
--- a/sc/source/filter/excel/xecontent.cxx
+++ b/sc/source/filter/excel/xecontent.cxx
@@ -1277,14 +1277,24 @@ XclExpCondfmt::~XclExpCondfmt()
 {
 }
 
-bool XclExpCondfmt::IsValid() const
+bool XclExpCondfmt::IsValidForBinary() const
+{
+// ccf (2 bytes): An unsigned integer that specifies the count of CF 
records that follow this
+// record. MUST be greater than or equal to 0x0001, and less than or equal 
to 0x0003.
+
+SAL_WARN_IF( maCFList.GetSize() > 3, "sc.filter", "More than 3 conditional 
filters for cell(s), won't export");
+
+return !maCFList.IsEmpty() && maCFList.GetSize() <= 3 && 
!maXclRanges.empty();
+}
+
+bool XclExpCondfmt::IsValidForXml() const
 {
 return !maCFList.IsEmpty() && !maXclRanges.empty();
 }
 
 void XclExpCondfmt::Save( XclExpStream& rStrm )
 {
-if( IsValid() )
+if( IsValidForBinary() )
 {
 XclExpRecord::Save( rStrm );
 maCFList.Save( rStrm );
@@ -1304,7 +1314,7 @@ void XclExpCondfmt::WriteBody( XclExpStream& rStrm )
 
 void XclExpCondfmt::SaveXml( XclExpXmlStream& rStrm )
 {
-if( !IsValid() )
+if( !IsValidForXml() )
 return;
 
 sax_fastparser::FSHelperPtr& rWorksheet = rStrm.GetCurrentStream();
@@ -1475,8 +1485,7 @@ XclExpCondFormatBuffer::XclExpCondFormatBuffer( const 
XclExpRoot& rRoot, XclExtL
 itr != pCondFmtList->end(); ++itr)
 {
 XclExpCondfmtList::RecordRefType xCondfmtRec( new XclExpCondfmt( 
GetRoot(), **itr, xExtLst, nIndex ));
-if( xCondfmtRec->IsValid() )
-maCondfmtList.AppendRecord( xCondfmtRec );
+maCondfmtList.AppendRecord( xCondfmtRec );
 }
 }
 }
diff --git a/sc/source/filter/inc/xecontent.hxx 
b/sc/source/filter/inc/xecontent.hxx
index d8e9de5fd0c5..cbf41a377290 100644
--- a/sc/source/filter/inc/xecontent.hxx
+++ b/sc/source/filter/inc/xecontent.hxx
@@ -231,7 +231,8 @@ public:
 virtual ~XclExpCondfmt() override;
 
 /** Returns true, if this conditional format contains at least one cell 
range and CF record. */
-boolIsValid() const;
+boolIsValidForBinary() const;
+boolIsValidForXml() const;
 
 /** Writes the CONDFMT record with following CF records, if there is valid 
data. */
 virtual voidSave( XclExpStream& rStrm ) override;
commit fb4ab759650e08089c2753a80d8e6ba8266d7970
Author: Tor Lillqvist 

Re: master builds, starts but no UI

2017-03-26 Thread Arnaud Versini
The patch is here : https://gerrit.libreoffice.org/#/c/35720/

2017-03-26 19:13 GMT+02:00 Xisco Fauli :

> Issue reported here: https://bugs.documentfoundatio
> n.org/show_bug.cgi?id=106772
>
>
> El 26/03/17 a les 07:32, Jean-Baptiste Faure ha escrit:
>
> Hi,
>>
>> I confirm that the problematic commit is
>> https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdif
>> f;h=6903066822900a700ccf67ae70fa171c94644a9a
>>
>> Related: rhbz#1334915 tdf#100158 hack using startcenter icon under
>> wayland
>>
>> Conditions to reproduce :
>> 1/ use Xorg
>> 2/ use GTK3 vcl plugin
>>
>> Arnaud said me on irc that it works if you are using Wayland
>>
>> Best regards.
>> JBF
>>
>> Le 25/03/2017 à 23:23, Jean-Baptiste Faure a écrit :
>>
>>> Hi,
>>>
>>> Le 25/03/2017 à 09:20, Jean-Baptiste Faure a écrit :
>>>
 Hi,

 I don't know if it is only me: current master builds without problem, it
 starts but the UI does not show up and it does not crash, it does
 nothing (CPU = 0%). Under gdb I see nothing I am able to interpret.
 I am building under Ubuntu 16.04 x86-64

 I encounter this problem one or two days, somewhere after the commit
 7a46166beb714a3357f7fb04111ba98f6325065f
 I am currently bisecting.

>>>
>>> The StartCenter does not show up if you are not using Wayland.
>>>
>>> I guess that the problematic commit is :
>>> https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdif
>>> f;h=6903066822900a700ccf67ae70fa171c94644a9a
>>>
>>> Related: rhbz#1334915 tdf#100158 hack using startcenter icon under
>>> wayland
>>>
>>> I am currently building the master to confirm that this commit is the
>>> first bad. Currently my first good is the next commit :
>>> (42870930e2a625766288edc3ba956c1bcac198f0 starmath: Fix memory leak at
>>> double sub/superscripts) and my first bad is the previous one
>>> (51efa8fb3f92de9f8c02d2cc4e3bffcaa5caee7f Notebookbar: move shortcuts if
>>> multiple rows of tabs)
>>>
>>> Best regards.
>>> JBF
>>>
>>>
>>
>>
> --
> Xisco Faulí
> Libreoffice QA Team
> IRC: x1sc0
>
>
> ___
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/libreoffice
>



-- 
Arnaud Versini
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: master builds, starts but no UI

2017-03-26 Thread Xisco Fauli
Issue reported here: 
https://bugs.documentfoundation.org/show_bug.cgi?id=106772



El 26/03/17 a les 07:32, Jean-Baptiste Faure ha escrit:

Hi,

I confirm that the problematic commit is
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=6903066822900a700ccf67ae70fa171c94644a9a 



Related: rhbz#1334915 tdf#100158 hack using startcenter icon under
wayland

Conditions to reproduce :
1/ use Xorg
2/ use GTK3 vcl plugin

Arnaud said me on irc that it works if you are using Wayland

Best regards.
JBF

Le 25/03/2017 à 23:23, Jean-Baptiste Faure a écrit :

Hi,

Le 25/03/2017 à 09:20, Jean-Baptiste Faure a écrit :

Hi,

I don't know if it is only me: current master builds without 
problem, it

starts but the UI does not show up and it does not crash, it does
nothing (CPU = 0%). Under gdb I see nothing I am able to interpret.
I am building under Ubuntu 16.04 x86-64

I encounter this problem one or two days, somewhere after the commit
7a46166beb714a3357f7fb04111ba98f6325065f
I am currently bisecting.


The StartCenter does not show up if you are not using Wayland.

I guess that the problematic commit is :
https://gerrit.libreoffice.org/gitweb?p=core.git;a=commitdiff;h=6903066822900a700ccf67ae70fa171c94644a9a 



Related: rhbz#1334915 tdf#100158 hack using startcenter icon under 
wayland


I am currently building the master to confirm that this commit is the
first bad. Currently my first good is the next commit :
(42870930e2a625766288edc3ba956c1bcac198f0 starmath: Fix memory leak at
double sub/superscripts) and my first bad is the previous one
(51efa8fb3f92de9f8c02d2cc4e3bffcaa5caee7f Notebookbar: move shortcuts if
multiple rows of tabs)

Best regards.
JBF






--
Xisco Faulí
Libreoffice QA Team
IRC: x1sc0

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.1' - 3 commits - download.lst external/libxslt external/nss

2017-03-26 Thread Andras Timar
 download.lst   
  |4 
 
external/libxslt/0001-Fix-for-type-confusion-in-preprocessing-attributes.patch.1
 |   29 
 external/libxslt/ExternalPackage_xslt.mk   
  |2 
 external/libxslt/ExternalProject_xslt.mk   
  |4 
 external/libxslt/UnpackedTarball_xslt.mk   
  |8 
 external/libxslt/libxslt-android.patch 
  |   15 
 external/libxslt/libxslt-config-guess.patch.0  
  | 1921 --
 external/libxslt/libxslt-configure.patch.1 
  |   23 
 external/libxslt/libxslt-freebsd.patch.1   
  |   28 
 external/libxslt/libxslt-msvc.patch.2  
  |   13 
 external/libxslt/libxslt-vc15.patch
  |   13 
 external/nss/ExternalPackage_nss.mk
  |5 
 external/nss/ExternalProject_nss.mk
  |1 
 external/nss/Module_nss.mk 
  |2 
 external/nss/README
  |   64 
 external/nss/UnpackedTarball_nss.mk
  |5 
 external/nss/asan.patch.1  
  |   12 
 external/nss/clang-cl.patch.0  
  |   74 
 external/nss/nss-winXP-sdk.patch.1 
  |5 
 external/nss/nss.nowerror.patch
  |8 
 external/nss/nss.patch 
  |   26 
 external/nss/nss.utf8bom.patch.1   
  |   30 
 external/nss/nss.windowbuild.patch.0   
  |   55 
 external/nss/nss.windows.patch 
  |6 
 external/nss/nss_macosx.patch  
  |   10 
 external/nss/ubsan-alignment.patch.0   
  |   40 
 external/nss/ubsan.patch.0 
  |   34 
 27 files changed, 288 insertions(+), 2149 deletions(-)

New commits:
commit d6877736acfbd44417d79e8587739dfc92bf8123
Author: Andras Timar 
Date:   Sun Mar 26 18:31:29 2017 +0200

Revert "nss: remove nss-pem.patch"

This reverts commit 2a9e6ecc61b232be08d8b1687874289ca0fe8d43.

diff --git a/external/nss/ExternalPackage_nss.mk 
b/external/nss/ExternalPackage_nss.mk
index 6d568b7edfee..a7c1994e787a 100644
--- a/external/nss/ExternalPackage_nss.mk
+++ b/external/nss/ExternalPackage_nss.mk
@@ -64,4 +64,22 @@ $(eval $(call 
gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
 ))
 endif
 
+ifeq ($(SYSTEM_CURL),)
+ifeq ($(OS),IOS)
+# nothing
+else ifeq ($(OS),MACOSX)
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/libnsspem.dylib \
+))
+else ifeq ($(OS),WNT)
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/nsspem.dll \
+))
+else # OS!=WNT/MACOSX
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/libnsspem.so \
+))
+endif
+endif
+
 # vim: set noet sw=4 ts=4:
diff --git a/external/nss/ExternalProject_nss.mk 
b/external/nss/ExternalProject_nss.mk
index 07cc472b9fb2..03fd1f1ce135 100644
--- a/external/nss/ExternalProject_nss.mk
+++ b/external/nss/ExternalProject_nss.mk
@@ -89,6 +89,7 @@ $(call gb_ExternalProject_get_state_target,nss,build): $(call 
gb_ExternalProject

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnss3.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssckbi.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssdbm3.dylib \
+   
$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnsspem.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssutil3.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libplc4.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libplds4.dylib \
diff --git a/external/nss/UnpackedTarball_nss.mk 
b/external/nss/UnpackedTarball_nss.mk
index a0ac57173fd1..73a63da68416 100644
--- a/external/nss/UnpackedTarball_nss.mk
+++ b/external/nss/UnpackedTarball_nss.mk
@@ -40,6 +40,13 @@ $(eval $(call gb_UnpackedTarball_add_patches,nss,\
external/nss/nss.utf8bom.patch.1) \
 ))
 
+# nss-pem is only needed for internal curl to read the NSS CA database
+ifeq ($(SYSTEM_CURL),)
+$(eval $(call gb_Unpac

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.0' - external/nss

2017-03-26 Thread Andras Timar
 external/nss/ExternalPackage_nss.mk |   18 
 external/nss/ExternalProject_nss.mk |1 
 external/nss/UnpackedTarball_nss.mk |7 
 external/nss/nss-pem.patch  | 6376 
 4 files changed, 6402 insertions(+)

New commits:
commit 81da190a1e2ee181b9288537722c5d5fb1484060
Author: Andras Timar 
Date:   Sun Mar 26 18:34:18 2017 +0200

Revert "nss: remove nss-pem.patch"

This reverts commit 2a9e6ecc61b232be08d8b1687874289ca0fe8d43.

diff --git a/external/nss/ExternalPackage_nss.mk 
b/external/nss/ExternalPackage_nss.mk
index 6d568b7edfee..a7c1994e787a 100644
--- a/external/nss/ExternalPackage_nss.mk
+++ b/external/nss/ExternalPackage_nss.mk
@@ -64,4 +64,22 @@ $(eval $(call 
gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
 ))
 endif
 
+ifeq ($(SYSTEM_CURL),)
+ifeq ($(OS),IOS)
+# nothing
+else ifeq ($(OS),MACOSX)
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/libnsspem.dylib \
+))
+else ifeq ($(OS),WNT)
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/nsspem.dll \
+))
+else # OS!=WNT/MACOSX
+$(eval $(call gb_ExternalPackage_add_files,nss,$(LIBO_LIB_FOLDER),\
+   dist/out/lib/libnsspem.so \
+))
+endif
+endif
+
 # vim: set noet sw=4 ts=4:
diff --git a/external/nss/ExternalProject_nss.mk 
b/external/nss/ExternalProject_nss.mk
index 07cc472b9fb2..03fd1f1ce135 100644
--- a/external/nss/ExternalProject_nss.mk
+++ b/external/nss/ExternalProject_nss.mk
@@ -89,6 +89,7 @@ $(call gb_ExternalProject_get_state_target,nss,build): $(call 
gb_ExternalProject

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnss3.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssckbi.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssdbm3.dylib \
+   
$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnsspem.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libnssutil3.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libplc4.dylib \

$(gb_Package_SOURCEDIR_nss)/dist/out/lib/libplds4.dylib \
diff --git a/external/nss/UnpackedTarball_nss.mk 
b/external/nss/UnpackedTarball_nss.mk
index a0ac57173fd1..73a63da68416 100644
--- a/external/nss/UnpackedTarball_nss.mk
+++ b/external/nss/UnpackedTarball_nss.mk
@@ -40,6 +40,13 @@ $(eval $(call gb_UnpackedTarball_add_patches,nss,\
external/nss/nss.utf8bom.patch.1) \
 ))
 
+# nss-pem is only needed for internal curl to read the NSS CA database
+ifeq ($(SYSTEM_CURL),)
+$(eval $(call gb_UnpackedTarball_add_patches,nss,\
+   external/nss/nss-pem.patch \
+))
+endif
+
 ifeq ($(COM_IS_CLANG),TRUE)
 ifneq ($(filter -fsanitize=%,$(CC)),)
 $(eval $(call gb_UnpackedTarball_add_patches,nss,\
diff --git a/external/nss/nss-pem.patch b/external/nss/nss-pem.patch
new file mode 100644
index ..d41809d3ecaf
--- /dev/null
+++ b/external/nss/nss-pem.patch
@@ -0,0 +1,6376 @@
+diff --git a/a/nss/lib/ckfw/manifest.mn b/b/nss/lib/ckfw/manifest.mn
+index 20bebeb..4f10563 100644
+--- a/a/nss/lib/ckfw/manifest.mn
 b/b/nss/lib/ckfw/manifest.mn
+@@ -5,7 +5,7 @@
+ 
+ CORE_DEPTH = ../..
+ 
+-DIRS = builtins 
++DIRS = builtins pem
+ 
+ PRIVATE_EXPORTS = \
+   ck.h  \
+diff --git a/a/nss/lib/ckfw/pem/Makefile b/b/nss/lib/ckfw/pem/Makefile
+new file mode 100644
+index 000..aec3bbd
+--- /dev/null
 b/b/nss/lib/ckfw/pem/Makefile
+@@ -0,0 +1,107 @@
++# 
++# * BEGIN LICENSE BLOCK *
++# Version: MPL 1.1/GPL 2.0/LGPL 2.1
++#
++# The contents of this file are subject to the Mozilla Public License Version
++# 1.1 (the "License"); you may not use this file except in compliance with
++# the License. You may obtain a copy of the License at
++# http://www.mozilla.org/MPL/
++#
++# Software distributed under the License is distributed on an "AS IS" basis,
++# WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
++# for the specific language governing rights and limitations under the
++# License.
++#
++# The Original Code is the Netscape security libraries.
++#
++# The Initial Developer of the Original Code is
++# Netscape Communications Corporation.
++# Portions created by the Initial Developer are Copyright (C) 1994-2000
++# the Initial Developer. All Rights Reserved.
++#
++# Contributor(s):
++#
++# Alternatively, the contents of this file may be used under the terms of
++# either the GNU General Public License Version 2 or later (the "GPL"), or
++# the GNU Lesser General Public License Version 2.1 or later (the "LGPL"),
++# in which case the provisions of the GPL or the LGPL are applicable instead
++# of those above. If you wish to allow use of your version of this file only
++# under the terms of either the GPL or the LGPL, and not to allow others to
++# use your version of this file under t

[Libreoffice-commits] online.git: loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/src/layer/tile/CalcTileLayer.js |   21 +
 1 file changed, 21 insertions(+)

New commits:
commit d99dbade2b3a4090c4ff3fd593cd45a4ed6ff121
Author: Henry Castro 
Date:   Sun Mar 26 11:55:04 2017 -0400

loleaflet: handle annotation reply for Calc documents

Change-Id: I27d03313bf4cdc43e7356e78424ee236b90c5269

diff --git a/loleaflet/src/layer/tile/CalcTileLayer.js 
b/loleaflet/src/layer/tile/CalcTileLayer.js
index 0dd18011..1fa97c45 100644
--- a/loleaflet/src/layer/tile/CalcTileLayer.js
+++ b/loleaflet/src/layer/tile/CalcTileLayer.js
@@ -51,6 +51,7 @@ L.CalcTileLayer = L.TileLayer.extend({
map.on('zoomend', this._onZoomRowColumns, this);
map.on('updateparts', this._onUpdateParts, this);
map.on('AnnotationCancel', this._onAnnotationCancel, this);
+   map.on('AnnotationReply', this._onAnnotationReply, this);
map.on('AnnotationSave', this._onAnnotationSave, this);
},
 
@@ -86,6 +87,11 @@ L.CalcTileLayer = L.TileLayer.extend({
this._map.focus();
},
 
+   onAnnotationReply: function (annotation) {
+   annotation.reply();
+   annotation.focus();
+   },
+
showAnnotation: function (annotation) {
this._map.addLayer(annotation.mark);
this._map.addLayer(annotation);
@@ -119,6 +125,21 @@ L.CalcTileLayer = L.TileLayer.extend({
this._map.focus();
},
 
+   _onAnnotationReply: function (e) {
+   var comment = {
+   Id: {
+   type: 'string',
+   value: e.annotation._data.id
+   },
+   Text: {
+   type: 'string',
+   value: e.annotation._data.reply
+   }
+   };
+   this._map.sendUnoCommand('.uno:ReplyComment', comment);
+   this._map.focus();
+   },
+
_onAnnotationSave: function (e) {
var comment;
if (e.annotation._data.id === 'new') {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/src/layer/tile/ImpressTileLayer.js |   23 +++
 1 file changed, 23 insertions(+)

New commits:
commit 5e67e19a6c0d0e4d7ba32bfed5df4cf553cfe9e7
Author: Henry Castro 
Date:   Sun Mar 26 11:15:18 2017 -0400

loleaflet: handle annotation reply for Impress documents

Change-Id: I47edbc8807f5f7aa5a28b3a4fd79aeebc8026262

diff --git a/loleaflet/src/layer/tile/ImpressTileLayer.js 
b/loleaflet/src/layer/tile/ImpressTileLayer.js
index 1390216e..09015cbf 100644
--- a/loleaflet/src/layer/tile/ImpressTileLayer.js
+++ b/loleaflet/src/layer/tile/ImpressTileLayer.js
@@ -14,6 +14,7 @@ L.ImpressTileLayer = L.TileLayer.extend({
beforeAdd: function (map) {
map.on('updateparts', this.onUpdateParts, this);
map.on('AnnotationCancel', this.onAnnotationCancel, this);
+   map.on('AnnotationReply', this.onReplyClick, this);
map.on('AnnotationSave', this.onAnnotationSave, this);
map.on('AnnotationScrollUp', this.onAnnotationScrollUp, this);
map.on('AnnotationScrollDown', this.onAnnotationScrollDown, 
this);
@@ -52,6 +53,12 @@ L.ImpressTileLayer = L.TileLayer.extend({
draft.focus();
},
 
+   onAnnotationReply: function (annotation) {
+   var draft = L.annotation(this._map.getCenter(), 
annotation._data).addTo(this._map);
+   draft.reply();
+   draft.focus();
+   },
+
onAnnotationRemove: function (id) {
var comment = {
Id: {
@@ -90,6 +97,22 @@ L.ImpressTileLayer = L.TileLayer.extend({
this._map.focus();
},
 
+   onReplyClick: function (e) {
+   var comment = {
+   Id: {
+   type: 'string',
+   value: e.annotation._data.id
+   },
+   Text: {
+   type: 'string',
+   value: e.annotation._data.reply
+   }
+   };
+   this._map.sendUnoCommand('.uno:ReplyToAnnotation', comment);
+   this._map.removeLayer(e.annotation);
+   this._map.focus();
+   },
+
onAnnotationScrollDown: function (e) {
this._topAnnotation = Math.min(++this._topAnnotation, 
this._annotations[this._partHashes[this._selectedPart]].length - 1);
this.layoutAnnotations();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: basctl/sdi basctl/source include/editeng sc/sdi sc/source sd/sdi svx/sdi svx/source sw/sdi sw/source

2017-03-26 Thread Maxim Monastirsky
 basctl/sdi/baside.sdi|  119 +++
 basctl/source/basicide/baside3.cxx   |  364 +--
 basctl/source/basicide/basides1.cxx  |   25 ++
 basctl/source/basicide/basidesh.cxx  |   46 
 basctl/source/basicide/idetemp.hxx   |   57 -
 basctl/source/inc/baside3.hxx|1 
 basctl/source/inc/basidesh.hxx   |2 
 include/editeng/svxenum.hxx  |   10 
 sc/sdi/drawsh.sdi|   11 -
 sc/source/ui/app/typemap.cxx |1 
 sc/source/ui/drawfunc/drawsh5.cxx|7 
 sd/sdi/_drvwsh.sdi   |5 
 svx/sdi/fmslots.sdi  |   10 
 svx/sdi/svx.sdi  |   18 -
 svx/sdi/svxitems.sdi |   42 
 svx/source/form/fmshell.cxx  |6 
 sw/sdi/drwbassh.sdi  |   37 +++
 sw/sdi/wdrwbase.sdi  |   37 +++
 sw/source/uibase/shells/drwbassh.cxx |   16 -
 sw/source/uibase/shells/slotadd.cxx  |1 
 sw/source/uibase/uiview/view2.cxx|9 
 sw/source/uibase/uiview/viewstat.cxx |   21 --
 22 files changed, 419 insertions(+), 426 deletions(-)

New commits:
commit 12c6caa84a61f23df996a0396432fa4b8d5c4785
Author: Maxim Monastirsky 
Date:   Sat Mar 25 22:53:28 2017 +0300

Kill remaining PseudoSlots usage

Change-Id: I14a820b7c0baba8d8b17d07715bf17c42c73c8fb
TODO: Remove support code from idl and sfx2 modules
Reviewed-on: https://gerrit.libreoffice.org/35718
Tested-by: Jenkins 
Reviewed-by: Maxim Monastirsky 

diff --git a/basctl/sdi/baside.sdi b/basctl/sdi/baside.sdi
index e61f737e0e14..f4993919e5ae 100644
--- a/basctl/sdi/baside.sdi
+++ b/basctl/sdi/baside.sdi
@@ -409,6 +409,125 @@ shell basctl_Shell
 
 SID_CHOOSE_CONTROLS
 [
+StateMethod = GetState;
+]
+SID_INSERT_PUSHBUTTON
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_CHECKBOX
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_RADIOBUTTON
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_SPINBUTTON
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_FIXEDTEXT
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_GROUPBOX
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_LISTBOX
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_COMBOBOX
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_EDIT
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_HSCROLLBAR
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_VSCROLLBAR
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_SELECT
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_IMAGECONTROL
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_PROGRESSBAR
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_HFIXEDLINE
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_VFIXEDLINE
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_DATEFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_TIMEFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_NUMERICFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_CURRENCYFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_FORMATTEDFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_PATTERNFIELD
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_FILECONTROL
+[
+ExecMethod  = ExecuteDialog;
+StateMethod = GetState;
+]
+SID_INSERT_TREECONTROL
+[
 ExecMethod  = ExecuteDialog;
 StateMethod = GetState;
 ]
diff --git a/basctl/source/basicide/baside3.cxx 
b/basctl/source/basicide/baside3.cxx
index 821c34176072..ffccb5e2f1cb 100644
--- a/basctl/source/basicide/baside3.cxx
+++ b/basctl/source/basicide/baside3.cxx
@@ -29,7 +29,6 @@
 #include "dlgedmod.hxx"
 #include "dlgedview.hxx"
 #include "iderdll.hxx"
-#include "

[Libreoffice-commits] core.git: vcl/source

2017-03-26 Thread heiko tietze
 vcl/source/app/settings.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 3bae306bcf23c4baae65b2b8a3617f0b697dfc6a
Author: heiko tietze 
Date:   Sun Mar 26 14:57:33 2017 +0200

tdf#106458 Color widget should list an exact number of items

mnColorValueSetMaximumRowCount set to 9

Change-Id: I818512da16c1f31616f200253e72a8576996fa16
Reviewed-on: https://gerrit.libreoffice.org/35719
Tested-by: Jenkins 
Reviewed-by: Maxim Monastirsky 

diff --git a/vcl/source/app/settings.cxx b/vcl/source/app/settings.cxx
index 456aca3e109e..703f835c9b1d 100644
--- a/vcl/source/app/settings.cxx
+++ b/vcl/source/app/settings.cxx
@@ -550,7 +550,7 @@ ImplStyleData::ImplStyleData() :
 maEdgeBlendingBottomRightColor = RGB_COLORDATA(0x40, 0x40, 0x40);
 mnListBoxMaximumLineCount = 25;
 mnColorValueSetColumnCount = 12;
-mnColorValueSetMaximumRowCount = 8;
+mnColorValueSetMaximumRowCount = 9;
 maListBoxPreviewDefaultLogicSize = Size(15, 7);
 maListBoxPreviewDefaultPixelSize = Size(0, 0); // on-demand calculated in 
GetListBoxPreviewDefaultPixelSize()
 mnListBoxPreviewDefaultLineWidth = 1;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/src/layer/marker/Annotation.js |   16 
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit 563b90a20a8c4e318504579ca3a6568d95072fd4
Author: Henry Castro 
Date:   Sun Mar 26 10:19:10 2017 -0400

loleaflet: remove save button of the annotation object

Change-Id: I1aa19e3620d1bbd3a76fae32e97c4c32adcf0fe6

diff --git a/loleaflet/src/layer/marker/Annotation.js 
b/loleaflet/src/layer/marker/Annotation.js
index 0dc4c135..28c099f0 100644
--- a/loleaflet/src/layer/marker/Annotation.js
+++ b/loleaflet/src/layer/marker/Annotation.js
@@ -156,7 +156,7 @@ L.Annotation = L.Layer.extend({
this._nodeReplyText = L.DomUtil.create(tagTextArea, 
classTextArea, this._nodeReply);
 
buttons = L.DomUtil.create(tagDiv, empty, this._nodeModify);
-   this._createButton(buttons, _(' Save '), this._onSaveClick);
+   L.DomEvent.on(this._nodeModifyText, 'blur', this._onLostFocus, 
this);
this._createButton(buttons, cancel, this._onCancelClick);
buttons = L.DomUtil.create(tagDiv, empty, this._nodeReply);
this._createButton(buttons, _('Reply'), this._onReplyClick);
@@ -182,6 +182,13 @@ L.Annotation = L.Layer.extend({
this._map.fire('AnnotationCancel', {annotation: this});
},
 
+   _onLostFocus: function (e) {
+   L.DomEvent.stopPropagation(e);
+   this._data.text = this._contentText.innerHTML = 
this._nodeModifyText.value;
+   this.show();
+   this._map.fire('AnnotationSave', {annotation: this});
+   },
+
_onMouseClick: function (e) {
var target = e.target || e.srcElement;
L.DomEvent.stopPropagation(e);
@@ -215,13 +222,6 @@ L.Annotation = L.Layer.extend({
this._map.fire('AnnotationReply', {annotation: this});
},
 
-   _onSaveClick: function (e) {
-   L.DomEvent.stopPropagation(e);
-   this._data.text = this._contentText.innerHTML = 
this._nodeModifyText.value;
-   this.show();
-   this._map.fire('AnnotationSave', {annotation: this});
-   },
-
_updateLayout: function () {
var style = this._wrapper.style;
style.width = '';
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: loleaflet/src

2017-03-26 Thread Henry Castro
 loleaflet/src/layer/tile/CalcTileLayer.js |   11 +++
 1 file changed, 7 insertions(+), 4 deletions(-)

New commits:
commit 6da38c824440d444b45b6144ff088db367487d51
Author: Henry Castro 
Date:   Sun Mar 26 10:11:27 2017 -0400

loleaflet: fix when an annotation is removed from Calc documents

Calc server does not respond with cellPos property when an annotation
is removed

Change-Id: I7c15e112c549f10277fb2fb51bc12b3f16f5fc88

diff --git a/loleaflet/src/layer/tile/CalcTileLayer.js 
b/loleaflet/src/layer/tile/CalcTileLayer.js
index 35877c64..0dd18011 100644
--- a/loleaflet/src/layer/tile/CalcTileLayer.js
+++ b/loleaflet/src/layer/tile/CalcTileLayer.js
@@ -162,10 +162,10 @@ L.CalcTileLayer = L.TileLayer.extend({
if (textMsg.startsWith('comment:')) {
var obj = 
JSON.parse(textMsg.substring('comment:'.length + 1));
obj.comment.tab = obj.comment.id.substring(0, 
obj.comment.id.indexOf('.'));
-   obj.comment.cellPos = 
L.LOUtil.stringToBounds(obj.comment.cellPos);
-   obj.comment.cellPos = 
L.latLngBounds(this._twipsToLatLng(obj.comment.cellPos.getBottomLeft()),
-   
this._twipsToLatLng(obj.comment.cellPos.getTopRight()));
if (obj.comment.action === 'Add') {
+   obj.comment.cellPos = 
L.LOUtil.stringToBounds(obj.comment.cellPos);
+   obj.comment.cellPos = 
L.latLngBounds(this._twipsToLatLng(obj.comment.cellPos.getBottomLeft()),
+   
this._twipsToLatLng(obj.comment.cellPos.getTopRight()));
if (!this._annotations[obj.comment.tab]) {
this._annotations[obj.comment.tab] = {};
}
@@ -177,10 +177,13 @@ L.CalcTileLayer = L.TileLayer.extend({
var removed = 
this._annotations[obj.comment.tab][obj.comment.id];
if (removed) {
this.hideAnnotation(removed);
-   delete 
annotations[obj.comment.tab][obj.comment.id];
+   delete 
this._annotations[obj.comment.tab][obj.comment.id];
}
} else if (obj.comment.action === 'Modify') {
var modified = 
this._annotations[obj.comment.tab][obj.comment.id];
+   obj.comment.cellPos = 
L.LOUtil.stringToBounds(obj.comment.cellPos);
+   obj.comment.cellPos = 
L.latLngBounds(this._twipsToLatLng(obj.comment.cellPos.getBottomLeft()),
+   
this._twipsToLatLng(obj.comment.cellPos.getTopRight()));
if (modified) {
modified._annotation._data = 
obj.comment;

modified.setLatLngBounds(obj.comment.cellPos);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'aoo/trunk' - officecfg/registry

2017-03-26 Thread Matthias Seidel
 officecfg/registry/data/org/openoffice/Office/ExtensionManager.xcu |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c959f7f196dff719296388efb87f2e4fcc62baea
Author: Matthias Seidel 
Date:   Sun Mar 26 13:01:26 2017 +

Changed URL for extensions in extension manager

diff --git a/officecfg/registry/data/org/openoffice/Office/ExtensionManager.xcu 
b/officecfg/registry/data/org/openoffice/Office/ExtensionManager.xcu
index 4376d09b0605..39a7c7077a61 100644
--- a/officecfg/registry/data/org/openoffice/Office/ExtensionManager.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/ExtensionManager.xcu
@@ -26,7 +26,7 @@
 http://openoffice.org/2001/registry";>
   
 
-  
http://extensions.services.openoffice.org/getmore?cid=920794
+  http://extensions.openoffice.org/getmore?cid=920794
 
   
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sfx2/source shell/source slideshow/source

2017-03-26 Thread Andrea Gelmini
 sfx2/source/appl/appcfg.cxx   |2 +-
 sfx2/source/appl/appchild.cxx |2 +-
 sfx2/source/appl/shutdowniconw32.cxx  |2 +-
 sfx2/source/control/dispatch.cxx  |6 +++---
 sfx2/source/control/thumbnailview.cxx |4 ++--
 sfx2/source/dialog/dinfdlg.cxx|2 +-
 sfx2/source/doc/docfile.cxx   |2 +-
 sfx2/source/doc/objmisc.cxx   |2 +-
 sfx2/source/view/viewfrm.cxx  |2 +-
 shell/source/cmdmail/cmdmailsuppl.cxx |2 +-
 slideshow/source/engine/opengl/TransitionImpl.hxx |2 +-
 slideshow/source/engine/shapes/drawinglayeranimation.cxx  |2 +-
 slideshow/source/engine/slide/slideimpl.cxx   |2 +-
 slideshow/source/engine/slide/targetpropertiescreator.cxx |2 +-
 slideshow/source/engine/slideshowimpl.cxx |2 +-
 slideshow/source/inc/activitiesfactory.hxx|4 ++--
 16 files changed, 20 insertions(+), 20 deletions(-)

New commits:
commit 2eae33e82e5f91d6dbaf7e8e848f53644596059d
Author: Andrea Gelmini 
Date:   Sun Mar 26 14:27:42 2017 +0200

Fix typos

Change-Id: I5518e764fd8349c476e79bb888e93a1dab01ed1d
Reviewed-on: https://gerrit.libreoffice.org/35644
Tested-by: Jenkins 
Reviewed-by: Julien Nabet 

diff --git a/sfx2/source/appl/appcfg.cxx b/sfx2/source/appl/appcfg.cxx
index 03ae9db916aa..0f1b1bb8ebb2 100644
--- a/sfx2/source/appl/appcfg.cxx
+++ b/sfx2/source/appl/appcfg.cxx
@@ -709,7 +709,7 @@ void SfxApplication::SetOptions_Impl( const SfxItemSet& 
rSet )
 static_cast< SfxStringItem const * >(pItem)->GetValue(), batch);
 }
 
-// Secure-Referers
+// Secure-Referrer
 if ( SfxItemState::SET == rSet.GetItemState(SID_SECURE_URL, true, &pItem))
 {
 DBG_ASSERT(dynamic_cast< const SfxStringListItem *>( pItem ) !=  
nullptr, "StringListItem expected");
diff --git a/sfx2/source/appl/appchild.cxx b/sfx2/source/appl/appchild.cxx
index 1c36be3c1b77..0cee7193875a 100644
--- a/sfx2/source/appl/appchild.cxx
+++ b/sfx2/source/appl/appchild.cxx
@@ -74,7 +74,7 @@ void SfxApplication::RegisterChildWindowContext_Impl( 
SfxModule *pMod, sal_uInt1
 SfxChildWinFactory *pFac = &(*pFactories)[nFactory];
 if ( nId == pFac->nId )
 {
-// Factory found, registrer Context here.
+// Factory found, register Context here.
 pF = pFac;
 break;
 }
diff --git a/sfx2/source/appl/shutdowniconw32.cxx 
b/sfx2/source/appl/shutdowniconw32.cxx
index 434006120012..1a1b794794a7 100644
--- a/sfx2/source/appl/shutdowniconw32.cxx
+++ b/sfx2/source/appl/shutdowniconw32.cxx
@@ -274,7 +274,7 @@ LRESULT CALLBACK listenerWndProc( HWND hWnd, UINT uMsg, 
WPARAM wParam, LPARAM lP
 return TRUE;
 case WM_CREATE:
 {
-// request notfication when taskbar is recreated
+// request notification when taskbar is recreated
 // we then have to add our icon again
 s_uTaskbarRestart = 
RegisterWindowMessage(TEXT("TaskbarCreated"));
 s_uMsgKillTray = RegisterWindowMessage( 
SHUTDOWN_QUICKSTART_MESSAGE );
diff --git a/sfx2/source/control/dispatch.cxx b/sfx2/source/control/dispatch.cxx
index f6d19173b3c7..4303cd9e0485 100644
--- a/sfx2/source/control/dispatch.cxx
+++ b/sfx2/source/control/dispatch.cxx
@@ -483,7 +483,7 @@ SfxDispatcher::~SfxDispatcher()
 xImp->aIdle.Stop();
 xImp->xPoster->SetEventHdl( Link() );
 
-// Notify the stack varialbles in Call_Impl
+// Notify the stack variables in Call_Impl
 if ( xImp->pInCallAliveFlag )
 *xImp->pInCallAliveFlag = false;
 
@@ -1011,7 +1011,7 @@ const SfxPoolItem* SfxDispatcher::Execute(sal_uInt16 
nSlot, SfxCallMode nCall,
 
 @param nSlot the Id of the executing function
 @param eCall SfxCallMode::SYNCRHON, ..._ASYNCHRON or ..._SLOT
-@param pArgs Zero teminated C-Array of Parameters
+@param pArgs Zero terminated C-Array of Parameters
 @param pInternalArgs Zero terminated C-Array of Parameters
 
 @return const SfxPoolItem* Pointer to the SfxPoolItem valid to the next run
@@ -1611,7 +1611,7 @@ void SfxDispatcher::FlushImpl()
 if( bAwakeBindings )
 aToDoCopy.clear();
 
-// If more changes have occurred on the stach when
+// If more changes have occurred on the stack when
 // Activate/Deactivate/Delete:
 if (!xImp->bFlushed)
 // If Push/Pop has been called by someone, then also EnterReg was 
called!
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 8749a00e5ff2..5843d79c8a54 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailvi

[Libreoffice-commits] core.git: desktop/CppunitTest_desktop_app.mk

2017-03-26 Thread Stephan Bergmann
 desktop/CppunitTest_desktop_app.mk |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 887039a3b766c1f2f5a37130f4e7c2d116d89c5f
Author: Stephan Bergmann 
Date:   Sun Mar 26 08:08:51 2017 +

Extra linking against sofficeapp

...already included via gb_CppunitTest_use_library_objects

Change-Id: I4135a48c2ed60a36c15c85bf3720a3080e70837e
Reviewed-on: https://gerrit.libreoffice.org/35716
Tested-by: Jenkins 
Reviewed-by: Stephan Bergmann 

diff --git a/desktop/CppunitTest_desktop_app.mk 
b/desktop/CppunitTest_desktop_app.mk
index 1e7e3e0813c5..b809b6a7e8c6 100644
--- a/desktop/CppunitTest_desktop_app.mk
+++ b/desktop/CppunitTest_desktop_app.mk
@@ -34,7 +34,6 @@ $(eval $(call gb_CppunitTest_use_libraries,desktop_app, \
 salhelper \
 sb \
 sfx \
-sofficeapp \
 svl \
 svx \
 svxcore \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: 2 commits - basctl/uiconfig framework/source

2017-03-26 Thread Maxim Monastirsky
 basctl/uiconfig/basicide/toolbar/insertcontrolsbar.xml |3 +--
 framework/source/uielement/subtoolbarcontroller.cxx|1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

New commits:
commit b0f97cfc1cd6de86c31c00b9e33d7d5805215826
Author: Maxim Monastirsky 
Date:   Sun Mar 26 16:27:13 2017 +0300

Remove useless toolbar break

Seems to not affect the layout in anyway, but instead
always hides the last item when the toolbar is docked.

Change-Id: Ib33c6757f4a9b5037648516e3da3b1f9f08acf40

diff --git a/basctl/uiconfig/basicide/toolbar/insertcontrolsbar.xml 
b/basctl/uiconfig/basicide/toolbar/insertcontrolsbar.xml
index 87de703703c4..aec92a8868fa 100644
--- a/basctl/uiconfig/basicide/toolbar/insertcontrolsbar.xml
+++ b/basctl/uiconfig/basicide/toolbar/insertcontrolsbar.xml
@@ -57,6 +57,5 @@
  
  
  
-
- 
+ 
 
commit 91053763a7189c8ac03a59dc996e6333404f846e
Author: Maxim Monastirsky 
Date:   Sun Mar 26 16:20:50 2017 +0300

Fix sub-toolbar tearoff for .uno:ChooseControls

Regression of 81d4fbc0daa54889ccb09e6a3fadff9c70d99448
("tdf#42029 Use a floating toolbar to show clipped items").

Change-Id: Id32a49109477745122d78354a5d3bd94cb126310

diff --git a/framework/source/uielement/subtoolbarcontroller.cxx 
b/framework/source/uielement/subtoolbarcontroller.cxx
index 2f372633a315..266ebbab247c 100644
--- a/framework/source/uielement/subtoolbarcontroller.cxx
+++ b/framework/source/uielement/subtoolbarcontroller.cxx
@@ -245,7 +245,6 @@ css::uno::Reference< css::awt::XWindow > 
SubToolBarController::createPopupWindow
 // calc and set size for popup mode
 Size aSize = pToolBar->CalcPopupWindowSizePixel();
 pToolBar->SetSizePixel( aSize );
-pToolBar->SetMenuType( ToolBoxMenuType::NONE );
 // open subtoolbox in popup mode
 vcl::Window::GetDockingManager()->StartPopupMode( 
pToolBox, pToolBar );
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'refs/notes/commits' - 54/8f330f31bc238e041e4bfb7b413b8ea641d6ad

2017-03-26 Thread Caolán McNamara
 54/8f330f31bc238e041e4bfb7b413b8ea641d6ad |1 +
 1 file changed, 1 insertion(+)

New commits:
commit b72191ca65dcb260c104d06a8ad25c647c45ea47
Author: Caolán McNamara 
Date:   Sun Mar 26 11:37:59 2017 +0100

Notes added by 'git notes add'

diff --git a/54/8f330f31bc238e041e4bfb7b413b8ea641d6ad 
b/54/8f330f31bc238e041e4bfb7b413b8ea641d6ad
new file mode 100644
index ..8e5c182e7789
--- /dev/null
+++ b/54/8f330f31bc238e041e4bfb7b413b8ea641d6ad
@@ -0,0 +1 @@
+ignore: aoo
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits