blob: d5c0dc4087ba7474d205573b952d1750784ddf10 [file] [log] [blame]
Adam Lesinski2ae4a872015-11-02 16:10:55 -08001/*
2 * Copyright (C) 2015 The Android Open Source Project
3 *
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at
7 *
8 * http://www.apache.org/licenses/LICENSE-2.0
9 *
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
15 */
16
Adam Lesinski2ae4a872015-11-02 16:10:55 -080017#include "link/ManifestFixer.h"
Adam Lesinskice5e56e2016-10-21 17:56:45 -070018
19#include <unordered_set>
20
21#include "android-base/logging.h"
22
Adam Lesinskicacb28f2016-10-19 12:18:14 -070023#include "ResourceUtils.h"
Adam Lesinski2ae4a872015-11-02 16:10:55 -080024#include "util/Util.h"
Adam Lesinskicc5609d2016-04-05 12:41:07 -070025#include "xml/XmlActionExecutor.h"
Adam Lesinski467f1712015-11-16 17:35:44 -080026#include "xml/XmlDom.h"
Adam Lesinski2ae4a872015-11-02 16:10:55 -080027
Adam Lesinskid5083f62017-01-16 15:07:21 -080028using android::StringPiece;
29
Adam Lesinski2ae4a872015-11-02 16:10:55 -080030namespace aapt {
31
Adam Lesinskicc5609d2016-04-05 12:41:07 -070032/**
Adam Lesinskicacb28f2016-10-19 12:18:14 -070033 * This is how PackageManager builds class names from AndroidManifest.xml
34 * entries.
Adam Lesinskicc5609d2016-04-05 12:41:07 -070035 */
Adam Lesinskice5e56e2016-10-21 17:56:45 -070036static bool NameIsJavaClassName(xml::Element* el, xml::Attribute* attr,
Adam Lesinskicc5609d2016-04-05 12:41:07 -070037 SourcePathDiagnostics* diag) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -070038 // We allow unqualified class names (ie: .HelloActivity)
39 // Since we don't know the package name, we can just make a fake one here and
40 // the test will be identical as long as the real package name is valid too.
Adam Lesinskice5e56e2016-10-21 17:56:45 -070041 Maybe<std::string> fully_qualified_class_name =
42 util::GetFullyQualifiedClassName("a", attr->value);
Adam Lesinskicc5609d2016-04-05 12:41:07 -070043
Adam Lesinskice5e56e2016-10-21 17:56:45 -070044 StringPiece qualified_class_name = fully_qualified_class_name
45 ? fully_qualified_class_name.value()
46 : attr->value;
Adam Lesinskid0f116b2016-07-08 15:00:32 -070047
Adam Lesinskice5e56e2016-10-21 17:56:45 -070048 if (!util::IsJavaClassName(qualified_class_name)) {
49 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070050 << "attribute 'android:name' in <" << el->name
51 << "> tag must be a valid Java class name");
Adam Lesinski52364f72016-01-11 13:10:24 -080052 return false;
Adam Lesinskicacb28f2016-10-19 12:18:14 -070053 }
54 return true;
55}
56
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080057static bool OptionalNameIsJavaClassName(xml::Element* el, SourcePathDiagnostics* diag) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -070058 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
59 return NameIsJavaClassName(el, attr, diag);
Adam Lesinskicacb28f2016-10-19 12:18:14 -070060 }
61 return true;
62}
63
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080064static bool RequiredNameIsJavaClassName(xml::Element* el, SourcePathDiagnostics* diag) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -070065 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
66 return NameIsJavaClassName(el, attr, diag);
Adam Lesinskicacb28f2016-10-19 12:18:14 -070067 }
Adam Lesinskice5e56e2016-10-21 17:56:45 -070068 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070069 << "<" << el->name << "> is missing attribute 'android:name'");
70 return false;
Adam Lesinski52364f72016-01-11 13:10:24 -080071}
72
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -080073static bool RequiredNameIsJavaPackage(xml::Element* el, SourcePathDiagnostics* diag) {
74 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
75 return util::IsJavaPackageName(attr->value);
76 }
77 diag->Error(DiagMessage(el->line_number)
78 << "<" << el->name << "> is missing attribute 'android:name'");
79 return false;
80}
81
Adam Lesinskice5e56e2016-10-21 17:56:45 -070082static bool VerifyManifest(xml::Element* el, SourcePathDiagnostics* diag) {
83 xml::Attribute* attr = el->FindAttribute({}, "package");
Adam Lesinskicacb28f2016-10-19 12:18:14 -070084 if (!attr) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -070085 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070086 << "<manifest> tag is missing 'package' attribute");
87 return false;
Adam Lesinskice5e56e2016-10-21 17:56:45 -070088 } else if (ResourceUtils::IsReference(attr->value)) {
89 diag->Error(
90 DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070091 << "attribute 'package' in <manifest> tag must not be a reference");
92 return false;
Adam Lesinskice5e56e2016-10-21 17:56:45 -070093 } else if (!util::IsJavaPackageName(attr->value)) {
94 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -070095 << "attribute 'package' in <manifest> tag is not a valid Java "
96 "package name: '"
97 << attr->value << "'");
98 return false;
99 }
100 return true;
Adam Lesinski52364f72016-01-11 13:10:24 -0800101}
102
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700103/**
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700104 * The coreApp attribute in <manifest> is not a regular AAPT attribute, so type
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700105 * checking on it is manual.
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700106 */
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700107static bool FixCoreAppAttribute(xml::Element* el, SourcePathDiagnostics* diag) {
108 if (xml::Attribute* attr = el->FindAttribute("", "coreApp")) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700109 std::unique_ptr<BinaryPrimitive> result =
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700110 ResourceUtils::TryParseBool(attr->value);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700111 if (!result) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700112 diag->Error(DiagMessage(el->line_number)
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700113 << "attribute coreApp must be a boolean");
114 return false;
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700115 }
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700116 attr->compiled_value = std::move(result);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700117 }
118 return true;
Adam Lesinski6b17d2c2016-08-10 11:37:06 -0700119}
120
Adam Lesinski86d67df2017-01-31 13:47:27 -0800121// Checks that <uses-feature> has android:glEsVersion or android:name, not both (or neither).
122static bool VerifyUsesFeature(xml::Element* el, SourcePathDiagnostics* diag) {
123 bool has_name = false;
124 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "name")) {
125 if (attr->value.empty()) {
126 diag->Error(DiagMessage(el->line_number)
127 << "android:name in <uses-feature> must not be empty");
128 return false;
129 }
130 has_name = true;
131 }
132
133 bool has_gl_es_version = false;
134 if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "glEsVersion")) {
135 if (has_name) {
136 diag->Error(DiagMessage(el->line_number)
137 << "cannot define both android:name and android:glEsVersion in <uses-feature>");
138 return false;
139 }
140 has_gl_es_version = true;
141 }
142
143 if (!has_name && !has_gl_es_version) {
144 diag->Error(DiagMessage(el->line_number)
145 << "<uses-feature> must have either android:name or android:glEsVersion attribute");
146 return false;
147 }
148 return true;
149}
150
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700151bool ManifestFixer::BuildRules(xml::XmlActionExecutor* executor,
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700152 IDiagnostics* diag) {
153 // First verify some options.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700154 if (options_.rename_manifest_package) {
155 if (!util::IsJavaPackageName(options_.rename_manifest_package.value())) {
156 diag->Error(DiagMessage() << "invalid manifest package override '"
157 << options_.rename_manifest_package.value()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700158 << "'");
159 return false;
160 }
161 }
162
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700163 if (options_.rename_instrumentation_target_package) {
164 if (!util::IsJavaPackageName(
165 options_.rename_instrumentation_target_package.value())) {
166 diag->Error(DiagMessage()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700167 << "invalid instrumentation target package override '"
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700168 << options_.rename_instrumentation_target_package.value()
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700169 << "'");
170 return false;
171 }
172 }
173
Adam Lesinski86d67df2017-01-31 13:47:27 -0800174 // Common <intent-filter> actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700175 xml::XmlNodeAction intent_filter_action;
176 intent_filter_action["action"];
177 intent_filter_action["category"];
178 intent_filter_action["data"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700179
Adam Lesinski86d67df2017-01-31 13:47:27 -0800180 // Common <meta-data> actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700181 xml::XmlNodeAction meta_data_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700182
Adam Lesinski86d67df2017-01-31 13:47:27 -0800183 // Common <uses-feature> actions.
184 xml::XmlNodeAction uses_feature_action;
185 uses_feature_action.Action(VerifyUsesFeature);
186
187 // Common component actions.
188 xml::XmlNodeAction component_action;
189 component_action.Action(RequiredNameIsJavaClassName);
190 component_action["intent-filter"] = intent_filter_action;
191 component_action["meta-data"] = meta_data_action;
192
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700193 // Manifest actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700194 xml::XmlNodeAction& manifest_action = (*executor)["manifest"];
195 manifest_action.Action(VerifyManifest);
196 manifest_action.Action(FixCoreAppAttribute);
197 manifest_action.Action([&](xml::Element* el) -> bool {
198 if (options_.version_name_default) {
199 if (el->FindAttribute(xml::kSchemaAndroid, "versionName") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700200 el->attributes.push_back(
201 xml::Attribute{xml::kSchemaAndroid, "versionName",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700202 options_.version_name_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700203 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700204 }
205
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700206 if (options_.version_code_default) {
207 if (el->FindAttribute(xml::kSchemaAndroid, "versionCode") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700208 el->attributes.push_back(
209 xml::Attribute{xml::kSchemaAndroid, "versionCode",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700210 options_.version_code_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700211 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700212 }
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800213 return true;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700214 });
215
216 // Meta tags.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700217 manifest_action["eat-comment"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700218
219 // Uses-sdk actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700220 manifest_action["uses-sdk"].Action([&](xml::Element* el) -> bool {
221 if (options_.min_sdk_version_default &&
222 el->FindAttribute(xml::kSchemaAndroid, "minSdkVersion") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700223 // There was no minSdkVersion defined and we have a default to assign.
224 el->attributes.push_back(
225 xml::Attribute{xml::kSchemaAndroid, "minSdkVersion",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700226 options_.min_sdk_version_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700227 }
228
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700229 if (options_.target_sdk_version_default &&
230 el->FindAttribute(xml::kSchemaAndroid, "targetSdkVersion") == nullptr) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700231 // There was no targetSdkVersion defined and we have a default to assign.
232 el->attributes.push_back(
233 xml::Attribute{xml::kSchemaAndroid, "targetSdkVersion",
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700234 options_.target_sdk_version_default.value()});
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700235 }
236 return true;
237 });
238
239 // Instrumentation actions.
Adam Lesinski86d67df2017-01-31 13:47:27 -0800240 manifest_action["instrumentation"].Action(RequiredNameIsJavaClassName);
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700241 manifest_action["instrumentation"].Action([&](xml::Element* el) -> bool {
242 if (!options_.rename_instrumentation_target_package) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700243 return true;
244 }
245
246 if (xml::Attribute* attr =
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700247 el->FindAttribute(xml::kSchemaAndroid, "targetPackage")) {
248 attr->value = options_.rename_instrumentation_target_package.value();
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700249 }
250 return true;
251 });
Adam Lesinski86d67df2017-01-31 13:47:27 -0800252 manifest_action["instrumentation"]["meta-data"] = meta_data_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700253
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700254 manifest_action["original-package"];
255 manifest_action["protected-broadcast"];
256 manifest_action["uses-permission"];
257 manifest_action["permission"];
258 manifest_action["permission-tree"];
259 manifest_action["permission-group"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700260 manifest_action["uses-configuration"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700261 manifest_action["supports-screens"];
Adam Lesinski86d67df2017-01-31 13:47:27 -0800262 manifest_action["uses-feature"] = uses_feature_action;
263 manifest_action["feature-group"]["uses-feature"] = uses_feature_action;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700264 manifest_action["compatible-screens"];
265 manifest_action["compatible-screens"]["screen"];
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700266 manifest_action["supports-gl-texture"];
Adam Lesinski5119e512016-12-05 19:48:20 -0800267 manifest_action["meta-data"] = meta_data_action;
268
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700269 // Application actions.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700270 xml::XmlNodeAction& application_action = manifest_action["application"];
271 application_action.Action(OptionalNameIsJavaClassName);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700272
Adam Lesinskiceb9b2f2017-02-16 12:05:42 -0800273 application_action["uses-library"].Action(RequiredNameIsJavaPackage);
274 application_action["library"].Action(RequiredNameIsJavaPackage);
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700275 application_action["meta-data"] = meta_data_action;
Adam Lesinski86d67df2017-01-31 13:47:27 -0800276 application_action["activity"] = component_action;
277 application_action["activity-alias"] = component_action;
278 application_action["service"] = component_action;
279 application_action["receiver"] = component_action;
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700280
281 // Provider actions.
Adam Lesinski86d67df2017-01-31 13:47:27 -0800282 application_action["provider"] = component_action;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700283 application_action["provider"]["grant-uri-permissions"];
284 application_action["provider"]["path-permissions"];
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700285
286 return true;
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800287}
288
Adam Lesinski52364f72016-01-11 13:10:24 -0800289class FullyQualifiedClassNameVisitor : public xml::Visitor {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700290 public:
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700291 using xml::Visitor::Visit;
Adam Lesinski52364f72016-01-11 13:10:24 -0800292
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700293 explicit FullyQualifiedClassNameVisitor(const StringPiece& package)
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700294 : package_(package) {}
Adam Lesinski52364f72016-01-11 13:10:24 -0800295
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700296 void Visit(xml::Element* el) override {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700297 for (xml::Attribute& attr : el->attributes) {
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700298 if (attr.namespace_uri == xml::kSchemaAndroid &&
299 class_attributes_.find(attr.name) != class_attributes_.end()) {
300 if (Maybe<std::string> new_value =
301 util::GetFullyQualifiedClassName(package_, attr.value)) {
302 attr.value = std::move(new_value.value());
Adam Lesinski52364f72016-01-11 13:10:24 -0800303 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700304 }
Adam Lesinski52364f72016-01-11 13:10:24 -0800305 }
306
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700307 // Super implementation to iterate over the children.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700308 xml::Visitor::Visit(el);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700309 }
310
311 private:
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700312 StringPiece package_;
313 std::unordered_set<StringPiece> class_attributes_ = {"name"};
Adam Lesinski52364f72016-01-11 13:10:24 -0800314};
315
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700316static bool RenameManifestPackage(const StringPiece& package_override,
317 xml::Element* manifest_el) {
318 xml::Attribute* attr = manifest_el->FindAttribute({}, "package");
Adam Lesinski52364f72016-01-11 13:10:24 -0800319
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700320 // We've already verified that the manifest element is present, with a package
321 // name specified.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700322 CHECK(attr != nullptr);
Adam Lesinski52364f72016-01-11 13:10:24 -0800323
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700324 std::string original_package = std::move(attr->value);
Adam Lesinskid5083f62017-01-16 15:07:21 -0800325 attr->value = package_override.to_string();
Adam Lesinski52364f72016-01-11 13:10:24 -0800326
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700327 FullyQualifiedClassNameVisitor visitor(original_package);
328 manifest_el->Accept(&visitor);
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700329 return true;
Adam Lesinski52364f72016-01-11 13:10:24 -0800330}
331
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700332bool ManifestFixer::Consume(IAaptContext* context, xml::XmlResource* doc) {
333 xml::Element* root = xml::FindRootElement(doc->root.get());
334 if (!root || !root->namespace_uri.empty() || root->name != "manifest") {
335 context->GetDiagnostics()->Error(DiagMessage(doc->file.source)
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700336 << "root tag must be <manifest>");
337 return false;
338 }
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800339
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700340 if ((options_.min_sdk_version_default ||
341 options_.target_sdk_version_default) &&
342 root->FindChild({}, "uses-sdk") == nullptr) {
Adam Lesinskie343eb12016-10-27 16:31:58 -0700343 // Auto insert a <uses-sdk> element. This must be inserted before the
344 // <application> tag. The device runtime PackageParser will make SDK version
345 // decisions while parsing <application>.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700346 std::unique_ptr<xml::Element> uses_sdk = util::make_unique<xml::Element>();
347 uses_sdk->name = "uses-sdk";
Adam Lesinskie343eb12016-10-27 16:31:58 -0700348 root->InsertChild(0, std::move(uses_sdk));
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700349 }
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800350
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700351 xml::XmlActionExecutor executor;
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700352 if (!BuildRules(&executor, context->GetDiagnostics())) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700353 return false;
354 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700355
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700356 if (!executor.Execute(xml::XmlActionExecutorPolicy::kWhitelist,
357 context->GetDiagnostics(), doc)) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700358 return false;
359 }
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700360
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700361 if (options_.rename_manifest_package) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700362 // Rename manifest package outside of the XmlActionExecutor.
Adam Lesinskie343eb12016-10-27 16:31:58 -0700363 // We need to extract the old package name and FullyQualify all class
364 // names.
Adam Lesinskice5e56e2016-10-21 17:56:45 -0700365 if (!RenameManifestPackage(options_.rename_manifest_package.value(),
366 root)) {
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700367 return false;
Adam Lesinskicc5609d2016-04-05 12:41:07 -0700368 }
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700369 }
370 return true;
Adam Lesinski2ae4a872015-11-02 16:10:55 -0800371}
372
Adam Lesinskicacb28f2016-10-19 12:18:14 -0700373} // namespace aapt