Chromium Code Reviews
DescriptionRevert of Fix a notification update issue when navigating to a different URL in the same origin (patchset #2 id:20001 of https://codereview.chromium.org/2720073002/ )
Reason for revert:
The CL is an incomplete fix, as there's another underlying issue in MediaSessionServiceImpl wrt RenderFrame loading.
Original issue's description:
> Fix a notification update issue when navigating to a different URL in the same origin
>
> We should reset the notification info for all non-in-page navigation.
>
> BUG=696716
>
> Review-Url: https://codereview.chromium.org/2720073002
> Cr-Commit-Position: refs/heads/master@{#453573}
> Committed: https://chromium.googlesource.com/chromium/src/+/7ed877a5634408b1b4c5791f57ad7ffbfc03d13f
TBR=avayvod@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=696716
Review-Url: https://codereview.chromium.org/2718373003
Cr-Commit-Position: refs/heads/master@{#453607}
Committed: https://chromium.googlesource.com/chromium/src/+/bc96964f1e70347ca4e9b4b470c101a49033c3fc
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|
|||||||||||||||||||||||||||||||||||||