Chromium Code Reviews
Help | Chromium Project | Sign in
(6)

Issue 12221030: Add WebP icon files. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 6 months ago by Stephen
Modified:
2 years, 6 months ago
Reviewers:
Sergey Ulanov, Wez
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Add WebP icon files. This change adds the necessary WebP icon files in order to support changing our icons to WebP. A separate CL will be used to actually wire them up. BUG=169382 TEST=Manual TBR=wez@chromium.org NOTRY=True Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=180860

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -0 lines) Patch
A remoting/resources/chromoting128.webp View Binary file 0 comments Download
A remoting/resources/chromoting16.webp View Binary file 0 comments Download
A remoting/resources/chromoting256.webp View Binary file 0 comments Download
A remoting/resources/chromoting32.webp View Binary file 0 comments Download
A remoting/resources/chromoting48.webp View Binary file 0 comments Download
A remoting/resources/chromoting512.webp View Binary file 0 comments Download
A remoting/resources/disclosure_arrow_down.webp View Binary file 0 comments Download
A remoting/resources/disclosure_arrow_right.webp View Binary file 0 comments Download
A remoting/resources/host_setup_instructions.webp View Binary file 0 comments Download
A remoting/resources/icon_cross.webp View Binary file 0 comments Download
A remoting/resources/icon_host.webp View Binary file 0 comments Download
A remoting/resources/icon_pencil.webp View Binary file 0 comments Download
A remoting/resources/icon_warning.webp View Binary file 0 comments Download
A remoting/resources/infographic_my_computers.webp View Binary file 0 comments Download
A remoting/resources/infographic_remote_assistance.webp View Binary file 0 comments Download
A remoting/resources/tick.webp View Binary file 0 comments Download
A remoting/webapp/dividerbottom.webp View Binary file 0 comments Download
A remoting/webapp/dividertop.webp View Binary file 0 comments Download
A remoting/webapp/scale-to-fit.webp View Binary file 0 comments Download
Trybot results:
Project "None" does not have a commit queue.

Messages

Total messages: 12 (0 generated)
Stephen
2 years, 6 months ago (2013-02-06 01:08:09 UTC) #1
commit-bot: I haz the power
No reviewers yet.
2 years, 6 months ago (2013-02-06 01:10:10 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
2 years, 6 months ago (2013-02-06 01:24:34 UTC) #3
Sergey Ulanov
LGTM
2 years, 6 months ago (2013-02-06 01:43:37 UTC) #4
Sergey Ulanov
I think you need to add NOTRY=true in the description, so CQ can land it ...
2 years, 6 months ago (2013-02-06 01:45:11 UTC) #5
Stephen
On 2013/02/06 01:45:11, sergeyu wrote: > I think you need to add NOTRY=true in the ...
2 years, 6 months ago (2013-02-06 01:46:42 UTC) #6
Stephen
2 years, 6 months ago (2013-02-06 01:47:07 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/skonig@chromium.org/12221030/1
2 years, 6 months ago (2013-02-06 01:47:25 UTC) #8
commit-bot: I haz the power
Can't process patch for file remoting/resources/chromoting48.webp. Binary file support is temporarilly disabled due to a ...
2 years, 6 months ago (2013-02-06 01:47:27 UTC) #9
Sergey Ulanov
On 2013/02/06 01:47:27, I haz the power (commit-bot) wrote: > Can't process patch for file ...
2 years, 6 months ago (2013-02-06 01:49:38 UTC) #10
Sergey Ulanov
Committed manually as r180860 (presubmit successful).
2 years, 6 months ago (2013-02-06 02:11:53 UTC) #11
Stephen
2 years, 6 months ago (2013-02-06 05:24:46 UTC) #12
Message was sent while issue was closed.
On 2013/02/06 02:11:53, sergeyu wrote:
> Committed manually as r180860 (presubmit successful).

Thanks!
Sign in to reply to this message.

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