Closed
Bug 1055328
Opened 10 years ago
Closed 10 years ago
[B2G][Flame][Browser] Error messages lack text
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:2.1+, b2g-v1.4 unaffected, b2g-v2.0 unaffected, b2g-v2.1 verified, b2g-v2.2 verified)
Tracking | Status | |
---|---|---|
b2g-v1.4 | --- | unaffected |
b2g-v2.0 | --- | unaffected |
b2g-v2.1 | --- | verified |
b2g-v2.2 | --- | verified |
People
(Reporter: AdamA, Assigned: zbraniecki)
References
Details
(Keywords: regression, Whiteboard: [2.1-flame-test-run-1])
Attachments
(6 files)
Description:
The error messages that appear do not have any text. This occurs in the browser, hosted apps, and apps that bring the user to the browser.
Some of the error messages that do not have text are "server not found", "unable to connect", and "no access to the internet right now".
Repro Steps:
1) Update a Flame device to BuildID: 20140818040201
2) Disconnect from the internet
3) Launch browser and go to any page
4) Observe text on error pages
Actual:
There is no text on the error pages
Expected:
It is expected that error pages tell the user what the problem is
Environmental Variables:
Device: Flame 2.1 Master (319mb)
BuildID: 20140818040201
Gaia: aa8aace12d65956dd9525da5dac66e0d3b28597f
Gecko: 0aaa2d3d15cc
Version: 34.0a1 (2.1 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0
Notes: strings, problem, airplane, wifi, wi-fi
Repro frequency: 100%
Link to failed test case:
See attached: screenshot, logcat
Reporter | ||
Comment 1•10 years ago
|
||
Reporter | ||
Comment 2•10 years ago
|
||
Reporter | ||
Comment 3•10 years ago
|
||
Reporter | ||
Comment 4•10 years ago
|
||
This issue also occurs on 2.1 Flame (512), and 2.1 Buri.
Environmental Variables:
Device: Flame 2.1 Master (512mb)
BuildID: 20140818040201
Gaia: aa8aace12d65956dd9525da5dac66e0d3b28597f
Gecko: 0aaa2d3d15cc
Version: 34.0a1 (2.1 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0
Device: Buri 2.1 Master
BuildID: 20140818073016
Gaia: ba1992f2addc5a84afc2eab426f222a6bf2962ba
Gecko: bf27e27c994d
Version: 34.0a1 (2.1 Master)
Firmware: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0
Result:
Error messages do not have text.
------------------------------------------------------------------------
This issue does not occur on 2.0 Flame, 2.0 Buri, 1.4 Flame, and 1.4 Buri.
Environmental Variables:
Device: Flame 2.0 (319mb)
BuildID: 20140818000201
Gaia: fb2dd31abed2803eb7ad67eb4c52abb48de1e0f7
Gecko: 09f7a7184c71
Version: 32.0 (2.0)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
Device: Flame 1.4 (319mb)
Build ID: 20140818063007
Gaia: 21bec64497dc06a7f12071d573570ba8fea598ae
Gecko: 07d78d0f9bef
Version: 30.0 (1.4)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0
Device: Buri 2.0
Build ID: 20140818063008
Gaia: 640ce38ca03f1e26a4524ff4215b8b3f7731e2f0
Gecko: 692c93509dc9
Version: 32.0 (2.0)
Firmware Version: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
Device: Buri 1.4
BuildID: 20140818063007
Gaia: 21bec64497dc06a7f12071d573570ba8fea598ae
Gecko: 07d78d0f9bef
Version: 30.0 (1.4)
Firmware: v1.2device.cfg
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0
Result:
Error messages have text.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Comment 5•10 years ago
|
||
[Blocking Requested - why for this release]:
This is a regression from 2.0 and important strings are missing so nominating 2.1?
blocking-b2g: --- → 2.1?
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
Updated•10 years ago
|
QA Contact: jmitchell
Updated•10 years ago
|
Component: Gaia::Browser → Gaia::System
Comment 6•10 years ago
|
||
Mozilla Central Regression Window:
Last Working:
Device: Flame Master
Build ID: 20140805072639
Gaia: 19bf9795263e2ccc15d824a52ebf23c2670fa9b9
Gecko: 2aaedcdf69f6
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
First Broken:
Device: Flame Master
Build ID: 20140805074901
Gaia: e93780f9da8b34f370a4113abd4df9780d58e443
Gecko: 1aab16f74817
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Last Working Gaia First Broken Gecko: Issue does NOT reproduce
Gaia: 19bf9795263e2ccc15d824a52ebf23c2670fa9b9
Gecko: 1aab16f74817
First Broken Gaia Last Working Gecko: Issue DOES reproduce
Gaia: e93780f9da8b34f370a4113abd4df9780d58e443
Gecko: 2aaedcdf69f6
Gaia pushlog: https://github.com/mozilla-b2g/gaia/compare/19bf9795263e2ccc15d824a52ebf23c2670fa9b9...e93780f9da8b34f370a4113abd4df9780d58e443
B2G Inbound Regression Window:
Last Working:
Device: Flame Master
Build ID: 20140804131728
Gaia: 19bf9795263e2ccc15d824a52ebf23c2670fa9b9
Gecko: 7f81be7db528
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
First Broken:
Device: Flame Master
Build ID: 20140804133226
Gaia: f58bb4c40abbebd9ff9b32fb2a39bf3f44340991
Gecko: 2be403f035f5
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Last Working Gaia First Broken Gecko: Issue does NOT reproduce
Gaia: 19bf9795263e2ccc15d824a52ebf23c2670fa9b9
Gecko: 2be403f035f5
First Broken Gaia Last Working Gecko: Issue DOES reproduce
Gaia: f58bb4c40abbebd9ff9b32fb2a39bf3f44340991
Gecko: 7f81be7db528
Gaia pushlog: https://github.com/mozilla-b2g/gaia/compare/19bf9795263e2ccc15d824a52ebf23c2670fa9b9...f58bb4c40abbebd9ff9b32fb2a39bf3f44340991
Possibly broken by bug 994370 or bug 1048359
Starting with bug 994370 - Zibi can you take a look?
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
Flags: needinfo?(gandalf)
Keywords: regressionwindow-wanted
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(pbylenga)
Assignee | ||
Comment 7•10 years ago
|
||
That's regression from bug 994370. netError requires inline locales, so if we want to get rid of them, we need to figure out a solution for netError until we fix bug 444165.
Taking.
Assignee: nobody → gandalf
Status: NEW → ASSIGNED
Flags: needinfo?(gandalf)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Assignee | ||
Comment 8•10 years ago
|
||
Assignee | ||
Comment 9•10 years ago
|
||
The patch updates webapp-optimize to embed l10n resources in netError case and l10n.js to react to that.
Stas, can you look at this?
Attachment #8476914 -
Flags: review?(stas)
Updated•10 years ago
|
blocking-b2g: 2.1? → 2.1+
Assignee | ||
Comment 12•10 years ago
|
||
Comment on attachment 8476914 [details] [diff] [review]
bug1055328.diff
Review of attachment 8476914 [details] [diff] [review]:
-----------------------------------------------------------------
Yuren, can you take a look at this?
From your perspective it mainly adds an option to inject the whole ab-CD.json to <head/> if data-no-fetch is set (only net_error).
Attachment #8476914 -
Flags: review?(yurenju.mozilla)
Assignee | ||
Comment 13•10 years ago
|
||
Flod, we're moving resources out of system.properties to new net_error.properties
Assignee | ||
Comment 14•10 years ago
|
||
Comment on attachment 8476914 [details] [diff] [review]
bug1055328.diff
Actually, I'd prefer to get sr from yuren on the build changes. Stas is doing the review.
Attachment #8476914 -
Flags: review?(yurenju.mozilla) → superreview?(yurenju.mozilla)
Comment 15•10 years ago
|
||
Comment on attachment 8476914 [details] [diff] [review]
bug1055328.diff
Review of attachment 8476914 [details] [diff] [review]:
-----------------------------------------------------------------
This is much better from the semantic point of view. Thanks! r=me.
Attachment #8476914 -
Flags: review?(stas) → review+
Comment 16•10 years ago
|
||
Comment on attachment 8476914 [details] [diff] [review]
bug1055328.diff
Patch looks good but we should have an integration test for it.
so we can add an app into dev_apps like test-l10n-optimize, add this app into build/config/phone/apps-engineering.list and use |make build-test-integration| to verify the content in profile by build.test.js[1] to avoid regression.
[1] https://github.com/mozilla-b2g/gaia/blob/master/build/test/integration/build.test.js#L366-L370
Attachment #8476914 -
Flags: superreview?(yurenju.mozilla) → superreview-
Comment 17•10 years ago
|
||
marked as sr- because if I just cancel superreview, it will looks like a patch which is ready to land with r+.
Assignee | ||
Comment 18•10 years ago
|
||
Comment on attachment 8476913 [details]
pull request
Added the test. Requesting review on the test :)
Attachment #8476913 -
Flags: review?(yurenju.mozilla)
Comment 19•10 years ago
|
||
Comment on attachment 8476913 [details]
pull request
r=yurenju if nit is addressed, thanks!
Attachment #8476913 -
Flags: review?(yurenju.mozilla) → review+
Assignee | ||
Comment 20•10 years ago
|
||
Assignee | ||
Comment 21•10 years ago
|
||
L20n.js: https://github.com/l20n/l20n.js/commit/a80d69cd549c203348e5082d4c57146f6f153d2b
p.s. I see some errors on TBPL with my checkin but verified that it is not the result of this patch:
- For Gij: https://tbpl.mozilla.org/php/getParsedLog.php?id=47200832&tree=Gaia-Try&full=1
- bug 1046828 comment 43
- Gip and Gu errors show up before my checkin
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Target Milestone: --- → 2.1 S4 (12sep)
Updated•10 years ago
|
status-b2g-v2.2:
--- → fixed
Comment 22•10 years ago
|
||
This issue is verified fixed on the Flame 2.1 and 2.2
The error messages display properly without missing text.
Flame 2.1
Device: Flame 2.1 KK (319mb) (Full Flash)
BuildID: 20141012001201
Gaia: d18e130216cd3960cd327179364d9f71e42debda
Gecko: 610ee0e6a776
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 34.0a2 (2.1)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Flame 2.2
Device: Flame 2.2 Master KK (319mb) (Full Flash)
BuildID: 20141012040203
Gaia: 717ad4e8b7fc10ab8248500d00ba5ba0977fa8ab
Gecko: 44168a7af20d
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
Flame 2.2
Device: Flame 2.2 Master KK (319mb) (Full Flash)
BuildID: 20141012040203
Gaia: 717ad4e8b7fc10ab8248500d00ba5ba0977fa8ab
Gecko: 44168a7af20d
Gonk: 52c909e821d107d414f851e267dedcd7aae2cebf
Version: 35.0a1 (2.2 Master)
Firmware: V180
User Agent: Mozilla/5.0 (Mobile; rv:35.0) Gecko/35.0 Firefox/35.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage?][lead-review+]
Flags: needinfo?(dharris)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?][lead-review+] → [QAnalyst-Triage+][lead-review+]
Flags: needinfo?(dharris)
You need to log in
before you can comment on or make changes to this bug.
Description
•