[Webkit-unassigned] [Bug 31358] [Qt] Various doc fixes

[bugzilla-daemon at webkit.org]

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





--- Comment #13 from Antonio Gomes (tonikitoo) <tonikitoo at webkit.org>  2009-11-11 12:17:50 PST ---
(In reply to comment #12)
> I'm afraid that that code was changed maybe before it got integrated. So we
> need to test it to make sure the docs are ok. Antonio can you do that?

So i just tested it w/ a basic app (see paste below), whereas no size is
specified to "webview".  When it opens it sizes 800x600, that is the preferred
size specified by QGraphicsWebView::sizeHint method. If I comment out this
method, QGraphicsWebView instance will have an invalid size. So it is
independent on the containing view.

So

"If the width and height of the item is not set, they will dynamically
adjust to a size appropriate for the content. This width may be large (e.g.,
980 pixels or more) for typical online Web pages."

is wrong imo.





#include <QApplication>
#include "qgraphicswebview.h"
#include <QGraphicsView>

int main(int argc, char *argv[])
{
    QApplication::setGraphicsSystem(QLatin1String("raster"));
    QApplication app(argc, argv);
    QGraphicsScene scene;
    QGraphicsView view(&scene);

    view.setOptimizationFlags(QGraphicsView::DontSavePainterState
                            | QGraphicsView::DontAdjustForAntialiasing);

    view.setViewportUpdateMode(QGraphicsView::MinimalViewportUpdate);
    view.setFrameShape(QFrame::NoFrame);

    QGraphicsWebView *webView = new QGraphicsWebView;
    webView->setFocus();

    scene.addItem(webView);
    scene.setActiveWindow(webView);

    view.resize(600, 480);
    view.show();

    webView->load(QUrl("http://google.com"));

    return app.exec();
}

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