This commit is (quite) last in the urlbar encodings fix series. It fixes:

BUG: 233159
BUG: 234168
Moreover it "cleans" road to fix (not yet, but we are near..) another bug
CCBUG: 230771
at least from the rekonq side.

Changes in there
- cleaning and fixing filterurljob class (responsible for the encodings)
- using urls instead of strings in the resolver class (work with right data..)
- letting first box appearance without item selection and adding one signal to use
the filterurljob class directly.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/urlbar/completionwidget.cpp b/src/urlbar/completionwidget.cpp
index 740f1471..0733b1f0 100644
--- a/src/urlbar/completionwidget.cpp
+++ b/src/urlbar/completionwidget.cpp
@@ -97,7 +97,6 @@ void CompletionWidget::sizeAndPosition()
 
 void CompletionWidget::popup()
 {
-    down();
     sizeAndPosition();
     if (!isVisible()) 
         show();
@@ -210,7 +209,10 @@ bool CompletionWidget::eventFilter( QObject *o, QEvent *e )
                 case Qt::Key_Enter:
                 case Qt::Key_Return:
                         hide();
-                        emit chosenUrl(_list.at(_currentIndex).url, Rekonq::CurrentTab);
+                        if(_currentIndex >= 0)
+                            emit chosenUrl(_list.at(_currentIndex).url, Rekonq::CurrentTab);
+                        else
+                            emit loadTypedUrl();
                         ev->accept();
                         return true;
                     break;