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

Side by Side Diff: chrome/installer/mac/sign_versioned_dir.sh.in

Issue 2884693002: Rename the XPC identifier to use the outer bundle (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 #!/bin/bash -p 1 #!/bin/bash -p
2 2
3 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 3 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 # Using codesign, sign the contents of the versioned directory. Namely, this 7 # Using codesign, sign the contents of the versioned directory. Namely, this
8 # includes the framework and helper app. After signing, the signatures are 8 # includes the framework and helper app. After signing, the signatures are
9 # verified. 9 # verified.
10 10
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 107
108 codesign_with_options "${app_mode_loader_tmp}" \ 108 codesign_with_options "${app_mode_loader_tmp}" \
109 "${enforcement_flags_helpers}" \ 109 "${enforcement_flags_helpers}" \
110 "app_mode_loader" 110 "app_mode_loader"
111 111
112 cp "${app_mode_loader_tmp}" "${app_mode_loader}" 112 cp "${app_mode_loader_tmp}" "${app_mode_loader}"
113 rm -f "${app_mode_loader_tmp}" 113 rm -f "${app_mode_loader_tmp}"
114 114
115 codesign_with_options "${notification_service}" \ 115 codesign_with_options "${notification_service}" \
116 "${enforcement_flags_helpers}" \ 116 "${enforcement_flags_helpers}" \
117 "com.google.Chrome.framework.AlertNotificationService" 117 "com.google.Chrome.AlertNotificationService"
Mark Mentovai 2017/05/16 18:20:08 Here too.
Miguel Garcia 2017/05/19 08:43:56 Done.
118 118
119 # Only sign widevine plugin if it is present in the bundle. 119 # Only sign widevine plugin if it is present in the bundle.
120 # ${enforcement_flags*} are meaningless for dynamic libraries. 120 # ${enforcement_flags*} are meaningless for dynamic libraries.
121 if [[ -f "${widevine_plugin}" ]]; then 121 if [[ -f "${widevine_plugin}" ]]; then
122 codesign_with_options "${widevine_plugin}" "" "widevinecdmadapter" 122 codesign_with_options "${widevine_plugin}" "" "widevinecdmadapter"
123 fi 123 fi
124 124
125 # The framework is a dylib, so ${enforcement_flags_helpers} are meaningless. 125 # The framework is a dylib, so ${enforcement_flags_helpers} are meaningless.
126 codesign_with_options "${framework}" "" "com.google.Chrome.framework" 126 codesign_with_options "${framework}" "" "com.google.Chrome.framework"
127 127
128 codesign_with_options "${helper_app}" \ 128 codesign_with_options "${helper_app}" \
129 "${enforcement_flags_app}" \ 129 "${enforcement_flags_app}" \
130 "com.google.Chrome.helper" 130 "com.google.Chrome.helper"
131 131
132 if [[ -f "${widevine_plugin}" ]]; then 132 if [[ -f "${widevine_plugin}" ]]; then
133 codesign_display_and_verify "${widevine_plugin}" 133 codesign_display_and_verify "${widevine_plugin}"
134 fi 134 fi
135 135
136 codesign_display_and_verify "${crashpad_handler}" --deep 136 codesign_display_and_verify "${crashpad_handler}" --deep
137 codesign_display_and_verify "${app_mode_loader}" --ignore-resources 137 codesign_display_and_verify "${app_mode_loader}" --ignore-resources
138 codesign_display_and_verify "${notification_service}" --deep 138 codesign_display_and_verify "${notification_service}" --deep
139 # The framework contains KeystoneRegistration.framework, which uses 139 # The framework contains KeystoneRegistration.framework, which uses
140 # custom resource rules, so use --no-strict to verify. 140 # custom resource rules, so use --no-strict to verify.
141 codesign_display_and_verify "${framework}" --deep --no-strict 141 codesign_display_and_verify "${framework}" --deep --no-strict
142 codesign_display_and_verify "${framework}" --strict 142 codesign_display_and_verify "${framework}" --strict
143 codesign_display_and_verify "${helper_app}" --deep 143 codesign_display_and_verify "${helper_app}" --deep
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698