Screen.availLeft and Screen.availTop should be unsigned, since they can be negative for screens to the left of/above the primary screen.
Screen.availLeft and Screen.availTop should be *signed*
Created attachment 70275 [details] Patch
Comment on attachment 70275 [details] Patch +int Screen::availLeft() const { if (!m_frame) return 0; return static_cast<unsigned>(screenAvailableRect(m_frame->view()).x()); Why cast to unsigned?
Created attachment 70278 [details] Patch
Fixed the casts.
Comment on attachment 70278 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=70278&action=review r=me > WebCore/manual-tests/screen-availLeft.html:5 > + <script type="text/javascript" charset="utf-8"> Charset is useless for inline scripts. But it's fine to land as is to make sure it's not causing a crash somewhere. > WebCore/manual-tests/screen-availLeft.html:11 > + function log(s) > + { > + var output = document.getElementById('output'); > + output.innerHTML += s + "<br>"; > + } That's a lot of code for one lag() call.
http://trac.webkit.org/changeset/69599