<?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>91914</bug_id>
          
          <creation_ts>2012-07-20 18:16:56 -0700</creation_ts>
          <short_desc>[EFL] Remove unused headers from ewk_security_origin</short_desc>
          <delta_ts>2012-07-20 20:58:47 -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>All</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>0</everconfirmed>
          <reporter name="Kihong Kwon">kihong.kwon</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>gyuyoung.kim</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>rakuco</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>675128</commentid>
    <comment_count>0</comment_count>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-07-20 18:16:56 -0700</bug_when>
    <thetext>There are some unused header in the ewk_security_origin.h/cpp
We need to remove them.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675140</commentid>
    <comment_count>1</comment_count>
      <attachid>153638</attachid>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-07-20 18:55:20 -0700</bug_when>
    <thetext>Created attachment 153638
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675145</commentid>
    <comment_count>2</comment_count>
      <attachid>153638</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-07-20 19:02:37 -0700</bug_when>
    <thetext>Comment on attachment 153638
Patch

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

I don&apos;t think we can land any patchess without review except for gardening and urgent build break, crash problems

&gt; Source/WebKit/efl/ChangeLog:6
&gt; +        Unreviewed EFL gardening.

This patch is not gardening.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675146</commentid>
    <comment_count>3</comment_count>
      <attachid>153640</attachid>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-07-20 19:07:04 -0700</bug_when>
    <thetext>Created attachment 153640
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675147</commentid>
    <comment_count>4</comment_count>
      <attachid>153640</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-07-20 19:08:52 -0700</bug_when>
    <thetext>Comment on attachment 153640
Patch

LGTM</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675148</commentid>
    <comment_count>5</comment_count>
      <attachid>153640</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-07-20 19:11:17 -0700</bug_when>
    <thetext>Comment on attachment 153640
Patch

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

&gt; Source/WebKit/efl/ewk/ewk_security_origin.h:36
&gt; +#include &lt;Eina.h&gt;

It looks we don&apos;t need to move to .h file.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675149</commentid>
    <comment_count>6</comment_count>
      <attachid>153640</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-07-20 19:20:25 -0700</bug_when>
    <thetext>Comment on attachment 153640
Patch

Eina_List is being used by ewk_security_origin.h.  It looks Eina.h files has included by include chain so far. So, there was a little misunderstanding.

LGTM now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675158</commentid>
    <comment_count>7</comment_count>
      <attachid>153640</attachid>
    <who name="Noam Rosenthal">noam</who>
    <bug_when>2012-07-20 20:02:06 -0700</bug_when>
    <thetext>Comment on attachment 153640
Patch

rs=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675163</commentid>
    <comment_count>8</comment_count>
      <attachid>153640</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-07-20 20:58:42 -0700</bug_when>
    <thetext>Comment on attachment 153640
Patch

Clearing flags on attachment: 153640

Committed r123287: &lt;http://trac.webkit.org/changeset/123287&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>675164</commentid>
    <comment_count>9</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-07-20 20:58:47 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>153638</attachid>
            <date>2012-07-20 18:55:20 -0700</date>
            <delta_ts>2012-07-20 19:06:56 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-91914-20120721105515.patch</filename>
            <type>text/plain</type>
            <size>1941</size>
            <attacher name="Kihong Kwon">kihong.kwon</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTIzMjgxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Vm
