New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 599870 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Feature



Sign in to add a comment

Investigate Physical Web Memory Usage

Project Member Reported by mmo...@chromium.org, Apr 1 2016

Issue description

General catchall bug to report on investigations.  We can file specific bugs when we find any sources of issues.
 
One area to investigate: It appears we keep a map of iconUrl -> Bitmap which does not get cleared until Chrome is killed.

As we move to support larger icons, this may prove a problem.

Comment 2 by cco3@chromium.org, Apr 7 2016

mmocny@, that's in NearbyUrlsAdapter, which is tied to the ListUrlsActivity.  It clears whenever the Activity is removed from memory.

Comment 3 by cco3@chromium.org, Apr 7 2016

Owner: cco3@chromium.org
Status: Started (was: Available)
Cc: agrieve@chromium.org
I was talking with @agrieve and he mentioned that the UMA reports often have noise, so this is likely just an anomaly.  There are other automated scans on memory impact, and if no one has spoken up, its likely we haven't done anything noticeable.

That said, its also possible that this only happens with beacons around, and that our testers don't have any nearby.

I think a bit of due diligence will go a long way.  Andrew suggested a tool we can use to help understand memory usage, but I forgot what it was.

Comment 5 by cco3@chromium.org, Apr 8 2016

Was it MAT?  I've played with that some.

Comment 6 by cco3@chromium.org, Apr 11 2016

Status: Fixed (was: Started)
Things look good/as expected so far.  I'm going to mark this fixed unless we have other specific inquiries.
pwheap.png
170 KB View Download

Sign in to add a comment