Move sensitive field parceling bool to constructor
This addresses API review comments recommending to use a copy constructor with additional parameters instead of a dedicated method. makeSensitiveFieldsParcelingCopy becomes LinkProperties(base, true). Bug: 150877475 Test: atest FrameworksNetTests NetworkStackTests NetworkStackNextTests Change-Id: Ib145ca7f36dcbee6ef47d09862a181fc04a28f03
This commit is contained in:
@@ -1028,7 +1028,8 @@ public class LinkPropertiesTest {
|
||||
source.setCaptivePortalApiUrl(CAPPORT_API_URL);
|
||||
source.setCaptivePortalData((CaptivePortalData) getCaptivePortalData());
|
||||
source.setDhcpServerAddress((Inet4Address) GATEWAY1);
|
||||
assertParcelSane(source.makeSensitiveFieldsParcelingCopy(), 18 /* fieldCount */);
|
||||
assertParcelSane(new LinkProperties(source, true /* parcelSensitiveFields */),
|
||||
18 /* fieldCount */);
|
||||
|
||||
// Verify that without using a sensitiveFieldsParcelingCopy, sensitive fields are cleared.
|
||||
final LinkProperties sanitized = new LinkProperties(source);
|
||||
|
||||
Reference in New Issue
Block a user