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

Issue 227433011: Merge 261530 "Update chromoting host icons" (Closed)

Created:
6 years, 8 months ago by Lambros
Modified:
6 years, 8 months ago
Reviewers:
Lambros
CC:
chromium-reviews, chromoting-reviews_chromium.org
Visibility:
Public.

Description

Merge 261530 "Update chromoting host icons" > Update chromoting host icons > > Update the host icons to the new style which includes the Chrome logo. > This affects all chromoting components, including the Android app and > the webapp. > > All icons are based off the 512x512 PNG in the linked bug. > > The smaller sizes were generated with ImageMagick, for example: > convert chromoting512.png -resize 48x48 chromoting48.png > > ICO file was generated with: > convert chromoting16.png chromoting32.png chromoting48.png chromoting.ico > > WebP versions were generated using cwebp: > for i in chromoting*.png; do \ > cwebp -lossless -q 100 -m 6 $i -o ${i%.png}.webp; done > > Android empty_host_list icon was made by cropping 512x512 to 510x510, > then shrinking by a factor of 3, then converting to grayscale, then > darkening the image using GIMP and cropping to a few-pixels-wide > margin. > > All PNGs were optimized by running: > tools/resources/optimize-png-files.sh -o2 remoting/resources > > BUG=334830 > R=jamiewalch@chromium.org > > Review URL: https://codereview.chromium.org/222673002 TBR=lambroslambrou@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=262245

Patch Set 1 #

Messages

Total messages: 2 (0 generated)
Lambros
6 years, 8 months ago (2014-04-07 23:20:48 UTC) #1
Lambros
6 years, 8 months ago (2014-04-07 23:20:57 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r262245.

Powered by Google App Engine
This is Rietveld 408576698