Another try at fixing --enable-logging on Windows

Another try at fixing --enable-logging on Windows

Per details logged on connect bug and crbug, when linked as a Windows
subsystem binary, the _fileno() for stdin/stdout/stderr are incorrectly set
to 0,1,2 rather than all -2 (because there's no console attached). This 
was broken between VS2010->VS2012 and is still broken in the latest VS2013
update.

So, instead of only checking _fileno, we also check the underlying HANDLE
to confirm that it's valid for that stream before aborting.

BUG=358267
BUG=https://connect.microsoft.com/VisualStudio/feedback/details/785119/#tabs

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

[brettw, 2014-06-09 16:49:58]

lgtm

[scottmg, 2014-06-09 16:58:29]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2014-06-09 16:59:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/scottmg@chromium.org/324493003/1

[commit-bot: I haz the power, 2014-06-10 05:02:11]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_clang_dbg/bui...)

[commit-bot: I haz the power, 2014-06-10 11:04:16]

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

[commit-bot: I haz the power, 2014-06-10 11:04:17]

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

[scottmg, 2014-06-10 14:45:43]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2014-06-10 14:47:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/scottmg@chromium.org/324493003/1

[commit-bot: I haz the power, 2014-06-10 15:47:09]

Change committed as 276039