Closed Bug 596155 Opened 10 years ago Closed 10 years ago

MTF widget is using deprecated API.

Categories

(Core Graveyard :: Widget: Qt, defect)

x86
MeeGo
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: roger.wang, Assigned: roger.wang)

References

Details

Attachments

(1 file)

With Meego Touch Framework 0.20.38-1, fennec built with MTF failed to render a black browser window, with a warning of using deprecated API MSceneWindow::appear(MSceneWindow::DeletionPolicy policy).

The attached patch fixes this and make the browser window shown correctly.
Blocks: 583135
Attachment #475008 - Flags: review?
(In reply to comment #0)
> Created attachment 475008 [details] [diff] [review]
> using updated api
> 
> With Meego Touch Framework 0.20.38-1, fennec built with MTF failed to render a
                                            s/failed to/failed and/
> black browser window, with a warning of using deprecated API
> MSceneWindow::appear(MSceneWindow::DeletionPolicy policy).
> 
> The attached patch fixes this and make the browser window shown correctly.
Attachment #475008 - Flags: review?(romaxa)
Attachment #475008 - Flags: review?(doug.turner)
Attachment #475008 - Flags: review?
Comment on attachment 475008 [details] [diff] [review]
using updated api

oleg is a good enough reviewer for this.
Attachment #475008 - Flags: review?(doug.turner)
Comment on attachment 475008 [details] [diff] [review]
using updated api

Make sense:
http://apidocs.meego.com/mtf/class_m_scene_window.html#a517747ad2df965dd67098ed1133aceaf
Please use appear(QGraphicsScene) or appear(MWindow) instead.
Attachment #475008 - Flags: review?(romaxa) → review+
Keywords: checkin-needed
a=npodb
Assignee: nobody → roger.wang
http://hg.mozilla.org/mozilla-central/rev/f8a74e29e55f
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.