<?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>62649</bug_id>
          
          <creation_ts>2011-06-14 10:32:59 -0700</creation_ts>
          <short_desc>Incorrect #if[n]def for building without DISPATCH_VM_PRESSURE</short_desc>
          <delta_ts>2011-06-14 16:30:42 -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>WebCore Misc.</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>
          <dependson>62652</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Saboff">msaboff</reporter>
          <assigned_to name="Michael Saboff">msaboff</assigned_to>
          <cc>aestes</cc>
    
    <cc>joepeck</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>420540</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 10:32:59 -0700</bug_when>
    <thetext>The #ifdef DISPATCH_VM_PRESSURE on line 43 of MemoryPressureHandlerMac.mm should actually be #ifndef.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420565</commentid>
    <comment_count>1</comment_count>
      <attachid>97140</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 10:49:56 -0700</bug_when>
    <thetext>Created attachment 97140
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420567</commentid>
    <comment_count>2</comment_count>
      <attachid>97140</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2011-06-14 10:51:12 -0700</bug_when>
    <thetext>Comment on attachment 97140
Proposed patch

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420571</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 10:55:12 -0700</bug_when>
    <thetext>Committed r88823: &lt;http://trac.webkit.org/changeset/88823&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420595</commentid>
    <comment_count>4</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2011-06-14 11:21:44 -0700</bug_when>
    <thetext>DISPATCH_VM_PRESSURE is an enum, not a macro, so this check doesn&apos;t actually work and ends up breaking the build on machines that do have the system header.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420663</commentid>
    <comment_count>5</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2011-06-14 12:46:50 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/88829 rolled this out. Reopening.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420775</commentid>
    <comment_count>6</comment_count>
      <attachid>97173</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 15:35:33 -0700</bug_when>
    <thetext>Created attachment 97173
Updated patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420779</commentid>
    <comment_count>7</comment_count>
      <attachid>97176</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 15:38:14 -0700</bug_when>
    <thetext>Created attachment 97176
Right updated patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420781</commentid>
    <comment_count>8</comment_count>
      <attachid>97176</attachid>
    <who name="Joseph Pecoraro">joepeck</who>
    <bug_when>2011-06-14 15:39:28 -0700</bug_when>
    <thetext>Comment on attachment 97176
Right updated patch

Thanks again.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420831</commentid>
    <comment_count>9</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-06-14 16:30:42 -0700</bug_when>
    <thetext>Committed r88871: &lt;http://trac.webkit.org/changeset/88871&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>97140</attachid>
            <date>2011-06-14 10:49:56 -0700</date>
            <delta_ts>2011-06-14 15:35:33 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>62649.patch</filename>
            <type>text/plain</type>
            <size>1328</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDg4ODIxKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDYtMTQgIE1pY2hhZWwg
