Skip to content
This repository has been archived by the owner on Aug 20, 2020. It is now read-only.

Commit

Permalink
WaylandWidndow: Remove unecessary switch/case block
Browse files Browse the repository at this point in the history
The checks for shell surface types are already done in UpdateShellSurface
  • Loading branch information
Eduardo Lima (Etrunko) authored and kalyankondapally committed Nov 27, 2013
1 parent 66c6c86 commit bf1dc26
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions wayland/window.cc
Expand Up @@ -45,21 +45,7 @@ void WaylandWindow::SetShellType(ShellType type)
shell_surface_ = new WaylandShellSurface(this);

type_ = type;
switch (type_) {
case TOPLEVEL:
shell_surface_->UpdateShellSurface(TOPLEVEL);
break;
case MENU:
shell_surface_->UpdateShellSurface(MENU);
break;
case FULLSCREEN:
case TRANSIENT:
case CUSTOM:
NOTREACHED() << "UnSupported Shell Type.";
break;
default:
break;
}
shell_surface_->UpdateShellSurface(type_);
}

void WaylandWindow::SetWindowTitle(const string16& title) {
Expand Down

0 comments on commit bf1dc26

Please sign in to comment.