I was trying to use TestInProcessContextProvider in some unit tests (run cc_unittests --gtest_filter=OopPixelTest.DrawColor for one example), and I continually get this error in the logs:

[56111:56113:0302/140420.284366:1210019862049:ERROR:gl_context_virtual.cc(39)] Trying to make virtual context current without decoder.

It seems a lot like the decoder is always destroyed first and then in MakeCurrent, it prints this error because there's no decoder.  I'm not sure if we should rearrange these calls or maybe just not have this error log, but the current state of affairs doesn't seem right.

It looks like https://chromium-review.googlesource.com/929894 rearranged this shutdown code, so assigning to ccameron to investigate.