This CL is a follow up to https://codereview.chromium.org/416683002/.

This CL is a follow up to https://codereview.chromium.org/416683002/.

It updates several comments and renames member variables to increase clarity and
better match the style guide.

R=wtc@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288282

[wtc, 2014-08-07 23:25:31]

Patch set 1 LGTM.

https://codereview.chromium.org/448293002/diff/1/net/socket/ssl_client_socket...
File net/socket/ssl_client_socket_openssl.h (right):

https://codereview.chromium.org/448293002/diff/1/net/socket/ssl_client_socket...
net/socket/ssl_client_socket_openssl.h:173: static void InfoCallback(const SSL*
ssl, int type, int /*val*/);

Nit: The Style Guide only recommends commenting out unused function parameters
in the definition. So in the declaration we don't need to comment out 'val'.

Here is an excerpt of the Style Guide:

If some parameters are unused, comment out the variable name in the function
definition:

// Always have named parameters in interfaces.
class Shape {
 public:
  virtual void Rotate(double radians) = 0;
}

// Always have named parameters in the declaration.
class Circle : public Shape {
 public:
  virtual void Rotate(double radians);
}

// Comment out unused named parameters in definitions.
void Circle::Rotate(double /*radians*/) {}

https://codereview.chromium.org/448293002/diff/1/net/test/spawned_test_server...
File net/test/spawned_test_server/base_test_server.h (right):

https://codereview.chromium.org/448293002/diff/1/net/test/spawned_test_server...
net/test/spawned_test_server/base_test_server.h:208: // disabled, the server
will generate an empty session ID in the

Nit: generate => "return" or "use"

[mshelley, 2014-08-07 23:34:49]

The CQ bit was checked by mshelley@chromium.org

[commit-bot: I haz the power, 2014-08-08 00:01:18]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mshelley@chromium.org/448293002/20001

[commit-bot: I haz the power, 2014-08-08 09:39:39]

Change committed as 288282

[wtc, 2014-08-08 12:03:42]

Patch set 2 LGTM.