723151 - No drawable resources defined for eclair

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

all our drawables are drawalbe-[m/h/xh/]dpi-v[8/11/14]. Its that last part, specifying versions, that excludes eclair. The "right" thing to do according to the android docs is to have a complete set of your base resources in res/drawable, res/layout, res/strings etc etc. We're certainly violating that for drawables, not sure about the rest of them.

With this patch I can run fennec on my N1 running eclair

Comment 1

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

Patch is partly missing

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch

not sure what went wrong there

Comment 3

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

Comment on attachment 593539 [details] [diff] [review]
patch

Looks good, but I want Sriram to be aware of this too and look for any issues

Comment 4

[Sriram Ramasubramanian [:sriram]]

Comment on attachment 593539 [details] [diff] [review]
patch

Review of attachment 593539 [details] [diff] [review]:
-----------------------------------------------------------------

The v8 was added thinking that we wouldn't be supporting eclair. I am happy with this change :)
I am not sure if we want LDPI version. If I remember correctly we don't support LDPI.

Comment 5

[Brad Lassey [:blassey] (use needinfo?)]

(In reply to Sriram Ramasubramanian [:sriram] from comment #4)
> Comment on attachment 593539 [details] [diff] [review]
> patch
> 
> Review of attachment 593539 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> The v8 was added thinking that we wouldn't be supporting eclair. I am happy
> with this change :)
> I am not sure if we want LDPI version. If I remember correctly we don't
> support LDPI.

we don't, but sync has an LDPI resource.

Comment 6

[Sriram Ramasubramanian [:sriram]]

Probably we need to ask Sync team if they really want those resources (I believe they use eclipse and eclipse created the ldpi folder by default). If they don't want, we can remove it some day. If they want it, we should think of copying low resolution images into drawable-ldpi to make us LDPI devices ready.

Comment 7

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/f7a88134a0f7

Comment 8

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 593539 [details] [diff] [review]
patch

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
crash on startup on eclair
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
String changes made by this patch:

Comment 9

[Cristian Nicolae (:xti)]

Verified fixed on:

Mozilla/5.0 (Android;Linux armv7l;rv:13.0a1)Gecko/20120203
Firefox/13.0a1 Fennec/13.0a1
Device: Samsung Galaxy S
OS: Android 2.1-update1

Comment 10

[Cristian Nicolae (:xti)]

Aurora nom?

Comment 11

[Aaron Train [:aaronmt]]

(In reply to Cristian Nicolae (:xti) from comment #10)
> Aurora nom?

See the patch above. It's question marked already.

Comment 12

[Cristian Nicolae (:xti)]

(In reply to Aaron Train [:aaronmt] from comment #11)
> (In reply to Cristian Nicolae (:xti) from comment #10)
> > Aurora nom?
> 
> See the patch above. It's question marked already.

Sorry, I just missed it. Thanks

Comment 13

[Alex Keybl [:akeybl]]

Comment on attachment 593539 [details] [diff] [review]
patch

[Triage Comment]
Mobile only - approved for Aurora 12 and Beta 11.

Comment 14

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/7c4652b0f3b0

Comment 15

[Brad Lassey [:blassey] (use needinfo?)]

https://hg.mozilla.org/releases/mozilla-beta/rev/02d946fad4c7