New issue
Advanced search Search tips

Issue 736009 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 739091



Sign in to add a comment

Form validation bubble doesn't work in OOPIFs

Project Member Reported by lukasza@chromium.org, Jun 22 2017

Issue description

Repro steps:

1. Launch chrome with --site-per-process flag
2. Navigate to http://anforowicz.github.io/form-validation-bubble
3. Enter invalid characters into the left and into the right frame
   (the test page considers any non-"z" character to be invalid)

Expected behavior: form validation bubble is shown for both frames

Actual behavior: form validation bubble is not shown for the cross-site frame


 
Cc: alex...@chromium.org
alexmos@ points out in https://crrev.com/2949593003/#msg21 that ViewHostMsg_ShowValidationMessage will be dropped when the RenderView is swapped out.
Cc: tkent@chromium.org
Components: Blink>Forms>Validation
tkent@ points out in  https://crbug.com/734729#c5  that it might be possible today to draw the form-validation-bubble in the renderer.  This would automatically support OOPIFs and would also get rid of other issues (like  issue 734729  and  issue 733940 ).
Description: Show this description

Comment 4 by tkent@chromium.org, Jun 29 2017

Status: Available (was: Untriaged)

Comment 5 by tkent@chromium.org, Jul 6 2017

Blockedon: 739091
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2752293b8965ddfdcf7a3780573e795188a62f1f

commit 2752293b8965ddfdcf7a3780573e795188a62f1f
Author: Kent Tamura <tkent@chromium.org>
Date: Thu Aug 17 01:04:51 2017

ValidationBubbleInRenderer: Enable it by default.

Bug:  432243 ,  597044 ,  734729 ,  736009 ,  736792 ,  739091 
Change-Id: I15a9a24cd86be34dca315f898e556b044bf0acfd
Reviewed-on: https://chromium-review.googlesource.com/607770
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Keishi Hattori <keishi@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495027}
[modify] https://crrev.com/2752293b8965ddfdcf7a3780573e795188a62f1f/content/browser/web_contents/web_contents_impl_browsertest.cc
[modify] https://crrev.com/2752293b8965ddfdcf7a3780573e795188a62f1f/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.json5

Comment 7 by tkent@chromium.org, Aug 17 2017

Labels: M-62
Owner: tkent@chromium.org
Status: Fixed (was: Available)

Sign in to add a comment