Bug 29876
| Summary: | [Qt] Add QWebFrame::setHtml(QIODevice *) overload | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Tor Arne Vestbø <vestbo> |
| Component: | WebKit Qt | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Enhancement | CC: | hausmann, jedrzej.nowacki, tonikitoo |
| Priority: | P2 | Keywords: | Qt |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Other | ||
| OS: | OS X 10.5 | ||
| Bug Depends on: | 31115, 31552 | ||
| Bug Blocks: | |||
Tor Arne Vestbø
This bug report originated from issue QTBUG-2911
http://bugreports.qt.nokia.com/browse/QTBUG-2911
--- Description ---
Currently QWebFrame has a setHtml(QString), but an overload for QIODevice would be cool.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Jędrzej Nowacki
This is change for 4.7.
What is the use case? For a small file it is definitely better to use the QString and the setContent(), for a big file the load(QUrl) method should be enough.
The only use case I found is when a page is created on the runtime and it is big. So big that it is worth to stream it. Am I right?
Simon Hausmann
I agree with Jędrzej. setHtml() is a synchronous API that exists for convenience.
It is not suited for streaming (one potential use-case for QIODevice)