Ubuntu
webbrowser-app package

Regression: QmlTests::BookmarksViewWide::test_delete_bookmark() is now failing

Bug #1526940 reported by Olivier Tilloy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
New
Undecided
Unassigned
webbrowser-app (Ubuntu)
Fix Released
Critical
Olivier Tilloy

Bug Description

A very recent landing (possible ubuntu-ui-toolkit 1.3.1761+15.04.20151216.1-0ubuntu1) seems to have introduced a regression in webbrowser-app builds: the following unit test is now reliably failing on all CI and silo builds:

FAIL! : QmlTests::BookmarksViewWide::test_delete_bookmark() property count
   Actual (): 3
   Expected (): 2
   Loc: [/tmp/buildd/webbrowser-app-0.23+16.04.20151204bzr1300pkg0vivid1295/tests/unittests/qml/tst_BookmarksViewWide.qml(108)]

Related branches

lp:~osomon/webbrowser-app/workaround-1526940
Michael Sheldon (community): Approve
PS Jenkins bot: Needs Fixing (continuous-integration)
Diff: 15 lines (+4/-1)
1 file modified
tests/unittests/qml/WebbrowserTestCase.qml (+4/-1)
Michael Sheldon (michael-sheldon)
Changed in webbrowser-app (Ubuntu):
status: New → Confirmed
Revision history for this message
Olivier Tilloy (osomon) wrote :

Added a ubuntu-ui-toolkit task: we have a workaround that "fixes" the failing unit test, but that very much looks like a regression in the UITK itself. What’s happening is that if we don’t specify a non-null delay as the last parameter of flick(), the list item is swiped to the right but the confirm action is only partially visible, it’s offset by most of its width to the left. Specifying a delay of 1 is enough to trigger the expected behaviour.

Changed in webbrowser-app (Ubuntu):
assignee: nobody → Olivier Tilloy (osomon)
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package webbrowser-app - 0.23+16.04.20151221-0ubuntu1

---------------
webbrowser-app (0.23+16.04.20151221-0ubuntu1) xenial; urgency=medium

  [ Michael Sheldon ]
  * Add support for handling downloads internally within the browser.
    (LP: #1354391)

 -- CI Train Bot <email address hidden> Mon, 21 Dec 2015 20:39:36 +0000

Changed in webbrowser-app (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Zsombor Egri (zsombi) wrote :

Olivier, this is an issue which Andrea also identified, and it's cause by some Qt stuff iirc.

Revision history for this message
Andrea Bernabei (faenil) wrote :

Yes, I extensively debugged that, please see https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1549256

Revision history for this message
Andrea Bernabei (faenil) wrote :

Please remove the workaround once the UITK fix is released :)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.