Media Metrix browser tracking broken

VERIFIED FIXED

Status

P3
normal
VERIFIED FIXED
18 years ago
14 years ago

People

(Reporter: law, Assigned: law)

Tracking

Trunk
x86
Windows 98

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [rtm++]Fix in hand, reviewed and approved)

Attachments

(1 attachment)

(Assignee)

Description

18 years ago
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 don@netscape.com 2000-10-11 10:28 -------

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

Comment 1

18 years ago
Created attachment 16776 [details] [diff] [review]
Proposed fix
(Assignee)

Comment 2

18 years ago
Updating bug to reflect status of its twin in Bugscape.
Status: NEW → ASSIGNED
Keywords: rtm
Whiteboard: [rtm+]Fix in hand, reviewed and approved
(Assignee)

Comment 3

18 years ago
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
Cc'ing jud, looks like a typo (should have been cut and paste, no retyping).

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

Comment 6

18 years ago
rtm++
Whiteboard: [rtm+]Fix in hand, reviewed and approved → [rtm++]Fix in hand, reviewed and approved
(Assignee)

Comment 7

18 years ago
Fix checked in, trunk and branch.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
rubberstamp vrfy for the branch...
Keywords: vtrunk
...and rubberstamp vrfy from the trunk pov.
Status: RESOLVED → VERIFIED
Keywords: vtrunk
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.