diff options
author | Jocelyn Turcotte <jocelyn.turcotte@digia.com> | 2014-02-12 12:59:00 +0100 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2014-02-13 14:28:04 +0100 |
commit | 71edccabd4c359eb1ac9d6f6c3220b0d308f3a7b (patch) | |
tree | 4fdd781b1371c9ee4218cac37e9f44dcc5dbb509 /src/core/qt_render_view_observer_host.cpp | |
parent | 0bb596fbebd9317ef7acf00860cbcca45193fcee (diff) |
Move the requestID argument before the result
This tries to get the order of declaration to be consistent.
It also follows the order used by Chromium in some places
where the routing_id usually comes first, then the request_id
and then the result of the asynchronous request.
Change-Id: I88e164dee67e1631161a222f7dd7a4679c3d3acd
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
Diffstat (limited to 'src/core/qt_render_view_observer_host.cpp')
-rw-r--r-- | src/core/qt_render_view_observer_host.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/core/qt_render_view_observer_host.cpp b/src/core/qt_render_view_observer_host.cpp index 4b3cc0189..be811e0b9 100644 --- a/src/core/qt_render_view_observer_host.cpp +++ b/src/core/qt_render_view_observer_host.cpp @@ -75,12 +75,12 @@ bool QtRenderViewObserverHost::OnMessageReceived(const IPC::Message& message) } -void QtRenderViewObserverHost::onDidFetchDocumentMarkup(const base::string16& markup, quint64 requestId) +void QtRenderViewObserverHost::onDidFetchDocumentMarkup(quint64 requestId, const base::string16& markup) { - m_adapterClient->didFetchDocumentMarkup(toQt(markup), requestId); + m_adapterClient->didFetchDocumentMarkup(requestId, toQt(markup)); } -void QtRenderViewObserverHost::onDidFetchDocumentInnerText(const base::string16& innerText, quint64 requestId) +void QtRenderViewObserverHost::onDidFetchDocumentInnerText(quint64 requestId, const base::string16& innerText) { - m_adapterClient->didFetchDocumentInnerText(toQt(innerText), requestId); + m_adapterClient->didFetchDocumentInnerText(requestId, toQt(innerText)); } |