Sandbox: Fix a violation to the syle plugin.

Sandbox: Fix a violation to the syle plugin.

BUG=426577
R=thakis@chromium.org

Committed: https://crrev.com/77c9ad9986492b1791139fe0dcc41fab00a40539
Cr-Commit-Position: refs/heads/master@{#338210}

[Nico, 2015-07-09 23:54:29]

Thanks!

The message is

../..\sandbox/win/src/sharedmem_ipc_server.h(76,3) :  error: [chromium-style]
Complex class/struct needs an explicit out-of-line constructor.
  struct ServerControl {
  ^
../..\sandbox/win/src/sharedmem_ipc_server.h(76,3) :  error: [chromium-style]
Complex class/struct needs an explicit out-of-line destructor.

so I think you need a constructor in addition to the destructor too.

lgtm with that

[rvargas (doing something else), 2015-07-10 00:04:54]

updated. Thanks.

[rvargas (doing something else), 2015-07-10 00:05:13]

The CQ bit was checked by rvargas@chromium.org

[rvargas (doing something else), 2015-07-10 00:05:14]

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/1228093004/#ps20001
(title: "But wait, we'll also bundle a constructor for the same price")

[commit-bot: I haz the power, 2015-07-10 00:06:38]

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

[Nico, 2015-07-10 02:35:34]

Looks like the cq has a problem. I filed
https://code.google.com/p/chromium/issues/detail?id=508783 for that and I'm
going to land this manually now.

[commit-bot: I haz the power, 2015-07-10 02:36:22]

Committed patchset #2 (id:20001)

[Nico, 2015-07-10 02:37:20]

On 2015/07/10 02:36:22, commit-bot: I haz the power wrote:
> Committed patchset #2 (id:20001)

Hm, looks like just complaining here was enough to make it go :-)

[commit-bot: I haz the power, 2015-07-10 02:37:28]

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