summaryrefslogtreecommitdiffstats
path: root/chrome
diff options
context:
space:
mode:
Diffstat (limited to 'chrome')
-rw-r--r--chrome/browser/login_prompt.h6
1 files changed, 6 insertions, 0 deletions
diff --git a/chrome/browser/login_prompt.h b/chrome/browser/login_prompt.h
index 5027409..6279c7a 100644
--- a/chrome/browser/login_prompt.h
+++ b/chrome/browser/login_prompt.h
@@ -28,6 +28,12 @@ class URLRequest;
// the URLRequest that needs it. Used by the automation proxy for testing.
// These functions should be (and are, in LoginHandlerImpl) implemented in
// a thread safe manner.
+//
+// TODO(erg): Refactor the common code from all LoginHandler subclasses into a
+// common controller class. All the methods below have the same copy/pasted
+// implementation. This is more difficult then it should be because all these
+// subclasses are also base::RefCountedThreadSafe<> and I'm not sure how to get
+// ownership correct. http://crbug.com/14909
class LoginHandler {
public:
// Builds the platform specific LoginHandler. Used from within