This is an automated email from the git hooks/post-receive script. richard pushed a commit to branch tor-browser-91.8.0esr-11.0-1 in repository tor-browser. commit af2a7d1b0c6621382d73573494872de05c4c9fd5 Author: Emilio Cobos Álvarez <emilio@crisal.io> AuthorDate: Tue Mar 29 12:15:52 2022 +0000 Bug 1760674. r=mak Differential Revision: https://phabricator.services.mozilla.com/D141958 --- docshell/base/BaseHistory.cpp | 13 ------------- dom/base/Link.cpp | 10 +--------- 2 files changed, 1 insertion(+), 22 deletions(-) diff --git a/docshell/base/BaseHistory.cpp b/docshell/base/BaseHistory.cpp index 12ea1d9357231..7aff7607db2c0 100644 --- a/docshell/base/BaseHistory.cpp +++ b/docshell/base/BaseHistory.cpp @@ -116,10 +116,6 @@ void BaseHistory::RegisterVisitedCallback(nsIURI* aURI, Link* aLink) { // This will not catch a case where it is registered for two different URIs. MOZ_DIAGNOSTIC_ASSERT(!links->mLinks.Contains(aLink), "Already tracking this Link object!"); - // FIXME(emilio): We should consider changing this (see the entry.Remove() - // call in NotifyVisitedInThisProcess). - MOZ_DIAGNOSTIC_ASSERT(links->mStatus != VisitedStatus::Visited, - "We don't keep tracking known-visited links"); links->mLinks.AppendElement(aLink); @@ -205,15 +201,6 @@ void BaseHistory::NotifyVisitedInThisProcess(nsIURI* aURI, for (Link* link : links.mLinks.BackwardRange()) { link->VisitedQueryFinished(visited); } - - // We never go from visited -> unvisited. - // - // FIXME(emilio): It seems unfortunate to remove a link to a visited uri and - // then re-add it to the document to trigger a new visited query. It shouldn't - // if we keep track of mStatus. - if (visited) { - entry.Remove(); - } } void BaseHistory::SendPendingVisitedResultsToChildProcesses() { diff --git a/dom/base/Link.cpp b/dom/base/Link.cpp index b139b8260293e..72f00c030d5cd 100644 --- a/dom/base/Link.cpp +++ b/dom/base/Link.cpp @@ -60,25 +60,17 @@ bool Link::ElementHasHref() const { void Link::VisitedQueryFinished(bool aVisited) { MOZ_ASSERT(mRegistered, "Setting the link state of an unregistered Link!"); - MOZ_ASSERT(mState == State::Unvisited, - "Why would we want to know our visited state otherwise?"); auto newState = aVisited ? State::Visited : State::Unvisited; // Set our current state as appropriate. mState = newState; - // We will be no longer registered if we're visited, as it'd be pointless, we - // never transition from visited -> unvisited. - if (aVisited) { - mRegistered = false; - } - MOZ_ASSERT(LinkState() == NS_EVENT_STATE_VISITED || LinkState() == NS_EVENT_STATE_UNVISITED, "Unexpected state obtained from LinkState()!"); - // Tell the element to update its visited state + // Tell the element to update its visited state. mElement->UpdateState(true); if (StaticPrefs::layout_css_always_repaint_on_unvisited()) { -- To stop receiving notification emails like this one, please contact the administrator of this repository.