Diff
Modified: trunk/LayoutTests/ChangeLog (238069 => 238070)
--- trunk/LayoutTests/ChangeLog 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/LayoutTests/ChangeLog 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,3 +1,15 @@
+2018-11-10 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r238065.
+
+ Breaks internal builds.
+
+ Reverted changeset:
+
+ "Make it possible to edit images inline"
+ https://bugs.webkit.org/show_bug.cgi?id=191352
+ https://trac.webkit.org/changeset/238065
+
2018-11-10 Chris Dumez <cdu...@apple.com>
Unreviewed, fix typo in r238066.
Modified: trunk/LayoutTests/TestExpectations (238069 => 238070)
--- trunk/LayoutTests/TestExpectations 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/LayoutTests/TestExpectations 2018-11-10 21:01:36 UTC (rev 238070)
@@ -48,7 +48,6 @@
http/tests/gzip-content-encoding [ Skip ]
http/tests/cookies/same-site [ Skip ]
system-preview [ Skip ]
-editing/images [ Skip ]
# window.showModalDialog is only tested in DumpRenderTree on Mac.
editing/execCommand/show-modal-dialog-during-execCommand.html [ Skip ]
Modified: trunk/LayoutTests/resources/ui-helper.js (238069 => 238070)
--- trunk/LayoutTests/resources/ui-helper.js 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/LayoutTests/resources/ui-helper.js 2018-11-10 21:01:36 UTC (rev 238070)
@@ -398,24 +398,4 @@
})()`, scaleAsString => resolve(parseFloat(scaleAsString)))
});
}
-
- static drawSquareInEditableImage()
- {
- if (!this.isWebKit2())
- return Promise.resolve();
-
- return new Promise(resolve => testRunner.runUIScript(`uiController.drawSquareInEditableImage()`, resolve));
- }
-
- static numberOfStrokesInEditableImage()
- {
- if (!this.isWebKit2())
- return Promise.resolve();
-
- return new Promise(resolve => {
- testRunner.runUIScript(`(() => {
- uiController.uiScriptComplete(uiController.numberOfStrokesInEditableImage);
- })()`, numberAsString => resolve(parseInt(numberAsString, 10)))
- });
- }
}
Modified: trunk/Source/WebCore/ChangeLog (238069 => 238070)
--- trunk/Source/WebCore/ChangeLog 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/ChangeLog 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,3 +1,15 @@
+2018-11-10 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r238065.
+
+ Breaks internal builds.
+
+ Reverted changeset:
+
+ "Make it possible to edit images inline"
+ https://bugs.webkit.org/show_bug.cgi?id=191352
+ https://trac.webkit.org/changeset/238065
+
2018-11-10 Tim Horton <timothy_hor...@apple.com>
Make it possible to edit images inline
Modified: trunk/Source/WebCore/html/HTMLAttributeNames.in (238069 => 238070)
--- trunk/Source/WebCore/html/HTMLAttributeNames.in 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/html/HTMLAttributeNames.in 2018-11-10 21:01:36 UTC (rev 238070)
@@ -420,5 +420,3 @@
x-apple-data-detectors-result
x-apple-pdf-annotation
-
-x-apple-editable-image
Modified: trunk/Source/WebCore/html/HTMLImageElement.cpp (238069 => 238070)
--- trunk/Source/WebCore/html/HTMLImageElement.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/html/HTMLImageElement.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -737,11 +737,4 @@
}
#endif
-GraphicsLayer::EmbeddedViewID HTMLImageElement::editableImageViewID() const
-{
- if (!m_editableImageViewID)
- m_editableImageViewID = GraphicsLayer::nextEmbeddedViewID();
- return m_editableImageViewID;
}
-
-}
Modified: trunk/Source/WebCore/html/HTMLImageElement.h (238069 => 238070)
--- trunk/Source/WebCore/html/HTMLImageElement.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/html/HTMLImageElement.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -25,7 +25,6 @@
#include "DecodingOptions.h"
#include "FormNamedItem.h"
-#include "GraphicsLayer.h"
#include "GraphicsTypes.h"
#include "HTMLElement.h"
#include "HTMLImageLoader.h"
@@ -114,8 +113,6 @@
WEBCORE_EXPORT bool isSystemPreviewImage() const;
#endif
- GraphicsLayer::EmbeddedViewID editableImageViewID() const;
-
protected:
HTMLImageElement(const QualifiedName&, Document&, HTMLFormElement* = 0);
@@ -152,14 +149,6 @@
ImageCandidate bestFitSourceFromPictureElement();
-#if ENABLE(SERVICE_CONTROLS)
- void updateImageControls();
- void tryCreateImageControls();
- void destroyImageControls();
- bool hasImageControls() const;
- bool childShouldCreateRenderer(const Node&) const override;
-#endif
-
HTMLImageLoader m_imageLoader;
HTMLFormElement* m_form;
HTMLFormElement* m_formSetByParser;
@@ -172,7 +161,13 @@
bool m_experimentalImageMenuEnabled;
bool m_hadNameBeforeAttributeChanged { false }; // FIXME: We only need this because parseAttribute() can't see the old value.
- mutable GraphicsLayer::EmbeddedViewID m_editableImageViewID { 0 };
+#if ENABLE(SERVICE_CONTROLS)
+ void updateImageControls();
+ void tryCreateImageControls();
+ void destroyImageControls();
+ bool hasImageControls() const;
+ bool childShouldCreateRenderer(const Node&) const override;
+#endif
friend class HTMLPictureElement;
};
Modified: trunk/Source/WebCore/page/Settings.yaml (238069 => 238070)
--- trunk/Source/WebCore/page/Settings.yaml 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/page/Settings.yaml 2018-11-10 21:01:36 UTC (rev 238070)
@@ -763,6 +763,3 @@
videoQualityIncludesDisplayCompositingEnabled:
initial: false
conditional: VIDEO
-
-editableImagesEnabled:
- initial: false
Modified: trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/GraphicsLayer.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -739,12 +739,6 @@
traverse(*maskLayer, traversalFunc);
}
-GraphicsLayer::EmbeddedViewID GraphicsLayer::nextEmbeddedViewID()
-{
- static GraphicsLayer::EmbeddedViewID nextEmbeddedViewID;
- return ++nextEmbeddedViewID;
-}
-
void GraphicsLayer::dumpLayer(TextStream& ts, LayerTreeAsTextBehavior behavior) const
{
ts << indent << "(" << "GraphicsLayer";
Modified: trunk/Source/WebCore/platform/graphics/GraphicsLayer.h (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/GraphicsLayer.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/GraphicsLayer.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -476,21 +476,11 @@
Media,
Canvas,
BackgroundColor,
- Plugin,
- EmbeddedView
+ Plugin
};
- enum class ContentsLayerEmbeddedViewType : uint8_t {
- None = 0,
- EditableImage,
- };
-
- using EmbeddedViewID = uint64_t;
- static EmbeddedViewID nextEmbeddedViewID();
-
// Pass an invalid color to remove the contents layer.
virtual void setContentsToSolidColor(const Color&) { }
- virtual void setContentsToEmbeddedView(GraphicsLayer::ContentsLayerEmbeddedViewType, EmbeddedViewID) { }
virtual void setContentsToPlatformLayer(PlatformLayer*, ContentsLayerPurpose) { }
virtual bool usesContentsLayer() const { return false; }
Modified: trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -373,11 +373,6 @@
#endif
}
-Ref<PlatformCALayer> GraphicsLayerCA::createPlatformCALayerForEmbeddedView(PlatformCALayer::LayerType layerType, GraphicsLayer::EmbeddedViewID, PlatformCALayerClient* owner)
-{
- return GraphicsLayerCA::createPlatformCALayer(layerType, owner);
-}
-
Ref<PlatformCAAnimation> GraphicsLayerCA::createPlatformCAAnimation(PlatformCAAnimation::AnimationType type, const String& keyPath)
{
#if PLATFORM(COCOA)
@@ -1147,39 +1142,6 @@
noteLayerPropertyChanged(ContentsImageChanged);
}
-void GraphicsLayerCA::setContentsToEmbeddedView(GraphicsLayer::ContentsLayerEmbeddedViewType layerType, GraphicsLayer::EmbeddedViewID embeddedViewID)
-{
- bool contentsLayerChanged = false;
-
- if (layerType != GraphicsLayer::ContentsLayerEmbeddedViewType::None) {
- m_contentsLayerPurpose = ContentsLayerPurpose::EmbeddedView;
-
- PlatformCALayer::LayerType platformType;
- switch (layerType) {
- case GraphicsLayer::ContentsLayerEmbeddedViewType::EditableImage:
- platformType = PlatformCALayer::LayerTypeEditableImageLayer;
- break;
- default:
- ASSERT_NOT_REACHED();
- platformType = PlatformCALayer::LayerTypeLayer;
- break;
- }
-
- if (!m_contentsLayer || m_contentsLayer->layerType() != platformType || m_contentsLayer->embeddedViewID() != embeddedViewID) {
- m_contentsLayer = createPlatformCALayerForEmbeddedView(platformType, embeddedViewID, this);
- m_contentsLayer->setAnchorPoint(FloatPoint3D());
- contentsLayerChanged = true;
- }
- } else {
- m_contentsLayerPurpose = ContentsLayerPurpose::None;
- contentsLayerChanged = m_contentsLayer;
- m_contentsLayer = nullptr;
- }
-
- if (contentsLayerChanged)
- noteSublayersChanged();
-}
-
void GraphicsLayerCA::setContentsToPlatformLayer(PlatformLayer* platformLayer, ContentsLayerPurpose purpose)
{
if (m_contentsLayer && platformLayer == m_contentsLayer->platformLayer())
Modified: trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/GraphicsLayerCA.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -133,7 +133,6 @@
WEBCORE_EXPORT void removeAnimation(const String& animationName) override;
WEBCORE_EXPORT void setContentsToImage(Image*) override;
- WEBCORE_EXPORT void setContentsToEmbeddedView(GraphicsLayer::ContentsLayerEmbeddedViewType, GraphicsLayer::EmbeddedViewID) override;
#if PLATFORM(IOS_FAMILY)
WEBCORE_EXPORT PlatformLayer* contentsLayerForMedia() const override;
#endif
@@ -229,7 +228,6 @@
virtual Ref<PlatformCALayer> createPlatformCALayer(PlatformCALayer::LayerType, PlatformCALayerClient* owner);
virtual Ref<PlatformCALayer> createPlatformCALayer(PlatformLayer*, PlatformCALayerClient* owner);
- virtual Ref<PlatformCALayer> createPlatformCALayerForEmbeddedView(PlatformCALayer::LayerType, GraphicsLayer::EmbeddedViewID, PlatformCALayerClient* owner);
virtual Ref<PlatformCAAnimation> createPlatformCAAnimation(PlatformCAAnimation::AnimationType, const String& keyPath);
PlatformCALayer* primaryLayer() const { return m_structuralLayer.get() ? m_structuralLayer.get() : m_layer.get(); }
Modified: trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.cpp (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -213,9 +213,6 @@
case PlatformCALayer::LayerTypeRootLayer:
ts << "root-layer";
break;
- case PlatformCALayer::LayerTypeEditableImageLayer:
- ts << "editable-image";
- break;
case PlatformCALayer::LayerTypeBackdropLayer:
ts << "backdrop-layer";
break;
Modified: trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.h (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/PlatformCALayer.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -76,8 +76,7 @@
LayerTypeLightSystemBackdropLayer,
LayerTypeDarkSystemBackdropLayer,
LayerTypeScrollingLayer,
- LayerTypeEditableImageLayer,
- LayerTypeCustom,
+ LayerTypeCustom
};
enum FilterType { Linear, Nearest, Trilinear };
@@ -237,8 +236,6 @@
virtual GraphicsLayer::CustomAppearance customAppearance() const = 0;
virtual void updateCustomAppearance(GraphicsLayer::CustomAppearance) = 0;
- virtual GraphicsLayer::EmbeddedViewID embeddedViewID() const = 0;
-
virtual TiledBacking* tiledBacking() = 0;
virtual void drawTextAtPoint(CGContextRef, CGFloat x, CGFloat y, CGSize scale, CGFloat fontSize, const char* text, size_t length, CGFloat strokeWidthAsPercentageOfFontSize = 0, Color strokeColor = Color()) const;
Modified: trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.h (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -168,8 +168,6 @@
GraphicsLayer::CustomAppearance customAppearance() const override { return m_customAppearance; }
void updateCustomAppearance(GraphicsLayer::CustomAppearance) override;
- GraphicsLayer::EmbeddedViewID embeddedViewID() const override;
-
TiledBacking* tiledBacking() override;
Ref<PlatformCALayer> clone(PlatformCALayerClient* owner) const override;
Modified: trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm (238069 => 238070)
--- trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/platform/graphics/ca/cocoa/PlatformCALayerCocoa.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -223,7 +223,6 @@
case LayerTypeScrollingLayer:
// Scrolling layers only have special behavior with PlatformCALayerRemote.
// fallthrough
- case LayerTypeEditableImageLayer:
case LayerTypeWebLayer:
layerClass = [WebLayer class];
break;
@@ -1025,12 +1024,6 @@
#endif
}
-GraphicsLayer::EmbeddedViewID PlatformCALayerCocoa::embeddedViewID() const
-{
- ASSERT_NOT_REACHED();
- return 0;
-}
-
static NSString *layerContentsFormat(bool acceleratesDrawing, bool wantsDeepColor, bool supportsSubpixelAntialiasedFonts)
{
#if PLATFORM(IOS_FAMILY)
Modified: trunk/Source/WebCore/rendering/RenderImage.cpp (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderImage.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderImage.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -212,28 +212,6 @@
return ImageSizeChangeForAltText;
}
-bool RenderImage::isEditableImage() const
-{
- if (!settings().editableImagesEnabled())
- return false;
-
- if (!element())
- return false;
-
- return element()->hasAttributeWithoutSynchronization(x_apple_editable_imageAttr);
-}
-
-bool RenderImage::requiresLayer() const
-{
- if (RenderReplaced::requiresLayer())
- return true;
-
- if (isEditableImage())
- return true;
-
- return false;
-}
-
void RenderImage::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)
{
if (!hasInitializedStyle())
Modified: trunk/Source/WebCore/rendering/RenderImage.h (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderImage.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderImage.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -76,8 +76,6 @@
bool hasNonBitmapImage() const;
- bool isEditableImage() const;
-
protected:
void willBeDestroyed() override;
@@ -107,8 +105,6 @@
bool isImage() const override { return true; }
bool isRenderImage() const final { return true; }
-
- bool requiresLayer() const override;
void paintReplaced(PaintInfo&, const LayoutPoint&) override;
void paintIncompleteImageOutline(PaintInfo&, LayoutPoint, LayoutUnit) const;
Modified: trunk/Source/WebCore/rendering/RenderLayer.cpp (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderLayer.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderLayer.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -546,7 +546,6 @@
|| renderer().isVideo()
|| renderer().isEmbeddedObject()
|| renderer().isRenderIFrame()
- || (renderer().isRenderImage() && downcast<RenderImage>(renderer()).isEditableImage())
|| (renderer().style().specifiesColumns() && !isRenderViewLayer())
|| renderer().isInFlowRenderFragmentedFlow();
}
@@ -6068,7 +6067,6 @@
|| renderer().isCanvas()
|| renderer().isVideo()
|| renderer().isEmbeddedObject()
- || (renderer().isRenderImage() && downcast<RenderImage>(renderer()).isEditableImage())
|| renderer().isRenderIFrame()
|| renderer().isInFlowRenderFragmentedFlow();
}
Modified: trunk/Source/WebCore/rendering/RenderLayerBacking.cpp (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -766,14 +766,6 @@
if (is<RenderWidget>(renderer()))
layerConfigChanged = RenderLayerCompositor::parentFrameContentLayers(&downcast<RenderWidget>(renderer()));
- if (is<RenderImage>(renderer()) && downcast<RenderImage>(renderer()).isEditableImage()) {
- auto element = renderer().element();
- if (is<HTMLImageElement>(element)) {
- m_graphicsLayer->setContentsToEmbeddedView(GraphicsLayer::ContentsLayerEmbeddedViewType::EditableImage, downcast<HTMLImageElement>(element)->editableImageViewID());
- layerConfigChanged = true;
- }
- }
-
return layerConfigChanged;
}
Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -2016,7 +2016,6 @@
|| requiresCompositingForVideo(renderer)
|| requiresCompositingForFrame(renderer)
|| requiresCompositingForPlugin(renderer)
- || requiresCompositingForEditableImage(renderer)
|| requiresCompositingForOverflowScrolling(*renderer.layer());
}
@@ -2076,7 +2075,6 @@
|| requiresCompositingForVideo(renderer)
|| requiresCompositingForFrame(renderer)
|| requiresCompositingForPlugin(renderer)
- || requiresCompositingForEditableImage(renderer)
|| requiresCompositingForOverflowScrolling(layer)
|| renderer.isTransparent()
|| renderer.hasMask()
@@ -2120,8 +2118,6 @@
reasons.add(CompositingReason::Plugin);
else if (requiresCompositingForFrame(renderer))
reasons.add(CompositingReason::IFrame);
- else if (requiresCompositingForEditableImage(renderer))
- reasons.add(CompositingReason::EmbeddedView);
if ((canRender3DTransforms() && renderer.style().backfaceVisibility() == BackfaceVisibility::Hidden))
reasons.add(CompositingReason::BackfaceVisibilityHidden);
@@ -2451,19 +2447,7 @@
IntRect contentBox = snappedIntRect(pluginRenderer.contentBoxRect());
return contentBox.height() * contentBox.width() > 1;
}
-
-bool RenderLayerCompositor::requiresCompositingForEditableImage(RenderLayerModelObject& renderer) const
-{
- if (!renderer.isRenderImage())
- return false;
- auto& image = downcast<RenderImage>(renderer);
- if (!image.isEditableImage())
- return false;
-
- return true;
-}
-
bool RenderLayerCompositor::requiresCompositingForFrame(RenderLayerModelObject& renderer) const
{
if (!is<RenderWidget>(renderer))
Modified: trunk/Source/WebCore/rendering/RenderLayerCompositor.h (238069 => 238070)
--- trunk/Source/WebCore/rendering/RenderLayerCompositor.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebCore/rendering/RenderLayerCompositor.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -81,7 +81,6 @@
WillChange = 1 << 23,
Root = 1 << 24,
IsolatesCompositedBlendingDescendants = 1 << 25,
- EmbeddedView = 1 << 26,
};
// RenderLayerCompositor manages the hierarchy of
@@ -427,7 +426,6 @@
bool requiresCompositingForFrame(RenderLayerModelObject&) const;
bool requiresCompositingForFilters(RenderLayerModelObject&) const;
bool requiresCompositingForWillChange(RenderLayerModelObject&) const;
- bool requiresCompositingForEditableImage(RenderLayerModelObject&) const;
bool requiresCompositingForScrollableFrame() const;
bool requiresCompositingForPosition(RenderLayerModelObject&, const RenderLayer&, RenderLayer::ViewportConstrainedNotCompositedReason* = nullptr) const;
bool requiresCompositingForOverflowScrolling(const RenderLayer&) const;
Modified: trunk/Source/WebKit/ChangeLog (238069 => 238070)
--- trunk/Source/WebKit/ChangeLog 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/ChangeLog 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,3 +1,15 @@
+2018-11-10 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r238065.
+
+ Breaks internal builds.
+
+ Reverted changeset:
+
+ "Make it possible to edit images inline"
+ https://bugs.webkit.org/show_bug.cgi?id=191352
+ https://trac.webkit.org/changeset/238065
+
2018-11-10 Tim Horton <timothy_hor...@apple.com>
Make it possible to edit images inline
Deleted: trunk/Source/WebKit/Platform/spi/ios/PencilKitSPI.h (238069 => 238070)
--- trunk/Source/WebKit/Platform/spi/ios/PencilKitSPI.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/Platform/spi/ios/PencilKitSPI.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,35 +0,0 @@
-/*
- * Copyright (C) 2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#if USE(APPLE_INTERNAL_SDK)
-
-#import <PencilKit/PencilKit.h>
-
-#else
-
-@interface PKCanvasView : UIView
-@end
-
-#endif
Modified: trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm (238069 => 238070)
--- trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -367,7 +367,6 @@
case WebCore::PlatformCALayer::LayerTypeContentsProvidedLayer:
case WebCore::PlatformCALayer::LayerTypeShapeLayer:
case WebCore::PlatformCALayer::LayerTypeScrollingLayer:
- case WebCore::PlatformCALayer::LayerTypeEditableImageLayer:
case WebCore::PlatformCALayer::LayerTypeCustom:
ASSERT_NOT_REACHED();
break;
Modified: trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.h (238069 => 238070)
--- trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -103,8 +103,6 @@
WebCore::GraphicsLayer::PlatformLayerID layerID;
WebCore::PlatformCALayer::LayerType type;
- WebCore::GraphicsLayer::EmbeddedViewID embeddedViewID;
-
uint32_t hostingContextID;
float hostingDeviceScaleFactor;
};
Modified: trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.mm (238069 => 238070)
--- trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerTreeTransaction.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -41,7 +41,6 @@
RemoteLayerTreeTransaction::LayerCreationProperties::LayerCreationProperties()
: layerID(0)
, type(WebCore::PlatformCALayer::LayerTypeLayer)
- , embeddedViewID(0)
, hostingContextID(0)
, hostingDeviceScaleFactor(1)
{
@@ -51,7 +50,6 @@
{
encoder << layerID;
encoder.encodeEnum(type);
- encoder << embeddedViewID;
encoder << hostingContextID;
encoder << hostingDeviceScaleFactor;
}
@@ -65,9 +63,6 @@
if (!decoder.decodeEnum(result.type))
return std::nullopt;
- if (!decoder.decode(result.embeddedViewID))
- return std::nullopt;
-
if (!decoder.decode(result.hostingContextID))
return std::nullopt;
Modified: trunk/Source/WebKit/Shared/WebPreferences.yaml (238069 => 238070)
--- trunk/Source/WebKit/Shared/WebPreferences.yaml 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/Shared/WebPreferences.yaml 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1185,10 +1185,6 @@
defaultValue: false
condition: ENABLE(VIDEO)
-EditableImagesEnabled:
- type: bool
- defaultValue: false
-
# For experimental features:
# The type should be boolean.
# You must provide a humanReadableName and humanReadableDescription for all experimental features. They
Modified: trunk/Source/WebKit/SourcesCocoa.txt (238069 => 238070)
--- trunk/Source/WebKit/SourcesCocoa.txt 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/SourcesCocoa.txt 2018-11-10 21:01:36 UTC (rev 238070)
@@ -344,6 +344,9 @@
UIProcess/Gamepad/ios/UIGamepadProviderIOS.mm
UIProcess/Gamepad/mac/UIGamepadProviderMac.mm
+UIProcess/ios/DragDropInteractionState.mm
+UIProcess/ios/WebDataListSuggestionsDropdownIOS.mm
+
UIProcess/ios/forms/WKAirPlayRoutePicker.mm
UIProcess/ios/forms/WKDatePickerViewController.mm
UIProcess/ios/forms/WKFileUploadPanel.mm
@@ -367,7 +370,6 @@
UIProcess/ios/fullscreen/WKFullScreenViewController.mm
UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm
-UIProcess/ios/DragDropInteractionState.mm
UIProcess/ios/InputViewUpdateDeferrer.mm
UIProcess/ios/LayerRepresentation.mm
UIProcess/ios/PageClientImplIOS.mm
@@ -376,7 +378,6 @@
UIProcess/ios/SmartMagnificationController.mm
UIProcess/ios/TextCheckerIOS.mm
UIProcess/ios/ViewGestureControllerIOS.mm
-UIProcess/ios/WebDataListSuggestionsDropdownIOS.mm
UIProcess/ios/WebPageProxyIOS.mm
UIProcess/ios/WebProcessProxyIOS.mm
UIProcess/ios/WKActionSheet.mm
@@ -384,7 +385,6 @@
UIProcess/ios/WKApplicationStateTrackingView.mm
UIProcess/ios/WKContentView.mm @no-unify
UIProcess/ios/WKContentViewInteraction.mm @no-unify
-UIProcess/ios/WKDrawingView.mm
UIProcess/ios/WKGeolocationProviderIOS.mm
UIProcess/ios/WKGeolocationProviderIOSObjCSecurityOrigin.mm
UIProcess/ios/WKInspectorNodeSearchGestureRecognizer.mm
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -645,8 +645,6 @@
pageConfiguration->preferenceValues().set(WebKit::WebPreferencesKey::colorFilterEnabledKey(), WebKit::WebPreferencesStore::Value(!![_configuration _colorFilterEnabled]));
- pageConfiguration->preferenceValues().set(WebKit::WebPreferencesKey::editableImagesEnabledKey(), WebKit::WebPreferencesStore::Value(!![_configuration _editableImagesEnabled]));
-
#if ENABLE(LEGACY_ENCRYPTED_MEDIA)
pageConfiguration->preferenceValues().set(WebKit::WebPreferencesKey::legacyEncryptedMediaAPIEnabledKey(), WebKit::WebPreferencesStore::Value(!![_configuration _legacyEncryptedMediaAPIEnabled]));
#endif
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -159,7 +159,6 @@
BOOL _incompleteImageBorderEnabled;
BOOL _shouldDeferAsynchronousScriptsUntilAfterDocumentLoad;
BOOL _drawsBackground;
- BOOL _editableImagesEnabled;
RetainPtr<NSString> _mediaContentTypesRequiringHardwareSupport;
RetainPtr<NSArray<NSString *>> _additionalSupportedImageTypes;
@@ -247,8 +246,6 @@
_shouldDeferAsynchronousScriptsUntilAfterDocumentLoad = NO;
_drawsBackground = YES;
- _editableImagesEnabled = NO;
-
return self;
}
@@ -408,8 +405,6 @@
configuration->_shouldDeferAsynchronousScriptsUntilAfterDocumentLoad = self->_shouldDeferAsynchronousScriptsUntilAfterDocumentLoad;
configuration->_drawsBackground = self->_drawsBackground;
- configuration->_editableImagesEnabled = self->_editableImagesEnabled;
-
return configuration;
}
@@ -1027,16 +1022,6 @@
return _allowMediaContentTypesRequiringHardwareSupportAsFallback;
}
-- (void)_setEditableImagesEnabled:(BOOL)enabled
-{
- _editableImagesEnabled = enabled;
-}
-
-- (BOOL)_editableImagesEnabled
-{
- return _editableImagesEnabled;
-}
-
@end
@implementation WKWebViewConfiguration (WKDeprecated)
Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfigurationPrivate.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -105,8 +105,6 @@
// The input of this SPI is an array of image UTI (Uniform Type Identifier).
@property (nonatomic, copy, setter=_setAdditionalSupportedImageTypes:) NSArray<NSString *> *_additionalSupportedImageTypes WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
-@property (nonatomic, setter=_setEditableImagesEnabled:) BOOL _editableImagesEnabled WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));
-
@end
#endif
Modified: trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.h (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -33,7 +33,6 @@
OBJC_CLASS CAAnimation;
OBJC_CLASS WKAnimationDelegate;
-OBJC_CLASS WKEmbeddedView;
namespace WebKit {
@@ -76,7 +75,6 @@
private:
LayerOrView *createLayer(const RemoteLayerTreeTransaction::LayerCreationProperties&, const RemoteLayerTreeTransaction::LayerProperties*);
- RetainPtr<WKEmbeddedView> createEmbeddedView(const RemoteLayerTreeTransaction::LayerCreationProperties&, const RemoteLayerTreeTransaction::LayerProperties*);
static void setLayerID(CALayer *, WebCore::GraphicsLayer::PlatformLayerID);
void layerWillBeRemoved(WebCore::GraphicsLayer::PlatformLayerID);
@@ -85,8 +83,6 @@
LayerOrView *m_rootLayer { nullptr };
HashMap<WebCore::GraphicsLayer::PlatformLayerID, RetainPtr<LayerOrView>> m_layers;
HashMap<WebCore::GraphicsLayer::PlatformLayerID, RetainPtr<WKAnimationDelegate>> m_animationDelegates;
- HashMap<WebCore::GraphicsLayer::EmbeddedViewID, RetainPtr<WKEmbeddedView>> m_embeddedViews;
- HashMap<WebCore::GraphicsLayer::PlatformLayerID, WebCore::GraphicsLayer::EmbeddedViewID> m_layerToEmbeddedViewMap;
bool m_isDebugLayerTreeHost { false };
};
Modified: trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/RemoteLayerTree/RemoteLayerTreeHost.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -151,18 +151,11 @@
void RemoteLayerTreeHost::layerWillBeRemoved(WebCore::GraphicsLayer::PlatformLayerID layerID)
{
- auto animationDelegateIter = m_animationDelegates.find(layerID);
- if (animationDelegateIter != m_animationDelegates.end()) {
- [animationDelegateIter->value invalidate];
- m_animationDelegates.remove(animationDelegateIter);
+ auto iter = m_animationDelegates.find(layerID);
+ if (iter != m_animationDelegates.end()) {
+ [iter->value invalidate];
+ m_animationDelegates.remove(iter);
}
-
- auto embeddedViewIter = m_layerToEmbeddedViewMap.find(layerID);
- if (embeddedViewIter != m_layerToEmbeddedViewMap.end()) {
- m_embeddedViews.remove(embeddedViewIter->value);
- m_layerToEmbeddedViewMap.remove(embeddedViewIter);
- }
-
m_layers.remove(layerID);
}
@@ -225,8 +218,6 @@
}
m_layers.clear();
- m_embeddedViews.clear();
- m_layerToEmbeddedViewMap.clear();
m_rootLayer = nullptr;
}
@@ -263,7 +254,6 @@
case PlatformCALayer::LayerTypePageTiledBackingLayer:
case PlatformCALayer::LayerTypeTiledBackingTileLayer:
case PlatformCALayer::LayerTypeScrollingLayer:
- case PlatformCALayer::LayerTypeEditableImageLayer:
layer = adoptNS([[CALayer alloc] init]);
break;
case PlatformCALayer::LayerTypeTransformLayer:
Modified: trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeHostIOS.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeHostIOS.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeHostIOS.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -29,18 +29,155 @@
#if PLATFORM(IOS_FAMILY)
#import "RemoteLayerTreeDrawingAreaProxy.h"
-#import "RemoteLayerTreeViews.h"
#import "UIKitSPI.h"
-#import "WKDrawingView.h"
#import "WebPageProxy.h"
#import <UIKit/UIScrollView.h>
#import <pal/spi/cocoa/QuartzCoreSPI.h>
-#import <wtf/SoftLinking.h>
using namespace WebCore;
-namespace WebKit {
+@interface UIView (WKHitTesting)
+- (UIView *)_findDescendantViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event;
+@end
+@implementation UIView (WKHitTesting)
+
+// UIView hit testing assumes that views should only hit test subviews that are entirely contained
+// in the view. This is not true of web content.
+// We only want to find UIScrollViews here. Other views are ignored.
+- (UIView *)_recursiveFindDescendantScrollViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event
+{
+ if (self.clipsToBounds && ![self pointInside:point withEvent:event])
+ return nil;
+
+ __block UIView *foundView = nil;
+ [[self subviews] enumerateObjectsUsingBlock:^(UIView *view, NSUInteger idx, BOOL *stop) {
+ CGPoint subviewPoint = [view convertPoint:point fromView:self];
+
+ if ([view pointInside:subviewPoint withEvent:event] && [view isKindOfClass:[UIScrollView class]] && view.isUserInteractionEnabled)
+ foundView = view;
+
+ if (![view subviews])
+ return;
+
+ if (UIView *hitView = [view _recursiveFindDescendantScrollViewAtPoint:subviewPoint withEvent:event])
+ foundView = hitView;
+ }];
+
+ return foundView;
+}
+
+- (UIView *)_findDescendantViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event
+{
+ return [self _recursiveFindDescendantScrollViewAtPoint:point withEvent:event];
+}
+
+@end
+
+@interface WKCompositingView : UIView
+@end
+
+@implementation WKCompositingView
+
+- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
+{
+ return [self _findDescendantViewAtPoint:point withEvent:event];
+}
+
+- (NSString *)description
+{
+ NSString *viewDescription = [super description];
+ NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
+ return [viewDescription stringByAppendingString:webKitDetails];
+}
+
+@end
+
+@interface WKTransformView : WKCompositingView
+@end
+
+@implementation WKTransformView
+
++ (Class)layerClass
+{
+ return [CATransformLayer self];
+}
+
+@end
+
+@interface WKSimpleBackdropView : WKCompositingView
+@end
+
+@implementation WKSimpleBackdropView
+
++ (Class)layerClass
+{
+ return [CABackdropLayer self];
+}
+
+@end
+
+@interface WKShapeView : WKCompositingView
+@end
+
+@implementation WKShapeView
+
++ (Class)layerClass
+{
+ return [CAShapeLayer self];
+}
+
+@end
+
+@interface WKRemoteView : WKCompositingView
+@end
+
+@implementation WKRemoteView
+
+- (instancetype)initWithFrame:(CGRect)frame contextID:(uint32_t)contextID
+{
+ if ((self = [super initWithFrame:frame])) {
+ CALayerHost *layer = (CALayerHost *)self.layer;
+ layer.contextId = contextID;
+#if PLATFORM(IOSMAC)
+ // When running iOS apps on macOS, kCAContextIgnoresHitTest isn't respected; instead, we avoid
+ // hit-testing to the remote context by disabling hit-testing on its host layer. See
+ // <rdar://problem/40591107> for more details.
+ layer.allowsHitTesting = NO;
+#endif
+ }
+
+ return self;
+}
+
++ (Class)layerClass
+{
+ return NSClassFromString(@"CALayerHost");
+}
+
+@end
+
+#if USE(UIREMOTEVIEW_CONTEXT_HOSTING)
+@interface WKUIRemoteView : _UIRemoteView
+@end
+
+@implementation WKUIRemoteView
+
+- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
+{
+ return [self _findDescendantViewAtPoint:point withEvent:event];
+}
+
+- (NSString *)description
+{
+ NSString *viewDescription = [super description];
+ NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
+ return [viewDescription stringByAppendingString:webKitDetails];
+}
+
+@end
+#endif
+
static RetainPtr<UIView> createRemoteView(pid_t pid, uint32_t contextID)
{
#if USE(UIREMOTEVIEW_CONTEXT_HOSTING)
@@ -58,6 +195,27 @@
return adoptNS([[WKRemoteView alloc] initWithFrame:CGRectZero contextID:contextID]);
}
+@interface WKBackdropView : _UIBackdropView
+@end
+
+@implementation WKBackdropView
+
+- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
+{
+ return [self _findDescendantViewAtPoint:point withEvent:event];
+}
+
+- (NSString *)description
+{
+ NSString *viewDescription = [super description];
+ NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
+ return [viewDescription stringByAppendingString:webKitDetails];
+}
+
+@end
+
+namespace WebKit {
+
LayerOrView *RemoteLayerTreeHost::createLayer(const RemoteLayerTreeTransaction::LayerCreationProperties& properties, const RemoteLayerTreeTransaction::LayerProperties* layerProperties)
{
RetainPtr<LayerOrView>& view = m_layers.add(properties.layerID, nullptr).iterator->value;
@@ -103,14 +261,15 @@
view = adoptNS([[WKShapeView alloc] init]);
break;
case PlatformCALayer::LayerTypeScrollingLayer:
- if (!m_isDebugLayerTreeHost)
- view = adoptNS([[WKChildScrollView alloc] init]);
- else // The debug indicator parents views under layers, which can cause crashes with UIScrollView.
+ if (!m_isDebugLayerTreeHost) {
+ auto scrollView = adoptNS([[UIScrollView alloc] init]);
+#if __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000
+ [scrollView setContentInsetAdjustmentBehavior:UIScrollViewContentInsetAdjustmentNever];
+#endif
+ view = scrollView;
+ } else // The debug indicator parents views under layers, which can cause crashes with UIScrollView.
view = adoptNS([[UIView alloc] init]);
break;
- case PlatformCALayer::LayerTypeEditableImageLayer:
- view = createEmbeddedView(properties, layerProperties);
- break;
default:
ASSERT_NOT_REACHED();
}
@@ -120,32 +279,6 @@
return view.get();
}
-RetainPtr<WKEmbeddedView> RemoteLayerTreeHost::createEmbeddedView(const RemoteLayerTreeTransaction::LayerCreationProperties& properties, const RemoteLayerTreeTransaction::LayerProperties* layerProperties)
-{
- Class embeddedViewClass = nil;
- switch (properties.type) {
- case PlatformCALayer::LayerTypeEditableImageLayer:
- embeddedViewClass = [WKDrawingView class];
- break;
- default:
- break;
- }
-
- if (!embeddedViewClass || m_isDebugLayerTreeHost)
- return adoptNS([[UIView alloc] init]);
-
- auto result = m_embeddedViews.ensure(properties.embeddedViewID, [&] {
- return adoptNS([[embeddedViewClass alloc] init]);
- });
- auto view = result.iterator->value;
- if (!result.isNewEntry)
- m_layerToEmbeddedViewMap.remove([view layerID]);
- [view setLayerID:properties.layerID];
- m_embeddedViews.set(properties.embeddedViewID, view);
- m_layerToEmbeddedViewMap.set(properties.layerID, properties.embeddedViewID);
- return view;
-}
-
} // namespace WebKit
#endif // PLATFORM(IOS_FAMILY)
Deleted: trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.h (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,70 +0,0 @@
-/*
- * Copyright (C) 2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#if PLATFORM(IOS_FAMILY)
-
-#import "UIKitSPI.h"
-#import <WebCore/GraphicsLayer.h>
-
-@protocol WKNativelyInteractible <NSObject>
-@end
-
-@interface WKCompositingView : UIView
-@end
-
-@interface WKTransformView : WKCompositingView
-@end
-
-@interface WKSimpleBackdropView : WKCompositingView
-@end
-
-@interface WKShapeView : WKCompositingView
-@end
-
-@interface WKRemoteView : WKCompositingView
-
-- (instancetype)initWithFrame:(CGRect)frame contextID:(uint32_t)contextID;
-
-@end
-
-@interface WKUIRemoteView : _UIRemoteView
-@end
-
-@interface WKBackdropView : _UIBackdropView
-@end
-
-@interface WKChildScrollView : UIScrollView
-@end
-
-@interface WKEmbeddedView : UIView
-
-@property (nonatomic, assign) WebCore::GraphicsLayer::EmbeddedViewID embeddedViewID;
-@property (nonatomic, assign) WebCore::GraphicsLayer::PlatformLayerID layerID;
-
-@end
-
-#endif // PLATFORM(IOS_FAMILY)
Deleted: trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/RemoteLayerTree/ios/RemoteLayerTreeViews.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,207 +0,0 @@
-/*
- * Copyright (C) 2014-2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import "config.h"
-#import "RemoteLayerTreeViews.h"
-
-#if PLATFORM(IOS_FAMILY)
-
-#import "RemoteLayerTreeHost.h"
-#import "UIKitSPI.h"
-#import "WKDrawingView.h"
-#import <pal/spi/cocoa/QuartzCoreSPI.h>
-#import <wtf/SoftLinking.h>
-
-@interface UIView (WKHitTesting)
-- (UIView *)_web_findDescendantViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event;
-@end
-
-@implementation UIView (WKHitTesting)
-
-// UIView hit testing assumes that views should only hit test subviews that are entirely contained
-// in the view. This is not true of web content.
-// We only want to find views that allow native interaction here. Other views are ignored.
-- (UIView *)_web_recursiveFindDescendantInteractibleViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event
-{
- if (self.clipsToBounds && ![self pointInside:point withEvent:event])
- return nil;
-
- __block UIView *foundView = nil;
- [[self subviews] enumerateObjectsUsingBlock:^(UIView *view, NSUInteger idx, BOOL *stop) {
- CGPoint subviewPoint = [view convertPoint:point fromView:self];
-
- if (view.isUserInteractionEnabled && [view pointInside:subviewPoint withEvent:event]) {
- if ([view conformsToProtocol:@protocol(WKNativelyInteractible)]) {
- foundView = view;
-
- if (![view subviews])
- return;
-
- if (UIView *hitView = [view hitTest:subviewPoint withEvent:event])
- foundView = hitView;
- } else if ([view isKindOfClass:[WKChildScrollView class]])
- foundView = view;
- }
-
- if (![view subviews])
- return;
-
- if (UIView *hitView = [view _web_recursiveFindDescendantInteractibleViewAtPoint:subviewPoint withEvent:event])
- foundView = hitView;
- }];
-
- return foundView;
-}
-
-- (UIView *)_web_findDescendantViewAtPoint:(CGPoint)point withEvent:(UIEvent *)event
-{
- return [self _web_recursiveFindDescendantInteractibleViewAtPoint:point withEvent:event];
-}
-
-@end
-
-@implementation WKCompositingView
-
-- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
-{
- return [self _web_findDescendantViewAtPoint:point withEvent:event];
-}
-
-- (NSString *)description
-{
- NSString *viewDescription = [super description];
- NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
- return [viewDescription stringByAppendingString:webKitDetails];
-}
-
-@end
-
-@implementation WKTransformView
-
-+ (Class)layerClass
-{
- return [CATransformLayer self];
-}
-
-@end
-
-@implementation WKSimpleBackdropView
-
-+ (Class)layerClass
-{
- return [CABackdropLayer self];
-}
-
-@end
-
-@implementation WKShapeView
-
-+ (Class)layerClass
-{
- return [CAShapeLayer self];
-}
-
-@end
-
-@implementation WKRemoteView
-
-- (instancetype)initWithFrame:(CGRect)frame contextID:(uint32_t)contextID
-{
- if ((self = [super initWithFrame:frame])) {
- CALayerHost *layer = (CALayerHost *)self.layer;
- layer.contextId = contextID;
-#if PLATFORM(IOSMAC)
- // When running iOS apps on macOS, kCAContextIgnoresHitTest isn't respected; instead, we avoid
- // hit-testing to the remote context by disabling hit-testing on its host layer. See
- // <rdar://problem/40591107> for more details.
- layer.allowsHitTesting = NO;
-#endif
- }
-
- return self;
-}
-
-+ (Class)layerClass
-{
- return NSClassFromString(@"CALayerHost");
-}
-
-@end
-
-#if USE(UIREMOTEVIEW_CONTEXT_HOSTING)
-@implementation WKUIRemoteView
-
-- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
-{
- return [self _web_findDescendantViewAtPoint:point withEvent:event];
-}
-
-- (NSString *)description
-{
- NSString *viewDescription = [super description];
- NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
- return [viewDescription stringByAppendingString:webKitDetails];
-}
-
-@end
-#endif
-
-@implementation WKBackdropView
-
-- (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
-{
- return [self _web_findDescendantViewAtPoint:point withEvent:event];
-}
-
-- (NSString *)description
-{
- NSString *viewDescription = [super description];
- NSString *webKitDetails = [NSString stringWithFormat:@" layerID = %llu \"%@\"", WebKit::RemoteLayerTreeHost::layerID(self.layer), self.layer.name ? self.layer.name : @""];
- return [viewDescription stringByAppendingString:webKitDetails];
-}
-
-@end
-
-@implementation WKChildScrollView
-
-- (instancetype)initWithFrame:(CGRect)frame
-{
- self = [super initWithFrame:frame];
- if (!self)
- return nil;
-
-#if __IPHONE_OS_VERSION_MIN_REQUIRED >= 110000
- self.contentInsetAdjustmentBehavior = UIScrollViewContentInsetAdjustmentNever;
-#endif
-
- return self;
-}
-
-@end
-
-@implementation WKEmbeddedView
-@end
-
-#endif // PLATFORM(IOS_FAMILY)
Deleted: trunk/Source/WebKit/UIProcess/ios/WKDrawingView.h (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/ios/WKDrawingView.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/ios/WKDrawingView.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,36 +0,0 @@
-/*
- * Copyright (C) 2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#if PLATFORM(IOS_FAMILY)
-
-#import "RemoteLayerTreeHost.h"
-#import "RemoteLayerTreeViews.h"
-
-@interface WKDrawingView : WKEmbeddedView <WKNativelyInteractible>
-
-@end
-
-
-#endif // PLATFORM(IOS_FAMILY)
Deleted: trunk/Source/WebKit/UIProcess/ios/WKDrawingView.mm (238069 => 238070)
--- trunk/Source/WebKit/UIProcess/ios/WKDrawingView.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/UIProcess/ios/WKDrawingView.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,64 +0,0 @@
-/*
- * Copyright (C) 2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import "config.h"
-#import "WKDrawingView.h"
-
-#if PLATFORM(IOS_FAMILY)
-
-#import "PencilKitSPI.h"
-#import <wtf/RetainPtr.h>
-
-SOFT_LINK_PRIVATE_FRAMEWORK(PencilKit);
-SOFT_LINK_CLASS(PencilKit, PKCanvasView);
-
-@implementation WKDrawingView {
- RetainPtr<PKCanvasView> _pencilView;
-}
-
-- (id)init
-{
- self = [super init];
- if (!self)
- return nil;
-
- _pencilView = adoptNS([allocPKCanvasViewInstance() initWithFrame:CGRectZero]);
-
- [_pencilView setUserInteractionEnabled:YES];
- [_pencilView setOpaque:NO];
-
- [self addSubview:_pencilView.get()];
-
- return self;
-}
-
-- (void)layoutSubviews
-{
- [_pencilView setFrame:self.bounds];
-}
-
-@end
-
-#endif // PLATFORM(IOS_FAMILY)
Modified: trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj (238069 => 238070)
--- trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj 2018-11-10 21:01:36 UTC (rev 238070)
@@ -703,10 +703,6 @@
2DABA7761A82B42100EF0F1A /* APIHistoryClient.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DABA7751A82B42100EF0F1A /* APIHistoryClient.h */; };
2DACE64E18ADBFF000E4CA76 /* _WKThumbnailViewInternal.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DACE64D18ADBFF000E4CA76 /* _WKThumbnailViewInternal.h */; };
2DAF06D618BD1A470081CEB1 /* SmartMagnificationController.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DAF06D418BD1A470081CEB1 /* SmartMagnificationController.h */; };
- 2DC18FAD218910490025A88D /* WKDrawingView.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DC18FAB218910480025A88D /* WKDrawingView.h */; };
- 2DC18FB0218912640025A88D /* PencilKitSPI.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DC18FAF218912640025A88D /* PencilKitSPI.h */; };
- 2DC18FB3218A6E9E0025A88D /* RemoteLayerTreeViews.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DC18FB1218A6E9E0025A88D /* RemoteLayerTreeViews.h */; };
- 2DC18FB4218A6E9E0025A88D /* RemoteLayerTreeViews.mm in Sources */ = {isa = PBXBuildFile; fileRef = 2DC18FB2218A6E9E0025A88D /* RemoteLayerTreeViews.mm */; };
2DC6D9C318C44A610043BAD4 /* WKWebViewContentProviderRegistry.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DC6D9C118C44A610043BAD4 /* WKWebViewContentProviderRegistry.h */; };
2DD12A081A8177F3002C74E6 /* WKPageRenderingProgressEvents.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DD12A071A8177F3002C74E6 /* WKPageRenderingProgressEvents.h */; settings = {ATTRIBUTES = (Private, ); }; };
2DD45ADE1E5F8972006C355F /* InputViewUpdateDeferrer.h in Headers */ = {isa = PBXBuildFile; fileRef = 2DD45ADC1E5F8972006C355F /* InputViewUpdateDeferrer.h */; };
@@ -2744,11 +2740,6 @@
2DAF06D518BD1A470081CEB1 /* SmartMagnificationController.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; name = SmartMagnificationController.mm; path = ios/SmartMagnificationController.mm; sourceTree = "<group>"; };
2DAF06D818BD23BA0081CEB1 /* SmartMagnificationController.messages.in */ = {isa = PBXFileReference; lastKnownFileType = text; name = SmartMagnificationController.messages.in; path = ios/SmartMagnificationController.messages.in; sourceTree = "<group>"; };
2DAF4FFA1B636181006013D6 /* ViewGestureController.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = ViewGestureController.cpp; sourceTree = "<group>"; };
- 2DC18FAB218910480025A88D /* WKDrawingView.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = WKDrawingView.h; path = ios/WKDrawingView.h; sourceTree = "<group>"; };
- 2DC18FAC218910480025A88D /* WKDrawingView.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; name = WKDrawingView.mm; path = ios/WKDrawingView.mm; sourceTree = "<group>"; };
- 2DC18FAF218912640025A88D /* PencilKitSPI.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = PencilKitSPI.h; sourceTree = "<group>"; };
- 2DC18FB1218A6E9E0025A88D /* RemoteLayerTreeViews.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RemoteLayerTreeViews.h; sourceTree = "<group>"; };
- 2DC18FB2218A6E9E0025A88D /* RemoteLayerTreeViews.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = RemoteLayerTreeViews.mm; sourceTree = "<group>"; };
2DC4CF7A1D2DE24B00ECCC94 /* WebPageCocoa.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebPageCocoa.mm; sourceTree = "<group>"; };
2DC6D9C118C44A610043BAD4 /* WKWebViewContentProviderRegistry.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WKWebViewContentProviderRegistry.h; sourceTree = "<group>"; };
2DC6D9C218C44A610043BAD4 /* WKWebViewContentProviderRegistry.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WKWebViewContentProviderRegistry.mm; sourceTree = "<group>"; };
@@ -5473,8 +5464,6 @@
isa = PBXGroup;
children = (
0F0C365718C051BA00F607D7 /* RemoteLayerTreeHostIOS.mm */,
- 2DC18FB1218A6E9E0025A88D /* RemoteLayerTreeViews.h */,
- 2DC18FB2218A6E9E0025A88D /* RemoteLayerTreeViews.mm */,
0F0C365B18C05CA100F607D7 /* RemoteScrollingCoordinatorProxyIOS.mm */,
0F931C1A18C5711900DBA7C3 /* ScrollingTreeOverflowScrollingNodeIOS.h */,
0F931C1B18C5711900DBA7C3 /* ScrollingTreeOverflowScrollingNodeIOS.mm */,
@@ -5773,8 +5762,6 @@
0FCB4E3D18BBE044000FCFC9 /* WKContentView.mm */,
0FCB4E6A18BBF26A000FCFC9 /* WKContentViewInteraction.h */,
0FCB4E6B18BBF26A000FCFC9 /* WKContentViewInteraction.mm */,
- 2DC18FAB218910480025A88D /* WKDrawingView.h */,
- 2DC18FAC218910480025A88D /* WKDrawingView.mm */,
0FCB4E3F18BBE044000FCFC9 /* WKGeolocationProviderIOS.h */,
0FCB4E4018BBE044000FCFC9 /* WKGeolocationProviderIOS.mm */,
0FCB4E4118BBE044000FCFC9 /* WKGeolocationProviderIOSObjCSecurityOrigin.mm */,
@@ -8563,7 +8550,6 @@
CE1A0BCF1A48E6C60054EF74 /* ManagedConfigurationSPI.h */,
A13B3DA1207F39DE0090C58D /* MobileWiFiSPI.h */,
3178AF9720E2A7F80074DE94 /* PDFKitSPI.h */,
- 2DC18FAF218912640025A88D /* PencilKitSPI.h */,
CE1A0BD01A48E6C60054EF74 /* TCCSPI.h */,
CE1A0BD11A48E6C60054EF74 /* TextInputSPI.h */,
CEE4AE2A1A5DCF430002F49B /* UIKitSPI.h */,
@@ -9083,7 +9069,6 @@
C574A58112E66681002DFE98 /* PasteboardTypes.h in Headers */,
C1E123BA20A11573002646F4 /* PDFContextMenu.h in Headers */,
E19582D3153CBFD700B60875 /* PDFKitImports.h in Headers */,
- 2DC18FB0218912640025A88D /* PencilKitSPI.h in Headers */,
5C298DA01C3DF02100470AFE /* PendingDownload.h in Headers */,
832ED18C1E2FE157006BA64A /* PerActivityStateCPUUsageSampler.h in Headers */,
5CE85B201C88E64B0070BFCE /* PingLoad.h in Headers */,
@@ -9142,7 +9127,6 @@
2DDE0AFA18298CC900F97EAA /* RemoteLayerTreePropertyApplier.h in Headers */,
0FF264A01A1FF9CC001FE759 /* RemoteLayerTreeScrollingPerformanceData.h in Headers */,
1AF1AC6C1651759E00C17D7F /* RemoteLayerTreeTransaction.h in Headers */,
- 2DC18FB3218A6E9E0025A88D /* RemoteLayerTreeViews.h in Headers */,
E1B78471163F24690007B692 /* RemoteNetworkingContext.h in Headers */,
1A5704FC1BE1751100874AF1 /* RemoteObjectInvocation.h in Headers */,
1AC1338018590AE400F3EC05 /* RemoteObjectRegistry.h in Headers */,
@@ -9556,7 +9540,6 @@
BC017D2116263308007054F5 /* WKDOMTextIterator.h in Headers */,
1AB7D78D1288CD9A00CFD08C /* WKDownload.h in Headers */,
F409BA181E6E64BC009DA28E /* WKDragDestinationAction.h in Headers */,
- 2DC18FAD218910490025A88D /* WKDrawingView.h in Headers */,
1AF4592F19464B2000F9D4A2 /* WKError.h in Headers */,
BCFD548C132D82680055D816 /* WKErrorCF.h in Headers */,
37B5045219EEF31300CE2CF8 /* WKErrorPrivate.h in Headers */,
@@ -10699,7 +10682,6 @@
2D54C31B212F4DA60049C174 /* ProcessLauncher.cpp in Sources */,
2D72A1FA212BF46E00517A20 /* RemoteLayerTreeDrawingArea.mm in Sources */,
0FF24A2D1879E4BC003ABF0C /* RemoteLayerTreeDrawingAreaProxyMessageReceiver.cpp in Sources */,
- 2DC18FB4218A6E9E0025A88D /* RemoteLayerTreeViews.mm in Sources */,
1AC1338518590C4600F3EC05 /* RemoteObjectRegistryMessageReceiver.cpp in Sources */,
0F5947A7187B517600437857 /* RemoteScrollingCoordinatorMessageReceiver.cpp in Sources */,
A55BA8261BA25CFD007CD33D /* RemoteWebInspectorProxyMessageReceiver.cpp in Sources */,
Modified: trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp (238069 => 238070)
--- trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -57,11 +57,6 @@
return PlatformCALayerRemote::create(platformLayer, owner, m_context);
}
-Ref<PlatformCALayer> GraphicsLayerCARemote::createPlatformCALayerForEmbeddedView(PlatformCALayer::LayerType layerType, GraphicsLayer::EmbeddedViewID embeddedViewID, PlatformCALayerClient* owner)
-{
- return PlatformCALayerRemote::createForEmbeddedView(layerType, embeddedViewID, owner, m_context);
-}
-
Ref<PlatformCAAnimation> GraphicsLayerCARemote::createPlatformCAAnimation(PlatformCAAnimation::AnimationType type, const String& keyPath)
{
return PlatformCAAnimationRemote::create(type, keyPath);
Modified: trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.h (238069 => 238070)
--- trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -49,7 +49,6 @@
Ref<WebCore::PlatformCALayer> createPlatformCALayer(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient* owner) override;
Ref<WebCore::PlatformCALayer> createPlatformCALayer(PlatformLayer*, WebCore::PlatformCALayerClient* owner) override;
- Ref<WebCore::PlatformCALayer> createPlatformCALayerForEmbeddedView(WebCore::PlatformCALayer::LayerType, GraphicsLayer::EmbeddedViewID, WebCore::PlatformCALayerClient* owner) override;
Ref<WebCore::PlatformCAAnimation> createPlatformCAAnimation(WebCore::PlatformCAAnimation::AnimationType, const String& keyPath) override;
// PlatformCALayerRemote can't currently proxy directly composited image contents, so opt out of this optimization.
Modified: trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp (238069 => 238070)
--- trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -61,13 +61,6 @@
return PlatformCALayerRemoteCustom::create(platformLayer, owner, context);
}
-Ref<PlatformCALayerRemote> PlatformCALayerRemote::createForEmbeddedView(LayerType layerType, GraphicsLayer::EmbeddedViewID embeddedViewID, PlatformCALayerClient* owner, RemoteLayerTreeContext& context)
-{
- RefPtr<PlatformCALayerRemote> layer = adoptRef(new PlatformCALayerRemote(layerType, embeddedViewID, owner, context));
- context.layerWasCreated(*layer, layerType);
- return layer.releaseNonNull();
-}
-
Ref<PlatformCALayerRemote> PlatformCALayerRemote::create(const PlatformCALayerRemote& other, WebCore::PlatformCALayerClient* owner, RemoteLayerTreeContext& context)
{
auto layer = adoptRef(*new PlatformCALayerRemote(other, owner, context));
@@ -87,12 +80,6 @@
}
}
-PlatformCALayerRemote::PlatformCALayerRemote(LayerType layerType, GraphicsLayer::EmbeddedViewID embeddedViewID, PlatformCALayerClient* owner, RemoteLayerTreeContext& context)
- : PlatformCALayerRemote(layerType, owner, context)
-{
- m_embeddedViewID = embeddedViewID;
-}
-
PlatformCALayerRemote::PlatformCALayerRemote(const PlatformCALayerRemote& other, PlatformCALayerClient* owner, RemoteLayerTreeContext& context)
: PlatformCALayer(other.layerType(), owner)
, m_acceleratesDrawing(other.acceleratesDrawing())
@@ -858,11 +845,6 @@
m_properties.notePropertiesChanged(RemoteLayerTreeTransaction::CustomAppearanceChanged);
}
-GraphicsLayer::EmbeddedViewID PlatformCALayerRemote::embeddedViewID() const
-{
- return m_embeddedViewID;
-}
-
Ref<PlatformCALayer> PlatformCALayerRemote::createCompatibleLayer(PlatformCALayer::LayerType layerType, PlatformCALayerClient* client) const
{
return PlatformCALayerRemote::create(layerType, client, *m_context);
Modified: trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h (238069 => 238070)
--- trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -42,7 +42,6 @@
static Ref<PlatformCALayerRemote> create(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient*, RemoteLayerTreeContext&);
static Ref<PlatformCALayerRemote> create(PlatformLayer *, WebCore::PlatformCALayerClient*, RemoteLayerTreeContext&);
static Ref<PlatformCALayerRemote> create(const PlatformCALayerRemote&, WebCore::PlatformCALayerClient*, RemoteLayerTreeContext&);
- static Ref<PlatformCALayerRemote> createForEmbeddedView(WebCore::PlatformCALayer::LayerType, WebCore::GraphicsLayer::EmbeddedViewID, WebCore::PlatformCALayerClient*, RemoteLayerTreeContext&);
virtual ~PlatformCALayerRemote();
@@ -174,8 +173,6 @@
WebCore::GraphicsLayer::CustomAppearance customAppearance() const override;
void updateCustomAppearance(WebCore::GraphicsLayer::CustomAppearance) override;
- WebCore::GraphicsLayer::EmbeddedViewID embeddedViewID() const override;
-
WebCore::TiledBacking* tiledBacking() override { return nullptr; }
Ref<WebCore::PlatformCALayer> clone(WebCore::PlatformCALayerClient* owner) const override;
@@ -200,7 +197,6 @@
protected:
PlatformCALayerRemote(WebCore::PlatformCALayer::LayerType, WebCore::PlatformCALayerClient* owner, RemoteLayerTreeContext&);
- PlatformCALayerRemote(WebCore::PlatformCALayer::LayerType, WebCore::GraphicsLayer::EmbeddedViewID, WebCore::PlatformCALayerClient* owner, RemoteLayerTreeContext&);
PlatformCALayerRemote(const PlatformCALayerRemote&, WebCore::PlatformCALayerClient*, RemoteLayerTreeContext&);
void updateClonedLayerProperties(PlatformCALayerRemote& clone, bool copyContents = true) const;
@@ -221,8 +217,6 @@
PlatformCALayerRemote* m_maskLayer { nullptr };
HashMap<String, RefPtr<WebCore::PlatformCAAnimation>> m_animations;
- WebCore::GraphicsLayer::EmbeddedViewID m_embeddedViewID;
-
bool m_acceleratesDrawing { false };
bool m_wantsDeepColorBackingStore { false };
Modified: trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.mm (238069 => 238070)
--- trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -70,7 +70,6 @@
RemoteLayerTreeTransaction::LayerCreationProperties creationProperties;
creationProperties.layerID = layerID;
creationProperties.type = type;
- creationProperties.embeddedViewID = layer.embeddedViewID();
if (layer.isPlatformCALayerRemoteCustom()) {
creationProperties.hostingContextID = layer.hostingContextID();
Modified: trunk/Tools/ChangeLog (238069 => 238070)
--- trunk/Tools/ChangeLog 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/ChangeLog 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,3 +1,15 @@
+2018-11-10 Ryan Haddad <ryanhad...@apple.com>
+
+ Unreviewed, rolling out r238065.
+
+ Breaks internal builds.
+
+ Reverted changeset:
+
+ "Make it possible to edit images inline"
+ https://bugs.webkit.org/show_bug.cgi?id=191352
+ https://trac.webkit.org/changeset/238065
+
2018-11-10 Michael Catanzaro <mcatanz...@igalia.com>
[WPE][GTK] API test /webkit/WebKitSettings/webkit-settings is failing
Modified: trunk/Tools/DumpRenderTree/ios/UIScriptControllerIOS.mm (238069 => 238070)
--- trunk/Tools/DumpRenderTree/ios/UIScriptControllerIOS.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/DumpRenderTree/ios/UIScriptControllerIOS.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -407,15 +407,6 @@
{
}
-void UIScriptController::drawSquareInEditableImage()
-{
}
-long UIScriptController::numberOfStrokesInEditableImage()
-{
- return 0;
-}
-
-}
-
#endif // PLATFORM(IOS_FAMILY)
Modified: trunk/Tools/TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl (238069 => 238070)
--- trunk/Tools/TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl 2018-11-10 21:01:36 UTC (rev 238070)
@@ -272,7 +272,4 @@
void firstResponderSuppressionForWebView(boolean shouldSuppress);
void makeWindowContentViewFirstResponder();
readonly attribute boolean isWindowContentViewFirstResponder;
-
- void drawSquareInEditableImage();
- readonly attribute long numberOfStrokesInEditableImage;
};
Modified: trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.cpp (238069 => 238070)
--- trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -481,15 +481,6 @@
{
}
-void UIScriptController::drawSquareInEditableImage()
-{
-}
-
-long UIScriptController::numberOfStrokesInEditableImage()
-{
- return 0;
-}
-
#endif
#if !PLATFORM(COCOA)
Modified: trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.h (238069 => 238070)
--- trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/TestRunnerShared/UIScriptContext/UIScriptController.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -194,9 +194,6 @@
void makeWindowContentViewFirstResponder();
bool isWindowContentViewFirstResponder() const;
- void drawSquareInEditableImage();
- long numberOfStrokesInEditableImage();
-
private:
UIScriptController(UIScriptContext&);
Deleted: trunk/Tools/TestRunnerShared/spi/PencilKitTestSPI.h (238069 => 238070)
--- trunk/Tools/TestRunnerShared/spi/PencilKitTestSPI.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/TestRunnerShared/spi/PencilKitTestSPI.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1,62 +0,0 @@
-/*
- * Copyright (C) 2018 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#import <wtf/Platform.h>
-
-#if PLATFORM(IOS_FAMILY)
-
-#if USE(APPLE_INTERNAL_SDK)
-
-#import <PencilKit/PencilKit.h>
-
-#import <PencilKit/PKDrawing_Private.h>
-#import <PencilKit/PKStroke_Private.h>
-
-#else
-
-typedef NSInteger PKInkType;
-
-@interface PKInk : NSObject
-+ (PKInk *)inkWithType:(PKInkType)type color:(UIColor *)color weight:(CGFloat)weight;
-@end
-
-@interface PKStroke : NSObject
-- (id)_initWithPath:(CGPathRef)path ink:(PKInk *)ink inputScale:(CGFloat)inputScale;
-@end
-
-@interface PKDrawing : NSObject
-- (PKStroke *)_addStroke:(PKStroke *)stroke;
-@property (nonatomic, readonly) NSArray<PKStroke *> *_allStrokes;
-@end
-
-@interface PKCanvasView : UIView
-@property (nonatomic, copy) PKDrawing *drawing;
-@end
-
-#endif // USE(APPLE_INTERNAL_SDK)
-
-#endif // PLATFORM(IOS_FAMILY)
Modified: trunk/Tools/WebKitTestRunner/TestController.cpp (238069 => 238070)
--- trunk/Tools/WebKitTestRunner/TestController.cpp 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/WebKitTestRunner/TestController.cpp 2018-11-10 21:01:36 UTC (rev 238070)
@@ -1239,8 +1239,6 @@
testOptions.shouldIgnoreMetaViewport = parseBooleanTestHeaderValue(value);
else if (key == "spellCheckingDots")
testOptions.shouldShowSpellCheckingDots = parseBooleanTestHeaderValue(value);
- else if (key == "enableEditableImages")
- testOptions.enableEditableImages = parseBooleanTestHeaderValue(value);
pairStart = pairEnd + 1;
}
}
Modified: trunk/Tools/WebKitTestRunner/TestOptions.h (238069 => 238070)
--- trunk/Tools/WebKitTestRunner/TestOptions.h 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/WebKitTestRunner/TestOptions.h 2018-11-10 21:01:36 UTC (rev 238070)
@@ -64,7 +64,6 @@
bool checkForWorldLeaks { false };
bool shouldIgnoreMetaViewport { false };
bool shouldShowSpellCheckingDots { false };
- bool enableEditableImages { false };
float deviceScaleFactor { 1 };
Vector<String> overrideLanguages;
@@ -106,8 +105,7 @@
|| runSingly != options.runSingly
|| checkForWorldLeaks != options.checkForWorldLeaks
|| shouldShowSpellCheckingDots != options.shouldShowSpellCheckingDots
- || shouldIgnoreMetaViewport != options.shouldIgnoreMetaViewport
- || enableEditableImages != options.enableEditableImages)
+ || shouldIgnoreMetaViewport != options.shouldIgnoreMetaViewport)
return false;
if (experimentalFeatures != options.experimentalFeatures)
Modified: trunk/Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj (238069 => 238070)
--- trunk/Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj 2018-11-10 21:01:36 UTC (rev 238070)
@@ -256,7 +256,6 @@
29A8FCE1345E7021006AA5A6 /* JSAccessibilityTextMarkerRange.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSAccessibilityTextMarkerRange.cpp; sourceTree = "<group>"; };
29A8FCE1345E7021006AA5A7 /* JSAccessibilityTextMarkerRange.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSAccessibilityTextMarkerRange.h; sourceTree = "<group>"; };
29A8FCE4145F0464009045A6 /* AccessibilityTextMarkerRangeMac.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = AccessibilityTextMarkerRangeMac.mm; sourceTree = "<group>"; };
- 2D0FF14A2192C78300C975D6 /* PencilKitTestSPI.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = PencilKitTestSPI.h; path = ../TestRunnerShared/spi/PencilKitTestSPI.h; sourceTree = "<group>"; };
2DB6187D1D7D58D400978D19 /* CoreGraphicsTestSPI.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = CoreGraphicsTestSPI.h; path = ../TestRunnerShared/spi/CoreGraphicsTestSPI.h; sourceTree = "<group>"; };
2DCE2CD11B84524500C7F832 /* TestControllerCocoa.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; name = TestControllerCocoa.mm; path = cocoa/TestControllerCocoa.mm; sourceTree = "<group>"; };
2DD4C4981D6E7D3B0007379C /* EventSerializerMac.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = EventSerializerMac.h; path = mac/EventSerializerMac.h; sourceTree = "<group>"; };
@@ -597,7 +596,6 @@
isa = PBXGroup;
children = (
2DB6187D1D7D58D400978D19 /* CoreGraphicsTestSPI.h */,
- 2D0FF14A2192C78300C975D6 /* PencilKitTestSPI.h */,
31268EB91EF06A3B001963E2 /* UIKitTestSPI.h */,
);
name = spi;
Modified: trunk/Tools/WebKitTestRunner/cocoa/TestControllerCocoa.mm (238069 => 238070)
--- trunk/Tools/WebKitTestRunner/cocoa/TestControllerCocoa.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/WebKitTestRunner/cocoa/TestControllerCocoa.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -154,14 +154,11 @@
#endif
if (options.enableAttachmentElement)
- [copiedConfiguration _setAttachmentElementEnabled:YES];
+ [copiedConfiguration _setAttachmentElementEnabled: YES];
if (options.enableColorFilter)
- [copiedConfiguration _setColorFilterEnabled:YES];
+ [copiedConfiguration _setColorFilterEnabled: YES];
- if (options.enableEditableImages)
- [copiedConfiguration _setEditableImagesEnabled:YES];
-
if (options.applicationManifest.length()) {
auto manifestPath = [NSString stringWithUTF8String:options.applicationManifest.c_str()];
NSString *text = [NSString stringWithContentsOfFile:manifestPath usedEncoding:nullptr error:nullptr];
Modified: trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm (238069 => 238070)
--- trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm 2018-11-10 20:12:06 UTC (rev 238069)
+++ trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm 2018-11-10 21:01:36 UTC (rev 238070)
@@ -29,7 +29,6 @@
#if PLATFORM(IOS_FAMILY)
#import "HIDEventGenerator.h"
-#import "PencilKitTestSPI.h"
#import "PlatformWebView.h"
#import "StringFunctions.h"
#import "TestController.h"
@@ -784,43 +783,6 @@
return foundDataListSuggestionsPickerView;
}
-PKCanvasView *findEditableImageCanvas()
-{
- TestRunnerWKWebView *webView = TestController::singleton().mainWebView()->platformView();
- Class pkCanvasViewClass = NSClassFromString(@"PKCanvasView");
- __block PKCanvasView *canvasView = nil;
- forEachViewInHierarchy(webView.window, ^(UIView *subview, BOOL *stop) {
- if (![subview isKindOfClass:pkCanvasViewClass])
- return;
-
- canvasView = (PKCanvasView *)subview;
- *stop = YES;
- });
- return canvasView;
}
-void UIScriptController::drawSquareInEditableImage()
-{
- Class pkDrawingClass = NSClassFromString(@"PKDrawing");
- Class pkInkClass = NSClassFromString(@"PKInk");
- Class pkStrokeClass = NSClassFromString(@"PKStroke");
-
- PKCanvasView *canvasView = findEditableImageCanvas();
- RetainPtr<PKDrawing> drawing = adoptNS([[pkDrawingClass alloc] init]);
- RetainPtr<CGPathRef> path = adoptCF(CGPathCreateWithRect(CGRectMake(0, 0, 50, 50), NULL));
- RetainPtr<PKInk> ink = [pkInkClass inkWithType:0 color:UIColor.greenColor weight:100.0];
- RetainPtr<PKStroke> stroke = adoptNS([[pkStrokeClass alloc] _initWithPath:path.get() ink:ink.get() inputScale:1]);
- [drawing _addStroke:stroke.get()];
-
- [canvasView setDrawing:drawing.get()];
-}
-
-long UIScriptController::numberOfStrokesInEditableImage()
-{
- PKCanvasView *canvasView = findEditableImageCanvas();
- return canvasView.drawing._allStrokes.count;
-}
-
-}
-
#endif // PLATFORM(IOS_FAMILY)