<?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>70561</bug_id>
          
          <creation_ts>2011-10-20 15:58:36 -0700</creation_ts>
          <short_desc>Consider making WebScreenInfoFactory an abstract factory, and moving concrete implementation outside of WebKit</short_desc>
          <delta_ts>2011-10-20 15:58:36 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit API</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>
          
          <blocked>70559</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Fady Samuel">fsamuel</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>rjkroege</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>487960</commentid>
    <comment_count>0</comment_count>
    <who name="Fady Samuel">fsamuel</who>
    <bug_when>2011-10-20 15:58:36 -0700</bug_when>
    <thetext>This is not entirely a Chromium only change, but it strikes me as odd that WebScreenInfoFactory has an implementation that uses X11 but that code is only called from the browser. It seems that conceptually, that implementation does not belong in WebKit at all. This is a bit of a yak shave.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>