Remove layer for readback

Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537
Committed: https://crrev.com/b5e173cdb92e95a18b35deca996b4463033996a3
Cr-Commit-Position: refs/heads/master@{#379230}

[no sievers, 2016-02-25 23:26:11]

sievers@chromium.org changed reviewers:
+ sievers@chromium.org

[no sievers, 2016-02-25 23:26:11]

https://codereview.chromium.org/1732323002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/1732323002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/render_widget_host_view_android.cc:911:
compositor->RequestCopyOfOutputOnRootLayer(std::move(request));
You want to use |layer_| here or you'll get the whole output (with the omnibox,
or even the whole tab stack if you happen to be inside that).

[Jinsuk Kim, 2016-02-26 00:53:17]

Description was changed from

==========
Remove layer for readback

BUG=574537
==========

to

==========
Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest
fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537
==========

[Jinsuk Kim, 2016-02-26 00:53:30]

https://codereview.chromium.org/1732323002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/1732323002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/render_widget_host_view_android.cc:911:
compositor->RequestCopyOfOutputOnRootLayer(std::move(request));
On 2016/02/25 23:26:11, sievers wrote:
> You want to use |layer_| here or you'll get the whole output (with the
omnibox,
> or even the whole tab stack if you happen to be inside that).

Thanks for helping out. Fixed.

[no sievers, 2016-02-26 18:32:28]

lgtm

[Jinsuk Kim, 2016-03-02 03:56:14]

jinsukkim@chromium.org changed reviewers:
+ dtrainor@chromium.org

[Jinsuk Kim, 2016-03-02 03:56:15]

dtrainor@chromium.org: Please review changes in ui/android

[David Trainor- moved to gerrit, 2016-03-04 05:13:13]

lgtm

[Jinsuk Kim, 2016-03-04 05:14:02]

The CQ bit was checked by jinsukkim@chromium.org

[commit-bot: I haz the power, 2016-03-04 05:14:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1732323002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1732323002/20001

[commit-bot: I haz the power, 2016-03-04 06:32:37]

Description was changed from

==========
Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest
fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537
==========

to

==========
Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest
fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537
==========

[commit-bot: I haz the power, 2016-03-04 06:32:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-04 06:33:45]

Description was changed from

==========
Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest
fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537
==========

to

==========
Remove layer for readback

A dedicated layer for readback is not necessary any more thanks to the latest
fixes crrev.com/1398633003, etc. Removed related logic.

BUG=574537

Committed: https://crrev.com/b5e173cdb92e95a18b35deca996b4463033996a3
Cr-Commit-Position: refs/heads/master@{#379230}
==========

[commit-bot: I haz the power, 2016-03-04 06:33:46]

Patchset 2 (id:??) landed as
https://crrev.com/b5e173cdb92e95a18b35deca996b4463033996a3
Cr-Commit-Position: refs/heads/master@{#379230}