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

Issue 2369963003: Remove OnSuspend/OnResume notifications on Android. (Closed)

Created:
4 years, 2 months ago by DaleCurtis
Modified:
4 years, 2 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2840
Project:
chromium
Visibility:
Public.

Description

Remove OnSuspend/OnResume notifications on Android. These are incorrectly mixing disparate concepts. Chrome code expects OnSuspend/OnResume to mean the system is about to go to sleep, not just that there is no more visible activity. This prevents background tasks like background audio from working properly since the network connections are force killed. We could workaround this by improving when OnSuspend/OnResume is sent, but it seems no one really cares, so just delete these. In fact a couple pieces of code explicitly reject OnSupend/OnResume from Android since they do not mean what is expected. An alternate signal for app suspension should be sent if this is ever needed again in the future. BUG=644515 TEST=background audio playback works. Review-Url: https://codereview.chromium.org/2324923002 Cr-Commit-Position: refs/heads/master@{#417691} (cherry picked from commit 610004f26299d1eb31b18f7dce3e2d3740eb8c04) Committed: https://chromium.googlesource.com/chromium/src/+/9ea1d94129fd5eb34437c3d3f0dccee91e7d0e3f

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -42 lines) Patch
M base/android/java/src/org/chromium/base/PowerMonitor.java View 4 chunks +1 line, -27 lines 0 comments Download
M base/power_monitor/power_monitor_device_source_android.cc View 1 chunk +3 lines, -7 lines 0 comments Download
M media/renderers/audio_renderer_impl.cc View 3 chunks +1 line, -8 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
DaleCurtis
4 years, 2 months ago (2016-09-26 22:12:34 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
9ea1d94129fd5eb34437c3d3f0dccee91e7d0e3f.

Powered by Google App Engine
This is Rietveld 408576698