Reland "[graphite] Context::insertRecording returns more status info on failure"
This reverts commit 1bacd2930bdb62cf79c808fe3c4814832de38757.
Reason for revert: bool operator can't be explicit; was causing chrome to use the enum operator instead and then convert *that* to a truthy value
Bug: b/412570905
Original change's description:
> Revert "[graphite] Context::insertRecording returns more status info on failure"
>
> This reverts commit 5b4a6c99f50632255325bb09de161650ebb0f5bf.
>
> Reason for revert: breaking chrome roll?
>
> Bug: b/412570905
> Original change's description:
> > [graphite] Context::insertRecording returns more status info on failure
> >
> > Now returns an InsertStatus value, which is a class that mimics an
> > enum class but defines an operator bool() to ease migration.
> >
> > Bug: b/412570905
> > Change-Id: I0d7068f62151ca16867dd0be7a2c5cd1f3a18d78
> > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1009176
> > Reviewed-by: Thomas Smith <thomsmit@google.com>
> > Commit-Queue: Michael Ludwig <michaelludwig@google.com>
>
> Bug: b/412570905
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Change-Id: Ie7a4519d70c5205b642fc285fa34dc8ff1cbfecb
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1009951
> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Auto-Submit: Michael Ludwig <michaelludwig@google.com>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bug: b/412570905
Change-Id: I028735cefbf3c7ef413a669bd8c1f1a0177b25a3
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1011436
Reviewed-by: Thomas Smith <thomsmit@google.com>
Commit-Queue: Michael Ludwig <michaelludwig@google.com>
Auto-Submit: Michael Ludwig <michaelludwig@google.com>
6 files changed
