819942 - Test failure "Default bookmarks have been imported" due to restoreDefaultBookmarks() in lib/places

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect, P1)

Description

[Daniela Petrovici]

A lot of failures appeared with "Default bookmarks have been imported" error message. Reproducible on all platforms:
http://mozmill-ci.blargon7.com/#/functional/report/352218d7e3125c857fc1d37167f162a7
http://mozmill-ci.blargon7.com/#/endurance/report/352218d7e3125c857fc1d37167f1de7f

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Please get this fixed as soon as possible given that it affects all supported platforms for Firefox 20.

Comment 2

[Alex Lakatos[:AlexLakatos]]

Attachment: patch v1.0

there was a change on how BookmarkHTMLUtils.importFromURL is called

Comment 3

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 690399 [details] [diff] [review]
patch v1.0

Review of attachment 690399 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the quick fix Alex.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to Alex Lakatos[:AlexLakatos] from comment #2)
> there was a change on how BookmarkHTMLUtils.importFromURL is called

Please reference the bug id in the dependency as usual for regressions! Thanks.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+2]]

http://hg.mozilla.org/qa/mozmill-tests/rev/818d34639ea1 (default)