<?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>169417</bug_id>
          
          <creation_ts>2017-03-09 09:36:24 -0800</creation_ts>
          <short_desc>[WK2] Prevent app nap for the active tab</short_desc>
          <delta_ts>2017-03-09 10:54:11 -0800</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>WebKit 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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>andersca</cc>
    
    <cc>barraclough</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ggaren</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1285341</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-03-09 09:36:24 -0800</bug_when>
    <thetext>Prevent app nap for the active tab to minimize the risks of breakage.

From the AppNap documentation [1], the app is only a candidate for app nap if it not the foreground app. However, in the Safari case, the foreground app is Safari (UIProcess) and the same policy does not apply to our foreground WebContent process unless we explicitly make it so.

[1] https://developer.apple.com/library/content/documentation/Performance/Conceptual/power_efficiency_guidelines_osx/AppNap.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285343</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-03-09 09:36:44 -0800</bug_when>
    <thetext>&lt;rdar://problem/30876965&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285347</commentid>
    <comment_count>2</comment_count>
      <attachid>303926</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-03-09 09:39:00 -0800</bug_when>
    <thetext>Created attachment 303926
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285365</commentid>
    <comment_count>3</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2017-03-09 10:02:20 -0800</bug_when>
    <thetext>&lt;rdar://problem/30318924&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285398</commentid>
    <comment_count>4</comment_count>
      <attachid>303926</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-03-09 10:54:05 -0800</bug_when>
    <thetext>Comment on attachment 303926
Patch

Clearing flags on attachment: 303926

Committed r213649: &lt;http://trac.webkit.org/changeset/213649&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1285399</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-03-09 10:54:11 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>303926</attachid>
            <date>2017-03-09 09:39:00 -0800</date>
            <delta_ts>2017-03-09 10:54:05 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-169417-20170309093859.patch</filename>
            <type>text/plain</type>
            <size>2569</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjEzNjQzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggZmZkOTk2MDNkNmE4ZjVj
