<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [GTK] Hangs when showing Google search results"
   href="https://bugs.webkit.org/show_bug.cgi?id=168699#c18">Comment # 18</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [GTK] Hangs when showing Google search results"
   href="https://bugs.webkit.org/show_bug.cgi?id=168699">bug 168699</a>
              from <span class="vcard"><a class="email" href="mailto:cgarcia&#64;igalia.com" title="Carlos Garcia Campos &lt;cgarcia&#64;igalia.com&gt;"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=168699#c17">comment #17</a>)
<span class="quote">&gt; (In reply to <a href="show_bug.cgi?id=168699#c16">comment #16</a>)
&gt; &gt; Right, now that EFL is gone, that's exactly my plan. We could also make it
&gt; &gt; ifdef free by converting our socket monitor into a platform-specific class
&gt; &gt; with platform implementations. However, now that there's no other unix ports
&gt; &gt; using this code, it's not easy for me to do that refactoring without knowing
&gt; &gt; the other implementations.
&gt; 
&gt; Let's do this. Solution is to rename it to ConnectionGLib.cpp and just
&gt; remove ConnectionUnix.cpp. If some future port wants to use it, they can
&gt; bring it back at that time.</span >

No, I haven't proposed this, a possible ConnectionGLib would be quite different (and simpler) using the GSocket API por everything. ConnectionUnix is common code except for the socket monitor, so let's just change the ifdefs to USE(GLIB). What I proposed was to make the socket monitor a common interface with platform implementations not the whole file. When we added this implementation, we (GTK, EFL and Qt ports) decided not to use platform APIs for this, to share the implementation.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>