The default bug view has changed. See this FAQ.

Port a number of recent places changes

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

unspecified
seamonkey2.1b2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
I have a list of recent places changes in Firefox I need to go through and see to port them to SeaMonkey.
(Assignee)

Updated

7 years ago
Depends on: 424542
(Assignee)

Updated

7 years ago
Depends on: 458026
(Assignee)

Comment 1

7 years ago
Note that bug 574688 is only on the list for the sidebar changes, which are independent of the actual statusbar-realted changes.
Depends on: 574688
(Assignee)

Updated

7 years ago
Depends on: 477882
(Assignee)

Comment 2

7 years ago
Created attachment 489558 [details] [diff] [review]
port those bugs

Due to bug 611001 not having been done, I needed to add the .markPageAsFollowedLink() somewhere else but I seem to have found the right spot, so that the test succeeds. I hope the other changes are easy to verify and not too controversial...
Attachment #489558 - Flags: review?(neil)
(Assignee)

Comment 3

7 years ago
Oh, I forgot to mention that I took the chance to take care of the merging of the sidebarUtils as they have been practically identical for bookmarks and history anyhow.

As a side note, onBeforeLinkTraversal has been introduced for bug 575561, which is an app tab matter, but I saw errors in the console due to this method missing (when executing the markPageAsFollowedLink test), so I implemented it in a dummy version.

Comment 4

6 years ago
Comment on attachment 489558 [details] [diff] [review]
port those bugs

>+  // Called before links are navigated to to allow us to retarget them if needed.
>+  onBeforeLinkTraversal: function(originalTarget, linkURI, linkNode, isAppTab) {
>+    // We doesn't support app tabs yet but are seeing failures without this.
The method was added to nsIXULBrowserWindow.idl so a) the implementation belongs after setOverLink b) we don't need this comment but I guess you could add an nsIXULBrowserWindow heading instead to make it clearer.

>+               .filter(function (tag) tag.length > 0); // Kill empty tags.
[filter(function(tag) tag) would probably work.]
Attachment #489558 - Flags: review?(neil) → review+
(Assignee)

Comment 5

6 years ago
Landed as http://hg.mozilla.org/comm-central/rev/65159580f815
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2

Updated

6 years ago
Blocks: 609034

Updated

6 years ago
Duplicate of this bug: 609034

Updated

6 years ago
Blocks: 631770
You need to log in before you can comment on or make changes to this bug.