Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(14)

Unified Diff: LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html

Issue 129643002: Fix link highlights on links that contain images. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html
diff --git a/LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html b/LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html
index 31c3f612e701c36b51785c50bc8c73a647d6f65f..a98dd8317ef832f08ce2168cc59ef5a7af09ee5c 100644
--- a/LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html
+++ b/LayoutTests/compositing/gestures/gesture-tapHighlight-img-expected.html
@@ -1,29 +1,20 @@
<!DOCTYPE html>
<html>
<head>
-<script src="../../resources/js-test.js"></script>
+<script src="resources/link-highlight-helper.js"></script>
+<link rel="stylesheet" type="text/css" href="resources/link-highlight-style.css">
</head>
<body onload="runTest();">
-<div style="-webkit-transform: translateZ(0); padding: 20px;">
- <div id="targetLink" style="width: 320px; height: 240px; cursor: pointer; -webkit-tap-highlight-color: rgb(0, 255, 0)"></div>
+<div style="padding: 20px">
+ <div id="targetLink" style="-webkit-transform: translateZ(0); margin:0px; padding: 0px; width: 320px;">
+ <a href="#"><img id="targetImg" width="320" height="240" src="resources/dice.png"></a>
</div>
+</div>
<script>
function runTest() {
- var clientRect = document.getElementById('targetLink').getBoundingClientRect();
- x = (clientRect.left + clientRect.right) / 2;
- y = (clientRect.top + clientRect.bottom) / 2;
- if (window.testRunner) {
+ createCompositedHighlight(document.getElementById('targetImg'));
+ if (window.testRunner)
testRunner.dumpAsTextWithPixelResults();
- testRunner.waitUntilDone();
- }
-
- if (window.eventSender) {
- eventSender.gestureShowPress(x, y);
- window.setTimeout(function() { window.testRunner.notifyDone(); }, 0);
- } else {
- debug("This test requires DumpRenderTree.");
- debug("This test is successful if the image below is covered in a green rectangle with rounded corners.");
- }
}
</script>
</script>

Powered by Google App Engine
This is Rietveld 408576698