Closed Bug 1777204 Opened 2 years ago Closed 1 year ago

Remove unused PreloaderBase::NotifyValidating and PreloaderBase::NotifyValidated

Categories

(Core :: DOM: Networking, task, P3)

task

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox111 --- fixed

People

(Reporter: manuel, Assigned: rstbroker)

Details

(Keywords: good-first-bug, Whiteboard: [necko-triaged])

Attachments

(1 file)

https://searchfox.org/mozilla-central/rev/0069a0cdc3cd136ea7411521219df6665f260be7/uriloader/preload/PreloaderBase.h#68-74:

  // Called when this currently existing load has to be asynchronously
  // revalidated before it can be used.  This prevents link preload DOM nodes
  // being notified until the validation is resolved.
  void NotifyValidating();
  // Called when the validation process has been done.  This will notify
  // associated link DOM nodes.
  void NotifyValidated(nsresult aStatus);

They aren't and weren't used. Should probably be removed.

Assignee: mbucher → nobody
Status: ASSIGNED → NEW
Keywords: good-first-bug
Whiteboard: [necko-triaged]
Summary: PreloaderBase::NotifyValidating and PreloaderBase::NotifyValidated are unused → Remove unused PreloaderBase::NotifyValidating and PreloaderBase::NotifyValidated
Assignee: nobody → rstbroker
Status: NEW → ASSIGNED
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/95f1f794cfc6
Removed unused PreloaderBase::NotifyValidating() & PreloaderBase::NotifyValidated() methods. r=manuel
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: