Avoid repeating frame info calculation in Java layer

Avoid repeating frame info calculation in Java layer

Native layer is already calculating some frame info about
top control (position + update flag). This CL passes
the values to Java layer so that it won't repeat those
operations again.

Review-Url: https://codereview.chromium.org/2841033002
Cr-Commit-Position: refs/heads/master@{#467871}
Committed: https://chromium.googlesource.com/chromium/src/+/61654002540ff418d34c869a745aadd4d6e71c59

[Jinsuk Kim, 2017-04-27 00:24:21]

Description was changed from

==========
Avoid repeating frame info in Java layer

Native layer is already calculating the frame info about
top control. This CL passes the values to Java layer so that it
won't repeat those operations again.
==========

to

==========
Avoid repeating frame info calculation in Java layer

Native layer is already calculating some frame info about
top control (position + update flag). This CL passes
the values to Java layer so that it won't repeat those
operations again.
==========

[Jinsuk Kim, 2017-04-27 05:12:53]

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

[boliu, 2017-04-27 17:30:50]

lgtm % comment

https://codereview.chromium.org/2841033002/diff/1/content/browser/android/con...
File content/browser/android/content_view_core_impl.h (right):

https://codereview.chromium.org/2841033002/diff/1/content/browser/android/con...
content/browser/android/content_view_core_impl.h:255: // All sizes and offsets
are in CSS pixels as cached by the renderer.
update comment

[Jinsuk Kim, 2017-04-27 23:47:03]

https://codereview.chromium.org/2841033002/diff/1/content/browser/android/con...
File content/browser/android/content_view_core_impl.h (right):

https://codereview.chromium.org/2841033002/diff/1/content/browser/android/con...
content/browser/android/content_view_core_impl.h:255: // All sizes and offsets
are in CSS pixels as cached by the renderer.
On 2017/04/27 17:30:50, boliu wrote:
> update comment

Done.

[Jinsuk Kim, 2017-04-28 02:13:04]

The CQ bit was checked by jinsukkim@chromium.org

[Jinsuk Kim, 2017-04-28 02:13:04]

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

[commit-bot: I haz the power, 2017-04-28 02:14:09]

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-28 03:27:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493345584133860,
"parent_rev": "4bceb9805881b8c25d39efe3c94b0c5da714a556", "commit_rev":
"61654002540ff418d34c869a745aadd4d6e71c59"}

[commit-bot: I haz the power, 2017-04-28 03:27:58]

Description was changed from

==========
Avoid repeating frame info calculation in Java layer

Native layer is already calculating some frame info about
top control (position + update flag). This CL passes
the values to Java layer so that it won't repeat those
operations again.
==========

to

==========
Avoid repeating frame info calculation in Java layer

Native layer is already calculating some frame info about
top control (position + update flag). This CL passes
the values to Java layer so that it won't repeat those
operations again.

Review-Url: https://codereview.chromium.org/2841033002
Cr-Commit-Position: refs/heads/master@{#467871}
Committed:
https://chromium.googlesource.com/chromium/src/+/61654002540ff418d34c869a745a...
==========

[commit-bot: I haz the power, 2017-04-28 03:27:59]

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