Places/Status Meetings/2009-04-20

« previous week | index | next week »

Places Team Meeting Details

• Mondays at 9:00am Pacific
• irc.mozilla.org #places

Work Priorities

• Blockers (1, 0, http://tinyurl.com/dgbhmn)
  • bug 488783 - Tags list no longer sorted (alphabetized), mak: patch ready but investigating on sorting issues noticed while writing test.
• Needs branch landing (1, -2, http://tinyurl.com/c4droh)
• Wanted (33, 0, http://tinyurl.com/cg78qy)
• Possible Wontfix (41, +1, http://tinyurl.com/ajqgm7)
• Places bugs on m-c but not yet on 1.9.1 (19, -12, http://tinyurl.com/686pp9)
• Waiting for approval1.9.1 (3, -8, http://tinyurl.com/cs2gng)
• Requiring tests (in-testsuite?) (81, -3, http://tinyurl.com/d3eqav)

• Bug Triage (1354, -4 change), http://tinyurl.com/a3j9p6)

Status updates

• sdwilsh
  • Working on some weave-impact bugs for 1.9.1
    • bug 487660 Places depends on buggy behavior of SQLite's SUBSTR (needs branch landing)
    • bug 487511 nsINavHistoryObserver has no "onBeforeDeleteURI" callback (needs review dietrich)
    • bug 468307 Multiple observer events fire for addition, removal (approach changed; needs work)

• mak
  • blocker bug 487040 Crash [@ nsNavHistoryResult::OnItemAdded ]. Landed on trunk and branch, re-enabled crashing test on branch and waiting results
  • bug 484019 Fix roots titles with preventive maintenance. First patch ready, need to evaluate how to handle checks on Library left pane folders.
  • bug 487777 History menu is slow and takes ages to open. Fixed on trunk and branch.
  • Fixed some of the places "sort operation required" storage warnings.
  • various reviews and pushes
  • blocker bug 488783 Tags list no longer sorted (alphabetized) in latest Shiretoko nightly. Patch ready, but i noticed some strange behavior while writing a test, investigating them. ETA should be end of today.
  • 488966 bug Bug 488966 Add a last_visit_date column with an index to moz_places. Wan to investigate if it would be possible fixing this for 3.5, and if really would make history menu lightning fast.

• adw
  • Clear recent history sprint bug 480169 - landed on trunk and branch
  • Weave bookmarks-restore notifications bug 470314 - landed on trunk and branch
  • Library polish bug 430148 - review requested from mano
  • hasTag bug 399799 - put up an nsIVariant patch for Dietrich's review; need to investigate alphabetizing array of tags returned per Marco's comment
  • bookmarks initialization is not fault tolerant bug 478912 - posted a WIP patch that addresses most but not all of the issues linked to in the bug, but the bug is very terse; could use some guidance here, or maybe Marco or Dietrich could use the patch as a starting point since they know what the bug's about

• ddahl
  • Patch up for bug 416580 "Disable copying and other commands on "special" Library folders".
  • still sidelined by bug bug 473337 fixing makefile? for SeaMonkey builds
  • test and patch up for bug 485442, one more test to finish.
  • starting new Places wanted bugs
  • Weekend nerd-out sessions are revealing more nuances of SQLite and Storage:)

• dietrich
  • reviews, etc.

Documentation

• Bugs with dev-doc-needed (http://is.gd/iVWS)

• MDC
  • the migration guide needs history api and dynamic container examples. (dietrich)
  • the design documents linked from MDC need to be written. can probably cull from the old design overview docs.
    • Bookmarks (dietrich, not started)
    • Annotations (sdwilsh, not started)
  • Documentation non-existent:
    • dynamic containers (MaK77)
      • will do as soon as they are working properly

• WMO
  • Places:Live_Bookmarks (decide on owner)
  • need documentation on how to write SQL queries against the places back-end, for core developers (Places/Places_SQL_queries_best_practices)
    • Invariants and triggers (sdwilsh)

• Needs porting from WMO to MDC:
  • Places:Transactions (decide on owner)

Roundtable

• A Places (or non-Places-specific that we use for Places...) dashboard that shows live changes, bugzilla queries, graphs would be nice to have
  • https://wiki.mozilla.org/User:Dietrich/Dashboard
  • http://www.toolness.com/wp/?p=550

• Places team in person meeting during All Hands week: Tracy would like to meet Thursday

• bug 445704 needs to be ported to 3.0.10 branch (freeze is April 21st)