[Webkit-unassigned] [Bug 31718] Framework to show form validation message for invalid controls

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=31718





--- Comment #41 from Kent Tamura <tkent at chromium.org>  2010-10-27 03:34:20 PST ---
(In reply to comment #40)
> (In reply to comment #39)
> > Created an attachment (id=71726)
 --> (https://bugs.webkit.org/attachment.cgi?id=71726&action=review) [details] [details]
> > Patch 12 (rebased and simplified)
> 
> Story for stepping into the middle of the story here. I thought you mentioned using shadow DOM in one of the previous comments, but this patch still uses the same page->chrome->foo machinery as the others.
> 
> Also, why does Chrome.cpp code is doing so much work? Shouldn't it be basically just a shim to shuttle information between browser and renderer?

The original intention was
 1) Showing a validation message by a native way.  It means Chromium needs IPC messagings.
 2) Showing just one validation message in one page even if the page consists of multiple frames

As you know, 1 is invalid for now because I have a shadow DOM implementation.

As for 2, I have changed my mind :-)   Showing multiple validation messages for multiple forms would be acceptable.  It happens only if a page has multiple forms and a user tries to submit them in short time.

We can simplify the patch.  Simpler is better.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.