Chromium Code Reviews
DescriptionMerge 227148 "Mac: Do not ask to handle power events in processe..."
> Mac: Do not ask to handle power events in processes with no PowerManager.
>
> Otherwise Chrome causes a 30-second delay trying to put a Mac to sleep.
>
> Only the main browser process, and nacl processes, create a
> PowerMonitorDeviceSource. However, renderers and other processes still
> call the static function PowerMonitorDeviceSource::AllocateSystemIOPorts.
> This results in the process asking to receive power change
> notifications, but not hooking up a function to acknowledge them. This
> results in 30-second timeouts while trying to sleep a Mac.
>
> This change removes plugin, renderer, utility and worker processes from
> the list of process types that register to receive power change events
> from OS X.
>
> BUG=132336
> TEST=Put a Mac to sleep with Chrome running. After waking, run `pmset -g
> log` from Terminal. There should be no new entries for Chrome under
> applicationresponse.timedout [10.6] or on a `Timedout` line [10.8]
>
> Review URL: https://codereview.chromium.org/25954005
TBR=tapted@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=227608
Patch Set 1 #
Messages
Total messages: 2 (0 generated)
|
|||||||||||||||||||