author MilindL <i.milind.luthra@gmail.com>

Wed, 04 Oct 2017 13:50:25 +0530

changeset 700539 df80df559ca8bfee012d4c3a01dbd8e7021ca939

parent 685447 967c95cee709756596860ed2a3e6ac06ea3a053f

child 700540 f40421807d8390b32979e8e3b6737f52056b0222

child 700542 24008650188fb354a08a2df22d2f2382aa84f479

push id 89886

push user bmo:i.milind.luthra@gmail.com

push date Mon, 20 Nov 2017 10:17:19 +0000

reviewers mak

bugs 1293445

milestone 58.0a1

toolkit/components/places/Database.cpp file | annotate | diff | comparison | revisions

toolkit/components/places/Database.h file | annotate | diff | comparison | revisions

toolkit/components/places/tests/head_common.js file | annotate | diff | comparison | revisions

toolkit/components/places/tests/migration/places_v41.sqlite file | annotate | diff | comparison | revisions

toolkit/components/places/tests/migration/test_current_from_v41.js file | annotate | diff | comparison | revisions

toolkit/components/places/tests/migration/xpcshell.ini file | annotate | diff | comparison | revisions
--- a/toolkit/components/places/Database.cpp
+++ b/toolkit/components/places/Database.cpp
@@ -5,30 +5,32 @@
 #include "mozilla/ArrayUtils.h"
 #include "mozilla/Attributes.h"
 #include "mozilla/DebugOnly.h"
 #include "mozilla/ScopeExit.h"
 
 #include "Database.h"
 
 #include "nsIAnnotationService.h"
+#include "nsClassHashtable.h"
 #include "nsINavBookmarksService.h"
 #include "nsIInterfaceRequestorUtils.h"
 #include "nsIFile.h"
 #include "nsIWritablePropertyBag2.h"
 
 #include "nsNavHistory.h"
 #include "nsPlacesTables.h"
 #include "nsPlacesIndexes.h"
 #include "nsPlacesTriggers.h"
 #include "nsPlacesMacros.h"
 #include "nsVariant.h"
 #include "SQLFunctions.h"
 #include "Helpers.h"
 #include "nsFaviconService.h"
+#include "nsNavHistoryQuery.h"
 
 #include "nsAppDirectoryServiceDefs.h"
 #include "nsDirectoryServiceUtils.h"
 #include "prenv.h"
 #include "prsystem.h"
 #include "nsPrintfCString.h"
 #include "mozilla/Preferences.h"
 #include "mozilla/Services.h"
