Changeset 121120 in webkit

Timestamp:

Jun 24, 2012, 11:44:02 AM (14 years ago)

Author:

abarth@webkit.org

Message:

The comment in test_expectations_android.txt is unclear as to the purpose of this file.
​https://bugs.webkit.org/show_bug.cgi?id=89832

Reviewed by Ojan Vafai.

This patch attempts to clarify the purpose of this file and what will
happen to it in the future.

• platform/chromium/test_expectations_android.txt:

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• platform/chromium/test_expectations_android.txt (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2012-06-24  Adam Barth  <abarth@webkit.org>
+
+        The comment in test_expectations_android.txt is unclear as to the purpose of this file.
+        https://bugs.webkit.org/show_bug.cgi?id=89832
+
+        Reviewed by Ojan Vafai.
+
+        This patch attempts to clarify the purpose of this file and what will
+        happen to it in the future.
+
+        * platform/chromium/test_expectations_android.txt:
+
 2012-06-24  Christophe Dumez  <christophe.dumez@intel.com>
 

trunk/LayoutTests/platform/chromium/test_expectations_android.txt

@@ -1 +1 @@
 // This file should remain empty. It is used by the chromium-android port in
-// its downstream repository to manage test expecations downstream. Rather than
-// add expectations to this file upstream, please add them to the normal
-// TestExpectations file with the ANDROID keyword.
+// its downstream repository to manage test expecations downstream.  When the
+// chromium-android port runs tests on build.webkit.org, test expectations
+// should be put in the Chromium TestExpectations file as normal rather than
+// here.  We will delete this file once we finish upstreaming the
+// chromium-android port.