U2Fib2ZmICA8bXNhYm9mZkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgSW5jb3JyZWN0ICNpZltuXWRlZiBmb3IgYnVpbGRpbmcgd2l0
aG91dCBESVNQQVRDSF9WTV9QUkVTU1VSRQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9NjI2NDkKKworICAgICAgICBUaGUgI2lmZGVmIERJU1BBVENIX1ZN
X1BSRVNTVVJFIGFjdHVhbGx5IG5lZWRzIHRvIGJlICNpZm5kZWYKKyAgICAgICAgdG8gaGFuZGxl
IHdoZW4gRElTUEFUQ0hfVk1fUFJFU1NVUkUgaXMgTk9UIGRlZmluZWQgaW4gdGhlCisgICAgICAg
IHN5c3RlbSBoZWFkZXIgZmlsZXMuCisKKyAgICAgICAgTm8gZnVuY3Rpb25hbCBjaGFuZ2VzLCBm
aXhpbmcgYnVpbGQgaXNzdWUgdGhlcmVmb3JlIG5vIHRlc3RzLgorCisgICAgICAgICogcGxhdGZv
cm0vbWFjL01lbW9yeVByZXNzdXJlSGFuZGxlck1hYy5tbToKKwogMjAxMS0wNi0xNCAgVmlhdGNo
ZXNsYXYgT3N0YXBlbmtvICA8b3N0YXBlbmtvLnZpYXRjaGVzbGF2QG5va2lhLmNvbT4KIAogICAg
ICAgICBSZXZpZXdlZCBieSBMYXN6bG8gR29tYm9zLgpJbmRleDogU291cmNlL1dlYkNvcmUvcGxh
dGZvcm0vbWFjL01lbW9yeVByZXNzdXJlSGFuZGxlck1hYy5tbQo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3Vy
Y2UvV2ViQ29yZS9wbGF0Zm9ybS9tYWMvTWVtb3J5UHJlc3N1cmVIYW5kbGVyTWFjLm1tCShyZXZp
c2lvbiA4ODgyMCkKKysrIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL21hYy9NZW1vcnlQcmVzc3Vy
ZUhhbmRsZXJNYWMubW0JKHdvcmtpbmcgY29weSkKQEAgLTQwLDcgKzQwLDcgQEAKIERJU1BBVENI
X0VYUE9SVCBjb25zdCBzdHJ1Y3QgZGlzcGF0Y2hfc291cmNlX3R5cGVfcyBfZGlzcGF0Y2hfc291
cmNlX3R5cGVfdm07CiAjZW5kaWYKIAotI2lmZGVmIERJU1BBVENIX1ZNX1BSRVNTVVJFCisjaWZu
ZGVmIERJU1BBVENIX1ZNX1BSRVNTVVJFCiBlbnVtIHsKICBESVNQQVRDSF9WTV9QUkVTU1VSRSA9
IDB4ODAwMDAwMDAsCiB9Owo=
</data>
<flag name="review"
          id="91004"
          type_id="1"
          status="+"
          setter="joepeck"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>97173</attachid>
            <date>2011-06-14 15:35:33 -0700</date>
            <delta_ts>2011-06-14 15:38:14 -0700</delta_ts>
            <desc>Updated patch</desc>
            <filename>62649.patch</filename>
            <type>text/plain</type>
            <size>1328</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDg4ODIxKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDYtMTQgIE1pY2hhZWwg
U2Fib2ZmICA8bXNhYm9mZkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgSW5jb3JyZWN0ICNpZltuXWRlZiBmb3IgYnVpbGRpbmcgd2l0
aG91dCBESVNQQVRDSF9WTV9QUkVTU1VSRQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9NjI2NDkKKworICAgICAgICBUaGUgI2lmZGVmIERJU1BBVENIX1ZN
X1BSRVNTVVJFIGFjdHVhbGx5IG5lZWRzIHRvIGJlICNpZm5kZWYKKyAgICAgICAgdG8gaGFuZGxl
IHdoZW4gRElTUEFUQ0hfVk1fUFJFU1NVUkUgaXMgTk9UIGRlZmluZWQgaW4gdGhlCisgICAgICAg
IHN5c3RlbSBoZWFkZXIgZmlsZXMuCisKKyAgICAgICAgTm8gZnVuY3Rpb25hbCBjaGFuZ2VzLCBm
aXhpbmcgYnVpbGQgaXNzdWUgdGhlcmVmb3JlIG5vIHRlc3RzLgorCisgICAgICAgICogcGxhdGZv
cm0vbWFjL01lbW9yeVByZXNzdXJlSGFuZGxlck1hYy5tbToKKwogMjAxMS0wNi0xNCAgVmlhdGNo
ZXNsYXYgT3N0YXBlbmtvICA8b3N0YXBlbmtvLnZpYXRjaGVzbGF2QG5va2lhLmNvbT4KIAogICAg
ICAgICBSZXZpZXdlZCBieSBMYXN6bG8gR29tYm9zLgpJbmRleDogU291cmNlL1dlYkNvcmUvcGxh
dGZvcm0vbWFjL01lbW9yeVByZXNzdXJlSGFuZGxlck1hYy5tbQo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3Vy
Y2UvV2ViQ29yZS9wbGF0Zm9ybS9tYWMvTWVtb3J5UHJlc3N1cmVIYW5kbGVyTWFjLm1tCShyZXZp
c2lvbiA4ODgyMCkKKysrIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL21hYy9NZW1vcnlQcmVzc3Vy
ZUhhbmRsZXJNYWMubW0JKHdvcmtpbmcgY29weSkKQEAgLTQwLDcgKzQwLDcgQEAKIERJU1BBVENI
X0VYUE9SVCBjb25zdCBzdHJ1Y3QgZGlzcGF0Y2hfc291cmNlX3R5cGVfcyBfZGlzcGF0Y2hfc291
cmNlX3R5cGVfdm07CiAjZW5kaWYKIAotI2lmZGVmIERJU1BBVENIX1ZNX1BSRVNTVVJFCisjaWZu
ZGVmIERJU1BBVENIX1ZNX1BSRVNTVVJFCiBlbnVtIHsKICBESVNQQVRDSF9WTV9QUkVTU1VSRSA9
IDB4ODAwMDAwMDAsCiB9Owo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>97176</attachid>
            <date>2011-06-14 15:38:14 -0700</date>
            <delta_ts>2011-06-14 15:39:28 -0700</delta_ts>
            <desc>Right updated patch</desc>
            <filename>62649-2.patch</filename>
            <type>text/plain</type>
            <size>1366</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDg4ODU3KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTggQEAKKzIwMTEtMDYtMTQgIE1pY2hhZWwg
