stub out debug::StackTrace().Print() when using uclibc

stub out debug::StackTrace().Print() when using uclibc

This is a followup from https://codereview.chromium.org/228753003 which expands the subset of chromium that should be possible to build with uClibc.

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

[Mostyn Bramley-Moore, 2014-06-25 21:01:58]

This expands a previous CL that made it possible to build content with uClibc,
to include sync and content_tests.

@Paweł: PTAL at browser_test_base.cc

@Nicolas: PTAL at sync_client.cc - is is likely that embedded linux systems
would use this code (these are the most likely linux systems to use uClibc)?  If
not we should stop this part of the CL.

@Benjamin: can you test this, to see if it solves your build problems?

[Nicolas Zea, 2014-06-25 21:27:15]

Seems wrong to have to #ifdef out all calls of Print, rather than just have
Print do nothing if uclibc isn't supported.

That said, sync_client is a debug tool only, so should't hold up any other work.
sync LGTM

[Vitaly Buka (NO REVIEWS), 2014-06-25 22:11:32]

lgtm

[Vitaly Buka (NO REVIEWS), 2014-06-25 22:11:46]

lgtm

[commit-bot: I haz the power, 2014-06-25 22:13:00]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mostynb@opera.com/354873002/1

[commit-bot: I haz the power, 2014-06-26 01:43:50]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/25725)

[commit-bot: I haz the power, 2014-06-26 01:47:43]

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

[commit-bot: I haz the power, 2014-06-26 01:47:44]

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

[Mostyn Bramley-Moore, 2014-06-26 21:13:50]

@sky: can you please take a look at content/public/test/browser_test_base.cc ?

[sky, 2014-06-26 22:01:39]

One comment, could we instead make base::debug::StackTrace().Print a no-op in
this config so that we don't need to change all call sites?

[Mostyn Bramley-Moore, 2014-06-26 23:05:32]

On 2014/06/26 22:01:39, sky wrote:
> One comment, could we instead make base::debug::StackTrace().Print a no-op in
> this config so that we don't need to change all call sites?

Sure, done.

@Mark: does this look OK to you?  I have not modified stack_trace_unittest.cc,
since calling these stubs in unittests feels like a (small) waste of time for
uClibc devs.

[sky, 2014-06-26 23:31:12]

THis means you no longer need me:
-sky

[Mark Mentovai, 2014-06-27 03:23:42]

LGTM

[Mostyn Bramley-Moore, 2014-06-27 05:40:49]

The CQ bit was checked by mostynb@opera.com

[Mostyn Bramley-Moore, 2014-06-27 05:41:06]

The CQ bit was unchecked by mostynb@opera.com

[Mostyn Bramley-Moore, 2014-06-27 05:41:43]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2014-06-27 05:42:28]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mostynb@opera.com/354873002/20001

[commit-bot: I haz the power, 2014-06-27 09:21:12]

Change committed as 280289