diff options
author | Benjamin Poulain <ikipou@gmail.com> | 2010-11-13 16:35:39 +0100 |
---|---|---|
committer | Benjamin Poulain <benjamin.poulain@nokia.com> | 2010-11-20 18:52:51 +0100 |
commit | eddae681ffdff853d3dceb0cefd1e57f89cdd0a9 (patch) | |
tree | ab1debd08b8aca58dfb1db77bad8f4a8d0a2e81e | |
parent | Use QObject cleaning for the destruction of WebView children (diff) | |
download | rekonq-eddae681ffdff853d3dceb0cefd1e57f89cdd0a9.tar.xz |
Update the name of WebTab attributes to follow the conventions
Most classes of Rekonq uses m_ as a prefix for attribute,
this patch update WebTab to follow this convention.
Reviewed by: Pierre Rossi
Reviewed by: Andrea Diamantini
-rw-r--r-- | src/webtab.cpp | 62 | ||||
-rw-r--r-- | src/webtab.h | 18 |
2 files changed, 40 insertions, 40 deletions
diff --git a/src/webtab.cpp b/src/webtab.cpp index 0a55060b..0bc7503c 100644 --- a/src/webtab.cpp +++ b/src/webtab.cpp @@ -48,9 +48,9 @@ WebTab::WebTab(QWidget *parent) : QWidget(parent) - , _view(new WebView(this)) - , _bar(new UrlBar(this)) - , _progress(0) + , m_webView(new WebView(this)) + , m_urlBar(new UrlBar(this)) + , m_progress(0) { setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding); @@ -58,13 +58,13 @@ WebTab::WebTab(QWidget *parent) l->setMargin(0); l->setSpacing(0); - l->addWidget(_view); - _view->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding); + l->addWidget(m_webView); + m_webView->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding); // fix focus handling - setFocusProxy(_view); + setFocusProxy(m_webView); - KWebWallet *wallet = _view->page()->wallet(); + KWebWallet *wallet = m_webView->page()->wallet(); if (wallet) { @@ -72,15 +72,15 @@ WebTab::WebTab(QWidget *parent) this, SLOT(createWalletBar(const QString &, const QUrl &))); } - connect(_view, SIGNAL(loadProgress(int)), this, SLOT(updateProgress(int))); - connect(_view, SIGNAL(loadFinished(bool)), this, SLOT(loadFinished(bool))); + connect(m_webView, SIGNAL(loadProgress(int)), this, SLOT(updateProgress(int))); + connect(m_webView, SIGNAL(loadFinished(bool)), this, SLOT(loadFinished(bool))); } WebTab::~WebTab() { - _walletBar.clear(); - _previewSelectorBar.clear(); + m_walletBar.clear(); + m_previewSelectorBar.clear(); } @@ -95,20 +95,20 @@ KUrl WebTab::url() void WebTab::updateProgress(int p) { - _progress = p; + m_progress = p; emit loadProgressing(); } void WebTab::loadFinished(bool) { - _progress = 0; + m_progress = 0; } bool WebTab::isPageLoading() { - return _progress != 0; + return m_progress != 0; } @@ -121,37 +121,37 @@ void WebTab::createWalletBar(const QString &key, const QUrl &url) return; KWebWallet *wallet = page()->wallet(); - if(_walletBar.isNull()) { - _walletBar = new WalletBar(this); - _walletBar.data()->onSaveFormData(key, url); - qobject_cast<QVBoxLayout *>(layout())->insertWidget(0, _walletBar.data() ); + if(m_walletBar.isNull()) { + m_walletBar = new WalletBar(this); + m_walletBar.data()->onSaveFormData(key, url); + qobject_cast<QVBoxLayout *>(layout())->insertWidget(0, m_walletBar.data() ); } else { disconnect(wallet); - _walletBar.data()->notifyUser(); + m_walletBar.data()->notifyUser(); } - connect(_walletBar.data(), SIGNAL(saveFormDataAccepted(const QString &)), + connect(m_walletBar.data(), SIGNAL(saveFormDataAccepted(const QString &)), wallet, SLOT(acceptSaveFormDataRequest(const QString &)), Qt::UniqueConnection); - connect(_walletBar.data(), SIGNAL(saveFormDataRejected(const QString &)), + connect(m_walletBar.data(), SIGNAL(saveFormDataRejected(const QString &)), wallet, SLOT(rejectSaveFormDataRequest(const QString &)), Qt::UniqueConnection); } void WebTab::createPreviewSelectorBar(int index) { - if(_previewSelectorBar.isNull()) { - _previewSelectorBar = new PreviewSelectorBar(index, this); - qobject_cast<QVBoxLayout *>(layout())->insertWidget(0, _previewSelectorBar.data()); + if(m_previewSelectorBar.isNull()) { + m_previewSelectorBar = new PreviewSelectorBar(index, this); + qobject_cast<QVBoxLayout *>(layout())->insertWidget(0, m_previewSelectorBar.data()); } else { - disconnect(_previewSelectorBar.data()); - _previewSelectorBar.data()->setIndex(index); - _previewSelectorBar.data()->notifyUser(); + disconnect(m_previewSelectorBar.data()); + m_previewSelectorBar.data()->setIndex(index); + m_previewSelectorBar.data()->notifyUser(); } - connect(page(), SIGNAL(loadStarted()), _previewSelectorBar.data(), SLOT(loadProgress()), Qt::UniqueConnection); - connect(page(), SIGNAL(loadProgress(int)), _previewSelectorBar.data(), SLOT(loadProgress()), Qt::UniqueConnection); - connect(page(), SIGNAL(loadFinished(bool)), _previewSelectorBar.data(), SLOT(loadFinished()), Qt::UniqueConnection); - connect(page()->mainFrame(), SIGNAL(urlChanged(QUrl)), _previewSelectorBar.data(), SLOT(verifyUrl()), Qt::UniqueConnection); + connect(page(), SIGNAL(loadStarted()), m_previewSelectorBar.data(), SLOT(loadProgress()), Qt::UniqueConnection); + connect(page(), SIGNAL(loadProgress(int)), m_previewSelectorBar.data(), SLOT(loadProgress()), Qt::UniqueConnection); + connect(page(), SIGNAL(loadFinished(bool)), m_previewSelectorBar.data(), SLOT(loadFinished()), Qt::UniqueConnection); + connect(page()->mainFrame(), SIGNAL(urlChanged(QUrl)), m_previewSelectorBar.data(), SLOT(verifyUrl()), Qt::UniqueConnection); } void WebTab::insertBar(NotificationBar *bar) diff --git a/src/webtab.h b/src/webtab.h index 3fd4f793..04d0a6e3 100644 --- a/src/webtab.h +++ b/src/webtab.h @@ -53,10 +53,10 @@ public: explicit WebTab(QWidget *parent = 0); ~WebTab(); - inline WebView *view() { return _view; } - UrlBar *urlBar() { return _bar; } - inline WebPage *page() { return view()->page(); } - inline int progress() { return _progress; } + inline WebView *view() const { return m_webView; } + UrlBar *urlBar() const { return m_urlBar; } + inline WebPage *page() const { return view()->page(); } + inline int progress() const { return m_progress; } KUrl url(); void createPreviewSelectorBar(int index); @@ -76,13 +76,13 @@ signals: void loadProgressing(); private: - WebView *_view; - UrlBar *_bar; + WebView *const m_webView; + UrlBar *const m_urlBar; - int _progress; + int m_progress; - QWeakPointer<WalletBar> _walletBar; - QWeakPointer<PreviewSelectorBar> _previewSelectorBar; + QWeakPointer<WalletBar> m_walletBar; + QWeakPointer<PreviewSelectorBar> m_previewSelectorBar; }; #endif |