While I support backfilling tests I don't understand why this is pri 1.

Plus adding tests for that bugfix is non-trivial, as there's no existing suite for ValidationMessageBubble or for its subclasses. So I probably won't get to this for a while.

Thanks for the update. Not a P1 bug, reduced the priority.

Please add the test suite when you get a chance.TE's main goal here is to maximize the automation test coverage.

***Bulk Edit***

Considering Missing tests are P2, but please prioritize and add them asap.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/819580022f1d2c8dc1eaa4d8614e765a55f95a4b

commit 819580022f1d2c8dc1eaa4d8614e765a55f95a4b
Author: bsep <bsep@chromium.org>
Date: Wed May 17 20:26:19 2017

Backfill some UI tests.

* Added a test for DialogClientView's button layout.
* Added a test for non client hit results in BubbleFrameView.
* Added a test for ValidationMessageBubbleView's anchor rect at HighDPI.

BUG=702196, 680019 

Review-Url: https://codereview.chromium.org/2861533003
Cr-Commit-Position: refs/heads/master@{#472550}

[add] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/chrome/browser/ui/views/validation_message_bubble_view_unittest.cc
[modify] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/chrome/test/BUILD.gn
[modify] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/content/test/test_render_view_host.cc
[modify] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/content/test/test_render_view_host.h
[modify] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/ui/views/window/dialog_client_view_unittest.cc
[modify] https://crrev.com/819580022f1d2c8dc1eaa4d8614e765a55f95a4b/ui/views/window/dialog_delegate_unittest.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d77878b9c9f4f4db96a21e0974843a7e8cd77b24

commit d77878b9c9f4f4db96a21e0974843a7e8cd77b24
Author: bsep <bsep@chromium.org>
Date: Wed May 17 20:40:13 2017

Revert of Backfill some UI tests. (patchset #9 id:160001 of https://codereview.chromium.org/2861533003/ )

Reason for revert:
Caused compile failure

Original issue's description:
> Backfill some UI tests.
>
> * Added a test for DialogClientView's button layout.
> * Added a test for non client hit results in BubbleFrameView.
> * Added a test for ValidationMessageBubbleView's anchor rect at HighDPI.
>
> BUG=702196, 680019 
>
> Review-Url: https://codereview.chromium.org/2861533003
> Cr-Commit-Position: refs/heads/master@{#472550}
> Committed: https://chromium.googlesource.com/chromium/src/+/819580022f1d2c8dc1eaa4d8614e765a55f95a4b

TBR=sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=702196, 680019 

Review-Url: https://codereview.chromium.org/2891803002
Cr-Commit-Position: refs/heads/master@{#472554}

[delete] https://crrev.com/525974cdba654d1611d169a34397304334797175/chrome/browser/ui/views/validation_message_bubble_view_unittest.cc
[modify] https://crrev.com/d77878b9c9f4f4db96a21e0974843a7e8cd77b24/chrome/test/BUILD.gn
[modify] https://crrev.com/d77878b9c9f4f4db96a21e0974843a7e8cd77b24/content/test/test_render_view_host.cc
[modify] https://crrev.com/d77878b9c9f4f4db96a21e0974843a7e8cd77b24/content/test/test_render_view_host.h
[modify] https://crrev.com/d77878b9c9f4f4db96a21e0974843a7e8cd77b24/ui/views/window/dialog_client_view_unittest.cc
[modify] https://crrev.com/d77878b9c9f4f4db96a21e0974843a7e8cd77b24/ui/views/window/dialog_delegate_unittest.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9

commit ea63253fa4fdcf6db310bff500e72a8aac4ae8f9
Author: bsep <bsep@chromium.org>
Date: Thu May 18 20:48:25 2017

Backfill some UI tests.

* Added a test for DialogClientView's button layout.
* Added a test for non client hit results in BubbleFrameView.
* Added a test for ValidationMessageBubbleView's anchor rect at HighDPI.

BUG=702196, 680019 

Review-Url: https://codereview.chromium.org/2861533003
Cr-Original-Commit-Position: refs/heads/master@{#472550}
Committed: https://chromium.googlesource.com/chromium/src/+/819580022f1d2c8dc1eaa4d8614e765a55f95a4b
Review-Url: https://codereview.chromium.org/2861533003
Cr-Commit-Position: refs/heads/master@{#472923}

[add] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/chrome/browser/ui/views/validation_message_bubble_view_unittest.cc
[modify] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/chrome/test/BUILD.gn
[modify] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/content/test/test_render_view_host.cc
[modify] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/content/test/test_render_view_host.h
[modify] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/ui/views/window/dialog_client_view_unittest.cc
[modify] https://crrev.com/ea63253fa4fdcf6db310bff500e72a8aac4ae8f9/ui/views/window/dialog_delegate_unittest.cc