<?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>95843</bug_id>
          
          <creation_ts>2012-09-05 05:29:09 -0700</creation_ts>
          <short_desc>[CMake] Add notifications path to the WebKit2/CMakeLists.txt</short_desc>
          <delta_ts>2012-09-06 05:28:59 -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>WebKit2</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>0</everconfirmed>
          <reporter name="Kihong Kwon">kihong.kwon</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>gyuyoung.kim</cc>
    
    <cc>rakuco</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>711973</commentid>
    <comment_count>0</comment_count>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-09-05 05:29:09 -0700</bug_when>
    <thetext>Add WebCore/Modules/Notifications path to the WebKit2/CMakeLists.txt.
Because there are some build errors when ENABLE_NOTIFICATIONS feature is enabled.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>711977</commentid>
    <comment_count>1</comment_count>
      <attachid>162227</attachid>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-09-05 05:31:26 -0700</bug_when>
    <thetext>Created attachment 162227
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712024</commentid>
    <comment_count>2</comment_count>
      <attachid>162227</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2012-09-05 07:21:16 -0700</bug_when>
    <thetext>Comment on attachment 162227
Patch

Looks good to me. 

Should the ChangeLog title include the [EFL] prefix as well ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712030</commentid>
    <comment_count>3</comment_count>
    <who name="Ryuan Choi">ryuan.choi</who>
    <bug_when>2012-09-05 07:31:00 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 162227 [details])
&gt; Looks good to me. 
&gt; 
&gt; Should the ChangeLog title include the [EFL] prefix as well ?

I think that it looks enough because CMakeLists.txt is not port specific. (although only Efl uses WebKit2 among the cmake ports)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712080</commentid>
    <comment_count>4</comment_count>
      <attachid>162227</attachid>
    <who name="Gyuyoung Kim">gyuyoung.kim</who>
    <bug_when>2012-09-05 08:27:21 -0700</bug_when>
    <thetext>Comment on attachment 162227
Patch

I also think this is not port specific. But,it looks [CMAKE] prefix is more useful.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712721</commentid>
    <comment_count>5</comment_count>
      <attachid>162384</attachid>
    <who name="Kihong Kwon">kihong.kwon</who>
    <bug_when>2012-09-05 18:01:33 -0700</bug_when>
    <thetext>Created attachment 162384
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>713188</commentid>
    <comment_count>6</comment_count>
      <attachid>162384</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-06 05:28:56 -0700</bug_when>
    <thetext>Comment on attachment 162384
Patch for landing

Clearing flags on attachment: 162384

Committed r127731: &lt;http://trac.webkit.org/changeset/127731&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>713189</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-06 05:28:59 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>162227</attachid>
            <date>2012-09-05 05:31:26 -0700</date>
            <delta_ts>2012-09-05 18:01:33 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-95843-20120905213108.patch</filename>
            <type>text/plain</type>
            <size>1417</size>
            <attacher name="Kihong Kwon">kihong.kwon</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTI3NTc4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggZmE4MDM4MWI3MWJiZWVk
NjVkYmNhZjg0YTE0Nzc2ODI2MWJhNDg3Ni4uYzg2Mjk5ZDVmNjc4NWI1M2NlNTc1MGZkMjFlNzRi
NGQ5NzY4YWY2YyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA5LTA1ICBLaWhv
bmcgS3dvbiAgPGtpaG9uZy5rd29uQHNhbXN1bmcuY29tPgorCisgICAgICAgIFtXSzJdIEFkZCBu
b3RpZmljYXRpb25zIHBhdGggdG8gdGhlIFdlYktpdDIvQ01ha2VMaXN0cy50eHQKKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk1ODQzCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgQWRkIFdlYkNvcmUvTW9kdWxl
cy9Ob3RpZmljYXRpb25zIHRvIHRoZSBXZWJLaXQyX0lOQ0xVREVfRElSRUNUT1JJRVMgb2YgV2Vi
S2l0Mi9DTWFrZUxpc3RzLnR4dC4KKyAgICAgICAgVGhlcmUgYXJlIHNvbWUgYnVpbGQgZXJyb3Jz
IHdoZW4gRU5BQkxFX05PVElGSUNBVElPTlMgZmVhdHVyZSBpcyBlbmFibGVkLgorCisgICAgICAg
ICogQ01ha2VMaXN0cy50eHQ6CisKIDIwMTItMDktMDUgIEthdXN0dWJoIEF0cmF3YWxrYXIgIDxr
YXVzdHViaEBtb3Rvcm9sYS5jb20+CiAKICAgICAgICAgW0RSVF0gTFRDOjogTW92ZSBwcmludGlu
ZyByZWxhdGVkIEFQSXMgZnJvbSBMYXlvdXRUZXN0Q29udHJvbGxlciB0byBJbnRlcm5hbHMKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NNYWtlTGlzdHMudHh0IGIvU291cmNlL1dlYktpdDIv
Q01ha2VMaXN0cy50eHQKaW5kZXggYzQzZGQ0YWM4ZWFmNjNlMmVmOTZkYzhjMzIyNTBiYTFhZjlm
MTgzOC4uZjJiYTc0NzFiZjRlN2FmZDdlYTViOGMyYWU2Y2M1YzUzMmI5NGQxNyAxMDA2NDQKLS0t
IGEvU291cmNlL1dlYktpdDIvQ01ha2VMaXN0cy50eHQKKysrIGIvU291cmNlL1dlYktpdDIvQ01h
a2VMaXN0cy50eHQKQEAgLTQ4LDYgKzQ4LDcgQEAgU0VUKFdlYktpdDJfSU5DTFVERV9ESVJFQ1RP
UklFUwogICAgICIke1dFQkNPUkVfRElSfS9Nb2R1bGVzL2JhdHRlcnkiCiAgICAgIiR7V0VCQ09S
RV9ESVJ9L01vZHVsZXMvaW50ZW50cyIKICAgICAiJHtXRUJDT1JFX0RJUn0vTW9kdWxlcy9uZXR3
b3JraW5mbyIKKyAgICAiJHtXRUJDT1JFX0RJUn0vTW9kdWxlcy9ub3RpZmljYXRpb25zIgogICAg
ICIke1dFQkNPUkVfRElSfS9Nb2R1bGVzL3ZpYnJhdGlvbiIKICAgICAiJHtXRUJDT1JFX0RJUn0v
YWNjZXNzaWJpbGl0eSIKICAgICAiJHtXRUJDT1JFX0RJUn0vYmluZGluZ3MvanMiCg==
</data>
<flag name="review"
          id="173262"
          type_id="1"
          status="+"
          setter="laszlo.gombos"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>162384</attachid>
            <date>2012-09-05 18:01:33 -0700</date>
            <delta_ts>2012-09-06 05:28:55 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>patch</filename>
            <type>text/plain</type>
            <size>1258</size>
            <attacher name="Kihong Kwon">kihong.kwon</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NNYWtlTGlzdHMudHh0IGIvU291cmNlL1dlYktp
