Part 1: review in #35 2. method should inheritdoc since doc is on the interface.
Part 1: #35 comment part of 6. adding comment about what array intersect is doing. might want an exception when attempted to update things not allowed.
START PART 2:  https://www.drupal.org/node/2301273
apply patch https://www.drupal.org/files/issues/menu-2256521-MenuLinkTree-review-do-not-test.patch
convert createLink to addDefinition
docs fixes suggested by dawehner
Fix t() to $this->t()
some dawehner fixes
reapply review #15
some doc fixes
Part 2: 2301723 Added namespace in MenuParentFormSelector.php
START PART 3: https://www.drupal.org/node/2301313
apply patch https://www.drupal.org/files/issues/menu-2256521-MenuLinkContentUI-review-do-not-test.patch
rename updateLink to updateDefinition
START PART 4: https://www.drupal.org/node/2301317
apply patch https://www.drupal.org/files/issues/menu-2256521-conversion-review-do-not-test.patch
and resolve 8.x conflicts
fix hook_uninstall to use the correct service
rename updateLink to updateDefinition
rename deleteLink to removeDefinition
START PART 5: https://www.drupal.org/node/2301319
apply patch https://www.drupal.org/files/issues/menu-2256521-remove-dead-code.patch
and resolve conflicts with 8.x