From 8806bb67acdcba5a9fd54677907d1b7fa79921ea Mon Sep 17 00:00:00 2001
From: Teo Mrnjavac <teo@kde.org>
Date: Thu, 24 Mar 2016 20:03:05 +0100
Subject: [PATCH] Revert "Merge pull request #239 from calamares/qtwebengine"

This reverts commit 8f0fafa1472e57e9efa935745998b17007a494c6, reversing
changes made to b8729bf1b391e48e987d1c83c214072c63d12565.
---
 src/modules/webview/CMakeLists.txt     | 41 ++++----------------------
 src/modules/webview/WebViewConfig.h.in |  7 -----
 src/modules/webview/WebViewStep.cpp    | 18 +----------
 src/modules/webview/WebViewStep.h      | 13 ++------
 4 files changed, 10 insertions(+), 69 deletions(-)
 delete mode 100644 src/modules/webview/WebViewConfig.h.in

diff --git a/src/modules/webview/CMakeLists.txt b/src/modules/webview/CMakeLists.txt
index c4b0107a3..eb5c81a68 100644
--- a/src/modules/webview/CMakeLists.txt
+++ b/src/modules/webview/CMakeLists.txt
@@ -1,42 +1,11 @@
-list( APPEND CALA_WEBVIEW_INCLUDE_DIRECTORIES
-    ${PROJECT_BINARY_DIR}/src/libcalamaresui
-)
-list( APPEND CALA_WEBVIEW_LINK_LIBRARIES
-    calamaresui
-)
+find_package( Qt5 ${QT_VERSION} CONFIG REQUIRED WebKit WebKitWidgets )
 
-
-if ( Qt5Core_VERSION VERSION_LESS 5.6 )
-    find_package( Qt5 ${QT_VERSION} CONFIG REQUIRED WebKit WebKitWidgets )
-
-    list( APPEND CALA_WEBVIEW_INCLUDE_DIRECTORIES
-        ${QT_QTWEBKIT_INCLUDE_DIR}
-    )
-    list( APPEND CALA_WEBVIEW_LINK_LIBRARIES
-        Qt5::WebKit
-        Qt5::WebKitWidgets
-    )
-    set( WEBVIEW_WITH_WEBKIT 1 )
-else()
-    find_package( Qt5 ${QT_VERSION} CONFIG REQUIRED WebEngine WebEngineWidgets )
-
-    list( APPEND CALA_WEBVIEW_INCLUDE_DIRECTORIES
-        ${QT_QTWEBENGINE_INCLUDE_DIR}
-    )
-    list( APPEND CALA_WEBVIEW_LINK_LIBRARIES
-        Qt5::WebEngine
-        Qt5::WebEngineWidgets
-    )
-    set( WEBVIEW_WITH_WEBENGINE 1 )
-endif()
-
-include_directories( ${CALA_WEBVIEW_INCLUDE_DIRECTORIES} )
+include_directories( ${PROJECT_BINARY_DIR}/src/libcalamaresui
+                     ${QT_QTWEBKIT_INCLUDE_DIR} )
 
 set( CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH}
                        ${CMAKE_CURRENT_SOURCE_DIR}/CMakeModules )
 
