Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(903)

Issue 543543002: android: Don't invalidate UI resources if context is lost (Closed)

Created:
6 years, 3 months ago by no sievers
Modified:
6 years, 3 months ago
Reviewers:
CC:
chromium-reviews, darin-cc_chromium.org, jam
Base URL:
https://chromium.googlesource.com/chromium/src.git@clankium
Project:
chromium
Visibility:
Public.

Description

android: Don't invalidate UI resources if context is lost This callback implies that the resources are gone (because LTH was deleted). What ends up happening is that the resource clients get deleted while LTH still tries to call GetBitmap() on them. This still needs some working invalidation mechanism for Thumbnails, which don't have a persistent Bitmap they return from GetBitmap(). BUG=406879 NOTRY=True TBR=dtrainor@chromium.org Committed: https://crrev.com/f5186515e0607b24d77a0bca13634f0158865b8e Cr-Commit-Position: refs/heads/master@{#293252}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1 line) Patch
M content/browser/renderer_host/compositor_impl_android.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 4 (1 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sievers@chromium.org/543543002/1
6 years, 3 months ago (2014-09-04 02:51:02 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1) as 48f59d19817a5d4987d986865b5e3220c444f48e
6 years, 3 months ago (2014-09-04 02:53:20 UTC) #3
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 03:29:33 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/f5186515e0607b24d77a0bca13634f0158865b8e
Cr-Commit-Position: refs/heads/master@{#293252}

Powered by Google App Engine
This is Rietveld 408576698