getImageData now finalizesFrame and disableAcceleration updates composit

getImageData now finalizesFrame and disableAcceleration updates composit

Both are needed for proper accelerate to software conversion.
getImageData wasn't calling needFinalizeFrame, so frames that only
called it, wouldn't be moved to software.
Also, when converting from accelerate, background-color/alpha wasn't
being properly propagated because we didn't updated compositing.

BUG=

Review-Url: https://codereview.chromium.org/2802353002
Cr-Commit-Position: refs/heads/master@{#463307}
Committed: https://chromium.googlesource.com/chromium/src/+/67774f8b275a884f3571cbb0f24a687c73380a7e

[fserb, 2017-04-07 20:36:56]

fserb@chromium.org changed reviewers:
+ junov@chromium.org

[Justin Novosad, 2017-04-09 01:54:07]

lgtm

[fserb, 2017-04-09 18:11:12]

The CQ bit was checked by fserb@chromium.org

[commit-bot: I haz the power, 2017-04-09 18:11:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-09 18:13:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-09 18:13:10]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[fserb, 2017-04-10 15:56:55]

The CQ bit was checked by fserb@chromium.org

[fserb, 2017-04-10 15:56:55]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2802353002/#ps20001
(title: "merge")

[commit-bot: I haz the power, 2017-04-10 15:57:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 17:23:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1491839815210910,
"parent_rev": "281faa09393f7448e894309581d531429abb4fa1", "commit_rev":
"67774f8b275a884f3571cbb0f24a687c73380a7e"}

[commit-bot: I haz the power, 2017-04-10 17:24:39]

Description was changed from

==========
getImageData now finalizesFrame and disableAcceleration updates composit

Both are needed for proper accelerate to software conversion.
getImageData wasn't calling needFinalizeFrame, so frames that only
called it, wouldn't be moved to software.
Also, when converting from accelerate, background-color/alpha wasn't
being properly propagated because we didn't updated compositing.

BUG=
==========

to

==========
getImageData now finalizesFrame and disableAcceleration updates composit

Both are needed for proper accelerate to software conversion.
getImageData wasn't calling needFinalizeFrame, so frames that only
called it, wouldn't be moved to software.
Also, when converting from accelerate, background-color/alpha wasn't
being properly propagated because we didn't updated compositing.

BUG=

Review-Url: https://codereview.chromium.org/2802353002
Cr-Commit-Position: refs/heads/master@{#463307}
Committed:
https://chromium.googlesource.com/chromium/src/+/67774f8b275a884f3571cbb0f24a...
==========

[commit-bot: I haz the power, 2017-04-10 17:24:40]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/67774f8b275a884f3571cbb0f24a...