-configure_file( ${CMAKE_CURRENT_SOURCE_DIR}/WebViewConfig.h.in
-                ${CMAKE_CURRENT_BINARY_DIR}/WebViewConfig.h )
 
 calamares_add_plugin( webview
     TYPE viewmodule
@@ -44,6 +13,8 @@ calamares_add_plugin( webview
     SOURCES
         WebViewStep.cpp
     LINK_LIBRARIES
-        ${CALA_WEBVIEW_LINK_LIBRARIES}
+        calamaresui
+        Qt5::WebKit
+        Qt5::WebKitWidgets
     SHARED_LIB
 )
diff --git a/src/modules/webview/WebViewConfig.h.in b/src/modules/webview/WebViewConfig.h.in
deleted file mode 100644
index 6611e44c0..000000000
--- a/src/modules/webview/WebViewConfig.h.in
+++ /dev/null
@@ -1,7 +0,0 @@
-#ifndef CALAMARESWEBVIEWCONFIG_H
-#define CALAMARESWEBVIEWCONFIG_H
-
-#cmakedefine WEBVIEW_WITH_WEBENGINE
-#cmakedefine WEBVIEW_WITH_WEBKIT
-
-#endif // CALAMARESWEBVIEWCONFIG_H
diff --git a/src/modules/webview/WebViewStep.cpp b/src/modules/webview/WebViewStep.cpp
index b1d50bf7c..7fa5b5989 100644
--- a/src/modules/webview/WebViewStep.cpp
+++ b/src/modules/webview/WebViewStep.cpp
@@ -1,7 +1,6 @@
 /* === This file is part of Calamares - <http://github.com/calamares> ===
  *
  *   Copyright 2015, Rohan Garg <rohan@garg.io>
- *   Copyright 2016, Teo Mrnjavac <teo@kde.org>
  *
  *   Calamares is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -20,12 +19,7 @@
 #include "WebViewStep.h"
 
 #include <QVariant>
-
-#ifdef WEBVIEW_WITH_WEBKIT
 #include <QWebView>
-#else
-#include <QWebEngineView>
-#endif
 
 CALAMARES_PLUGIN_FACTORY_DEFINITION( WebViewStepFactory, registerPlugin<WebViewStep>(); )
 
@@ -33,17 +27,7 @@ WebViewStep::WebViewStep( QObject* parent )
     : Calamares::ViewStep( parent )
 {
     emit nextStatusChanged( true );
-    m_view = new C_QWEBVIEW();
-#ifdef WEBVIEW_WITH_WEBKIT
-    m_view->settings()->setFontFamily( QWebSettings::StandardFont,
-                                       m_view->settings()->
-                                       fontFamily( QWebSettings::SansSerifFont ) );
-    m_view->setRenderHints( QPainter::Antialiasing |
-                            QPainter::TextAntialiasing |
-                            QPainter::HighQualityAntialiasing |
-                            QPainter::SmoothPixmapTransform |
-                            QPainter::NonCosmeticDefaultPen );
-#endif
+    m_view = new QWebView;
 }
 
 
diff --git a/src/modules/webview/WebViewStep.h b/src/modules/webview/WebViewStep.h
index 9d8405a97..af95ea269 100644
--- a/src/modules/webview/WebViewStep.h
+++ b/src/modules/webview/WebViewStep.h
@@ -1,7 +1,6 @@
 /* === This file is part of Calamares - <http://github.com/calamares> ===
  *
  *   Copyright 2015, Rohan Garg <rohan@garg.io>
- *   Copyright 2016, Teo Mrnjavac <teo@kde.org>
  *
  *   Calamares is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -20,7 +19,7 @@
 #ifndef WEBVIEWPLUGIN_H
 #define WEBVIEWPLUGIN_H
 
-#include "WebViewConfig.h"
+#include <QObject>
 
 #include <utils/PluginFactory.h>
 #include <viewpages/ViewStep.h>
@@ -29,13 +28,7 @@
 
 #include <QVariantMap>
 
-#ifdef WEBVIEW_WITH_WEBKIT
-#define C_QWEBVIEW QWebView
-#else
-#define C_QWEBVIEW QWebEngineView
-#endif
-
-class C_QWEBVIEW;
+class QWebView;
 
 class PLUGINDLLEXPORT WebViewStep : public Calamares::ViewStep
 {
@@ -64,7 +57,7 @@ public:
     void setConfigurationMap( const QVariantMap& configurationMap ) override;
 
 private:
-    C_QWEBVIEW *m_view;
+    QWebView *m_view;
     QString m_url;
     QString m_prettyName;
 };