The default bug view has changed. See this FAQ.

Provide helper function for opening the library in tests

RESOLVED FIXED in Firefox 7

Status

()

Firefox
Bookmarks & History
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

Trunk
Firefox 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 538825 [details] [diff] [review]
patch
Attachment #538825 - Flags: review?(mak77)
(Assignee)

Updated

6 years ago
Attachment #538825 - Flags: review?(mak77) → review?(sdwilsh)
any reason to change reviewer here? I can do this, I'd rather not disturb sdwilsh for this stuff.
(Assignee)

Comment 2

6 years ago
Feel free to steal from Shawn. I just had the impression that you're busy with other stuff, since this waited a few days even though I expected this to be a very quick review.
Comment on attachment 538825 [details] [diff] [review]
patch

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

Sorry sometimes context switching is expensive, I'm running through reviews queue now.

So, I like this, the only doubt I still have is whether we may need an additional executeSoon due to the fact some of the Library trees stuff is inited lazily (especially the details pane overlay). I know waitForFocus does that once and I assume if you ran this through try so it may be enough as it is.
But as a protection vs randomness, I'd prefer it this may land first on Places and bake a couple days. I can push it if you don't want to clone the branch.
Attachment #538825 - Flags: review?(sdwilsh) → review+
(Assignee)

Comment 4

6 years ago
Feel free to land this on places. That tree appears to be in a bad shape, though...
Yes, I was working on that to be able to merge.
I have a local patch that should fix all livemarks tests random failures, I'll need a review on that but then it should go green. Since these changes are on b-c tests and other failures are on chrome-tests it should be fine.

Will land in the next minutes along with your other bug 663630.
Sorry, it took a while to figure out how to make Mercurial correctly handle your name :)
http://hg.mozilla.org/projects/places/rev/3c59da5d57ac
Whiteboard: [fixed-in-places]
http://hg.mozilla.org/mozilla-central/rev/3c59da5d57ac
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 7
Blocks: 725549
(Assignee)

Updated

5 years ago
Whiteboard: [fixed-in-places]
You need to log in before you can comment on or make changes to this bug.