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

Issue 2689213012: DevTools: Add icons to performance pane tabs. (Closed)

Created:
3 years, 10 months ago by alph
Modified:
3 years, 10 months ago
Reviewers:
kozy, paulirish, caseq
CC:
apavlov+blink_chromium.org, blink-reviews, caseq+blink_chromium.org, chromium-reviews, devtools-reviews_chromium.org, kozyatinskiy+blink_chromium.org, lushnikov+blink_chromium.org, pfeldman+blink_chromium.org, pfeldman
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

DevTools: Add icons to performance pane tabs. BUG=693238 Review-Url: https://codereview.chromium.org/2689213012 Cr-Commit-Position: refs/heads/master@{#451321} Committed: https://chromium.googlesource.com/chromium/src/+/8376a0c3554ffb329e74e899f4277f7a80b52448

Patch Set 1 #

Messages

Total messages: 14 (6 generated)
alph
3 years, 10 months ago (2017-02-16 22:47:23 UTC) #2
paulirish
I like the icons. It adds value to the view behind each. lgtm I am ...
3 years, 10 months ago (2017-02-17 00:50:06 UTC) #3
alph
On 2017/02/17 00:50:06, paulirish wrote: > I like the icons. It adds value to the ...
3 years, 10 months ago (2017-02-17 00:52:31 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2689213012/1
3 years, 10 months ago (2017-02-17 00:53:56 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/121198)
3 years, 10 months ago (2017-02-17 02:52:22 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2689213012/1
3 years, 10 months ago (2017-02-17 16:28:43 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/8376a0c3554ffb329e74e899f4277f7a80b52448
3 years, 10 months ago (2017-02-17 17:12:29 UTC) #13
alph
3 years, 10 months ago (2017-02-24 02:53:15 UTC) #14
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2714923002/ by alph@chromium.org.

The reason for reverting is: Per UX folks request.

Powered by Google App Engine
This is Rietveld 408576698