bC9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0L2VmbC9DaGFuZ2VMb2cKaW5kZXggM2U3YjU0Nzkz
ZTk5MjZkYWQxODg5ODNlMzRlZDM2NjM0NjU3MDVkYi4uY2E2OTI5NTJlZDJjYTA5ZTdkZjJlMmI5
ODQ0ODlhMzRjYzY3MTFlZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9lZmwvQ2hhbmdlTG9n
CisrKyBiL1NvdXJjZS9XZWJLaXQvZWZsL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEy
LTA3LTIwICBLaWhvbmcgS3dvbiAgPGtpaG9uZy5rd29uQHNhbXN1bmcuY29tPgorCisgICAgICAg
IFtFRkxdIFJlbW92ZSB1bnVzZWQgaGVhZGVycyBmcm9tIGV3a19zZWN1cml0eV9vcmlnaW4KKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkxOTE0CisKKyAg
ICAgICAgVW5yZXZpZXdlZCBFRkwgZ2FyZGVuaW5nLgorCisgICAgICAgIFRoZXJlIGFyZSBzb21l
IHVudXNlZCBoZWFkZXIgaW4gdGhlIGV3a19zZWN1cml0eV9vcmlnaW4uaC9jcHAKKyAgICAgICAg
V2UgbmVlZCB0byByZW1vdmUgdGhlbS4KKworICAgICAgICAqIGV3ay9ld2tfc2VjdXJpdHlfb3Jp
Z2luLmNwcDoKKyAgICAgICAgKiBld2svZXdrX3NlY3VyaXR5X29yaWdpbi5oOgorCiAyMDEyLTA3
LTIwICBDaHJpc3RvcGhlIER1bWV6ICA8Y2hyaXN0b3BoZS5kdW1lekBpbnRlbC5jb20+CiAKICAg
ICAgICAgW0VGTF0gUHJveHkgY29uZmlndXJhdGlvbiBzaG91bGQgaG9ub3IgdGhlIG5vX3Byb3h5
IGVudmlyb25tZW50IHZhcmlhYmxlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2VmbC9ld2sv
ZXdrX3NlY3VyaXR5X29yaWdpbi5jcHAgYi9Tb3VyY2UvV2ViS2l0L2VmbC9ld2svZXdrX3NlY3Vy
aXR5X29yaWdpbi5jcHAKaW5kZXggZDdlMzA0YzFhYzQ5NzM3MTYxZDFjNjAyOGE5ZmNiNDQwMTc0
MDJmZC4uMWI2ODg0ZThjMTk3ZmU4NjhlYmZiZTA3NjBmNDY2NzhjYzcwNDUzYyAxMDA2NDQKLS0t
IGEvU291cmNlL1dlYktpdC9lZmwvZXdrL2V3a19zZWN1cml0eV9vcmlnaW4uY3BwCisrKyBiL1Nv
dXJjZS9XZWJLaXQvZWZsL2V3ay9ld2tfc2VjdXJpdHlfb3JpZ2luLmNwcApAQCAtMjQsMTEgKzI0
LDkgQEAKICNpbmNsdWRlICJBcHBsaWNhdGlvbkNhY2hlU3RvcmFnZS5oIgogI2luY2x1ZGUgIkRh
dGFiYXNlVHJhY2tlci5oIgogI2luY2x1ZGUgIlNlY3VyaXR5T3JpZ2luLmgiCi0jaW5jbHVkZSAi
ZXdrX3ByaXZhdGUuaCIKICNpbmNsdWRlICJld2tfc2VjdXJpdHlfb3JpZ2luX3ByaXZhdGUuaCIK
ICNpbmNsdWRlICJld2tfd2ViX2RhdGFiYXNlLmgiCiAjaW5jbHVkZSAiZXdrX3dlYl9kYXRhYmFz
ZV9wcml2YXRlLmgiCi0jaW5jbHVkZSA8RWluYS5oPgogI2luY2x1ZGUgPHd0Zi9SZWZQdHIuaD4K
ICNpbmNsdWRlIDx3dGYvVW51c2VkUGFyYW0uaD4KICNpbmNsdWRlIDx3dGYvdGV4dC9DU3RyaW5n
Lmg+CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2VmbC9ld2svZXdrX3NlY3VyaXR5X29yaWdp
bi5oIGIvU291cmNlL1dlYktpdC9lZmwvZXdrL2V3a19zZWN1cml0eV9vcmlnaW4uaAppbmRleCA2
ZTA0YzkyMWJiZmNhYzM1NDc1MGFmYmZmMTk5MWFjMTI3YmJmNjIyLi45OTk2OTcyZDcxYjU2M2Fi
NDJhYjY0NmEwNDlhYTkyN2JjOTBlZWZjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0L2VmbC9l
d2svZXdrX3NlY3VyaXR5X29yaWdpbi5oCisrKyBiL1NvdXJjZS9XZWJLaXQvZWZsL2V3ay9ld2tf
c2VjdXJpdHlfb3JpZ2luLmgKQEAgLTMzLDcgKzMzLDcgQEAKICNpZm5kZWYgZXdrX3NlY3VyaXR5
X29yaWdpbl9oCiAjZGVmaW5lIGV3a19zZWN1cml0eV9vcmlnaW5faAogCi0jaW5jbHVkZSA8RXZh
cy5oPgorI2luY2x1ZGUgPEVpbmEuaD4KIAogI2lmZGVmIF9fY3BsdXNwbHVzCiBleHRlcm4gIkMi
IHsK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>153640</attachid>
            <date>2012-07-20 19:07:04 -0700</date>
            <delta_ts>2012-07-20 20:58:42 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-91914-20120721110659.patch</filename>
            <type>text/plain</type>
            <size>1943</size>
            <attacher name="Kihong Kwon">kihong.kwon</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTIzMjgxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2Vm
