Get all tests passing with --all_incompatible_changes under Bazel 0.19

[brandjon]

#105 is one example of this.

[brandjon]

Aside from #105, here are the remaining flags that break us.

• --incompatible_disable_deprecated_attr_params
• --incompatible_disallow_data_transition
• --incompatible_disable_deprecated_attr_params
• --incompatible_disallow_filetype
• --incompatible_depset_is_not_iterable
• --incompatible_new_actions_api
• --incompatible_no_support_tools_in_action_inputs
• --incompatible_package_name_is_a_function

[brandjon]

Upstream fixes (will update this comment as I go):

• protobuf #5367
• skydoc #123
• rules_nodejs #421

[brandjon]

I'm going to have to put this down for now, no way I can migrate all the upstream deps. I'll tackle whichever ones are needed to keep our CI passing for the short-term future.