Minor fixes for layout test writing documentation.

Minor fixes for layout test writing documentation.

The main reason for this CL is to change the recommended path for
accessing testharness.js from HTTP tests. The file can be accessed using
both the old path (/js-test-resources/testharness.js) and the new path
(/resources/testharness.js). However, our testharnessresult.js uses the
CSS selector script[src*="/resources/testharness"] to identify
JavaScript tests, causing tests that use the old path to fail without a
text baseline.

The CL also adds an "under construction" unicode character to the
guideline for adding a <meta charset> to every test, as it is still
under discussion.

BUG=

Committed: https://crrev.com/e781948d69c56ab6bfc89f318a1e107cb81a566f
Cr-Commit-Position: refs/heads/master@{#439266}

[pwnall, 2016-12-16 07:22:43]

pwnall@chromium.org changed reviewers:
+ jsbell@chromium.org

[pwnall, 2016-12-16 07:22:44]

PTAL?

This is the line that I'm referencing:
https://cs.chromium.org/chromium/src/third_party/WebKit/LayoutTests/resources...

This issue caused me to waste a bit of time debugging my test and the testing
infastructure, so I'd like to fix it before other poor souls bump into it.

[jsbell, 2016-12-17 00:03:21]

lgtm

[pwnall, 2016-12-17 00:45:05]

The CQ bit was checked by pwnall@chromium.org

[pwnall, 2016-12-17 00:45:17]

On 2016/12/17 00:03:21, jsbell wrote:
> lgtm

Thank you for the quick turnaround!

[commit-bot: I haz the power, 2016-12-17 00:45:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-17 00:58:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481935505706670, "parent_rev":
"5e3c8166aeaaf0a37e431495f21a197dfc97c281", "commit_rev":
"e43c8c2a54c5c6c2650794400b111a7d2bab9ac4"}

[commit-bot: I haz the power, 2016-12-17 00:59:19]

Description was changed from

==========
Minor fixes for layout test writing documentation.

The main reason for this CL is to change the recommended path for
accessing testharness.js from HTTP tests. The file can be accessed using
both the old path (/js-test-resources/testharness.js) and the new path
(/resources/testharness.js). However, our testharnessresult.js uses the
CSS selector script[src*="/resources/testharness"] to identify
JavaScript tests, causing tests that use the old path to fail without a
text baseline.

The CL also adds an "under construction" unicode character to the
guideline for adding a <meta charset> to every test, as it is still
under discussion.

BUG=
==========

to

==========
Minor fixes for layout test writing documentation.

The main reason for this CL is to change the recommended path for
accessing testharness.js from HTTP tests. The file can be accessed using
both the old path (/js-test-resources/testharness.js) and the new path
(/resources/testharness.js). However, our testharnessresult.js uses the
CSS selector script[src*="/resources/testharness"] to identify
JavaScript tests, causing tests that use the old path to fail without a
text baseline.

The CL also adds an "under construction" unicode character to the
guideline for adding a <meta charset> to every test, as it is still
under discussion.

BUG=

Review-Url: https://codereview.chromium.org/2579953002
==========

[commit-bot: I haz the power, 2016-12-17 00:59:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-17 01:02:56]

Description was changed from

==========
Minor fixes for layout test writing documentation.

The main reason for this CL is to change the recommended path for
accessing testharness.js from HTTP tests. The file can be accessed using
both the old path (/js-test-resources/testharness.js) and the new path
(/resources/testharness.js). However, our testharnessresult.js uses the
CSS selector script[src*="/resources/testharness"] to identify
JavaScript tests, causing tests that use the old path to fail without a
text baseline.

The CL also adds an "under construction" unicode character to the
guideline for adding a <meta charset> to every test, as it is still
under discussion.

BUG=

Review-Url: https://codereview.chromium.org/2579953002
==========

to

==========
Minor fixes for layout test writing documentation.

The main reason for this CL is to change the recommended path for
accessing testharness.js from HTTP tests. The file can be accessed using
both the old path (/js-test-resources/testharness.js) and the new path
(/resources/testharness.js). However, our testharnessresult.js uses the
CSS selector script[src*="/resources/testharness"] to identify
JavaScript tests, causing tests that use the old path to fail without a
text baseline.

The CL also adds an "under construction" unicode character to the
guideline for adding a <meta charset> to every test, as it is still
under discussion.

BUG=

Committed: https://crrev.com/e781948d69c56ab6bfc89f318a1e107cb81a566f
Cr-Commit-Position: refs/heads/master@{#439266}
==========

[commit-bot: I haz the power, 2016-12-17 01:02:58]

Patchset 1 (id:??) landed as
https://crrev.com/e781948d69c56ab6bfc89f318a1e107cb81a566f
Cr-Commit-Position: refs/heads/master@{#439266}