Title: [183700] trunk/Source/WebCore
Revision
183700
Author
[email protected]
Date
2015-05-01 16:59:58 -0700 (Fri, 01 May 2015)

Log Message

Inline some small methods instead of them being EXPORT'd and out of line
https://bugs.webkit.org/show_bug.cgi?id=144498

Patch by Joseph Pecoraro <[email protected]> on 2015-05-01
Reviewed by Darin Adler.

* page/Page.cpp:
(WebCore::Page::testTrigger): Deleted.
(WebCore::Page::clearTrigger): Deleted.
(WebCore::Page::expectsWheelEventTriggers): Deleted.
* page/Page.h:
(WebCore::Page::testTrigger):
(WebCore::Page::clearTrigger):
(WebCore::Page::expectsWheelEventTriggers):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (183699 => 183700)


--- trunk/Source/WebCore/ChangeLog	2015-05-01 23:52:06 UTC (rev 183699)
+++ trunk/Source/WebCore/ChangeLog	2015-05-01 23:59:58 UTC (rev 183700)
@@ -1,3 +1,19 @@
+2015-05-01  Joseph Pecoraro  <[email protected]>
+
+        Inline some small methods instead of them being EXPORT'd and out of line
+        https://bugs.webkit.org/show_bug.cgi?id=144498
+
+        Reviewed by Darin Adler.
+
+        * page/Page.cpp:
+        (WebCore::Page::testTrigger): Deleted.
+        (WebCore::Page::clearTrigger): Deleted.
+        (WebCore::Page::expectsWheelEventTriggers): Deleted.
+        * page/Page.h:
+        (WebCore::Page::testTrigger):
+        (WebCore::Page::clearTrigger):
+        (WebCore::Page::expectsWheelEventTriggers):
+
 2015-05-01  Dan Bernstein  <[email protected]>
 
         WebCore part of <rdar://problem/8636045> Back/forward navigation to an error page in Safari breaks the back-forward list

Modified: trunk/Source/WebCore/page/Page.cpp (183699 => 183700)


--- trunk/Source/WebCore/page/Page.cpp	2015-05-01 23:52:06 UTC (rev 183699)
+++ trunk/Source/WebCore/page/Page.cpp	2015-05-01 23:59:58 UTC (rev 183700)
@@ -1725,11 +1725,6 @@
 }
 #endif
 
-RefPtr<WheelEventTestTrigger> Page::testTrigger() const
-{
-    return m_testTrigger;
-}
-
 WheelEventTestTrigger& Page::ensureTestTrigger()
 {
     if (!m_testTrigger)
@@ -1738,14 +1733,4 @@
     return *m_testTrigger;
 }
 
-void Page::clearTrigger()
-{
-    m_testTrigger = nullptr;
-}
-
-bool Page::expectsWheelEventTriggers() const
-{
-    return !!m_testTrigger;
-}
-
 } // namespace WebCore

Modified: trunk/Source/WebCore/page/Page.h (183699 => 183700)


--- trunk/Source/WebCore/page/Page.h	2015-05-01 23:52:06 UTC (rev 183699)
+++ trunk/Source/WebCore/page/Page.h	2015-05-01 23:59:58 UTC (rev 183700)
@@ -440,10 +440,10 @@
     WEBCORE_EXPORT void setShouldPlayToPlaybackTarget(uint64_t, bool);
 #endif
 
-    WEBCORE_EXPORT RefPtr<WheelEventTestTrigger> testTrigger() const;
+    RefPtr<WheelEventTestTrigger> testTrigger() const { return m_testTrigger; }
     WEBCORE_EXPORT WheelEventTestTrigger& ensureTestTrigger();
-    WEBCORE_EXPORT void clearTrigger();
-    WEBCORE_EXPORT bool expectsWheelEventTriggers() const;
+    void clearTrigger() { m_testTrigger = nullptr; }
+    bool expectsWheelEventTriggers() const { return !!m_testTrigger; }
 
 private:
     WEBCORE_EXPORT void initGroup();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to