511982 - Acid 2 Test on n810 does not render at all

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Aakash Desai [:aakashd]]

Steps to Reproduce:
1. Go to http://www.webstandards.org/files/acid2/test.html
2. lick on "Take the Acid2 Test".

Actual Results:
Nothing happens

Expected Results:
The smiley face (in some sort of form should show up)

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

I believe we need to add back the "scroll" event support that was in CanvasBrowser, but not ported to the tile system.

See bug 452286 and bug 470861 and bug 499388

Comment 2

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

Attachment: Patch v0.1

mainly a backport of the patch for bug 452286

Comment 3

[:Gavin Sharp [email: [e-mail]]]

(In reply to comment #2)
> Created an attachment (id=396722) [details]
> Patch v0.1
> 
> mainly a backport of the patch for bug 452286

I guess this probably belongs in bug 512277...

Comment 4

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

i guess you're right!

Comment 6

[Roy Frostig :froystig]

Comment on attachment 396722 [details] [diff] [review]
Patch v0.1

>+    let zoomLevel = this._browserViewportState.zoomLevel;
>+    Browser.contentScrollboxScroller.scrollTo(scrollX * zoomLevel, scrollY * zoomLevel);

We're pretty consistent with this everywhere else, so collapse these two lines to:

>     Browser.contentScrollboxScroller.scrollTo(this.browserToViewport(scrollX), this.browserToViewport(scrollY));

r+ with that.

Comment 7

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

Attachment: Address Roy's comments

Comment 8

[:Gavin Sharp [email: [e-mail]]]

https://hg.mozilla.org/mobile-browser/rev/63986b01506d

Comment 9

[Aakash Desai [:aakashd]]

So, the smiley is showing up, but it looks maimed and disfigured...so, that'll hopefully be taken care of with bug 499388

verified FIXED on builds: 

Mozilla/5.0 (Windows; U; WindowsCE 5.2; en-US; rv:1.9.3a1pre) Gecko/20090901 Fennec/1.0a3

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.2a2pre) Gecko/20090901
Fennec/1.0b4pre

Comment 10

[Aakash Desai [:aakashd]]

This bug is on our litmus BFT via testcase:

https://litmus.mozilla.org/show_test.cgi?id=7386