views: Enable ViewTest.RemoveNotification on Linux.

views: Enable ViewTest.RemoveNotification on Linux.

BUG=139083
R=ben@chromium.org
NOTRY=True

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

[tfarina, 2012-07-28 21:38:53]

It passed locally (Linux with chromeos=1) and on linux_chromeos. Do we need to
test it somewhere else?

[Ben Goodger (Google), 2012-07-28 21:43:36]

lgtm


On Sat, Jul 28, 2012 at 2:38 PM, <tfarina@chromium.org> wrote:

> Reviewers: Ben Goodger (Google),
>
> Message:
> It passed locally (Linux with chromeos=1) and on linux_chromeos. Do we
> need to
> test it somewhere else?
>
> Description:
> views: Enable ViewTest.RemoveNotification on Linux.
>
> BUG=139083
> R=ben@chromium.org
>
>
> Please review this at
http://codereview.chromium.**org/10821083/<http://codereview.chromium.org/108...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M ui/views/view_unittest.cc
>
>
> Index: ui/views/view_unittest.cc
> diff --git a/ui/views/view_unittest.cc b/ui/views/view_unittest.cc
> index ad3949d8cedce835017f78aabb5027**216f5cce21..**
> 65bb80d5d665b8cce25d3f830ec645**edb6adf489 100644
> --- a/ui/views/view_unittest.cc
> +++ b/ui/views/view_unittest.cc
> @@ -818,13 +818,7 @@ TEST_F(ViewTest, DISABLED_Painting) {
>  }
>  */
>
> -#if defined(OS_WIN)
>  TEST_F(ViewTest, RemoveNotification) {
> -#else
> -// TODO(beng): stopped working with widget hierarchy split,
> -//             http://crbug.com/82364
> -TEST_F(ViewTest, DISABLED_RemoveNotification) {
> -#endif
>    ViewStorage* vs = ViewStorage::GetInstance();
>    Widget* widget = new Widget;
>    widget->Init(Widget::**InitParams(Widget::InitParams:**:TYPE_POPUP));
>
>
>

[commit-bot: I haz the power, 2012-07-30 14:31:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/10821083/7001

[commit-bot: I haz the power, 2012-07-30 14:31:29]

Change committed as 148950