<?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>116707</bug_id>
          
          <creation_ts>2013-05-24 00:31:10 -0700</creation_ts>
          <short_desc>[EFL][WK2] Remove unneeded WebCore namespace use in ewk</short_desc>
          <delta_ts>2013-05-27 23:48:20 -0700</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="Gyuyoung Kim">gyuyoung.kim</reporter>
          <assigned_to name="Gyuyoung Kim">gyuyoung.kim</assigned_to>
          <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>rakuco</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>893196</commentid>
    <comment_count>0</comment_count>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-24 00:31:10 -0700</bug_when>
    <thetext>SSIA</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893197</commentid>
    <comment_count>1</comment_count>
      <attachid>202778</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-24 00:32:19 -0700</bug_when>
    <thetext>Created attachment 202778
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893202</commentid>
    <comment_count>2</comment_count>
      <attachid>202778</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2013-05-24 00:36:44 -0700</bug_when>
    <thetext>Comment on attachment 202778
Patch

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

&gt; Source/WebKit2/UIProcess/API/efl/ewk_security_origin.cpp:33
&gt;  #include &lt;WebCore/SecurityOrigin.h&gt;

Could we remove this header include as well?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893203</commentid>
    <comment_count>3</comment_count>
      <attachid>202779</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-24 00:40:35 -0700</bug_when>
    <thetext>Created attachment 202779
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893204</commentid>
    <comment_count>4</comment_count>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-24 00:40:56 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 202778 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=202778&amp;action=review
&gt; 
&gt; &gt; Source/WebKit2/UIProcess/API/efl/ewk_security_origin.cpp:33
&gt; &gt;  #include &lt;WebCore/SecurityOrigin.h&gt;
&gt; 
&gt; Could we remove this header include as well?

