feat(large-video): background blur through canvas and feature flag
To reduce the amount of motion that has to be blurred, use a canvas to essentially set the FPS of the video background. This canvas component is behind a temporary feature flag, as well as being able to disable the blur, so it can be played around with on deployed environments.
This commit is contained in:
parent
0831c16d7e
commit
f015f4edc3
|
@ -163,7 +163,17 @@ var interfaceConfig = {
|
||||||
/**
|
/**
|
||||||
* Specify mobile app scheme for opening the app from the mobile browser.
|
* Specify mobile app scheme for opening the app from the mobile browser.
|
||||||
*/
|
*/
|
||||||
// APP_SCHEME: 'org.jitsi.meet'
|
// APP_SCHEME: 'org.jitsi.meet',
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Temporary feature flag to debug performance with the large video
|
||||||
|
* background blur. On initial implementation, blur was always enabled so a
|
||||||
|
* falsy value here will be used to keep blur enabled, as will the value
|
||||||
|
* "video", and will render the blur over a video element. The value
|
||||||
|
* "canvas" will display the blur over a canvas element, while the value
|
||||||
|
* "off" will prevent the background from rendering.
|
||||||
|
*/
|
||||||
|
// _BACKGROUND_BLUR: undefined
|
||||||
};
|
};
|
||||||
|
|
||||||
/* eslint-enable no-unused-vars, no-var, max-len */
|
/* eslint-enable no-unused-vars, no-var, max-len */
|
||||||
|
|
|
@ -7,7 +7,8 @@ import ReactDOM from 'react-dom';
|
||||||
import { browser } from '../../../react/features/base/lib-jitsi-meet';
|
import { browser } from '../../../react/features/base/lib-jitsi-meet';
|
||||||
import {
|
import {
|
||||||
ORIENTATION,
|
ORIENTATION,
|
||||||
LargeVideoBackground
|
LargeVideoBackground,
|
||||||
|
LargeVideoBackgroundCanvas
|
||||||
} from '../../../react/features/large-video';
|
} from '../../../react/features/large-video';
|
||||||
/* eslint-enable no-unused-vars */
|
/* eslint-enable no-unused-vars */
|
||||||
|
|
||||||
|
@ -689,14 +690,20 @@ export class VideoContainer extends LargeContainer {
|
||||||
_updateBackground() {
|
_updateBackground() {
|
||||||
// Do not the background display on browsers that might experience
|
// Do not the background display on browsers that might experience
|
||||||
// performance issues from the presence of the background.
|
// performance issues from the presence of the background.
|
||||||
if (browser.isFirefox()
|
if (interfaceConfig._BACKGROUND_BLUR === 'off'
|
||||||
|
|| browser.isFirefox()
|
||||||
|| browser.isSafariWithWebrtc()
|
|| browser.isSafariWithWebrtc()
|
||||||
|| browser.isTemasysPluginUsed()) {
|
|| browser.isTemasysPluginUsed()) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// eslint-disable-next-line no-unused-vars
|
||||||
|
const Background = interfaceConfig._BACKGROUND_BLUR === 'canvas'
|
||||||
|
? LargeVideoBackgroundCanvas
|
||||||
|
: LargeVideoBackground;
|
||||||
|
|
||||||
ReactDOM.render(
|
ReactDOM.render(
|
||||||
<LargeVideoBackground
|
<Background
|
||||||
hidden = { this._hideBackground }
|
hidden = { this._hideBackground }
|
||||||
mirror = {
|
mirror = {
|
||||||
this.stream
|
this.stream
|
||||||
|
@ -708,6 +715,7 @@ export class VideoContainer extends LargeContainer {
|
||||||
= { this.$video.hasClass(LOCAL_PROBLEM_FILTER_CLASS) }
|
= { this.$video.hasClass(LOCAL_PROBLEM_FILTER_CLASS) }
|
||||||
showRemoteProblemFilter
|
showRemoteProblemFilter
|
||||||
= { this.$video.hasClass(REMOTE_PROBLEM_FILTER_CLASS) }
|
= { this.$video.hasClass(REMOTE_PROBLEM_FILTER_CLASS) }
|
||||||
|
videoElement = { this.$video && this.$video[0] }
|
||||||
videoTrack = { this.stream } />,
|
videoTrack = { this.stream } />,
|
||||||
document.getElementById('largeVideoBackgroundContainer')
|
document.getElementById('largeVideoBackgroundContainer')
|
||||||
);
|
);
|
||||||
|
|
|
@ -23,7 +23,7 @@ export const ORIENTATION = {
|
||||||
* @private
|
* @private
|
||||||
* @type {Object}
|
* @type {Object}
|
||||||
*/
|
*/
|
||||||
const ORIENTATION_TO_CLASS = {
|
export const ORIENTATION_TO_CLASS = {
|
||||||
[ORIENTATION.LANDSCAPE]: 'fit-full-width',
|
[ORIENTATION.LANDSCAPE]: 'fit-full-width',
|
||||||
[ORIENTATION.PORTRAIT]: 'fit-full-height'
|
[ORIENTATION.PORTRAIT]: 'fit-full-height'
|
||||||
};
|
};
|
||||||
|
|
|
@ -0,0 +1,229 @@
|
||||||
|
// @flow
|
||||||
|
|
||||||
|
import React, { Component } from 'react';
|
||||||
|
|
||||||
|
import { ORIENTATION, ORIENTATION_TO_CLASS } from './LargeVideoBackground';
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The type of the React {@code Component} props of
|
||||||
|
* {@link LargeVideoBackgroundCanvas}.
|
||||||
|
*/
|
||||||
|
type Props = {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Additional CSS class names to add to the root of the component.
|
||||||
|
*/
|
||||||
|
className: String,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether or not the background should have its visibility hidden.
|
||||||
|
*/
|
||||||
|
hidden: boolean,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether or not the video should display flipped horizontally, so left
|
||||||
|
* becomes right and right becomes left.
|
||||||
|
*/
|
||||||
|
mirror: boolean,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether the component should ensure full width of the video is displayed
|
||||||
|
* (landscape) or full height (portrait).
|
||||||
|
*/
|
||||||
|
orientationFit: String,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether or not to display a filter on the video to visually indicate a
|
||||||
|
* problem with the video being displayed.
|
||||||
|
*/
|
||||||
|
showLocalProblemFilter: boolean,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether or not to display a filter on the video to visually indicate a
|
||||||
|
* problem with the video being displayed.
|
||||||
|
*/
|
||||||
|
showRemoteProblemFilter: boolean,
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The video stream to display.
|
||||||
|
*/
|
||||||
|
videoElement: Object
|
||||||
|
};
|
||||||
|
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Implements a React Component which shows a video element intended to be used
|
||||||
|
* as a background to fill the empty space of container with another video.
|
||||||
|
*
|
||||||
|
* @extends Component
|
||||||
|
*/
|
||||||
|
export class LargeVideoBackgroundCanvas extends Component<Props> {
|
||||||
|
_canvasEl: Object;
|
||||||
|
|
||||||
|
_updateCanvasInterval: number;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Initializes new {@code LargeVideoBackgroundCanvas} instance.
|
||||||
|
*
|
||||||
|
* @param {*} props - The read-only properties with which the new instance
|
||||||
|
* is to be initialized.
|
||||||
|
*/
|
||||||
|
constructor(props: Props) {
|
||||||
|
super(props);
|
||||||
|
|
||||||
|
// Bind event handlers so they are only bound once per instance.
|
||||||
|
this._setCanvasEl = this._setCanvasEl.bind(this);
|
||||||
|
this._updateCanvas = this._updateCanvas.bind(this);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* If the canvas is not hidden, sets the initial interval to update the
|
||||||
|
* image displayed in the canvas.
|
||||||
|
*
|
||||||
|
* @inheritdoc
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
componentDidMount() {
|
||||||
|
if (this.props.videoElement && !this.props.hidden) {
|
||||||
|
this._updateCanvas();
|
||||||
|
this._setUpdateCanvasInterval();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Starts or stops the interval to update the image displayed in the canvas.
|
||||||
|
*
|
||||||
|
* @inheritdoc
|
||||||
|
* @param {Object} nextProps - The read-only React {@code Component} props
|
||||||
|
* with which the new instance is to be initialized.
|
||||||
|
*/
|
||||||
|
componentWillReceiveProps(nextProps: Props) {
|
||||||
|
if (this.props.hidden && !nextProps.hidden) {
|
||||||
|
this._clearCanvas();
|
||||||
|
this._setUpdateCanvasInterval();
|
||||||
|
}
|
||||||
|
|
||||||
|
if ((!this.props.hidden && nextProps.hidden)
|
||||||
|
|| !nextProps.videoElement) {
|
||||||
|
this._clearCanvas();
|
||||||
|
this._clearUpdateCanvasInterval();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Clears the interval for updating the image displayed in the canvas.
|
||||||
|
*
|
||||||
|
* @inheritdoc
|
||||||
|
*/
|
||||||
|
componentWillUnmount() {
|
||||||
|
this._clearUpdateCanvasInterval();
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Implements React's {@link Component#render()}.
|
||||||
|
*
|
||||||
|
* @inheritdoc
|
||||||
|
* @returns {ReactElement}
|
||||||
|
*/
|
||||||
|
render() {
|
||||||
|
const {
|
||||||
|
hidden,
|
||||||
|
mirror,
|
||||||
|
orientationFit,
|
||||||
|
showLocalProblemFilter,
|
||||||
|
showRemoteProblemFilter
|
||||||
|
} = this.props;
|
||||||
|
const orientationClass = orientationFit
|
||||||
|
? ORIENTATION_TO_CLASS[orientationFit] : '';
|
||||||
|
const classNames = `large-video-background ${mirror ? 'flip-x' : ''} ${
|
||||||
|
hidden ? 'invisible' : ''} ${orientationClass} ${
|
||||||
|
showLocalProblemFilter ? 'videoProblemFilter' : ''} ${
|
||||||
|
showRemoteProblemFilter ? 'remoteVideoProblemFilter' : ''}`;
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div className = { classNames }>
|
||||||
|
<canvas
|
||||||
|
id = 'largeVideoBackground'
|
||||||
|
ref = { this._setCanvasEl } />
|
||||||
|
</div>
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Removes any image displayed on the canvas.
|
||||||
|
*
|
||||||
|
* @private
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
_clearCanvas() {
|
||||||
|
const cavnasContext = this._canvasEl.getContext('2d');
|
||||||
|
|
||||||
|
cavnasContext.clearRect(
|
||||||
|
0, 0, this._canvasEl.width, this._canvasEl.height);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Clears the interval for updating the image displayed in the canvas.
|
||||||
|
*
|
||||||
|
* @private
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
_clearUpdateCanvasInterval() {
|
||||||
|
clearInterval(this._updateCanvasInterval);
|
||||||
|
}
|
||||||
|
|
||||||
|
_setCanvasEl: () => void;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Sets the instance variable for the component's canvas element so it can
|
||||||
|
* be accessed directly for drawing on.
|
||||||
|
*
|
||||||
|
* @param {Object} element - The DOM element for the component's canvas.
|
||||||
|
* @private
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
_setCanvasEl(element) {
|
||||||
|
this._canvasEl = element;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Starts the interval for updating the image displayed in the canvas.
|
||||||
|
*
|
||||||
|
* @private
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
_setUpdateCanvasInterval() {
|
||||||
|
this._clearUpdateCanvasInterval();
|
||||||
|
this._updateCanvasInterval = setInterval(this._updateCanvas, 200);
|
||||||
|
}
|
||||||
|
|
||||||
|
_updateCanvas: () => void;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Draws the current frame of the passed in video element onto the canvas.
|
||||||
|
*
|
||||||
|
* @private
|
||||||
|
* @returns {void}
|
||||||
|
*/
|
||||||
|
_updateCanvas() {
|
||||||
|
const { videoElement } = this.props;
|
||||||
|
const { videoWidth, videoHeight } = videoElement;
|
||||||
|
const {
|
||||||
|
height: canvasHeight,
|
||||||
|
width: canvasWidth
|
||||||
|
} = this._canvasEl;
|
||||||
|
const cavnasContext = this._canvasEl.getContext('2d');
|
||||||
|
|
||||||
|
if (this.props.orientationFit === ORIENTATION.LANDSCAPE) {
|
||||||
|
const heightScaledToFit = (canvasWidth / videoWidth) * videoHeight;
|
||||||
|
|
||||||
|
cavnasContext.drawImage(
|
||||||
|
videoElement, 0, 0, canvasWidth, heightScaledToFit);
|
||||||
|
} else {
|
||||||
|
const widthScaledToFit = (canvasHeight / videoHeight) * videoWidth;
|
||||||
|
|
||||||
|
cavnasContext.drawImage(
|
||||||
|
videoElement, 0, 0, widthScaledToFit, canvasHeight);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
|
@ -1,2 +1,3 @@
|
||||||
export { default as LargeVideo } from './LargeVideo';
|
export { default as LargeVideo } from './LargeVideo';
|
||||||
export * from './LargeVideoBackground';
|
export * from './LargeVideoBackground';
|
||||||
|
export * from './LargeVideoBackgroundCanvas';
|
||||||
|
|
Loading…
Reference in New Issue