850567 - [Open_][Gallery]longKey set the Gallery albums to wallpaper, the crop box and picture don't match.(617001922211)

Status: RESOLVED DUPLICATE of bug 846236

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

Description

[Firefox_Mozilla]

Attachment: screenshot

Steps to reproduce:
1. LongKeydown idle set wallpaper,select Gallery;
2. Select a photo set, see the crop box;
Actual results:
Crop box does not match with selected picture, see attached file;
Expected results:
Crop box match Selected picture
mozilla build id：20130225070200(AU025)

Comment 1

[Michael Vines [:m1] [:evilmachines]]

Assuming we still have this issue on the tip, tef+ as it's pretty broken UI.

qawanted to check if this is still seen at tip please!

Comment 2

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: selection to crop after selecting image from gallery

It matches somewhat.   It's asking to crop the gallery image so that it fits the screen.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot after cropped selection.

screenshot after cropped selection.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Looking at the images... I don't really think this is a bug, unless I'm not understanding what's being reported?

Comment 5

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

fyi : I used 

Gecko  http://hg.mozilla.org/releases/mozilla-b2g18/rev/2a8c49f56bb6
Gaia   acd6546cdd03284b3b8c12432fb9955868fc4c2b
BuildID 20130313070202
Version 18.0

Comment 6

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

I cannot reproduce this issue with the latest beta build,
 - Build ID: 20130314114915
 - Channel: beta


I think the bug reported is about the size of crop box (white box) does not match the image.
Take attachment 724317 [details] as an example, the top and the bottom edge of the crop box should hover the picture's edges.

Could someone provide the image used to reproduce this issue?

Thanks.

Comment 7

[Firefox_Mozilla]

we add an image which can reproduce this issue. it can use this image to try.

Comment 8

[Firefox_Mozilla]

Attachment: image

Comment 9

[Firefox_Mozilla]

The attachment 725239 [details] can reproduce this issue in build ID:20130225070200. It means the size of crop box (white box) does not match the image, which mentioned in comment 7 .

Comment 10

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

I cannot reproduce with the attached image (attachment 725239 [details]) and the same build 20130225070200.
The build should be this one on v1.0.1,
https://pvtbuilds.mozilla.org/pub/mozilla.org/b2g/nightly/mozilla-b2g18_v1_0_1-unagi/2013/02/2013-02-25-07-02-00/

Please note that this looks the same as case 1 (landscape picture, attachment 716476 [details]) in Bug 826211, and we could not reproduce case 1 in that bug.

Mark testcase-wanted for help to get to the specific case as case 1.

Comment 11

[Andrew Overholt [:overholt]]

ni(reporter) as we wait for a test case

Comment 12

[Alex Keybl [:akeybl]]

Placing back on the nom list since all signs point to this being either a one-off image issue or completely unreproducible.

Comment 13

[Michael Vines [:m1] [:evilmachines]]

(clearing tef? flag for now pending more detail from reporter.)

Comment 14

[Firefox_Mozilla]

we still can reproduce it in build 20130310070203. Is it caused by hwcomposer?

Comment 15

[Michael Vines [:m1] [:evilmachines]]

Maybe, what do you think Diego?

Comment 16

[Diego Wilson [:diego]]

Yeah, looks like hwcomposer is missing some offset

Comment 17

[Michael Vines [:m1] [:evilmachines]]

(thanks, tef+ and please to be fixing)

Comment 18

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Which device did you use?

Comment 19

[Diego Wilson [:diego]]

I found the problem. HwcComposer2d is not handling the combination of rotation and Y-axis flip at the same time. Since this is an uncommon use case I'll just make it fallback to GPU rendering.

Comment 20

[Diego Wilson [:diego]]

Oops, please ignore comment 19. It was intended for bug 846236

Comment 21

[Diego Wilson [:diego]]

Turns out this bug has the same root cause as bug 846236. More details there.

Comment 22

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

If it's the same root cause, closing the issue as a dup of that bug.

Comment 23

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Steps are not clear on how to reproduce this issue for unagi.  Marking the other bug as test case wanted.