dDIvQ01ha2VMaXN0cy50eHQKaW5kZXggYzQzZGQ0YS4uZjJiYTc0NyAxMDA2NDQKLS0tIGEvU291
cmNlL1dlYktpdDIvQ01ha2VMaXN0cy50eHQKKysrIGIvU291cmNlL1dlYktpdDIvQ01ha2VMaXN0
cy50eHQKQEAgLTQ4LDYgKzQ4LDcgQEAgU0VUKFdlYktpdDJfSU5DTFVERV9ESVJFQ1RPUklFUwog
ICAgICIke1dFQkNPUkVfRElSfS9Nb2R1bGVzL2JhdHRlcnkiCiAgICAgIiR7V0VCQ09SRV9ESVJ9
L01vZHVsZXMvaW50ZW50cyIKICAgICAiJHtXRUJDT1JFX0RJUn0vTW9kdWxlcy9uZXR3b3JraW5m
byIKKyAgICAiJHtXRUJDT1JFX0RJUn0vTW9kdWxlcy9ub3RpZmljYXRpb25zIgogICAgICIke1dF
QkNPUkVfRElSfS9Nb2R1bGVzL3ZpYnJhdGlvbiIKICAgICAiJHtXRUJDT1JFX0RJUn0vYWNjZXNz
aWJpbGl0eSIKICAgICAiJHtXRUJDT1JFX0RJUn0vYmluZGluZ3MvanMiCmRpZmYgLS1naXQgYS9T
b3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXgg
ZmE4MDM4MS4uY2YyYzU0MSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisr
KyBiL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA5LTA1
ICBLaWhvbmcgS3dvbiAgPGtpaG9uZy5rd29uQHNhbXN1bmcuY29tPgorCisgICAgICAgIFtDTWFr
ZV0gQWRkIG5vdGlmaWNhdGlvbnMgcGF0aCB0byB0aGUgV2ViS2l0Mi9DTWFrZUxpc3RzLnR4dAor
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTU4NDMKKwor
ICAgICAgICBSZXZpZXdlZCBieSBMYXN6bG8gR29tYm9zLgorCisgICAgICAgIEFkZCBXZWJDb3Jl
L01vZHVsZXMvTm90aWZpY2F0aW9ucyB0byB0aGUgV2ViS2l0Ml9JTkNMVURFX0RJUkVDVE9SSUVT
IG9mIFdlYktpdDIvQ01ha2VMaXN0cy50eHQuCisgICAgICAgIFRoZXJlIGFyZSBzb21lIGJ1aWxk
IGVycm9ycyB3aGVuIEVOQUJMRV9OT1RJRklDQVRJT05TIGZlYXR1cmUgaXMgZW5hYmxlZC4KKwor
ICAgICAgICAqIENNYWtlTGlzdHMudHh0OgorCiAyMDEyLTA5LTA1ICBLYXVzdHViaCBBdHJhd2Fs
a2FyICA8a2F1c3R1YmhAbW90b3JvbGEuY29tPgogCiAgICAgICAgIFtEUlRdIExUQzo6IE1vdmUg
cHJpbnRpbmcgcmVsYXRlZCBBUElzIGZyb20gTGF5b3V0VGVzdENvbnRyb2xsZXIgdG8gSW50ZXJu
YWxzCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>