Sure !</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893205</commentid>
    <comment_count>5</comment_count>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-24 00:43:26 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; (In reply to comment #2)
&gt; &gt; (From update of attachment 202778 [details] [details])
&gt; &gt; View in context: https://bugs.webkit.org/attachment.cgi?id=202778&amp;action=review
&gt; &gt; 
&gt; &gt; &gt; Source/WebKit2/UIProcess/API/efl/ewk_security_origin.cpp:33
&gt; &gt; &gt;  #include &lt;WebCore/SecurityOrigin.h&gt;
&gt; &gt; 
&gt; &gt; Could we remove this header include as well?
&gt; 
&gt; Sure !

Oops. EFL WK2 EWS is pending now. There was no problem when I remove it on my local source.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>893206</commentid>
    <comment_count>6</comment_count>
      <attachid>202779</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2013-05-24 00:52:18 -0700</bug_when>
    <thetext>Comment on attachment 202779
Patch

Looks fine, r=me. Please make sure it builds locally before landing as the ews is sluggish today.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>894229</commentid>
    <comment_count>7</comment_count>
      <attachid>202779</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-27 23:48:13 -0700</bug_when>
    <thetext>Comment on attachment 202779
Patch

Clearing flags on attachment: 202779

Committed r150787: &lt;http://trac.webkit.org/changeset/150787&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>894230</commentid>
    <comment_count>8</comment_count>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2013-05-27 23:48:20 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>202778</attachid>
            <date>2013-05-24 00:32:19 -0700</date>
            <delta_ts>2013-05-24 00:40:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-116707-20130524163103.patch</filename>
            <type>text/plain</type>
            <size>1858</size>
            <attacher name="Gyuyoung Kim">gyuyoung.kim</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUwNjI1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYjAxNGQwMzNhMjM4NTY4
MzgwMDZjNThmYWNhNzQ1YTQ1NDM2MjE3NC4uZjg0YjJmNzZkNDA1YTBkZTk4MTRlOGJjMDVjOWFj
NmYwYTVlOWQ1MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTA1LTI0ICBHeXV5
b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ac2Ftc3VuZy5jb20+CisKKyAgICAgICAgW0VGTF1bV0sy
XSBSZW1vdmUgdW5uZWVkZWQgV2ViQ29yZSBuYW1lc3BhY2UgdXNlIGluIGV3aworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE2NzA3CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL2Vm
bC9ld2tfc2VjdXJpdHlfb3JpZ2luLmNwcDogUmVtb3ZlICJ1c2luZyBuYW1lc3BhY2UgV2ViQ29y
ZSIuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHA6IGRp
dHRvLgorCiAyMDEzLTA1LTIzICBSeXVhbiBDaG9pICA8cnl1YW4uY2hvaUBzYW1zdW5nLmNvbT4K
IAogICAgICAgICBbV0syXVtFRkxdW1FUXSBCdWlsZCBicmVhayBhZnRlciByMTUwNjEwCmRpZmYg
LS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC9ld2tfc2VjdXJpdHlfb3Jp
Z2luLmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9v
cmlnaW4uY3BwCmluZGV4IDE4NGFkMmY0NjkwMzMyM2JkMzJhOGMyNzEwNjg5NGE2NWI0NDM2YjYu
LjA2MDMwYmFjNTcwMDBjNWFjZDQ4N2NlYjcwZmQ4MTA0MjM4MjJmYzUgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9vcmlnaW4uY3BwCisr
KyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9vcmlnaW4u
Y3BwCkBAIC0zMiw3ICszMiw2IEBACiAjaW5jbHVkZSAiZXdrX3NlY3VyaXR5X29yaWdpbl9wcml2
YXRlLmgiCiAjaW5jbHVkZSA8V2ViQ29yZS9TZWN1cml0eU9yaWdpbi5oPgogCi11c2luZyBuYW1l
c3BhY2UgV2ViQ29yZTsKIHVzaW5nIG5hbWVzcGFjZSBXZWJLaXQ7CiAKIEV3a1NlY3VyaXR5T3Jp
Z2luOjpFd2tTZWN1cml0eU9yaWdpbihXS1NlY3VyaXR5T3JpZ2luUmVmIG9yaWdpblJlZikKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a190ZXh0X2NoZWNr
ZXIuY3BwIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tl
ci5jcHAKaW5kZXggODlhMzgyZjYyNmIyY2Q1Y2JiMjViMjA1MjA0ZDk5NmNlOTYzN2QyOC4uOGY0
OTk0YjBmYWE3YWIzNTZiZWJiOGMwNWM5YjBmNjc4NTlkZGNkNyAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHAKKysrIGIvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHAKQEAgLTMw
LDcgKzMwLDYgQEAKICNpZiBFTkFCTEUoU1BFTExDSEVDSykKICNpbmNsdWRlICJUZXh0Q2hlY2tl
ckNsaWVudEVmbC5oIgogCi11c2luZyBuYW1lc3BhY2UgV2ViQ29yZTsKIHVzaW5nIG5hbWVzcGFj
ZSBXZWJLaXQ7CiAKIHN0YXRpYyBFaW5hX0xpc3QqIGNvbnZlcnRMYW5ndWFnZXNUb0VpbmFMaXN0
KGNvbnN0IFZlY3RvcjxTdHJpbmc+JiBsYW5ndWFnZXMpCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>202779</attachid>
            <date>2013-05-24 00:40:35 -0700</date>
            <delta_ts>2013-05-27 23:48:13 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-116707-20130524163919.patch</filename>
            <type>text/plain</type>
            <size>1912</size>
            <attacher name="Gyuyoung Kim">gyuyoung.kim</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUwNjI1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYjAxNGQwMzNhMjM4NTY4
MzgwMDZjNThmYWNhNzQ1YTQ1NDM2MjE3NC4uZjg0YjJmNzZkNDA1YTBkZTk4MTRlOGJjMDVjOWFj
NmYwYTVlOWQ1MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTA1LTI0ICBHeXV5
b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ac2Ftc3VuZy5jb20+CisKKyAgICAgICAgW0VGTF1bV0sy
XSBSZW1vdmUgdW5uZWVkZWQgV2ViQ29yZSBuYW1lc3BhY2UgdXNlIGluIGV3aworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE2NzA3CisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvQVBJL2Vm
bC9ld2tfc2VjdXJpdHlfb3JpZ2luLmNwcDogUmVtb3ZlICJ1c2luZyBuYW1lc3BhY2UgV2ViQ29y
ZSIuCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHA6IGRp
dHRvLgorCiAyMDEzLTA1LTIzICBSeXVhbiBDaG9pICA8cnl1YW4uY2hvaUBzYW1zdW5nLmNvbT4K
IAogICAgICAgICBbV0syXVtFRkxdW1FUXSBCdWlsZCBicmVhayBhZnRlciByMTUwNjEwCmRpZmYg
LS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC9ld2tfc2VjdXJpdHlfb3Jp
Z2luLmNwcCBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9v
cmlnaW4uY3BwCmluZGV4IDE4NGFkMmY0NjkwMzMyM2JkMzJhOGMyNzEwNjg5NGE2NWI0NDM2YjYu
LmI0NzZmNDQ0NDU2YTI1NDE2MjkxNjM1ZDZjMWVhMjA2YmI1ZDhiZGUgMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9vcmlnaW4uY3BwCisr
KyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a19zZWN1cml0eV9vcmlnaW4u
Y3BwCkBAIC0zMCw5ICszMCw3IEBACiAjaW5jbHVkZSAiV0tTZWN1cml0eU9yaWdpbi5oIgogI2lu
Y2x1ZGUgIldLU3RyaW5nLmgiCiAjaW5jbHVkZSAiZXdrX3NlY3VyaXR5X29yaWdpbl9wcml2YXRl
LmgiCi0jaW5jbHVkZSA8V2ViQ29yZS9TZWN1cml0eU9yaWdpbi5oPgogCi11c2luZyBuYW1lc3Bh
Y2UgV2ViQ29yZTsKIHVzaW5nIG5hbWVzcGFjZSBXZWJLaXQ7CiAKIEV3a1NlY3VyaXR5T3JpZ2lu
OjpFd2tTZWN1cml0eU9yaWdpbihXS1NlY3VyaXR5T3JpZ2luUmVmIG9yaWdpblJlZikKZGlmZiAt
LWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a190ZXh0X2NoZWNrZXIu
Y3BwIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5j
cHAKaW5kZXggODlhMzgyZjYyNmIyY2Q1Y2JiMjViMjA1MjA0ZDk5NmNlOTYzN2QyOC4uOGY0OTk0
YjBmYWE3YWIzNTZiZWJiOGMwNWM5YjBmNjc4NTlkZGNkNyAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHAKKysrIGIvU291cmNl
L1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvZXdrX3RleHRfY2hlY2tlci5jcHAKQEAgLTMwLDcg
KzMwLDYgQEAKICNpZiBFTkFCTEUoU1BFTExDSEVDSykKICNpbmNsdWRlICJUZXh0Q2hlY2tlckNs
aWVudEVmbC5oIgogCi11c2luZyBuYW1lc3BhY2UgV2ViQ29yZTsKIHVzaW5nIG5hbWVzcGFjZSBX
ZWJLaXQ7CiAKIHN0YXRpYyBFaW5hX0xpc3QqIGNvbnZlcnRMYW5ndWFnZXNUb0VpbmFMaXN0KGNv
bnN0IFZlY3RvcjxTdHJpbmc+JiBsYW5ndWFnZXMpCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>