New issue
Advanced search Search tips

Issue 736792 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug

Blocked on:
issue 739091

Blocking:
issue 671916



Sign in to add a comment

ValidationMessageBubbleView positions wrong on MacViewsBrowser

Reported by alshaba...@yandex-team.ru, Jun 26 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 YaBrowser/17.7.1.99 Yowser/2.5 Safari/537.36

Steps to reproduce the problem:
1. Open a site with <input required>
2. Submit a form without filling the <input>
3. Form validation bubble will appear

What is the expected behavior?
Bubble points to the <input> element

What went wrong?
Bubble points to a wrong place

Did this work before? N/A 

Chrome version: 59.0.3071.86  Channel: n/a
OS Version: OS X 10.12.5
Flash Version: Shockwave Flash 26.0 r0
 
Monosnap 2017-06-26 19-59-02.png
372 KB View Download

Comment 1 by rsesek@chromium.org, Jun 26 2017

 Issue 736794  has been merged into this issue.

Comment 2 by rsesek@chromium.org, Jun 26 2017

Components: Blink>Forms>Validation
Labels: Proj-MacViews
Labels: Needs-Triage-M59

Comment 4 by tapted@chromium.org, Jun 26 2017

Blocking: 671916
Labels: phase4
Status: Available (was: Unconfirmed)
Summary: ValidationMessageBubbleView positions wrong on MacViewsBrowser (was: ValidationMessageBubbleView positions wrong on MacViews)

Comment 5 Deleted

Comment 6 Deleted

Comment 7 by tkent@chromium.org, Jul 13 2017

Blockedon: 739091
Project Member

Comment 8 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 9 by tkent@chromium.org, Aug 17 2017

Owner: tkent@chromium.org
Status: WontFix (was: Available)
We have switched to a new implementation, and will remove ValidationMessageBubbleView.

Sign in to add a comment