Commit 340a2278 authored by wanghui's avatar wanghui Committed by Commit Bot

Fix pack chrome.7z with ffmpeg component build.

Pack the ffmpeg.dll into chrome.7z when "is_component_ffmpeg" is true.

Bug:1100242

Change-Id: I9e82739b9edf38c75238f08bb0c0b160a600d926
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2269408Reviewed-by: default avatarGreg Thompson <grt@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#783916}
parent 2879f845
...@@ -7,6 +7,7 @@ import("//build/config/features.gni") ...@@ -7,6 +7,7 @@ import("//build/config/features.gni")
import("//build/config/ui.gni") import("//build/config/ui.gni")
import("//chrome/process_version_rc_template.gni") import("//chrome/process_version_rc_template.gni")
import("//components/nacl/features.gni") import("//components/nacl/features.gni")
import("//third_party/ffmpeg/ffmpeg_options.gni")
import("//third_party/icu/config.gni") import("//third_party/icu/config.gni")
import("//tools/v8_context_snapshot/v8_context_snapshot.gni") import("//tools/v8_context_snapshot/v8_context_snapshot.gni")
import("//ui/base/ui_features.gni") import("//ui/base/ui_features.gni")
...@@ -192,6 +193,10 @@ template("generate_mini_installer") { ...@@ -192,6 +193,10 @@ template("generate_mini_installer") {
args += [ "--component_build=1" ] args += [ "--component_build=1" ]
} }
if (is_component_ffmpeg) {
args += [ "--component_ffmpeg_build=1" ]
}
if (skip_archive_compression) { if (skip_archive_compression) {
args += [ "--skip_archive_compression" ] args += [ "--skip_archive_compression" ]
} else { } else {
......
...@@ -65,6 +65,9 @@ MEIPreload\preloaded_data.pb: %(VersionDir)s\MEIPreload\ ...@@ -65,6 +65,9 @@ MEIPreload\preloaded_data.pb: %(VersionDir)s\MEIPreload\
[HIDPI] [HIDPI]
chrome_200_percent.pak: %(VersionDir)s\ chrome_200_percent.pak: %(VersionDir)s\
[FFMPEG]
ffmpeg.dll: %(VersionDir)s\
[TOUCH] [TOUCH]
[GOOGLE_CHROME] [GOOGLE_CHROME]
......
...@@ -94,7 +94,8 @@ def CompressUsingLZMA(build_dir, compressed_file, input_file, verbose): ...@@ -94,7 +94,8 @@ def CompressUsingLZMA(build_dir, compressed_file, input_file, verbose):
def CopyAllFilesToStagingDir(config, distribution, staging_dir, build_dir, def CopyAllFilesToStagingDir(config, distribution, staging_dir, build_dir,
enable_hidpi, include_snapshotblob, verbose): enable_hidpi, include_snapshotblob,
component_build, component_ffmpeg_build, verbose):
"""Copies the files required for installer archive. """Copies the files required for installer archive.
Copies all common files required for various distributions of Chromium and Copies all common files required for various distributions of Chromium and
also files for the specific Chromium build specified by distribution. also files for the specific Chromium build specified by distribution.
...@@ -117,6 +118,10 @@ def CopyAllFilesToStagingDir(config, distribution, staging_dir, build_dir, ...@@ -117,6 +118,10 @@ def CopyAllFilesToStagingDir(config, distribution, staging_dir, build_dir,
CopySectionFilesToStagingDir(config, 'SNAPSHOTBLOB', staging_dir, build_dir, CopySectionFilesToStagingDir(config, 'SNAPSHOTBLOB', staging_dir, build_dir,
verbose) verbose)
if component_build != '1' and component_ffmpeg_build == '1':
CopySectionFilesToStagingDir(config, 'FFMPEG', staging_dir, build_dir,
verbose)
# The 'SafeConfigParser' makes all strings lowercase - which works fine on # The 'SafeConfigParser' makes all strings lowercase - which works fine on
# a cases-insensitive NTFS partition, but makes no sense when trying to build # a cases-insensitive NTFS partition, but makes no sense when trying to build
# mini_installer.exe on a linux box. This function can be used to make glob # mini_installer.exe on a linux box. This function can be used to make glob
...@@ -538,6 +543,8 @@ def main(options): ...@@ -538,6 +543,8 @@ def main(options):
staging_dir, options.build_dir, staging_dir, options.build_dir,
options.enable_hidpi, options.enable_hidpi,
options.include_snapshotblob, options.include_snapshotblob,
options.component_build,
options.component_ffmpeg_build,
options.verbose) options.verbose)
if options.component_build == '1': if options.component_build == '1':
...@@ -603,6 +610,8 @@ def _ParseOptions(): ...@@ -603,6 +610,8 @@ def _ParseOptions():
help='Whether to include the V8 snapshot blob.') help='Whether to include the V8 snapshot blob.')
parser.add_option('--component_build', default='0', parser.add_option('--component_build', default='0',
help='Whether this archive is packaging a component build.') help='Whether this archive is packaging a component build.')
parser.add_option('--component_ffmpeg_build', default='0',
help='Whether this archive is packaging with ffmpeg component build.')
parser.add_option('--skip_archive_compression', parser.add_option('--skip_archive_compression',
action='store_true', default=False, action='store_true', default=False,
help='This will turn off compression of chrome.7z into chrome.packed.7z ' help='This will turn off compression of chrome.7z into chrome.packed.7z '
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment