Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(219)

Issue 6582003: Merge 75969 - Fix a ChromeFrame crash which occurs in the field typically dur... (Closed)

Created:
9 years, 10 months ago by ananta
Modified:
9 years, 6 months ago
Reviewers:
ananta
CC:
chromium-reviews, amit
Visibility:
Public.

Description

Merge 75969 - Fix a ChromeFrame crash which occurs in the field typically during upgrades when multiple ChromeFrame versions are loaded in IE. We have code in ChromeFrame which attempts to fallback to the older version in case it is already loaded. This is done by creating a named shared memory block which contains the version of the loaded dll. Newer versions look at this block and fallback to the older version if any. However due to a small oversight this never worked as the shared memory block was created unnamed. Thus the fallbacks never occurred leading to multiple chrome frame binaries attempting to patch urlmon, etc. Fixes bug http://code.google.com/p/chromium/issues/detail?id=74047 BUG=74047 Review URL: http://codereview.chromium.org/6581045 TBR=ananta@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75970

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M chrome_frame/dll_redirector.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
ananta
9 years, 10 months ago (2011-02-24 22:21:12 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698