Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Issue 23578034: Change logos. (Closed)

Created:
7 years, 3 months ago by samarth
Modified:
7 years, 3 months ago
Reviewers:
jfweitz, Jered
CC:
chromium-reviews, skanuj+watch_chromium.org, melevin+watch_chromium.org, dhollowa+watch_chromium.org, dougw+watch_chromium.org, donnd+watch_chromium.org, mad+watch_chromium.org, dominich, jfweitz+watch_chromium.org, David Black, arv+watch_chromium.org, samarth+watch_chromium.org, kmadhusu+watch_chromium.org
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M chrome/browser/resources/local_ntp/images/2x/ntp_google_logo.png View Binary file 0 comments Download
M chrome/browser/resources/local_ntp/images/2x/ntp_white_google_logo.png View Binary file 0 comments Download
M chrome/browser/resources/local_ntp/local_ntp.css View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
samarth
PTAL.
7 years, 3 months ago (2013-09-16 18:12:00 UTC) #1
Jered
On 2013/09/16 18:12:00, samarth wrote: > PTAL. lgtm Please crush the pngs. I wish the ...
7 years, 3 months ago (2013-09-16 18:14:00 UTC) #2
samarth
They should already be crushed. Did you expect them to be smaller?
7 years, 3 months ago (2013-09-16 18:16:58 UTC) #3
jfweitz
lgtm
7 years, 3 months ago (2013-09-16 18:22:53 UTC) #4
Jered
On 2013/09/16 18:16:58, samarth wrote: > They should already be crushed. Did you expect them ...
7 years, 3 months ago (2013-09-16 18:23:04 UTC) #5
samarth
7 years, 3 months ago (2013-09-16 18:34:28 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r223372 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698