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 411 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,

[enne (OOO), 2016/10/19 19:04:03]

Why did this need to change?

[dshwang, 2016/10/20 18:57:56]

It's the bug I mentioned above. without this change,
GLRenderer::EnqueueTextureQuad() don't know texture size. It's ok so far,
because the method doesn't use texture size. However, now we need texture size
to calculate one unit texel.

The reason is that ResourceProvider::CreateResourceFromTextureMailbox() set size
to Resource using size in TextureMailbox.
https://cs.chromium.org/chromium/src/cc/resources/resource_provider.cc?q=Crea...

I have a plan to fix it in next CL by replacing cc::TextureMailbox(3 params) to
cc::TextureMailbox(6 params) in all place.

+                                   false, false);
 
   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 689 matching lines @@
     default;
 
 void Canvas2DLayerBridge::Logger::reportHibernationEvent(
     HibernationEvent event) {
   DEFINE_STATIC_LOCAL(EnumerationHistogram, hibernationHistogram,
                       ("Canvas.HibernationEvents", HibernationEventCount));
   hibernationHistogram.count(event);
 }
 
 }  // namespace blink