get rid of redundant should_record_bridge_info() call in options_act()

In options_act() we have

    if ((!old_options || !old_options->EntryStatistics) &&
        options->EntryStatistics && !should_record_bridge_info(options)) {

But right above that we have

    /* Only collect other relay-only statistics on relays. */
    if (!public_server_mode(options)) {
      options->CellStatistics = 0;
      options->EntryStatistics = 0;

So the only way EntryStatistics could be non-zero at that lower point is if public_server_mode() is true.

So the check to !should_record_bridge_info() will always be true, and is redundant to check.