1006890 - [OPEN C] "+" label sign missing from Contact Picture with CSP 1.0 Parser enabled

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Contacts, defect)

Description

[Marcia Knous [:marcia]]

Attachment: Screenshot of issue on Open C Master branch

Open C, while running:

Gaia   4af716f09747edfbea637f5b60f7fd7d0183f19b
SourceStamp 81651ad5e43c
BuildID 20140506040204
Version 32.0a1
4/29 Base Version

STR:
1. Open the contact app
2. Observe the attached screenshot - The add picture field is missing the plus button.

Comment 1

[Marcia Knous [:marcia]]

I reset the phone, and the issue persists.

Comment 2

[Marcia Knous [:marcia]]

Attachment: Screenshot of Open C on 1.4

Comment 3

[Noemí Freire (:noemi)]

Attachment: Screenshot of Hamachi_master.png

The issue does not occur on today's (5/7) master build:
Device: Hamachi
BuildId: 20140507072737
Gecko: 8bb825e
Gaia: 1f0cd52
Platform version: 32.0a1

Comment 4

[Jason Smith [:jsmith]]

(In reply to Noemí Freire (:noemi) from comment #3)
> Created attachment 8418672 [details]
> Screenshot of Hamachi_master.png
> 
> The issue does not occur on today's (5/7) master build:
> Device: Hamachi
> BuildId: 20140507072737
> Gecko: 8bb825e
> Gaia: 1f0cd52
> Platform version: 32.0a1

That would mean that this is a high-res specific issue.

Comment 5

[Natalya Kot [:nkot]]

(In reply to Jason Smith [:jsmith] from comment #4)
> (In reply to Noemí Freire (:noemi) from comment #3)
> > Created attachment 8418672 [details]
> > Screenshot of Hamachi_master.png
> > 
> > The issue does not occur on today's (5/7) master build:
> > Device: Hamachi
> > BuildId: 20140507072737
> > Gecko: 8bb825e
> > Gaia: 1f0cd52
> > Platform version: 32.0a1
> 
> That would mean that this is a high-res specific issue.


It doesn't occur on today's master build on Open_C either, should we close it as WFM?

Open_C 
Build ID: 20140507040203
Gecko: 417acde736e7
Gaia: 870a5c518742665d36b17e7e88c2ab07d440b94c
Platform Version: 32.0a1
P821A10V1.0.0B06_LOG_DL

Comment 6

[Chris Kreinbring [:CKreinbring]]

Regression windows on Flame 2.0 mozilla RIL

No repro
Build ID: 20140505193016
Gecko: https://hg.mozilla.org/mozilla-central/rev/56aa5c49bfc4
Gaia: e8a08a3f7a608993f0b302371e016e73faceea70
Platform Version: 32.0a1
Firmware Version: V1.2-device.cfg

Repro
Build ID: 20140505223005
Gecko: https://hg.mozilla.org/mozilla-central/rev/81651ad5e43c
Gaia: 4af716f09747edfbea637f5b60f7fd7d0183f19b
Platform Version: 32.0a1
Firmware Version: V1.2-device.cfg

Old Gaia / New Gecko - Fail
New Gaia / Old Gecko - OK
Gecko push log: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=56aa5c49bfc4&tochange=81651ad5e43c

No repro
Build ID: 20140505203002
Gecko: https://hg.mozilla.org/integration/b2g-inbound/rev/fc63732cd4f1
Gaia: d902ffefa2aa4ad4e3cc987d90a532dcc95d8228
Platform Version: 32.0a1
Firmware Version: V1.2-device.cfg

Repro
Build ID: 20140506023004
Gecko: https://hg.mozilla.org/integration/b2g-inbound/rev/82bb5983a805
Gaia: cfc40f41bf417c2d242033fefd53b7aa3f3bf71c
Platform Version: 32.0a1
Firmware Version: V1.2-device.cfg

Push log: http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=fc63732cd4f1&tochange=82bb5983a805

Comment 7

[Jason Smith [:jsmith]]

(In reply to Natalya Kot [:nkot] from comment #5)
> 
> It doesn't occur on today's master build on Open_C either, should we close
> it as WFM?

Don't think so - that means the root cause is due to the CSP parser landing. I'll rework the title to indicate that.

Comment 8

[Paul Theriault [:pauljt] (no longer reading bugmail)]

I've made this block the Gaia CSP fixes, instead of flipping the pref. This concerns me since I didn't notice this bug when auditing Gaia for inline style usage. Unless maybe its 1.4 only.

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to Paul Theriault [:pauljt] from comment #8)
> I've made this block the Gaia CSP fixes, instead of flipping the pref. This
> concerns me since I didn't notice this bug when auditing Gaia for inline
> style usage. Unless maybe its 1.4 only.

Can't see it on Flame / Master with speccompliant CSP enabled.

Comment 10

[BMO Automation]

Firefox OS is not being worked on