MukeshKhulve / android-screenshot-library

Automatically exported from code.google.com/p/android-screenshot-library
Other
0 stars 0 forks source link

Colors are inverted on Motorola Droid #1

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Colors are inverted on Motorola Droid.  Works perfectly on the Nexus One.  
Attached file shows 3 layers of screenshots.  The middle layer, which is a 
screenshot-of-a-screenshot, has correct colors.

What version of the product are you using? On what operating system?
ASL 1.1 on Motorola Droid running Android 2.2.1

Original issue reported on code.google.com by stanal...@gmail.com on 14 Dec 2010 at 6:16

Attachments:

GoogleCodeExporter commented 9 years ago

After some testing, it seems the RED and BLUE components are the ones being 
reversed.  If I reverse these in ScreenshotService.writeImageFile, then the 
file is written correctly on the Droid (now the Nexus One would be reversed).  
Since it seems that the ScreenShotService is retrieving the raw data already 
reversed, so the issue is probably in the lower level asl-native binary or 
fbshot.c code.  Unfortunately, I do not know enough about this to figure out a 
solution for all phones.

Original comment by stanal...@gmail.com on 14 Dec 2010 at 7:00

GoogleCodeExporter commented 9 years ago
It seems that Motorola Droid has a different frame buffer format, essentially 
BGR instead of usual RGB found on most phones. Unfortunately, I don't have this 
device around so I cannot verify the hyphothesis.

Nevertheless, I have deployed a new version of the library (1.2) which now 
takes the framebuffer format into account. Please check whether it fixes the 
issue.

Original comment by xion.dev@gmail.com on 3 Jan 2011 at 1:41

GoogleCodeExporter commented 9 years ago

Great, I downloaded 1.2 and that fixed the problem!  Thanks.

Original comment by stanal...@gmail.com on 3 Jan 2011 at 5:51

GoogleCodeExporter commented 9 years ago

Original comment by karol.ku...@polidea.pl on 14 Jun 2011 at 1:22