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

Issue 8055021: Merge 100224 - Make Chrome Frame registration and unregistration more robust and make it return e... (Closed)

Created:
9 years, 2 months ago by robertshield
Modified:
9 years, 2 months ago
Reviewers:
robertshield
CC:
chromium-reviews, amit
Visibility:
Public.

Description

Merge 100224 - Make Chrome Frame registration and unregistration more robust and make it return extra information in the failure code. Specifically, there are a series of registration changes made by Chrome Frame at registration and unregistration time. Previously, if any of those changes failed, the previous changes were left and subsequent changes were not made. During registration this could result in partial registration data being left if one of the items failed which is bad but wasn't a problem in practice, because if ever registration failed, unregistration would then be performed by the installer. Also the most likely step to fail happens first. During unregistration, if any of the steps fail, the remaining unregistration steps will not be taken. This is bad, as uninstallation is a best effort, "clean up everything you can" type thing. Changed this to for registration roll back the previous N-1 steps of step N fails and for unregistration to attempt to perform all N steps disregarding failures. Updated installer logging. BUG=94848 TEST=Unregistration of e.g. the user-agent modification should always be performed even if an earlier step fails. Review URL: http://codereview.chromium.org/7824010 TBR=robertshield@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=102982

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+420 lines, -290 lines) Patch
M chrome/installer/util/self_reg_work_item.cc View 2 chunks +26 lines, -3 lines 0 comments Download
M chrome_frame/chrome_tab.cc View 12 chunks +394 lines, -287 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
robertshield
9 years, 2 months ago (2011-09-27 18:41:06 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698