clear up method parameters for _modify_rules

'table_name' is always used anyway, so there's no reason to have it as
optional especially as it's not perfectly clear that it's only needed
when current_lines is empty

'binary' was never used

Change-Id: I2c6bcf3fcff1c097d1f5c5ae34aa3428a94dabdc
Merge "clear up method parameters for _modify_rules"
Merge remote-tracking branch 'm_src_repo/master' into build-temp