From 47175752068e8a0a8b073460148903be0e9d43cd Mon Sep 17 00:00:00 2001 From: guoweis Date: Mon, 14 Sep 2015 11:14:10 -0700 Subject: [PATCH] Remove NOTREACHED() in default switch case which could be hit. This happens when new enum is added in WebRTC. BUG= Review URL: https://codereview.chromium.org/1336143003 Cr-Commit-Position: refs/heads/master@{#348658} --- content/renderer/media/rtc_peer_connection_handler.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/content/renderer/media/rtc_peer_connection_handler.cc b/content/renderer/media/rtc_peer_connection_handler.cc index 9c50906e3da9..06953b591538 100644 --- a/content/renderer/media/rtc_peer_connection_handler.cc +++ b/content/renderer/media/rtc_peer_connection_handler.cc @@ -532,7 +532,8 @@ class PeerConnectionUMAObserver : public webrtc::UMAObserver { counter, counter_max); break; default: - NOTREACHED(); + // The default clause is expected to reach when new enum types are + // added. break; } } @@ -555,7 +556,9 @@ class PeerConnectionUMAObserver : public webrtc::UMAObserver { value); break; default: - NOTREACHED(); + // The default clause is expected to reach when new enum types are + // added. + break; } } }; -- 2.11.4.GIT