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

Issue 10911137: Remove bundled NPAPI Flash on Windows. (Closed)

Created:
8 years, 3 months ago by yzshen1
Modified:
8 years, 3 months ago
Reviewers:
brettw
CC:
chromium-reviews, sadrul, jam, darin-cc_chromium.org, stuartmorgan+watch_chromium.org, ben+watch_chromium.org
Visibility:
Public.

Description

Remove bundled NPAPI Flash on Windows. This change also takes care of turning on bundled Pepper Flash, if NPAPI Flash is enabled while Pepper Flash is disabled. BUG=146869 TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155507

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -13 lines) Patch
M chrome/browser/browser_process_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/component_updater/npapi_flash_component_installer.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/plugin_prefs.cc View 4 chunks +37 lines, -0 lines 0 comments Download
M chrome/browser/plugin_prefs_factory.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/common/chrome_content_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/chrome_paths.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/common/chrome_paths.cc View 1 chunk +3 lines, -1 line 0 comments Download
M chrome/common/pref_names.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/pref_names.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M third_party/adobe/flash/flash_player.gyp View 1 chunk +0 lines, -9 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
lgtm
8 years, 3 months ago (2012-09-07 19:30:31 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yzshen@chromium.org/10911137/1
8 years, 3 months ago (2012-09-07 19:33:51 UTC) #2
commit-bot: I haz the power
8 years, 3 months ago (2012-09-07 22:33:32 UTC) #3
Change committed as 155507

Powered by Google App Engine
This is Rietveld 408576698