Replace fetching control addresses with IsControllingAddress

[anorth]

A few other actors fetch the control addresses from the miner, then perform the ritual of resolving and comparing an address against them. Replace them with calls to the new IsControllingAddress method. Remove the existing accessor if possible.

[jennijuju]

@sudo-shashank will work on this!

[sudo-shashank]

@anorth where can I find more context on new IsControllingAddress?

[jennijuju]

builtin-actors/actors/miner/src/lib.rs

Lines 252 to 271 in 46415d2

	fn is_controlling_address(
	rt: &mut impl Runtime,
	params: IsControllingAddressParam,
	) -> Result<IsControllingAddressReturn, ActorError> {
	rt.validate_immediate_caller_accept_any()?;
	let input = match rt.resolve_address(&params.address) {
	Some(a) => Address::new_id(a),
	None => return Ok(IsControllingAddressReturn { is_controlling: false }),
	};
	let state: State = rt.state()?;
	let info = get_miner_info(rt.store(), &state)?;
	let is_controlling = info
	.control_addresses
	.iter()
	.chain(&[info.worker, info.owner])
	.into_iter()
	.any(|a| *a == input);
	Ok(IsControllingAddressReturn { is_controlling })
	}

[jennijuju]

your PR should be targeting next branch as well!