706822 - Delay when opening the AwesomeScreen and intermittent UI pauses when scrolling through listing of items

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect, P2)

Description

[Aaron Train [:aaronmt]]

Opening the AwesomeBar:

I/ActivityManager(  110): Displayed org.mozilla.fennec/org.mozilla.gecko.AwesomeBar: 1s+56ms

Scrolling through list of all items:

E/CursorWindow(  793): not growing since there are already 28 row(s), max size 1048576
E/Cursor  (  793): Failed allocating 230918 bytes for blob at 31,9
D/Cursor  (  793): finish_program_and_get_row_count row 40
D/Cursor  (  793): skip_rows row 3
E/CursorWindow(  793): need to grow: mSize = 1048576, size = 230918, freeSpace() = 91148, numRows = 29
E/CursorWindow(  793): not growing since there are already 29 row(s), max size 1048576
E/Cursor  (  793): Failed allocating 230918 bytes for blob at 31,9
D/Cursor  (  793): finish_program_and_get_row_count row 40
D/Cursor  (  793): skip_rows row 2
E/CursorWindow(  793): need to grow: mSize = 1048576, size = 230918, freeSpace() = 3116, numRows = 30
E/CursorWindow(  793): not growing since there are already 30 row(s), max size 1048576
E/Cursor  (  793): Failed allocating 230918 bytes for blob at 31,9
D/Cursor  (  793): finish_program_and_get_row_count row 40
D/Cursor  (  793): skip_rows row 1
E/CursorWindow(  793): need to grow: mSize = 1048576, size = 30614, freeSpace() = 716, numRows = 30
E/CursorWindow(  793): not growing since there are already 30 row(s), max size 1048576
E/Cursor  (  793): Failed allocating 30614 bytes for blob at 30,9
D/Cursor  (  793): finish_program_and_get_row_count row 41
E/CursorWindow(  793): need to grow: mSize = 1048576, size = 1150, freeSpace() = 955, numRows = 31
E/CursorWindow(  793): not growing since there are already 31 row(s), max size 1048576
E/Cursor  (  793): Failed allocating 1150 bytes for blob at 30,8
D/Cursor  (  793): finish_program_and_get_row_count row 41

--
Samsung Nexus S (Android 2.3.6)
20111201040252
http://hg.mozilla.org/projects/birch/rev/d71c91775f9b

Comment 1

[Aaron Train [:aaronmt]]

http://hg.mozilla.org/projects/birch/pushloghtml?fromchange=8e9a466b85bb&tochange=24f415e415f2

Comment 2

[Lucas Rocha (:lucasr)]

Blassey, maybe caused by one your of screenshot patches?

Comment 3

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

Attachment: patch

Comment 4

[Sriram Ramasubramanian [:sriram]]

As per the documentation, the data should be stored in onPause().

http://developer.android.com/reference/android/app/Activity.html

"Note that it is important to save persistent data in onPause() instead of onSaveInstanceState(Bundle) because the latter is not part of the lifecycle callbacks, so will not be called in every situation as described in its documentation."

Also, there is a big discussion in: http://stackoverflow.com/a/2909211/1052787

I think, we might need to save it in onStop(), to avoid the flags.

Comment 5

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

This already landed

Comment 6

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/a1a85e201c73

Comment 7

[Catalin Suciu [:csuciu]]

Cannot reproduce this on: 
Nightly 13.0a1(2012-03-05)
Device: Samsung Galaxy S2
OS: Android 2.3.4