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

Issue 222673002: Update chromoting host icons (Closed)

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

Description

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 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261530

Patch Set 1 : #

Messages

Total messages: 6 (0 generated)
Lambros
6 years, 8 months ago (2014-04-02 18:47:10 UTC) #1
Sergey Ulanov
The scaled images look blurry. Just tried scaling it in Gimp using Cubic and Lanczos ...
6 years, 8 months ago (2014-04-02 19:57:50 UTC) #2
Jamie
Since Sergey seems to be more discerning about these things than I am, I'm happy ...
6 years, 8 months ago (2014-04-02 20:54:34 UTC) #3
Sergey Ulanov
On 2014/04/02 20:54:34, Jamie wrote: > Since Sergey seems to be more discerning about these ...
6 years, 8 months ago (2014-04-02 21:31:19 UTC) #4
Jamie
On 2014/04/02 21:31:19, Sergey Ulanov wrote: > On 2014/04/02 20:54:34, Jamie wrote: > > Since ...
6 years, 8 months ago (2014-04-03 00:30:31 UTC) #5
Lambros
6 years, 8 months ago (2014-04-03 21:32:34 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r261530 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698