[Webkit-unassigned] [Bug 35932] [EFL] Add ewk_view.{cpp, h} to WK/efl/ewk.

[bugzilla-daemon at webkit.org]

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


Gustavo Noronha (kov) <gns at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #50331|review?                     |review-
               Flag|                            |




--- Comment #5 from Gustavo Noronha (kov) <gns at gnome.org>  2010-03-16 13:24:11 PST ---
(From update of attachment 50331)
 169 static inline void _ewk_view_smart_changed(Ewk_View_Smart_Data* sd)

There are several inline markers in this code. In WebKit land we're averse to
premature, unmeasured optimization. If you have measured that this helps, this
should be OK, but if you haven't I'd recommend leaving this up to the compiler.

 478 static inline WTF::PassRefPtr<WebCore::Frame>
_ewk_view_core_frame_new(Ewk_View_Smart_Data* sd, Ewk_View_Private_Data* priv,
WebCore::HTMLFrameOwnerElement* owner)
[...]
 488     return WebCore::Frame::create(priv->page, owner, flc).get();

This is wrong. Frame::create will adopt the ref a RefPtr is holding, and return
a PassRefPtr already. What you're doing here is you are getting the raw
pointer, and returning it. When the returned PassRefPtr goes out of scope the
frame will likely be deleted.

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