Put all of SkPostConfig.h inside top level ifdef.

Put all of SkPostConfig.h inside top level ifdef.

Due to inconsistent indentation, the last third of this header is outside
of the top level ifdef header guard. This CL fixes that, as well as
makes the indents more consistent to avoid this issue in the future.

R=mtklein@google.com, reed@google.com

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

[bungeman-skia, 2013-12-03 15:38:31]

Let's fight over the style choices.

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h
File include/core/SkPostConfig.h (left):

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:241: #if 0
This was simply removed as it appears to be '#if 0'ed for some time now.

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:343: #endif
This '#endif' actually closes the header guard 'ifndef'.

[mtklein, 2013-12-03 16:55:17]

lgtm with small tweaks

(No surprise, seems I wrote SK_ALWAYS_INLINE in this style myself!)

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h
File include/core/SkPostConfig.h (right):

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:174: #    define SK_DEBUGBREAK(cond) do { if
(!(cond)) { SkNO_RETURN_HINT(); __debugbreak(); }} while (false)
While we're at this, can you wrap this to 100?

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:390: #endif
  // SkThing_DEFINED

[bungeman-skia, 2013-12-03 17:07:59]

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h
File include/core/SkPostConfig.h (right):

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:174: #    define SK_DEBUGBREAK(cond) do { if
(!(cond)) { SkNO_RETURN_HINT(); __debugbreak(); }} while (false)
On 2013/12/03 16:55:17, mtklein wrote:
> While we're at this, can you wrap this to 100?

Done.

https://codereview.chromium.org/102153002/diff/1/include/core/SkPostConfig.h#...
include/core/SkPostConfig.h:390: #endif
On 2013/12/03 16:55:17, mtklein wrote:
>   // SkThing_DEFINED

Done.

[reed1, 2013-12-04 16:33:05]

lgtm

[bungeman-chromium, 2013-12-04 16:40:44]

Committed patchset #2 manually as r12482.