GitLab

Remove TestExtensionDir::WriteManifestWithSingleQuotes(). This was a
convenience function introduced before we had support for string
literals in the language.  Now that we have string literals, those are
preferred:
- They are more consistent
- They preserve newlines
- They don't clutter each line with double-quotes
- It is mildly faster, since we don't have to do string replacement.

Remove the old WriteManifestWithSingleQuotes() function, and update
all call sites.

This CL should have no behavior change.

Bug: None
Change-Id: I2e65c514530316d2d25fd75cb33ca2ffcb1a244e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1975073
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Nathan Parker <nparker@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#727621}