Commit a5ade726 authored by Vaclav Brozek's avatar Vaclav Brozek Committed by Commit Bot

Use IsWebElementVisible consistently

On some places in the autofill code, IsWebElementVisible is used,
on other that function is "inlined".

This CL changes one such inlined call to call IsWebElementVisible
explicitly.

There should be no visible behaviour changes caused by this CL.

Bug: 833838
Change-Id: Ia0efb92d1a2b781467af929848c8999d8bb72d34
Reviewed-on: https://chromium-review.googlesource.com/1017202Reviewed-by: default avatarRoger McFarlane <rogerm@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#553159}
parent 0d5bf221
...@@ -1501,8 +1501,7 @@ void WebFormControlElementToFormField( ...@@ -1501,8 +1501,7 @@ void WebFormControlElementToFormField(
IsTextAreaElement(element) || IsTextAreaElement(element) ||
IsSelectElement(element)) { IsSelectElement(element)) {
field->is_autofilled = element.IsAutofilled(); field->is_autofilled = element.IsAutofilled();
if (!g_prevent_layout) field->is_focusable = IsWebElementVisible(element);
field->is_focusable = element.IsFocusable();
field->should_autocomplete = element.AutoComplete(); field->should_autocomplete = element.AutoComplete();
// Use 'text-align: left|right' if set or 'direction' otherwise. // Use 'text-align: left|right' if set or 'direction' otherwise.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment