Title: [111414] trunk/Tools
Revision
111414
Author
[email protected]
Date
2012-03-20 11:12:49 -0700 (Tue, 20 Mar 2012)

Log Message

2012-02-24  Darin Fisher  <[email protected]>

        Add more Chromium WebKit API reviewers
        https://bugs.webkit.org/show_bug.cgi?id=81576

        Reviewed by Adam Barth and David Levin.

        Simplify watchlist to lump all WebKit API into one.

        * Scripts/webkitpy/common/config/watchlist:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (111413 => 111414)


--- trunk/Tools/ChangeLog	2012-03-20 18:08:15 UTC (rev 111413)
+++ trunk/Tools/ChangeLog	2012-03-20 18:12:49 UTC (rev 111414)
@@ -1,3 +1,14 @@
+2012-02-24  Darin Fisher  <[email protected]>
+
+        Add more Chromium WebKit API reviewers
+        https://bugs.webkit.org/show_bug.cgi?id=81576
+
+        Reviewed by Adam Barth and David Levin.
+
+        Simplify watchlist to lump all WebKit API into one.
+
+        * Scripts/webkitpy/common/config/watchlist:
+
 2012-03-20  Gyuyoung Kim  <[email protected]>
 
         Convert hasSpellingMarker to use Internals interface.

Modified: trunk/Tools/Scripts/webkitpy/common/config/watchlist (111413 => 111414)


--- trunk/Tools/Scripts/webkitpy/common/config/watchlist	2012-03-20 18:08:15 UTC (rev 111413)
+++ trunk/Tools/Scripts/webkitpy/common/config/watchlist	2012-03-20 18:12:49 UTC (rev 111414)
@@ -20,10 +20,8 @@
         },
         "ChromiumPublicApi": {
             "filename": r"Source/WebKit/chromium/public/"
+                        r"|Source/Platform/chromium/public/",
         },
-        "ChromiumPlatformApi": {
-            "filename": r"Source/Platform/chromium/public/"
-        },
         "AppleMacPublicApi": {
             "filename": r"Source/WebCore/bindings/objc/PublicDOMInterfaces.h"
         },
@@ -148,8 +146,7 @@
         # two different accounts as far as bugzilla is concerned.
         "ChromiumDumpRenderTree": [ "[email protected]", ],
         "ChromiumGraphics": [ "[email protected]", "[email protected]" ],
-        "ChromiumPublicApi": [ "[email protected]", "[email protected]" ],
-        "ChromiumPlatformApi": [ "[email protected]", "[email protected]", "[email protected]" ],
+        "ChromiumPublicApi": [ "[email protected]", "[email protected]", "[email protected]", "[email protected]", "[email protected]" ],
         "AppleMacPublicApi": [ "[email protected]" ],
         "Forms": [ "[email protected]", ],
         "GStreamerGraphics": [ "[email protected]", "[email protected]", "[email protected]" ],
@@ -174,10 +171,10 @@
         "ScrollingCoordinator": [ "[email protected]", "[email protected]", "[email protected]" ],
     },
     "MESSAGE_RULES": {
-        "ChromiumPublicApi": [ "Please wait for approval from [email protected] before submitting "
-                               "because this patch contains changes to the Chromium public API.", ],
-        "ChromiumPlatformApi": [ "Please wait for approval from [email protected], [email protected] or [email protected] before submitting "
-                                 "because this patch contains changes to the Chromium platform API.", ],
+        "ChromiumPublicApi": [ "Please wait for approval from [email protected], [email protected], "
+                               "[email protected], [email protected] or [email protected] before "
+                               "submitting, as this patch contains changes to the Chromium public API. "
+                               "See also https://trac.webkit.org/wiki/ChromiumWebKitAPI." ],
         "AppleMacPublicApi": [ "Please wait for approval from [email protected] (or another member "
                                "of the Apple Safari Team) before submitting "
                                "because this patch contains changes to the Apple Mac "
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to