Replace WebElement::hasTagName with hasHTMLTagName in autofill

Replace WebElement::hasTagName with hasHTMLTagName in autofill

BUG=374725, 358235
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272000

[keishi, 2014-05-20 14:57:21]

CL replacing https://codereview.chromium.org/295603002/

Replaces all usage of hasTagName in autofill module with hasHTMLTagName

[Ilya Sherman, 2014-05-20 14:59:11]

LGTM, thanks!

https://codereview.chromium.org/293993004/diff/20001/components/autofill/cont...
File components/autofill/content/renderer/page_click_tracker.cc (right):

https://codereview.chromium.org/293993004/diff/20001/components/autofill/cont...
components/autofill/content/renderer/page_click_tracker.cc:36: const
WebInputElement GetWebInputElement(const WebNode& node) {
nit: This function name should remain "GetTextWebInputElement", as the function
specifically checks IsTextInput().  All <textarea>s are text fields, but not all
<input>s are.

[keishi, 2014-05-21 01:55:17]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2014-05-21 01:56:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/keishi@chromium.org/293993004/40001

[commit-bot: I haz the power, 2014-05-21 07:58:30]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  android_aosp on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_aosp/builds/7...)
  linux_chromium_chromeos_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/7393)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/10624)

[commit-bot: I haz the power, 2014-05-21 08:24:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-21 08:24:31]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)

[keishi, 2014-05-21 09:39:04]

The CQ bit was checked by keishi@chromium.org

[commit-bot: I haz the power, 2014-05-21 09:40:20]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/keishi@chromium.org/293993004/60001

[commit-bot: I haz the power, 2014-05-21 22:50:37]

Change committed as 272000