[Home] Fix top-offset for native pages

[Home] Fix top-offset for native pages

BUG=706683
Review-Url: https://codereview.chromium.org/2855033002
Cr-Commit-Position: refs/heads/master@{#469396}
Committed: https://chromium.googlesource.com/chromium/src/+/63c287aa94594af1018f6c3b966aa72e6f315801

[Theresa, 2017-05-02 22:32:28]

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

[commit-bot: I haz the power, 2017-05-02 22:33:05]

Dry run: CQ is trying da patch.

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

[Theresa, 2017-05-02 22:33:16]

twellington@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Theresa, 2017-05-02 22:33:16]

ptal

[commit-bot: I haz the power, 2017-05-02 23:01:05]

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

[commit-bot: I haz the power, 2017-05-02 23:01:06]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[Theresa, 2017-05-02 23:15:10]

I need to update NativePageFactoryTest.java to fix the compilation error.

[Theresa, 2017-05-04 16:23:57]

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

[commit-bot: I haz the power, 2017-05-04 16:24:19]

Dry run: CQ is trying da patch.

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

[gone, 2017-05-04 17:04:22]

https://codereview.chromium.org/2855033002/diff/20001/chrome/android/java/src...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsPage.java
(right):

https://codereview.chromium.org/2855033002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsPage.java:109:
ApiCompatibilityUtils.getPaddingStart(recentTabsRoot),
getPaddingEnd?

[Theresa, 2017-05-04 17:56:14]

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

[Theresa, 2017-05-04 17:56:15]

https://codereview.chromium.org/2855033002/diff/20001/chrome/android/java/src...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsPage.java
(right):

https://codereview.chromium.org/2855033002/diff/20001/chrome/android/java/src...
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsPage.java:109:
ApiCompatibilityUtils.getPaddingStart(recentTabsRoot),
On 2017/05/04 17:04:22, slow (dfalcantara) wrote:
> getPaddingEnd?

Done.

[gone, 2017-05-04 17:59:23]

lgtm

[Theresa, 2017-05-04 18:00:53]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2017-05-04 18:01:18]

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-05-04 18:41:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493920853779450,
"parent_rev": "c84e3575dfa679edeeb4efe7c906d86425117e5f", "commit_rev":
"63c287aa94594af1018f6c3b966aa72e6f315801"}

[commit-bot: I haz the power, 2017-05-04 18:41:37]

Description was changed from

==========
[Home] Fix top-offset for native pages

BUG=706683
==========

to

==========
[Home] Fix top-offset for native pages

BUG=706683

Review-Url: https://codereview.chromium.org/2855033002
Cr-Commit-Position: refs/heads/master@{#469396}
Committed:
https://chromium.googlesource.com/chromium/src/+/63c287aa94594af1018f6c3b966a...
==========

[commit-bot: I haz the power, 2017-05-04 18:41:38]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/63c287aa94594af1018f6c3b966a...