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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 22432002: Add app_current_window to app_container watchlist. (Closed)

Created:
7 years, 4 months ago by scheib
Modified:
7 years, 4 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add app_current_window to app_container watchlist. notry=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215942

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M WATCHLISTS View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
scheib
7 years, 4 months ago (2013-08-06 15:25:04 UTC) #1
Ken Rockot(use gerrit already)
On 2013/08/06 15:25:04, scheib wrote: lgtm
7 years, 4 months ago (2013-08-06 15:54:06 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
7 years, 4 months ago (2013-08-06 15:58:33 UTC) #3
scheib
;) Oh, yes, I perennially forget that even non OWNER covered files need a committer ...
7 years, 4 months ago (2013-08-06 16:03:00 UTC) #4
scheib
7 years, 4 months ago (2013-08-06 16:03:28 UTC) #5
asargent_no_longer_on_chrome
lgtm
7 years, 4 months ago (2013-08-06 17:07:53 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/scheib@chromium.org/22432002/1
7 years, 4 months ago (2013-08-06 17:41:41 UTC) #7
commit-bot: I haz the power
7 years, 4 months ago (2013-08-06 19:08:18 UTC) #8
Message was sent while issue was closed. 
Change committed as 215942

Powered by Google App Engine
This is Rietveld 408576698