Title: [173623] trunk/Source/WebKit2
- Revision
- 173623
- Author
- [email protected]
- Date
- 2014-09-15 10:10:23 -0700 (Mon, 15 Sep 2014)
Log Message
Should call [BKSProcessAssertion invalidate]
https://bugs.webkit.org/show_bug.cgi?id=136804
<rdar://problem/17897780>
Reviewed by Darin Adler.
We should invalidate the process assertion before releasing it. Fortunately this is really easy,
since there is a simple 1:1 relationship with ProcessAssertion, with their lifetimes matching exactly.
* UIProcess/ProcessAssertion.cpp:
(WebKit::ProcessAssertion::~ProcessAssertion):
- no-op implementation.
* UIProcess/ProcessAssertion.h:
- added destructor.
* UIProcess/ios/ProcessAssertionIOS.mm:
(WebKit::ProcessAssertion::~ProcessAssertion):
- destructor invalidates the BKSProcessAssertion.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (173622 => 173623)
--- trunk/Source/WebKit2/ChangeLog 2014-09-15 15:59:39 UTC (rev 173622)
+++ trunk/Source/WebKit2/ChangeLog 2014-09-15 17:10:23 UTC (rev 173623)
@@ -1,3 +1,23 @@
+2014-09-13 Gavin Barraclough <[email protected]>
+
+ Should call [BKSProcessAssertion invalidate]
+ https://bugs.webkit.org/show_bug.cgi?id=136804
+ <rdar://problem/17897780>
+
+ Reviewed by Darin Adler.
+
+ We should invalidate the process assertion before releasing it. Fortunately this is really easy,
+ since there is a simple 1:1 relationship with ProcessAssertion, with their lifetimes matching exactly.
+
+ * UIProcess/ProcessAssertion.cpp:
+ (WebKit::ProcessAssertion::~ProcessAssertion):
+ - no-op implementation.
+ * UIProcess/ProcessAssertion.h:
+ - added destructor.
+ * UIProcess/ios/ProcessAssertionIOS.mm:
+ (WebKit::ProcessAssertion::~ProcessAssertion):
+ - destructor invalidates the BKSProcessAssertion.
+
2014-09-15 Gyuyoung Kim <[email protected]>
REGRESSION(r170243): [EFL][WK2] Dirty image is shown when new page is loading on fixed layout
Modified: trunk/Source/WebKit2/UIProcess/ProcessAssertion.cpp (173622 => 173623)
--- trunk/Source/WebKit2/UIProcess/ProcessAssertion.cpp 2014-09-15 15:59:39 UTC (rev 173622)
+++ trunk/Source/WebKit2/UIProcess/ProcessAssertion.cpp 2014-09-15 17:10:23 UTC (rev 173623)
@@ -35,6 +35,10 @@
{
}
+ProcessAssertion::~ProcessAssertion()
+{
+}
+
void ProcessAssertion::setState(AssertionState assertionState)
{
if (m_assertionState == assertionState)
Modified: trunk/Source/WebKit2/UIProcess/ProcessAssertion.h (173622 => 173623)
--- trunk/Source/WebKit2/UIProcess/ProcessAssertion.h 2014-09-15 15:59:39 UTC (rev 173622)
+++ trunk/Source/WebKit2/UIProcess/ProcessAssertion.h 2014-09-15 17:10:23 UTC (rev 173623)
@@ -42,7 +42,8 @@
class ProcessAssertion {
public:
ProcessAssertion(pid_t, AssertionState);
-
+ ~ProcessAssertion();
+
AssertionState state() const { return m_assertionState; }
void setState(AssertionState);
Modified: trunk/Source/WebKit2/UIProcess/ios/ProcessAssertionIOS.mm (173622 => 173623)
--- trunk/Source/WebKit2/UIProcess/ios/ProcessAssertionIOS.mm 2014-09-15 15:59:39 UTC (rev 173622)
+++ trunk/Source/WebKit2/UIProcess/ios/ProcessAssertionIOS.mm 2014-09-15 17:10:23 UTC (rev 173623)
@@ -158,6 +158,11 @@
m_assertion = adoptNS([[BKSProcessAssertion alloc] initWithPID:pid flags:flagsForState(assertionState) reason:BKSProcessAssertionReasonExtension name:@"Web content visible" withHandler:handler]);
}
+ProcessAssertion::~ProcessAssertion()
+{
+ [m_assertion invalidate];
+}
+
void ProcessAssertion::setState(AssertionState assertionState)
{
if (m_assertionState == assertionState)
@@ -201,6 +206,10 @@
{
}
+ProcessAssertion::~ProcessAssertion()
+{
+}
+
void ProcessAssertion::setState(AssertionState assertionState)
{
m_assertionState = assertionState;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
