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

VERIFIED FIXED

Status

()

Firefox for Android
General
P2
normal
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: aaronmt, Assigned: blassey)

Tracking

({perf, regression})

unspecified
ARM
Android
perf, regression
Points:
---

Firefox Tracking Flags

(firefox11 fixed, fennec11+)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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

6 years ago
Keywords: perf, regression
(Reporter)

Comment 1

6 years ago
http://hg.mozilla.org/projects/birch/pushloghtml?fromchange=8e9a466b85bb&tochange=24f415e415f2
Blassey, maybe caused by one your of screenshot patches?
Assignee: nobody → blassey.bugs
Priority: -- → P2
Created attachment 578307 [details] [diff] [review]
patch
Attachment #578307 - Flags: review?(doug.turner)
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

6 years ago
Attachment #578307 - Flags: review?(doug.turner) → review+
(Reporter)

Updated

6 years ago
Keywords: checkin-needed
This already landed
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
tracking-fennec: --- → 11+
https://hg.mozilla.org/releases/mozilla-aurora/rev/a1a85e201c73
status-firefox11: --- → fixed
Cannot reproduce this on: 
Nightly 13.0a1(2012-03-05)
Device: Samsung Galaxy S2
OS: Android 2.3.4
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.