Disable warning C4702 in logging.cc

Disable warning C4702 in logging.cc

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

[Sébastien Marchand, 2014-06-03 18:32:57]

PTAL, we still need to fix the root cause of this but in the meantime this will
unblock my work on the PGO builder.

[Peter Kasting, 2014-06-03 18:34:09]

LGTM

[Sébastien Marchand, 2014-06-03 18:34:24]

The CQ bit was checked by sebmarchand@chromium.org

[commit-bot: I haz the power, 2014-06-03 18:35:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sebmarchand@chromium.org/315663003/1

[Sébastien Marchand, 2014-06-03 18:45:40]

The CQ bit was unchecked by sebmarchand@chromium.org

[Sébastien Marchand, 2014-06-03 18:47:38]

Putting this behind a "#if defined(OS_WIN)" condition could be a good idea...

[Sébastien Marchand, 2014-06-03 18:47:51]

PTAnL

[Peter Kasting, 2014-06-03 18:51:13]

LGTM

It's too bad we can't put this around the EAT_STREAM_PARAMETERS define.  I hate
the C preprocessor.

(I wonder if this problem would get easier if we converted some of these defines
into actual functions.)

[Sébastien Marchand, 2014-06-03 18:52:09]

The CQ bit was checked by sebmarchand@chromium.org

[commit-bot: I haz the power, 2014-06-03 18:52:41]

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

[scottmg, 2014-06-03 18:58:28]

It can go in the macro by using __pragma instead of #pragma. I think that would
be better than disabling it at the one callsite.

[Sébastien Marchand, 2014-06-03 19:00:32]

Yeah, I just wanted to fix this temporarily until we decide on the proper way to
fix the bug.

[Sébastien Marchand, 2014-06-03 21:36:35]

+brettw@ for owner approval

[brettw, 2014-06-03 22:55:19]

lgtm

[commit-bot: I haz the power, 2014-06-04 09:10:13]

Change committed as 274738