Getting rid of immediate ack in DelegatedFrameHost (Relanding)

Getting rid of immediate ack in DelegatedFrameHost (Relanding)

This is a relanding of https://codereview.chromium.org/2710263002/ with a
fix. The original CL was causing flakes with
CaptureScreenshotTest.CaptureScreenshotArea. Apparently taking screenshots
depends on latency infos and latency infos somehow get lost when the frame
size is wrong.

BUG=692880
Review-Url: https://codereview.chromium.org/2711263004
Cr-Commit-Position: refs/heads/master@{#452924}
Committed: https://chromium.googlesource.com/chromium/src/+/7f5ee4fc924e13877c064c4b73388cdffff76ecd

[Saman Sami, 2017-02-24 19:51:20]

The CQ bit was checked by samans@chromium.org to run a CQ dry run

[Saman Sami, 2017-02-24 19:51:53]

Description was changed from

==========
Getting rid of immediate ack in DelegatedFrameHost (Relanding)

This is a relanding of https://codereview.chromium.org/2710263002/ with a
fix. The original CL was causing flakes with
CaptureScreenshotTest.CaptureScreenshotArea. Apparently taking screenshots
depends on latency infos and latency infos somehow get lost when the frame
size is wrong.

BUG=692880
==========

to

==========
Getting rid of immediate ack in DelegatedFrameHost (Relanding)

This is a relanding of https://codereview.chromium.org/2710263002/ with a
fix. The original CL was causing flakes with
CaptureScreenshotTest.CaptureScreenshotArea. Apparently taking screenshots
depends on latency infos and latency infos somehow get lost when the frame
size is wrong.

BUG=692880
==========

[Saman Sami, 2017-02-24 19:51:53]

samans@chromium.org changed reviewers:
+ fsamuel@chromium.org

[commit-bot: I haz the power, 2017-02-24 19:52:02]

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

[Saman Sami, 2017-02-24 20:11:00]

fsamuel@: PTAL

[Fady Samuel, 2017-02-24 20:14:12]

LGTM. I guess we can figure out what is going on with LatencyInfo afterward.

[Saman Sami, 2017-02-24 20:16:25]

On 2017/02/24 20:14:12, Fady Samuel wrote:
> LGTM. I guess we can figure out what is going on with LatencyInfo afterward.

I actually know to some extent how LatencyInfo is used for screenshots. What I'm
not sure is how the frames of wrong size get lost after submission to
SurfaceFactory. Maybe in Display::DrawAndSwap?

[Saman Sami, 2017-02-24 20:29:39]

The CQ bit was unchecked by samans@chromium.org

[Saman Sami, 2017-02-24 20:29:39]

The CQ bit was checked by samans@chromium.org

[commit-bot: I haz the power, 2017-02-24 20:29:54]

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-02-24 21:23:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487968179921330, "parent_rev":
"453ee830bdd7e8a3ac3f05c0f348deb75f35f4ae", "commit_rev":
"7f5ee4fc924e13877c064c4b73388cdffff76ecd"}

[commit-bot: I haz the power, 2017-02-24 21:23:49]

Description was changed from

==========
Getting rid of immediate ack in DelegatedFrameHost (Relanding)

This is a relanding of https://codereview.chromium.org/2710263002/ with a
fix. The original CL was causing flakes with
CaptureScreenshotTest.CaptureScreenshotArea. Apparently taking screenshots
depends on latency infos and latency infos somehow get lost when the frame
size is wrong.

BUG=692880
==========

to

==========
Getting rid of immediate ack in DelegatedFrameHost (Relanding)

This is a relanding of https://codereview.chromium.org/2710263002/ with a
fix. The original CL was causing flakes with
CaptureScreenshotTest.CaptureScreenshotArea. Apparently taking screenshots
depends on latency infos and latency infos somehow get lost when the frame
size is wrong.

BUG=692880

Review-Url: https://codereview.chromium.org/2711263004
Cr-Commit-Position: refs/heads/master@{#452924}
Committed:
https://chromium.googlesource.com/chromium/src/+/7f5ee4fc924e13877c064c4b7338...
==========

[commit-bot: I haz the power, 2017-02-24 21:23:50]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/7f5ee4fc924e13877c064c4b7338...