Revert windows-specific part of r15937 for debug level 1.

Revert windows-specific part of r15937 for debug level 1.

The change in r15937 might have broken a mozilla test under windows. See https://codereview.chromium.org/19384011/

Each case of debug 0,1,2 is now stated explicitly for better clarity in the msvs section. This causes some minor code duplications, which we can squeeze again after toolchain.gypi is in a stable state.

R=dpranke@chromium.org, jkummerow@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=16018

[Michael Achenbach, 2013-08-01 08:05:08]

PTAL

[Jakob Kummerow, 2013-08-01 09:42:48]

lgtm

[Dirk Pranke, 2013-08-01 17:14:25]

lgtm also. Did this fix the problem?

https://codereview.chromium.org/21495002/diff/1/build/toolchain.gypi
File build/toolchain.gypi (left):

https://codereview.chromium.org/21495002/diff/1/build/toolchain.gypi#oldcode67
build/toolchain.gypi:67: #     This may be hard or impossible to debug. This may
still be
Good fix. This comment was just wrong.

[Jakob Kummerow, 2013-08-01 17:19:32]

On 2013/08/01 17:14:25, Dirk Pranke wrote:
> lgtm also. Did this fix the problem?

The problem has been fixed by https://chromiumcodereview.appspot.com/20607005. I
still think we should land this patch too.

[Dirk Pranke, 2013-08-01 17:21:04]

On 2013/08/01 17:19:32, Jakob wrote:
> On 2013/08/01 17:14:25, Dirk Pranke wrote:
> > lgtm also. Did this fix the problem?
> 
> The problem has been fixed by https://chromiumcodereview.appspot.com/20607005.
I
> still think we should land this patch too.

Ah. I feel better that it wasn't this change that caused that problem. We should
definitely fix the comment, and I'm fine w/ landing this patch and consolidating
more slowly when we're sure things are stable.

[Michael Achenbach, 2013-08-02 07:58:28]

Committed patchset #1 manually as r16018 (presubmit successful).