DO NOT COMMIT (will be split in sub-components) - clang-tidy WaitableEvent refactor

media/base/pipeline_impl.cc

 // Copyright (c) 2012 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.
 
 #include "media/base/pipeline_impl.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 77 matching lines @@
 void PipelineImpl::Stop() {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
   DVLOG(2) << __FUNCTION__;
 
   if (media_task_runner_ != main_task_runner_) {
     // This path is executed by production code where the two task runners -
     // main and media - live on different threads.
     // TODO(alokp): It may be possible to not have to wait for StopTask by
     // moving the members accessed on media thread into a class/struct and
     // DeleteSoon the instance on the media thread.
-    base::WaitableEvent waiter(false, false);
+    base::WaitableEvent waiter(base::WaitableEvent::ResetPolicy::AUTOMATIC,
+                               base::WaitableEvent::InitialState::NOT_SIGNALED);
     base::Closure stop_cb =
         base::Bind(&base::WaitableEvent::Signal, base::Unretained(&waiter));
     // If posting the task fails or the posted task fails to run,
     // we will wait here forever. So add a CHECK to make sure we do not run
     // into those situations.
     CHECK(weak_factory_.HasWeakPtrs());
     CHECK(media_task_runner_->PostTask(
         FROM_HERE, base::Bind(&PipelineImpl::StopTask, weak_this_, stop_cb)));
     waiter.Wait();
   } else {
@@ skipping 841 matching lines @@
   if (demuxer_->GetStream(DemuxerStream::AUDIO)) {
     metadata.has_audio = true;
   }
 
   main_task_runner_->PostTask(
       FROM_HERE,
       base::Bind(&Pipeline::Client::OnMetadata, weak_client_, metadata));
 }
 
 }  // namespace media