Don't make a top-level namespace called "internal"

webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc

Index: webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc
diff --git a/webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc b/webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc
index 2507027689705b328d64f74304dd6ff2fee3930d..85f7edcaded1c1d20de75fd76b7679c06ce9b36c 100644
--- a/webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc
+++ b/webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.cc
@@ -24,6 +24,7 @@
 #include "webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_nalu.h"
 #include "webrtc/video_frame.h"
 
+namespace webrtc {
 namespace internal {

[magjed_webrtc, 2017/04/13 09:35:44]

Can you make this an anonymous namespace instead? Also, if you put it inside the
webrtc namespace, can you take the opportunity to remove the unnecessary
'webrtc::'?

[kwiberg-webrtc, 2017/04/13 11:49:03]

Done.

 
 static const int64_t kMsPerSec = 1000;
@@ -72,8 +73,6 @@ void VTDecompressionOutputCallback(void* decoder,
 
 }  // namespace internal
 
-namespace webrtc {
-
 H264VideoToolboxDecoder::H264VideoToolboxDecoder()
     : callback_(nullptr),
       video_format_(nullptr),