arc_bridge: Add more logging, plus cleanup

components/arc/arc_bridge_bootstrap.cc

 // Copyright 2015 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 "components/arc/arc_bridge_bootstrap.h"
 
 #include <fcntl.h>
 #include <grp.h>
 #include <unistd.h>
 
@@ skipping 110 matching lines @@
   base::WeakPtrFactory<ArcBridgeBootstrapImpl> weak_factory_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ArcBridgeBootstrapImpl);
 };
 
 ArcBridgeBootstrapImpl::ArcBridgeBootstrapImpl() : weak_factory_(this) {}
 
 ArcBridgeBootstrapImpl::~ArcBridgeBootstrapImpl() {
   DCHECK(thread_checker_.CalledOnValidThread());
-  DCHECK(state_ == State::STOPPED);
+  VLOG(1) << "State: " << static_cast<uint32_t>(state_);
+  DCHECK(state_ == State::STOPPED || state_ == State::STOPPING);
 }
 
 void ArcBridgeBootstrapImpl::Start() {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(delegate_);
   if (state_ != State::STOPPED) {
     VLOG(1) << "Start() called when instance is not stopped";
     return;
   }
   SetState(State::SOCKET_CREATING);
@@ skipping 180 matching lines @@
   DCHECK(delegate_);
   SetState(State::STOPPED);
   delegate_->OnStopped();
 }
 
 void ArcBridgeBootstrapImpl::SetState(State state) {
   DCHECK(thread_checker_.CalledOnValidThread());
   // DCHECK on enum classes not supported.
   DCHECK(state_ != state);
   state_ = state;
+  VLOG(2) << "State: " << static_cast<uint32_t>(state_);
 }
 
 }  // namespace
 
-ArcBridgeBootstrap::ArcBridgeBootstrap() {}
-
-ArcBridgeBootstrap::~ArcBridgeBootstrap() {}
-
 // static
 std::unique_ptr<ArcBridgeBootstrap> ArcBridgeBootstrap::Create() {
   return base::WrapUnique(new ArcBridgeBootstrapImpl());
 }
 
 }  // namespace arc