U2Fib2ZmICA8bXNhYm9mZkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgSW5jb3JyZWN0ICNpZltuXWRlZiBmb3IgYnVpbGRpbmcgd2l0
aG91dCBESVNQQVRDSF9WTV9QUkVTU1VSRQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9NjI2NDkKKworICAgICAgICBUYWtlIDIuCisgICAgICAgIFRoZSAj
aWZkZWYgRElTUEFUQ0hfVk1fUFJFU1NVUkUgZG9lc24ndCB3b3JrLiAgQ29tYmluZWQgdGhlIGNv
ZGUKKyAgICAgICAgaW5zaWRlIHRoaXMgI2lmZGVmIHdpdGggdGhlIHByaW9yICNpZm5kZWYgRElT
UEFUQ0hfU09VUkNFX1RZUEVfVk0uCisKKyAgICAgICAgTm8gZnVuY3Rpb25hbCBjaGFuZ2VzLCBm
aXhpbmcgYnVpbGQgaXNzdWUgdGhlcmVmb3JlIG5vIHRlc3RzLgorCisgICAgICAgICogcGxhdGZv
cm0vbWFjL01lbW9yeVByZXNzdXJlSGFuZGxlck1hYy5tbToKKwogMjAxMS0wNi0xNCAgSmVmZnJl
eSBQZmF1ICA8anBmYXVAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERhdmlkIEh5
YXR0LgpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbWFjL01lbW9yeVByZXNzdXJlSGFu
ZGxlck1hYy5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9tYWMvTWVt
b3J5UHJlc3N1cmVIYW5kbGVyTWFjLm1tCShyZXZpc2lvbiA4ODgyMykKKysrIFNvdXJjZS9XZWJD
b3JlL3BsYXRmb3JtL21hYy9NZW1vcnlQcmVzc3VyZUhhbmRsZXJNYWMubW0JKHdvcmtpbmcgY29w
eSkKQEAgLTM4LDkgKzM4LDcgQEAKICNpZm5kZWYgRElTUEFUQ0hfU09VUkNFX1RZUEVfVk0KICNk
ZWZpbmUgRElTUEFUQ0hfU09VUkNFX1RZUEVfVk0gKCZfZGlzcGF0Y2hfc291cmNlX3R5cGVfdm0p
CiBESVNQQVRDSF9FWFBPUlQgY29uc3Qgc3RydWN0IGRpc3BhdGNoX3NvdXJjZV90eXBlX3MgX2Rp
c3BhdGNoX3NvdXJjZV90eXBlX3ZtOwotI2VuZGlmCiAKLSNpZm5kZWYgRElTUEFUQ0hfVk1fUFJF
U1NVUkUKIGVudW0gewogIERJU1BBVENIX1ZNX1BSRVNTVVJFID0gMHg4MDAwMDAwMCwKIH07Cg==
</data>
<flag name="review"
          id="91041"
          type_id="1"
          status="+"
          setter="joepeck"
    />
          </attachment>
      

    </bug>

</bugzilla>