Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on argb visual flag

Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on argb visual flag (patchset #30 id:580001 of https://codereview.chromium.org/1998653002/ )

Reason for revert:
The unittests added in widget_unittest.cc fail on Win10. See the report with failure output here:

https://chromium-swarm.appspot.com/user/task/2f9a5518a647e510 (scroll down to see the output of running the tests)

Original issue's description:
> Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on argb visual flag
>
> If depth is 32 and if there is a compositing manager then transparency is supported.
>
> Do not force transparency if Widget::InitParams::TRANSLUCENT_WINDOW is set whereas transparency is not supported.
> Which is the same behavior as on OS_WIN.
>
> Also add a new test WidgetTest.Transparency_DesktopWidget to widget_unittest.cc which ensures that transparency state of the native window matches what returns ShouldWindowContentsBeTransparent().
>
> BUG=589509
>
> R=erg@chromium.org, mgiuca@chromium.org, sadrul@chromium.org
>
> TEST=views_unittests --gtest_filter=*WidgetTest.Transparency*
>
> CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/29d8c51452a83dcbc359679e8ce36af41af64e36
> Cr-Commit-Position: refs/heads/master@{#401826}

TBR=sadrul@chromium.org,jbauman@chromium.org,piman@chromium.org,sky@chromium.org,erg@chromium.org,mgiuca@chromium.org,j.isorce@samsung.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589509
Committed: https://crrev.com/b6287c064906f9a2139c4be8499befe5cc572e5d
Cr-Commit-Position: refs/heads/master@{#401835}

[blundell, 2016-06-24 08:50:13]

The CQ bit was checked by blundell@chromium.org

[blundell, 2016-06-24 08:50:13]

Created Revert of Make
DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on argb
visual flag

[commit-bot: I haz the power, 2016-06-24 08:51:09]

Description was changed from

==========
Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent()
relies on argb visual flag (patchset #30 id:580001 of
https://codereview.chromium.org/1998653002/ )

Reason for revert:
The unittests added in widget_unittest.cc fail on Win10. See the report with
failure output here:

https://chromium-swarm.appspot.com/user/task/2f9a5518a647e510 (scroll down to
see the output of running the tests)

Original issue's description:
> Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on
argb visual flag
> 
> If depth is 32 and if there is a compositing manager then transparency is
supported.
> 
> Do not force transparency if Widget::InitParams::TRANSLUCENT_WINDOW is set
whereas transparency is not supported.
> Which is the same behavior as on OS_WIN.
> 
> Also add a new test WidgetTest.Transparency_DesktopWidget to
widget_unittest.cc which ensures that transparency state of the native window
matches what returns ShouldWindowContentsBeTransparent().
> 
> BUG=589509
> 
> R=erg@chromium.org, mgiuca@chromium.org, sadrul@chromium.org
> 
> TEST=views_unittests --gtest_filter=*WidgetTest.Transparency*
> 
>
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/29d8c51452a83dcbc359679e8ce36af41af64e36
> Cr-Commit-Position: refs/heads/master@{#401826}

TBR=sadrul@chromium.org,jbauman@chromium.org,piman@chromium.org,sky@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589509
==========

to

==========
Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent()
relies on argb visual flag (patchset #30 id:580001 of
https://codereview.chromium.org/1998653002/ )

Reason for revert:
The unittests added in widget_unittest.cc fail on Win10. See the report with
failure output here:

https://chromium-swarm.appspot.com/user/task/2f9a5518a647e510 (scroll down to
see the output of running the tests)

Original issue's description:
> Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on
argb visual flag
> 
> If depth is 32 and if there is a compositing manager then transparency is
supported.
> 
> Do not force transparency if Widget::InitParams::TRANSLUCENT_WINDOW is set
whereas transparency is not supported.
> Which is the same behavior as on OS_WIN.
> 
> Also add a new test WidgetTest.Transparency_DesktopWidget to
widget_unittest.cc which ensures that transparency state of the native window
matches what returns ShouldWindowContentsBeTransparent().
> 
> BUG=589509
> 
> R=erg@chromium.org, mgiuca@chromium.org, sadrul@chromium.org
> 
> TEST=views_unittests --gtest_filter=*WidgetTest.Transparency*
> 
>
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/29d8c51452a83dcbc359679e8ce36af41af64e36
> Cr-Commit-Position: refs/heads/master@{#401826}

TBR=sadrul@chromium.org,jbauman@chromium.org,piman@chromium.org,sky@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589509
==========

[commit-bot: I haz the power, 2016-06-24 08:52:27]

Description was changed from

==========
Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent()
relies on argb visual flag (patchset #30 id:580001 of
https://codereview.chromium.org/1998653002/ )

Reason for revert:
The unittests added in widget_unittest.cc fail on Win10. See the report with
failure output here:

https://chromium-swarm.appspot.com/user/task/2f9a5518a647e510 (scroll down to
see the output of running the tests)

Original issue's description:
> Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on
argb visual flag
> 
> If depth is 32 and if there is a compositing manager then transparency is
supported.
> 
> Do not force transparency if Widget::InitParams::TRANSLUCENT_WINDOW is set
whereas transparency is not supported.
> Which is the same behavior as on OS_WIN.
> 
> Also add a new test WidgetTest.Transparency_DesktopWidget to
widget_unittest.cc which ensures that transparency state of the native window
matches what returns ShouldWindowContentsBeTransparent().
> 
> BUG=589509
> 
> R=erg@chromium.org, mgiuca@chromium.org, sadrul@chromium.org
> 
> TEST=views_unittests --gtest_filter=*WidgetTest.Transparency*
> 
>
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
> 
> Committed: https://crrev.com/29d8c51452a83dcbc359679e8ce36af41af64e36
> Cr-Commit-Position: refs/heads/master@{#401826}

TBR=sadrul@chromium.org,jbauman@chromium.org,piman@chromium.org,sky@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589509
==========

to

==========
Revert of Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent()
relies on argb visual flag (patchset #30 id:580001 of
https://codereview.chromium.org/1998653002/ )

Reason for revert:
The unittests added in widget_unittest.cc fail on Win10. See the report with
failure output here:

https://chromium-swarm.appspot.com/user/task/2f9a5518a647e510 (scroll down to
see the output of running the tests)

Original issue's description:
> Make DesktopWindowTreeHostX11::ShouldWindowContentsBeTransparent() relies on
argb visual flag
>
> If depth is 32 and if there is a compositing manager then transparency is
supported.
>
> Do not force transparency if Widget::InitParams::TRANSLUCENT_WINDOW is set
whereas transparency is not supported.
> Which is the same behavior as on OS_WIN.
>
> Also add a new test WidgetTest.Transparency_DesktopWidget to
widget_unittest.cc which ensures that transparency state of the native window
matches what returns ShouldWindowContentsBeTransparent().
>
> BUG=589509
>
> R=erg@chromium.org, mgiuca@chromium.org, sadrul@chromium.org
>
> TEST=views_unittests --gtest_filter=*WidgetTest.Transparency*
>
>
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
>
> Committed: https://crrev.com/29d8c51452a83dcbc359679e8ce36af41af64e36
> Cr-Commit-Position: refs/heads/master@{#401826}

TBR=sadrul@chromium.org,jbauman@chromium.org,piman@chromium.org,sky@chromium....
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=589509

Committed: https://crrev.com/b6287c064906f9a2139c4be8499befe5cc572e5d
Cr-Commit-Position: refs/heads/master@{#401835}
==========