<?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>96109</bug_id>
          
          <creation_ts>2012-09-07 08:00:46 -0700</creation_ts>
          <short_desc>[chromium] Global Selection test broken on Chromium Linux</short_desc>
          <delta_ts>2013-04-09 17:07:09 -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>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Fady Samuel">fsamuel</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>allan.jensen</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>714524</commentid>
    <comment_count>0</comment_count>
    <who name="Fady Samuel">fsamuel</who>
    <bug_when>2012-09-07 08:00:46 -0700</bug_when>
    <thetext>editing/pasteboard/paste-global-selection.html is broken and should be fixed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>714704</commentid>
    <comment_count>1</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-09-07 10:34:35 -0700</bug_when>
    <thetext>From https://bugs.webkit.org/show_bug.cgi?id=88238#c49 :
There is no code in WebKit that updates the global selection for Chromium. The chromium code that does that the update is outside WebKit, which is why the automated test will not work in the chromium port.

--

For non-selection clipboard tests, we use a mock clipboard. We probably need to make a mock selection clipboard for this test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>714706</commentid>
    <comment_count>2</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-09-07 10:35:16 -0700</bug_when>
    <thetext>Alternately, we should add a method to internals for setting the global selection clipboard that all ports can use (and doesn&apos;t depend on X).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>714764</commentid>
    <comment_count>3</comment_count>
    <who name="Allan Sandfeld Jensen">allan.jensen</who>
    <bug_when>2012-09-07 11:12:27 -0700</bug_when>
    <thetext>A(In reply to comment #2)
&gt; Alternately, we should add a method to internals for setting the global selection clipboard that all ports can use (and doesn&apos;t depend on X).

Not sure about internals, while it would be nice if all ports could pass the tests the same; I think it makes more sense to me to add it to TestRunner. For Chromium that only means updating DumpRenderTree and adding something saving and recalling the selection (like the Chrome/Chromium application would).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>729375</commentid>
    <comment_count>4</comment_count>
    <who name="Allan Sandfeld Jensen">allan.jensen</who>
    <bug_when>2012-09-27 03:13:26 -0700</bug_when>
    <thetext>An alternative solution is to move the chromium update of the global selection into WebKit. Unless there is a good reason it is outside, I think that would be the best solution, and would mean we could also unify the code updating of the selection buffer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>729793</commentid>
    <comment_count>5</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-09-27 13:10:37 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; An alternative solution is to move the chromium update of the global selection into WebKit. Unless there is a good reason it is outside, I think that would be the best solution, and would mean we could also unify the code updating of the selection buffer.

We could do this for DRT, but in a real Chromium.  WebKit is in a sandbox that doesn&apos;t have direct access to X.  We make an IPC call to the browser process that does sets the global selection clipboard.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>730408</commentid>
    <comment_count>6</comment_count>
    <who name="Allan Sandfeld Jensen">allan.jensen</who>
    <bug_when>2012-09-28 02:40:31 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #4)
&gt; &gt; An alternative solution is to move the chromium update of the global selection into WebKit. Unless there is a good reason it is outside, I think that would be the best solution, and would mean we could also unify the code updating of the selection buffer.
&gt; 
&gt; We could do this for DRT, but in a real Chromium.  WebKit is in a sandbox that doesn&apos;t have direct access to X.  We make an IPC call to the browser process that does sets the global selection clipboard.

My point was to move the code that captures the changes to the selection to WebKit, the part that makes the X calls would still be hidden behind the PasteboardChromium.cpp api.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>872493</commentid>
    <comment_count>7</comment_count>
    <who name="Stephen Chenney">schenney</who>
    <bug_when>2013-04-09 17:07:09 -0700</bug_when>
    <thetext>Marked LayoutTest bugs, bugs with Chromium IDs, and some others as WontFix. Test failure bugs still are trackable via TestExpectations or disabled unit tests.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>