LanguageState should be owned by TranslateManager

components/translate/core/browser/translate_ui_delegate.h

Index: components/translate/core/browser/translate_ui_delegate.h
diff --git a/components/translate/core/browser/translate_ui_delegate.h b/components/translate/core/browser/translate_ui_delegate.h
index ab6986e3f32619b8452bc7f2b440e23eb296d26b..48eb5869b9f845edaf6e80b234a23de9661af850 100644
--- a/components/translate/core/browser/translate_ui_delegate.h
+++ b/components/translate/core/browser/translate_ui_delegate.h
@@ -10,6 +10,7 @@
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 #include "base/strings/string16.h"
 #include "components/translate/core/common/translate_errors.h"
 
@@ -26,7 +27,7 @@ class TranslateUIDelegate {
   enum { NO_INDEX = -1, };
 
   TranslateUIDelegate(TranslateClient* translate_client,

[droger, 2014/06/05 08:36:35]

Optional cleanup while you are here:
1) remove the translate_client parameter, and get it from the manager (there is
a translate_client() method on the manager).
2) remove the translate_client_ member, it is not actually used.

[nshaik, 2014/06/05 18:45:30]

Done.

-                      TranslateManager* translate_manager,
+                      const base::WeakPtr<TranslateManager>& translate_manager,
                       const std::string& original_language,
                       const std::string& target_language);
   virtual ~TranslateUIDelegate();
@@ -100,7 +101,7 @@ class TranslateUIDelegate {
 
   TranslateClient* translate_client_;
   TranslateDriver* translate_driver_;
-  TranslateManager* translate_manager_;
+  base::WeakPtr<TranslateManager> translate_manager_;
 
   typedef std::pair<std::string, base::string16> LanguageNamePair;