Do not share FBOs/VAOs/Transform feedback objects across contexts

gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc

Index: gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc
diff --git a/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc b/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc
index d913f37c53ef0927dc5c6b56b4bd58a278c9cb7a..a5b0812c8b3af51169b05143ecbbb8854dada4f3 100644
--- a/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc
+++ b/gpu/command_buffer/service/gles2_cmd_decoder_unittest_base.cc
@@ -542,6 +542,7 @@ void GLES2DecoderTestBase::ResetDecoder() {
   EXPECT_EQ(GL_NO_ERROR, GetGLError());
   if (!decoder_->WasContextLost()) {
     EXPECT_CALL(*gl_, DeleteBuffersARB(1, _)).Times(2).RetiresOnSaturation();
+    EXPECT_CALL(*gl_, DeleteFramebuffersEXT(1, _)).Times(AnyNumber());

[Chandan, 2017/05/18 13:33:36]

The no. of DeleteFramebuffers calls possible here is equal to the size of the
framebuffers_ map in FramebufferManager at the time of its destruction i.e. when
the map is cleared, which can be >=0. Therefore, I have added AnyNumber() as the
cardinality here.

[Zhenyao Mo, 2017/05/18 17:39:53]

If you set it to Times(1).RetiresOnSaturation(), are any tests failing?  If not,
I'd prefer that.  AnyNumber() is too allowing, so usually we avoid doing that if
we can.

[Chandan, 2017/05/18 18:10:09]

Yes, around 24 tests were failing for the reason that I have mentioned above.
For some test cases, no. of DeleteFramebuffers calls from
FramebufferManager::Destroy() was 0 since |framebuffers_| map size was 0. For
some it was 1 and for some it was 2. Therefore, Times(1).RetiresOnSaturation()
didn't help.

     if (group_->feature_info()->feature_flags().native_vertex_array_object) {
       EXPECT_CALL(*gl_,
                   DeleteVertexArraysOES(1, Pointee(kServiceVertexArrayId)))