Closed
Bug 706822
Opened 13 years ago
Closed 13 years ago
Delay when opening the AwesomeScreen and intermittent UI pauses when scrolling through listing of items
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(firefox11 fixed, fennec11+)
VERIFIED
FIXED
People
(Reporter: aaronmt, Assigned: blassey)
Details
(Keywords: perf, regression)
Attachments
(1 file)
1.92 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Updated•13 years ago
|
Keywords: perf,
regression
Reporter | ||
Comment 1•13 years ago
|
||
http://hg.mozilla.org/projects/birch/pushloghtml?fromchange=8e9a466b85bb&tochange=24f415e415f2
Comment 2•13 years ago
|
||
Blassey, maybe caused by one your of screenshot patches?
Updated•13 years ago
|
Assignee: nobody → blassey.bugs
Priority: -- → P2
Assignee | ||
Comment 3•13 years ago
|
||
Attachment #578307 -
Flags: review?(doug.turner)
Comment 4•13 years ago
|
||
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.
Updated•13 years ago
|
Attachment #578307 -
Flags: review?(doug.turner) → review+
Reporter | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 5•13 years ago
|
||
This already landed
Assignee | ||
Updated•13 years ago
|
tracking-fennec: --- → 11+
Assignee | ||
Comment 6•13 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/a1a85e201c73
status-firefox11:
--- → fixed
Comment 7•12 years ago
|
||
Cannot reproduce this on: Nightly 13.0a1(2012-03-05) Device: Samsung Galaxy S2 OS: Android 2.3.4
Status: RESOLVED → VERIFIED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•