Changeset 242130 in webkit


Ignore:
Timestamp:
Feb 27, 2019, 9:33:44 AM (6 years ago)
Author:
Simon Fraser
Message:

scrolling/ios/hit-testing-iframe* tests need to hide the tap highlight
https://bugs.webkit.org/show_bug.cgi?id=195099

Reviewed by Frederic Wang.

These tests were failing because the ref test captured the tap highlight, so hide
it with -webkit-tap-highlight-color: transparent;

  • fast/scrolling/ios/hit-testing-iframe-001.html:
  • fast/scrolling/ios/hit-testing-iframe-002.html:
  • fast/scrolling/ios/hit-testing-iframe-003.html:
  • fast/scrolling/ios/hit-testing-iframe-004.html:
  • fast/scrolling/ios/hit-testing-iframe-005.html:
  • fast/scrolling/ios/hit-testing-iframe-006.html:
  • fast/scrolling/ios/mixing-user-and-programmatic-scroll-006.html:
  • platform/ios-wk2/TestExpectations: hit-testing-iframe-006.html passes now.
Location:
trunk/LayoutTests
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r242129 r242130  
     12019-02-27  Simon Fraser  <simon.fraser@apple.com>
     2
     3        scrolling/ios/hit-testing-iframe* tests need to hide the tap highlight
     4        https://bugs.webkit.org/show_bug.cgi?id=195099
     5
     6        Reviewed by Frederic Wang.
     7
     8        These tests were failing because the ref test captured the tap highlight, so hide
     9        it with -webkit-tap-highlight-color: transparent;
     10
     11        * fast/scrolling/ios/hit-testing-iframe-001.html:
     12        * fast/scrolling/ios/hit-testing-iframe-002.html:
     13        * fast/scrolling/ios/hit-testing-iframe-003.html:
     14        * fast/scrolling/ios/hit-testing-iframe-004.html:
     15        * fast/scrolling/ios/hit-testing-iframe-005.html:
     16        * fast/scrolling/ios/hit-testing-iframe-006.html:
     17        * fast/scrolling/ios/mixing-user-and-programmatic-scroll-006.html:
     18        * platform/ios-wk2/TestExpectations: hit-testing-iframe-006.html passes now.
     19
    1202019-02-27  Ulrich Pflueger  <up@nanocosmos.de>
    221
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-001.html

    r242052 r242130  
    3737    <div style="position: absolute; top: 3em; width: 100px; height: 100px; background: green;">
    3838      <iframe id="clickInContentBox" style="left: 0px; top: 0px;" scrolling="yes" onclick="this.style.background='red'" srcdoc="
    39           <body style='margin: 0; width: 200px; height: 200px'>
    40               <div style='position: absolute; width: 100px; height: 100px; background: red;'
     39          <body style='margin: 0; width: 200px; height: 200px;'>
     40              <div style='position: absolute; width: 100px; height: 100px; background: red; -webkit-tap-highlight-color: transparent;'
    4141                   onclick='this.style.background=&quot;green&quot;'></div>
    4242          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-002.html

    r242052 r242130  
    3838      <iframe id="clickInPaddingBoxOutOfContentBox" style="left: 0px; top: 0px; background: red;" onclick="this.style.background='green'" scrolling="yes" srcdoc="
    3939          <body style='margin: 0; width: 200px; height: 200px'>
    40               <div style='position: absolute; width: 100px; height: 100px; background: green;'
     40              <div style='position: absolute; width: 100px; height: 100px; background: green; -webkit-tap-highlight-color: transparent;'
    4141                   onclick='this.style.background=&quot;red&quot;'></div>
    4242          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-003.html

    r242052 r242130  
    3838      <iframe id="clickElementAboveFrame" style="left: 0px; top: 0px;" scrolling="yes" onclick="this.style.background='red'" srcdoc="
    3939          <body style='margin: 0; width: 200px; height: 200px; background: green;'>
    40               <div style='position: absolute; width: 100px; height: 100px; background: green;'
     40              <div style='position: absolute; width: 100px; height: 100px; background: green; -webkit-tap-highlight-color: transparent;'
    4141                   onclick='this.style.background=&quot;red&quot;'></div>
    4242          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-004.html

    r242052 r242130  
    3838      <iframe id="clickElementInsideFrame" style="left: 0px; top: 0px;" scrolling="yes" onclick="this.style.background='red'" srcdoc="
    3939          <body style='margin: 0; width: 200px; height: 200px; background: green;'>
    40               <div style='position: absolute; left: 10px; top: 10px; width: 50px; height: 50px; background: red;'
     40              <div style='position: absolute; left: 10px; top: 10px; width: 50px; height: 50px; background: red; -webkit-tap-highlight-color: transparent;'
    4141                   onclick='this.style.background=&quot;green&quot;'></div>
    4242          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-005.html

    r242052 r242130  
    4141          <body style='margin: 0; width: 200px; height: 200px; background: green;'>
    4242             <div style='position: absolute; width: 75px; height: 75px; background: red;'></div>
    43               <div style='position: absolute; left: 110px; top: 110px; width: 50px; height: 50px; background: red;'
     43              <div style='position: absolute; left: 110px; top: 110px; width: 50px; height: 50px; background: red; -webkit-tap-highlight-color: transparent;'
    4444                   onclick='this.style.background=&quot;green&quot;'></div>
    4545          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/hit-testing-iframe-006.html

    r242052 r242130  
    4949          <body style='margin: 0; width: 200px; height: 200px; background: green;'>
    5050             <div style='position: absolute; width: 75px; height: 75px; background: red;'></div>
    51               <div style='position: absolute; left: 0px; top: 150px; width: 50px; height: 50px; background: red;'
     51              <div style='position: absolute; left: 0px; top: 150px; width: 50px; height: 50px; background: red; -webkit-tap-highlight-color: transparent;'
    5252                   onclick='this.style.background=&quot;green&quot;'></div>
    5353          </body>" onload="runTest()">
  • trunk/LayoutTests/fast/scrolling/ios/mixing-user-and-programmatic-scroll-006.html

    r242046 r242130  
    5858        <iframe id="programmaticYAndUserXAndHitTesting" style="left: 0px; top: 0px;" scrolling="yes" srcdoc="
    5959            <body style='margin: 0; width: 200px; height: 200px'>
    60                 <div style='left: 100px; top: 100px; position: absolute; width: 100px; height: 100px; background: red;' onclick='this.style.background=&quot;green&quot;'></div>
     60                <div style='left: 100px; top: 100px; position: absolute; width: 100px; height: 100px; background: red; -webkit-tap-highlight-color: transparent;' onclick='this.style.background=&quot;green&quot;'></div>
    6161            </body>" onload="runTest()">
    6262        </iframe>
  • trunk/LayoutTests/platform/ios-wk2/TestExpectations

    r242052 r242130  
    10761076
    10771077fast/scrolling/ios/scroll-events-back-forward-after-pageshow.html [ Pass Failure ]
    1078 webkit.org/b/193997 fast/scrolling/ios/hit-testing-iframe-006.html [ ImageOnlyFailure ]
    10791078
    10801079webkit.org/b/157589 fast/text-autosizing/ios/text-autosizing-after-back.html [ Pass Timeout ]
Note: See TracChangeset for help on using the changeset viewer.