Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1347)

Unified Diff: media/tools/layout_tests/testname/media.csv

Issue 10261034: Per issue 125757, added more test cases to be monitored in the layout test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Enable recursion and reduce the number of entries in test case CSV file. Created 8 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/tools/layout_tests/testname/media.csv
diff --git a/media/tools/layout_tests/testname/media.csv b/media/tools/layout_tests/testname/media.csv
index cf3b561865cfe2a4439a8f51258f738507a5b0ae..f0773fa24fc527079fe7ed3389fbb6e9d20ad652 100644
--- a/media/tools/layout_tests/testname/media.csv
+++ b/media/tools/layout_tests/testname/media.csv
@@ -1,25 +1,26 @@
-http/tests/media/\S+.html,
-media/\S+.html,
-media/track/\S+.html,
-http/tests/security/\S+video\S+.html,
-fast/layers/video-layer.html,
-compositing/geometry/clipped-video-controller.html,
-compositing/geometry/video-fixed-scrolling.html,
-compositing/geometry/video-opacity-overlay.html,
-compositing/overflow/overflow-compositing-descendant.html,
-compositing/overflow/scroll-ancestor-update.html,
-compositing/reflections/load-video-in-reflection.html,
-compositing/self-painting-layers.html,
-compositing/self-painting-layers2.html,
-compositing/video/video-background-color.html,
-compositing/video/video-poster.html,
-fast/canvas/webgl/context-lost.html,
-fast/canvas/webgl/tex-image-and-sub-image-2d-with-video.html,
-fast/dom/beforeload/remove-video-in-beforeload-listener.html,
-fast/dom/beforeload/video-before-load.html,
-fast/dom/shadow/frameless-media-element-crash.html,
-http/tests/appcache/video.html,
-http/tests/canvas/webgl/origin-clean-conformance.html,
-http/tests/security/contentSecurityPolicy/media-src-allowed.html,
-http/tests/security/contentSecurityPolicy/media-src-blocked.html,
-platform/chromium/media/\S+.html,
+"http/tests/media/"
scherkus (not reviewing) 2012/05/02 00:50:55 why are " needed now? it causes for quite a messy
imasaki1 2012/05/03 23:31:14 Done.
+"media/"
+"http/tests/security/\S+video\S+.html"
ddorwin 2012/05/01 22:32:37 Do we need the S+.html?
imasaki1 2012/05/03 23:31:14 Yes. Currently, we cannot mix regular expression a
+"fast/layers/video-layer.html"
+"compositing/geometry/clipped-video-controller.html"
+"compositing/geometry/video-fixed-scrolling.html"
+"compositing/geometry/video-opacity-overlay.html"
+"compositing/overflow/overflow-compositing-descendant.html"
+"compositing/overflow/scroll-ancestor-update.html"
+"compositing/reflections/load-video-in-reflection.html"
+"compositing/self-painting-layers.html"
+"compositing/self-painting-layers2.html"
+"compositing/video/video-background-color.html"
+"compositing/video/video-poster.html"
+"fast/canvas/webgl/context-lost.html"
+"fast/canvas/webgl/tex-image-and-sub-image-2d-with-video.html"
+"fast/dom/beforeload/remove-video-in-beforeload-listener.html"
+"fast/dom/beforeload/video-before-load.html"
+"fast/dom/shadow/frameless-media-element-crash.html"
+"http/tests/appcache/video.html"
+"http/tests/canvas/webgl/origin-clean-conformance.html"
+"http/tests/security/contentSecurityPolicy/media-src-allowed.html"
+"http/tests/security/contentSecurityPolicy/media-src-blocked.html"
+"platform/chromium/media/\S+.html"
ddorwin 2012/05/01 22:32:37 Do we need S+.html?
imasaki1 2012/05/03 23:31:14 Removed.
+"fast/events/constructors/media-key-event-constructor.html"
ddorwin 2012/05/01 22:32:37 Please alphabetize these and some of those above.
imasaki1 2012/05/03 23:31:14 Done.
+"fast/events/constructors/track-event-constructor.html"

Powered by Google App Engine
This is Rietveld 408576698