Fix win style errors.

Fix win style errors.

Recently introduced at https://codereview.chromium.org/1180313007/.

BUG=493414
TBR=tsepez

Committed: https://crrev.com/f19538207d2f833192ea7e4ca367da91cdcc7a8b
Cr-Commit-Position: refs/heads/master@{#334741}

[erikchen, 2015-06-16 22:16:28]

erikchen@chromium.org changed reviewers:
+ thakis@chromium.org

[erikchen, 2015-06-16 22:16:28]

thakis: Please review.

Note that the errors don't show up on my local win machine, and they don't show
up on the trybots, so I have no idea if this fixes all of the problems. Are you
using a different gyp config to get the errors/warnings?

[Nico, 2015-06-16 22:21:00]

Thanks! lgtm with the comment.

One of the two errors was actually from
https://codereview.chromium.org/1185133006

See
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/h20a68y8Q_Q
for your question.

https://codereview.chromium.org/1179973004/diff/1/ipc/ipc_channel_win.h
File ipc/ipc_channel_win.h (right):

https://codereview.chromium.org/1179973004/diff/1/ipc/ipc_channel_win.h#newco...
ipc/ipc_channel_win.h:113: base::WeakPtrFactory<ChannelWin> weak_factory_;
You'll also have to change the order in the initializer list in the constructor
definition in the cc file.

[erikchen, 2015-06-16 22:24:16]

https://codereview.chromium.org/1179973004/diff/1/ipc/ipc_channel_win.h
File ipc/ipc_channel_win.h (right):

https://codereview.chromium.org/1179973004/diff/1/ipc/ipc_channel_win.h#newco...
ipc/ipc_channel_win.h:113: base::WeakPtrFactory<ChannelWin> weak_factory_;
On 2015/06/16 22:21:00, Nico wrote:
> You'll also have to change the order in the initializer list in the
constructor
> definition in the cc file.

So you're right. Thanks!

[erikchen, 2015-06-16 22:24:24]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2015-06-16 22:24:24]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1179973004/#ps10003
(title: "Comments from thakis.")

[commit-bot: I haz the power, 2015-06-16 22:25:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1179973004/10003

[commit-bot: I haz the power, 2015-06-16 22:36:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-16 22:36:53]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Nico, 2015-06-16 22:38:17]

thakis@chromium.org changed reviewers:
+ tsepez@chromium.org

[Nico, 2015-06-16 22:38:18]

+tsepez for ipc/owners

[Nico, 2015-06-16 22:38:40]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2015-06-16 22:39:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1179973004/10003

[Tom Sepez, 2015-06-16 22:49:10]

lgtm

[commit-bot: I haz the power, 2015-06-16 23:16:12]

Committed patchset #2 (id:10003)

[commit-bot: I haz the power, 2015-06-16 23:16:58]

Patchset 2 (id:??) landed as
https://crrev.com/f19538207d2f833192ea7e4ca367da91cdcc7a8b
Cr-Commit-Position: refs/heads/master@{#334741}