Fix some issues found looking at the code.

Patch from Gaetano Mendola <mendola@gmail.com>
Original review: http://codereview.chromium.org/4273

I added some additions on my part and two unit test fix due to the added DCHECK. Reduced atl header inclusion.
Review URL: http://codereview.chromium.org/5009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2730 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 4 deletions

diff --git a/base/event_recorder.h b/base/event_recorder.h
index 500ad1a..a258d3a 100644
--- a/base/event_recorder.h
+++ b/base/event_recorder.h
@@ -37,7 +37,7 @@ class EventRecorder {
   // Starts recording events.
   // Will clobber the file if it already exists.
   // Returns true on success, or false if an error occurred.
-  bool StartRecording(std::wstring &filename);
+  bool StartRecording(const std::wstring& filename);
 
   // Stops recording.
   void StopRecording();
@@ -47,7 +47,7 @@ class EventRecorder {
 
   // Plays events previously recorded.
   // Returns true on success, or false if an error occurred.
-  bool StartPlayback(std::wstring &filename);
+  bool StartPlayback(const std::wstring& filename);
 
   // Stops playback.
   void StopPlayback();
@@ -68,7 +68,9 @@ class EventRecorder {
       : is_recording_(false),
         is_playing_(false),
         journal_hook_(NULL),
-        file_(NULL) {
+        file_(NULL),
+        playback_first_msg_time_(0),
+        playback_start_time_(0) {
   }
   ~EventRecorder();
 
@@ -88,4 +90,3 @@ class EventRecorder {
 }  // namespace base
 
 #endif // BASE_EVENT_RECORDER_H_
-