Closed
      
        Bug 641716
      
      
        Opened 14 years ago
          Closed 3 years ago
      
        
    
  
[SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """, since bug 527935 landing     
    Categories
(Toolkit :: Form Manager, defect)
Tracking
()
        RESOLVED
        INCOMPLETE
        
    
  
| Tracking | Status | |
|---|---|---|
| status2.0 | --- | unaffected | 
| status1.9.1 | --- | ? | 
People
(Reporter: sgautherie, Unassigned)
References
(Blocks 1 open bug, )
Details
(Keywords: regression, Whiteboard: [almost perma-orange])
Attachments
(1 obsolete file)
Last good:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey2.0/1296705165.1296707956.16725.gz&fulltext=1
Linux comm-1.9.1 test mochitests on 2011/02/02 19:52:45
rev:07c40b67dae9
moz:cf6f4bb351f7
{
88180 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | Checking form1 input - "v" should equal "v"
}
+
Same changesets for OSX and Windows.
First bad:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey2.0/1296869668.1296871918.12812.gz
Linux comm-1.9.1 test mochitests on 2011/02/04 17:34:28
rev:07c40b67dae9
moz:27ddaf8a6c89
{
88188 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | Starting test #211
88189 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | This test is privledged.
88190 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | popup should be closed - false should equal false
88191 ERROR TEST-UNEXPECTED-FAIL | /tests/toolkit/components/satchel/test/test_bug_511615.html | Checking form1 input - "v" should equal ""
}
+
Same changesets for OSX and Windows.
Regression timeframe:
http://hg.mozilla.org/releases/mozilla-1.9.1/pushloghtml?fromchange=cf6f4bb351f7&tochange=27ddaf8a6c89
Bug 527935!
"You are not authorized to access bug #527935."
| Reporter | ||
| Updated•14 years ago
           | 
Summary: [SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """ → [SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """, since bug 527935 landing
| Comment 1•14 years ago
           | ||
Serge: does this only happen on 1.9.1?
Dolske: can you please look into this?  This looks like a test problem to me, the same kind of stuff you fixed in bug 527935...
| Comment 2•14 years ago
           | ||
Probably something in <http://hg.mozilla.org/releases/mozilla-1.9.1/rev/27ddaf8a6c89#l12.1> is wrong.  I tested it locally in Firefox before landing the patch both on 1.9.2 and 1.9.1, but I'm not sure what SeaMonkey could have different here...
| Reporter | ||
| Comment 3•14 years ago
           | ||
(In reply to comment #1)
> Serge: does this only happen on 1.9.1?
Yes, no failure report with SeaMonkey 2.1 (m-2.0).
          status2.0:
          --- → unaffected
| Reporter | ||
| Comment 4•14 years ago
           | ||
With SM 2.0, if I manually press escape after test has ended, the 'v' is removed.
Yet, I kind of wonder why, with (FF 3.x or) SM 2.1, this escape both 1) closes the popup and 2) removes the 'v' ... as I am ++/- unable to reproduce that behavior after the test has ended :-|
Note that, before bug 527935, the test didn't expect the 'v' to be removed.
| Reporter | ||
| Comment 5•14 years ago
           | ||
Unrelated, but while there.
        Attachment #519454 -
        Flags: review?(ehsan)
| Reporter | ||
| Comment 6•14 years ago
           | ||
(Guessing:) Could it be related to some XPFE vs Toolkit autocomplete widget difference?
| Comment 7•14 years ago
           | ||
Probably what happens in XPFE is that ESC closes the popup but is eaten and so the autocomplete doesn't revert the text value.
| Comment 8•14 years ago
           | ||
Having said that, the test passes on my build, so...
| Comment 9•14 years ago
           | ||
Comment on attachment 519454 [details] [diff] [review]
(Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]
r=me only if you land this on mozilla-central too.
And please change the commit message to something like "Bug 641716 - Correct the spelling of doKeyUnprivledged".
        Attachment #519454 -
        Flags: review?(ehsan) → review+
| Comment 10•14 years ago
           | ||
Comment on attachment 519454 [details] [diff] [review]
(Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]
Actually, r-.  Landing this patch as part of this bug will only make things more confusing.  Please file a new bug and attach the patch there.
        Attachment #519454 -
        Flags: review+ → review-
| Updated•14 years ago
           | 
        Attachment #519454 -
        Attachment is obsolete: true
| Updated•14 years ago
           | 
Whiteboard: [perma-orange]
| Reporter | ||
| Updated•14 years ago
           | 
Whiteboard: [almost perma-orange]
| Updated•14 years ago
           | 
Whiteboard: [almost perma-orange] → [almost perma-orange]
| Reporter | ||
| Comment 11•14 years ago
           | ||
Bug still there.
| Reporter | ||
| Updated•13 years ago
           | 
        Attachment #519454 -
        Attachment description: (Av1) s/privledged/priviledged/g, Add 2 comments → (Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]
| Comment 12•3 years ago
           | ||
There haven't been any activity on this bug for more than a decade.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
          You need to log in
          before you can comment on or make changes to this bug.
        
Description
•