<?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>20632</bug_id>
          
          <creation_ts>2008-09-03 13:30:56 -0700</creation_ts>
          <short_desc>WebKitWebFrame should be replaced with GdomFrame</short_desc>
          <delta_ts>2010-09-10 08:08:29 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</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>0</everconfirmed>
          <reporter name="Luke Kenneth Casson Leighton">lkcl</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>alp</cc>
    
    <cc>mrobinson</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>90057</commentid>
    <comment_count>0</comment_count>
    <who name="Luke Kenneth Casson Leighton">lkcl</who>
    <bug_when>2008-09-03 13:30:56 -0700</bug_when>
    <thetext>... or the other way round.
GdomFrame contains auto-generated the complete set of bindings, whereas WebKitWebFrame is manually created glib code.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91041</commentid>
    <comment_count>1</comment_count>
    <who name="Alp Toker">alp</who>
    <bug_when>2008-09-13 07:17:18 -0700</bug_when>
    <thetext>(In reply to comment #0)
&gt; ... or the other way round.
&gt; GdomFrame contains auto-generated the complete set of bindings, whereas
&gt; WebKitWebFrame is manually created glib code.
&gt; 

The core WebKit/GTK+ API is intentionally hand-written to provide a tailored interface for developers. There isn&apos;t anything in the DOM that would work as a good replacement for WebKitWebFrame without customisation so the convention in WebKit ports is to provide accessors to the DOM binding from the core API rather than trying to use the DOM directly for all interaction with the browser engine.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>91064</commentid>
    <comment_count>2</comment_count>
    <who name="Luke Kenneth Casson Leighton">lkcl</who>
    <bug_when>2008-09-13 09:06:08 -0700</bug_when>
    <thetext>sorry, i&apos;d forgotten about this bugreport - i&apos;ve since worked out
that a much better suggestion would be for WebKitWebFrame to _use_
GdomFrame.

i.e. rather than having a Frame* have a GdomFrame*.

the reasoning is that there are a number of _functions_ which are
hand-created in WebKitWebFrame which could be replaced with
the gdom_frame_* equivalents.

plus, in some cases, especially the Location* ones - i remember
seeing someone talking on the irc about doing a URL class or
something, for WebKitWebFrame - of course, you now have gdom_location_*

ok - actually, you _don&apos;t_ have gdom_location_* yet because it&apos;s
a custom function in JSLocation which i haven&apos;t cut/paste over
to GDOM* yet.... )

... but you get the idea.

also, it would make it clearer to the qt porters that the possibility
exists to do exactly the same trick in there.  yes, they might
squawk a bit at #include &lt;gobject.h&gt; their competitor, but that would
kick them into gear to do a Qt bindings generator.

if WebKitWebFrame itself continues to have a Frame* then it remains
unclear as to exactly how other ports should go about adding in the
gdom bindings.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>277193</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2010-09-10 08:08:29 -0700</bug_when>
    <thetext>I think that we want to follow the behavior of other ports here and keep them as seperate entities.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>