[Webkit-unassigned] [Bug 131696] Only webkit2 pages should have a page throttler.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 16 19:12:19 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=131696


Gavin Barraclough <barraclough at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #229502|review?                     |review+
               Flag|                            |




--- Comment #7 from Gavin Barraclough <barraclough at apple.com>  2014-04-16 19:12:38 PST ---
(From update of attachment 229502)
View in context: https://bugs.webkit.org/attachment.cgi?id=229502&action=review

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1750
> +    if (m_page->pageThrottler())

WebPage's m_page should always have a throttler (per call to createPageThrottler, above).
This should just be an ASSERT.

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1819
> +    if(m_page->pageThrottler())

ditto.

> Source/WebKit2/WebProcess/WebPage/WebPage.cpp:1851
> +    if (m_page->pageThrottler())

ditto.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list