Apply suggestions from code review
Co-authored-by: J. Ryan Stinnett <jryans@gmail.com>pull/21833/head
parent
0167c3cefb
commit
0eb0d242d5
|
@ -179,7 +179,8 @@ export default class InteractiveAuthComponent extends React.Component {
|
||||||
}, () => {
|
}, () => {
|
||||||
if (oldStage !== stageType) {
|
if (oldStage !== stageType) {
|
||||||
this._setFocus();
|
this._setFocus();
|
||||||
} else if (!stageState.error && this._stageComponent.current &&
|
} else if (
|
||||||
|
!stageState.error && this._stageComponent.current &&
|
||||||
this._stageComponent.current.attemptFailed
|
this._stageComponent.current.attemptFailed
|
||||||
) {
|
) {
|
||||||
this._stageComponent.current.attemptFailed();
|
this._stageComponent.current.attemptFailed();
|
||||||
|
|
|
@ -626,6 +626,7 @@ export class SSOAuthEntry extends React.Component {
|
||||||
window.removeEventListener("message", this._onReceiveMessage);
|
window.removeEventListener("message", this._onReceiveMessage);
|
||||||
if (this._popupWindow) {
|
if (this._popupWindow) {
|
||||||
this._popupWindow.close();
|
this._popupWindow.close();
|
||||||
|
this._popupWindow = null;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -639,6 +640,7 @@ export class SSOAuthEntry extends React.Component {
|
||||||
if (event.data === "authDone" && event.origin === this.props.matrixClient.getHomeserverUrl()) {
|
if (event.data === "authDone" && event.origin === this.props.matrixClient.getHomeserverUrl()) {
|
||||||
if (this._popupWindow) {
|
if (this._popupWindow) {
|
||||||
this._popupWindow.close();
|
this._popupWindow.close();
|
||||||
|
this._popupWindow = null;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
Loading…
Reference in New Issue