Use the NTP background color instead of white when there is no other info.

Use the NTP background color instead of white when there is no other info.

To do this, observe the RenderViewReady callback in ContentsWebView.

BUG=470669
Committed: https://crrev.com/dd94ab1cdce9d3b0e45d8b4689d0b4865fb2d942
Cr-Commit-Position: refs/heads/master@{#391081}

[chrishtr, 2016-05-02 18:51:24]

Description was changed from

==========
none

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

to

==========
Use the NTP background color instead of white when there is no other info.

To do this, observe the RenderViewReady callback in ContentsWebView.

BUG=470669
==========

[chrishtr, 2016-05-02 18:52:46]

chrishtr@chromium.org changed reviewers:
+ sky@chromium.org

[chrishtr, 2016-05-02 18:52:56]

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

[commit-bot: I haz the power, 2016-05-02 18:53:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1939913002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1939913002/40001

[commit-bot: I haz the power, 2016-05-02 19:44:42]

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

[commit-bot: I haz the power, 2016-05-02 19:44:43]

Dry run: This issue passed the CQ dry run.

[sky, 2016-05-02 20:04:33]

LGTM

https://codereview.chromium.org/1939913002/diff/40001/chrome/browser/ui/views...
File chrome/browser/ui/views/frame/contents_web_view.cc (right):

https://codereview.chromium.org/1939913002/diff/40001/chrome/browser/ui/views...
chrome/browser/ui/views/frame/contents_web_view.cc:72: if (rwhv)
nit: move inside if block on 70 (or early return if web_contents() is null).

[chrishtr, 2016-05-02 20:24:02]

https://codereview.chromium.org/1939913002/diff/40001/chrome/browser/ui/views...
File chrome/browser/ui/views/frame/contents_web_view.cc (right):

https://codereview.chromium.org/1939913002/diff/40001/chrome/browser/ui/views...
chrome/browser/ui/views/frame/contents_web_view.cc:72: if (rwhv)
On 2016/05/02 at 20:04:33, sky wrote:
> nit: move inside if block on 70 (or early return if web_contents() is null).

Done.

[chrishtr, 2016-05-02 20:24:08]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-05-02 20:24:08]

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

[commit-bot: I haz the power, 2016-05-02 20:24:28]

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

[commit-bot: I haz the power, 2016-05-02 22:47:21]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-02 22:48:40]

Description was changed from

==========
Use the NTP background color instead of white when there is no other info.

To do this, observe the RenderViewReady callback in ContentsWebView.

BUG=470669
==========

to

==========
Use the NTP background color instead of white when there is no other info.

To do this, observe the RenderViewReady callback in ContentsWebView.

BUG=470669

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

[commit-bot: I haz the power, 2016-05-02 22:48:41]

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