DescriptionRevert of Move navigation-induced hiding of form-validation-bubble to the browser process. (patchset #2 id:20001 of https://codereview.chromium.org/2949593003/ )
Reason for revert:
Causing failures on WebKit Linux Trusty Leak.
({"numberOfLiveDocuments":[1,2],"numberOfLiveNodes":[4,62],"numberOfLiveResources":[0,2],"numberOfLiveSuspendableObjects":[2,3]})
Original issue's description:
> Move navigation-induced hiding of form-validation-bubble to the browser process.
>
> Moving hiding of the form validation bubble to the browser process has
> the following benefits:
> - It allows simplifying/removing code from Blink (in particular it
> allows removing some of the code added in r459701).
> - It makes the hiding immune against compromised renderers.
> - It makes the hiding work even if ViewHostMsg_HideValidationMessage IPC
> gets filtered out (i.e. see CanSendWhileSwappedOut method in
> content::SwappedOutMessages). This in turn will allow relanding
> of r479538.
>
> BUG=733940
> TEST=Manually verified that bugs 733940, 704560, 673163 are still fixed.
>
> Review-Url: https://codereview.chromium.org/2949593003
> Cr-Commit-Position: refs/heads/master@{#481681}
> Committed: https://chromium.googlesource.com/chromium/src/+/15399f887a0e8498253dc4496446eee83158e96a
TBR=tkent@chromium.org,alexmos@chromium.org,estade@chromium.org,avi@chromium.org,lukasza@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=733940
Review-Url: https://codereview.chromium.org/2956593003
Cr-Commit-Position: refs/heads/master@{#481827}
Committed: https://chromium.googlesource.com/chromium/src/+/77083a5a889ea093f3343a68c543359c6a16f206
Patch Set 1 #Messages
Total messages: 6 (3 generated)
|