[Clean-up] Remove unnecessary default: NOTREACHED() case.

ishermandom · Commit bot · commit 267aeeb8d85c · 2014-09-26T22:40:31.000Z
The (clang) compiler is smart enough to catch this error at compile-time, which is better than waiting until runtime. BUG=none TEST=none R=keybuk@chromium.org Review URL: https://codereview.chromium.org/575343002 Cr-Commit-Position: refs/heads/master@{#297060}
diff --git a/extensions/browser/api/bluetooth_socket/bluetooth_api_socket.cc b/extensions/browser/api/bluetooth_socket/bluetooth_api_socket.cc
@@ -140,8 +140,6 @@ void BluetoothApiSocket::OnSocketReceiveError(
     case device::BluetoothSocket::kSystemError:
       error_reason = BluetoothApiSocket::kSystemError;
       break;
-    default:
-      NOTREACHED();
   }
   error_callback.Run(error_reason, message);
 }

Original file line number	Diff line number	Diff line change
`@@ -140,8 +140,6 @@ void BluetoothApiSocket::OnSocketReceiveError(`
`140`	`140`	`case device::BluetoothSocket::kSystemError:`
`141`	`141`	`error_reason = BluetoothApiSocket::kSystemError;`
`142`	`142`	`break;`
`143`		`- default:`
`144`		`- NOTREACHED();`
`145`	`143`	`}`
`146`	`144`	`error_callback.Run(error_reason, message);`
`147`	`145`	`}`