<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[168712] releases/WebKitGTK/webkit-2.4</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/168712">168712</a></dd>
<dt>Author</dt> <dd>gns@gnome.org</dd>
<dt>Date</dt> <dd>2014-05-13 12:29:57 -0700 (Tue, 13 May 2014)</dd>
</dl>

<h3>Log Message</h3>
<pre>Merge 165060 - [GTK] Simplify the GObject DOM bindings API break check into one step
https://bugs.webkit.org/show_bug.cgi?id=129571

Reviewed by Carlos Garcia Campos.

.:

* Source/PlatformGTK.cmake: We don't need to run the API break check before gtkdoc generation.
Add the check to 'make check.'

Source/WebCore:

* bindings/gobject/GNUmakefile.am: We no longer generate the webkitdom.symbols file in the
DerivedSources directory. All the logic is handled internally in the script now.
* bindings/scripts/gobject-run-api-break-test: Removed.

Tools:

* GNUmakefile.am: Add the API break check to 'make check.'
* gtk/check-for-webkitdom-api-breaks: Added. A combination of the two removed scripts.
* gtk/check-gdom-symbols: Removed.
* gtk/webkitdom.py: Added.
(header_name_list_from_gtkdoc_config_file): Gets the names of all headers from the gtkdoc config file.
(get_all_webkitdom_symbol_files): Find the symbol files from the names.


Conflicts:
        ChangeLog
        Source/WebCore/ChangeLog
        Tools/ChangeLog</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#releasesWebKitGTKwebkit24SourcePlatformGTKcmake">releases/WebKitGTK/webkit-2.4/Source/PlatformGTK.cmake</a></li>
<li><a href="#releasesWebKitGTKwebkit24SourceWebCoreGNUmakefileam">releases/WebKitGTK/webkit-2.4/Source/WebCore/GNUmakefile.am</a></li>
<li><a href="#releasesWebKitGTKwebkit24SourceWebCorePlatformGTKcmake">releases/WebKitGTK/webkit-2.4/Source/WebCore/PlatformGTK.cmake</a></li>
<li><a href="#releasesWebKitGTKwebkit24SourceWebCorebindingsgobjectGNUmakefileam">releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/gobject/GNUmakefile.am</a></li>
<li><a href="#releasesWebKitGTKwebkit24ToolsGNUmakefileam">releases/WebKitGTK/webkit-2.4/Tools/GNUmakefile.am</a></li>
<li><a href="#releasesWebKitGTKwebkit24ToolsgtkGNUmakefileam">releases/WebKitGTK/webkit-2.4/Tools/gtk/GNUmakefile.am</a></li>
</ul>

<h3>Added Paths</h3>
<ul>
<li><a href="#releasesWebKitGTKwebkit24Toolsgtkcheckforwebkitdomapibreaks">releases/WebKitGTK/webkit-2.4/Tools/gtk/check-for-webkitdom-api-breaks</a></li>
<li><a href="#releasesWebKitGTKwebkit24Toolsgtkwebkitdompy">releases/WebKitGTK/webkit-2.4/Tools/gtk/webkitdom.py</a></li>
</ul>

