Revert of Remove <title> in writing_layout_tests documentation (patchset #2 id:20001 of https://codereview.chromium.org/2617083002/ )
Reason for revert:
pwnall@ pointed out that <title> is used in next_default_test_name()
Original issue's description:
> Remove <title> in writing_layout_tests documentation as it
> doesn't appear in test results (thanks to Alan for pointing it out).
>
> Instead we should focus on writing good test descriptions.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2617083002
> Cr-Commit-Position: refs/heads/master@{#441876}
> Committed: https://chromium.googlesource.com/chromium/src/+/f0c6c6901e6addca3e3025c07f999333a9c53901
[email protected],[email protected],[email protected],[email protected]
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=
Review-Url: https://codereview.chromium.org/2624533003
Cr-Commit-Position: refs/heads/master@{#442446}
diff --git a/docs/testing/writing_layout_tests.md b/docs/testing/writing_layout_tests.md
index 13d563b..f2767c2 100644
--- a/docs/testing/writing_layout_tests.md
+++ b/docs/testing/writing_layout_tests.md
@@ -338,6 +338,7 @@
```html
<!doctype html>
<meta charset="utf-8">
+<title>JavaScript: the true literal</title>
<link rel="help" href="https://tc39.github.io/ecma262/#sec-boolean-literals">
<meta name="assert" value="The true literal is equal to itself and immutable">
<script src="/resources/testharness.js"></script>
@@ -374,7 +375,9 @@
Some points that are not immediately obvious from the example:
-* The `<meta name="assert">` describes the purpose of the entire file.
+* The `<meta name="assert">` describes the purpose of the entire file, and
+ is not redundant to `<title>`. Don't add a `<meta name="assert">` when the
+ information in the `<title>` is sufficient.
* When calling an `assert_` function that compares two values, the first
argument is the actual value (produced by the functionality being tested), and
the second argument is the expected value (known good, golden). The order
@@ -390,6 +393,9 @@
redundant.
* Do not start test case descriptions with redundant terms like "Testing"
or "Test for".
+ * Test files with a single test case should omit the test case description.
+ The file's `<title>` should be sufficient to describe the scenario being
+ tested.
* Asynchronous tests have a few subtleties.
* The `async_test` wrapper calls its function with a test case argument that
is used to signal when the test case is done, and to connect assertion
@@ -510,6 +516,7 @@
```html
<!doctype html>
<meta charset="utf-8">
+<title>DOM: Event.isTrusted for UI events</title>
<link rel="help" href="https://dom.spec.whatwg.org/#dom-event-istrusted">
<link rel="help" href="https://dom.spec.whatwg.org/#constructing-events">
<meta name="assert"