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

Issue 603623002: Rename PromisesPanel to PromisePane. (Closed)

Created:
6 years, 3 months ago by Alexandra Mikhaylova
Modified:
6 years, 2 months ago
Reviewers:
apavlov, vsevik, aandrey
CC:
blink-reviews, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, malch+blink_chromium.org, yurys+blink_chromium.org, lushnikov+blink_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, paulirish+reviews_chromium.org, apavlov+blink_chromium.org, devtools-reviews_chromium.org, sergeyv+blink_chromium.org, aandrey+blink_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Rename PromisesPanel to PromisePane. BUG=348919 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184365

Patch Set 1 #

Total comments: 2

Patch Set 2 : Address comments #

Patch Set 3 : REBASE #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -268 lines) Patch
M Source/devtools/devtools.gypi View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
A + Source/devtools/front_end/promises/PromisePane.js View 1 2 6 chunks +6 lines, -7 lines 0 comments Download
D Source/devtools/front_end/promises/PromisesPanel.js View 1 2 1 chunk +0 lines, -201 lines 0 comments Download
M Source/devtools/front_end/promises/module.json View 1 2 2 chunks +4 lines, -4 lines 0 comments Download
A + Source/devtools/front_end/promises/promisePane.css View 1 2 0 chunks +-1 lines, --1 lines 0 comments Download
D Source/devtools/front_end/promises/promisesPanel.css View 1 2 1 chunk +0 lines, -55 lines 0 comments Download

Messages

Total messages: 17 (6 generated)
Alexandra Mikhaylova
6 years, 3 months ago (2014-09-24 14:49:28 UTC) #2
aandrey
lgtm https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js File Source/devtools/front_end/promises/PromisePane.js (right): https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js#newcode16 Source/devtools/front_end/promises/PromisePane.js:16: buttonsBar.classList.add("promise-tracker-controls"); var buttonsBar = this.element.createChild("div", "promise-tracker-controls");
6 years, 3 months ago (2014-09-24 14:51:39 UTC) #3
Alexandra Mikhaylova
https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js File Source/devtools/front_end/promises/PromisePane.js (right): https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js#newcode16 Source/devtools/front_end/promises/PromisePane.js:16: buttonsBar.classList.add("promise-tracker-controls"); On 2014/09/24 14:51:39, aandrey wrote: > var buttonsBar ...
6 years, 3 months ago (2014-09-24 15:20:46 UTC) #4
Alexandra Mikhaylova
On 2014/09/24 15:20:46, Alexandra Mikhaylova wrote: > https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js > File Source/devtools/front_end/promises/PromisePane.js (right): > > https://codereview.chromium.org/603623002/diff/1/Source/devtools/front_end/promises/PromisePane.js#newcode16 ...
6 years, 2 months ago (2014-09-25 14:53:13 UTC) #6
aandrey
ping?
6 years, 2 months ago (2014-10-24 11:44:11 UTC) #7
vsevik
rs lgtm
6 years, 2 months ago (2014-10-24 11:49:31 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/603623002/20001
6 years, 2 months ago (2014-10-24 11:50:45 UTC) #11
commit-bot: I haz the power
Failed to apply patch for Source/devtools/front_end/promises/PromisePane.js: While running patch -p1 --forward --force --no-backup-if-mismatch; A Source/devtools/front_end/promises/PromisePane.js ...
6 years, 2 months ago (2014-10-24 11:51:05 UTC) #13
Alexandra Mikhaylova
On 2014/10/24 11:51:05, I haz the power (commit-bot) wrote: > Failed to apply patch for ...
6 years, 2 months ago (2014-10-24 14:16:37 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/603623002/40001
6 years, 2 months ago (2014-10-24 14:17:09 UTC) #16
commit-bot: I haz the power
6 years, 2 months ago (2014-10-24 15:49:49 UTC) #17
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as 184365

Powered by Google App Engine
This is Rietveld 408576698