53842 - History page is displaying a blank window

Status: VERIFIED FIXED

(Core :: DOM: Navigation, defect, P2)

Description

[srinivasa yarlagadda]

Build: NS6PR3(ID:2000090913) for Sparc_Solaris
       NS6PR3(ID:2000080720) for Intle_SOalris
Description:
Set the Preferences->History->Browsing history to Some days(say 9).
Visit some sites and look into Tasks->Tools->History. It shows a blank page.

Comment 1

[Radha on family leave (not reading bugmail)]

Don, Now that slamm's gone, who is this going to?

Comment 2

[Stephan Litke]

since .jar packing for linux (since moz build 2000092013) the history
window is allways empty, IF you install 'Navigator only' from the
moz...-sea.tar.gz package.
Reason: the history window seems to depend on the file
components/libmork.so,
which is installed only together with Mail/News.

Comment 3

[Radha on family leave (not reading bugmail)]

seems to be a JAR packaging issue. CC'ing Don, Warren, rjc please advise what 
can be done.

Comment 4

[don]

Hmmm, who usually fixes jar packaging issues like this?  Do we need to fix this
for RTM?  How heinous is it to have a blank history window in a Navigator-only
install?

Comment 5

[don]

Nominate for RTM.

Comment 6

[don]

We still need an answer of this one ...

Comment 7

[don]

Over to Bill because he thinks he knows how to fix this ...

Comment 8

[don]

Adding to cc ...

Comment 9

[Bill Law]

Any theories on what this might have to do with .jar packaging?

.jar files only contain .xul/.js/.dtd etc. files, not .so's.  I think the
problem with the missing .so is that it simply isn't in a navigator-only
install.  That would be the case regardless of .jar packaging, no?

In any case, I'm going to look into moving this .so to the navigator component.
 There are issues of dependencies on yet more libraries...

Comment 10

[Bill Law]

Attachment: Proposed fix

Comment 11

[Bill Law]

We reproduced the problem on Win98.  I don't think it's got anything to do with
.jars.  mork library is needed by the browser so it should appear there in the
various packages-* files.

I've attached a patch that does that.  Am looking for review and super-review,
at which point we'll go for [rtm++].

Comment 12

[don]

Fix in hand, waiting on review ...

Comment 13

[Sean Su]

r=ssu

Comment 14

[Daniel Veditz [:dveditz]]

sr=dveditz for packaging issues (not that I count officially :-) )

Comment 15

[don]

Dan, I think you DO count officially for packaging. :-)

PDT, fix in hand, reviewed and approved.  Please approve for "++".

Comment 16

[Bill Law]

Fix checked in on trunk.  Awaiting [rtm++] for branch.

Comment 17

[James "Kovu" Russell]

*** Bug 43836 has been marked as a duplicate of this bug. ***

Comment 18

[don]

Updated status whiteboard.

Comment 19

[selmer (gone)]

need info: is this Solaris only?  We couldn't repro this on Win95 or Win98.
(Although we couldn't clear the history or delete any history items on either
platform which seems like a bigger problem.)

Comment 20

[srinivasa yarlagadda]

I saw this problem on i386_Solaris and Sparc_Solaris platforms. I did't checked
on Linux.

Comment 21

[Claudius Gayle]

fyi, this is/was all platforms, see law's 10/06 comments. The key is the nav only install.

Comment 22

[don]

PDT, this is a Navigator-only install problem.  Making this "+" again since it's
safely working on the trunk.  Please approve.

Comment 23

[Phil Peterson]

[rtm++] to make history work in nav-only install

Comment 24

[Blake Ross]

*** Bug 56091 has been marked as a duplicate of this bug. ***

Comment 25

[Bill Law]

Fix checked in on branch.

Comment 26

[Claudius Gayle]

VERIFIED Fixed on branch build with a Nav only install 2000101608. adding vtrunk keyword.

Comment 27

[Asa Dotzler [:asa]]

Verified Fixed on trunk builds.  Nav Only install History looks fine.
linux 101808 RedHat 6.2
win32 101804 NT 4
mac 101804 Mac OS9
Setting bug to Verified and removing vtrunk keyword