Commit 725013a7 authored by commit-queue@webkit.org's avatar commit-queue@webkit.org
Browse files

2011-05-13 Sheriff Bot <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r86447.
        http://trac.webkit.org/changeset/86447
        https://bugs.webkit.org/show_bug.cgi?id=60809

        "Broke some uses of EventSender object on Qt" (Requested by
        mwenge on #webkit).

        * platform/qt/Skipped:
        * platform/qt/http/tests/plugins/plugin-document-has-focus-expected.txt: Removed.
2011-05-13  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r86447.
        http://trac.webkit.org/changeset/86447
        https://bugs.webkit.org/show_bug.cgi?id=60809

        "Broke some uses of EventSender object on Qt" (Requested by
        mwenge on #webkit).

        * WebCoreSupport/DumpRenderTreeSupportQt.cpp:
        * WebCoreSupport/DumpRenderTreeSupportQt.h:
2011-05-13  Sheriff Bot  <webkit.review.bot@gmail.com>

        Unreviewed, rolling out r86447.
        http://trac.webkit.org/changeset/86447
        https://bugs.webkit.org/show_bug.cgi?id=60809

        "Broke some uses of EventSender object on Qt" (Requested by
        mwenge on #webkit).

        * DumpRenderTree/qt/DumpRenderTreeQt.cpp:
        (WebCore::WebPage::WebPage):
        (WebCore::DumpRenderTree::DumpRenderTree):
        (WebCore::DumpRenderTree::initJSObjects):
        (WebCore::DumpRenderTree::createWindow):
        * DumpRenderTree/qt/DumpRenderTreeQt.h:
        (WebCore::DumpRenderTree::eventSender):
        * DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:
        (webkit_test_plugin_new_instance):
        (webkit_test_plugin_set_window):
        (webkit_test_plugin_handle_event):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@86464 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent c2bcfc8b
2011-05-13 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r86447.
http://trac.webkit.org/changeset/86447
https://bugs.webkit.org/show_bug.cgi?id=60809
"Broke some uses of EventSender object on Qt" (Requested by
mwenge on #webkit).
* platform/qt/Skipped:
* platform/qt/http/tests/plugins/plugin-document-has-focus-expected.txt: Removed.
2011-05-13 Adam Barth <abarth@webkit.org>
 
Reviewed by Eric Seidel.
......@@ -357,6 +357,8 @@ plugins/embed-attributes-style.html
# ---- depends on QuickTime support
plugins/embed-attributes-setting.html
plugins/plugin-initiate-popup-window.html
# ---- to be reviewed
http/tests/plugins/plugin-document-has-focus.html
# [Qt] plugins/keyboard-events.html fails after r72717
# https://bugs.webkit.org/show_bug.cgi?id=50050
......
CONSOLE MESSAGE: line 0: PLUGIN: keyDown 'a'
CONSOLE MESSAGE: line 0: PLUGIN: keyUp 'a'
This tests that a full page plugin gets keyboard focus without the need to first click it.
2011-05-13 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r86447.
http://trac.webkit.org/changeset/86447
https://bugs.webkit.org/show_bug.cgi?id=60809
"Broke some uses of EventSender object on Qt" (Requested by
mwenge on #webkit).
* WebCoreSupport/DumpRenderTreeSupportQt.cpp:
* WebCoreSupport/DumpRenderTreeSupportQt.h:
2011-05-13 Sam Weinig <sam@webkit.org>
Reviewed by Anders Carlsson.
......
......@@ -1142,11 +1142,6 @@ QString DumpRenderTreeSupportQt::layerTreeAsText(QWebFrame* frame)
return coreFrame->layerTreeAsText();
}
void DumpRenderTreeSupportQt::setView(QWebPage* page, QWidget* view)
{
page->d->view = view;
}
// Provide a backward compatibility with previously exported private symbols as of QtWebKit 4.6 release
void QWEBKIT_EXPORT qt_resumeActiveDOMObjects(QWebFrame* frame)
......
......@@ -107,7 +107,6 @@ public:
static void setCaretBrowsingEnabled(QWebPage* page, bool value);
static void setMediaType(QWebFrame* qframe, const QString& type);
static void setDumpRenderTreeModeEnabled(bool b);
static void setView(QWebPage*, QWidget* view);
static void garbageCollectorCollect();
static void garbageCollectorCollectOnAlternateThread(bool waitUntilDone);
......
2011-05-13 Sheriff Bot <webkit.review.bot@gmail.com>
Unreviewed, rolling out r86447.
http://trac.webkit.org/changeset/86447
https://bugs.webkit.org/show_bug.cgi?id=60809
"Broke some uses of EventSender object on Qt" (Requested by
mwenge on #webkit).
* DumpRenderTree/qt/DumpRenderTreeQt.cpp:
(WebCore::WebPage::WebPage):
(WebCore::DumpRenderTree::DumpRenderTree):
(WebCore::DumpRenderTree::initJSObjects):
(WebCore::DumpRenderTree::createWindow):
* DumpRenderTree/qt/DumpRenderTreeQt.h:
(WebCore::DumpRenderTree::eventSender):
* DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:
(webkit_test_plugin_new_instance):
(webkit_test_plugin_set_window):
(webkit_test_plugin_handle_event):
2011-05-13 Robert Hogan <robert@webkit.org>
Reviewed by Benjamin Poulain.
......
......@@ -136,7 +136,6 @@ public:
WebPage::WebPage(QObject* parent, DumpRenderTree* drt)
: QWebPage(parent)
, m_webInspector(0)
, m_eventSender(0)
, m_drt(drt)
{
QWebSettings* globalSettings = QWebSettings::globalSettings();
......@@ -170,14 +169,6 @@ WebPage::~WebPage()
delete m_webInspector;
}
EventSender* WebPage::eventSender()
{
if (!m_eventSender)
m_eventSender = new EventSender(this);
return m_eventSender;
}
QWebInspector* WebPage::webInspector()
{
if (!m_webInspector) {
......@@ -473,7 +464,7 @@ DumpRenderTree::DumpRenderTree()
connect(m_controller, SIGNAL(geolocationPermissionSet()), this, SLOT(geolocationPermissionSet()));
connect(m_controller, SIGNAL(done()), this, SLOT(dump()));
m_eventSender = m_page->eventSender();
m_eventSender = new EventSender(m_page);
m_textInputController = new TextInputController(m_page);
m_plainTextController = new PlainTextController(m_page);
m_gcController = new GCController(m_page);
......@@ -784,7 +775,7 @@ void DumpRenderTree::initJSObjects()
QWebFrame *frame = qobject_cast<QWebFrame*>(sender());
Q_ASSERT(frame);
frame->addToJavaScriptWindowObject(QLatin1String("layoutTestController"), m_controller);
frame->addToJavaScriptWindowObject(QLatin1String("eventSender"), qobject_cast<WebPage*>(frame->page())->eventSender());
frame->addToJavaScriptWindowObject(QLatin1String("eventSender"), m_eventSender);
frame->addToJavaScriptWindowObject(QLatin1String("textInputController"), m_textInputController);
frame->addToJavaScriptWindowObject(QLatin1String("GCController"), m_gcController);
frame->addToJavaScriptWindowObject(QLatin1String("plainText"), m_plainTextController);
......@@ -1106,10 +1097,6 @@ QWebPage *DumpRenderTree::createWindow()
QObject* container = new QObject(m_mainView);
// create a QWebPage we want to return
QWebPage* page = static_cast<QWebPage*>(new WebPage(container, this));
// QWebPage::setView() will display the page, so use a minimal version
// that is just enough to allow EventSender to interact with the page's view
DumpRenderTreeSupportQt::setView(page, m_mainView);
// gets cleaned up in closeRemainingWindows()
windows.append(container);
......
......@@ -89,6 +89,7 @@ public:
void resetToConsistentStateBeforeTesting(const QUrl&);
LayoutTestController *layoutTestController() const { return m_controller; }
EventSender *eventSender() const { return m_eventSender; }
TextInputController *textInputController() const { return m_textInputController; }
QString persistentStoragePath() const { return m_persistentStoragePath; }
NetworkAccessManager *networkAccessManager() const { return m_networkAccessManager; }
......@@ -183,7 +184,6 @@ public:
WebPage(QObject* parent, DumpRenderTree*);
virtual ~WebPage();
QWebInspector* webInspector();
EventSender* eventSender();
void closeWebInspector();
QWebPage *createWindow(QWebPage::WebWindowType);
......@@ -216,7 +216,6 @@ private slots:
private:
QWebInspector* m_webInspector;
EventSender* m_eventSender;
QList<QWebFrame*> m_pendingGeolocationRequests;
DumpRenderTree *m_drt;
};
......
......@@ -98,10 +98,6 @@ webkit_test_plugin_new_instance(NPMIMEType mimetype,
obj->testWindowOpen = TRUE;
else if (strcasecmp(argn[i], "onSetWindow") == 0 && !obj->onSetWindow)
obj->onSetWindow = strdup(argv[i]);
else if (!strcasecmp(argn[i], "src") && strstr(argv[i], "plugin-document-has-focus.pl")) {
browser->setvalue(instance, NPPVpluginWindowBool, false);
obj->testKeyboardFocusForPlugins = TRUE;
}
}
browser->getvalue(instance, NPNVprivateModeBool, (void *)&obj->cachedPrivateBrowsingMode);
......@@ -167,11 +163,6 @@ webkit_test_plugin_set_window(NPP instance, NPWindow *window)
obj->testWindowOpen = FALSE;
}
if (obj->testKeyboardFocusForPlugins) {
obj->eventLogging = true;
browser->setvalue(instance, NPPVpluginWindowBool, false);
executeScript(obj, "eventSender.keyDown('A');");
}
}
return obj->pluginTest->NPP_SetWindow(instance, window);
......@@ -301,11 +292,6 @@ webkit_test_plugin_handle_event(NPP instance, void* event)
break;
case KeyRelease:
pluginLog(instance, "keyUp '%c'", keyEventToChar(&evt->xkey));
if (obj->testKeyboardFocusForPlugins) {
obj->eventLogging = false;
obj->testKeyboardFocusForPlugins = FALSE;
executeScript(obj, "layoutTestController.notifyDone();");
}
break;
case KeyPress:
pluginLog(instance, "keyDown '%c'", keyEventToChar(&evt->xkey));
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment