http://trac.webkit.org/changeset/113656
https://bugs.webkit.org/show_bug.cgi?id=83542
test is broken on chromium-win (Requested by simonjam on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-04-09
Source/WebCore:
* Modules/intents/Intent.cpp:
(WebCore::Intent::Intent):
* Modules/intents/Intent.h:
(Intent):
* Modules/intents/Intent.idl:
* bindings/scripts/CodeGeneratorV8.pm:
(GenerateParametersCheck):
* bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
Source/WebKit/chromium:
* public/WebIntent.h:
(WebIntent):
* src/WebIntent.cpp:
Tools:
* DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::dispatchIntent):
LayoutTests:
* webintents/web-intents-invoke-port-expected.txt: Removed.
* webintents/web-intents-invoke-port.html: Removed.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@113671
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2012-04-09 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r113656.
+ http://trac.webkit.org/changeset/113656
+ https://bugs.webkit.org/show_bug.cgi?id=83542
+
+ test is broken on chromium-win (Requested by simonjam on
+ #webkit).
+
+ * webintents/web-intents-invoke-port-expected.txt: Removed.
+ * webintents/web-intents-invoke-port.html: Removed.
+
2012-04-09 Abhishek Arya <inferno@chromium.org>
ASSERTION FAILED: !attached() in Node::attach.
+++ /dev/null
-Received Web Intent: action=action1 type=mime/type1
-Have 1 ports
-PASS successfullyParsed is true
-
-TEST COMPLETE
-* sent intent
-
+++ /dev/null
-<html>
- <head>
- <script src="../fast/js/resources/js-test-pre.js"></script>
- <script src="resources/web-intents-testing.js"></script>
- <script>
- var channel = new MessageChannel();
- channel.port2.onMessage = function(e) {
- debug("* got message");
- };
-
- function buttonClicked() {
- try {
- var intent1 = new WebKitIntent("action1", "mime/type1", channel.port1, "not a port");
- testFailed("Should have thrown TypeError");
- } catch (e) {
- if (!(e instanceof TypeError)) testFailed("Should throw type error on non-port arg");
- }
- try {
- var intent1 = new WebKitIntent("action1", "mime/type1", channel.port1, ["not a port"]);
- testFailed("Should have thrown TypeError");
- } catch (e) {
- if (!(e instanceof TypeError)) testFailed("Should throw type error on non-port arg");
- }
-
- navigator.webkitStartActivity(new WebKitIntent("action1", "mime/type1", channel.port1, [channel.port1]));
- debug("* sent intent");
- }
- </script>
- </head>
-<body onload="simulateButtonPress()">
-<input type="button" id="button" value="Start Web Intent" onmouseup="buttonClicked()">
-<script src="../fast/js/resources/js-test-post.js"></script>
-</body>
-</html>
+2012-04-09 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r113656.
+ http://trac.webkit.org/changeset/113656
+ https://bugs.webkit.org/show_bug.cgi?id=83542
+
+ test is broken on chromium-win (Requested by simonjam on
+ #webkit).
+
+ * Modules/intents/Intent.cpp:
+ (WebCore::Intent::Intent):
+ * Modules/intents/Intent.h:
+ (Intent):
+ * Modules/intents/Intent.idl:
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GenerateParametersCheck):
+ * bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
+
2012-04-09 Abhishek Arya <inferno@chromium.org>
ASSERTION FAILED: !attached() in Node::attach.
#if ENABLE(WEB_INTENTS)
#include "ExceptionCode.h"
-#include "MessagePort.h"
#include "SerializedScriptValue.h"
namespace WebCore {
return adoptRef(new Intent(action, type, data));
}
-PassRefPtr<Intent> Intent::create(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data, const MessagePortArray& ports, ExceptionCode& ec)
-{
- if (action.isEmpty()) {
- ec = SYNTAX_ERR;
- return 0;
- }
- if (type.isEmpty()) {
- ec = SYNTAX_ERR;
- return 0;
- }
-
- OwnPtr<MessagePortChannelArray> channels = MessagePort::disentanglePorts(&ports, ec);
-
- return adoptRef(new Intent(action, type, data, channels.release()));
-}
-
Intent::Intent(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data)
: m_action(action)
, m_type(type)
{
if (data)
- m_data = data;
- else
- m_data = SerializedScriptValue::nullValue();
-}
-
-Intent::Intent(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data, PassOwnPtr<MessagePortChannelArray> ports)
- : m_action(action)
- , m_type(type)
- , m_ports(ports)
-{
- if (data)
- m_data = data;
+ m_data = SerializedScriptValue::createFromWire(data->toWireString());
else
m_data = SerializedScriptValue::nullValue();
}
return m_data.get();
}
-MessagePortChannelArray* Intent::messagePorts() const
-{
- return m_ports.get();
-}
-
} // namespace WebCore
#endif // ENABLE(WEB_INTENTS)
#if ENABLE(WEB_INTENTS)
-#include "MessagePort.h"
-#include "MessagePortChannel.h"
#include <wtf/Forward.h>
#include <wtf/PassRefPtr.h>
#include <wtf/RefCounted.h>
class Intent : public RefCounted<Intent> {
public:
static PassRefPtr<Intent> create(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data, ExceptionCode&);
- static PassRefPtr<Intent> create(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data, const MessagePortArray& ports, ExceptionCode&);
const String& action() const;
const String& type() const;
int identifier() const;
void setIdentifier(int);
- MessagePortChannelArray* messagePorts() const;
-
private:
Intent(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data);
- Intent(const String& action, const String& type, PassRefPtr<SerializedScriptValue> data, PassOwnPtr<MessagePortChannelArray> ports);
String m_action;
String m_type;
RefPtr<SerializedScriptValue> m_data;
- OwnPtr<MessagePortChannelArray> m_ports;
};
} // namespace WebCore
module window {
interface [
Conditional=WEB_INTENTS,
- Constructor(in DOMString action, in DOMString type, in [Optional=DefaultIsNullString, TransferList=transferList] SerializedScriptValue data, in [Optional=DefaultIsUndefined] Array transferList),
+ Constructor(in DOMString action, in DOMString type, in [Optional=DefaultIsNullString] SerializedScriptValue data),
ConstructorRaisesException
] Intent {
readonly attribute DOMString action;
die "IDL error: TransferList refers to a nonexistent argument";
}
- AddToImplIncludes("wtf/ArrayBuffer.h");
+ AddToImplIncludes("ArrayBuffer.h");
AddToImplIncludes("MessagePort.h");
$TransferListName = ucfirst($transferListName);
$parameterCheckString .= " MessagePortArray messagePortArray$TransferListName;\n";
#if ENABLE(Condition1) || ENABLE(Condition2)
+#include "ArrayBuffer.h"
#include "ExceptionCode.h"
#include "MessagePort.h"
#include "RuntimeEnabledFeatures.h"
#include "V8DOMWrapper.h"
#include "V8IsolatedContext.h"
#include "V8Proxy.h"
-#include <wtf/ArrayBuffer.h>
#include <wtf/UnusedParam.h>
namespace WebCore {
+2012-04-09 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r113656.
+ http://trac.webkit.org/changeset/113656
+ https://bugs.webkit.org/show_bug.cgi?id=83542
+
+ test is broken on chromium-win (Requested by simonjam on
+ #webkit).
+
+ * public/WebIntent.h:
+ (WebIntent):
+ * src/WebIntent.cpp:
+
2012-04-09 James Robinson <jamesr@chromium.org>
[chromium] More unreviewed OwnPtr<>::operator= related clang build fixes in unit tests.
#ifndef WebIntent_h
#define WebIntent_h
-#include "WebMessagePortChannel.h"
#include "platform/WebCommon.h"
#include "platform/WebPrivatePtr.h"
#include "platform/WebString.h"
-#include "platform/WebVector.h"
namespace WebCore { class Intent; }
WEBKIT_EXPORT WebString type() const;
WEBKIT_EXPORT WebString data() const;
- // Caller takes ownership of the ports.
- WEBKIT_EXPORT WebMessagePortChannelArray* messagePortChannelsRelease() const;
-
#if WEBKIT_IMPLEMENTATION
WebIntent(const WTF::PassRefPtr<WebCore::Intent>&);
#endif
#include "WebIntent.h"
#include "Intent.h"
-#include "PlatformMessagePortChannel.h"
#include "SerializedScriptValue.h"
namespace WebKit {
#endif
}
-WebMessagePortChannelArray* WebIntent::messagePortChannelsRelease() const
-{
- // Note: see PlatformMessagePortChannel::postMessageToRemote.
- WebMessagePortChannelArray* webChannels = 0;
- WebCore::MessagePortChannelArray* messagePorts = m_private->messagePorts();
- if (messagePorts) {
- webChannels = new WebMessagePortChannelArray(messagePorts->size());
- for (size_t i = 0; i < messagePorts->size(); ++i) {
- WebCore::PlatformMessagePortChannel* platformChannel = messagePorts->at(i)->channel();
- (*webChannels)[i] = platformChannel->webChannelRelease();
- (*webChannels)[i]->setClient(0);
- }
- }
-
- return webChannels;
-}
-
} // namespace WebKit
+2012-04-09 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r113656.
+ http://trac.webkit.org/changeset/113656
+ https://bugs.webkit.org/show_bug.cgi?id=83542
+
+ test is broken on chromium-win (Requested by simonjam on
+ #webkit).
+
+ * DumpRenderTree/chromium/WebViewHost.cpp:
+ (WebViewHost::dispatchIntent):
+
2012-04-09 Takashi Toyoshima <toyoshim@chromium.org>
Update pywebsocket to 0.7.4.
printf("Received Web Intent: action=%s type=%s\n",
request.intent().action().utf8().data(),
request.intent().type().utf8().data());
- WebMessagePortChannelArray* ports = request.intent().messagePortChannelsRelease();
m_currentRequest = request;
- if (ports) {
- printf("Have %zu ports\n", ports->size());
- for (size_t i = 0; i < ports->size(); ++i)
- (*ports)[i]->destroy();
- delete ports;
- }
}
// Public functions -----------------------------------------------------------