New issue
Advanced search Search tips

Issue 702360 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

gfx_unittests --single-process-tests fails on linux

Project Member Reported by petermayo@chromium.org, Mar 16 2017

Issue description

...
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] TextEliderTest.TestFilenameEliding
[  FAILED  ] RenderTextHarfBuzzTest.Multiline_SurrogatePairsOrCombiningChars/HarfBuzz, where GetParam() = 0


whereas run with no arguments it passes.
Also happens under rr, because: single process.

Still debugging ...
 
./out/Debug/gfx_unittests --single-process-tests --gtest_filter="PlatformFontLinuxTest.*:TextEliderTest.TestFilenameEliding" fails.

Looks like PlatformFontLinuxTest.DefaultFont changes a default/prerequisite font for TestFilenameEliding
TextEliderTest.TestFilenameEliding is a conflict on the Default Font List.
RenderTextHarfBuzzTest.Multiline_SurrogatePairsOrCombiningChars/HarfBuzz, where GetParam() = 0 is also a result of the change in the default font.
Status: Started (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce6b684e85eb02e3ff92e9b5e84fe2cfd1908f8f

commit ce6b684e85eb02e3ff92e9b5e84fe2cfd1908f8f
Author: petermayo <petermayo@chromium.org>
Date: Fri Mar 17 00:30:49 2017

Reset Default font on exit from PlatformFontLinuxTest.

Without this we have an order dependency on other unit tests
in gfx_unittests in the same process.

BUG= 702360 

Review-Url: https://codereview.chromium.org/2760463003
Cr-Commit-Position: refs/heads/master@{#457625}

[modify] https://crrev.com/ce6b684e85eb02e3ff92e9b5e84fe2cfd1908f8f/ui/gfx/platform_font_linux_unittest.cc

Status: Verified (was: Started)

Sign in to add a comment