Prevent redirector from returning the current module in cases where it fails to look up the first...

Prevent redirector from returning the current module in cases where it fails to look up the first module in. This prevents a bug that can cause user-level Chrome Frame to redirect to itself when a system-level Chrome Frame is present.

BUG=96016
TEST=Upgrade a user-level CF install to a system-level CF install while user-level CF is rendering. Observe that the current explorer instance does not hang when new CF tabs are created.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=100904

[grt (UTC plus 2), 2011-09-12 17:27:15]

LGTM with one small debug logging nit.

http://codereview.chromium.org/7866043/diff/4001/chrome/installer/test/altern...
File chrome/installer/test/alternate_version_generator.cc (left):

http://codereview.chromium.org/7866043/diff/4001/chrome/installer/test/altern...
chrome/installer/test/alternate_version_generator.cc:339: if (image_handle.Get()
!= INVALID_HANDLE_VALUE) {
Greg is a booboo head.

http://codereview.chromium.org/7866043/diff/4001/chrome_frame/dll_redirector.cc
File chrome_frame/dll_redirector.cc (right):

http://codereview.chromium.org/7866043/diff/4001/chrome_frame/dll_redirector....
chrome_frame/dll_redirector.cc:234: DPLOG(ERROR) << "DllRedirector: Could not
get address of "
DPLOG_IF(ERROR, !proc_ptr) << ...

[robertshield, 2011-09-12 17:55:48]

Thanks!

http://codereview.chromium.org/7866043/diff/4001/chrome_frame/dll_redirector.cc
File chrome_frame/dll_redirector.cc (right):

http://codereview.chromium.org/7866043/diff/4001/chrome_frame/dll_redirector....
chrome_frame/dll_redirector.cc:234: DPLOG(ERROR) << "DllRedirector: Could not
get address of "
On 2011/09/12 17:27:15, grt wrote:
> DPLOG_IF(ERROR, !proc_ptr) << ...

You and your fancy macros..