| commit | 8473a47ab58361330e27fa53a6bb768ad9031dae | |
| author | luken <luken@chromium.org> | |
| Wed, 15 Oct 2014 17:43:39 +0000 (15 10:43 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 15 Oct 2014 17:44:04 +0000 (15 17:44 +0000) | ||
| tree | 4bc6bf0027d4d88d381d45c0ef3b3c35e97cb49e | treesnapshot (tar.gz zip) |
| parent | a469cf58da7180af4934b5bb4eb9baed95455500 | commitdiff |
fixing CrOS link disambiguation disable
CrOS has TOOLKIT_VIEWS enabled and uses Aura so it has picked up a few of the
parts of the link disambiguation popup now in effect on Windows. This has
caused a regression in their UI where Blink is expecting link disambiguation to
show a bubble but no bubble is being shown. This code updates the RenderViewImpl
to have the correct value in the RendererPrefs so it can report to blink that
the link disambiguation was not handled and Blink should act like it did before
I enabled the bubble for Windows.
BUG=421163
Review URL: https://codereview.chromium.org/647113002
Cr-Commit-Position: refs/heads/master@{#299708}
CrOS has TOOLKIT_VIEWS enabled and uses Aura so it has picked up a few of the
parts of the link disambiguation popup now in effect on Windows. This has
caused a regression in their UI where Blink is expecting link disambiguation to
show a bubble but no bubble is being shown. This code updates the RenderViewImpl
to have the correct value in the RendererPrefs so it can report to blink that
the link disambiguation was not handled and Blink should act like it did before
I enabled the bubble for Windows.
BUG=421163
Review URL: https://codereview.chromium.org/647113002
Cr-Commit-Position: refs/heads/master@{#299708}