Test and css changes for dom_distiller inclusion of video.

Test and css changes for dom_distiller inclusion of video.

Adds tests for code in https://codereview.chromium.org/322553005/
Also, update stylesheet to restrict max video size.
BUG=376107
TBR=blundell

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=276842

[cjhopman, 2014-06-06 23:11:06]

https://codereview.chromium.org/318723004/diff/1/components/test/data/dom_dis...
File components/test/data/dom_distiller/simple_article.html (right):

https://codereview.chromium.org/318723004/diff/1/components/test/data/dom_dis...
components/test/data/dom_distiller/simple_article.html:14: <video width="500"
height="400" controls>
I'd rather create a copy of this simple article with the video stuff added so
that this simple article stays simple as we add tests for more stuff.

[Yaron, 2014-06-12 00:46:22]

https://codereview.chromium.org/318723004/diff/1/components/test/data/dom_dis...
File components/test/data/dom_distiller/simple_article.html (right):

https://codereview.chromium.org/318723004/diff/1/components/test/data/dom_dis...
components/test/data/dom_distiller/simple_article.html:14: <video width="500"
height="400" controls>
On 2014/06/06 23:11:05, cjhopman wrote:
> I'd rather create a copy of this simple article with the video stuff added so
> that this simple article stays simple as we add tests for more stuff.

Done.

[cjhopman, 2014-06-12 17:43:11]

lgtm

[Yaron, 2014-06-12 17:45:07]

The CQ bit was checked by yfriedman@chromium.org

[commit-bot: I haz the power, 2014-06-12 17:47:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yfriedman@chromium.org/318723004/20001

[commit-bot: I haz the power, 2014-06-12 22:07:10]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-06-12 22:13:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-12 22:13:16]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[Yaron, 2014-06-12 22:53:33]

tbr'ing blundell for test data file

[Yaron, 2014-06-12 22:53:37]

The CQ bit was checked by yfriedman@chromium.org

[commit-bot: I haz the power, 2014-06-12 22:58:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yfriedman@chromium.org/318723004/20001

[commit-bot: I haz the power, 2014-06-12 23:05:41]

Change committed as 276842

[blundell, 2014-06-13 07:31:37]

rubberstamp lgtm