<?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>37442</bug_id>
          
          <creation_ts>2010-04-12 01:20:31 -0700</creation_ts>
          <short_desc>[PATCH] [Qt] Fix compilation with system tray disabled.</short_desc>
          <delta_ts>2010-04-26 09:07:36 -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 Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Ismail Donmez">ismail</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>cshu</cc>
    
    <cc>hausmann</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>211570</commentid>
    <comment_count>0</comment_count>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:20:31 -0700</bug_when>
    <thetext>Notification support should be disabled if QT_NO_SYSTEMTRAYICON is defined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211571</commentid>
    <comment_count>1</comment_count>
      <attachid>53158</attachid>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:22:04 -0700</bug_when>
    <thetext>Created attachment 53158
Compile if QT_NO_SYSTEMTRAYICON is defined</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211572</commentid>
    <comment_count>2</comment_count>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:25:40 -0700</bug_when>
    <thetext>Patch missing some more checks. I will fix it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211574</commentid>
    <comment_count>3</comment_count>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:34:44 -0700</bug_when>
    <thetext>WebCore::ChromeClient::notificationPresenter() is abstract, it should be implemented in all conditions.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211575</commentid>
    <comment_count>4</comment_count>
      <attachid>53159</attachid>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:36:05 -0700</bug_when>
    <thetext>Created attachment 53159
Implement WebCore::ChromeClient::notificationPresenter() in all cases</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211576</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-04-12 01:37:50 -0700</bug_when>
    <thetext>Attachment 53159 did not pass style-queue:

Failed to run &quot;WebKitTools/Scripts/check-webkit-style&quot; exit_code: 1
WARNING: File exempt from style guide. Skipping: &quot;WebKit/qt/Api/qwebpage.cpp&quot;
WebKit/qt/WebCoreSupport/ChromeClientQt.cpp:482:  Use 0 instead of NULL.  [readability/null] [5]
Total errors found: 1 in 6 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211577</commentid>
    <comment_count>6</comment_count>
      <attachid>53160</attachid>
    <who name="Ismail Donmez">ismail</who>
    <bug_when>2010-04-12 01:38:58 -0700</bug_when>
    <thetext>Created attachment 53160
Fix style error</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>211928</commentid>
    <comment_count>7</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-04-13 06:58:49 -0700</bug_when>
    <thetext>Wouldn&apos;t it be better to disable ENABLE(NOTIFICATIONS) if there&apos;s no system tray icon class instead of adding another condition to every use of ENABLE(NOTIFICATIONS)?

It may be easiest to do that in WebCore/config.h</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>212268</commentid>
    <comment_count>8</comment_count>
      <attachid>53160</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-04-13 23:32:37 -0700</bug_when>
    <thetext>Comment on attachment 53160
Fix style error

I agree with Simon&apos;s proposal, I&apos;d like to give an r- this time. I similar mechanism exists to disable certain WebKit features if sqlite is not available.

Would the following work ..

