test_runner: Migrate MockWebRTCDTMFSenderHandler to Chromium C++ style

content/shell/renderer/test_runner/mock_webrtc_dtmf_sender_handler.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCKWEBRTCDTMFSENDERHANDLER_H_
 #define CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCKWEBRTCDTMFSENDERHANDLER_H_
 
 #include "base/basictypes.h"
 #include "content/shell/renderer/test_runner/TestCommon.h"
 #include "content/shell/renderer/test_runner/WebTask.h"
 #include "third_party/WebKit/public/platform/WebMediaStreamTrack.h"
 #include "third_party/WebKit/public/platform/WebRTCDTMFSenderHandler.h"
 #include "third_party/WebKit/public/platform/WebString.h"
 
 namespace content {
 
 class WebTestDelegate;
 
 class MockWebRTCDTMFSenderHandler : public blink::WebRTCDTMFSenderHandler {
-public:
-    MockWebRTCDTMFSenderHandler(const blink::WebMediaStreamTrack&, WebTestDelegate*);
+ public:
+  MockWebRTCDTMFSenderHandler(const blink::WebMediaStreamTrack&,

[jochen (gone - plz use gerrit), 2014/06/03 12:55:00]

nit. add parameter names

[Nikhil, 2014/06/04 05:33:55]

Done.

+                              WebTestDelegate*);
 
-    virtual void setClient(blink::WebRTCDTMFSenderHandlerClient*) OVERRIDE;
+  virtual void setClient(blink::WebRTCDTMFSenderHandlerClient*) OVERRIDE;

[jochen (gone - plz use gerrit), 2014/06/03 12:55:00]

nit. add a comment saying which interface you're overriding here (similar to the
WebTask comment below)

[Nikhil, 2014/06/04 05:33:55]

Done.

 
-    virtual blink::WebString currentToneBuffer() OVERRIDE;
+  virtual blink::WebString currentToneBuffer() OVERRIDE;
 
-    virtual bool canInsertDTMF() OVERRIDE;
-    virtual bool insertDTMF(const blink::WebString& tones, long duration, long interToneGap) OVERRIDE;
+  virtual bool canInsertDTMF() OVERRIDE;
+  virtual bool insertDTMF(const blink::WebString& tones,
+                          long duration,
+                          long inter_tone_gap) OVERRIDE;
 
-    // WebTask related methods
-    WebTaskList* mutable_task_list() { return &m_taskList; }
-    void clearToneBuffer() { m_toneBuffer.reset(); }
+  // WebTask related methods
+  WebTaskList* mutable_task_list() { return &task_list_; }
+  void ClearToneBuffer() { tone_buffer_.reset(); }
 
-private:
-    MockWebRTCDTMFSenderHandler();
+ private:
+  MockWebRTCDTMFSenderHandler();
 
-    blink::WebRTCDTMFSenderHandlerClient* m_client;
-    blink::WebMediaStreamTrack m_track;
-    blink::WebString m_toneBuffer;
-    WebTaskList m_taskList;
-    WebTestDelegate* m_delegate;
+  blink::WebRTCDTMFSenderHandlerClient* client_;
+  blink::WebMediaStreamTrack track_;
+  blink::WebString tone_buffer_;
+  WebTaskList task_list_;
+  WebTestDelegate* delegate_;
 
-    DISALLOW_COPY_AND_ASSIGN(MockWebRTCDTMFSenderHandler);
+  DISALLOW_COPY_AND_ASSIGN(MockWebRTCDTMFSenderHandler);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_SHELL_RENDERER_TEST_RUNNER_MOCKWEBRTCDTMFSENDERHANDLER_H_