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

Issue 11615034: Remove unused Flash-related code: (Closed)

Created:
7 years, 11 months ago by yzshen1
Modified:
7 years, 11 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Remove unused Flash-related code: - field trial code for Pepper Flash. - registration and component-update code for NPAPI Flash. TEST=None BUG=None Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=174741

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -340 lines) Patch
M chrome/browser/browser_process_impl.cc View 3 chunks +0 lines, -21 lines 0 comments Download
M chrome/browser/chrome_content_browser_client.cc View 2 chunks +0 lines, -2 lines 0 comments Download
M chrome/browser/component_updater/flash_component_installer.h View 1 chunk +0 lines, -3 lines 0 comments Download
D chrome/browser/component_updater/npapi_flash_component_installer.cc View 1 chunk +0 lines, -224 lines 0 comments Download
M chrome/browser/component_updater/pepper_flash_component_installer.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/chrome_content_client.h View 1 chunk +0 lines, -13 lines 0 comments Download
M chrome/common/chrome_content_client.cc View 4 chunks +4 lines, -22 lines 0 comments Download
M chrome/common/chrome_paths.h View 1 chunk +3 lines, -6 lines 1 comment Download
M chrome/common/chrome_paths.cc View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/common/chrome_switches.h View 2 chunks +0 lines, -2 lines 0 comments Download
M chrome/common/chrome_switches.cc View 3 chunks +1 line, -7 lines 0 comments Download
M chrome/common/pepper_flash.h View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/common/pepper_flash.cc View 1 chunk +0 lines, -27 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
yzshen1
Hi, Brett and Carlos. Would you please take a look? Carlos: component updater code. Brett: ...
7 years, 11 months ago (2012-12-27 23:10:57 UTC) #1
cpu_(ooo_6.6-7.5)
component updater lgtm
7 years, 11 months ago (2012-12-27 23:21:48 UTC) #2
brettw
lgtm
7 years, 11 months ago (2012-12-28 19:15:36 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yzshen@chromium.org/11615034/1
7 years, 11 months ago (2012-12-28 19:24:56 UTC) #4
commit-bot: I haz the power
7 years, 11 months ago (2012-12-28 21:31:20 UTC) #5
Message was sent while issue was closed.
Change committed as 174741

Powered by Google App Engine
This is Rietveld 408576698