Closed
Bug 920532
Opened 11 years ago
Closed 11 years ago
Lots of "System JS : ERROR file:///builds/slave/test/build/b2g/components/MozKeyboard.js:380 TypeError: this._inputcontext is null" when running Gaia UI tests
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(firefox25 wontfix, firefox26 wontfix, firefox27 fixed, b2g-v1.2 fixed)
People
(Reporter: RyanVM, Assigned: fabrice)
References
Details
Looks like a regression from bug 905573. https://tbpl.mozilla.org/php/getParsedLog.php?id=28230001&tree=B2g-Inbound
Reporter | ||
Comment 1•11 years ago
|
||
This spams 200+ times in the log and makes finding other failures *much* harder to find (as I ran into this morning).
Assignee | ||
Comment 2•11 years ago
|
||
Let's see if this simple check is enough: https://tbpl.mozilla.org/?tree=Try&rev=9e1d78eb8447
Assignee | ||
Comment 3•11 years ago
|
||
Trying to get Gu to run: https://tbpl.mozilla.org/?tree=Try&rev=b9c67670faf4
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/77a15a2e18fa
https://hg.mozilla.org/mozilla-central/rev/77a15a2e18fa
Assignee: nobody → fabrice
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
Reopening because now the UI tests run again against mc and this checks the wrong variable.
Comment 7•11 years ago
|
||
this.inputcontext is a getter method that returns this._inputcontext, so it looks to me like this patch should work. Maybe I'm missing something, though. In any case, the test failures that caused bug 884752 to be backed out make me think that we need to uplift this to aurora. Jan: do you still see the error on m-c builds, or are you just basing this on https://bugzilla.mozilla.org/show_bug.cgi?id=884752#c51
Comment 8•11 years ago
|
||
Jan, thank you for diagnosing the bug. Let's create a new bug other than reopen this bug if we don't need to back out the patch of this bug.
Comment 9•11 years ago
|
||
Sorry, I think I was asleep or something. I just based it on the comment in bug 884752, haven't seen it myself.
Comment 10•11 years ago
|
||
Close based on comment 8.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Comment 12•11 years ago
|
||
Can we get this patch uplifted to B2G26 branch? It seems this get it harder to examine the following failure as Comment 1 also mentioned, https://tbpl-dev.allizom.org/?tree=Mozilla-B2g26-v1.2&rev=fc1043ab2d67
Reporter | ||
Comment 13•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/163f34a949e9
status-b2g-v1.2:
--- → fixed
status-firefox25:
--- → wontfix
status-firefox26:
--- → wontfix
status-firefox27:
--- → fixed
Flags: needinfo?(ryanvm)
Keywords: checkin-needed
Whiteboard: [checkin-needed-b2g26]
You need to log in
before you can comment on or make changes to this bug.
Description
•