Make importers return new_next_id.

Though it is not currently the case, it is valid as per contract
that an importer returns modify_ids with empty entries. In this
case, archive builder can not save next_id appropriately to avoid
duplicated modifications.

This patch updates AbstractImporter and its implementations to
return new_next_id to allow such cases.

BUG=chromium:733103
TEST=bin/run_tests && bin/run_yapf -i && bin/run_lint

Change-Id: I8cda782b5a7c4b7389daefbe84a118bd91735d9b
Reviewed-on: https://chromium-review.googlesource.com/694921
Commit-Ready: Shuhei Takahashi <nya@chromium.org>
Tested-by: Shuhei Takahashi <nya@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
9 files changed