Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-14 13:55:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1447950 - use a switch statement in place of chained if-else in AsyncTabSwitcher.handleEvent. r=dao

Browse Source
This commit is contained in:
Daniel Almeida 2018-03-23 19:00:01 +01:00
parent 9e35535e7d
commit 673edf8ee3
1 changed files with 23 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
browser/modules/AsyncTabSwitcher.jsm
View File

@ -959,22 +959,29 @@ class AsyncTabSwitcher {
this._processing = true;
this.preActions();
if (event.type == "MozLayerTreeReady") {
this.onLayersReady(event.originalTarget);
}
if (event.type == "MozAfterPaint") {
this.onPaint();
} else if (event.type == "MozLayerTreeCleared") {
this.onLayersCleared(event.originalTarget);
} else if (event.type == "TabRemotenessChange") {
this.onRemotenessChange(event.target);
} else if (event.type == "sizemodechange" ||
event.type == "occlusionstatechange") {
this.onSizeModeOrOcclusionStateChange();
} else if (event.type == "SwapDocShells") {
this.onSwapDocShells(event.originalTarget, event.detail);
} else if (event.type == "EndSwapDocShells") {
this.onEndSwapDocShells(event.originalTarget, event.detail);
switch (event.type) {
case "MozLayerTreeReady":
this.onLayersReady(event.originalTarget);
break;
case "MozAfterPaint":
this.onPaint();
break;
case "MozLayerTreeCleared":
this.onLayersCleared(event.originalTarget);
break;
case "TabRemotenessChange":
this.onRemotenessChange(event.target);
break;
case "sizemodechange":
case "occlusionstatechange":
this.onSizeModeOrOcclusionStateChange();
break;
case "SwapDocShells":
this.onSwapDocShells(event.originalTarget, event.detail);
break;
case "EndSwapDocShells":
this.onEndSwapDocShells(event.originalTarget, event.detail);
break;
}
this.postActions();