Fix more cases of unreachable code on Windows

components/search_provider_logos/logo_tracker_unittest.cc

 // Copyright 2014 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/search_provider_logos/logo_tracker.h"
 
 #include <vector>
 
 #include "base/base64.h"
 #include "base/bind.h"
@@ skipping 206 matching lines @@
   virtual const LogoMetadata* GetCachedLogoMetadataInternal() {
     return metadata_.get();
   }
 
   virtual void SetCachedLogoInternal(const EncodedLogo* logo) {
     logo_.reset(logo ? new EncodedLogo(*logo) : NULL);
     metadata_.reset(logo ? new LogoMetadata(logo->metadata) : NULL);
   }
 
   virtual scoped_ptr<EncodedLogo> GetCachedLogo() OVERRIDE {
+    OnGetCachedLogo();
     return make_scoped_ptr(logo_ ? new EncodedLogo(*logo_) : NULL);
-    OnGetCachedLogo();
   }
 
  private:
   scoped_ptr<LogoMetadata> metadata_;
   scoped_ptr<EncodedLogo> logo_;
 };
 
 class MockLogoObserver : public LogoObserver {
  public:
   virtual ~MockLogoObserver() {}
@@ skipping 421 matching lines @@
   MockLogoObserver listener2;
   listener2.ExpectFreshLogo(&logo);
   logo_tracker_->GetLogo(&listener2);
 
   base::RunLoop().RunUntilIdle();
 }
 
 }  // namespace
 
 }  // namespace search_provider_logos