/usr/bin/unity8-dash:7:QQuickItem::y:ListViewWithPageHeader::ListItem::y:ListViewWithPageHeader::onModelUpdated:ListViewWithPageHeader::qt_static_metacall:QMetaObject::activate
Bug #1433056 reported by
errors.ubuntu.com bug bridge
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
unity8 (Ubuntu) |
Fix Released
|
High
|
Albert Astals Cid |
Bug Description
The Ubuntu Error Tracker has been receiving reports about a problem regarding unity8. This problem was most recently seen with version 8.02+15.
Related branches
lp://staging/~aacid/unity8/lvwph_firstVisibleIndex_remove
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Michael Zanetti (community): Approve
-
Diff: 47 lines (+18/-8)2 files modifiedplugins/Dash/listviewwithpageheader.cpp (+7/-8)
tests/plugins/Dash/listviewwithpageheadertest.cpp (+11/-0)
Changed in unity8 (Ubuntu): | |
status: | New → Triaged |
importance: | Undecided → High |
assignee: | nobody → Albert Astals Cid (aacid) |
Changed in unity8 (Ubuntu): | |
status: | Triaged → In Progress |
To post a comment you must log in.
I can't prove that the attached branch really fixes the backtrace of this bug but this crash is caused because on line
const bool firstItemWithVi ewOnTop = insert.index == 0 && m_firstVisibleIndex == 0 && m_visibleItems. first() ->y() + m_clipItem->y() > contentY();
insert.index is 0 and m_firstVisibleIndex is 0 but m_visibleItems. first() ->y() crashes (i.e. it's probably empty) and this fixes a case in which m_firstVisibleIndex may get out of sync so I'd say let's merge this fix and let's monitor errors.ubuntu.com to see if it continues or it is indeed fixed by this.