Python: cleaned up the code for the WebBrowserWindow plugin.

src/plugins/miscellaneous/WebBrowserWindow/src/webbrowserwindowplugin.cpp

@@ -159,6 +159,17 @@ void WebBrowserWindowPlugin::handleUrl(const QUrl &pUrl)
     // We don't handle this interface...
 }
 
+//==============================================================================
+// Python interface
+//==============================================================================
+
+void WebBrowserWindowPlugin::registerPythonClasses(PyObject *pModule)
+{
+    // Register our Python classes
+
+    new WebBrowserWindowPythonWrapper(pModule, this);
+}
+
 //==============================================================================
 // Window interface
 //==============================================================================
@@ -188,17 +199,6 @@ QDockWidget * WebBrowserWindowPlugin::windowWidget() const
     return mWebBrowserWindowWindow;
 }
 
-//==============================================================================
-// Python interface
-//==============================================================================
-
-void WebBrowserWindowPlugin::registerPythonClasses(PyObject *pModule)
-{
-    // Register our Python classes
-
-    new WebBrowserWindowPythonWrapper(pModule, this);
-}
-
 //==============================================================================
 // Plugin specific
 //==============================================================================
@@ -207,18 +207,18 @@ WebBrowserWindowPlugin * WebBrowserWindowPlugin::instance()
 {
     // Return the 'global' instance of our plugin
 
-    static WebBrowserWindowPlugin pluginInstance;
+    static WebBrowserWindowPlugin instance;
 
-    return static_cast<WebBrowserWindowPlugin *>(Core::globalInstance("OpenCOR::WebBrowserWindow::WebBrowserWindowPlugin",
-                                                 &pluginInstance));
+    return static_cast<WebBrowserWindowPlugin *>(Core::globalInstance("OpenCOR::WebBrowserWindow::WebBrowserWindowPlugin::instance()",
+                                                 &instance));
 }
 
 
 //==============================================================================
 
-WebBrowserWindowWidget * WebBrowserWindowPlugin::webBrowserWidget() const
+WebBrowserWindowWidget * WebBrowserWindowPlugin::webBrowserWindowWidget() const
 {
-    // Return our Web Browser widget
+    // Return our Web Browser window widget
 
     return mWebBrowserWindowWidget;
 }

src/plugins/miscellaneous/WebBrowserWindow/src/webbrowserwindowplugin.h

@@ -68,7 +68,7 @@ class WebBrowserWindowPlugin : public QObject, public I18nInterface,
 
     static WebBrowserWindowPlugin * instance();
 
-    WebBrowserWindowWidget * webBrowserWidget() const;
+    WebBrowserWindowWidget * webBrowserWindowWidget() const;
 
 private:
     QAction *mWebBrowserWindowAction = nullptr;

src/plugins/miscellaneous/WebBrowserWindow/src/webbrowserwindowpythonwrapper.cpp

@@ -18,10 +18,12 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *******************************************************************************/
 
 //==============================================================================
-// Python wrapper for web browser window
+// Web Browser window Python wrapper
 //==============================================================================
 
+#include "pythonqtsupport.h"
 #include "webbrowserwindowplugin.h"
+#include "webbrowserwindowpythonwrapper.h"
 #include "webbrowserwindowwidget.h"
 
 //==============================================================================
@@ -30,11 +32,6 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 //==============================================================================
 
-#include "pythonqtsupport.h"
-#include "webbrowserwindowpythonwrapper.h"
-
-//==============================================================================
-
 #include <array>
 
 //==============================================================================
@@ -44,14 +41,14 @@ namespace WebBrowserWindow {
 
 //==============================================================================
 
-static PyObject * browserWebView(PyObject *pSelf, PyObject *pArgs)
+static PyObject * webBrowserWindowWebView(PyObject *pSelf, PyObject *pArgs)
 {
     Q_UNUSED(pSelf)
     Q_UNUSED(pArgs)
 
-    // Return a Python object to our Web browser's Web view
+    // Return a Python object to our Web browser window's Web view
 
-    return PythonQtSupport::wrapQObject(WebBrowserWindowPlugin::instance()->webBrowserWidget()->webView());
+    return PythonQtSupport::wrapQObject(WebBrowserWindowPlugin::instance()->webBrowserWindowWidget()->webView());
 }
 //==============================================================================
 
@@ -62,7 +59,7 @@ WebBrowserWindowPythonWrapper::WebBrowserWindowPythonWrapper(PyObject *pModule,
     // Add some Python wrappers
 
     static std::array<PyMethodDef, 2> PythonWebBrowserWindowMethods = {{
-                                                                          { "browserWebView",  browserWebView, METH_VARARGS, "browserWebView()\n\nReturn a QWebView of OpenCOR's web browser." },
+                                                                          { "webBrowserWindowWebView",  webBrowserWindowWebView, METH_VARARGS, "Return OpenCOR's Web Browser window's QWebView object." },
                                                                           { nullptr, nullptr, 0, nullptr }
                                                                       }};
 

src/plugins/miscellaneous/WebBrowserWindow/src/webbrowserwindowpythonwrapper.h

@@ -18,7 +18,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *******************************************************************************/
 
 //==============================================================================
-// Python wrapper for web browser window
+// Web Browser window Python wrapper
 //==============================================================================
 
 #pragma once
@@ -36,9 +36,6 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 //==============================================================================
 
 namespace OpenCOR {
-
-//==============================================================================
-
 namespace WebBrowserWindow {
 
 //==============================================================================

src/plugins/miscellaneous/WebBrowserWindow/src/webbrowserwindowwindow.cpp

@@ -261,7 +261,7 @@ void WebBrowserWindowWindow::saveSettings(QSettings &pSettings) const
 
 //==============================================================================
 
-WebBrowserWindowWidget *WebBrowserWindowWindow::widget() const
+WebBrowserWindowWidget * WebBrowserWindowWindow::widget() const
 {
     return mWebBrowserWindowWidget;
 }