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)