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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 10389158: Fix typo (Closed)

Created:
8 years, 7 months ago by amit
Modified:
8 years, 7 months ago
Reviewers:
Philippe, mmenke
CC:
chromium-reviews, cbentzel+watch_chromium.org, darin-cc_chromium.org
Visibility:
Public.

Description

Fix typo Remove unnecessary ';' that's messing with xcode 4. BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137456

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M net/socket/ssl_client_socket_openssl.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Philippe
lgtm
8 years, 7 months ago (2012-05-16 07:47:15 UTC) #1
Philippe
On 2012/05/16 07:47:15, Philippe wrote: > lgtm Thanks for fixing this!
8 years, 7 months ago (2012-05-16 07:52:13 UTC) #2
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
8 years, 7 months ago (2012-05-16 16:15:55 UTC) #3
mmenke
On 2012/05/16 16:15:55, I haz the power (commit-bot) wrote: > No LGTM from a valid ...
8 years, 7 months ago (2012-05-16 16:23:17 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/amit@chromium.org/10389158/1
8 years, 7 months ago (2012-05-16 16:31:18 UTC) #5
commit-bot: I haz the power
8 years, 7 months ago (2012-05-16 18:10:20 UTC) #6 
Change committed as 137456

Powered by Google App Engine
This is Rietveld 408576698