Revert of x11: Don't wait on a MapNotify on override-redirect windows.

Revert of x11: Don't wait on a MapNotify on override-redirect windows. (patchset #3 id:40001 of https://codereview.chromium.org/1896363004/ )

Reason for revert:
Patch suspected to cause blank sublist on menus.

BUG=606661
Original issue's description:
> x11: Don't wait on a MapNotify on override-redirect windows.
>
> Some window managers do not send MapNotify on override-redirect
> windows. Enlightenment doesn't send one, XFCE does, and I don't know
> which behaviour is more common.
>
> This makes mapping resilient against whether we would have received a
> MapNotify message on override-redirect window. On override-redirect
> windows, we immediately trigger the code that would run in the future
> MapNotify handler, and ignore it in the future MapNotify message if it
> is already run.
>
> BUG=381732
>
> Committed: https://crrev.com/28bb70402ddc4e68963f536388730c63598f8d65
> Cr-Commit-Position: refs/heads/master@{#389572}

TBR=danakj@chromium.org,thomasanderson@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=381732
Committed: https://crrev.com/64a7c6401449a843118d9fc208798674a5a8c7ea
Cr-Commit-Position: refs/heads/master@{#390140}

[Elliot Glaysher, 2016-04-27 18:14:54]

Created Revert of x11: Don't wait on a MapNotify on override-redirect windows.

[commit-bot: I haz the power, 2016-04-27 18:15:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1931583002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1931583002/1

[commit-bot: I haz the power, 2016-04-27 18:31:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1931583002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1931583002/1

[commit-bot: I haz the power, 2016-04-27 18:59:54]

Description was changed from

==========
Revert of x11: Don't wait on a MapNotify on override-redirect windows. (patchset
#3 id:40001 of https://codereview.chromium.org/1896363004/ )

Reason for revert:
Patch suspected to cause blank sublist on menus.

BUG=606661

Original issue's description:
> x11: Don't wait on a MapNotify on override-redirect windows.
> 
> Some window managers do not send MapNotify on override-redirect
> windows. Enlightenment doesn't send one, XFCE does, and I don't know
> which behaviour is more common.
> 
> This makes mapping resilient against whether we would have received a
> MapNotify message on override-redirect window. On override-redirect
> windows, we immediately trigger the code that would run in the future
> MapNotify handler, and ignore it in the future MapNotify message if it
> is already run.
> 
> BUG=381732
> 
> Committed: https://crrev.com/28bb70402ddc4e68963f536388730c63598f8d65
> Cr-Commit-Position: refs/heads/master@{#389572}

TBR=danakj@chromium.org,thomasanderson@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=381732
==========

to

==========
Revert of x11: Don't wait on a MapNotify on override-redirect windows. (patchset
#3 id:40001 of https://codereview.chromium.org/1896363004/ )

Reason for revert:
Patch suspected to cause blank sublist on menus.

BUG=606661

Original issue's description:
> x11: Don't wait on a MapNotify on override-redirect windows.
> 
> Some window managers do not send MapNotify on override-redirect
> windows. Enlightenment doesn't send one, XFCE does, and I don't know
> which behaviour is more common.
> 
> This makes mapping resilient against whether we would have received a
> MapNotify message on override-redirect window. On override-redirect
> windows, we immediately trigger the code that would run in the future
> MapNotify handler, and ignore it in the future MapNotify message if it
> is already run.
> 
> BUG=381732
> 
> Committed: https://crrev.com/28bb70402ddc4e68963f536388730c63598f8d65
> Cr-Commit-Position: refs/heads/master@{#389572}

TBR=danakj@chromium.org,thomasanderson@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=381732
==========

[commit-bot: I haz the power, 2016-04-27 18:59:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-30 17:11:39]

Patchset 1 (id:??) landed as
https://crrev.com/64a7c6401449a843118d9fc208798674a5a8c7ea
Cr-Commit-Position: refs/heads/master@{#390140}

[commit-bot: I haz the power, 2016-04-30 17:51:06]

Description was changed from

==========
Revert of x11: Don't wait on a MapNotify on override-redirect windows. (patchset
#3 id:40001 of https://codereview.chromium.org/1896363004/ )

Reason for revert:
Patch suspected to cause blank sublist on menus.

BUG=606661

Original issue's description:
> x11: Don't wait on a MapNotify on override-redirect windows.
> 
> Some window managers do not send MapNotify on override-redirect
> windows. Enlightenment doesn't send one, XFCE does, and I don't know
> which behaviour is more common.
> 
> This makes mapping resilient against whether we would have received a
> MapNotify message on override-redirect window. On override-redirect
> windows, we immediately trigger the code that would run in the future
> MapNotify handler, and ignore it in the future MapNotify message if it
> is already run.
> 
> BUG=381732
> 
> Committed: https://crrev.com/28bb70402ddc4e68963f536388730c63598f8d65
> Cr-Commit-Position: refs/heads/master@{#389572}

TBR=danakj@chromium.org,thomasanderson@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=381732
==========

to

==========
Revert of x11: Don't wait on a MapNotify on override-redirect windows. (patchset
#3 id:40001 of https://codereview.chromium.org/1896363004/ )

Reason for revert:
Patch suspected to cause blank sublist on menus.

BUG=606661

Original issue's description:
> x11: Don't wait on a MapNotify on override-redirect windows.
>
> Some window managers do not send MapNotify on override-redirect
> windows. Enlightenment doesn't send one, XFCE does, and I don't know
> which behaviour is more common.
>
> This makes mapping resilient against whether we would have received a
> MapNotify message on override-redirect window. On override-redirect
> windows, we immediately trigger the code that would run in the future
> MapNotify handler, and ignore it in the future MapNotify message if it
> is already run.
>
> BUG=381732
>
> Committed: https://crrev.com/28bb70402ddc4e68963f536388730c63598f8d65
> Cr-Commit-Position: refs/heads/master@{#389572}

TBR=danakj@chromium.org,thomasanderson@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=381732

Committed: https://crrev.com/64a7c6401449a843118d9fc208798674a5a8c7ea
Cr-Commit-Position: refs/heads/master@{#390140}
==========