New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 595035 link

Starred by 3 users

Issue metadata

Status: Archived
Owner:
Closed: Sep 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Feature



Sign in to add a comment

Replace Skia vector icons with Cocoa equivalent

Project Member Reported by shrike@chromium.org, Mar 15 2016

Issue description

Right now the Mac version of Chrome uses Skia codepaths to generate NSImages from vector icons, which is inefficient on several levels. Chrome Mac should instead:

1. Create NSImages with NSCustomImageRep representations that contain the vector drawing code
2. Make it easy to set the vector image's color (rather than replace the whole image with one that uses the desired color).
 
Project Member

Comment 1 by sheriffbot@chromium.org, Jun 1 2016

Labels: -M-52 M-53 MovedFrom-52
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 2 by sdy@chromium.org, Jun 30 2016

Cc: shrike@chromium.org
Labels: -Type-Bug Type-Feature
Owner: sdy@chromium.org

Comment 3 by shrike@chromium.org, Jul 11 2016

On the Views side there's code that parses the vector icon files, and then more code that rasterizes each drawing directive. I think we want to add some code on the Mac side that takes the parsed drawing directives from Views and creates an NSBezierPath that's held by the NSCustomImageRep.

I have a small Mac program that does this NSBezierPath generation mostly correctly. When you're ready to start on this bug please ping me and I'll get it to you. I also think you should put together a design doc so that we can discuss the implementation before you get started writing code - I will help you get started with that as well.

Comment 4 by sdy@chromium.org, Jul 11 2016

Will do.  Issue 589943  is still first, right?

Comment 5 by shrike@chromium.org, Jul 11 2016

Correct.

Project Member

Comment 6 by sheriffbot@chromium.org, Jul 15 2016

Labels: -M-53 MovedFrom-53
This issue has been moved once and is lower than Pri-1. Removing the milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 7 by sdy@chromium.org, Feb 23 2017

Labels: -MovedFrom-52 -MovedFrom-53 Hotlist-GoodFirstBug HelpWanted
Status: Available (was: Assigned)

Comment 8 by shrike@chromium.org, Mar 28 2017

Labels: -HelpWanted -Hotlist-GoodFirstBug M-61
Owner: lgrey@chromium.org
Status: Assigned (was: Available)

Comment 9 by shrike@chromium.org, Mar 28 2017

Labels: -M-61 M-60
Status: Archived (was: Assigned)
Archiving old bugs that have only received trivial updates for some time.

If you feel this issue should still be addressed, feel free to reopen it or to file a new issue. Thanks!

Sign in to add a comment