[Webkit-unassigned] [Bug 69127] [EFL] Enabling the Page Visibility API

[bugzilla-daemon at webkit.org]

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





--- Comment #14 from Dongwoo Joshua Im <dw.im at samsung.com>  2011-09-30 19:55:42 PST ---
(In reply to comment #13)
> (From update of attachment 109273 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=109273&action=review
> 
> > ChangeLog:12
> > +        * Source/cmake/OptionsEfl.cmake:
> > +        * Source/cmakeconfig.h.cmake:
> 
> The description above is too generic; please describe what is being changed in these specific files.
> 

I add some more description.

> > Source/WebKit/efl/ChangeLog:12
> > +        Browser(or, other application) should call this newly added function
> > +        when page visibility is changed.
> 
> Do you mean users should call the API when the page visibility changes or when they want to change it? If it is really the former, how are they informed that this change has happened?
> 

If browser call this API, then WebKit will fire a "visibilitychange" event.
Web application need to add an event listener to the "visibilitychange" event, and event handling code.

You can find an example code here. (http://www.w3.org/TR/page-visibility/#introduction)


> > Source/WebKit/efl/ChangeLog:15
> > +        (ewk_view_visibility_state_set): Setter function of EFL port.
> 
> Don't you also need a getter?
> 

I don't think a getter is needed, because browser will not try to get the visibility status from WebKit. The visibility status is maintained by browser.
Browser inform the status to WebKit to fire the "visibilitychange" event which the web application might be listening.

> > Source/WebKit/efl/ewk/ewk_view.h:2183
> > + * When the visibility status of page is changed,
> > + * this API should be called by the application who use WebKit, such as browser.
> 
> Weird line break here, could you make both lines have approximately the same width?
> 

I'll modify this.

> > Source/WebKit/efl/ewk/ewk_view.h:2185
> > + * When the visibility status of page is changed,
> > + * this API should be called by the application who use WebKit, such as browser.
> > + * Then the web application will get the visibility status of the page on which it runs,
> > + * it could stop or slow down if the visibility of the page is hidden.
> 
> The whole description is a bit unclear to me, can you phrase it in another way?

I'll try to write again.

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