<!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>[51733] trunk/WebKitSite</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/51733">51733</a></dd>
<dt>Author</dt> <dd>eric@webkit.org</dd>
<dt>Date</dt> <dd>2009-12-05 16:30:28 -0800 (Sat, 05 Dec 2009)</dd>
</dl>

<h3>Log Message</h3>
<pre>2009-12-05  Chris Jerdonek  &lt;chris.jerdonek@gmail.com&gt;

        Reviewed by Darin Adler.

        Refactored web site documentation so that advice to add
        WebKitTools/Scripts to your shell path does not appear
        on multiple pages.

        https://bugs.webkit.org/show_bug.cgi?id=32112

        * building/build.html:
        * building/checkout.html:
        * building/run.html:
        * coding/coding-style.html:
        * coding/contributing.html:
        * coding/scripts.html: Added.
        * quality/testing.html:</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkWebKitSiteChangeLog">trunk/WebKitSite/ChangeLog</a></li>
<li><a href="#trunkWebKitSitebuildingbuildhtml">trunk/WebKitSite/building/build.html</a></li>
<li><a href="#trunkWebKitSitebuildingcheckouthtml">trunk/WebKitSite/building/checkout.html</a></li>
<li><a href="#trunkWebKitSitebuildingrunhtml">trunk/WebKitSite/building/run.html</a></li>
<li><a href="#trunkWebKitSitecodingcodingstylehtml">trunk/WebKitSite/coding/coding-style.html</a></li>
<li><a href="#trunkWebKitSitecodingcontributinghtml">trunk/WebKitSite/coding/contributing.html</a></li>
<li><a href="#trunkWebKitSitequalitytestinghtml">trunk/WebKitSite/quality/testing.html</a></li>
</ul>

