<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>71945</bug_id>
          
          <creation_ts>2011-11-09 13:00:42 -0800</creation_ts>
          <short_desc>Eliminate the notion of a window coordinate system from WebCore</short_desc>
          <delta_ts>2011-11-29 13:16:03 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Platform</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>71954</dependson>
    
    <dependson>71947</dependson>
    
    <dependson>71955</dependson>
    
    <dependson>72338</dependson>
    
    <dependson>72397</dependson>
    
    <dependson>72408</dependson>
    
    <dependson>73352</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Anders Carlsson">andersca</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>darin</cc>
    
    <cc>fsamuel</cc>
    
    <cc>sam</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>499278</commentid>
    <comment_count>0</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2011-11-09 13:00:42 -0800</bug_when>
    <thetext>WebCore shouldn&apos;t need to know anything about window coordinates - especially not in Chromium and WebKit2 where the WebCore widget hierarchy is in a completely different process from the actual window!

We should try to eliminate calls to convertToContainingWindow/convertFromContainingWindow etc and replace them with convertToRootView/convertFromRootView.

The only case where window coordinates are important is in Mac WebKit1 due to the way it works with AppKit, but we should try to keep that code in the WebKit1 code.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>499285</commentid>
    <comment_count>1</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2011-11-09 13:07:05 -0800</bug_when>
    <thetext>Awesome plan!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>499443</commentid>
    <comment_count>2</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-11-09 16:11:28 -0800</bug_when>
    <thetext>Don’t window coordinates matter for things like window.screen, window.screenLeft, window.screenTop, window.innerHeight, window.outerHeight?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>511046</commentid>
    <comment_count>3</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2011-11-29 13:16:03 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; Don’t window coordinates matter for things like window.screen, window.screenLeft, window.screenTop, window.innerHeight, window.outerHeight?

Nope. for window.outerHeight/window.innerHeight we have a ChromeClient function to get the window rect (in screen coordinates). Nowhere in WebCore itself do we need to use window coordinates.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>