Closed Bug 1769312 Opened 3 years ago Closed 2 years ago

Replace timer in browser_library_row_is_visible.js with an event

Categories

(Firefox :: Bookmarks & History, defect, P3)

Product:
Firefox
Component:
Bookmarks & History
Type:
defect

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: jteow, Unassigned)

Details

When using the Bookmark Library and selecting an element, the details panel will expand. One example where this happens is if the selection is on a folder and then the user selects a bookmark.

When a bookmark is selected, ensureRowIsVisible is called which scrolls the list down to the appropriate row if it is not visible. Unfortunately, neither MozMousePixelScroll nor DOMMouseScroll are triggered when ensureRowIsVisible scrolls the list. This makes it tricky to determine when to check the row for visibility.

For now, I'm using a timer to pass the tests in browser_library_row_is_visible.js, but it should be replace with an event to be more durable.

Closing it out as invalid since the patch associated with Bug 1786518 removes the test associated with this bug.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.