[FEATURE][NECKO] nsURLReloadFromHistory support

VERIFIED FIXED in M10

Status

SeaMonkey
UI Design
P3
normal
VERIFIED FIXED
19 years ago
14 years ago

People

(Reporter: Nisheeth Ranjan, Assigned: Radha on family leave (not reading bugmail))

Tracking

Trunk
x86
Windows NT

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
A new URL reload type got added to the tip that will need to be supported in
netlib.  This is used by session history to inform the webshell that a history
traversal is happening.  The webshell is then smart and scrolls the document in
certain cases rather than loading the document via netlib all the time.

Warren, gagan, and me will work together to fix this once Necko lands...
(Reporter)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M9
(Reporter)

Comment 1

19 years ago
Setting target milestone to M9 and accepting bug...

Comment 2

19 years ago
Changing all Networking Library/Browser bugs to Networking-Core component for
Browser.

Occasionally, Bugzilla will burp and cause Verified bugs to reopen when I do
this in a bulk change.  If this happens, I will fix. ;-)
(Reporter)

Updated

19 years ago
Target Milestone: M9 → M10
(Reporter)

Comment 3

19 years ago
Moving bugs blocked by Necko to M10...
(Reporter)

Updated

19 years ago
Assignee: nisheeth → radha
Status: ASSIGNED → NEW
(Reporter)

Comment 4

19 years ago
Radha, as per our email discussion earlier today, I'm assigning this bug to you
because you'll work with Warren to fix this.  Thanks.

Comment 5

19 years ago
I don't think we want an actual "reload type" in necko for this, but some
higher-level routine that first tries history, and then moves on to necko. But
I don't have the background on this yet. Let's talk about it.
Status: NEW → ASSIGNED
nsWebShell::LoadURL() takes a load type as an argument. It used to be a
nsURLReloadType and in the new world, it is a nsLoadType  as defined in
nsIChannel. Right now it has only LOAD_NORMAL and LOAD_BACKGROUND defined. I
thought I could add a LOAD_HISTORY too. Let me know if you have other
suggestions.

Comment 7

19 years ago
I think it's ok for the web shell to define another bit that gets or'd into the
nsLoadType flags and passed around. I just didn't think that that bit should be
defined by necko proper, because necko has no notion of a history. But it's not
really a big deal if putting it there is a lot more convenient.

Necko should however define a bit for "reload from cache" since it will know
about the cache.

Updated

19 years ago
Component: Networking-Core → XPApps
Summary: [Necko] Need to support new reload type nsURLReloadFromHistory → [FEATURE][NECKO] nsURLReloadFromHistory support
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
This is no more a feature for NECKO. LOAD_HISTORY has been taken care at
nsiSessionHistory and webshell. Markign fixed.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 9

19 years ago
nisheeth, I assume you agree with the resolution
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.