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

Unified Diff: trunk/src/content/test/data/media/webrtc_test_utilities.js

Issue 466983002: Revert 288702 "Content browsertest for verifying that a black fr..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 4 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
« no previous file with comments | « trunk/src/content/test/data/media/peerconnection-call.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/content/test/data/media/webrtc_test_utilities.js
===================================================================
--- trunk/src/content/test/data/media/webrtc_test_utilities.js (revision 289082)
+++ trunk/src/content/test/data/media/webrtc_test_utilities.js (working copy)
@@ -52,14 +52,6 @@
callback);
}
-function detectBlackVideo(videoElementName, callback) {
- detectVideo(videoElementName,
- function (pixels, previous_pixels) {
- return isVideoBlack(pixels);
- },
- callback);
-}
-
function detectVideo(videoElementName, predicate, callback) {
console.log('Looking at video in element ' + videoElementName);
@@ -105,11 +97,6 @@
detectVideoStopped(videoElement, function () { eventOccured(); });
}
-function waitForBlackVideo(videoElement) {
- addExpectedEvent();
- detectBlackVideo(videoElement, function () { eventOccured(); });
-}
-
// Calculates the current frame rate and compares to |expected_frame_rate|
// |callback| is triggered with value |true| if the calculated frame rate
// is +-1 the expected or |false| if five calculations fail to match
@@ -184,16 +171,6 @@
return false;
}
-function isVideoBlack(pixels) {
- for (var i = 0; i < pixels.length; i++) {
- // |pixels| is in RGBA. Ignore the alpha channel.
- if (pixels[i] != 0 && (i + 1) % 4 != 0) {
- return false;
- }
- }
- return true;
-}
-
// This function matches |left| and |right| and fails the test if the
// values don't match using normal javascript equality (i.e. the hard
// types of the operands aren't checked).
« no previous file with comments | « trunk/src/content/test/data/media/peerconnection-call.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698