<h3>Added Paths</h3>
<ul>
<li><a href="#trunkWebKitSitecodingscriptshtml">trunk/WebKitSite/coding/scripts.html</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkWebKitSiteChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/ChangeLog (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/ChangeLog        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/ChangeLog        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -1,3 +1,21 @@
</span><ins>+2009-12-05  Chris Jerdonek  &lt;chris.jerdonek@gmail.com&gt;
+
+        Reviewed by Darin Adler.
+
+        Refactored web site documentation so that advice to add
+        WebKitTools/Scripts to your shell path does not appear 
+        on multiple pages.
+        
+        https://bugs.webkit.org/show_bug.cgi?id=32112
+        
+        * building/build.html:
+        * building/checkout.html:
+        * building/run.html:
+        * coding/coding-style.html:
+        * coding/contributing.html:
+        * coding/scripts.html: Added.
+        * quality/testing.html:
+
</ins><span class="cx"> 2009-12-04  Marc-Antoine Ruel  &lt;maruel@chromium.org&gt;
</span><span class="cx"> 
</span><span class="cx">         Reviewed by Eric Seidel.
</span></span></pre></div>
<a id="trunkWebKitSitebuildingbuildhtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/building/build.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/building/build.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/building/build.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -17,10 +17,11 @@
</span><span class="cx"> &lt;/div&gt;
</span><span class="cx"> 
</span><span class="cx"> &lt;ol&gt;
</span><del>-&lt;li&gt;&lt;p&gt;Type this command to build WebKit:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/build-webkit&lt;/p&gt;
-&lt;p&gt;Use the &lt;tt&gt;--debug&lt;/tt&gt; option for a debug build, which includes debugging symbols and assertions:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/build-webkit --debug&lt;/p&gt;
</del><ins>+&lt;li&gt;&lt;p&gt;Run the &lt;tt&gt;build-webkit&lt;/tt&gt; &lt;a href=&quot;/coding/scripts.html&quot;&gt;script&lt;/a&gt;
+to build WebKit.&lt;/p&gt;
+&lt;p&gt;Use the &lt;tt&gt;--debug&lt;/tt&gt; option for a debug build, which includes
+debugging symbols and assertions:&lt;/p&gt;
+&lt;p class=&quot;code&quot;&gt;build-webkit --debug&lt;/p&gt;
</ins><span class="cx"> &lt;/li&gt;
</span><span class="cx"> &lt;/ol&gt;
</span><span class="cx"> 
</span><span class="lines">@@ -38,19 +39,19 @@
</span><span class="cx"> &lt;p&gt;Don't forget that if you have any questions or problems building WebKit, feel free to &lt;a href=&quot;/contact.html&quot;&gt;get in touch!&lt;/a&gt;&lt;/p&gt;
</span><span class="cx"> &lt;/div&gt;
</span><span class="cx"> 
</span><ins>+&lt;p&gt;Once your build has finished, you can &lt;a href=&quot;run.html&quot;&gt;run it inside
+Safari.&lt;/a&gt;&lt;/p&gt;
+
</ins><span class="cx"> &lt;h2&gt;Setting a Default Configuration&lt;/h2&gt;
</span><span class="cx"> &lt;ol&gt;
</span><del>-&lt;li&gt;&lt;p&gt;To set a default build configuration for &lt;tt&gt;build-webkit&lt;/tt&gt; and other scripts, use the &lt;tt&gt;set-webkit-configuration&lt;/tt&gt; script:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/set-webkit-configuration --debug&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/set-webkit-configuration --release&lt;/p&gt;
</del><ins>+&lt;li&gt;&lt;p&gt;To set a default build configuration for &lt;tt&gt;build-webkit&lt;/tt&gt; and
+other &lt;a href=&quot;/coding/scripts.html&quot;&gt;scripts&lt;/a&gt;, use the
+&lt;tt&gt;set-webkit-configuration&lt;/tt&gt; script:&lt;/p&gt;
+&lt;p class=&quot;code&quot;&gt;set-webkit-configuration --debug&lt;/p&gt;
+&lt;p class=&quot;code&quot;&gt;set-webkit-configuration --release&lt;/p&gt;
</ins><span class="cx"> &lt;/li&gt;
</span><span class="cx"> &lt;/ol&gt;
</span><span class="cx"> 
</span><del>-&lt;hr&gt;
-&lt;p&gt;It's handy to put &lt;tt&gt;WebKit/WebKitTools/Scripts&lt;/tt&gt; in your shell path so you can type commands like &lt;tt&gt;build-webkit&lt;/tt&gt; without specifying a full path.&lt;/p&gt;
-
-&lt;p&gt;Once your build has finished, you can &lt;A href=&quot;run.html&quot;&gt;run it inside Safari.&lt;/a&gt;&lt;/p&gt;
-
</del><span class="cx"> &lt;?php
</span><span class="cx">     include(&quot;../footer.inc&quot;);
</span><span class="cx"> ?&gt;
</span></span></pre></div>
<a id="trunkWebKitSitebuildingcheckouthtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/building/checkout.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/building/checkout.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/building/checkout.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -40,14 +40,16 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;div class=&quot;windows-instructions&quot;&gt;
</span><span class="cx"> &lt;h4&gt;Windows&lt;/h4&gt;
</span><ins>+&lt;ol&gt;
</ins><span class="cx"> &lt;li&gt;&lt;p&gt;Install the WebKit Support Libraries&lt;/p&gt;
</span><span class="cx"> &lt;p&gt;Download the &lt;a href=&quot;http://developer.apple.com/opensource/internet/webkit_sptlib_agree.html&quot;&gt;WebKit Support Libraries&lt;/a&gt; to the root of your source tree (&lt;tt&gt;C:\cygwin\home\&amp;lt;username&amp;gt;\WebKit&lt;/tt&gt;).&lt;/p&gt;
</span><span class="cx"> &lt;p&gt;If the file is incorrectly named, rename it to &lt;tt&gt;WebKitSupportLibrary.zip&lt;/tt&gt;. Do not extract its contents.&lt;/p&gt;
</span><ins>+&lt;/ol&gt;
</ins><span class="cx"> &lt;/div&gt;
</span><span class="cx"> 
</span><span class="cx"> &lt;ol&gt;
</span><del>-&lt;li&gt;&lt;p&gt;Type this command to update your source tree:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/update-webkit&lt;/p&gt;
</del><ins>+&lt;li&gt;&lt;p&gt;Run the &lt;tt&gt;update-webkit&lt;/tt&gt; &lt;a href=&quot;/coding/scripts.html&quot;&gt;script&lt;/a&gt;
+to update your source tree.&lt;/p&gt;
</ins><span class="cx"> &lt;p&gt;If you downloaded the tarball, this will bring it up to date.  Windows users
</span><span class="cx"> must always execute this command after first obtaining the code, since it will
</span><span class="cx"> download additional libraries that are needed to build.&lt;/p&gt;
</span><span class="lines">@@ -57,11 +59,9 @@
</span><span class="cx"> &lt;a href=&quot;build.html&quot;&gt;building WebKit&lt;/a&gt;.&lt;/p&gt;
</span><span class="cx"> 
</span><span class="cx"> &lt;h2&gt;Keeping up to Date&lt;/h2&gt;
</span><del>-&lt;p&gt;At any time, you can retype the &lt;tt&gt;update-webkit&lt;/tt&gt; command to update your source tree:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/update-webkit&lt;/p&gt;
</del><ins>+&lt;p&gt;At any time, you can rerun the &lt;tt&gt;update-webkit&lt;/tt&gt; script to update
+your source tree.&lt;/p&gt;
</ins><span class="cx"> 
</span><del>-&lt;hr&gt;
-&lt;p&gt;It's handy to put &lt;tt&gt;WebKit/WebKitTools/Scripts&lt;/tt&gt; in your shell path so you can type commands like &lt;tt&gt;update-webkit&lt;/tt&gt; without specifying a full path.&lt;/p&gt;
</del><span class="cx"> &lt;?php
</span><span class="cx"> include(&quot;../footer.inc&quot;);
</span><span class="cx"> ?&gt;
</span></span></pre></div>
<a id="trunkWebKitSitebuildingrunhtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/building/run.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/building/run.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/building/run.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -5,10 +5,10 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;h2&gt;Running WebKit&lt;/h2&gt;
</span><span class="cx"> &lt;ol&gt;
</span><del>-&lt;li&gt;&lt;p&gt;Type this command to run Safari with the WebKit you built:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/run-safari&lt;/p&gt;
</del><ins>+&lt;li&gt;&lt;p&gt;Execute the &lt;tt&gt;run-safari&lt;/tt&gt; &lt;a href=&quot;/coding/scripts.html&quot;&gt;script&lt;/a&gt;
+to run Safari with the WebKit version you &lt;a href=&quot;build.html&quot;&gt;built&lt;/a&gt;.&lt;/p&gt;
</ins><span class="cx"> &lt;p&gt;Use the &lt;tt&gt;--debug&lt;/tt&gt; option for a debug build:&lt;/p&gt;
</span><del>-&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/run-safari --debug&lt;/p&gt;
</del><ins>+&lt;p class=&quot;code&quot;&gt;run-safari --debug&lt;/p&gt;
</ins><span class="cx"> &lt;/li&gt;
</span><span class="cx"> &lt;/ol&gt;
</span><span class="cx"> 
</span><span class="lines">@@ -20,10 +20,8 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;div class=&quot;windows-instructions&quot;&gt;
</span><span class="cx"> &lt;h4&gt;Windows&lt;/h4&gt;
</span><del>-&lt;p&gt;The &lt;tt&gt;run-safari&lt;/tt&gt; script launches the Safari executable with the &lt;tt&gt;/frameworkPath&lt;/tt&gt; command line switch set to point to your build products.  The &lt;tt&gt;/debug&lt;/tt&gt; command line switch will be set if you pass --debug to run-safari.&lt;/p&gt;
</del><ins>+&lt;p&gt;The &lt;tt&gt;run-safari&lt;/tt&gt; script launches the Safari executable with the &lt;tt&gt;/frameworkPath&lt;/tt&gt; command line switch set to point to your build products.  The &lt;tt&gt;/debug&lt;/tt&gt; command line switch will be set if you pass --debug to &lt;tt&gt;run-safari.&lt;/tt&gt;&lt;/p&gt;
</ins><span class="cx"> &lt;/div&gt;
</span><del>-&lt;hr&gt;
-&lt;p&gt;It's handy to put &lt;tt&gt;WebKit/WebKitTools/Scripts&lt;/tt&gt; in your shell path so you can type commands like &lt;tt&gt;run-safari&lt;/tt&gt; without specifying a full path.&lt;/p&gt;
</del><span class="cx"> 
</span><span class="cx"> &lt;?php
</span><span class="cx">     include(&quot;../footer.inc&quot;);
</span></span></pre></div>
<a id="trunkWebKitSitecodingcodingstylehtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/coding/coding-style.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/coding/coding-style.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/coding/coding-style.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -742,7 +742,7 @@
</span><span class="cx"> &lt;/li&gt;
</span><span class="cx"> 
</span><span class="cx"> &lt;li&gt;It is acceptable, however, to use &quot;using&quot; declarations at the end of 
</span><del>-header files in the WTF sub-libary to include one or more names in 
</del><ins>+header files in the WTF sub-library to include one or more names in 
</ins><span class="cx"> the WTF namespace into the global scope.
</span><span class="cx"> 
</span><span class="cx"> &lt;h4 class=&quot;right&quot;&gt;Right:&lt;/h4&gt;
</span></span></pre></div>
<a id="trunkWebKitSitecodingcontributinghtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/coding/contributing.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/coding/contributing.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/coding/contributing.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -4,9 +4,17 @@
</span><span class="cx"> ?&gt;
</span><span class="cx"> &lt;h2&gt;Contributing Code&lt;/h2&gt;
</span><span class="cx"> &lt;p&gt;Contributing code to the WebKit project is a straightforward process.
</span><del>-Once you have the code &lt;a href=&quot;/building/checkout.html&quot;&gt;checked out&lt;/a&gt;, &lt;a href=&quot;/building/build.html&quot;&gt;built&lt;/a&gt;, and made your changes, you'll need to do a few things in order to get it landed in the tree:&lt;/p&gt;
</del><ins>+The project maintains several &lt;a href=&quot;scripts.html&quot;&gt;scripts&lt;/a&gt;
+to assist you with this process.&lt;/p&gt;
+
+&lt;p&gt;Once you have &lt;a href=&quot;/building/checkout.html&quot;&gt;checked out&lt;/a&gt;,
+&lt;a href=&quot;/building/build.html&quot;&gt;built&lt;/a&gt;, and made your changes to the code,
+you'll need to do a few things to get your changes landed in the tree:&lt;/p&gt;
+
</ins><span class="cx"> &lt;ol&gt;
</span><del>-    &lt;li&gt;Make sure your changes meet the &lt;a href=&quot;/coding/coding-style.html&quot;&gt;code style guidelines&lt;/a&gt;. &lt;tt&gt;check-webkit-style&lt;/tt&gt; may be of help.&lt;/li&gt;
</del><ins>+    &lt;li&gt;Make sure your changes meet the &lt;a href=&quot;/coding/coding-style.html&quot;&gt;code
+        style guidelines&lt;/a&gt;. The &lt;tt&gt;check-webkit-style&lt;/tt&gt; script may be of
+        help.&lt;/li&gt;
</ins><span class="cx">     &lt;li&gt;Run the layout tests using the &lt;tt&gt;run-webkit-tests&lt;/tt&gt; script and make sure they all pass.
</span><span class="cx">         See the &lt;a href=&quot;/quality/testwriting.html&quot;&gt;testing page&lt;/a&gt; for more information, as well as what you need to do if you've modified JavaScriptCore.&lt;/li&gt;
</span><span class="cx">     &lt;li&gt;Add any new files and layout tests to Subversion using the &lt;tt&gt;svn add&lt;/tt&gt; command.&lt;/li&gt;
</span><span class="lines">@@ -18,6 +26,7 @@
</span><span class="cx">     &lt;li&gt;Please watch for any regressions it may have caused (hopefully none)!&lt;/li&gt;
</span><span class="cx"> &lt;/ol&gt;
</span><span class="cx"> 
</span><ins>+&lt;p&gt;More detail about these steps is below.&lt;/p&gt;
</ins><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;Code Style Guidelines&lt;/h3&gt;
</span><span class="cx"> &lt;p&gt;Patches must comply with the &lt;a href=&quot;/coding/coding-style.html&quot;&gt;code style guidelines&lt;/a&gt;.
</span><span class="lines">@@ -51,14 +60,20 @@
</span><span class="cx"> &lt;p&gt;The line WARNING: NO TEST CASES ADDED OR CHANGED appears if prepare-ChangeLog did not detect the addition of test cases.  If your patch does not require test cases (or test cases are not possible), you should include a line stating such.  Otherwise all changes require test cases which should be mentioned in the ChangeLog.&lt;/p&gt;
</span><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;Create the patch&lt;/h3&gt;
</span><del>-&lt;p&gt;WebKit uses the &lt;tt&gt;svn-create-patch&lt;/tt&gt; script to create patches.  &lt;tt&gt;svn-create-patch&lt;/tt&gt; is just a small wrapper around Subversion's &lt;tt&gt;diff&lt;/tt&gt; command which knows how to better handle moved, added, and deleted files. This command is best run from the top level of your checkout to make
-sure no changes are left out of your patch. It is not necessary to break a patch into multiple files.&lt;/p&gt;
</del><ins>+&lt;p&gt;WebKit uses &lt;tt&gt;svn-create-patch&lt;/tt&gt; to create patches. The 
+&lt;tt&gt;svn-create-patch&lt;/tt&gt; script is a small wrapper around Subversion's 
+&lt;tt&gt;diff&lt;/tt&gt; command that better handles moved, added, and deleted files.
+This command is best run from the top level of your checkout 
+to make sure no changes are left out of your patch. It is not necessary to 
+break a patch into multiple files.&lt;/p&gt;
</ins><span class="cx"> 
</span><del>-&lt;p&gt;&lt;tt&gt;svn-create-patch&lt;/tt&gt; does not create a file automatically, you need to redirect the output yourself using something like: &lt;tt&gt;svn-create-patch &gt; MyExcellentPatch.txt&lt;/tt&gt;&lt;/p&gt;
</del><ins>+&lt;p&gt;The &lt;tt&gt;svn-create-patch&lt;/tt&gt; script does not create a file automatically.
+You need to redirect the output yourself using something like--&lt;/p&gt;
+&lt;p class=&quot;code&quot;&gt;svn-create-patch &gt; MyExcellentPatch.txt&lt;/p&gt;
</ins><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;Patch review&lt;/h3&gt;
</span><del>-&lt;p&gt;Once you have a patch file, it must be reviewed by one of the approved WebKit reviewers. To request a review, attach the patch
-to the bug report, and mark the patch with the flag &lt;tt&gt;review:?&lt;/tt&gt;. The reviewer will typically either approve the patch
</del><ins>+&lt;p&gt;Once you have a patch file, one of the approved WebKit reviewers must review it.
+To request a review, attach the patch to the bug report, and mark the patch with the flag &lt;tt&gt;review:?&lt;/tt&gt;. The reviewer will typically either approve the patch
</ins><span class="cx"> (by responding with an &lt;tt&gt;r=me&lt;/tt&gt; in the bug report and marking the patch &lt;tt&gt;review:+&lt;/tt&gt;) or request revisions
</span><span class="cx"> to the patch (and mark the patch &lt;tt&gt;review:-&lt;/tt&gt;). In rare cases a patch may be permanently rejected, meaning that the reviewer
</span><span class="cx"> believes the feature should never be committed to the tree. The review process can consist of multiple iterations between you and
</span><span class="lines">@@ -75,12 +90,6 @@
</span><span class="cx"> &lt;h4 id=&quot;commitqueue&quot;&gt;Optional: Use of the WebKit Commit Bot&lt;/h4&gt;
</span><span class="cx"> &lt;p&gt;WebKit provides an automated system (commit-queue) for landing patches for any who would like to use it.  To use the commit-queue, set the &lt;tt&gt;commit-queue:?&lt;/tt&gt; flag on your patch.  A committer will set &lt;tt&gt;commit-queue:+&lt;/tt&gt; and an automated process will download, build, run the layout tests, and submit your patch on your behalf.  If the &lt;a href=&quot;http://build.webkit.org/&quot;&gt;WebKit buildbots&lt;/a&gt; are passing, your patch should be landed within 15 minutes after &lt;tt&gt;commit-queue:+&lt;/tt&gt; is set. See the &lt;a href=&quot;https://trac.webkit.org/wiki/CommitQueue&quot;&gt;commit-queue documentation&lt;/a&gt; for more information.&lt;/p&gt;
</span><span class="cx"> 
</span><del>-&lt;h2&gt;WebKit Scripts&lt;/h2&gt;
-&lt;p&gt;&lt;tt&gt;WebKitTools/Scripts&lt;/tt&gt; contains a number of scripts to help make life easier when submitting a patch. All scripts mentioned
-on this page (and on the rest of the site as well) are located here unless otherwise mentioned.&lt;/p&gt;
-&lt;p&gt;It's handy to put this directory in your shell path so you can just type the commands without having to specify the path to
-the script each time.&lt;/p&gt;
-
</del><span class="cx"> &lt;h2&gt;Obtaining Commit and Review Privileges&lt;/h2&gt;
</span><span class="cx"> &lt;p&gt;Our &lt;a href=&quot;commit-review-policy.html&quot;&gt;Committer and Reviewer policy&lt;/a&gt; provides details on obtaining commit and review privileges.&lt;/p&gt;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkWebKitSitecodingscriptshtml"></a>
<div class="addfile"><h4>Added: trunk/WebKitSite/coding/scripts.html (0 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/coding/scripts.html                                (rev 0)
+++ trunk/WebKitSite/coding/scripts.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -0,0 +1,46 @@
</span><ins>+&lt;?php
+    $title=&quot;WebKit Scripts&quot;;
+    include(&quot;../header.inc&quot;);
+?&gt;
+&lt;h2&gt;WebKit Scripts&lt;/h2&gt;
+
+&lt;p&gt;The WebKit project maintains several scripts to assist with things
+like building and testing code, checking style, and preparing patches.&lt;p&gt;
+
+&lt;p&gt;The &lt;tt&gt;WebKitTools/Scripts&lt;/tt&gt;
+&lt;a href=&quot;http://trac.webkit.org/browser/trunk/WebKitTools/Scripts&quot;&gt;folder&lt;/a&gt;
+of the WebKit source tree contains these scripts.
+In fact, unless otherwise mentioned, all of the scripts mentioned
+on this site are located in this folder.&lt;/p&gt;
+
+&lt;h3&gt;Running a script&lt;/h3&gt;
+
+&lt;p&gt;To run the &lt;tt&gt;build-webkit&lt;/tt&gt; script, for example, type the
+following from the command line:&lt;/p&gt;
+
+&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/build-webkit&lt;/p&gt;
+
+&lt;p&gt;Similarly, to run the &lt;tt&gt;build-webkit&lt;/tt&gt; script with the
+&lt;tt&gt;--help&lt;/tt&gt; option, type the following:&lt;p&gt;
+
+&lt;p class=&quot;code&quot;&gt;WebKit/WebKitTools/Scripts/build-webkit --help&lt;/p&gt;
+
+&lt;h3&gt;Adding to your path&lt;/h3&gt;
+
+&lt;p&gt;It is convenient to add &lt;tt&gt;WebKit/WebKitTools/Scripts&lt;/tt&gt; to your
+shell path so you can run WebKit scripts without typing the full path.&lt;p&gt;
+
+&lt;p&gt;For example, this will allow you to type the following for the two
+examples above:&lt;/p&gt;
+
+&lt;p class=&quot;code&quot;&gt;build-webkit&lt;/p&gt;
+&lt;p class=&quot;code&quot;&gt;build-webkit --help&lt;/p&gt;
+
+&lt;p&gt;The instructions throughout this site assume you have added
+&lt;tt&gt;WebKit/WebKitTools/Scripts&lt;/tt&gt; to your shell path.
+If you do not add this directory to your shell path, simply always include
+the full path as in the first examples above.&lt;/p&gt;
+
+&lt;?php
+    include(&quot;../footer.inc&quot;);
+?&gt;
</ins></span></pre></div>
<a id="trunkWebKitSitequalitytestinghtml"></a>
<div class="modfile"><h4>Modified: trunk/WebKitSite/quality/testing.html (51732 => 51733)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/WebKitSite/quality/testing.html        2009-12-05 23:59:24 UTC (rev 51732)
+++ trunk/WebKitSite/quality/testing.html        2009-12-06 00:30:28 UTC (rev 51733)
</span><span class="lines">@@ -6,9 +6,9 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;The WebKit Tests&lt;/h3&gt;
</span><span class="cx"> 
</span><del>-&lt;p&gt;Before patches can land in any of the frameworks in the repository, the layout regression tests must be run. To run these tests, execute the &lt;tt&gt;run-webkit-tests&lt;/tt&gt; script.&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKitTools/Scripts/run-webkit-tests&lt;/p&gt;
-&lt;p&gt;It's handy to put the &lt;tt&gt;WebKitTools/Scripts&lt;/tt&gt; directory in your shell path so you can type commands like &lt;tt&gt;run-webkit-tests&lt;/tt&gt; without specifying the path to the script.&lt;/p&gt;
</del><ins>+&lt;p&gt;Before patches can land in any of the frameworks in the repository, the
+layout regression tests must pass. To run these tests, execute the
+&lt;tt&gt;run-webkit-tests&lt;/tt&gt; &lt;a href=&quot;/coding/scripts.html&quot;&gt;script&lt;/a&gt;.&lt;/p&gt;
</ins><span class="cx"> 
</span><span class="cx"> &lt;p&gt;The script will dump the render trees for all of the pages and diff the results against the expected correct results.  If no
</span><span class="cx"> differences are found, then the patch has passed the tests.  If any tests fail, then the patch cannot be committed until the
</span><span class="lines">@@ -60,9 +60,9 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;How to run the tests&lt;/h3&gt;
</span><span class="cx"> 
</span><del>-&lt;p&gt;Execute the &quot;run-javascriptcore-tests&quot; script:&lt;/p&gt;
-&lt;p class=&quot;code&quot;&gt;WebKitTools/Scripts/run-javascriptcore-tests&lt;/p&gt;
-&lt;p&gt;The script will run all the tests and summarize how the results differ from what is currently expected.&lt;/p&gt;
</del><ins>+&lt;p&gt;Execute the &lt;tt&gt;run-javascriptcore-tests&lt;/tt&gt; &lt;a href=&quot;/coding/scripts.html&quot;&gt;script&lt;/a&gt;.
+The script will run all the tests and summarize how the results differ
+from what is currently expected.&lt;/p&gt;
</ins><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;What just happened&lt;/h3&gt;
</span><span class="cx"> &lt;p&gt;
</span><span class="lines">@@ -80,7 +80,8 @@
</span><span class="cx"> 
</span><span class="cx"> &lt;h3&gt;What happens if I caused a regression?&lt;/h3&gt;
</span><span class="cx"> 
</span><del>-&lt;p&gt;It's not the end of the world.  Go back and fix your bug and re-run &lt;tt&gt;run-javascriptcore-tests&lt;/tt&gt;  as many times as necessary.&lt;/p&gt;
</del><ins>+&lt;p&gt;It's not the end of the world.  Go back and fix your bug and rerun
+&lt;tt&gt;run-javascriptcore-tests&lt;/tt&gt; as many times as necessary.&lt;/p&gt;
</ins><span class="cx"> 
</span><span class="cx"> &lt;?php
</span><span class="cx">     include(&quot;../footer.inc&quot;);
</span></span></pre>
</div>
</div>

</body>
</html>