Improve WebValidationMessageClient interface and implementation.

Improve WebValidationMessageClient interface and implementation.

* Change the anchor position argument of showValidationMessage from the
  screen coordinate system to the root view coordinate system.  Blink
  can't provide a rectangle in the screen coordinate on Android port
  because HostWindow::rootViewToScreen does nothing on Android, and
  we'd like to make it consistent.  This change breaks validation message
  bubble position temporarily until UI implementations for non-Android is
  updated.

* Add moveValidationMessage to avoid recreation of UI widgets.

* Take account of the length of title attribute value for display duration.

BUG=146212
R=keishi@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=152176

[tkent, 2013-06-07 05:13:32]

Would you review this please?
This shouldn't landed until https://codereview.chromium.org/16583005/ is ready.

[tkent, 2013-06-10 22:27:11]

This is ready.  Keishi, would you review this please?

[keishi, 2013-06-11 02:25:33]

lgtm

[commit-bot: I haz the power, 2013-06-11 02:29:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tkent@chromium.org/16180009/1

[commit-bot: I haz the power, 2013-06-11 02:29:42]

Failed to apply patch for public/webview/WebValidationMessageClient.h:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  A         public/webview
  Created missing directory public/webview.
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: public/webview/WebValidationMessageClient.h
  |diff --git a/public/webview/WebValidationMessageClient.h
b/public/webview/WebValidationMessageClient.h
  |index
6ba1555272942b015102b4c3df0b3d2ba2e8867c..26a07b15ebe10e0eaed33ae46e314910b421dc19
100644
  |--- a/public/webview/WebValidationMessageClient.h
  |+++ b/public/webview/WebValidationMessageClient.h
  --------------------------
  No file to patch.  Skipping patch.
  1 out of 1 hunk ignored

Patch:       public/webview/WebValidationMessageClient.h
Index: public/webview/WebValidationMessageClient.h
diff --git a/public/webview/WebValidationMessageClient.h
b/public/webview/WebValidationMessageClient.h
index
6ba1555272942b015102b4c3df0b3d2ba2e8867c..26a07b15ebe10e0eaed33ae46e314910b421dc19
100644
--- a/public/webview/WebValidationMessageClient.h
+++ b/public/webview/WebValidationMessageClient.h
@@ -44,11 +44,13 @@ public:
     // Show a notification popup for the specified form vaidation messages
     // besides the anchor rectangle. An implementation of this function should
     // not hide the popup until hideValidationMessage call.
-    virtual void showValidationMessage(const WebRect& anchorInScreen, const
WebString& mainText, const WebString& supplementalText, WebTextDirection hint) {
}
+    virtual void showValidationMessage(const WebRect& anchorInRootView, const
WebString& mainText, const WebString& supplementalText, WebTextDirection hint) {
}
 
     // Hide notifation popup for form validation messages.
     virtual void hideValidationMessage() { }
 
+    // Move the existing notifation popup for the new anchor position.
+    virtual void moveValidationMessage(const WebRect& anchorInRootView) { }
 
 protected:
     virtual ~WebValidationMessageClient() { }

[commit-bot: I haz the power, 2013-06-11 02:33:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tkent@chromium.org/16180009/9001

[tkent, 2013-06-11 03:19:07]

Committed patchset #2 manually as r152176 (presubmit successful).