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

Issue 21149003: Clear access code on successful connection. (Closed)

Created:
7 years, 4 months ago by Jamie
Modified:
7 years, 4 months ago
Reviewers:
Lambros
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, weitaosu+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Clear access code on successful connection. BUG=262203 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215774

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M remoting/webapp/client_screen.js View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Jamie
ptal
7 years, 4 months ago (2013-07-29 22:37:13 UTC) #1
Lambros
lgtm Do we also need to clear the field in the case that the user ...
7 years, 4 months ago (2013-07-30 23:14:13 UTC) #2
Jamie
On 2013/07/30 23:14:13, Lambros wrote: > lgtm > Do we also need to clear the ...
7 years, 4 months ago (2013-07-30 23:16:50 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jamiewalch@chromium.org/21149003/1
7 years, 4 months ago (2013-07-30 23:36:23 UTC) #4
commit-bot: I haz the power
Step "update" is always a major failure. Look at the try server FAQ for more ...
7 years, 4 months ago (2013-07-31 01:07:05 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jamiewalch@chromium.org/21149003/1
7 years, 4 months ago (2013-08-05 22:31:04 UTC) #6
commit-bot: I haz the power
7 years, 4 months ago (2013-08-06 02:59:38 UTC) #7
Message was sent while issue was closed.
Change committed as 215774

Powered by Google App Engine
This is Rietveld 408576698