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

Issue 8893015: The chromoting client logs a session ID to the server, (Closed)

Created:
9 years ago by simonmorris
Modified:
9 years ago
Reviewers:
Jamie
CC:
chromium-reviews, jamiewalch+watch_chromium.org, hclam+watch_chromium.org, simonmorris+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

The chromoting client logs a session ID to the server, instead of a per-install ID. BUG=106208 TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114127

Patch Set 1 #

Patch Set 2 : Allow for session ID expiry. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+125 lines, -37 lines) Patch
M remoting/webapp/me2mom/log_to_server.js View 1 6 chunks +82 lines, -32 lines 0 comments Download
M remoting/webapp/me2mom/server_log_entry.js View 1 3 chunks +43 lines, -5 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
simonmorris
PTAL
9 years ago (2011-12-09 23:15:47 UTC) #1
simonmorris
This CL has session ID expiry now, so it's ready to be looked at.
9 years ago (2011-12-12 22:49:51 UTC) #2
Jamie
9 years ago (2011-12-12 23:25:54 UTC) #3
lgtm

Powered by Google App Engine
This is Rietveld 408576698