<h3>Removed Paths</h3>
<ul>
<li><a href="#releasesWebKitGTKwebkit24SourceWebCorebindingsscriptsgobjectrunapibreaktest">releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/scripts/gobject-run-api-break-test</a></li>
<li><a href="#releasesWebKitGTKwebkit24Toolsgtkcheckgdomsymbols">releases/WebKitGTK/webkit-2.4/Tools/gtk/check-gdom-symbols</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="releasesWebKitGTKwebkit24SourcePlatformGTKcmake"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Source/PlatformGTK.cmake (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Source/PlatformGTK.cmake        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Source/PlatformGTK.cmake        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -4,7 +4,6 @@
</span><span class="cx"> add_custom_target(gir ALL DEPENDS ${GObjectIntrospectionTargets})
</span><span class="cx"> 
</span><span class="cx"> set(DocumentationDependencies
</span><del>-    generate-gdom-symbols-file
</del><span class="cx">     &quot;${CMAKE_SOURCE_DIR}/Source/WebKit/gtk/docs/webkitenvironment.xml&quot;
</span><span class="cx"> )
</span><span class="cx"> 
</span><span class="lines">@@ -51,7 +50,8 @@
</span><span class="cx"> endif ()
</span><span class="cx"> 
</span><span class="cx"> add_custom_target(check
</span><del>-    COMMAND &quot;${TOOLS_DIR}/Scripts/run-gtk-tests&quot;
</del><ins>+    COMMAND ${TOOLS_DIR}/Scripts/run-gtk-tests
+    COMMAND ${TOOLS_DIR}/gtk/check-for-webkitdom-api-breaks
</ins><span class="cx"> )
</span><span class="cx"> 
</span><span class="cx"> if (ENABLE_WEBKIT AND ENABLE_WEBKIT2)
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24SourceWebCoreGNUmakefileam"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Source/WebCore/GNUmakefile.am (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Source/WebCore/GNUmakefile.am        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Source/WebCore/GNUmakefile.am        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -682,7 +682,6 @@
</span><span class="cx">         Source/WebCore/bindings/scripts/StaticString.pm \
</span><span class="cx">         Source/WebCore/bindings/scripts/generate-bindings.pl \
</span><span class="cx">         Source/WebCore/bindings/scripts/gobject-generate-headers.pl \
</span><del>-        Source/WebCore/bindings/scripts/gobject-run-api-break-test \
</del><span class="cx">         Source/WebCore/bindings/scripts/preprocessor.pm \
</span><span class="cx">         Source/WebCore/bindings/scripts/preprocess-idls.pl \
</span><span class="cx">         Source/WebCore/ChangeLog \
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24SourceWebCorePlatformGTKcmake"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Source/WebCore/PlatformGTK.cmake (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Source/WebCore/PlatformGTK.cmake        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Source/WebCore/PlatformGTK.cmake        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -734,16 +734,5 @@
</span><span class="cx">                   bindings/gobject/WebKitDOMObject.h
</span><span class="cx">             DESTINATION &quot;${WEBKITGTK_HEADER_INSTALL_DIR}/webkitdom&quot;
</span><span class="cx">     )
</span><del>-
-    add_custom_command(
-        OUTPUT ${DERIVED_SOURCES_GOBJECT_DOM_BINDINGS_DIR}/webkitdom.symbols
-        DEPENDS ${GObjectDOMBindingsSymbolsFiles} ${CMAKE_SOURCE_DIR}/Tools/gtk/check-gdom-symbols
-        COMMAND ln -n -s -f ${WEBCORE_DIR}/bindings/gobject/WebKitDOM*.symbols ${DERIVED_SOURCES_GOBJECT_DOM_BINDINGS_DIR}
-        COMMAND ${CMAKE_SOURCE_DIR}/Tools/gtk/check-gdom-symbols
-    )
-
-    add_custom_target(generate-gdom-symbols-file
-        DEPENDS GObjectDOMBindings ${DERIVED_SOURCES_GOBJECT_DOM_BINDINGS_DIR}/webkitdom.symbols
-    )
</del><span class="cx"> endif ()
</span><span class="cx"> 
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24SourceWebCorebindingsgobjectGNUmakefileam"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/gobject/GNUmakefile.am (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/gobject/GNUmakefile.am        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/gobject/GNUmakefile.am        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -483,7 +483,6 @@
</span><span class="cx"> DerivedSources/webkitdom/webkitdomdefines.h: $(WebCore)/bindings/scripts/gobject-generate-headers.pl $(WebCore)/bindings/gobject/GNUmakefile.am
</span><span class="cx">         $(AM_V_GEN)echo $(gdom_class_list) | $(PERL) $&lt; defines &gt; $@
</span><span class="cx"> 
</span><del>-noinst_DATA += DerivedSources/webkitdom/webkitdom.symbols
</del><span class="cx"> gdom_symbol_files += $(patsubst %.h,%.symbols, $(filter DerivedSources/webkitdom/WebKitDOM%.h, $(webkitgtk_gdom_built_h_api)))
</span><span class="cx"> 
</span><span class="cx"> $(top_builddir)/DerivedSources/webkitdom/WebKitDOMObject.symbols: $(WebCore)/bindings/gobject/WebKitDOMObject.symbols
</span><span class="lines">@@ -501,9 +500,6 @@
</span><span class="cx"> DerivedSources/webkitdom/WebKitDOM%.symbols: DerivedSources/webkitdom/WebKitDOM%.h
</span><span class="cx">         @true
</span><span class="cx"> 
</span><del>-DerivedSources/webkitdom/webkitdom.symbols: $(gdom_symbol_files) $(WebCore)/bindings/gobject/webkitdom.symbols $(WebCore)/bindings/scripts/gobject-run-api-break-test $(srcdir)/Tools/gtk/check-gdom-symbols
-        $(AM_V_GEN)$(srcdir)/Tools/gtk/check-gdom-symbols
-
</del><span class="cx"> EXTRA_DIST += \
</span><span class="cx">         $(WebCore)/bindings/gobject/WebKitDOMCustom.symbols \
</span><span class="cx">         $(WebCore)/bindings/gobject/WebKitDOMDeprecated.symbols \
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24SourceWebCorebindingsscriptsgobjectrunapibreaktest"></a>
<div class="delfile"><h4>Deleted: releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/scripts/gobject-run-api-break-test (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/scripts/gobject-run-api-break-test        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Source/WebCore/bindings/scripts/gobject-run-api-break-test        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -1,67 +0,0 @@
</span><del>-#!/usr/bin/env python
-# Copyright (C) 2013 Igalia S.L.
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
-
-import difflib
-import optparse
-import shutil
-import sys
-
-def check_api(options, from_file, to_file):
-    from_lines = open(from_file, 'r').readlines()
-    from_lines.sort()
-    to_lines = open(to_file, 'r').readlines()
-    to_lines.sort()
-
-    diff = difflib.unified_diff(from_lines, to_lines)
-
-    diff_out = &quot;&quot;
-    failed = False
-    for line in diff:
-        # Ignore initial lines since we don't use from/file filenames.
-        if line.startswith('+++') or line.startswith('---'):
-            continue
-
-        diff_out += line
-        if line[0] == '-':
-            failed = True
-
-    if diff_out:
-        if failed:
-            sys.stderr.write(&quot;API incompatibility detected in GObject DOM bindings\n&quot;)
-            sys.stderr.write(diff_out)
-            # This test can give false positives because the GObject
-            # DOM bindings API varies depending on the compilation options.
-            # So this shouldn't be made fatal until we figure out a way to handle it.
-            # See https://bugs.webkit.org/show_bug.cgi?id=121481
-            return 0
-
-        if options.reset_results:
-            sys.stdout.write(&quot;Reset results: %s\n&quot; % from_file)
-            shutil.copyfile(to_file, from_file)
-        else:
-            sys.stdout.write(&quot;API compatible changes found in GObject DOM bindings\n&quot;)
-            sys.stdout.write(&quot;To update the symbols file, execute %s --reset-results %s DerivedSources/webkitdom/webkitdom.symbols\n&quot; % (sys.argv[0], from_file))
-            sys.stdout.write(diff_out)
-
-    return 0
-
-
-option_parser = optparse.OptionParser(usage='usage: %prog [options] FROM_FILE TO_FILE')
-option_parser.add_option('--reset-results', action='store_true', dest='reset_results')
-options, args = option_parser.parse_args()
-sys.exit(check_api(options, args[0], args[1]))
-
</del></span></pre></div>
<a id="releasesWebKitGTKwebkit24ToolsGNUmakefileam"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Tools/GNUmakefile.am (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Tools/GNUmakefile.am        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Tools/GNUmakefile.am        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -267,6 +267,7 @@
</span><span class="cx"> 
</span><span class="cx"> check-local:
</span><span class="cx">         $(top_srcdir)/Tools/Scripts/run-gtk-tests --timeout=-1
</span><ins>+        $(top_srcdir)/Tools/gtk/check-for-webkitdom-api-breaks
</ins><span class="cx"> 
</span><span class="cx"> EXTRA_DIST += \
</span><span class="cx">         Tools/jhbuild/jhbuildutils.py \
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24ToolsgtkGNUmakefileam"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.4/Tools/gtk/GNUmakefile.am (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Tools/gtk/GNUmakefile.am        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Tools/gtk/GNUmakefile.am        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -1,4 +1,5 @@
</span><span class="cx"> EXTRA_DIST += \
</span><ins>+        Tools/gtk/check-for-webkitdom-api-breaks \
</ins><span class="cx">         Tools/gtk/common.py \
</span><span class="cx">         Tools/gtk/generate-feature-defines-files \
</span><span class="cx">         Tools/gtk/generate-gtkdoc \
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit24Toolsgtkcheckforwebkitdomapibreaks"></a>
<div class="addfile"><h4>Added: releases/WebKitGTK/webkit-2.4/Tools/gtk/check-for-webkitdom-api-breaks (0 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Tools/gtk/check-for-webkitdom-api-breaks                                (rev 0)
+++ releases/WebKitGTK/webkit-2.4/Tools/gtk/check-for-webkitdom-api-breaks        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -0,0 +1,78 @@
</span><ins>+#!/usr/bin/env python
+# Copyright (C) 2013, 2014 Igalia S.L.
+# Copyright (C) 2013 Gustavo Noronha Silva &lt;gns@gnome.org&gt;
+#
+# This library is free software; you can redistribute it and/or
+# modify it under the terms of the GNU Lesser General Public
+# License as published by the Free Software Foundation; either
+# version 2 of the License, or (at your option) any later version.
+#
+# This library is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this library; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+
+import argparse
+import common
+import os
+import sys
+import webkitdom
+
+EXPECTED_API_PATH = common.top_level_path('Source', 'WebCore', 'bindings', 'gobject', 'webkitdom.symbols')
+
+def read_built_api():
+    apis = set()
+    for file_path in webkitdom.get_all_webkitdom_symbol_files():
+        with open(file_path) as file_handle:
+            apis.update(set(file_handle.readlines()))
+    return apis
+
+def read_expected_api():
+    with open(EXPECTED_API_PATH) as file_handle:
+        return set(file_handle.readlines())
+
+def write_expected_api(new_expected_api):
+    with open(EXPECTED_API_PATH, 'w') as file_handle:
+        file_handle.writelines(new_expected_api)
+
+def check_api(options, expected_api, built_api):
+    missing_api = expected_api.difference(built_api)
+    new_api = built_api.difference(expected_api)
+
+    if missing_api:
+        sys.stderr.write(&quot;Missing API (API break!) detected in GObject DOM bindings\n&quot;)
+        sys.stderr.write(&quot;    %s\n&quot; % &quot;    &quot;.join(missing_api))
+        sys.stderr.flush()
+
+    if new_api:
+        sys.stdout.write(&quot;New API detected in GObject DOM bindings\n&quot;)
+        sys.stdout.write(&quot;    %s\n&quot; % &quot;    &quot;.join(new_api))
+
+    if missing_api:
+        # This test can give false positives because the GObject
+        # DOM bindings API varies depending on the compilation options.
+        # So this shouldn't be made fatal until we figure out a way to handle it.
+        # See https://bugs.webkit.org/show_bug.cgi?id=121481
+        sys.stderr.write(&quot;Re-add the missing API and rerun the %s.\n&quot; % __file__)
+        return 0
+
+    if new_api:
+        if options.reset_results:
+            sys.stdout.write(&quot;Resetting expected API\n&quot;)
+            write_expected_api(built_api)
+        else:
+            sys.stdout.write(&quot;API compatible changes found in GObject DOM bindings.\n&quot;)
+            sys.stdout.write(&quot;To update the symbols file, run %s --reset-results.\n&quot; % __file__)
+
+    return 0
+
+if __name__ == '__main__':
+    parser = argparse.ArgumentParser(description='Detect API breakage in the WebKitGTK+ GObject DOM bindings.')
+    parser.add_argument('--reset-results', action='store_true',
+                        help='When specified, rest the expected results file with the built results.')
+    options = parser.parse_args()
+    sys.exit(check_api(options, read_expected_api(), read_built_api()))
</ins><span class="cx">Property changes on: releases/WebKitGTK/webkit-2.4/Tools/gtk/check-for-webkitdom-api-breaks
</span><span class="cx">___________________________________________________________________
</span></span></pre></div>
<a id="svnexecutable"></a>
<div class="addfile"><h4>Added: svn:executable</h4></div>
<a id="releasesWebKitGTKwebkit24Toolsgtkcheckgdomsymbols"></a>
<div class="delfile"><h4>Deleted: releases/WebKitGTK/webkit-2.4/Tools/gtk/check-gdom-symbols (168711 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Tools/gtk/check-gdom-symbols        2014-05-13 19:29:34 UTC (rev 168711)
+++ releases/WebKitGTK/webkit-2.4/Tools/gtk/check-gdom-symbols        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -1,62 +0,0 @@
</span><del>-#!/usr/bin/env python
-# Copyright (C) 2013 Gustavo Noronha Silva &lt;gns@gnome.org&gt;
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
-
-import common
-import glob
-import os
-import subprocess
-import sys
-import tempfile
-
-def should_update_symbols_file(new, old):
-    if not os.path.exists(old):
-        return False
-
-    new = open(new).read()
-    old = open(old).read()
-
-    if len(new) != len(old):
-        return False
-
-    for i, byte in enumerate(new):
-        if byte != old[i]:
-            return False
-
-    return True
-
-if __name__ == '__main__':
-    with tempfile.NamedTemporaryFile() as tmp:
-        for file_path in glob.glob(os.path.join(common.build_path('DerivedSources'), 'webkitdom', '*.h')):
-            if not os.path.basename(file_path).startswith('WebKit') or file_path.endswith('Private.h'):
-                continue
-
-            file_path = file_path.replace('.h', '.symbols')
-            with open(file_path) as file_handle:
-                tmp.write(file_handle.read())
-        tmp.flush()
-
-        gdom_source_path = common.top_level_path('Source', 'WebCore', 'bindings')
-        api_break_test_path = os.path.join(gdom_source_path, 'scripts', 'gobject-run-api-break-test')
-        subprocess.call([sys.executable, api_break_test_path, os.path.join(gdom_source_path, 'gobject',  'webkitdom.symbols'), tmp.name])
-
-        generated_gdom_symbols_path = os.path.join(common.build_path('DerivedSources'), 'webkitdom', 'webkitdom.symbols')
-        if not should_update_symbols_file(tmp.name, generated_gdom_symbols_path):
-            source = open(tmp.name, 'r')
-            destination = open(generated_gdom_symbols_path, 'w')
-            destination.write(source.read())
-            destination.close()
-            source.close()
</del></span></pre></div>
<a id="releasesWebKitGTKwebkit24Toolsgtkwebkitdompy"></a>
<div class="addfile"><h4>Added: releases/WebKitGTK/webkit-2.4/Tools/gtk/webkitdom.py (0 => 168712)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.4/Tools/gtk/webkitdom.py                                (rev 0)
+++ releases/WebKitGTK/webkit-2.4/Tools/gtk/webkitdom.py        2014-05-13 19:29:57 UTC (rev 168712)
</span><span class="lines">@@ -0,0 +1,61 @@
</span><ins>+#!/usr/bin/env python
+# Copyright (C) 2013 Igalia S.L.
+#
+# This library is free software; you can redistribute it and/or
+# modify it under the terms of the GNU Lesser General Public
+# License as published by the Free Software Foundation; either
+# version 2 of the License, or (at your option) any later version.
+#
+# This library is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this library; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+# 02110-1301  USA
+
+import common
+import os
+import sys
+
+from ConfigParser import SafeConfigParser
+
+
+def header_name_list_from_gtkdoc_config_file():
+    config_file = common.build_path('gtkdoc-webkitdom.cfg')
+    if not os.path.isfile(config_file):
+        sys.stderr.write(&quot;Could not find config file at %s\n&quot; % config_file)
+        return sys.exit(1)
+
+    config = SafeConfigParser()
+    config.read(config_file)
+    module_name = config.sections()[0]
+    return [os.path.basename(f) for f in str(config.get(module_name, 'headers')).replace(';', ' ').split()]
+
+
+def get_all_webkitdom_symbol_files():
+    static_symbol_files_path = common.top_level_path('Source', 'WebCore', 'bindings', 'gobject')
+    generated_symbol_files_path = common.build_path('DerivedSources', 'webkitdom')
+
+    symbol_files = []
+    for header_name in header_name_list_from_gtkdoc_config_file():
+        # webkitdomdefines.h doesn't have a corresponding symbols file and webkitdom.symbols is a
+        # file containing the expected symbols results.
+        if header_name in (&quot;webkitdom.h&quot;, &quot;webkitdomdefines.h&quot;):
+            continue
+
+        symbol_file = header_name.replace(&quot;.h&quot;, &quot;.symbols&quot;)
+        path = os.path.join(static_symbol_files_path, symbol_file)
+        if os.path.exists(path):
+            symbol_files.append(path)
+            continue
+        path = os.path.join(generated_symbol_files_path, symbol_file)
+        if os.path.exists(path):
+            symbol_files.append(path)
+            continue
+        sys.stderr.write(&quot;Could not find symbol file for header: %s\n&quot; % header_name)
+        sys.exit(1)
+
+    return symbol_files
</ins><span class="cx">Property changes on: releases/WebKitGTK/webkit-2.4/Tools/gtk/webkitdom.py
</span><span class="cx">___________________________________________________________________
</span></span></pre></div>
<a id="svnexecutable"></a>
<div class="addfile"><h4>Added: svn:executable</h4></div>
</div>

</body>
</html>