<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>129128</bug_id>
          
          <creation_ts>2014-02-20 16:16:55 -0800</creation_ts>
          <short_desc>WinCairo should use coordinated graphics and tiled backing store</short_desc>
          <delta_ts>2014-03-03 08:59:30 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows 8</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>INVALID</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alex Christensen">alex.christensen</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>benjamin</cc>
    
    <cc>bfulgham</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ossy</cc>
    
    <cc>pnormand</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>983029</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Christensen">alex.christensen</who>
    <bug_when>2014-02-20 16:16:55 -0800</bug_when>
    <thetext>Now that everybody is using accelerated compositing, WinCairo should also be using coordinated graphics and tiled backing store.  GStreamer video doesn&apos;t show up on the screen without them, and I imagine there are other features that use them, too.

On a related note, I got WinCairo compiling and running with FeatureDefines.props. instead of FeatureDefinesCairo.props.  The differences between these two files could cause problems, and using the same feature defines would unite the ports more.  Do you think this would be worth upstreaming?  Specifically ENABLE_3D_RENDERING, ENABLE_CSS_FILTERS, ENABLE_CSS_SHAPE_INSIDE, ENABLE_IFRAME_SEAMLESS, ENABLE_MEDIA_STATISTICS, ENABLE_REQUEST_ANIMATION_FRAME, ENABLE_SUBPIXEL_LAYOUT, ENABLE_TEMPLATE_ELEMENT, and ENABLE_VIEW_MODE_CSS_MEDIA are different.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983082</commentid>
    <comment_count>1</comment_count>
      <attachid>224824</attachid>
    <who name="Alex Christensen">alex.christensen</who>
    <bug_when>2014-02-20 20:08:45 -0800</bug_when>
    <thetext>Created attachment 224824
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983146</commentid>
    <comment_count>2</comment_count>
      <attachid>224824</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2014-02-21 01:00:05 -0800</bug_when>
    <thetext>Comment on attachment 224824
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=224824&amp;action=review

&gt; Source/WTF/wtf/Platform.h:864
&gt; +#define WTF_USE_COORDINATED_GRAPHICS 1

But coordinated graphics is WebKit2 only feature, why do you need it for a WebKit1 only port?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983410</commentid>
    <comment_count>3</comment_count>
    <who name="Alex Christensen">alex.christensen</who>
    <bug_when>2014-02-21 15:47:09 -0800</bug_when>
    <thetext>&gt; But coordinated graphics is WebKit2 only feature, why do you need it for a WebKit1 only port?
I don&apos;t know, but GStreamer doesn&apos;t draw to the screen without it and tiled backing store.  Maybe there&apos;s an unintentional dependency somewhere, or maybe I should put this in the #if ENABLE(VIDEO) &amp;&amp; PLATFORM(WIN_CAIRO) section instead.

This hasn&apos;t fixed WebGL on WinCairo, but should I need coordinated graphics for WebGL, now that accelerated compositing is mandatory?  WebGL on Windows hasn&apos;t worked since that big change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>983600</commentid>
    <comment_count>4</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2014-02-22 09:43:45 -0800</bug_when>
    <thetext>The current GStreamer AC backend doesn&apos;t support COORDINATED_GRAPHICS (afaik), and I second Ossy&apos;s question. I&apos;m a bit confused here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>986360</commentid>
    <comment_count>5</comment_count>
    <who name="Alex Christensen">alex.christensen</who>
    <bug_when>2014-03-03 08:59:06 -0800</bug_when>
    <thetext>This was a fluke.  Video works fine without this.  I&apos;m not sure why it seemed to help.  The only reason I can think of was that I was testing at my apartment with a very flaky internet connection.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>224824</attachid>
            <date>2014-02-20 20:08:45 -0800</date>
            <delta_ts>2014-03-03 08:59:30 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-129128-20140220210844.patch</filename>
            <type>text/plain</type>
            <size>1112</size>
            <attacher name="Alex Christensen">alex.christensen</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XVEYvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XVEYvQ2hh
bmdlTG9nCShyZXZpc2lvbiAxNjQ0NjUpCisrKyBTb3VyY2UvV1RGL0NoYW5nZUxvZwkod29ya2lu
ZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDE0LTAyLTIwICBBbGV4IENocmlzdGVuc2VuICA8
YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgVXNlIENvb3JkaW5hdGVkIEdyYXBo
aWNzIGFuZCBUaWxlZCBCYWNraW5nIFN0b3JlIG9uIFdpbkNhaXJvLgorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI5MTI4CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiB3dGYvUGxhdGZvcm0uaDoKKyAgICAg
ICAgQWRkZWQgZGVmaW5pdGlvbnMgdG8gdXNlIENPT1JESU5BVEVEX0dSQVBISUNTIGFuZCBUSUxF
RF9CQUNLSU5HX1NUT1JFIGluIFdpbkNhaXJvLgorCiAyMDE0LTAyLTIwICBDc2FiYSBPc3p0cm9n
b27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CiAKICAgICAgICAgQWRkIFN0YWNrU3RhdHMgc291cmNl
cyB0byBjbWFrZSBhbmQgYXV0b3Rvb2xzIGJ1aWxkIGZpbGVzCkluZGV4OiBTb3VyY2UvV1RGL3d0
Zi9QbGF0Zm9ybS5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XVEYvd3RmL1BsYXRmb3JtLmgJKHJl
dmlzaW9uIDE2NDQ1NSkKKysrIFNvdXJjZS9XVEYvd3RmL1BsYXRmb3JtLmgJKHdvcmtpbmcgY29w
eSkKQEAgLTg2MSw2ICs4NjEsOCBAQAogCiAjaWYgUExBVEZPUk0oV0lOX0NBSVJPKQogI2RlZmlu
ZSBXVEZfVVNFX1RFWFRVUkVfTUFQUEVSIDEKKyNkZWZpbmUgV1RGX1VTRV9DT09SRElOQVRFRF9H
UkFQSElDUyAxCisjZGVmaW5lIFdURl9VU0VfVElMRURfQkFDS0lOR19TVE9SRSAxCiAjZW5kaWYK
IAogI2lmIFVTRShURVhUVVJFX01BUFBFUikgJiYgVVNFKDNEX0dSQVBISUNTKSAmJiAhZGVmaW5l
ZChXVEZfVVNFX1RFWFRVUkVfTUFQUEVSX0dMKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>