Cleanup: Remove some unneeded Extension* class forward declarations.

chrome/browser/extensions/ntp_overridden_bubble_controller.h

 // Copyright (c) 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_NTP_OVERRIDDEN_BUBBLE_CONTROLLER_H_
 #define CHROME_BROWSER_EXTENSIONS_NTP_OVERRIDDEN_BUBBLE_CONTROLLER_H_
 
 #include <string>

[Finnur, 2014/05/15 11:33:33]

This can be removed now also.

[Lei Zhang, 2014/05/20 00:17:08]

But we use std::string on line 20.

[Finnur, 2014/05/20 11:52:25]

Good point. Missed that during my scanning of the file.

 #include "chrome/browser/extensions/extension_message_bubble_controller.h"
 
-class ExtensionService;
-
 namespace extensions {
 
 class NtpOverriddenBubbleController : public ExtensionMessageBubbleController {
  public:
   explicit NtpOverriddenBubbleController(Profile* profile);
   virtual ~NtpOverriddenBubbleController();
 
   // Whether the controller knows that we should show the bubble for extension
   // with |extension_id|. Returns true if so.
   bool ShouldShow(const std::string& extension_id);
 
   // ExtensionMessageBubbleController:
   virtual bool CloseOnDeactivate() OVERRIDE;
 
  private:
   // A weak pointer to the profile we are associated with. Not owned by us.
   Profile* profile_;
 
   DISALLOW_COPY_AND_ASSIGN(NtpOverriddenBubbleController);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_NTP_OVERRIDDEN_BUBBLE_CONTROLLER_H_