Check that a transaction is opened before closing it

Check that a transaction is opened before closing it

The introduction of "Close All" provided a situation where we can close
a transaction that has not been opened which crashes the XPC service

BUG=698911
Review-Url: https://codereview.chromium.org/2735093004
Cr-Commit-Position: refs/heads/master@{#455273}
Committed: https://chromium.googlesource.com/chromium/src/+/a69a95ffd08ee6447a804526e1bf8c228f7a6952

[Miguel Garcia, 2017-03-07 16:55:22]

miguelg@chromium.org changed reviewers:
+ rsesek@chromium.org

[Robert Sesek, 2017-03-07 18:51:32]

LGTM

[Miguel Garcia, 2017-03-07 22:23:28]

The CQ bit was checked by miguelg@chromium.org

[commit-bot: I haz the power, 2017-03-07 22:24:20]

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

[commit-bot: I haz the power, 2017-03-07 23:01:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488925408217270, "parent_rev":
"ea37af717f1569dd93282e09b6fb49ff22b771f8", "commit_rev":
"a69a95ffd08ee6447a804526e1bf8c228f7a6952"}

[commit-bot: I haz the power, 2017-03-07 23:06:49]

Description was changed from

==========
Check that a transaction is opened before closing it

The introduction of "Close All" provided a situation where we can close
a transaction that has not been opened which crashes the XPC service

BUG=698911
==========

to

==========
Check that a transaction is opened before closing it

The introduction of "Close All" provided a situation where we can close
a transaction that has not been opened which crashes the XPC service

BUG=698911

Review-Url: https://codereview.chromium.org/2735093004
Cr-Commit-Position: refs/heads/master@{#455273}
Committed:
https://chromium.googlesource.com/chromium/src/+/a69a95ffd08ee6447a804526e1bf...
==========

[commit-bot: I haz the power, 2017-03-07 23:06:50]

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