Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use base::RunLoop.

Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use base::RunLoop. (https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed: https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8cc8941ddc4677

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707, 713820

[sgurun-gerrit only, 2017-04-20 18:43:07]

The CQ bit was checked by sgurun@chromium.org

[sgurun-gerrit only, 2017-04-20 18:43:07]

Created Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver
to use base::RunLoop. (patchset #3 id…

[sgurun-gerrit only, 2017-04-20 18:44:10]

Description was changed from

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (patchset #3 id… (patchset #11 id:200001 of
https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707
==========

to

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (patchset #3 id… (patchset #11 id:200001 of
https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707, 713820
==========

[commit-bot: I haz the power, 2017-04-20 18:44:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492713787272210, "parent_rev":
"38f02400540606a7c418f9fa5148d3a07fd8b4fa", "commit_rev":
"48c505a83ed6938f72442904b664fe0b6d7eaacf"}

[commit-bot: I haz the power, 2017-04-20 18:44:38]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492713787272210, "parent_rev":
"b26fed56d4ceba0ba676aafe34083d71ef261afb", "commit_rev":
"3d232ee96d76f7ab59d2b0cd89bb6949e91dd120"}

[commit-bot: I haz the power, 2017-04-20 18:44:50]

Description was changed from

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (patchset #3 id… (patchset #11 id:200001 of
https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707, 713820
==========

to

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (patchset #3 id… (patchset #11 id:200001 of
https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707

Review-Url: https://codereview.chromium.org/2828163004
Cr-Commit-Position: refs/heads/master@{#466082}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d232ee96d76f7ab59d2b0cd89bb...
==========

[commit-bot: I haz the power, 2017-04-20 18:44:51]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3d232ee96d76f7ab59d2b0cd89bb...

[sgurun-gerrit only, 2017-04-20 18:45:06]

Description was changed from

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (patchset #3 id… (patchset #11 id:200001 of
https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707

Review-Url: https://codereview.chromium.org/2828163004
Cr-Commit-Position: refs/heads/master@{#466082}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d232ee96d76f7ab59d2b0cd89bb...
==========

to

==========
Revert of Revert "Revert of Reland: Switch WindowedNotificationObserver to use
base::RunLoop. (https://codereview.chromium.org/2740783003/ )

Reason for revert:
speculative revert for crbug.com/713820

Original issue's description:
> Reland #2: Switch WindowedNotificationObserver to use base::RunLoop.
> 
> Now it will quit the message loop immediately after receiving the
> notification. Also it does not allow nested tasks anymore.
> 
> Previous CL (https://codereview.chromium.org/2720513003) is in patchset #1.
> Added fixes for:
>  - browser_tests.NavigatingExtensionPopupBrowserTest.DownloadViaPost with
PlzNavigate;
>  - interactive_ui_tests.ConstrainedWindowViewTest.NavigationOnBackspace.
> Also rephrased comment in print_preview_dialog_controller_browsertest.cc.
> 
> BUG=668707
> 
> Review-Url: https://codereview.chromium.org/2740783003
> Cr-Commit-Position: refs/heads/master@{#466016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/25911df55b8f939560df1f664f8c...

TBR=jam@chromium.org,vabr@chromium.org,ahest@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=668707, 713820
==========

[sgurun-gerrit only, 2017-04-20 18:45:12]

The CQ bit was checked by sgurun@chromium.org

[commit-bot: I haz the power, 2017-04-20 18:45:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-20 18:47:07]

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

[commit-bot: I haz the power, 2017-04-20 18:47:10]

Failed to apply the patch.
On branch working_branch
Your branch is up-to-date with 'origin/refs/heads/master'.
nothing to commit, working tree clean