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

Side by Side Diff: LayoutTests/editing/pasteboard/copy-crash.html

Issue 1133693002: Update most LayoutTests to be agnostic to scrollTopLeftInterop mode (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: tdresser cr feedback Created 5 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 <html> 1 <html>
2 <head> 2 <head>
3 <script src="../editing.js" language="JavaScript" type="text/JavaScript" ></ script> 3 <script src="../editing.js" language="JavaScript" type="text/JavaScript" ></ script>
4 <style type="text/css"> 4 <style type="text/css">
5 div.popup { 5 div.popup {
6 color: black !important; 6 color: black !important;
7 background: yellow !important; 7 background: yellow !important;
8 padding: 0.5em !important; 8 padding: 0.5em !important;
9 position: absolute !important; 9 position: absolute !important;
10 z-index: 20000 !important; 10 z-index: 20000 !important;
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 MPP_bind(this.handleOnLoad_, this), null); 97 MPP_bind(this.handleOnLoad_, this), null);
98 98
99 }; 99 };
100 100
101 PersonPopup.prototype.getPointerX_ = function(e) { 101 PersonPopup.prototype.getPointerX_ = function(e) {
102 var x, scrollLeft; 102 var x, scrollLeft;
103 103
104 if (e.pageX) { 104 if (e.pageX) {
105 x = e.pageX; 105 x = e.pageX;
106 } else if (e.clientX) { 106 } else if (e.clientX) {
107 x = e.clientX + document.body.scrollLeft; 107 x = e.clientX + document.scrollingElement.scrollLeft;
108 } else { 108 } else {
109 x = 0; 109 x = 0;
110 } 110 }
111 return x; 111 return x;
112 }; 112 };
113 113
114 PersonPopup.prototype.getPointerY_ = function(e) { 114 PersonPopup.prototype.getPointerY_ = function(e) {
115 var y, scrollTop; 115 var y, scrollTop;
116 116
117 if (e.pageY) { 117 if (e.pageY) {
118 y = e.pageY; 118 y = e.pageY;
119 } else if (e.clientY) { 119 } else if (e.clientY) {
120 y = e.clientY + document.body.scrollTop; 120 y = e.clientY + document.scrollingElement.scrollTop;
121 } else { 121 } else {
122 y = 0; 122 y = 0;
123 } 123 }
124 return y; 124 return y;
125 }; 125 };
126 126
127 PersonPopup.prototype.pointerCloseEnough_ = function(x, y) { 127 PersonPopup.prototype.pointerCloseEnough_ = function(x, y) {
128 var POINTER_TOLERANCE = 5; 128 var POINTER_TOLERANCE = 5;
129 if (this.pointerOutsidePopup) { 129 if (this.pointerOutsidePopup) {
130 if ((x >= this.targetPosX) && 130 if ((x >= this.targetPosX) &&
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
349 </p> 349 </p>
350 <div id="log_div"> 350 <div id="log_div">
351 This test checks the fix for https://bugs.webkit.org/show_bug.cgi?id=18506 . To test it manually: 351 This test checks the fix for https://bugs.webkit.org/show_bug.cgi?id=18506 . To test it manually:
352 <li/> Hover mouse over "Mouse Over" link 352 <li/> Hover mouse over "Mouse Over" link
353 <li/> Quickly jump to the yellow box that pops up and select "Select" link 353 <li/> Quickly jump to the yellow box that pops up and select "Select" link
354 <li/> Move mouse away so that pop up disappears 354 <li/> Move mouse away so that pop up disappears
355 <li/> Press the "Copy" keyboard accelerator - this should not cause any cr ash 355 <li/> Press the "Copy" keyboard accelerator - this should not cause any cr ash
356 </div> 356 </div>
357 </body> 357 </body>
358 </html> 358 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698