YzFmNjViMTc5YTQ1YzA1YmM3NzRhMzg4Yy4uNTdiZDYzYjM1NDg5NmQxYmI3N2U3OWFlMzExMWJm
ODY0Nzc4ZmM5YyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI0IEBACisyMDE3LTAzLTA5ICBDaHJp
cyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CisKKyAgICAgICAgW1dLMl0gUHJldmVudCBhcHAg
bmFwIGZvciB0aGUgYWN0aXZlIHRhYgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTY5NDE3CisgICAgICAgIDxyZGFyOi8vcHJvYmxlbS8zMDg3Njk2NT4K
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBQcmV2ZW50
IGFwcCBuYXAgZm9yIHRoZSBhY3RpdmUgdGFiIHRvIG1pbmltaXplIHRoZSByaXNrcyBvZiBicmVh
a2FnZS4KKworICAgICAgICBGcm9tIHRoZSBBcHBOYXAgZG9jdW1lbnRhdGlvbiBbMV0sIHRoZSBh
cHAgaXMgb25seSBhIGNhbmRpZGF0ZSBmb3IgYXBwIG5hcCBpZiBpdCBub3QgdGhlCisgICAgICAg
IGZvcmVncm91bmQgYXBwLiBIb3dldmVyLCBpbiB0aGUgU2FmYXJpIGNhc2UsIHRoZSBmb3JlZ3Jv
dW5kIGFwcCBpcyBTYWZhcmkgKFVJUHJvY2VzcykgYW5kCisgICAgICAgIHRoZSBzYW1lIHBvbGlj
eSBkb2VzIG5vdCBhcHBseSB0byBvdXIgZm9yZWdyb3VuZCBXZWJDb250ZW50IHByb2Nlc3MgdW5s
ZXNzIHdlIGV4cGxpY2l0bHkKKyAgICAgICAgbWFrZSBpdCBzby4gVGhpcyBwYXRjaCBhZGRyZXNz
ZXMgdGhlIGlzc3VlIGJ5IHRha2luZyBhIFVzZXJBY3Rpdml0eSB3aGVuIHRoZSB0YWIgaXMKKyAg
ICAgICAgYWN0aXZlIHRvIHByZXZlbnQgYXBwIG5hcC4KKworICAgICAgICBbMV0gaHR0cHM6Ly9k
ZXZlbG9wZXIuYXBwbGUuY29tL2xpYnJhcnkvY29udGVudC9kb2N1bWVudGF0aW9uL1BlcmZvcm1h
bmNlL0NvbmNlcHR1YWwvcG93ZXJfZWZmaWNpZW5jeV9ndWlkZWxpbmVzX29zeC9BcHBOYXAuaHRt
bAorCisgICAgICAgICogV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBhZ2UuY3BwOgorICAgICAgICAo
V2ViS2l0OjpXZWJQYWdlOjp1cGRhdGVUaHJvdHRsZVN0YXRlKToKKwogMjAxNy0wMy0wOSAgVGlt
IEhvcnRvbiAgPHRpbW90aHlfaG9ydG9uQGFwcGxlLmNvbT4KIAogICAgICAgICBXS1dlYlZpZXcg
c2hvdWxkIGF1dG9tYXRpY2FsbHkgcmVzcGVjdCBpbnNldHMgZnJvbSB0aGUgdmlldyBjb250cm9s
bGVyIGhpZXJhcmNoeQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQ
YWdlL1dlYlBhZ2UuY3BwIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL1dlYlBh
Z2UuY3BwCmluZGV4IGMzMGJhMTg3NmY2Y2FiNWE0MmM0ODE1NDE2OGFmMGViMmYxODJiNmMuLjNh
MjQwYTE4YzhlN2JiMzJmNTQ3ZDFiMjk4YTYyZmZkMGE1MzVlYmQgMTAwNjQ0Ci0tLSBhL1NvdXJj
ZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9XZWJQYWdlLmNwcAorKysgYi9Tb3VyY2UvV2Vi
S2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvV2ViUGFnZS5jcHAKQEAgLTU3Niw3ICs1NzYsOSBAQCB2
b2lkIFdlYlBhZ2U6OnVwZGF0ZVRocm90dGxlU3RhdGUoKQogICAgIC8vIFdlIHNob3VsZCBzdXBw
cmVzcyBpZiB0aGUgcGFnZSBpcyBub3QgYWN0aXZlLCBpcyB2aXN1YWxseSBpZGxlLCBhbmQgc3Vw
cmVzc2lvbiBpcyBlbmFibGVkLgogICAgIGJvb2wgaXNMb2FkaW5nID0gbV9hY3Rpdml0eVN0YXRl
ICYgQWN0aXZpdHlTdGF0ZTo6SXNMb2FkaW5nOwogICAgIGJvb2wgaXNQbGF5aW5nQXVkaW8gPSBt
X2FjdGl2aXR5U3RhdGUgJiBBY3Rpdml0eVN0YXRlOjpJc0F1ZGlibGU7Ci0gICAgYm9vbCBwYWdl
U3VwcHJlc3NlZCA9ICFpc0xvYWRpbmcgJiYgIWlzUGxheWluZ0F1ZGlvICYmIG1fcHJvY2Vzc1N1
cHByZXNzaW9uRW5hYmxlZCAmJiAobV9hY3Rpdml0eVN0YXRlICYgQWN0aXZpdHlTdGF0ZTo6SXNW
aXN1YWxseUlkbGUpOworICAgIGJvb2wgaXNWaXN1YWxseUlkbGUgPSBtX2FjdGl2aXR5U3RhdGUg
JiBBY3Rpdml0eVN0YXRlOjpJc1Zpc3VhbGx5SWRsZTsKKyAgICBib29sIHdpbmRvd0lzQWN0aXZl
ID0gbV9hY3Rpdml0eVN0YXRlICYgQWN0aXZpdHlTdGF0ZTo6V2luZG93SXNBY3RpdmU7CisgICAg
Ym9vbCBwYWdlU3VwcHJlc3NlZCA9ICF3aW5kb3dJc0FjdGl2ZSAmJiAhaXNMb2FkaW5nICYmICFp
c1BsYXlpbmdBdWRpbyAmJiBtX3Byb2Nlc3NTdXBwcmVzc2lvbkVuYWJsZWQgJiYgaXNWaXN1YWxs
eUlkbGU7CiAKICAgICAvLyBUaGUgVXNlckFjdGl2aXR5IGtlZXBzIHRoZSBwcm9jZXNzZXMgcnVu
bmFibGUuIFNvIGlmIHRoZSBwYWdlIHNob3VsZCBiZSBzdXBwcmVzc2VkLCBzdG9wIHRoZSBhY3Rp
dml0eS4KICAgICAvLyBJZiB0aGUgcGFnZSBzaG91bGQgbm90IGJlIHN1cHJlc3NlZCwgc3RhcnQg
aXQuCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>