HW draw workarounds

gpu/command_buffer/service/gles2_cmd_decoder.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "gpu/command_buffer/service/gles2_cmd_decoder.h"
 
 #include <stdio.h>
 
 #include <algorithm>
 #include <list>
@@ skipping 2822 matching lines @@
     }
 
     return false;
   }
 
   ProcessFinishedAsyncTransfers();
   if (workarounds().flush_on_context_switch)
     glFlush();
 
   // Rebind the FBO if it was unbound by the context.
+  /*
   if (workarounds().unbind_fbo_on_context_switch)
     RestoreFramebufferBindings();

[no sievers, 2013/05/16 00:09:55]

This should only be set on Vivante gpus. Why did you need this? I'm just asking
in case the workaround introduced a bug (and is not limited as it should be
somehow) :)

[joth, 2013/05/16 00:20:44]

Seems it's being set on Imagination gpus too?

gpu/command_buffer/service/feature_info.cc
    if (is_vivante || is_imagination) {
      workarounds_.unbind_fbo_on_context_switch = true;
    }

*IIRC* without this commented out, you'd get one frame but then would just
render blank (even with virtual context)

+  */
 
   return true;
 }
 
 void GLES2DecoderImpl::ProcessFinishedAsyncTransfers() {
   if (engine() && query_manager_.get())
     query_manager_->ProcessPendingTransferQueries();
 
   // TODO(epenner): Is there a better place to do this?
   // This needs to occur before we execute any batch of commands
@@ skipping 7564 matching lines @@
   return error::kNoError;
 }
 
 // Include the auto-generated part of this file. We split this because it means
 // we can easily edit the non-auto generated parts right here in this file
 // instead of having to edit some template or the code generator.
 #include "gpu/command_buffer/service/gles2_cmd_decoder_autogen.h"
 
 }  // namespace gles2
 }  // namespace gpu