Mojo: Add logging macros (and supporting code) for use in mojo/public/cpp.

Mojo: Add logging macros (and supporting code) for use in mojo/public/cpp.

The logging macros should be usable by anything that depends on
mojo/public/cpp/environment (i.e., everything except
mojo/public/cpp/{system,utility}).

The macros are a simplified version of the usual ones in base/logging.h,
omitting (D)CHECK_{EQ,NE,GT,...}. Also currently not present are
(D)VLOG, etc.

I'm considering also adding MOJO_(D)CHECK_OK(mojo_result) and
MOJO_(D)LOG_IF_NOT_OK(level, mojo_result), which might be used similarly
to (D)PCHECK and (D)PLOG (for errno/GetLastError()). But I'd do that
separately.

R=darin@chromium.org

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

[viettrungluu, 2014-06-19 04:10:57]

(Dependent on https://codereview.chromium.org/339403003/.)

[darin (slow to review), 2014-06-19 05:15:06]

LGTM

https://codereview.chromium.org/342653004/diff/1/mojo/public/cpp/environment/...
File mojo/public/cpp/environment/logging.h (right):

https://codereview.chromium.org/342653004/diff/1/mojo/public/cpp/environment/...
mojo/public/cpp/environment/logging.h:8: // TODO(vtl): It's weird that this is
in the environment directory, since its
perhaps we should have lib/ and lib/standalone/? dunno..

[viettrungluu, 2014-06-19 06:48:02]

On 2014/06/19 05:15:06, darin wrote:
> LGTM
> 
>
https://codereview.chromium.org/342653004/diff/1/mojo/public/cpp/environment/...
> File mojo/public/cpp/environment/logging.h (right):
> 
>
https://codereview.chromium.org/342653004/diff/1/mojo/public/cpp/environment/...
> mojo/public/cpp/environment/logging.h:8: // TODO(vtl): It's weird that this is
> in the environment directory, since its
> perhaps we should have lib/ and lib/standalone/? dunno..

My thought was that cpp/public/utility is a bit weird, since it's really "system
utility" (depending on cpp/public/system and things like pthreads, and nothing
more) ... which limits its usefulness somewhat.

(Of course, one might conceivably want to have utilities that depend on bindings
as well, in which case this logging stuff couldn't live there either.)

[viettrungluu, 2014-06-19 18:00:12]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-06-19 18:02:43]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/342653004/60001

[commit-bot: I haz the power, 2014-06-19 21:42:56]

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

[commit-bot: I haz the power, 2014-06-19 21:42:56]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[viettrungluu, 2014-06-19 22:11:38]

Committed patchset #4 manually as r278504.