Re-enable accidentally disabled DragPanels test on Windows.

Re-enable accidentally disabled DragPanels test on Windows.

modified:   chrome/browser/ui/panels/panel_browsertest.cc

BUG=
TEST=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=97151

[scherkus (not reviewing), 2011-08-16 23:29:12]

http://codereview.chromium.org/7631012/diff/1002/chrome/browser/ui/panels/pan...
File chrome/browser/ui/panels/panel_browsertest.cc (right):

http://codereview.chromium.org/7631012/diff/1002/chrome/browser/ui/panels/pan...
chrome/browser/ui/panels/panel_browsertest.cc:308: (!defined(OS_WIN) &&
defined(TOOLKIT_VIEWS))
drive-by... I wonder if you could re-write this to have three sections:

#if win/gtk
// test is enabled
#else
// test is disabled
#endif


...in short have a whitelist of OSes that you know it passes as opposed to a
blacklist

[prasadt, 2011-08-16 23:40:26]

http://codereview.chromium.org/7631012/diff/1002/chrome/browser/ui/panels/pan...
File chrome/browser/ui/panels/panel_browsertest.cc (right):

http://codereview.chromium.org/7631012/diff/1002/chrome/browser/ui/panels/pan...
chrome/browser/ui/panels/panel_browsertest.cc:308: (!defined(OS_WIN) &&
defined(TOOLKIT_VIEWS))
On 2011/08/16 23:29:12, scherkus wrote:
> drive-by... I wonder if you could re-write this to have three sections:
> 
> #if win/gtk
> // test is enabled
> #else
> // test is disabled
> #endif
> 
> 
> ...in short have a whitelist of OSes that you know it passes as opposed to a
> blacklist

Jenn suggested that and I thought about that too before I sent this out.  I
didn't find a straightforward way to test for linux only that does not include
chromeos.  Uploaded a new cl that uses TOOLKIT_GTK for whitelisting linux.

[prasadt, 2011-08-16 23:47:24]

Removed comment about linux desktop as technically TOOLKIT_GTK is set for any
platform that uses GTK.  BSD and Solaris for example.

[jennb, 2011-08-16 23:56:53]

LGTM

So much prettier!  Thanks.