Closed Bug 720291 Opened 13 years ago Closed 13 years ago

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)

Categories

(DevTools :: Style Editor, defect)

Product:
DevTools
Component:
Style Editor
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 12

People

(Reporter: philor, Assigned: cedricv)

References

Details

(Keywords: intermittent-failure, Whiteboard: [styleeditor])

Attachments

(1 file, 1 obsolete file)

patch v1.1
4.11 KB, patch
dcamp
: review+
Details | Diff | Splinter Review
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
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)
Cedric, is this fallout from the transitions landing?
(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.
Assignee: nobody → cedricv
Attached patch patch v1 (obsolete) — — Splinter Review
Attachment #591474 - Flags: review?(dcamp)
Attachment #591474 - Flags: review?(dcamp)
Status: NEW → ASSIGNED
Whiteboard: [orange] → [orange][styleeditor]
Attached patch patch v1.1 — — Splinter Review
Attachment #591474 - Attachment is obsolete: true
Attachment #591723 - Flags: review?(dcamp)
Attachment #591723 - Flags: review?(dcamp) → review+
Whiteboard: [orange][styleeditor] → [orange][styleeditor][land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/d42fcb70cb22
Whiteboard: [orange][styleeditor][land-in-fx-team] → [orange][styleeditor][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/d42fcb70cb22
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [orange][styleeditor][fixed-in-fx-team] → [orange][styleeditor]
Target Milestone: --- → Firefox 12
Whiteboard: [orange][styleeditor] → [styleeditor]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: