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

Issue 58483002: [Android WebView] Only throttle fallback tick (Closed)

Created:
7 years, 1 month ago by boliu
Modified:
7 years, 1 month ago
Reviewers:
joth
CC:
chromium-reviews, android-webview-reviews_chromium.org
Visibility:
Public.

Description

[Android WebView] Only throttle fallback tick When throttling, either when paused or when in background, only throttle fallback tick. There are apps relying on the fact that invalidates are not throttled when paused. When in background, framework should take care of not calling back with draw for invalidates. Internal bug b/11230312 BUG= NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232909

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M android_webview/browser/in_process_view_renderer.cc View 2 chunks +6 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
joth
lgtm Looks like we should be thinking about ways to wrap some focused unit tests ...
7 years, 1 month ago (2013-11-04 22:09:56 UTC) #1
boliu
On 2013/11/04 22:09:56, joth wrote: > lgtm > > Looks like we should be thinking ...
7 years, 1 month ago (2013-11-04 22:11:23 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/boliu@chromium.org/58483002/1
7 years, 1 month ago (2013-11-04 22:50:25 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/boliu@chromium.org/58483002/1
7 years, 1 month ago (2013-11-05 03:29:44 UTC) #4
commit-bot: I haz the power
7 years, 1 month ago (2013-11-05 04:38:26 UTC) #5
Message was sent while issue was closed.
Change committed as 232909

Powered by Google App Engine
This is Rietveld 408576698