56122 - Media Metrix browser tracking broken

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect, P3)

Description

[Bill Law]

I originally thought this was a commercial build problem but now I realize it is
a Mozilla build problem.  So I'm creating a new bug to Bugzilla.

Here is the complete description from the Bugscape bug (which is at
http://bugscape.mcom.com/show_bug.cgi?id=2845 for those lucky enough to be able
to see it).

[Bug description follows]

I was alerted by Jesse Burris of Media Metrix that their tool to track browser
usage was broken in PR3.  After some investigation, I have determined that some
packaging snafu has resulted in the omission of a required component type
library in the Netscape commercial build.

After adding some debugging code to navigator.js, I get this error on the
console when trying to access the nsIUrlWidget interface:

Error getting url widget service: [Exception... "Invalid InterfaceID"  nsresult:
 "0x80570018 (NS_ERROR_XPC_BAD_IID)"  location: "JS frame :: chrome://navigator/
content/navigator.js :: getService :: line 889"  data: no]

It appears as if the urlwidgt.xpt (xpconnect type library) is missing (or
whatever the equivalent is for our cached xpconnect type library story).


------- Additional Comments From Bill Law 2000-10-10 18:21 -------

Adding interested parties to cc: list and nominating for rtm.


------- Additional Comments From Bill Law 2000-10-10 18:44 -------

I think I have found the source of the problem: a typo in the package list.
I'll attacah a patch shortly.  I'll submit that to Sean Su for review.


------- Additional Comments From Bill Law 2000-10-10 18:45 -------

Created an attachment (id=392) Proposed patch


------- Additional Comments From Daniel Veditz 2000-10-11 00:54 -------

r=dveditz and module owner a= for the patch.


------- Additional Comments From [e-mail] 2000-10-11 10:28 -------

PDT folks, this is sort of a contractual obligation thang.  Please approve.

Comment 1

[Bill Law]

Attachment: Proposed fix

Comment 2

[Bill Law]

Updating bug to reflect status of its twin in Bugscape.

Comment 3

[Bill Law]

Here's a pointer to the change that broke this (this to aid super-reviewers in
assessing the validity of the patch)

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=packages-win&root=/cvsroot&subdir=mozilla/xpinstall/packager&command=DIFF_FRAMESET&rev1=1.108&rev2=1.109

Comment 4

[Brendan Eich [:brendan]]

Cc'ing jud, looks like a typo (should have been cut and paste, no retyping).

a=[e-mail]

Comment 5

[ekrock's old account (dead)]

This is a must-fix for RTM to enable Mozilla and Netscape 6 to have their market 
share tracked.

Comment 6

[Phil Peterson]

rtm++

Comment 7

[Bill Law]

Fix checked in, trunk and branch.

Comment 8

[sairuh (rarely reading bugmail)]

rubberstamp vrfy for the branch...

Comment 9

[sairuh (rarely reading bugmail)]

...and rubberstamp vrfy from the trunk pov.