Do not delete sub key if main key was not found in ChromeTestLauncherDelegate::PreSharding()

Do not delete sub key if main key was not found in ChromeTestLauncherDelegate::PreSharding()

Otherwise DeleteKey hits DCHECK(key_) when distrubution_key.Open() results
in ERROR_FILE_NOT_FOUND.

BUG=724350
Review-Url: https://codereview.chromium.org/2919523002
Cr-Commit-Position: refs/heads/master@{#475965}
Committed: https://chromium.googlesource.com/chromium/src/+/978ad0704f0ab11c422088c6b468aabdf10ff4b5

[gab, 2017-05-31 16:12:29]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-31 16:12:54]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[gab, 2017-05-31 16:13:19]

gab@chromium.org changed reviewers:
+ thakis@chromium.org

[gab, 2017-05-31 16:13:19]

Nico PTaL (and CQ), thanks!

[commit-bot: I haz the power, 2017-05-31 17:00:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-31 17:00:27]

Dry run: This issue passed the CQ dry run.

[Nico, 2017-05-31 17:32:51]

lgtm

What triggered this problem? Is this a recent regression?

[gab, 2017-05-31 17:33:11]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2017-05-31 17:33:32]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[gab, 2017-05-31 17:34:43]

On 2017/05/31 17:32:51, Nico (vacation Jun 3-11) wrote:
> lgtm
> 
> What triggered this problem? Is this a recent regression?

I wrote this code recently in a rush for https://crbug.com/721245.

[commit-bot: I haz the power, 2017-05-31 18:36:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496251991650570, "parent_rev":
"6bf9b5e003377e968f8f0c5e8b1cd6311701caf4", "commit_rev":
"978ad0704f0ab11c422088c6b468aabdf10ff4b5"}

[commit-bot: I haz the power, 2017-05-31 18:36:57]

Description was changed from

==========
Do not delete sub key if main key was not found in
ChromeTestLauncherDelegate::PreSharding()

Otherwise DeleteKey hits DCHECK(key_) when distrubution_key.Open() results
in ERROR_FILE_NOT_FOUND.

BUG=724350
==========

to

==========
Do not delete sub key if main key was not found in
ChromeTestLauncherDelegate::PreSharding()

Otherwise DeleteKey hits DCHECK(key_) when distrubution_key.Open() results
in ERROR_FILE_NOT_FOUND.

BUG=724350

Review-Url: https://codereview.chromium.org/2919523002
Cr-Commit-Position: refs/heads/master@{#475965}
Committed:
https://chromium.googlesource.com/chromium/src/+/978ad0704f0ab11c422088c6b468...
==========

[commit-bot: I haz the power, 2017-05-31 18:36:58]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/978ad0704f0ab11c422088c6b468...