contains(DEFINES, QT_NO_SYSTEMTRAYICON):DEFINES += ENABLE_NOTIFICATIONS=0</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>213904</commentid>
    <comment_count>9</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-04-18 21:33:39 -0700</bug_when>
    <thetext>*** Bug 37716 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214005</commentid>
    <comment_count>10</comment_count>
    <who name="Chang Shu">cshu</who>
    <bug_when>2010-04-19 06:45:25 -0700</bug_when>
    <thetext>(In reply to comment #8)
&gt; (From update of attachment 53160 [details])
&gt; I agree with Simon&apos;s proposal, I&apos;d like to give an r- this time. I similar
&gt; mechanism exists to disable certain WebKit features if sqlite is not available.
&gt; 
&gt; Would the following work ..
&gt; 
&gt; contains(DEFINES, QT_NO_SYSTEMTRAYICON):DEFINES += ENABLE_NOTIFICATIONS=0

I don&apos;t think the pri file is able to detect if QT_NO_SYSTEMTRAYICON is
defined.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214705</commentid>
    <comment_count>11</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-04-20 09:49:53 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #8)
&gt; &gt; (From update of attachment 53160 [details] [details])
&gt; &gt; I agree with Simon&apos;s proposal, I&apos;d like to give an r- this time. I similar
&gt; &gt; mechanism exists to disable certain WebKit features if sqlite is not available.
&gt; &gt; 
&gt; &gt; Would the following work ..
&gt; &gt; 
&gt; &gt; contains(DEFINES, QT_NO_SYSTEMTRAYICON):DEFINES += ENABLE_NOTIFICATIONS=0
&gt; 
&gt; I don&apos;t think the pri file is able to detect if QT_NO_SYSTEMTRAYICON is
&gt; defined.

Chang, you&apos;re right that unless QT_NO_SYSTEMTRAYICON is explicitly listed in the DEFINES qmake will not pick it up as it is only defined in a Qt header.

A compromise would be not to disable the ENABLE_NOTIFICATIONS feature but only guard the SysTray dependent code with SYSTEMTRAYICON. I think in the future we should allow QtWebKit clients to render the notification themselves.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214712</commentid>
    <comment_count>12</comment_count>
    <who name="Chang Shu">cshu</who>
    <bug_when>2010-04-20 09:59:35 -0700</bug_when>
    <thetext>&gt; &gt; &gt; contains(DEFINES, QT_NO_SYSTEMTRAYICON):DEFINES += ENABLE_NOTIFICATIONS=0
&gt; &gt; 
&gt; &gt; I don&apos;t think the pri file is able to detect if QT_NO_SYSTEMTRAYICON is
&gt; &gt; defined.
&gt; 
&gt; Chang, you&apos;re right that unless QT_NO_SYSTEMTRAYICON is explicitly listed in
&gt; the DEFINES qmake will not pick it up as it is only defined in a Qt header.
&gt; 
&gt; A compromise would be not to disable the ENABLE_NOTIFICATIONS feature but only
&gt; guard the SysTray dependent code with SYSTEMTRAYICON. I think in the future we
&gt; should allow QtWebKit clients to render the notification themselves.

I agree to guard the systemtrayicon code as we don&apos;t have a general solution and in this case, it&apos;s even a better solution as you have indicated for future enhancement. I am working on a patch if you guys don&apos;t mind.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214778</commentid>
    <comment_count>13</comment_count>
      <attachid>53847</attachid>
    <who name="Chang Shu">cshu</who>
    <bug_when>2010-04-20 11:08:12 -0700</bug_when>
    <thetext>Created attachment 53847
fix patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>214786</commentid>
    <comment_count>14</comment_count>
      <attachid>53847</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-04-20 11:17:09 -0700</bug_when>
    <thetext>Comment on attachment 53847
fix patch

Thanks Chang!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>215269</commentid>
    <comment_count>15</comment_count>
      <attachid>53847</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-04-21 06:08:22 -0700</bug_when>
    <thetext>Comment on attachment 53847
fix patch

Clearing flags on attachment: 53847

Committed r57972: &lt;http://trac.webkit.org/changeset/57972&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>215270</commentid>
    <comment_count>16</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-04-21 06:08:30 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>216973</commentid>
    <comment_count>17</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-04-26 03:39:45 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; (From update of attachment 53847 [details])
&gt; Clearing flags on attachment: 53847
&gt; 
&gt; Committed r57972: &lt;http://trac.webkit.org/changeset/57972&gt;

This patch is scheduled for integration into the 2.0 branch, but the earlier patches that implement this feature are not part of the branch.

Do we really need to include another feature in the release branch?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>217066</commentid>
    <comment_count>18</comment_count>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-04-26 06:50:14 -0700</bug_when>
    <thetext>Simon, this is not needed for the 2.0 branch. If I read the history of the bug correctly, you added the 35784 blocker to this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>217128</commentid>
    <comment_count>19</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2010-04-26 09:07:36 -0700</bug_when>
    <thetext>(In reply to comment #18)
&gt; Simon, this is not needed for the 2.0 branch. If I read the history of the bug
&gt; correctly, you added the 35784 blocker to this.

Hehe, thanks *gets the brown paper bag*</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53158</attachid>
            <date>2010-04-12 01:22:04 -0700</date>
            <delta_ts>2010-04-12 01:36:05 -0700</delta_ts>
            <desc>Compile if QT_NO_SYSTEMTRAYICON is defined</desc>
            <filename>notification.patch</filename>
            <type>text/plain</type>
            <size>2619</size>
            <attacher name="Ismail Donmez">ismail</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2UuY3BwIGIvV2ViS2l0L3F0L0FwaS9x
d2VicGFnZS5jcHAKaW5kZXggMTgwYzdhZi4uMWY2YTRhMSAxMDA2NDQKLS0tIGEvV2ViS2l0L3F0
L0FwaS9xd2VicGFnZS5jcHAKKysrIGIvV2ViS2l0L3F0L0FwaS9xd2VicGFnZS5jcHAKQEAgLTQw
Myw3ICs0MDMsNyBAQCBRV2ViUGFnZVByaXZhdGU6OlFXZWJQYWdlUHJpdmF0ZShRV2ViUGFnZSAq
cXEpCiAKICAgICBQYWdlR3JvdXA6OnNldFNob3VsZFRyYWNrVmlzaXRlZExpbmtzKHRydWUpOwog
ICAgIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKSAgICAKKyNpZiBFTkFCTEUoTk9USUZJQ0FU
SU9OUykgJiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAgICAgbm90aWZpY2F0aW9u
UHJlc2VudGVyQ2xpZW50ID0gbmV3IE5vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0KCk7CiAj
ZW5kaWYKIH0KZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hh
bmdlTG9nCmluZGV4IDJkMWViYzEuLjI4NTBiNWEgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFu
Z2VMb2cKKysrIGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEwLTA0
LTEyICDEsHNtYWlsIETDtm5tZXogIDxpc21haWxAbmFtdHJhYy5vcmc+CisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSBGaXggY29tcGlsYXRpb24g
d2l0aCBzeXN0ZW0gdHJheSBkaXNhYmxlZC4KKworICAgICAgICAqIEFwaS9xd2VicGFnZS5jcHA6
CisgICAgICAgIChRV2ViUGFnZVByaXZhdGU6OlFXZWJQYWdlUHJpdmF0ZSk6CisgICAgICAgICog
V2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwOgorICAgICAgICAqIFdlYkNvcmVTdXBw
b3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcDoKKyAgICAgICAgKiBXZWJDb3Jl
U3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRRdC5oOgorCiAyMDEwLTA0LTExICBT
aGVyaWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90QGdtYWlsLmNvbT4KIAogICAgICAgICBVbnJl
dmlld2VkLCByb2xsaW5nIG91dCByNTc0NjguCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29y
ZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwIGIvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L0No
cm9tZUNsaWVudFF0LmNwcAppbmRleCA2Mjg2YzUxLi40ZWRjMzMxIDEwMDY0NAotLS0gYS9XZWJL
aXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwCisrKyBiL1dlYktpdC9xdC9X
ZWJDb3JlU3VwcG9ydC9DaHJvbWVDbGllbnRRdC5jcHAKQEAgLTQ3NCw3ICs0NzQsNyBAQCB2b2lk
IENocm9tZUNsaWVudFF0OjpyZWFjaGVkTWF4QXBwQ2FjaGVTaXplKGludDY0X3QpCiB9CiAjZW5k
aWYKIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKQorI2lmIEVOQUJMRShOT1RJRklDQVRJT05T
KSAmJiAhZGVmaW5lZChRVF9OT19TWVNURU1UUkFZSUNPTikKIE5vdGlmaWNhdGlvblByZXNlbnRl
ciogQ2hyb21lQ2xpZW50UXQ6Om5vdGlmaWNhdGlvblByZXNlbnRlcigpIGNvbnN0CiB7CiAgICAg
cmV0dXJuIG1fd2ViUGFnZS0+ZC0+bm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50OwpkaWZmIC0t
Z2l0IGEvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVu
dFF0LmNwcCBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJD
bGllbnRRdC5jcHAKaW5kZXggMjZlOGZlNy4uMGE2MjdjMSAxMDA2NDQKLS0tIGEvV2ViS2l0L3F0
L1dlYkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcAorKysgYi9X
ZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50UXQuY3Bw
CkBAIC00OCw3ICs0OCw3IEBAIHZvaWQgUVdFQktJVF9FWFBPUlQgcXRfZHVtcF9ub3RpZmljYXRp
b24oYm9vbCBiKQogICAgIGR1bXBOb3RpZmljYXRpb24gPSBiOwogfQogCi0jaWYgRU5BQkxFKE5P
VElGSUNBVElPTlMpCisjaWYgRU5BQkxFKE5PVElGSUNBVElPTlMpICYmICFkZWZpbmVkKFFUX05P
X1NZU1RFTVRSQVlJQ09OKQogCiB1c2luZyBuYW1lc3BhY2UgV2ViQ29yZTsKIApkaWZmIC0tZ2l0
IGEvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0
LmggYi9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50
UXQuaAppbmRleCBjMzFiYTU3Li5iYjA5MzdmIDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvV2ViQ29y
ZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50UXQuaAorKysgYi9XZWJLaXQvcXQv
V2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50UXQuaApAQCAtMzQsNyAr
MzQsNyBAQAogCiAjaW5jbHVkZSA8UVN5c3RlbVRyYXlJY29uPgogCi0jaWYgRU5BQkxFKE5PVElG
SUNBVElPTlMpCisjaWYgRU5BQkxFKE5PVElGSUNBVElPTlMpICYmICFkZWZpbmVkKFFUX05PX1NZ
U1RFTVRSQVlJQ09OKQogCiBuYW1lc3BhY2UgV2ViQ29yZSB7CiBjbGFzcyBEb2N1bWVudDsK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53159</attachid>
            <date>2010-04-12 01:36:05 -0700</date>
            <delta_ts>2010-04-12 01:38:58 -0700</delta_ts>
            <desc>Implement WebCore::ChromeClient::notificationPresenter() in all cases</desc>
            <filename>notification.patch</filename>
            <type>text/plain</type>
            <size>3323</size>
            <attacher name="Ismail Donmez">ismail</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2UuY3BwIGIvV2ViS2l0L3F0L0FwaS9x
d2VicGFnZS5jcHAKaW5kZXggMTgwYzdhZi4uMWY2YTRhMSAxMDA2NDQKLS0tIGEvV2ViS2l0L3F0
L0FwaS9xd2VicGFnZS5jcHAKKysrIGIvV2ViS2l0L3F0L0FwaS9xd2VicGFnZS5jcHAKQEAgLTQw
Myw3ICs0MDMsNyBAQCBRV2ViUGFnZVByaXZhdGU6OlFXZWJQYWdlUHJpdmF0ZShRV2ViUGFnZSAq
cXEpCiAKICAgICBQYWdlR3JvdXA6OnNldFNob3VsZFRyYWNrVmlzaXRlZExpbmtzKHRydWUpOwog
ICAgIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKSAgICAKKyNpZiBFTkFCTEUoTk9USUZJQ0FU
SU9OUykgJiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAgICAgbm90aWZpY2F0aW9u
UHJlc2VudGVyQ2xpZW50ID0gbmV3IE5vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0KCk7CiAj
ZW5kaWYKIH0KZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hh
bmdlTG9nCmluZGV4IDJkMWViYzEuLjVmMzQ4M2YgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFu
Z2VMb2cKKysrIGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEwLTA0
LTEyICDEsHNtYWlsIETDtm5tZXogIDxpc21haWxAaW50ZWxsaWdlbnRwYXBlcnMuY29tPgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gRml4IGNv
bXBpbGF0aW9uIHdpdGggc3lzdGVtIHRyYXkgZGlzYWJsZWQuCisKKyAgICAgICAgKiBBcGkvcXdl
YnBhZ2UuY3BwOgorICAgICAgICAoUVdlYlBhZ2VQcml2YXRlOjpRV2ViUGFnZVByaXZhdGUpOgor
ICAgICAgICAqIFdlYkNvcmVTdXBwb3J0L0Nocm9tZUNsaWVudFF0LmNwcDoKKyAgICAgICAgKiBX
ZWJDb3JlU3VwcG9ydC9DaHJvbWVDbGllbnRRdC5oOgorICAgICAgICAqIFdlYkNvcmVTdXBwb3J0
L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcDoKKyAgICAgICAgKiBXZWJDb3JlU3Vw
cG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRRdC5oOgorCiAyMDEwLTA0LTExICBTaGVy
aWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90QGdtYWlsLmNvbT4KIAogICAgICAgICBVbnJldmll
d2VkLCByb2xsaW5nIG91dCByNTc0NjguCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1
cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwIGIvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L0Nocm9t
ZUNsaWVudFF0LmNwcAppbmRleCA2Mjg2YzUxLi4zMWVkMTIwIDEwMDY0NAotLS0gYS9XZWJLaXQv
cXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwCisrKyBiL1dlYktpdC9xdC9XZWJD
b3JlU3VwcG9ydC9DaHJvbWVDbGllbnRRdC5jcHAKQEAgLTQ3NCwxMiArNDc0LDE0IEBAIHZvaWQg
Q2hyb21lQ2xpZW50UXQ6OnJlYWNoZWRNYXhBcHBDYWNoZVNpemUoaW50NjRfdCkKIH0KICNlbmRp
ZgogCi0jaWYgRU5BQkxFKE5PVElGSUNBVElPTlMpCiBOb3RpZmljYXRpb25QcmVzZW50ZXIqIENo
cm9tZUNsaWVudFF0Ojpub3RpZmljYXRpb25QcmVzZW50ZXIoKSBjb25zdAogeworI2lmIEVOQUJM
RShOT1RJRklDQVRJT05TKSAmJiAhZGVmaW5lZChRVF9OT19TWVNURU1UUkFZSUNPTikKICAgICBy
ZXR1cm4gbV93ZWJQYWdlLT5kLT5ub3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnQ7Ci19CisjZWxz
ZQorICAgIHJldHVybiBOVUxMOwogI2VuZGlmCit9CiAKIHZvaWQgQ2hyb21lQ2xpZW50UXQ6OnJ1
bk9wZW5QYW5lbChGcmFtZSogZnJhbWUsIFBhc3NSZWZQdHI8RmlsZUNob29zZXI+IHBycEZpbGVD
aG9vc2VyKQogewpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L0Nocm9tZUNs
aWVudFF0LmggYi9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuaAppbmRl
eCA4NWViODU0Li5mNmMxNmE4IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQv
Q2hyb21lQ2xpZW50UXQuaAorKysgYi9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xp
ZW50UXQuaApAQCAtMTI4LDkgKzEyOCw3IEBAIG5hbWVzcGFjZSBXZWJDb3JlIHsKICAgICAgICAg
dmlydHVhbCB2b2lkIHJlYWNoZWRNYXhBcHBDYWNoZVNpemUoaW50NjRfdCBzcGFjZU5lZWRlZCk7
CiAjZW5kaWYKIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKQogICAgICAgICB2aXJ0dWFsIE5v
dGlmaWNhdGlvblByZXNlbnRlciogbm90aWZpY2F0aW9uUHJlc2VudGVyKCkgY29uc3Q7Ci0jZW5k
aWYKIAogI2lmIFVTRShBQ0NFTEVSQVRFRF9DT01QT1NJVElORykKICAgICAgICAgLy8gc2VlIENo
cm9tZUNsaWVudC5oCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZp
Y2F0aW9uUHJlc2VudGVyQ2xpZW50UXQuY3BwIGIvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05v
dGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcAppbmRleCAyNmU4ZmU3Li4wYTYyN2MxIDEw
MDY0NAotLS0gYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVy
Q2xpZW50UXQuY3BwCisrKyBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25Q
cmVzZW50ZXJDbGllbnRRdC5jcHAKQEAgLTQ4LDcgKzQ4LDcgQEAgdm9pZCBRV0VCS0lUX0VYUE9S
VCBxdF9kdW1wX25vdGlmaWNhdGlvbihib29sIGIpCiAgICAgZHVtcE5vdGlmaWNhdGlvbiA9IGI7
CiB9CiAKLSNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykKKyNpZiBFTkFCTEUoTk9USUZJQ0FUSU9O
UykgJiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAKIHVzaW5nIG5hbWVzcGFjZSBX
ZWJDb3JlOwogCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0
aW9uUHJlc2VudGVyQ2xpZW50UXQuaCBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmlj
YXRpb25QcmVzZW50ZXJDbGllbnRRdC5oCmluZGV4IGMzMWJhNTcuLmJiMDkzN2YgMTAwNjQ0Ci0t
LSBhL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRR
dC5oCisrKyBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJD
bGllbnRRdC5oCkBAIC0zNCw3ICszNCw3IEBACiAKICNpbmNsdWRlIDxRU3lzdGVtVHJheUljb24+
CiAKLSNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykKKyNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykg
JiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAKIG5hbWVzcGFjZSBXZWJDb3JlIHsK
IGNsYXNzIERvY3VtZW50Owo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53160</attachid>
            <date>2010-04-12 01:38:58 -0700</date>
            <delta_ts>2010-04-20 11:08:12 -0700</delta_ts>
            <desc>Fix style error</desc>
            <filename>notification.patch</filename>
            <type>text/plain</type>
            <size>3320</size>
            <attacher name="Ismail Donmez">ismail</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9BcGkvcXdlYnBhZ2UuY3BwIGIvV2ViS2l0L3F0L0FwaS9x
d2VicGFnZS5jcHAKaW5kZXggMTgwYzdhZi4uMWY2YTRhMSAxMDA2NDQKLS0tIGEvV2ViS2l0L3F0
L0FwaS9xd2VicGFnZS5jcHAKKysrIGIvV2ViS2l0L3F0L0FwaS9xd2VicGFnZS5jcHAKQEAgLTQw
Myw3ICs0MDMsNyBAQCBRV2ViUGFnZVByaXZhdGU6OlFXZWJQYWdlUHJpdmF0ZShRV2ViUGFnZSAq
cXEpCiAKICAgICBQYWdlR3JvdXA6OnNldFNob3VsZFRyYWNrVmlzaXRlZExpbmtzKHRydWUpOwog
ICAgIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKSAgICAKKyNpZiBFTkFCTEUoTk9USUZJQ0FU
SU9OUykgJiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAgICAgbm90aWZpY2F0aW9u
UHJlc2VudGVyQ2xpZW50ID0gbmV3IE5vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0KCk7CiAj
ZW5kaWYKIH0KZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQvcXQvQ2hh
bmdlTG9nCmluZGV4IDJkMWViYzEuLjVmMzQ4M2YgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9DaGFu
Z2VMb2cKKysrIGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEwLTA0
LTEyICDEsHNtYWlsIETDtm5tZXogIDxpc21haWxAaW50ZWxsaWdlbnRwYXBlcnMuY29tPgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gRml4IGNv
bXBpbGF0aW9uIHdpdGggc3lzdGVtIHRyYXkgZGlzYWJsZWQuCisKKyAgICAgICAgKiBBcGkvcXdl
YnBhZ2UuY3BwOgorICAgICAgICAoUVdlYlBhZ2VQcml2YXRlOjpRV2ViUGFnZVByaXZhdGUpOgor
ICAgICAgICAqIFdlYkNvcmVTdXBwb3J0L0Nocm9tZUNsaWVudFF0LmNwcDoKKyAgICAgICAgKiBX
ZWJDb3JlU3VwcG9ydC9DaHJvbWVDbGllbnRRdC5oOgorICAgICAgICAqIFdlYkNvcmVTdXBwb3J0
L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcDoKKyAgICAgICAgKiBXZWJDb3JlU3Vw
cG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRRdC5oOgorCiAyMDEwLTA0LTExICBTaGVy
aWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90QGdtYWlsLmNvbT4KIAogICAgICAgICBVbnJldmll
d2VkLCByb2xsaW5nIG91dCByNTc0NjguCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1
cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwIGIvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L0Nocm9t
ZUNsaWVudFF0LmNwcAppbmRleCA2Mjg2YzUxLi4zMWVkMTIwIDEwMDY0NAotLS0gYS9XZWJLaXQv
cXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuY3BwCisrKyBiL1dlYktpdC9xdC9XZWJD
b3JlU3VwcG9ydC9DaHJvbWVDbGllbnRRdC5jcHAKQEAgLTQ3NCwxMiArNDc0LDE0IEBAIHZvaWQg
Q2hyb21lQ2xpZW50UXQ6OnJlYWNoZWRNYXhBcHBDYWNoZVNpemUoaW50NjRfdCkKIH0KICNlbmRp
ZgogCi0jaWYgRU5BQkxFKE5PVElGSUNBVElPTlMpCiBOb3RpZmljYXRpb25QcmVzZW50ZXIqIENo
cm9tZUNsaWVudFF0Ojpub3RpZmljYXRpb25QcmVzZW50ZXIoKSBjb25zdAogeworI2lmIEVOQUJM
RShOT1RJRklDQVRJT05TKSAmJiAhZGVmaW5lZChRVF9OT19TWVNURU1UUkFZSUNPTikKICAgICBy
ZXR1cm4gbV93ZWJQYWdlLT5kLT5ub3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnQ7Ci19CisjZWxz
ZQorICAgIHJldHVybiAwOwogI2VuZGlmCit9CiAKIHZvaWQgQ2hyb21lQ2xpZW50UXQ6OnJ1bk9w
ZW5QYW5lbChGcmFtZSogZnJhbWUsIFBhc3NSZWZQdHI8RmlsZUNob29zZXI+IHBycEZpbGVDaG9v
c2VyKQogewpkaWZmIC0tZ2l0IGEvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L0Nocm9tZUNsaWVu
dFF0LmggYi9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50UXQuaAppbmRleCA4
NWViODU0Li5mNmMxNmE4IDEwMDY0NAotLS0gYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hy
b21lQ2xpZW50UXQuaAorKysgYi9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvQ2hyb21lQ2xpZW50
UXQuaApAQCAtMTI4LDkgKzEyOCw3IEBAIG5hbWVzcGFjZSBXZWJDb3JlIHsKICAgICAgICAgdmly
dHVhbCB2b2lkIHJlYWNoZWRNYXhBcHBDYWNoZVNpemUoaW50NjRfdCBzcGFjZU5lZWRlZCk7CiAj
ZW5kaWYKIAotI2lmIEVOQUJMRShOT1RJRklDQVRJT05TKQogICAgICAgICB2aXJ0dWFsIE5vdGlm
aWNhdGlvblByZXNlbnRlciogbm90aWZpY2F0aW9uUHJlc2VudGVyKCkgY29uc3Q7Ci0jZW5kaWYK
IAogI2lmIFVTRShBQ0NFTEVSQVRFRF9DT01QT1NJVElORykKICAgICAgICAgLy8gc2VlIENocm9t
ZUNsaWVudC5oCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0
aW9uUHJlc2VudGVyQ2xpZW50UXQuY3BwIGIvV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05vdGlm
aWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcAppbmRleCAyNmU4ZmU3Li4wYTYyN2MxIDEwMDY0
NAotLS0gYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xp
ZW50UXQuY3BwCisrKyBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVz
ZW50ZXJDbGllbnRRdC5jcHAKQEAgLTQ4LDcgKzQ4LDcgQEAgdm9pZCBRV0VCS0lUX0VYUE9SVCBx
dF9kdW1wX25vdGlmaWNhdGlvbihib29sIGIpCiAgICAgZHVtcE5vdGlmaWNhdGlvbiA9IGI7CiB9
CiAKLSNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykKKyNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykg
JiYgIWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAKIHVzaW5nIG5hbWVzcGFjZSBXZWJD
b3JlOwogCmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9u
UHJlc2VudGVyQ2xpZW50UXQuaCBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRp
b25QcmVzZW50ZXJDbGllbnRRdC5oCmluZGV4IGMzMWJhNTcuLmJiMDkzN2YgMTAwNjQ0Ci0tLSBh
L1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRRdC5o
CisrKyBiL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGll
bnRRdC5oCkBAIC0zNCw3ICszNCw3IEBACiAKICNpbmNsdWRlIDxRU3lzdGVtVHJheUljb24+CiAK
LSNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykKKyNpZiBFTkFCTEUoTk9USUZJQ0FUSU9OUykgJiYg
IWRlZmluZWQoUVRfTk9fU1lTVEVNVFJBWUlDT04pCiAKIG5hbWVzcGFjZSBXZWJDb3JlIHsKIGNs
YXNzIERvY3VtZW50Owo=
</data>
<flag name="review"
          id="36804"
          type_id="1"
          status="-"
          setter="laszlo.gombos"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>53847</attachid>
            <date>2010-04-20 11:08:12 -0700</date>
            <delta_ts>2010-04-21 06:08:22 -0700</delta_ts>
            <desc>fix patch</desc>
            <filename>bug37442_1.patch</filename>
            <type>text/plain</type>
            <size>1827</size>
            <attacher name="Chang Shu">cshu</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9xdC9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViS2l0L3F0L0NoYW5n
ZUxvZwkocmV2aXNpb24gNTc4OTcpCisrKyBXZWJLaXQvcXQvQ2hhbmdlTG9nCSh3b3JraW5nIGNv
cHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTAtMDQtMjAgIFNodSBDaGFuZyAgPGNoYW5nLnNodUBu
b2tpYS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgW1F0XSBGaXggU3ltYmlhbiBidWlsZCB3aGVyZSBRVF9OT19TWVNURU1UUkFZSUNPTiBpcyBk
ZWZpbmVkLgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
Mzc0NDIKKworICAgICAgICAqIFdlYkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNs
aWVudFF0LmNwcDoKKyAgICAgICAgKE5vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0OjpzaG93
KToKKyAgICAgICAgKiBXZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVzZW50ZXJDbGllbnRR
dC5oOgorCiAyMDEwLTA0LTE5ICBKb2NlbHluIFR1cmNvdHRlICA8am9jZWx5bi50dXJjb3R0ZUBu
b2tpYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgU2ltb24gSGF1c21hbm4uCkluZGV4OiBX
ZWJLaXQvcXQvV2ViQ29yZVN1cHBvcnQvTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50UXQuY3Bw
Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT0KLS0tIFdlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9Ob3RpZmljYXRpb25QcmVz
ZW50ZXJDbGllbnRRdC5jcHAJKHJldmlzaW9uIDU3Nzk0KQorKysgV2ViS2l0L3F0L1dlYkNvcmVT
dXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmNwcAkod29ya2luZyBjb3B5KQpA
QCAtNjcsMTAgKzY3LDExIEBAIGJvb2wgTm90aWZpY2F0aW9uUHJlc2VudGVyQ2xpZW50UXQ6OnNo
b3cKICAgICAgICAgICAgICAgICBRU3RyaW5nKG5vdGlmaWNhdGlvbi0+Y29udGVudHMoKS5ib2R5
KCkpLnRvVXRmOCgpLmNvbnN0RGF0YSgpKTsKICAgICAgICAgfQogICAgIH0KLSAgICAgICAgICAg
IAorCisjaWZuZGVmIFFUX05PX1NZU1RFTVRSQVlJQ09OCiAgICAgbV90cmF5LnNob3coKTsKICAg
ICBtX3RyYXkuc2hvd01lc3NhZ2Uobm90aWZpY2F0aW9uLT5jb250ZW50cygpLnRpdGxlKCksIG5v
dGlmaWNhdGlvbi0+Y29udGVudHMoKS5ib2R5KCksIFFTeXN0ZW1UcmF5SWNvbjo6SW5mb3JtYXRp
b24pOwotCisjZW5kaWYKICAgICByZXR1cm4gdHJ1ZTsKIH0KIApJbmRleDogV2ViS2l0L3F0L1dl
YkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0LmgKPT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot
LS0gV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05vdGlmaWNhdGlvblByZXNlbnRlckNsaWVudFF0
LmgJKHJldmlzaW9uIDU3Nzk0KQorKysgV2ViS2l0L3F0L1dlYkNvcmVTdXBwb3J0L05vdGlmaWNh
dGlvblByZXNlbnRlckNsaWVudFF0LmgJKHdvcmtpbmcgY29weSkKQEAgLTUyLDcgKzUyLDkgQEAg
cHVibGljOgogICAgIHZpcnR1YWwgTm90aWZpY2F0aW9uUHJlc2VudGVyOjpQZXJtaXNzaW9uIGNo
ZWNrUGVybWlzc2lvbihjb25zdCBLVVJMJik7CiAKIHByaXZhdGU6IAorI2lmbmRlZiBRVF9OT19T
WVNURU1UUkFZSUNPTgogICAgIFFTeXN0ZW1UcmF5SWNvbiBtX3RyYXk7CisjZW5kaWYKIH07CiB9
CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>