bC9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0L2VmbC9DaGFuZ2VMb2cKaW5kZXggM2U3YjU0Nzkz
ZTk5MjZkYWQxODg5ODNlMzRlZDM2NjM0NjU3MDVkYi4uMTk3ZmY3ZTcxZjE3ZmM2NWU5MjhiZjk2
OThlY2I5N2FiMDY2OTFkZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9lZmwvQ2hhbmdlTG9n
CisrKyBiL1NvdXJjZS9XZWJLaXQvZWZsL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEy
LTA3LTIwICBLaWhvbmcgS3dvbiAgPGtpaG9uZy5rd29uQHNhbXN1bmcuY29tPgorCisgICAgICAg
IFtFRkxdIFJlbW92ZSB1bnVzZWQgaGVhZGVycyBmcm9tIGV3a19zZWN1cml0eV9vcmlnaW4KKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTkxOTE0CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVGhlcmUgYXJlIHNv
bWUgdW51c2VkIGhlYWRlciBpbiB0aGUgZXdrX3NlY3VyaXR5X29yaWdpbi5oL2NwcAorICAgICAg
ICBXZSBuZWVkIHRvIHJlbW92ZSB0aGVtLgorCisgICAgICAgICogZXdrL2V3a19zZWN1cml0eV9v
cmlnaW4uY3BwOgorICAgICAgICAqIGV3ay9ld2tfc2VjdXJpdHlfb3JpZ2luLmg6CisKIDIwMTIt
MDctMjAgIENocmlzdG9waGUgRHVtZXogIDxjaHJpc3RvcGhlLmR1bWV6QGludGVsLmNvbT4KIAog
ICAgICAgICBbRUZMXSBQcm94eSBjb25maWd1cmF0aW9uIHNob3VsZCBob25vciB0aGUgbm9fcHJv
eHkgZW52aXJvbm1lbnQgdmFyaWFibGUKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZWZsL2V3
ay9ld2tfc2VjdXJpdHlfb3JpZ2luLmNwcCBiL1NvdXJjZS9XZWJLaXQvZWZsL2V3ay9ld2tfc2Vj
dXJpdHlfb3JpZ2luLmNwcAppbmRleCBkN2UzMDRjMWFjNDk3MzcxNjFkMWM2MDI4YTlmY2I0NDAx
NzQwMmZkLi4xYjY4ODRlOGMxOTdmZTg2OGViZmJlMDc2MGY0NjY3OGNjNzA0NTNjIDEwMDY0NAot
LS0gYS9Tb3VyY2UvV2ViS2l0L2VmbC9ld2svZXdrX3NlY3VyaXR5X29yaWdpbi5jcHAKKysrIGIv
U291cmNlL1dlYktpdC9lZmwvZXdrL2V3a19zZWN1cml0eV9vcmlnaW4uY3BwCkBAIC0yNCwxMSAr
MjQsOSBAQAogI2luY2x1ZGUgIkFwcGxpY2F0aW9uQ2FjaGVTdG9yYWdlLmgiCiAjaW5jbHVkZSAi
RGF0YWJhc2VUcmFja2VyLmgiCiAjaW5jbHVkZSAiU2VjdXJpdHlPcmlnaW4uaCIKLSNpbmNsdWRl
ICJld2tfcHJpdmF0ZS5oIgogI2luY2x1ZGUgImV3a19zZWN1cml0eV9vcmlnaW5fcHJpdmF0ZS5o
IgogI2luY2x1ZGUgImV3a193ZWJfZGF0YWJhc2UuaCIKICNpbmNsdWRlICJld2tfd2ViX2RhdGFi
YXNlX3ByaXZhdGUuaCIKLSNpbmNsdWRlIDxFaW5hLmg+CiAjaW5jbHVkZSA8d3RmL1JlZlB0ci5o
PgogI2luY2x1ZGUgPHd0Zi9VbnVzZWRQYXJhbS5oPgogI2luY2x1ZGUgPHd0Zi90ZXh0L0NTdHJp
bmcuaD4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZWZsL2V3ay9ld2tfc2VjdXJpdHlfb3Jp
Z2luLmggYi9Tb3VyY2UvV2ViS2l0L2VmbC9ld2svZXdrX3NlY3VyaXR5X29yaWdpbi5oCmluZGV4
IDZlMDRjOTIxYmJmY2FjMzU0NzUwYWZiZmYxOTkxYWMxMjdiYmY2MjIuLjk5OTY5NzJkNzFiNTYz
YWI0MmFiNjQ2YTA0OWFhOTI3YmM5MGVlZmMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvZWZs
L2V3ay9ld2tfc2VjdXJpdHlfb3JpZ2luLmgKKysrIGIvU291cmNlL1dlYktpdC9lZmwvZXdrL2V3
a19zZWN1cml0eV9vcmlnaW4uaApAQCAtMzMsNyArMzMsNyBAQAogI2lmbmRlZiBld2tfc2VjdXJp
dHlfb3JpZ2luX2gKICNkZWZpbmUgZXdrX3NlY3VyaXR5X29yaWdpbl9oCiAKLSNpbmNsdWRlIDxF
dmFzLmg+CisjaW5jbHVkZSA8RWluYS5oPgogCiAjaWZkZWYgX19jcGx1c3BsdXMKIGV4dGVybiAi
QyIgewo=
</data>

          </attachment>
      

    </bug>

</bugzilla>