Closed Bug 966805 Opened 10 years ago Closed 9 years ago

Intermittent browser_styleeditor_sourcemap_watching.js | div is blue after saving file - Got rgb(2, 0, 254), expected rgb(0, 0, 255)

Categories

(DevTools :: Style Editor, defect)

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

Tracking

(firefox31 wontfix, firefox32 fixed, firefox33 fixed, firefox34 fixed, firefox-esr24 unaffected, firefox-esr31 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 34
Tracking Flags:
Tracking Status
firefox31 --- wontfix
firefox32 --- fixed
firefox33 --- fixed
firefox34 --- fixed
firefox-esr24 --- unaffected
firefox-esr31 --- fixed

People

(Reporter: philor, Assigned: harth)

References

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Disable transitions for the test
1.48 KB, patch
bgrins
: review+
Details | Diff | Splinter Review 
https://tbpl.mozilla.org/php/getParsedLog.php?id=33977666&tree=Mozilla-Inbound
Rev5 MacOSX Mountain Lion 10.8 mozilla-inbound debug test mochitest-browser-chrome on 2014-02-02 11:11:32 PST for push 7e51a10dee23
slave: talos-mtnlion-r5-032

12:28:06     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | Console message: [JavaScript Warning: "XUL box for h3 element contained an inline #text child, forcing all its children to be wrapped in a block." {file: "chrome://browser/content/devtools/styleeditor.xul" line: 0}]
12:28:06     INFO -  --DOMWINDOW == 187 (0x139a90f98) [pid = 1211] [serial = 15037] [outer = 0x140eca6d8] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 186 (0x15ddf6378) [pid = 1211] [serial = 15007] [outer = 0x0] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 185 (0x13e9193d8) [pid = 1211] [serial = 15032] [outer = 0x0] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 184 (0x148e93798) [pid = 1211] [serial = 14996] [outer = 0x0] [url = data:text/html;charset=utf8,<!DOCTYPE%20html><html%20dir='ltr'>%20%20<head>%20%20%20%20<style>%20%20%20%20%20%20html,%20body%20{%20height:%20100%;%20}%20%20%20%20%20%20body%20{%20margin:%200;%20overflow:%20hidden;%20}%20%20%20%20%20%20.CodeMirror%20{%20width:%20100%;%20height:%20100%%20!important;%20line-height:%20normal!important}%20%20%20%20</style>%20%20%20%20<link%20rel='stylesheet'%20href='chrome://browser/skin/devtools/common.css'>%20%20%20%20<link%20rel='stylesheet'%20href='chrome://browser/content/devtools/codemirror/codemirror.css'>%20%20%20%20<link%20rel='stylesheet'%20href='chrome://browser/content/devtools/codemirror/dialog.css'>%20%20%20%20<link%20rel='stylesheet'%20href='chrome://browser/content/devtools/codemirror/mozilla.css'>%20%20%20%20<link%20rel='stylesheet'%20href='chrome://browser/content/devtools/codemirror/foldgutter.css'>%20%20</head>%20%20<body%20class='theme-body%20devtools-monospace'></body></html>]
12:28:06     INFO -  --DOMWINDOW == 183 (0x1478f1518) [pid = 1211] [serial = 15035] [outer = 0x15919e4b8] [url = about:blank]
12:28:06     INFO -  JavaScript strict warning: chrome://browser/content/devtools/codemirror/codemirror.js, line 721: reference to undefined property line.wrapClass
12:28:06     INFO -  JavaScript strict warning: chrome://browser/content/devtools/codemirror/codemirror.js, line 1040: reference to undefined property line.textClass
12:28:06     INFO -  JavaScript strict warning: chrome://browser/content/devtools/codemirror/codemirror.js, line 1020: reference to undefined property line.textClass
12:28:06     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | Console message: [JavaScript Warning: "ReferenceError: reference to undefined property line.wrapClass" {file: "chrome://browser/content/devtools/codemirror/codemirror.js" line: 721}]
12:28:06     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | Console message: [JavaScript Warning: "ReferenceError: reference to undefined property line.textClass" {file: "chrome://browser/content/devtools/codemirror/codemirror.js" line: 1040}]
12:28:06     INFO -  TEST-INFO | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | Console message: [JavaScript Warning: "ReferenceError: reference to undefined property line.textClass" {file: "chrome://browser/content/devtools/codemirror/codemirror.js" line: 1020}]
12:28:06     INFO -  --DOMWINDOW == 182 (0x140eca6d8) [pid = 1211] [serial = 15033] [outer = 0x0] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 181 (0x160c71c78) [pid = 1211] [serial = 15048] [outer = 0x13f793d78] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 180 (0x1311c5d68) [pid = 1211] [serial = 15022] [outer = 0x13021b0f8] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 179 (0x15ab8df08) [pid = 1211] [serial = 15010] [outer = 0x133740e48] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 178 (0x15ab2afb8) [pid = 1211] [serial = 14999] [outer = 0x148e6cdb8] [url = about:blank]
12:28:06     INFO -  --DOMWINDOW == 177 (0x15dddc878) [pid = 1211] [serial = 14989] [outer = 0x158991898] [url = about:blank]
12:28:07  WARNING -  TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | div is blue after saving file - Got rgb(2, 0, 254), expected rgb(0, 0, 255)
12:28:07     INFO -  Stack trace:
12:28:07     INFO -      JS frame :: chrome://mochikit/content/browser-test.js :: test_is :: line 705
12:28:07     INFO -      JS frame :: chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js :: test/</< :: line 53
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js :: observer :: line 63
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js :: emit :: line 96
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/devtools/server/protocol.js :: Front<.onPacket :: line 1055
12:28:07     INFO -      JS frame :: resource://gre/modules/devtools/dbg-client.jsm :: DebuggerClient.prototype.onPacket/< :: line 663
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: resolve :: line 118
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 43
12:28:07     INFO -      JS frame :: resource://gre/modules/commonjs/sdk/core/promise.js :: then :: line 153
12:28:07     INFO -      JS frame :: resource://gre/modules/devtools/dbg-client.jsm :: DebuggerClient.prototype.onPacket :: line 705
12:28:07     INFO -      JS frame :: resource://gre/modules/devtools/dbg-client.jsm -> resource://gre/modules/devtools/server/transport.js :: LDT_send/< :: line 258
12:28:07     INFO -      JS frame :: resource://gre/modules/devtools/DevToolsUtils.jsm -> resource://gre/modules/devtools/DevToolsUtils.js :: makeInfallible/< :: line 80
12:28:07     INFO -      native frame :: <unknown filename> :: <TOP_LEVEL> :: line 0
12:28:07     INFO -  [Parent 1211] WARNING: NS_ENSURE_TRUE(mMutable) failed: file ../../../../netwerk/base/src/nsSimpleURI.cpp, line 265
12:28:07     INFO -  TEST-INFO | MEMORY STAT vsize after test: 4143915008
12:28:07     INFO -  TEST-INFO | MEMORY STAT residentFast after test: 1068044288
12:28:07     INFO -  INFO TEST-END | chrome://mochitests/content/browser/browser/devtools/styleeditor/test/browser_styleeditor_sourcemap_watching.js | finished in 6278ms
Assignee: nobody → fayearthur
Aha, I bet this is catching it before the transition from red to blue finishes.
Any progress here?  This is a decent-frequency intermittent orange that's been happening since the test initially landed.
Blocks: 950915
Flags: needinfo?(fayearthur)
Trying something in bug 978653
Flags: needinfo?(fayearthur)
(In reply to Heather Arthur [:harth] from comment #106)
> Trying something in bug 978653

Looks like that didn't work :(
(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #119)
> (In reply to Heather Arthur [:harth] from comment #106)
> > Trying something in bug 978653
> 
> Looks like that didn't work :(

I thought maybe it hadn't made it to inbound yet, is that not the case?
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&fromchange=11ab722b0bcb&tochange=f56c157fa5ee suggests it was after bug 978653 landed on inbound (you can see it in the merge at the bottom), but no occurrences in the past couple days is encouraging :)
)=

Looks like it alleviated it, but didn't fix it. I filed bug 981041 which would eliminate it completely I bet.
Depends on: 965362
I found a bug in the read() function in this test which can cause it to not read everything that is available from the HTTP channel.  I have patches in bug 965362 up for review which fix this issue.  This bug may or may not be a result of this.
Depends on: 981041
Disabling transitions for this test.
Attachment #8462830 - Flags: review?(bgrinstead)
Attachment #8462830 - Flags: review?(bgrinstead) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/fe30b8d01baa
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 34
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee: fayearthur → nobody
Looks like this one might have been fixed after all. The last failures are from four months ago and seem residual from try and aurora.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 9 years ago
Resolution: --- → FIXED
Assignee: nobody → fayearthur
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: