[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-29 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java |4 

 android/source/src/java/org/libreoffice/ui/FileUtilities.java|5 
+
 2 files changed, 9 insertions(+)

New commits:
commit 83f8685504168574f00c24e6ef5d50026c1fda5f
Author: Michael Weghorn 
AuthorDate: Sat Nov 25 22:09:34 2023 +0100
Commit: Xisco Fauli 
CommitDate: Wed Nov 29 13:51:51 2023 +0100

android: Suggest file name for PDF export

When using the PDF export feature in Android Viewer,
suggest a file name for the PDF file that matches
the current display name (which is usually the
file name, or "untitled" for a newly created doc
that hasn't been saved yet).

This can be achieved by setting `Intent.EXTRA_TITLE`
for the `ACTION_CREATE_DOCUMENT`. [1]

The `DocumentsContract.EXTRA_INITIAL_URI` already
set previously already results in the same
directory as the doc being preselected in the
file chooser:

> Callers can set a document URI through
> DocumentsContract#EXTRA_INITIAL_URI to indicate the initial location of
> documents navigator. System will do its best to launch the navigator in
> the specified document if it's a folder, or the folder that contains the
> specified document if not.

Filling in the current file name was suggested
in a Google Play review comment for the app.

[1] 
https://developer.android.com/reference/android/content/Intent#ACTION_CREATE_DOCUMENT

Change-Id: Idbd4a89416089f927e0232ce65161b43059ca46d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159959
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit 872d26d4a859ff72e1e13068b798854185960de0)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160031
Reviewed-by: Xisco Fauli 

diff --git 
a/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java 
b/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
index b21ad02383d7..ba371bfd9c8a 100644
--- a/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
+++ b/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
@@ -350,7 +350,11 @@ public class LibreOfficeMainActivity extends 
AppCompatActivity implements Settin
 Intent intent = new Intent(Intent.ACTION_CREATE_DOCUMENT);
 intent.addCategory(Intent.CATEGORY_OPENABLE);
 intent.setType(FileUtilities.MIMETYPE_PDF);
+// suggest directory and file name based on the doc
 intent.putExtra(DocumentsContract.EXTRA_INITIAL_URI, mDocumentUri);
+final String displayName = toolbarTop.getTitle().toString();
+final String suggestedFileName = 
FileUtilities.stripExtensionFromFileName(displayName) + ".pdf";
+intent.putExtra(Intent.EXTRA_TITLE, suggestedFileName);
 
 startActivityForResult(intent, REQUEST_CODE_EXPORT_TO_PDF);
 }
diff --git a/android/source/src/java/org/libreoffice/ui/FileUtilities.java 
b/android/source/src/java/org/libreoffice/ui/FileUtilities.java
index 902b30ed7794..7fc8c3c84eb1 100644
--- a/android/source/src/java/org/libreoffice/ui/FileUtilities.java
+++ b/android/source/src/java/org/libreoffice/ui/FileUtilities.java
@@ -124,6 +124,11 @@ public class FileUtilities {
 return mimeType != null && mimeType.endsWith("template");
 }
 
+public static String stripExtensionFromFileName(final String fileName)
+{
+return fileName.split("\\.[A-Za-z0-9]*$")[0];
+}
+
 /**
  * Tries to retrieve the display (which should be the document name)
  * for the given URI using the given resolver.


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-29 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java |   21 
++
 1 file changed, 14 insertions(+), 7 deletions(-)

New commits:
commit 615705acd731150e029587daafc1226d1e07a108
Author: Michael Weghorn 
AuthorDate: Mon Nov 27 11:05:57 2023 +0100
Commit: Xisco Fauli 
CommitDate: Wed Nov 29 13:43:11 2023 +0100

tdf#158398 android: Draw light gray background for Calc headers

Similar to the desktop version, use a light gray background
color for the Calc header cells.

There was already code in place to draw darker gray background
to highlight the header cell when a cell in that row/column is
selected.
(The actually highlighted header cell didn't wasn't always the
correct one in a quick test, but that's independent of this change.)

Adapt that to always fill the rectangle, but use a lighter gray
(lower alpha value) when not selected.
Use a separate `Paint` object for the frame (stroke).
Set the frame color and text color to black instead of gray, for
better contrast to the light gray fill/background.

Change-Id: I0490811e928ebd1b3840242fc1aa4682b2786b00
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159989
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
Signed-off-by: Xisco Fauli 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160026

diff --git a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java 
b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
index eec0b5e4a88d..a285234bc8b0 100644
--- a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
+++ b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
@@ -10,6 +10,8 @@ import android.text.TextPaint;
 
 public class CalcHeaderCell extends CommonCanvasElement {
 private final TextPaint mTextPaint = new TextPaint();
+
+private final Paint mFramePaint = new Paint();
 private final Paint mBgPaint = new Paint();
 private final RectF mBounds;
 private final Rect mTextBounds = new Rect();
@@ -17,16 +19,20 @@ public class CalcHeaderCell extends CommonCanvasElement {
 
 public CalcHeaderCell(float left, float top, float width, float height, 
String text, boolean selected) {
 mBounds = new RectF(left, top, left + width, top + height);
+
+mFramePaint.setStyle(Style.STROKE);
+mFramePaint.setColor(Color.BLACK);
+
+mBgPaint.setStyle(Style.FILL);
+mBgPaint.setColor(Color.GRAY);
+// draw background more intensely when cell is selected
 if (selected) {
-// if the cell is selected, display filled
-mBgPaint.setStyle(Style.FILL_AND_STROKE);
+mBgPaint.setAlpha(100);
 } else {
-// if not, display only the frame
-mBgPaint.setStyle(Style.STROKE);
+mBgPaint.setAlpha(25);
 }
-mBgPaint.setColor(Color.GRAY);
-mBgPaint.setAlpha(100);  // hard coded for now
-mTextPaint.setColor(Color.GRAY);
+
+mTextPaint.setColor(Color.BLACK);
 mTextPaint.setTextSize(24f); // hard coded for now
 mTextPaint.setTextAlign(Paint.Align.CENTER);
 mText = text;
@@ -54,6 +60,7 @@ public class CalcHeaderCell extends CommonCanvasElement {
 @Override
 public void onDraw(Canvas canvas) {
 canvas.drawRect(mBounds, mBgPaint);
+canvas.drawRect(mBounds, mFramePaint);
 canvas.drawText(mText, mBounds.centerX(), mBounds.centerY() - 
mTextBounds.centerY(), mTextPaint);
 }
 }


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-29 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java |7 
++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit f4d9f597d1192a688900fda7f11f75c5804196ce
Author: Michael Weghorn 
AuthorDate: Mon Nov 27 10:55:51 2023 +0100
Commit: Xisco Fauli 
CommitDate: Wed Nov 29 13:42:45 2023 +0100

tdf#158398 android: Center Calc header text

Center the header text in the Calc header cell, to
make better use of the space. This is in line with
what the desktop version also does.

Setting the text alignment to `Paint.Align.CENTER`
is sufficient for centering horizontally.
There's no equivalent for centering vertically, so
calculate the position based on the text bounds.

Change-Id: Ia8d5d8434b703cb7daecd34ae70405883f22f0d5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159988
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
Signed-off-by: Xisco Fauli 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160025

diff --git a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java 
b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
index 1b2db6f65796..eec0b5e4a88d 100644
--- a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
+++ b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
@@ -4,6 +4,7 @@ import android.graphics.Canvas;
 import android.graphics.Color;
 import android.graphics.Paint;
 import android.graphics.Paint.Style;
+import android.graphics.Rect;
 import android.graphics.RectF;
 import android.text.TextPaint;
 
@@ -11,6 +12,7 @@ public class CalcHeaderCell extends CommonCanvasElement {
 private final TextPaint mTextPaint = new TextPaint();
 private final Paint mBgPaint = new Paint();
 private final RectF mBounds;
+private final Rect mTextBounds = new Rect();
 private final String mText;
 
 public CalcHeaderCell(float left, float top, float width, float height, 
String text, boolean selected) {
@@ -26,7 +28,10 @@ public class CalcHeaderCell extends CommonCanvasElement {
 mBgPaint.setAlpha(100);  // hard coded for now
 mTextPaint.setColor(Color.GRAY);
 mTextPaint.setTextSize(24f); // hard coded for now
+mTextPaint.setTextAlign(Paint.Align.CENTER);
 mText = text;
+
+mTextPaint.getTextBounds(mText, 0, mText.length(), mTextBounds);
 }
 
 /**
@@ -49,6 +54,6 @@ public class CalcHeaderCell extends CommonCanvasElement {
 @Override
 public void onDraw(Canvas canvas) {
 canvas.drawRect(mBounds, mBgPaint);
-canvas.drawText(mText, mBounds.left, mBounds.bottom, mTextPaint);
+canvas.drawText(mText, mBounds.centerX(), mBounds.centerY() - 
mTextBounds.centerY(), mTextPaint);
 }
 }


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-29 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java |   10 
+-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit e30ed7546869a4111e659cf01e1fd8d200340032
Author: Michael Weghorn 
AuthorDate: Mon Nov 27 10:49:32 2023 +0100
Commit: Xisco Fauli 
CommitDate: Wed Nov 29 13:41:51 2023 +0100

android: Make CalcHeaderCell fields final

Change-Id: I7e14145569428a2803f9376cf719e524aa8963a8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159987
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
Signed-off-by: Xisco Fauli 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160024

diff --git a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java 
b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
index c1f8e74e7ba2..1b2db6f65796 100644
--- a/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
+++ b/android/source/src/java/org/libreoffice/canvas/CalcHeaderCell.java
@@ -8,10 +8,10 @@ import android.graphics.RectF;
 import android.text.TextPaint;
 
 public class CalcHeaderCell extends CommonCanvasElement {
-private TextPaint mTextPaint = new TextPaint();
-private Paint mBgPaint = new Paint();
-private RectF mBounds;
-private String mText;
+private final TextPaint mTextPaint = new TextPaint();
+private final Paint mBgPaint = new Paint();
+private final RectF mBounds;
+private final String mText;
 
 public CalcHeaderCell(float left, float top, float width, float height, 
String text, boolean selected) {
 mBounds = new RectF(left, top, left + width, top + height);
@@ -51,4 +51,4 @@ public class CalcHeaderCell extends CommonCanvasElement {
 canvas.drawRect(mBounds, mBgPaint);
 canvas.drawText(mText, mBounds.left, mBounds.bottom, mTextPaint);
 }
-}
\ No newline at end of file
+}


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-24 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 10b76c9578c6a90a5d0e48a0cb9f18c2d9d68e84
Author: Michael Weghorn 
AuthorDate: Thu Nov 23 11:35:03 2023 +0100
Commit: Xisco Fauli 
CommitDate: Fri Nov 24 16:30:14 2023 +0100

tdf#158331 android: Fix "Save As" for non-experimental mode

While "Save" (i.e. saving back to the original location)
doesn't make sense when editing is disabled (because the
doc cannot be changed) and the menu entry is therefore
not available when the experimental editing mode is
disabled, "Save As" does make sense, e.g. in order to
save a copy of a file opened from another app.

The menu entry was there, but not working as expected,
a 0-byte file was created.

This is because `LibreOfficeMainActivity#saveFileToOriginalSource`
would return early if experimental mode is disabled.
No longer do that, but save the file as requested.

Change-Id: I5785b6060c4ba9cdf3e9c3591b9f941ab987bf4f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159857
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit 5b1538d92e947f3afd1b4815c9c4f4d7565278dd)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159765
Reviewed-by: Xisco Fauli 

diff --git 
a/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java 
b/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
index 54418f5cbc8f..b21ad02383d7 100644
--- a/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
+++ b/android/source/src/java/org/libreoffice/LibreOfficeMainActivity.java
@@ -416,7 +416,7 @@ public class LibreOfficeMainActivity extends 
AppCompatActivity implements Settin
 }
 
 public void saveFileToOriginalSource() {
-if (isReadOnlyMode() || mTempFile == null || mDocumentUri == null || 
!mDocumentUri.getScheme().equals(ContentResolver.SCHEME_CONTENT))
+if (mTempFile == null || mDocumentUri == null || 
!mDocumentUri.getScheme().equals(ContentResolver.SCHEME_CONTENT))
 return;
 
 boolean copyOK = false;


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-23 Thread Michael Weghorn (via logerrit)
 android/source/res/layout/activity_main.xml |4 ++--
 android/source/res/values/dimens.xml|1 -
 2 files changed, 2 insertions(+), 3 deletions(-)

New commits:
commit b5013b9d79270522d8b121258496373abe5f2d45
Author: Michael Weghorn 
AuthorDate: Wed Nov 22 09:27:45 2023 +0100
Commit: Michael Stahl 
CommitDate: Thu Nov 23 11:39:38 2023 +0100

tdf#158307 android a11y: Auto-detect toolbar height

Instead of hard-coding a toolbar height for the
cell address and formula bar in Calc, use
`android:layout_height="wrap_content"`, so the height
is determined based on the font size.
This avoids the text getting cut off when a large
font size has been selected in Android's accessibility
settings.

Change-Id: I9bfbe2b7a1db39ac11d9ed0ab15290f974782489
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159809
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit abed55b1ea7f04a0a522d54e369d170a71045cd5)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159757
Reviewed-by: Michael Stahl 

diff --git a/android/source/res/layout/activity_main.xml 
b/android/source/res/layout/activity_main.xml
index bd444f9fe79f..c03f0c93aea9 100644
--- a/android/source/res/layout/activity_main.xml
+++ b/android/source/res/layout/activity_main.xml
@@ -42,7 +42,7 @@
 256dp
 48dp
 24dp
-40dp
 96dp
 -11dp
 -7dp


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-17 Thread Michael Weghorn (via logerrit)
 android/source/res/drawable/calc.png|binary
 android/source/res/drawable/draw.png|binary
 android/source/res/drawable/impress.png |binary
 android/source/res/drawable/writer.png  |binary
 4 files changed

New commits:
commit f7de490b2901a872eb6830cc26f4423741fc65a8
Author: Michael Weghorn 
AuthorDate: Fri Nov 17 14:30:19 2023 +0100
Commit: Adolfo Jayme Barrientos 
CommitDate: Sat Nov 18 08:53:11 2023 +0100

android: Update app-specific/MIME type icons

Just as

commit 2105f638fa178f49210116bd914889599930b62a
Author: Michael Weghorn 
Date:   Fri Nov 17 09:11:42 2023 +0100

android: Update app icon to new startcenter icon

updated the main icon for the Android app, also update the
MIME type icons that are shown in the "Recent files" section
in LibreOfficeUIActivity to the new ones meant to be used
for Calc/Draw/Impress/Writer since 7.5.

Change-Id: I1d969a290caa3c23589e78151cd5bf70144c3099
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159568
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit cfabbffa14bc98a93fe63b96f0849fd33a66039d)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159576
Reviewed-by: Adolfo Jayme Barrientos 

diff --git a/android/source/res/drawable/calc.png 
b/android/source/res/drawable/calc.png
index 0c0d31444d11..240da6c8966d 100644
Binary files a/android/source/res/drawable/calc.png and 
b/android/source/res/drawable/calc.png differ
diff --git a/android/source/res/drawable/draw.png 
b/android/source/res/drawable/draw.png
index b3ee11426a04..8b8436789cc2 100644
Binary files a/android/source/res/drawable/draw.png and 
b/android/source/res/drawable/draw.png differ
diff --git a/android/source/res/drawable/impress.png 
b/android/source/res/drawable/impress.png
index 5909f05bf089..2de82d6bd42b 100644
Binary files a/android/source/res/drawable/impress.png and 
b/android/source/res/drawable/impress.png differ
diff --git a/android/source/res/drawable/writer.png 
b/android/source/res/drawable/writer.png
index 8a4e21e47174..6cc9e8483033 100644
Binary files a/android/source/res/drawable/writer.png and 
b/android/source/res/drawable/writer.png differ


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-10 Thread Michael Weghorn (via logerrit)
 android/source/build.gradle |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 1a39ca85650b5a0c9a3d3ec0b5539d14a60c6fcc
Author: Michael Weghorn 
AuthorDate: Tue Nov 7 17:27:49 2023 +0100
Commit: Xisco Fauli 
CommitDate: Sat Nov 11 00:01:53 2023 +0100

android: Don't add "-editing" suffix to version/vendor

Don't append "-editing" to the version string, which
would then end up to be something like
"24.2.0.0.alpha0+/7763190f7ec2/The Document Foundation-editing"
when the build was configured with
`--with-vendor="The Document Foundation"` and would result
in the "About" dialog/widget saying
"This release was supplied by The Document Foundation-editing."
(s. `AboutDialogFragment#onCreateDialog`).

While it's useful to have the different build flavors
(with or without the experimental editing support), I see no need
to have the build config reflected in the version string.
(Whether experimental editing support was enabled during the
build can easily be seen by checking whether the "Experimental
Mode" option is available in the settings.)

Change-Id: I48ddb3a842b9db4fc1f435683ed9d8e8e85898ed
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159079
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit b51a57e5ef15c9b09b8cda0a950231d234ae931b)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159036
Reviewed-by: Xisco Fauli 

diff --git a/android/source/build.gradle b/android/source/build.gradle
index d4776c88de90..e931124577b1 100644
--- a/android/source/build.gradle
+++ b/android/source/build.gradle
@@ -82,7 +82,6 @@ android {
 strippedUIEditing {
 dimension "default"
 buildConfigField 'boolean', 'ALLOW_EDITING', 'true'
-versionNameSuffix "-editing"
 }
 fullUI.dimension "default"
 }


[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - android/source

2023-11-10 Thread Michael Weghorn (via logerrit)
 android/source/src/java/org/mozilla/gecko/gfx/GLController.java |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 0a47e3d1034282853fb3b61a3b3bcf31f536c2de
Author: Michael Weghorn 
AuthorDate: Thu Nov 9 08:54:08 2023 +0100
Commit: Michael Stahl 
CommitDate: Fri Nov 10 10:29:44 2023 +0100

tdf#158125 android: Don't insist on RGB 565 EGL config

As the `eglChooseConfig` doc [1] says:

> eglChooseConfig returns in configs a list of all EGL frame buffer
> configurations that match the attributes specified
> [...]
> Attributes are matched in an attribute-specific manner. Some of the
> attributes, such as EGL_LEVEL, must match the specified value exactly.
> Others, such as, EGL_RED_SIZE must meet or exceed the specified minimum
> values.

The config/attribute list used for Android Viewer specifies
EGL_RED_SIZE=5, EGL_GREEN_SIZE=6, and EGL_BLUE_SIZE=5 and so
far, only configs using exactly those bit sizes were accepted,
causing 1 of the 11 devices used in automated tests in Google Play CI
crashing with this stack trace:

Exception org.mozilla.gecko.gfx.GLController$GLControllerException: No 
suitable EGL configuration found
  at org.mozilla.gecko.gfx.GLController.chooseConfig 
(GLController.java:219)
  at org.mozilla.gecko.gfx.GLController.initEGL (GLController.java:172)
  at org.mozilla.gecko.gfx.GLController.initEGLContext 
(GLController.java:176)
  at org.mozilla.gecko.gfx.GLController.initGLContext 
(GLController.java:57)
  at org.mozilla.gecko.gfx.RenderControllerThread.doSurfaceCreated 
(RenderControllerThread.java:132)
  at org.mozilla.gecko.gfx.RenderControllerThread.execute 
(RenderControllerThread.java:52)
  at org.mozilla.gecko.gfx.RenderControllerThread.run 
(RenderControllerThread.java:30)

Since only configs fulfilling the minimium specification
have been returned, I don't see a reason to insist on
having one that uses exactly the specified amount of bits
for the individual color components.

I also didn't see any rendering issues in a quick test (also using
the colorful Calc sheet attachment 188343 from tdf#156182) forcing the
use of a configuration using EGL_RED_SIZE=8, EGL_GREEN_SIZE=8, and
EGL_BLUE_SIZE=8 with an x86_64 AVD and on a Fairphone 3+ (arm64)
using this temporary local change:

diff --git 
a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java 
b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
index 45600e9f1e7c..9e7f348e9e72 100644
--- a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
+++ b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
@@ -171,7 +171,7 @@ public class GLController {
 mEGL.eglGetConfigAttrib(mEGLDisplay, config, 
EGL10.EGL_RED_SIZE, red);
 mEGL.eglGetConfigAttrib(mEGLDisplay, config, 
EGL10.EGL_GREEN_SIZE, green);
 mEGL.eglGetConfigAttrib(mEGLDisplay, config, 
EGL10.EGL_BLUE_SIZE, blue);
-if (red[0] == 5 && green[0] == 6 && blue[0] == 5) {
+if (red[0] == 8 && green[0] == 8 && blue[0] == 8) {
 return config;
 }
 }

Therefore, fall back to using another config that fulfils the
specification.
(Leave the previously required config as preferred one for now,
maybe it still has advantages, e.g. might be more efficient due
to not wasting extra bits for the color components that are
not needed for the rendering in LibreOffice Viewer. (?))

[1] https://registry.khronos.org/EGL/sdk/docs/man/html/eglChooseConfig.xhtml

Change-Id: I953d292248004bc6f7e9384ceef78c8a88c21e9e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159204
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 
(cherry picked from commit edc2dc80e1f6f7594722cd79ac6a930352252c90)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159044
Reviewed-by: Michael Stahl 

diff --git a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java 
b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
index e296f4760f68..3dcfbbdfdbff 100644
--- a/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
+++ b/android/source/src/java/org/mozilla/gecko/gfx/GLController.java
@@ -216,7 +216,8 @@ public class GLController {
 }
 }
 
-throw new GLControllerException("No suitable EGL configuration found");
+// if there's no 565 RGB configuration, select another one that 
fulfils the specification
+return configs[0];
 }
 
 private void createEGLSurface() {