Tim Guan-tin Chien [:timdream] (please needinfo) Reporter
	Description • 12 years ago

Today's build gives the lock screen the whole new look.

STR:

1. Boot up phone, observe lock screen

Expected:

1. Growing arrows animate from the unlock handle to unlock icon and camera icon

Actual:

1. Arrows are now drawn on the very top-left. On Otoro you can see the bottom-most one pixel flying just underneath the status bar.

Note:

1. Happen on SGS2 too. B2G Desktop not tested yet.
2. Since the arrows is mispositioned, bug 788412 is no longer observable.

	Andrew Overholt [:overholt]
	Comment 1 • 12 years ago

I can verify this.  Is it a gaia issue or a platform bug?

	Tim Guan-tin Chien [:timdream] (please needinfo) Reporter
	Comment 2 • 12 years ago

Platform. Gaia lock screen was kept unchanged like for more than a month.

	Andrew Overholt [:overholt]
	Comment 3 • 12 years ago

Thanks, Tim.

I'm taking a stab in the dark here that this issue is layout-related.  Jet, can someone on the layout team investigate?  Thanks.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 4 • 12 years ago

Moreover, this regressed in the last few days.  Sigh.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 5 • 12 years ago

Known-good commit: https://hg.mozilla.org/mozilla-central/rev/acd23e460291

Tim, what build were you testing?

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 6 • 12 years ago

Broken as of http://hg.mozilla.org/mozilla-central/rev/7771d1d6f0d8

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 7 • 12 years ago

Nothing particularly stands out in http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=acd23e460291&tochange=7771d1d6f0d8 .

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 8 • 12 years ago

Oh, missed a spot: my money is on bug 793132.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 9 • 12 years ago

It merged in through http://hg.mozilla.org/mozilla-central/rev/df69d95f636c .

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 10 • 12 years ago

Dammit, http://hg.mozilla.org/mozilla-central/rev/c23f6a95cdb3 has the

E/GeckoConsole(16690): [JavaScript Error: "TypeError: navigator.mozPower is null" {file: "app://system.gaiamobile.org/js/screen_manager.js" line: 196}]

error.

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 11 • 12 years ago

Matt sez

cjones: Take the code that was added to nsDisplayTransform::GetTransform() and add it to the CompositorParent caller of GetResultingTransformMatrix

	Chris Jones [:cjones] inactive; ni?/f?/r? if you need me Assignee
	Comment 13 • 12 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/6c59ce185898

	Ed Morley [:emorley]
	Comment 15 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/6c59ce185898