Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(111)

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp

Issue 2379653006: Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (Closed)
Patch Set: Moved include statement from surface_id_struct_traits.h to local_frame_id_struct_traits.h Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp b/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
index 16e3ae1e50969e8fb8c9109ed4bb4c8a7a5686d3..219505a8b271e94cbf8161c50ced91bca973ed1b 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptValueSerializer.cpp
@@ -357,7 +357,8 @@ void SerializedScriptValueWriter::writeTransferredOffscreenCanvas(
uint32_t clientId,
uint32_t sinkId,
uint32_t localId,
- uint64_t nonce) {
+ uint64_t nonceHigh,
+ uint64_t nonceLow) {
append(OffscreenCanvasTransferTag);
doWriteUint32(width);
doWriteUint32(height);
@@ -365,7 +366,8 @@ void SerializedScriptValueWriter::writeTransferredOffscreenCanvas(
doWriteUint32(clientId);
doWriteUint32(sinkId);
doWriteUint32(localId);
- doWriteUint64(nonce);
+ doWriteUint64(nonceHigh);
+ doWriteUint64(nonceLow);
}
void SerializedScriptValueWriter::writeTransferredSharedArrayBuffer(
@@ -1304,7 +1306,7 @@ ScriptValueSerializer::writeTransferredOffscreenCanvas(
offscreenCanvas->width(), offscreenCanvas->height(),
offscreenCanvas->getAssociatedCanvasId(), offscreenCanvas->clientId(),
offscreenCanvas->sinkId(), offscreenCanvas->localId(),
- offscreenCanvas->nonce());
+ offscreenCanvas->nonceHigh(), offscreenCanvas->nonceLow());
return nullptr;
}
@@ -1685,7 +1687,7 @@ bool SerializedScriptValueReader::readWithTag(
if (!m_version)
return false;
uint32_t width, height, canvasId, clientId, sinkId, localId;
- uint64_t nonce;
+ uint64_t nonceHigh, nonceLow;
if (!doReadUint32(&width))
return false;
if (!doReadUint32(&height))
@@ -1698,10 +1700,11 @@ bool SerializedScriptValueReader::readWithTag(
return false;
if (!doReadUint32(&localId))
return false;
- if (!doReadUint64(&nonce))
+ if (!doReadUint64(&nonceHigh) || !doReadUint64(&nonceLow))
return false;
if (!deserializer.tryGetTransferredOffscreenCanvas(
- width, height, canvasId, clientId, sinkId, localId, nonce, value))
+ width, height, canvasId, clientId, sinkId, localId, nonceHigh,
+ nonceLow, value))
return false;
break;
}
@@ -2552,11 +2555,12 @@ bool ScriptValueDeserializer::tryGetTransferredOffscreenCanvas(
uint32_t clientId,
uint32_t sinkId,
uint32_t localId,
- uint64_t nonce,
+ uint64_t nonceHigh,
+ uint64_t nonceLow,
v8::Local<v8::Value>* object) {
OffscreenCanvas* offscreenCanvas = OffscreenCanvas::create(width, height);
offscreenCanvas->setAssociatedCanvasId(canvasId);
- offscreenCanvas->setSurfaceId(clientId, sinkId, localId, nonce);
+ offscreenCanvas->setSurfaceId(clientId, sinkId, localId, nonceHigh, nonceLow);
*object = toV8(offscreenCanvas, m_reader.getScriptState());
if ((*object).IsEmpty())
return false;

Powered by Google App Engine
This is Rietveld 408576698