Remove offline fetching from nsPrefetchService.cpp

RESOLVED FIXED

Status

()

Core
Document Navigation
RESOLVED FIXED
10 years ago
7 years ago

People

(Reporter: dcamp, Assigned: dcamp)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
Created attachment 327505 [details] [diff] [review]
v1

Earlier versions of the offline code used the prefetch service to fetch files for the offline cache.  This was later moved to nsOfflineCacheUpdate.cpp, but I failed to clean up nsPrefetchService.cpp.

Attached patch removes offline fetching from nsPrefetchService.cpp.  It leaves the idl alone (to avoid extra breakage in 3.1), but returns NS_ERROR_NOT_IMPLEMENTED if asked to fetch to the offline cache.  Next time we're ok to break interfaces, we should remove these methods.
Attachment #327505 - Flags: superreview?(cbiesinger)
Attachment #327505 - Flags: review?(cbiesinger)
(Assignee)

Updated

10 years ago
Blocks: 442806
Probably need a bug on removing the methods then, marked blocking the relevant release.
(Assignee)

Updated

10 years ago
Blocks: 443015
(Assignee)

Updated

10 years ago
Attachment #327505 - Flags: review?(bzbarsky)
Comment on attachment 327505 [details] [diff] [review]
v1

>+++ b/uriloader/prefetch/nsIPrefetchService.idl	Mon Jun 30 17:18:00 2008 -0700
>      * @param aIncludeNormalItems include normal prefetch items in the list.

This arg must be true, or NOT_IMPLEMENTED is thrown, right?

> nsPrefetchService::NotifyLoadRequested(nsPrefetchNode *node)
>+    const char *topic = "prefetch-load-requested";

Make this static, or just pass the literal to the function directly?  There's no need for this temporary.

> nsPrefetchService::NotifyLoadCompleted(nsPrefetchNode *node)
>+    const char *topic = "prefetch-load-completed";

Same here.

r+sr=bzbarsky.

Oh, please add -p to your diff options?
Attachment #327505 - Flags: superreview?(cbiesinger)
Attachment #327505 - Flags: superreview+
Attachment #327505 - Flags: review?(cbiesinger)
Attachment #327505 - Flags: review?(bzbarsky)
Attachment #327505 - Flags: review+
(Assignee)

Comment 3

9 years ago
Created attachment 334500 [details] [diff] [review]
patch to commit
Attachment #327505 - Attachment is obsolete: true
(Assignee)

Updated

9 years ago
Keywords: checkin-needed
(Assignee)

Updated

9 years ago
Assignee: nobody → dcamp
(Assignee)

Comment 4

9 years ago
Pushed to mozilla-central as rev bbaf0d5fef61
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
(Assignee)

Comment 5

9 years ago
Bugs 442803, 442806, and 442812 were in the regression range for a places unit test failure and were all backed out in an attempt to fix it.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 6

9 years ago
Re-landed as changeset 90bc3d600db1.
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
Depends on: 619590
You need to log in before you can comment on or make changes to this bug.