Toolbar Tor profile icon

[petemill]

Fix brave/brave-browser#1384

Added two sizes of vector icons since it's text and wanted to scale appropriately (the icon displays at different sizes for touch UI vs normal)

Submitter Checklist:

• Submitted a ticket for my issue if one did not already exist.
• Used Github auto-closing keywords in the commit message.
• Added/updated tests for this change (for new code or code which already has tests).
• Ran git rebase -i to squash commits (if needed).
• Tagged reviewers and labelled the pull request as needed.
• Request a security/privacy review as needed.
• Add appropriate QA labels (QA/Yes or QA/No) to include the closed issue in milestone

Test Plan:

• Open Tor Window
• Observe that Avatar icon is 'Tor' icon

Reviewer Checklist:

• New files have MPL-2.0 license header.
• Request a security/privacy review as needed.
• Adequate test coverage exists to prevent regressions
• Verify test plan is specified in PR before merging to source

[petemill]

A lot of overhead in this PR since we didn't seem to previously have anywhere to put Brave icons that weren't just replacements for existing Chromium icons. This creates such a project.

[simonhong]

LGTM

[simonhong]

Just question, don't we use same theme for guest window?
Oops, I wrote to wrong PR

[petemill]

@simonhong I think Guest Window uses a different flag on profile since the icon is unchanged:

[petemill]

0.56.x 2c4f4e2
0.55.x 513d352

[bsclifton]

Added merged/0.55.x label 😄

[petemill]

@bsclifton I had added the merged/X labels on to the issue at brave/brave-browser#1384 as I thought that's where they are meant to go since the brave-browser issues are separated from the branches and PR in this repo. Is that not right?

edit: ok this is wrong, those labels are only for brave-browser PRs. Thanks @bsclifton

[bsclifton]

You got it 😄 Just to recap for others, for this repo, we just label the PR with the earliest (lowest version) version which the change was merged to