Make flattenables no longer depend on global static initializers.

Make flattenables no longer depend on global static initializers.

Instead, force all builds to call InitializeFlattenables.

Remove the make_debugger script, which was created to force
rebuilding without global static initializers so that all flattenables
would be linked. It is no longer necessary since all flattenables
will be linked thanks to InitializeFlattenables, which now can (and
must) be called when global static initializers are turned on.

BUG=https://code.google.com/p/skia/issues/detail?id=903
BUG=https://code.google.com/p/skia/issues/detail?id=902

Committed: https://code.google.com/p/skia/source/detail?r=5642

[DerekS, 2012-09-21 16:34:38]

shouldn't there be some changes to the gyp files and SkPostConfig.h as we no
longer need to support that variable?

https://codereview.appspot.com/6548044/diff/50/src/core/SkPathEffect.cpp
File src/core/SkPathEffect.cpp (right):

https://codereview.appspot.com/6548044/diff/50/src/core/SkPathEffect.cpp#newc...
src/core/SkPathEffect.cpp:175:
///////////////////////////////////////////////////////////////////////////////
nit...remove this line

https://codereview.appspot.com/6548044/diff/50/tests/GLProgramsTest.cpp
File tests/GLProgramsTest.cpp (left):

https://codereview.appspot.com/6548044/diff/50/tests/GLProgramsTest.cpp#oldco...
tests/GLProgramsTest.cpp:32: return (int)(r->nextF() * (max-min)) + min;
seems kinda random to remove this here.  Any reason?

[Leon, 2012-09-21 16:42:57]

> shouldn't there be some changes to the gyp files and SkPostConfig.h as we
> no longer need to support that variable?

My first patch did that, but then I decided there was no reason to remove
GLProgramsTest, which requires global static initializers. Maybe it wouldn't be
too hard to modify it so that it doesn't, but I figured what is most important
is that SkFlattenables do not require them.

https://codereview.appspot.com/6548044/diff/50/src/core/SkPathEffect.cpp
File src/core/SkPathEffect.cpp (right):

https://codereview.appspot.com/6548044/diff/50/src/core/SkPathEffect.cpp#newc...
src/core/SkPathEffect.cpp:175:
///////////////////////////////////////////////////////////////////////////////
On 2012/09/21 16:34:38, DerekS wrote:
> nit...remove this line

Will fix in next patch

https://codereview.appspot.com/6548044/diff/50/tests/GLProgramsTest.cpp
File tests/GLProgramsTest.cpp (left):

https://codereview.appspot.com/6548044/diff/50/tests/GLProgramsTest.cpp#oldco...
tests/GLProgramsTest.cpp:32: return (int)(r->nextF() * (max-min)) + min;
On 2012/09/21 16:34:38, DerekS wrote:
> seems kinda random to remove this here.  Any reason?

It is unused, and causes a warning. I was looking at this test because it
requires STATIC_GLOBAL_INITIALIZERS. Does not need to be in this CL though.

[DerekS, 2012-09-21 17:38:40]

lgtm.