@@ -1061,16 +1063,23 @@ Database::InitSchema(bool* aDatabaseMigr
 
       if (currentSchemaVersion < 41) {
         rv = MigrateV41Up();
         NS_ENSURE_SUCCESS(rv, rv);
       }
 
       // Firefox 58 uses schema version 41.
 
+      if (currentSchemaVersion < 42) {
+        rv = MigrateV42Up();
+        NS_ENSURE_SUCCESS(rv, rv);
+      }
+
+      // Firefox 59 uses schema version 42.
+
       // Schema Upgrades must add migration code here.
       // >>> IMPORTANT! <<<
       // NEVER MIX UP SYNC AND ASYNC EXECUTION IN MIGRATORS, YOU MAY LOCK THE
       // CONNECTION AND CAUSE FURTHER STEPS TO FAIL.
       // In case, set a bool and do the async work in the ScopeExit guard just
       // before the migration steps.
 
       rv = UpdateBookmarkRootTitles();
@@ -1810,16 +1819,128 @@ Database::MigrateV41Up() {
   NS_ENSURE_SUCCESS(rv, rv);
   rv = mMainConn->ExecuteSimpleSQL(NS_LITERAL_CSTRING(
     "DROP TABLE IF EXISTS moz_favicons"));
   NS_ENSURE_SUCCESS(rv, rv);
   return NS_OK;
 }
 
 nsresult
+Database::MigrateV42Up()
+{
+  MOZ_ASSERT(NS_IsMainThread());
+  nsCOMPtr<mozIStorageStatement> placesQueryStmt;
+  nsresult rv = mMainConn->CreateStatement(NS_LITERAL_CSTRING(
+    "SELECT id, url FROM moz_places "
+    "WHERE (url_hash BETWEEN hash(\"place\", \"prefix_lo\") AND "
+                            "hash(\"place\", \"prefix_hi\")) AND "
+    "url LIKE \"place:%type=7%\""
+  ), getter_AddRefs(placesQueryStmt));
+  NS_ENSURE_SUCCESS(rv, rv);
+
+  bool hasMoreQueries = false;
+  nsDataHashtable<nsUint32HashKey, nsCString> idURLMap;
+  while (NS_SUCCEEDED(rv = placesQueryStmt->ExecuteStep(&hasMoreQueries)) && hasMoreQueries) {
+    nsCString queryUrl;
+    int64_t queryId;
+    placesQueryStmt->GetInt64(0, &queryId);
+    placesQueryStmt->GetUTF8String(1, queryUrl);
+    idURLMap.Put((uint32_t) queryId, queryUrl);
+  }
+
+  if (idURLMap.Count() == 0) {
+    // Quit early, otherwise the replacementParamsArray will be empty causing
+    // BindParameters to fail.
+    return NS_OK;
+  }
+
+  nsNavHistory* historyService = nsNavHistory::GetHistoryService();
+  nsCOMPtr<mozIStorageAsyncStatement> replacementStmt;
+  nsCOMPtr<mozIStorageBindingParamsArray> replacementParamsArray;
+  rv = mMainConn->CreateAsyncStatement(NS_LITERAL_CSTRING(
+    "UPDATE moz_places SET url=:new_url,"
+    "url_hash=hash(:new_url) "
+    "WHERE id=:id"
+  ), getter_AddRefs(replacementStmt));
+  NS_ENSURE_SUCCESS(rv, rv);
+  rv = replacementStmt->NewBindingParamsArray(getter_AddRefs(replacementParamsArray));
+  NS_ENSURE_SUCCESS(rv, rv);
+
+  nsCOMPtr<mozIStorageStatement> tagStmt;
+  bool hasMoreTags;
+  nsString tagName;
+  for (auto iter = idURLMap.Iter(); !iter.Done(); iter.Next()) {
+    nsCString queryString = iter.UserData();
+    uint32_t queryId = iter.Key();
+    nsCOMArray<nsNavHistoryQuery> queries;
+    nsCOMPtr<nsNavHistoryQueryOptions> options;
+    historyService->QueryStringToQueryArray(queryString, &queries, getter_AddRefs(options));
+
+    for (uint32_t i = 0; i < queries.Length(); i++) {
+      uint16_t resultType;
+      rv = options->GetResultType(&resultType);
+      NS_ENSURE_SUCCESS(rv, rv);
+      if (resultType != nsNavHistoryQueryOptions::RESULTS_AS_TAG_CONTENTS) {
+        continue;
+      }
+
+      uint32_t folderCount;
+      int64_t* folders;
+      nsTArray<nsString> associatedTags;
+      queries[i]->GetFolders(&folderCount, &folders);
+      // The folder count should be exactly one for RESULTS_AS_TAG_CONTENTS.
+      MOZ_ASSERT(folderCount == 1);
+      queries[i]->SetFolders(nullptr, 0);
+
+      rv = mMainConn->CreateStatement(NS_LITERAL_CSTRING(
+        "SELECT b.title FROM moz_bookmarks b "
+        "JOIN moz_bookmarks h ON b.parent = h.id "
+        "WHERE b.id = :folder_id AND h.guid = \"tags________\""
+      ), getter_AddRefs(tagStmt));
+      NS_ENSURE_SUCCESS(rv, rv);
+      rv = tagStmt->BindInt64ByName(NS_LITERAL_CSTRING("folder_id"), folders[0]);
+      NS_ENSURE_SUCCESS(rv, rv);
+      rv = tagStmt->ExecuteStep(&hasMoreTags);
+      if (!hasMoreTags || NS_FAILED(rv)) {
+        // If the associated tag isn't found, don't rewrite the query.
+        continue;
+      }
+
+      rv = tagStmt->GetString(0, tagName);
+      NS_ENSURE_SUCCESS(rv, rv);
+      ToLowerCase(tagName);
+      associatedTags.AppendElement(tagName);
+      queries[i]->SetTags(associatedTags);
+    }
+    options->SetResultType(nsINavHistoryQueryOptions::RESULTS_AS_URI);
+    historyService->QueriesToQueryString((nsINavHistoryQuery**)queries.Elements(),
+                                         queries.Length(), options, queryString);
+
+    nsCOMPtr<mozIStorageBindingParams> replacementParams;
+    rv = replacementParamsArray->NewBindingParams(getter_AddRefs(replacementParams));
+    NS_ENSURE_SUCCESS(rv, rv);
+    rv = replacementParams->BindInt64ByName(NS_LITERAL_CSTRING("id"), queryId);
+    NS_ENSURE_SUCCESS(rv, rv);
+    rv = replacementParams->BindUTF8StringByName(NS_LITERAL_CSTRING("new_url"),
+                                                 queryString);
+    NS_ENSURE_SUCCESS(rv, rv);
+    rv = replacementParamsArray->AddParams(replacementParams);
+    NS_ENSURE_SUCCESS(rv, rv);
+  }
+
+  rv = replacementStmt->BindParameters(replacementParamsArray);
+  NS_ENSURE_SUCCESS(rv, rv);
+  nsCOMPtr<mozIStoragePendingStatement> pendingStmt;
+  rv = replacementStmt->ExecuteAsync(nullptr, getter_AddRefs(pendingStmt));
+  NS_ENSURE_SUCCESS(rv, rv);
+
+  return NS_OK;
+}
+
+nsresult
 Database::GetItemsWithAnno(const nsACString& aAnnoName, int32_t aItemType,
                            nsTArray<int64_t>& aItemIds)
 {
   nsCOMPtr<mozIStorageStatement> stmt;
   nsresult rv = mMainConn->CreateStatement(NS_LITERAL_CSTRING(
     "SELECT b.id FROM moz_items_annos a "
     "JOIN moz_anno_attributes n ON n.id = a.anno_attribute_id "
     "JOIN moz_bookmarks b ON b.id = a.item_id "
--- a/toolkit/components/places/Database.h
+++ b/toolkit/components/places/Database.h
@@ -14,17 +14,17 @@
 #include "mozilla/storage/StatementCache.h"
 #include "mozilla/Attributes.h"
 #include "nsIEventTarget.h"
 #include "Shutdown.h"
 #include "nsCategoryCache.h"
 
 // This is the schema version. Update it at any schema change and add a
 // corresponding migrateVxx method below.
-#define DATABASE_SCHEMA_VERSION 41
+#define DATABASE_SCHEMA_VERSION 42
 
 // Fired after Places inited.
 #define TOPIC_PLACES_INIT_COMPLETE "places-init-complete"
 // This topic is received when the profile is about to be lost.  Places does
 // initial shutdown work and notifies TOPIC_PLACES_SHUTDOWN to all listeners.
 // Any shutdown work that requires the Places APIs should happen here.
 #define TOPIC_PROFILE_CHANGE_TEARDOWN "profile-change-teardown"
 // Fired when Places is shutting down.  Any code should stop accessing Places
@@ -286,16 +286,17 @@ protected:
   nsresult MigrateV34Up();
   nsresult MigrateV35Up();
   nsresult MigrateV36Up();
   nsresult MigrateV37Up();
   nsresult MigrateV38Up();
   nsresult MigrateV39Up();
   nsresult MigrateV40Up();
   nsresult MigrateV41Up();
+  nsresult MigrateV42Up();
 
   nsresult UpdateBookmarkRootTitles();
 
   friend class ConnectionShutdownBlocker;
 
   int64_t CreateMobileRoot();
   nsresult GetItemsWithAnno(const nsACString& aAnnoName, int32_t aItemType,
                             nsTArray<int64_t>& aItemIds);
--- a/toolkit/components/places/tests/head_common.js
+++ b/toolkit/components/places/tests/head_common.js
@@ -1,17 +1,17 @@
 /* -*- indent-tabs-mode: nil; js-indent-level: 2 -*-
  * This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 // It is expected that the test files importing this file define Cu etc.
 /* global Cu, Ci, Cc, Cr */
 
-const CURRENT_SCHEMA_VERSION = 41;
+const CURRENT_SCHEMA_VERSION = 42;
 const FIRST_UPGRADABLE_SCHEMA_VERSION = 30;
 
 const NS_APP_USER_PROFILE_50_DIR = "ProfD";
 const NS_APP_PROFILE_DIR_STARTUP = "ProfDS";
 
 // Shortcuts to transitions type.
 const TRANSITION_LINK = Ci.nsINavHistoryService.TRANSITION_LINK;
 const TRANSITION_TYPED = Ci.nsINavHistoryService.TRANSITION_TYPED;
new file mode 100644
--- /dev/null
+++ b/toolkit/components/places/tests/migration/test_current_from_v41.js
@@ -0,0 +1,187 @@
+/* Any copyright is dedicated to the Public Domain.
+   http://creativecommons.org/publicdomain/zero/1.0/ */
+
+const { TYPE_BOOKMARK, TYPE_FOLDER } = Ci.nsINavBookmarksService;
+
+// Hashes are precalculated since URL generated each time is a constant.
+// They will need to be recalculated in case the test is changed.
+const gQueryData = [
+  {
+    statement: `INSERT INTO moz_places (url, guid, url_hash)
+                SELECT  'place:folder='||b.id||'&type=7&queryType=1', 'query1______', 268507487492160
+                FROM moz_bookmarks b
+                WHERE b.guid = 'tag1________';`,
+    guid: "query1______",
+    oldQueryString: null,
+    newQueryString: null
+  },
+  {
+    statement: `INSERT INTO moz_places (url, guid, url_hash)
+                SELECT  'place:folder='
+                ||(SELECT id FROM moz_bookmarks WHERE guid = 'tag1________')||
+                '&type=7&queryType=1&OR&folder='
+                ||(SELECT id FROM moz_bookmarks WHERE guid = 'tag2________')||
+                '&type=7&queryType=1', 'query2______', 268505990850137`,
+    guid: "query2______",
+    oldQueryString: null,
+    newQueryString: null
+  }
+];
+
+function makeTagGuid(tagName) {
+  // Be careful about having |tagName| > 12, since the guids
+  // will be unique only if tagName[0:12] is unique.
+  if (tagName.length >= 12) {
+    return tagName.slice(0, 12);
+  }
+
+  return tagName + "_".repeat(12 - tagName.length);
+}
+
+async function getIdForTag(tagName) {
+  return PlacesUtils.promiseItemId(makeTagGuid(tagName));
+}
+
+function copyPropertiesExcept(obj, except) {
+  let newObj = {};
+  for (const property in obj) {
+    if (obj.hasOwnProperty(property) &&
+        !except.includes(property) &&
+        typeof(obj[property]) !== "function") {
+      newObj[property] = obj[property];
+    }
+  }
+  return newObj;
+}
+
+async function insertPlacesQuery(db, qData) {
+  // Note that we cannot use Places.history here.
+  await db.execute(qData.statement);
+  const rows = await db.execute(`SELECT url FROM moz_places WHERE guid=:guid`,
+                                { guid: qData.guid });
+  Assert.equal(rows.length, 1);
+  qData.oldQueryString = rows[0].getResultByName("url");
+}
+
+async function validateMigration(qData) {
+  qData.newQueryString = (await PlacesUtils.history.fetch(qData.guid)).url;
+  do_print(`Old query string: ${qData.oldQueryString}`);
+  do_print(`New query string: ${qData.newQueryString}`);
+  let oldQueries = {}, oldOptions = {};
+  PlacesUtils.history.queryStringToQueries(qData.oldQueryString,
+                                           oldQueries,
+                                           { },
+                                           oldOptions);
+  let newQueries = {}, newOptions = {};
+  PlacesUtils.history.queryStringToQueries(qData.newQueryString,
+                                           newQueries,
+                                           { },
+                                           newOptions);
+
+  // Check options.
+  // Query types should be different.
+  Assert.equal(oldOptions.value.resultType, 7, "Old result type is 7.");
+  Assert.equal(newOptions.value.resultType, 0, "New result type is 0.");
+  // Rest of the options object should be the same.
+  Assert.deepEqual(copyPropertiesExcept(oldOptions.value, ["resultType"]),
+                   copyPropertiesExcept(newOptions.value, ["resultType"]),
+                   "Options are identical except for resultType.");
+
+  // Check queries.
+  Assert.equal(oldQueries.value.length, newQueries.value.length, "Number of queries is same.");
+  for (let i = 0; i < oldQueries.value.length; i++) {
+    // Number of tags == folderCount == 1.
+    // Folder ids should be replaced by correct tags.
+    Assert.equal(newQueries.value[i].tags.length, 1);
+    Assert.equal(oldQueries.value[i].getFolders().length, 1);
+    const tagName = newQueries.value[i].tags[0];
+    const folderId = oldQueries.value[i].getFolders()[0];
+    Assert.equal(folderId, (await getIdForTag(tagName)));
+
+    // Rest of the query object should be the same.
+    Assert.deepEqual(copyPropertiesExcept(oldQueries.value[i], ["folderCount", "tags"]),
+                     copyPropertiesExcept(newQueries.value[i], ["folderCount", "tags"]),
+                     "Query/Queries is/are identical except for folderCount and tags.");
+  }
+
+
+  // Query result URIs should be identical.
+  const oldRoot = PlacesUtils.history.executeQueries(oldQueries.value,
+                                                     oldQueries.value.length,
+                                                     oldOptions.value).root;
+  const newRoot = PlacesUtils.history.executeQueries(newQueries.value,
+                                                     newQueries.value.length,
+                                                     newOptions.value).root;
+
+  oldRoot.containerOpen = true;
+  newRoot.containerOpen = true;
+  const oldUris = [];
+  const newUris = [];
+  Assert.equal(oldRoot.childCount, newRoot.childCount, "Number of results is the same.");
+  for (let i = 0; i < oldRoot.childCount; i++) {
+    oldUris.push(oldRoot.getChild(i).uri);
+    newUris.push(newRoot.getChild(i).uri);
+  }
+
+  for (const uri of oldUris) {
+    Assert.ok(newUris.includes(uri));
+  }
+}
+
+add_task(async function setup() {
+  await setupPlacesDatabase("places_v41.sqlite");
+  let path = OS.Path.join(OS.Constants.Path.profileDir, DB_FILENAME);
+  let db = await Sqlite.openConnection({ path });
+  // Add pages.
+  await db.execute(`INSERT INTO moz_places (url, guid, rev_host, hidden, frecency, url_hash)
+                    VALUES ("http://test1.com/", "test1_______", "moc.1tset.", 0, -1, 125509083372812)
+                         , ("http://test2.com/", "test2_______", "moc.2tset.", 0, -1, 125508024173255)
+                         , ("http://test3.com/", "test3_______", "moc.3tset.", 0, -1, 125511593739455)
+                   `);
+  // Add bookmarks.
+  let now = Date.now() * 1000;
+  let index = 0;
+  await db.execute(`INSERT INTO moz_bookmarks (type, fk, parent, position, dateAdded, lastModified, guid)
+                    VALUES (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test1_______'), (SELECT id FROM moz_places WHERE guid = 'toolbar_____'), ${index++}, ${now}, ${now}, "bookmark1___")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test2_______'), (SELECT id FROM moz_places WHERE guid = 'toolbar_____'), ${index++}, ${now}, ${now}, "bookmark2___")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test3_______'), (SELECT id FROM moz_places WHERE guid = 'toolbar_____'), ${index++}, ${now}, ${now}, "bookmark3___")
+                   `);
+
+  // Add tag folders.
+  await db.execute(`INSERT INTO moz_bookmarks (type, parent, position, title, dateAdded, lastModified, guid)
+                    VALUES (${TYPE_FOLDER}, (SELECT id FROM moz_bookmarks WHERE guid = 'tags________'), ${index++}, "tag1", ${now}, ${now}, "${makeTagGuid("tag1")}")
+                         , (${TYPE_FOLDER}, (SELECT id FROM moz_bookmarks WHERE guid = 'tags________'), ${index++}, "tag2", ${now}, ${now}, "${makeTagGuid("tag2")}")
+                         , (${TYPE_FOLDER}, (SELECT id FROM moz_bookmarks WHERE guid = 'tags________'), ${index++}, "tag3", ${now}, ${now}, "${makeTagGuid("tag3")}")
+                   `);
+
+  // Add tag-bookmark association
+  await db.execute(`INSERT INTO moz_bookmarks (type, fk, parent, position, dateAdded, lastModified, guid)
+                    VALUES (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test1_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag1________'),${index++}, ${now}, ${now}, "bmkTagAsc1__")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test1_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag2________'),${index++}, ${now}, ${now}, "bmkTagAsc2__")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test1_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag3________'),${index++}, ${now}, ${now}, "bmkTagAsc3__")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test2_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag1________'),${index++}, ${now}, ${now}, "bmkTagAsc4__")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test2_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag2________'),${index++}, ${now}, ${now}, "bmkTagAsc5__")
+                         , (${TYPE_BOOKMARK}, (SELECT id FROM moz_places WHERE guid = 'test3_______'), (SELECT id FROM moz_bookmarks WHERE guid = 'tag1________'),${index++}, ${now}, ${now}, "bmkTagAsc6__")
+                  `);
+
+  // Add Places Queries.
+  for (const queryData of gQueryData) {
+    await insertPlacesQuery(db, queryData);
+  }
+  await db.close();
+});
+
+add_task(async function database_is_valid() {
+  // Accessing the database for the first time triggers migration.
+  Assert.equal(PlacesUtils.history.databaseStatus,
+               PlacesUtils.history.DATABASE_STATUS_UPGRADED);
+
+  let db = await PlacesUtils.promiseDBConnection();
+  Assert.equal((await db.getSchemaVersion()), CURRENT_SCHEMA_VERSION);
+
+  for (const queryData of gQueryData) {
+    await validateMigration(queryData);
+  }
+
+  await db.close();
+});
--- a/toolkit/components/places/tests/migration/xpcshell.ini
+++ b/toolkit/components/places/tests/migration/xpcshell.ini
@@ -3,17 +3,19 @@ head = head_migration.js
 
 support-files =
   places_outdated.sqlite
   places_v31.sqlite
   places_v34.sqlite
   places_v35.sqlite
   places_v36.sqlite
   places_v38.sqlite
+  places_v41.sqlite
 
 [test_current_from_downgraded.js]
 [test_current_from_outdated.js]
 [test_current_from_v31.js]
 [test_current_from_v34.js]
 [test_current_from_v34_no_roots.js]
 [test_current_from_v35.js]
 [test_current_from_v36.js]
 [test_current_from_v38.js]
+[test_current_from_v41.js]
author	MilindL <i.milind.luthra@gmail.com>
	Wed, 04 Oct 2017 13:50:25 +0530
changeset 700539	df80df559ca8bfee012d4c3a01dbd8e7021ca939
parent 685447	967c95cee709756596860ed2a3e6ac06ea3a053f
child 700540	f40421807d8390b32979e8e3b6737f52056b0222
child 700542	24008650188fb354a08a2df22d2f2382aa84f479
push id	89886
push user	bmo:i.milind.luthra@gmail.com
push date	Mon, 20 Nov 2017 10:17:19 +0000
reviewers	mak
bugs	1293445
milestone	58.0a1
toolkit/components/places/Database.cpp		file \| annotate \| diff \| comparison \| revisions
toolkit/components/places/Database.h		file \| annotate \| diff \| comparison \| revisions
toolkit/components/places/tests/head_common.js		file \| annotate \| diff \| comparison \| revisions
toolkit/components/places/tests/migration/places_v41.sqlite		file \| annotate \| diff \| comparison \| revisions
toolkit/components/places/tests/migration/test_current_from_v41.js		file \| annotate \| diff \| comparison \| revisions
toolkit/components/places/tests/migration/xpcshell.ini		file \| annotate \| diff \| comparison \| revisions