Fix HTML5 video blurry

third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 414 matching lines @@
     // there is no need for animations to be double buffered.
     mailboxInfo.m_image.reset();
   } else {
     // FIXME: We'd rather insert a syncpoint than perform a flush here,
     // but currently the canvas will flicker if we don't flush here.
     const GLuint64 fenceSync = gl->InsertFenceSyncCHROMIUM();
     gl->Flush();
     gl->GenSyncTokenCHROMIUM(fenceSync, syncToken.GetData());
   }
   mailboxInfo.m_mailbox = mailbox;
-  *outMailbox = cc::TextureMailbox(mailbox, syncToken, GL_TEXTURE_2D);
+  *outMailbox = cc::TextureMailbox(mailbox, syncToken, GL_TEXTURE_2D, m_size,
+                                   false, false);

[Justin Novosad, 2016/10/25 21:28:16]

These "false" literals are bad for code readability.  Please use temporary
variables.
Ex: 
bool isOverlayCandidate = false;
bool secureOutputOnly = false;
*outMailbox = cc::TextureMailbox( ... , isOverlayCandidate, secureOutputOnly);

[dshwang, 2016/10/26 15:26:28]

Done. Could you review again?

In addition, do you know how to run
Pixel_OffscreenCanvasUnaccelerated2DGPUCompositing test?
It should belong to gpu test, but Linux chromium doesn't support it..
content/test/gpu/gpu_tests/pixel_test_pages.py:      base_name +
'_OffscreenCanvasUnaccelerated2DGPUCompositing',
Available benchmarks for release are:
  cloud_storage_test_base.CloudStorageTestBase 
  device_trace_test                            Tests GPU Device traces show up
on devices that support it.
  gpu_process                                  Tests that accelerated content
triggers the creation of a GPU process
  gpu_rasterization                            Tests that GPU rasterization
produces valid content
  gpu_test_base.TestBase                       Base classes for all GPU tests in
this directory. Implements
  gpu_test_base_unittest.FakeTest              
  hardware_accelerated_feature                 Tests GPU acceleration is
reported as active for various features
  maps                                         Google Maps pixel tests.
  screenshot_sync                              Tests that screenhots are
properly synchronized with the frame one which
  trace_test                                   Tests GPU traces are plumbed
through properly.
  trace_test.TraceTestBase                     Base class for the trace tests.

 
   gl->BindTexture(GL_TEXTURE_2D, 0);
   // Because we are changing the texture binding without going through skia,
   // we must dirty the context.
   grContext->resetContext(kTextureBinding_GrGLBackendState);
   return true;
 }
 
 void Canvas2DLayerBridge::resetSkiaTextureBinding() {
   GrContext* grContext = m_contextProvider->grContext();
@@ skipping 690 matching lines @@
     default;
 
 void Canvas2DLayerBridge::Logger::reportHibernationEvent(
     HibernationEvent event) {
   DEFINE_STATIC_LOCAL(EnumerationHistogram, hibernationHistogram,
                       ("Canvas.HibernationEvents", HibernationEventCount));
   hibernationHistogram.count(event);
 }
 
 }  // namespace blink