Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(1)

Issue 2885023002: Add NTPCondensedLayout to test config. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 days ago by Michael van Ouwerkerk
Modified:
2 days, 22 hours ago
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add NTPCondensedLayout to test config. BUG=688291 Review-Url: https://codereview.chromium.org/2885023002 Cr-Commit-Position: refs/heads/master@{#473225} Committed: https://chromium.googlesource.com/chromium/src/+/de26ab98dcd8747564fd3e078db2862a53cb7246

Patch Set 1 #

Patch Set 2 : Rebase. #

Patch Set 3 : Disable condensed layout in tests that require a logo. #

Messages

Total messages: 22 (16 generated)
Michael van Ouwerkerk
Bernhard: please take a look from feature perspective Alexei: please take a look from a ...
6 days ago (2017-05-16 14:48:45 UTC) #2
Bernhard Bauer
lgtm
6 days ago (2017-05-16 15:30:30 UTC) #6
Alexei Svitkine (slow)
lgtm
5 days, 22 hours ago (2017-05-16 16:52:54 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2885023002/40001
3 days ago (2017-05-19 15:26:07 UTC) #18
commit-bot: I haz the power
Committed patchset #3 (id:40001) as https://chromium.googlesource.com/chromium/src/+/de26ab98dcd8747564fd3e078db2862a53cb7246
2 days, 22 hours ago (2017-05-19 17:22:00 UTC) #21
mikecase
2 days, 16 hours ago (2017-05-19 23:35:11 UTC) #22
Message was sent while issue was closed.
A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2898533003/ by mikecase@chromium.org.

The reason for reverting is: crbug/724698

May be causing tests failures of NewTabPageTest. Small cl, so just going to
revert..
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 650457f06