WebUI: add functional test for text_defaults.css

WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1

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

[Dan Beam, 2016-04-07 02:04:42]

+dschuyler@ for chrome/test/data/webui/
+sky@ for everything else

[sky, 2016-04-07 19:12:22]

LGTM

[dschuyler, 2016-04-07 20:54:31]

LGTM with nit

https://codereview.chromium.org/1863963004/diff/1/chrome/test/data/webui/text...
File chrome/test/data/webui/text_defaults_browsertest.js (right):

https://codereview.chromium.org/1863963004/diff/1/chrome/test/data/webui/text...
chrome/test/data/webui/text_defaults_browsertest.js:27: * @return {string} The
HTML decoded text of |maybeHtml|.
nit: The param name is called html and maybeHtml in
different places.  Either one is good imo.

[Dan Beam, 2016-04-07 21:03:41]

Description was changed from

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
==========

to

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1
==========

[Dan Beam, 2016-04-07 21:03:46]

https://codereview.chromium.org/1863963004/diff/1/chrome/test/data/webui/text...
File chrome/test/data/webui/text_defaults_browsertest.js (right):

https://codereview.chromium.org/1863963004/diff/1/chrome/test/data/webui/text...
chrome/test/data/webui/text_defaults_browsertest.js:27: * @return {string} The
HTML decoded text of |maybeHtml|.
On 2016/04/07 20:54:31, dschuyler wrote:
> nit: The param name is called html and maybeHtml in
> different places.  Either one is good imo.

Done.

[Dan Beam, 2016-04-07 21:03:56]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-04-07 21:06:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1863963004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1863963004/20001

[commit-bot: I haz the power, 2016-04-07 21:39:55]

Description was changed from

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1
==========

to

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1
==========

[commit-bot: I haz the power, 2016-04-07 21:39:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-07 21:41:15]

Description was changed from

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1
==========

to

==========
WebUI: add functional test for text_defaults.css

R=dschuyler@chromium.org,sky@chromium.org
BUG=601051
NOTRY=true  # see patchset 1

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

[commit-bot: I haz the power, 2016-04-07 21:41:16]

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