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

Issue 292203010: Test commit (Closed)

Created:
6 years, 7 months ago by mhm
Modified:
6 years, 7 months ago
Reviewers:
jww
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Just a test commit. NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273005

Patch Set 1 #

Total comments: 2

Patch Set 2 : Removing usernames from the code #

Patch Set 3 : A test whitespace file modification to try to commit a cl #

Patch Set 4 : Adding a white space to testing uplaoding a CL #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M build/whitespace_file.txt View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 16 (0 generated)
mhm
Please review my first bug
6 years, 7 months ago (2014-05-22 22:57:57 UTC) #1
jww
https://codereview.chromium.org/292203010/diff/1/build/whitespace_file.txt File build/whitespace_file.txt (right): https://codereview.chromium.org/292203010/diff/1/build/whitespace_file.txt#newcode114 build/whitespace_file.txt:114: This is a line for jww to test and ...
6 years, 7 months ago (2014-05-22 23:00:30 UTC) #2
mhm
Removing usernames from the code https://codereview.chromium.org/292203010/diff/1/build/whitespace_file.txt File build/whitespace_file.txt (right): https://codereview.chromium.org/292203010/diff/1/build/whitespace_file.txt#newcode114 build/whitespace_file.txt:114: This is a line ...
6 years, 7 months ago (2014-05-22 23:06:44 UTC) #3
jww
On 2014/05/22 23:06:44, mhm wrote: > Removing usernames from the code > > https://codereview.chromium.org/292203010/diff/1/build/whitespace_file.txt > ...
6 years, 7 months ago (2014-05-22 23:17:02 UTC) #4
mhm
The CQ bit was checked by mohammed@chromium.org
6 years, 7 months ago (2014-05-27 17:08:26 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mohammed@chromium.org/292203010/2
6 years, 7 months ago (2014-05-27 17:09:09 UTC) #6
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-05-27 17:10:44 UTC) #7
commit-bot: I haz the power
Failed to apply patch for build/whitespace_file.txt: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 7 months ago (2014-05-27 17:10:45 UTC) #8
mhm
6 years, 7 months ago (2014-05-27 18:43:19 UTC) #9
mhm
The CQ bit was checked by mohammed@chromium.org
6 years, 7 months ago (2014-05-27 18:43:29 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mohammed@chromium.org/292203010/30001
6 years, 7 months ago (2014-05-27 18:44:26 UTC) #11
jww
The CQ bit was unchecked by jww@chromium.org
6 years, 7 months ago (2014-05-27 18:45:06 UTC) #12
jww
On 2014/05/27 18:45:06, jww wrote: > The CQ bit was unchecked by mailto:jww@chromium.org This is ...
6 years, 7 months ago (2014-05-27 18:45:25 UTC) #13
mhm
6 years, 7 months ago (2014-05-27 19:00:18 UTC) #14
jww
lgtm
6 years, 7 months ago (2014-05-27 19:03:52 UTC) #15
commit-bot: I haz the power
6 years, 7 months ago (2014-05-27 19:06:50 UTC) #16
Message was sent while issue was closed.
Change committed as 273005

Powered by Google App Engine
This is Rietveld 408576698