Closed Bug 9405 Opened 25 years ago Closed 25 years ago

[FEATURE][NECKO] nsURLReloadFromHistory support

Categories

(SeaMonkey :: UI Design, defect, P3)

Product:
SeaMonkey
Component:
UI Design
Platform:
x86
Windows NT
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: nisheeth_mozilla, Assigned: radha)

Details

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...
Status: NEW → ASSIGNED
Target Milestone: M9
Setting target milestone to M9 and accepting bug...
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. ;-)
Target Milestone: M9 → M10
Moving bugs blocked by Necko to M10...
Assignee: nisheeth → radha
Status: ASSIGNED → NEW
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.
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.
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.
Component: Networking-Core → XPApps
Summary: [Necko] Need to support new reload type nsURLReloadFromHistory → [FEATURE][NECKO] nsURLReloadFromHistory support
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
This is no more a feature for NECKO. LOAD_HISTORY has been taken care at
nsiSessionHistory and webshell. Markign fixed.
Status: RESOLVED → VERIFIED
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.