Closed
Bug 389878
Opened 18 years ago
Closed 18 years ago
Gmail , Contacts and Settings links are broken
Categories
(Core :: General, defect)
Tracking
()
VERIFIED
FIXED
mozilla1.9alpha7
People
(Reporter: Peter6, Assigned: mrbkap)
References
Details
(Keywords: regression)
Attachments
(1 file)
4.39 KB,
patch
|
jst
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072620 Minefield/3.0a7pre ID:2007072620
repro
open FF
open Gmail
press on the Contacts link , left menu
result
nothing happens
Reporter | ||
Comment 1•18 years ago
|
||
regressionrange:
works in 20070726_1155_firefox-3.0a7pre.en-US.win32
fails in 20070726_1228_firefox-3.0a7pre.en-US.win32
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=1185476100&maxdate=1185478079&cvsroot=%2Fcvsroot
->bug 367911
I can't add the dependency coz of [sg]
Comment 2•18 years ago
|
||
And the Settings link, and more.
Reporter | ||
Updated•18 years ago
|
Summary: Gmail , Contacts link is broken → Gmail , Contacts and Settings links are broken
Reporter | ||
Comment 3•18 years ago
|
||
Labels->EditLabels is broken
Comment 4•18 years ago
|
||
I'm not sure that it's the same problem, but official Google Calendar gadget (http://www.google.com/ig/directory?hl=en&url=calendar.xml) is broken too
Comment 5•18 years ago
|
||
(In reply to comment #4)
> I'm not sure that it's the same problem, but official Google Calendar gadget
> (http://www.google.com/ig/directory?hl=en&url=calendar.xml) is broken too
>
Looks like the same problem after all:
works in 20070726_1155_firefox-3.0a7pre.en-US.win32
fails in 20070726_1228_firefox-3.0a7pre.en-US.win32
Updated•18 years ago
|
Updated•18 years ago
|
Flags: blocking1.9?
Assignee | ||
Comment 6•18 years ago
|
||
I'm trying to figure out what's going on here. gmail is rather difficult to debug.
Assignee: nobody → mrbkap
Reporter | ||
Comment 7•18 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072922 Minefield/3.0a7pre ID:2007072922
The problems are actually far worse.
Try Compose mail ->attach a file
result -> immediate crash.
Do I need to create another bug for the crash ?
Reporter | ||
Comment 8•18 years ago
|
||
(In reply to comment #7)
> Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007072922
> Minefield/3.0a7pre ID:2007072922
>
> The problems are actually far worse.
>
> Try Compose mail ->attach a file
> result -> immediate crash.
>
> Do I need to create another bug for the crash ?
>
nm, this problem seems to be gone again in the latest nightly
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007073005 Minefield/3.0a7pre ID:2007073005
Updated•18 years ago
|
Flags: blocking1.9? → blocking1.9+
Target Milestone: --- → mozilla1.9 M7
Comment 9•18 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9a7pre) Gecko/2007073005 Minefield/3.0a7pre ID:2007073005
Can reproduce problem on above build..
Assignee | ||
Comment 10•18 years ago
|
||
The problem was that I broke 'window.eval'. Because we now disallow, <not_global>.eval() will throw an exception. With XOWs, 'window' no longer refers to a true global object, instead, it refers to a wrapper, which is not a global object. This fix makes us wrap eval natives when we return them, so that we unwrap the global object before calling into eval.
Attachment #274662 -
Flags: superreview?(jst)
Attachment #274662 -
Flags: review?(jst)
Updated•18 years ago
|
Attachment #274662 -
Flags: superreview?(jst)
Attachment #274662 -
Flags: superreview+
Attachment #274662 -
Flags: review?(jst)
Attachment #274662 -
Flags: review+
Assignee | ||
Comment 11•18 years ago
|
||
Fix checked into trunk.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 12•18 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a7pre) Gecko/2007073113 Minefield/3.0a7pre ID:2007073113
VERIFIED FIXED
Updated•18 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•