<?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>103495</bug_id>
          
          <creation_ts>2012-11-28 02:08:02 -0800</creation_ts>
          <short_desc>[EFL][WK2][AC] Clear the viewport with white color</short_desc>
          <delta_ts>2012-11-28 05:23:31 -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 EFL</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</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="Joone Hur">joone</reporter>
          <assigned_to name="Joone Hur">joone</assigned_to>
          <cc>cdumez</cc>
    
    <cc>d-r</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kenneth</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>rakuco</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>yael</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>777657</commentid>
    <comment_count>0</comment_count>
    <who name="Joone Hur">joone</who>
    <bug_when>2012-11-28 02:08:02 -0800</bug_when>
    <thetext>When we run MiniBrowser, the default viewport color is black. 
We need to change the color with white.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777713</commentid>
    <comment_count>1</comment_count>
      <attachid>176448</attachid>
    <who name="Joone Hur">joone</who>
    <bug_when>2012-11-28 03:48:33 -0800</bug_when>
    <thetext>Created attachment 176448
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777719</commentid>
    <comment_count>2</comment_count>
      <attachid>176448</attachid>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2012-11-28 03:58:17 -0800</bug_when>
    <thetext>Comment on attachment 176448
Patch

Wouldn&apos;t transparent not be better?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777745</commentid>
    <comment_count>3</comment_count>
    <who name="Joone Hur">joone</who>
    <bug_when>2012-11-28 04:34:55 -0800</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 176448 [details])
&gt; Wouldn&apos;t transparent not be better?

Yes, maybe.
I checked the gtk+ port. It also clears the buffer with white color like this.
http://trac.webkit.org/browser/trunk/Source/WebKit2/WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp#L300</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777748</commentid>
    <comment_count>4</comment_count>
    <who name="Kenneth Rohde Christiansen">kenneth</who>
    <bug_when>2012-11-28 04:39:25 -0800</bug_when>
    <thetext>Sure but we are a mobile port and it should be possible to pan/pinch contents out of bounds and then have it bounce back again. In that case we don&apos;t want to show just white but be able to see the browser background below.

You can try the Qt MiniBrowser if you want to get an understanding of this. Maybe Thiago can show you, or you can try on an N9.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777754</commentid>
    <comment_count>5</comment_count>
      <attachid>176448</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-28 04:46:41 -0800</bug_when>
    <thetext>Comment on attachment 176448
Patch

Clearing flags on attachment: 176448

Committed r135997: &lt;http://trac.webkit.org/changeset/135997&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777755</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-11-28 04:46:45 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>777782</commentid>
    <comment_count>7</comment_count>
    <who name="Joone Hur">joone</who>
    <bug_when>2012-11-28 05:23:31 -0800</bug_when>
    <thetext>(In reply to comment #4)
&gt; Sure but we are a mobile port and it should be possible to pan/pinch contents out of bounds and then have it bounce back again. In that case we don&apos;t want to show just white but be able to see the browser background below.
&gt; 
&gt; You can try the Qt MiniBrowser if you want to get an understanding of this. Maybe Thiago can show you, or you can try on an N9.

I&apos;m not sure if all mobile browsers should have the same effect.
By the way, isn&apos;t this bounce effect an Apple patent?</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>176448</attachid>
            <date>2012-11-28 03:48:33 -0800</date>
            <delta_ts>2012-11-28 04:46:40 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-103495-20121128134528.patch</filename>
            <type>text/plain</type>
            <size>1498</size>
            <attacher name="Joone Hur">joone</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTM1OTY2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMWVjMmRlYWM5ZGFjZTlh
ZDg5N2FlNjQzZDNjMTdmNjBmMDMxY2M1OC4uZjBkMjMyMzc5OTlmNmNjYjBhMDk4OWI3ODhiZmQ2
NDI5NWFiNjhlMyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTExLTI4ICBKb29u
ZSBIdXIgIDxqb29uZS5odXJAaW50ZWwuY29tPgorCisgICAgICAgIFtFRkxdW1dLMl1bQUNdIENs
ZWFyIHRoZSB2aWV3cG9ydCB3aXRoIHdoaXRlIGNvbG9yCisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDM0OTUKKworICAgICAgICBXaGVuIGluaXRpYWxp
emluZyBhIEdMIHN1cmZhY2UsIHdlIHNob3VsZCBjbGVhciB0aGUgYnVmZmVyIHdpdGggd2hpdGUg
Y29sb3IuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
KiBVSVByb2Nlc3MvQVBJL2VmbC9Fd2tWaWV3SW1wbC5jcHA6CisgICAgICAgIChFd2tWaWV3SW1w
bDo6Y3JlYXRlR0xTdXJmYWNlKToKKwogMjAxMi0xMS0yNyAgSHVhbmcgRG9uZ3N1bmcgIDxsdXh0
ZWxsYUBjb21wYW55MTAwLm5ldD4KIAogICAgICAgICBSRUdSRVNTSU9OKHIxMzU2MjApOiBBU1NF
UlQobV9zdGF0ZS5kcmF3c0NvbnRlbnQgJiYgbV9zdGF0ZS5jb250ZW50c1Zpc2libGUgJiYgIW1f
c2l6ZS5pc0VtcHR5KCkpIGhpdHMgaW4gVGV4dHVyZU1hcHBlckxheWVyOjpwYWludFNlbGYoKS4K
ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL0V3a1ZpZXdJbXBs
LmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL0V3a1ZpZXdJbXBsLmNwcApp
bmRleCBjZTM4MjU0ZWIwMmIwNmRmYzA0YTkzZmFkMWU5MGI3NDFmNTZiODFhLi5kZTQwYThmOGNk
YzIwYmEzYmY0ODc4MmEzNDIyZGJlZDc4YWM0ZDdkIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9VSVByb2Nlc3MvQVBJL2VmbC9Fd2tWaWV3SW1wbC5jcHAKKysrIGIvU291cmNlL1dlYktpdDIv
VUlQcm9jZXNzL0FQSS9lZmwvRXdrVmlld0ltcGwuY3BwCkBAIC02NjUsNiArNjY1LDggQEAgYm9v
bCBFd2tWaWV3SW1wbDo6Y3JlYXRlR0xTdXJmYWNlKGNvbnN0IEludFNpemUmIHZpZXdTaXplKQog
CiAgICAgRXZhc19HTF9BUEkqIGdsID0gZXZhc19nbF9hcGlfZ2V0KGV2YXNHTCgpKTsKICAgICBn
bC0+Z2xWaWV3cG9ydCgwLCAwLCB2aWV3U2l6ZS53aWR0aCgpICsgc2QtPnZpZXcueCwgdmlld1Np
emUuaGVpZ2h0KCkgKyBzZC0+dmlldy55KTsKKyAgICBnbC0+Z2xDbGVhckNvbG9yKDEuMCwgMS4w
LCAxLjAsIDApOworICAgIGdsLT5nbENsZWFyKEdMX0NPTE9SX0JVRkZFUl9CSVQpOwogCiAgICAg
cmV0dXJuIHRydWU7CiB9Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>