Comment #61 on issue 6824 by bugdro...@chromium.org: No logon prompt  
for "Integrated Windows Authentication" (NTLM) only sites
http://code.google.com/p/chromium/issues/detail?id=6824

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=12290

------------------------------------------------------------------------
r12290 | w...@chromium.org | 2009-03-23 09:52:20 -0700 (Mon, 23 Mar 2009) |  
17 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/http/http_auth_handler_ntlm.cc?r1=12290&r2=12289
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/http/http_auth_handler_ntlm.h?r1=12290&r2=12289
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/net/http/http_network_transaction_unittest.cc?r1=12290&r2=12289

Make GetHostNameProc return a std::string.

Add the ScopedProcSetter helper class so that unit tests
restore the original GenerateRandom and GetHostName
functions on completion.

Merge NTLMAuthModule into HttpAuthHandlerNTLM.  The data
members domain_, username_, and password_ are moved.  The
Init method is inlined at its only call site.  The
GetNextToken method is moved.

Make generate_random_proc_ and get_host_name_proc_ static
members of the HttpAuthHandlerNTLM class.

R=eroman
BUG=6567,6824
Review URL: http://codereview.chromium.org/43113
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to