Closed Bug 394205 Opened 17 years ago Closed 16 years ago

decouple nsNavHistoryService and bookmarks import

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: dietrich, Assigned: dietrich)

References

Details

Attachments

(2 obsolete files)

Instead of setting the browser.places.importBookmarksHTML pref in the history service (1), we should expose a public property such as nsINavHistoryService.firstRun to indicate bookmarks, history or whatever needs to be imported.


1. http://mxr.mozilla.org/seamonkey/search?string=PREF_BROWSER_IMPORT_BOOKMARKS
Depends on: 384370
Blocks: 384370
No longer depends on: 384370
Assignee: nobody → dietrich
Attached patch fix (obsolete) — Splinter Review
pulled out of the patch for 384370
Attachment #291057 - Flags: review?(sspitzer)
hey dietrich, did this patch bit rot (due to changes from Marco and me) or is it still ready for review?
Attached patch v2 (obsolete) — Splinter Review
unrotted, and updated to take into account the new initialization-related prefs.
Attachment #291057 - Attachment is obsolete: true
Attachment #291057 - Flags: review?(sspitzer)
My testing scenarios:

- new profile, test, restart, test
- branch new profile, test, migrate to current, test
- a2 new profile, test, migrate to current, test

Where "test" is:
- confirm bookmarks and history are not empty
- confirm default bookmarks
- confirm smart bookmarks folder and contents
- confirm Library left pane contents
Status: NEW → ASSIGNED
Attachment #294772 - Flags: review?(sspitzer)
could you test also:

- delete all profile folder by hand and restart browser
- delete places.sqlite by hand and restart browser

this could cause a TS regression due to _initplaces changes
about the first point it's "delete all profile folder CONTENTS", not folder itself
Comment on attachment 294772 [details] [diff] [review]
v2

r=sspitzer, one nit:

+  *aImportBookmarks= mImportBookmarks;

nit:  space before =

But, I share some concerns that Marco raised.

First, please see bug #406114, which unexpectedly impacted Ts / Txul.

Second, see bug #406094, we should confirm that your fix doesn't regress that scenario.

Third, see bug #406833, a spin off of bug #406114.
Attachment #294772 - Flags: review?(sspitzer) → review+
Depends on: 462366
fixed in bug 462366.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
Attachment #294772 - Attachment is obsolete: true
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".

In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body   contains   places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.

Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.

Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: