Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Intermittent browser_styleeditor_new.js | content's background color has been updated to red - Got rgb(255, 5, 5) (or 6, 6), expected rgb(255, 0, 0)

RESOLVED FIXED in Firefox 12

Status

()

Firefox
Developer Tools: Style Editor
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: philor, Assigned: cedricv)

Tracking

({intermittent-failure})

Trunk
Firefox 12
intermittent-failure
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [styleeditor])

Attachments

(1 attachment, 1 obsolete attachment)

https://tbpl.mozilla.org/php/getParsedLog.php?id=8748453&tree=Mozilla-Inbound
Rev3 Fedora 12x64 mozilla-inbound pgo test mochitest-other on 2012-01-22 18:53:26 PST for push 2f9840b376a6

TEST-PASS | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_new.js | new editor shows 1 rule after modification
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_new.js | content's background color has been updated to red - Got rgb(255, 5, 5), expected rgb(255, 0, 0)
Stack trace:
    JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 402
    JS frame :: chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_new.js :: <TOP_LEVEL> :: line 110
    JS frame :: resource:///modules/devtools/StyleEditor.jsm :: SE__triggerAction :: line 913
    JS frame :: resource:///modules/devtools/StyleEditor.jsm :: SE__onTransitionEnd :: line 712
    native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0

TEST-PASS | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_new.js | StyleEditor's transition class has been removed from content
Comment hidden (Treeherder Robot)
That second one was when profiling merged the push in https://tbpl.mozilla.org/?rev=d43360499b86 if anything in there grabs your eye.
OS: Linux → All
Hardware: x86_64 → All
Summary: Intermittent browser_styleeditor_new.js | content's background color has been updated to red - Got rgb(255, 5, 5), expected rgb(255, 0, 0) → Intermittent browser_styleeditor_new.js | content's background color has been updated to red - Got rgb(255, 5, 5) (or 6, 6), expected rgb(255, 0, 0)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 10

6 years ago
Cedric, is this fallout from the transitions landing?
(Assignee)

Comment 11

6 years ago
(In reply to Dave Camp (:dcamp) from comment #10)
> Cedric, is this fallout from the transitions landing?

Yes it definitely is, I'm looking at it.
The 10% buffer over the transition duration to clean up the transition and "commit" the change might not be enough on our very busy/saturated try servers.
Will try to find a better way other than arbitrarily increasing the timer.
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Assignee)

Updated

6 years ago
Assignee: nobody → cedricv
(Assignee)

Comment 14

6 years ago
Created attachment 591474 [details] [diff] [review]
patch v1
Attachment #591474 - Flags: review?(dcamp)
(Assignee)

Updated

6 years ago
Attachment #591474 - Flags: review?(dcamp)
(Assignee)

Updated

6 years ago
Status: NEW → ASSIGNED
Whiteboard: [orange] → [orange][styleeditor]
(Assignee)

Comment 15

6 years ago
Created attachment 591723 [details] [diff] [review]
patch v1.1
Attachment #591474 - Attachment is obsolete: true
Attachment #591723 - Flags: review?(dcamp)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Updated

6 years ago
Attachment #591723 - Flags: review?(dcamp) → review+
(Assignee)

Updated

6 years ago
Whiteboard: [orange][styleeditor] → [orange][styleeditor][land-in-fx-team]

Comment 26

6 years ago
https://hg.mozilla.org/integration/fx-team/rev/d42fcb70cb22
Whiteboard: [orange][styleeditor][land-in-fx-team] → [orange][styleeditor][fixed-in-fx-team]
Comment hidden (Treeherder Robot)
https://hg.mozilla.org/mozilla-central/rev/d42fcb70cb22
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [orange][styleeditor][fixed-in-fx-team] → [orange][styleeditor]
Target Milestone: --- → Firefox 12
Keywords: intermittent-failure
Whiteboard: [orange][styleeditor] → [styleeditor]
You need to log in before you can comment on or make changes to this bug.