⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Nov 2, 2010:

11:59 PM Changeset in webkit [71215] by gyuyoung.kim@samsung.com
  • 7 edits in trunk

2010-11-02 Gyuyoung Kim <gyuyoung.kim@samsung.com>

Unreviewed: Add myself to the list of Committers.

  • Scripts/webkitpy/common/config/committers.py:
10:29 PM Changeset in webkit [71214] by Martin Robinson
  • 2 edits
    14 adds in trunk/WebKit2

2010-11-02 Amruth Raj <amruthraj@motorola.com> and Ravi Kasibhatla <ravi.kasibhatla@motorola.com>

Reviewed by Martin Robinson.

Changes to enable building WebKit2 for Gtk port.
(https://bugs.webkit.org/show_bug.cgi?id=37369)

  • Platform/gtk: Added.
  • Platform/gtk/SharedMemoryGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::SharedMemory::Handle::Handle): (WebKit::SharedMemory::Handle::~Handle): (WebKit::SharedMemory::Handle::encode): (WebKit::SharedMemory::Handle::decode): (WebKit::SharedMemory::create): (WebKit::SharedMemory::~SharedMemory): (WebKit::SharedMemory::createHandle): (WebKit::SharedMemory::systemPageSize):
  • Shared/gtk: Added.
  • Shared/gtk/BackingStoreGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::BackingStore::createGraphicsContext): (WebKit::BackingStore::paint):
  • Shared/gtk/PlatformCertificateInfo.h: Added. Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::PlatformCertificateInfo::PlatformCertificateInfo): (WebKit::PlatformCertificateInfo::encode): (WebKit::PlatformCertificateInfo::decode):
  • Shared/gtk/WebCoreArgumentCodersGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (CoreIPC::encodeResourceRequest): (CoreIPC::decodeResourceRequest):
  • UIProcess/Plugins/gtk: Added.
  • UIProcess/Plugins/gtk/PluginInfoStoreGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::PluginInfoStore::pluginsDirectories): (WebKit::PluginInfoStore::pluginPathsInDirectory): (WebKit::PluginInfoStore::individualPluginPaths): (WebKit::PluginInfoStore::getPluginInfo): (WebKit::PluginInfoStore::shouldUsePlugin):
  • WebProcess/InjectedBundle/InjectedBundle.h:
  • WebProcess/InjectedBundle/gtk: Added.
  • WebProcess/InjectedBundle/gtk/InjectedBundleGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::InjectedBundle::load): (WebKit::InjectedBundle::activateMacFontAscentHack):
  • WebProcess/Plugins/Netscape/gtk: Added.
  • WebProcess/Plugins/Netscape/gtk/NetscapePluginGtk.cpp: Added. Stubbed implementation for GTK port. Yet to implement. (WebKit::NetscapePlugin::platformPostInitialize): (WebKit::NetscapePlugin::platformDestroy): (WebKit::NetscapePlugin::platformGeometryDidChange): (WebKit::NetscapePlugin::platformPaint): (WebKit::toNP): (WebKit::NetscapePlugin::platformHandleMouseEvent): (WebKit::NetscapePlugin::platformHandleWheelEvent): (WebKit::NetscapePlugin::platformSetFocus): (WebKit::NetscapePlugin::platformHandleMouseEnterEvent): (WebKit::NetscapePlugin::platformHandleMouseLeaveEvent): (WebKit::NetscapePlugin::platformHandleKeyboardEvent):
  • WebProcess/WebCoreSupport/gtk: Added.
  • WebProcess/WebCoreSupport/gtk/WebFrameNetworkingContext.h: Added. Added. Stubbed implementation for GTK port. Yet to implement. (WebFrameNetworkingContext::create): (WebFrameNetworkingContext::WebFrameNetworkingContext):
9:53 PM WebKitGtkLayoutTests edited by Martin Robinson
Fixed the color depth argument on Xvfb calls (diff)
9:41 PM Changeset in webkit [71213] by crogers@google.com
  • 10 edits in trunk

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by James Robinson.

Add AudioProcessingEvent files
https://bugs.webkit.org/show_bug.cgi?id=48884

No new tests since audio API is not yet implemented.

  • webaudio/AudioProcessingEvent.cpp: Added. (WebCore::AudioProcessingEvent::create): (WebCore::AudioProcessingEvent::AudioProcessingEvent): (WebCore::AudioProcessingEvent::~AudioProcessingEvent): (WebCore::AudioProcessingEvent::isAudioProcessingEvent):
  • webaudio/AudioProcessingEvent.h: Added. (WebCore::AudioProcessingEvent::inputBuffer): (WebCore::AudioProcessingEvent::outputBuffer):
  • webaudio/AudioProcessingEvent.idl: Added.

2010-11-02 Noam Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Some refactor for texmap to enable WebKit2: remove globals, and allow TextureMapper to exist without a GraphicsContext.
This will allow rendering the TextureMapperNode tree without an active QPainter, into the current GL context.
Most of the changes simply move the globas in TextureMapperGL into members of that class.

[Texmap] [Qt] Texture mapper initial implementation
https://bugs.webkit.org/show_bug.cgi?id=47070

  • Api/qwebframe.cpp: (QWebFramePrivate::renderFromTiledBackingStore): (QWebFramePrivate::renderCompositedLayers): (QWebFramePrivate::renderRelativeCoords):
  • Api/qwebframe_p.h:
9:36 PM Changeset in webkit [71212] by crogers@google.com
  • 1 edit
    3 adds in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by James Robinson.

Add AudioProcessingEvent files
https://bugs.webkit.org/show_bug.cgi?id=48884

No new tests since audio API is not yet implemented.

  • webaudio/AudioProcessingEvent.cpp: Added. (WebCore::AudioProcessingEvent::create): (WebCore::AudioProcessingEvent::AudioProcessingEvent): (WebCore::AudioProcessingEvent::~AudioProcessingEvent): (WebCore::AudioProcessingEvent::isAudioProcessingEvent):
  • webaudio/AudioProcessingEvent.h: Added. (WebCore::AudioProcessingEvent::inputBuffer): (WebCore::AudioProcessingEvent::outputBuffer):
  • webaudio/AudioProcessingEvent.idl: Added.
9:23 PM Changeset in webkit [71211] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-11-02 Benjamin Kalman <kalman@chromium.org>

Reviewed by Ojan Vafai.

canvas/philip tests use BUG instead of BUGWK in chromium test_expectations.txt.
https://bugs.webkit.org/show_bug.cgi?id=48879

  • platform/chromium/test_expectations.txt: Update canvas/philip tests to use BUGWK instead of BUG.
9:16 PM Changeset in webkit [71210] by commit-queue@webkit.org
  • 11 edits
    2 adds in trunk

2010-11-02 Al Patrick <apatrick@chromium.org>

Reviewed by Kenneth Russell.

Added PluginLayerChromium, which composites plugin instances that have an associated OpenGL backing texture.

https://bugs.webkit.org/show_bug.cgi?id=48032

  • WebCore.gypi:
  • loader/SubframeLoader.cpp (WebCore::SubframeLoader::loadPlugin):
  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::setNeedsDisplay):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::initializeSharedObjects): (WebCore::LayerRendererChromium::cleanupSharedObjects):
  • platform/graphics/chromium/LayerRendererChromium.h: (WebCore::LayerRendererChromium::pluginLayerSharedValues):
  • platform/graphics/chromium/PluginLayerChromium.cpp: Added. (WebCore::PluginLayerChromium::SharedValues::SharedValues): (WebCore::PluginLayerChromium::SharedValues::~SharedValues): (WebCore::PluginLayerChromium::create): (WebCore::PluginLayerChromium::PluginLayerChromium): (WebCore::PluginLayerChromium::updateContents): (WebCore::PluginLayerChromium::draw):
  • platform/graphics/chromium/PluginLayerChromium.h: Added. (WebCore::PluginLayerChromium::drawsContent): (WebCore::PluginLayerChromium::setTextureId): (WebCore::PluginLayerChromium::SharedValues::shaderProgram): (WebCore::PluginLayerChromium::SharedValues::shaderSamplerLocation): (WebCore::PluginLayerChromium::SharedValues::shaderMatrixLocation): (WebCore::PluginLayerChromium::SharedValues::shaderAlphaLocation): (WebCore::PluginLayerChromium::SharedValues::initialized):

2010-11-02 Al Patrick <apatrick@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Plugin instances can propagate the ID of the OpenGL texture they render to.

https://bugs.webkit.org/show_bug.cgi?id=48032

  • public/WebPlugin.h: (WebKit::WebPlugin::getBackingTextureId):
  • public/WebPluginContainer.h: (WebKit::WebPluginContainer::commitBackingTexture):
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::commitBackingTexture): (WebKit::WebPluginContainerImpl::platformLayer): (WebKit::WebPluginContainerImpl::WebPluginContainerImpl):
  • src/WebPluginContainerImpl.h:
8:56 PM Changeset in webkit [71209] by commit-queue@webkit.org
  • 4 edits in trunk

2010-11-02 James Simonsen <simonjam@chromium.org>

Reviewed by Adam Barth.

mathml in html sometimes incorrectly parsed
https://bugs.webkit.org/show_bug.cgi?id=48105

  • html5lib/resources/webkit01.dat: Test case for bug.

2010-11-02 James Simonsen <simonjam@chromium.org>

Reviewed by Adam Barth.

mathml in html sometimes incorrectly parsed
https://bugs.webkit.org/show_bug.cgi?id=48105

  • html/parser/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTag): Fixed to match HTML5 spec.
8:46 PM Changeset in webkit [71208] by rolandsteiner@chromium.org
  • 2 edits in trunk/WebCore

2010-11-02 Chris Guillory <chris.guillory@google.com>

Reviewed by Dmitry Titov.

Null check obj in AXObjectCache::postPlatformNotification.
https://bugs.webkit.org/show_bug.cgi?id=48896

  • accessibility/chromium/AXObjectCacheChromium.cpp: (WebCore::AXObjectCache::postPlatformNotification):
8:31 PM Changeset in webkit [71207] by dimich@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Add Chromium baseline for a test expectations added
in http://trac.webkit.org/changeset/71192.

  • platform/chromium-mac/fast/canvas/canvas-zoom-expected.checksum: Added.
7:59 PM Changeset in webkit [71206] by commit-queue@webkit.org
  • 7 edits in trunk

2010-11-02 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Dumitru Daniliuc.

[FileSystem] Support not creating directories when queried by inspector.
https://bugs.webkit.org/show_bug.cgi?id=48169

  • fileapi/LocalFileSystem.cpp: (WebCore::openFileSystem): (WebCore::LocalFileSystem::readFileSystem): (WebCore::LocalFileSystem::requestFileSystem):
  • fileapi/LocalFileSystem.h:
  • platform/AsyncFileSystem.cpp: (WebCore::AsyncFileSystem::openFileSystem):
  • platform/AsyncFileSystem.h:

2010-11-02 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Dumitru Daniliuc.

[FileSystem] Support not creating directories when queried by inspector.
https://bugs.webkit.org/show_bug.cgi?id=48169

  • src/LocalFileSystemChromium.cpp: (WebCore::LocalFileSystem::readFileSystem): (WebCore::LocalFileSystem::requestFileSystem):
7:35 PM Changeset in webkit [71205] by crogers@google.com
  • 1 edit
    3 adds in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Add AudioBufferSourceNode files
https://bugs.webkit.org/show_bug.cgi?id=48012

No new tests since audio API is not yet implemented.

  • webaudio/AudioBufferSourceNode.cpp: Added. (WebCore::AudioBufferSourceNode::create): (WebCore::AudioBufferSourceNode::AudioBufferSourceNode): (WebCore::AudioBufferSourceNode::~AudioBufferSourceNode): (WebCore::AudioBufferSourceNode::process): (WebCore::AudioBufferSourceNode::provideInput): (WebCore::AudioBufferSourceNode::readFromBuffer): (WebCore::AudioBufferSourceNode::readFromBufferWithGrainEnvelope): (WebCore::AudioBufferSourceNode::reset): (WebCore::AudioBufferSourceNode::setBuffer): (WebCore::AudioBufferSourceNode::numberOfChannels): (WebCore::AudioBufferSourceNode::noteOn): (WebCore::AudioBufferSourceNode::noteGrainOn): (WebCore::AudioBufferSourceNode::noteOff): (WebCore::AudioBufferSourceNode::totalPitchRate):
  • webaudio/AudioBufferSourceNode.h: Added. (WebCore::AudioBufferSourceNode::buffer): (WebCore::AudioBufferSourceNode::looping): (WebCore::AudioBufferSourceNode::setLooping): (WebCore::AudioBufferSourceNode::gain): (WebCore::AudioBufferSourceNode::playbackRate): (WebCore::AudioBufferSourceNode::setPannerNode):
  • webaudio/AudioBufferSourceNode.idl: Added.
7:23 PM Changeset in webkit [71204] by Martin Robinson
  • 17 edits
    2 moves in trunk

2010-11-02 Martin Robinson <mrobinson@igalia.com>

Reviewed by Adam Barth.

[Cairo] Remove PlatformRefPtrCairo
https://bugs.webkit.org/show_bug.cgi?id=48192

Replace PlatformRefPtrCairo with a RefPtr specialization for Cairo and
Fontconfig types. This is the first step toward removing PlatformRefPtr,
whose job is better handled by RefPtr.

  • CMakeListsEfl.txt: Update the sources list to include RefPtrCairo instead of PlatformRefPtrCairo.
  • GNUmakefile.am: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • platform/graphics/GraphicsContext.h: Replace PlatformRefPtrCairo with RefPtrCairo.
  • platform/graphics/cairo/CairoUtilities.cpp: Ditto. (WebCore::drawPatternToCairoContext): Ditto.
  • platform/graphics/cairo/FontCacheFreeType.cpp: Ditto. (WebCore::FontCache::getFontDataForCharacters): Ditto. (WebCore::FontCache::createFontPlatformData): Ditto.
  • platform/graphics/cairo/FontPlatformDataFreeType.cpp: Ditto. (WebCore::FontPlatformData::FontPlatformData): Ditto. (WebCore::FontPlatformData::initializeWithFontFace): Ditto.
  • platform/graphics/cairo/FontPlatformDataFreeType.h: Ditto.
  • platform/graphics/cairo/GraphicsContextCairo.cpp: Ditto.
  • platform/graphics/cairo/ImageCairo.cpp: Ditto.
  • platform/graphics/cairo/RefPtrCairo.cpp: Renamed from WebCore/platform/graphics/cairo/PlatformRefPtrCairo.cpp. (WTF::refIfNotNull): Added. (WTF::derefIfNotNull): Added.
  • platform/graphics/cairo/RefPtrCairo.h: Renamed from WebCore/platform/graphics/cairo/PlatformRefPtrCairo.h.
  • platform/gtk/CursorGtk.cpp: Replace PlatformRefPtrCairo with RefPtrCairo. (WebCore::createNamedCursor): Ditto.
  • platform/gtk/DragImageGtk.cpp: Ditto. (WebCore::scaleDragImage): Ditto. (WebCore::dissolveDragImageToFraction): Ditto.
  • plugins/gtk/PluginViewGtk.cpp: Ditto. (WebCore::PluginView::paint): Ditto.

2010-11-02 Martin Robinson <mrobinson@igalia.com>

Reviewed by Adam Barth.

[Cairo] Remove PlatformRefPtrCairo
https://bugs.webkit.org/show_bug.cgi?id=48192

Replace instances of PlatformRefPtr smart pointers that wrap Cairo
types with RefPtrs.

  • WebCoreSupport/DragClientGtk.cpp: Use RefPtr instead of PlatformRefPtr. (WebKit::dragIconWindowDrawEventCallback):
  • WebCoreSupport/DragClientGtk.h: Ditto.
7:18 PM Changeset in webkit [71203] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit2

2010-11-02 Jing Jin <jjin@apple.com>

Reviewed by Alexey Proskuryakov.

Key down calls are not being sent to the application.
https://bugs.webkit.org/show_bug.cgi?id=48867

  • UIProcess/API/mac/WKView.mm: (-[WKView keyDown:]): Call super when ignoring a key down.
7:05 PM Changeset in webkit [71202] by commit-queue@webkit.org
  • 2 edits
    5 deletes in branches/chromium/552

Revert 71176 - Merge 71170

BUG=59554

TBR=inferno@chromium.org

7:02 PM Changeset in webkit [71201] by dimich@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Add Chromium baseline for test added http://trac.webkit.org/changeset/71177.

  • platform/chromium-mac/fast/repaint/selection-rl-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/selection-rl-expected.png: Added.
6:55 PM Changeset in webkit [71200] by crogers@google.com
  • 6 edits in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Simple followup changes to files affected by AudioNodeInput thread safety
https://bugs.webkit.org/show_bug.cgi?id=48661

No new tests since audio API is not yet implemented.

  • webaudio/AudioBasicProcessorNode.cpp: (WebCore::AudioBasicProcessorNode::checkNumberOfChannelsForInput):
  • webaudio/AudioChannelSplitter.cpp: (WebCore::AudioChannelSplitter::process):
  • webaudio/AudioDestinationNode.cpp: (WebCore::AudioDestinationNode::initialize): (WebCore::AudioDestinationNode::uninitialize): (WebCore::AudioDestinationNode::provideInput):
  • webaudio/AudioGainNode.cpp: (WebCore::AudioGainNode::checkNumberOfChannelsForInput):
  • webaudio/AudioPannerNode.cpp: (WebCore::AudioPannerNode::notifyAudioSourcesConnectedToNode):
6:40 PM Changeset in webkit [71199] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-11-02 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

Fix typo in webgl-test-util.js.
https://bugs.webkit.org/show_bug.cgi?id=48861

  • fast/canvas/webgl/resources/webgl-test-utils.js: (WebGLTestUtils):
6:20 PM Changeset in webkit [71198] by commit-queue@webkit.org
  • 9 edits in trunk

2010-11-02 Chris Guillory <chris.guillory@google.com>

Reviewed by Chris Fleizach.

Chromium: Propagate a document value changed notification on scroll.
https://bugs.webkit.org/show_bug.cgi?id=48817

  • platform/chromium/accessibility/post-notification-ValueChanged-expected.txt:
  • platform/chromium/accessibility/post-notification-ValueChanged.html:

2010-11-02 Chris Guillory <chris.guillory@google.com>

Reviewed by Chris Fleizach.

Chromium: Propagate a document value changed notification on scroll.
https://bugs.webkit.org/show_bug.cgi?id=48817

  • accessibility/AccessibilityObject.h: (WebCore::AccessibilityObject::isAccessibilityScrollbar):
  • accessibility/AccessibilityScrollbar.h: (WebCore::AccessibilityScrollbar::scrollbar): (WebCore::AccessibilityScrollbar::isAccessibilityScrollbar):
  • accessibility/chromium/AXObjectCacheChromium.cpp: (WebCore::AXObjectCache::postPlatformNotification):

2010-11-02 Chris Guillory <chris.guillory@google.com>

Reviewed by Chris Fleizach.

Chromium: Propagate a document value changed notification on scroll.
https://bugs.webkit.org/show_bug.cgi?id=48817

  • src/WebAccessibilityObject.cpp: (WebKit::WebAccessibilityObject::boundingBoxRect):
6:18 PM Changeset in webkit [71197] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.

  • platform/chromium/test_expectations.txt:
6:07 PM Changeset in webkit [71196] by Michael Nordman
  • 4 edits in trunk/LayoutTests

2010-11-02 Michael Nordman <Michael Nordman>

Reviewed by Alexey Proskuryakov.

[Chromium] Some work on making appcache layout tests less flaky.
https://bugs.webkit.org/show_bug.cgi?id=48002
1) Make network-simulator.php less slow.
In addition to redirecting back to the requested url, the response contains
multiple content-length headers. Chrome's network stacks considers that an
error condition too and picks up on it faster (one round trip instead of multiple).
This avoids timeouts due to using PHP via slow CGI on windows.
2) Removed canLoad() from appcache/fallback.html layout test, the load()
result is sufficient for determining if it could be loaded. So fewer requests are
made to conduct the test, cutting out more time.
3) Removed some unneeded code from offline-access-frame.html. Appcache events are guaranteed
to be deferred until after 'onload' has happened. This test was coded prior to that
guarantee being satisfied.

  • http/tests/appcache/fallback.html:
  • http/tests/resources/network-simulator.php:
  • http/tests/resources/offline-access-frame.html:
6:00 PM Changeset in webkit [71195] by crogers@google.com
  • 1 edit
    3 adds in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Add ConvolverNode files
https://bugs.webkit.org/show_bug.cgi?id=47941

No new tests since audio API is not yet implemented.

  • webaudio/ConvolverNode.cpp: Added. (WebCore::ConvolverNode::ConvolverNode): (WebCore::ConvolverNode::~ConvolverNode): (WebCore::ConvolverNode::process): (WebCore::ConvolverNode::reset): (WebCore::ConvolverNode::initialize): (WebCore::ConvolverNode::uninitialize): (WebCore::ConvolverNode::setBuffer): (WebCore::ConvolverNode::buffer):
  • webaudio/ConvolverNode.h: Added. (WebCore::ConvolverNode::create):
  • webaudio/ConvolverNode.idl: Added.
5:59 PM Changeset in webkit [71194] by Simon Fraser
  • 3 edits
    5 adds in trunk/WebCore

2010-11-02 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Overlapped nested iframes show smearing when scrolled
https://bugs.webkit.org/show_bug.cgi?id=48873

A FrameView goes into slow scrolling mode (not blitting) when told that it is
overlapped. However, a nested iframe could still try to use fast scrolling
when its parent is overlapped, which could result in bad rendering.

Fixed by making useSlowRepaints() and useSlowRepaintsIfNotOverlapped()
walk up their parent chain, asking if their parents also use slow
repaints.

All the places that call setCanBlitOnScroll() now call
updateCanBlitOnScrollRecursively() to ensure that the canBlitOnScroll
flag is updated correctly on subframes.

Repaint tests do not work on nested iframes, so added manual test.

  • manual-tests/frames/nested-iframe-blit-on-scroll.html: Added.
  • manual-tests/frames/resources/blit-on-scroll-subframe.html: Added.
  • manual-tests/frames/resources/blit-on-scroll-subsubframe.html: Added.
  • page/FrameView.cpp: (WebCore::FrameView::layout): (WebCore::FrameView::useSlowRepaints): (WebCore::FrameView::useSlowRepaintsIfNotOverlapped): (WebCore::FrameView::updateCanBlitOnScrollRecursively): (WebCore::FrameView::setUseSlowRepaints): (WebCore::FrameView::addSlowRepaintObject): (WebCore::FrameView::removeSlowRepaintObject): (WebCore::FrameView::addFixedObject): (WebCore::FrameView::removeFixedObject): (WebCore::FrameView::setIsOverlapped): (WebCore::FrameView::setContentIsOpaque):
  • page/FrameView.h:
5:44 PM Changeset in webkit [71193] by crogers@google.com
  • 1 edit
    2 adds in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Add RealtimeAnalyser files
https://bugs.webkit.org/show_bug.cgi?id=48810

No new tests since audio API is not yet implemented.

  • webaudio/RealtimeAnalyser.cpp: Added. (WebCore::RealtimeAnalyser::RealtimeAnalyser): (WebCore::RealtimeAnalyser::~RealtimeAnalyser): (WebCore::RealtimeAnalyser::reset): (WebCore::RealtimeAnalyser::setFftSize): (WebCore::RealtimeAnalyser::writeInput): (WebCore::RealtimeAnalyser::doFFTAnalysis): (WebCore::RealtimeAnalyser::getFloatFrequencyData): (WebCore::RealtimeAnalyser::getByteFrequencyData): (WebCore::RealtimeAnalyser::getByteTimeDomainData):
  • webaudio/RealtimeAnalyser.h: Added. (WebCore::RealtimeAnalyser::fftSize): (WebCore::RealtimeAnalyser::frequencyBinCount): (WebCore::RealtimeAnalyser::setMinDecibels): (WebCore::RealtimeAnalyser::minDecibels): (WebCore::RealtimeAnalyser::setMaxDecibels): (WebCore::RealtimeAnalyser::maxDecibels): (WebCore::RealtimeAnalyser::setSmoothingTimeConstant): (WebCore::RealtimeAnalyser::smoothingTimeConstant): (WebCore::RealtimeAnalyser::magnitudeBuffer):
5:30 PM Changeset in webkit [71192] by mihaip@chromium.org
  • 225 edits
    224 copies
    6 adds in trunk/LayoutTests

2010-11-02 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/canvas, fast/css-generated-content, fast/inline-block, fast/inline, and fast/overflow for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

5:25 PM Changeset in webkit [71191] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-11-02 Ilya Sherman <isherman@chromium.org>

Reviewed by Kent Tamura.

Avoid overlapping label text in autofill popup with icon.
Updates the menu width computation to take the icon width into account.
https://bugs.webkit.org/show_bug.cgi?id=48497

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::paintRow): (WebCore::PopupListBox::layout):
5:23 PM Changeset in webkit [71190] by zmo@google.com
  • 8 edits in trunk

2010-10-28 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Perform framebuffer attachment checking upon draw call rather than attachment
https://bugs.webkit.org/show_bug.cgi?id=46318

  • html/canvas/WebGLFramebuffer.cpp: (WebCore::getInternalFormat): Helper function to get an attachment's internal format. (WebCore::isUninitialized): Helper function to decide if an attachment is initialized or not. (WebCore::setInitialized): Helper funtion to set an attachment as being initialized. (WebCore::WebGLFramebuffer::setAttachment): No longer perform buffer initialization at this point. (WebCore::WebGLFramebuffer::removeAttachment): Ditto. (WebCore::WebGLFramebuffer::getColorBufferFormat): Make it const. (WebCore::WebGLFramebuffer::isIncomplete): Attachments conflict check. (WebCore::WebGLFramebuffer::onAccess): Check atatchment conflicts and buffer initialization if necessary. (WebCore::WebGLFramebuffer::initializeRenderbuffers): Return a boolean whether the framebuffer is complete or not.
  • html/canvas/WebGLFramebuffer.h: Add new function declaration and remove some outdated functions. (WebCore::WebGLFramebuffer::isColorAttached): (WebCore::WebGLFramebuffer::isDepthAttached): Make this private. (WebCore::WebGLFramebuffer::isStencilAttached): Ditto. (WebCore::WebGLFramebuffer::isDepthStencilAttached): Ditto.
  • html/canvas/WebGLRenderbuffer.h: (WebCore::WebGLRenderbuffer::setInternalFormat): Also set the renderbuffer as uninitialized.
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::bindFramebuffer): No longer perform framebuffer initialization at this point. (WebCore::WebGLRenderingContext::checkFramebufferStatus): Check WebGL specific attachment conflicts. (WebCore::WebGLRenderingContext::clear): Call WebGLFramebuffer::onAccess. (WebCore::WebGLRenderingContext::copyTexImage2D): Ditto. (WebCore::WebGLRenderingContext::copyTexSubImage2D): Ditto. (WebCore::WebGLRenderingContext::drawArrays): Ditto. (WebCore::WebGLRenderingContext::drawElements): Ditto. (WebCore::WebGLRenderingContext::framebufferRenderbuffer): No longer perform framebuffer initialization at this point. (WebCore::WebGLRenderingContext::readPixels): Call WebGLFramebuffer::onAccess. (WebCore::WebGLRenderingContext::renderbufferStorage): No longer perform framebuffer initialization at this point. (WebCore::WebGLRenderingContext::texImage2DBase): Ditto.

2010-10-28 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Perform framebuffer attachment checking upon draw call rather than attachment
https://bugs.webkit.org/show_bug.cgi?id=46318

  • fast/canvas/webgl/framebuffer-object-attachment-expected.txt:
  • fast/canvas/webgl/framebuffer-object-attachment.html:
5:06 PM Changeset in webkit [71189] by mihaip@chromium.org
  • 231 edits
    230 copies
    2 adds in trunk/LayoutTests

2010-11-02 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/flexbox, fast/frames, fast/invalid, fast/layers, and fast/lists for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

4:50 PM Changeset in webkit [71188] by andersca@apple.com
  • 4 edits in trunk/WebKit2

Add Null and Int32 to NPVariantData
https://bugs.webkit.org/show_bug.cgi?id=48887

Reviewed by John Sullivan.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::npVariantToNPVariantData):
(WebKit::NPRemoteObjectMap::npVariantDataToNPVariant):

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::NPVariantData):
(WebKit::NPVariantData::makeNull):
(WebKit::NPVariantData::makeInt32):
(WebKit::NPVariantData::encode):
(WebKit::NPVariantData::decode):

  • Shared/Plugins/NPVariantData.h:

(WebKit::NPVariantData::int32Value):

4:40 PM Changeset in webkit [71187] by mihaip@chromium.org
  • 323 edits
    322 copies in trunk/LayoutTests

2010-11-02 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/forms for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

4:36 PM Changeset in webkit [71186] by jam@chromium.org
  • 4 edits in trunk/WebKit/chromium

2010-10-29 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Fisher.

[chromium]: Allow plugins to use optimized scrolling
https://bugs.webkit.org/show_bug.cgi?id=48660

  • public/WebPluginContainer.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::scrollRect):
  • src/WebPluginContainerImpl.h:
4:25 PM Changeset in webkit [71185] by zmo@google.com
  • 4 edits
    2 adds in trunk

2010-11-01 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

blendFunc should generate INVALID_OPERATION if constant color and constant alpha are together as source and destination factors
https://bugs.webkit.org/show_bug.cgi?id=48674

Test: fast/canvas/webgl/webgl-specific.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::blendFunc): (WebCore::WebGLRenderingContext::blendFuncSeparate):
  • html/canvas/WebGLRenderingContext.h: (WebCore::WebGLRenderingContext::validateBlendFuncFactors): Helper function to do the checking.

2010-11-01 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

blendFunc should generate INVALID_OPERATION if constant color and constant alpha are together as source and destination factors
https://bugs.webkit.org/show_bug.cgi?id=48674

  • fast/canvas/webgl/webgl-specific-expected.txt: Added.
  • fast/canvas/webgl/webgl-specific.html: Added.
4:20 PM Changeset in webkit [71184] by dimich@chromium.org
  • 1 edit
    5 adds in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Add Chromium baseline for a test added in http://trac.webkit.org/changeset/71177.

  • platform/chromium-linux/fast/repaint/selection-rl-expected.checksum: Added.
  • platform/chromium-linux/fast/repaint/selection-rl-expected.png: Added.
  • platform/chromium-win/fast/repaint/selection-rl-expected.checksum: Added.
  • platform/chromium-win/fast/repaint/selection-rl-expected.png: Added.
  • platform/chromium-win/fast/repaint/selection-rl-expected.txt: Added.
4:16 PM Changeset in webkit [71183] by andersca@apple.com
  • 9 edits in trunk

Add support for NP_HasMethod and NP_Invoke
https://bugs.webkit.org/show_bug.cgi?id=48882

Reviewed by Adam Roben.

WebKit2:

  • Platform/CoreIPC/HandleMessage.h:

(CoreIPC::callMemberFunction):
Add new overload.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::hasMethod):
Call hasMethod and return the result.

(WebKit::NPObjectMessageReceiver::invoke):
Convert the arguments, call invoke and return the result.

(WebKit::NPObjectMessageReceiver::getProperty):
Release the converted variant.

  • Shared/Plugins/NPObjectMessageReceiver.messages.in:

Add HasMethod and Invoke messages.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::hasMethod):
Send the HasMethod message.

(WebKit::NPObjectProxy::invoke):
Send the Invoke message.

(WebKit::NPObjectProxy::NP_HasMethod):
Call hasMethod.

(WebKit::NPObjectProxy::NP_Invoke):
Call invoke.

LayoutTests:

Remove now passing tests.

  • platform/mac-wk2/Skipped:
4:12 PM Changeset in webkit [71182] by Simon Fraser
  • 3 edits
    6 adds in trunk

2010-11-02 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Iframes nested inside a compositing layer don't repaint correctly
https://bugs.webkit.org/show_bug.cgi?id=48880
<rdar://problem/8194698>

The isEnclosedInCompositingLayer() is used to modify the behavior of
-[WebClipView visibleRect:] in WebKit, so that scrolling-related repaints
are correct in composited iframes. Previously it only asked whether the
frame's renderer was in a compositing layer, but we actually need to
consult all ancestors.

Test: compositing/iframes/nested-iframe-scrolling.html

  • page/FrameView.cpp: (WebCore::FrameView::isEnclosedInCompositingLayer):
4:02 PM Changeset in webkit [71181] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.

  • platform/chromium/test_expectations.txt:
3:48 PM Changeset in webkit [71180] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Fix build.

  • TestWebKitAPI/PlatformUtilities.cpp:

(TestWebKitAPI::Util::createContextForInjectedBundleTest):

3:35 PM Changeset in webkit [71179] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-11-02 Marc-Antoine Ruel <maruel@chromium.org>

Reviewed by Dimitri Glazkov.

Disable incremental linking for webkit_unit_test and DumpRenderTRee on x86 Windows on chromium build.
https://bugs.webkit.org/show_bug.cgi?id=48836

It fails to link otherwise due to lack of virtual address space.

  • WebKit.gyp:
3:28 PM Changeset in webkit [71178] by andersca@apple.com
  • 8 edits in trunk

Implement NP_HasProperty for NPObjectProxy
https://bugs.webkit.org/show_bug.cgi?id=48876

Reviewed by Sam Weinig.

WebKit2:

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::hasProperty):
Call The hasProperty callback function.

  • Shared/Plugins/NPObjectMessageReceiver.messages.in:

Add HasProperty message.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::hasProperty):
Send the HasProperty message.

(WebKit::NPObjectProxy::NP_HasProperty):
Call hasProperty.

LayoutTests:

Remove now passing test.

  • platform/mac-wk2/Skipped:
3:21 PM Changeset in webkit [71177] by hyatt@apple.com
  • 13 edits
    4 adds in trunk

https://bugs.webkit.org/show_bug.cgi?id=48672

Reviewed by Dan Bernstein.

Make vertical glyph offset hit testing work with inline boxes. This allows selection to paint
properly for vertical text (although gap-filling is still broken).

Refactored and cleaned up the writing mode flipping functions, and also did some cleanup to
better share code between hit testing and painting.

WebCore:

  • rendering/InlineBox.cpp:

(WebCore::InlineBox::locationIncludingFlipping):
(WebCore::InlineBox::flipForWritingMode):

  • rendering/InlineBox.h:
  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::nodeAtPoint):
(WebCore::InlineFlowBox::paint):
(WebCore::InlineFlowBox::paintBoxDecorations):
(WebCore::InlineFlowBox::paintMask):

  • rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::selectionRect):
(WebCore::InlineTextBox::nodeAtPoint):
(WebCore::InlineTextBox::paint):
(WebCore::InlineTextBox::textPos):
(WebCore::InlineTextBox::offsetForPosition):
(WebCore::InlineTextBox::positionForOffset):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintChildren):
(WebCore::RenderBlock::paintFloats):
(WebCore::RenderBlock::hitTestFloats):
(WebCore::RenderBlock::hitTestContents):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computeRectForRepaint):
(WebCore::RenderBox::flipForWritingMode):
(WebCore::RenderBox::locationOffsetIncludingFlipping):

  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp:

(WebCore::RenderInline::clippedOverflowRectForRepaint):

  • rendering/RenderLineBoxList.cpp:

(WebCore::RenderLineBoxList::rangeIntersectsRect):
(WebCore::RenderLineBoxList::anyLineIntersectsRect):
(WebCore::RenderLineBoxList::lineIntersectsDirtyRect):
(WebCore::RenderLineBoxList::paint):
(WebCore::RenderLineBoxList::hitTest):

  • rendering/RenderLineBoxList.h:
  • rendering/RenderText.cpp:

(WebCore::RenderText::positionForPoint):

LayoutTests:

  • fast/repaint/selection-rl.html: Added.
  • platform/mac/fast/repaint/selection-rl-expected.checksum: Added.
  • platform/mac/fast/repaint/selection-rl-expected.png: Added.
  • platform/mac/fast/repaint/selection-rl-expected.txt: Added.
3:15 PM Changeset in webkit [71176] by inferno@chromium.org
  • 2 edits
    5 copies in branches/chromium/552

Merge 71170

BUG=59554

3:06 PM Changeset in webkit [71175] by weinig@apple.com
  • 21 edits in trunk

Add a way to send startup messages on the context which can be posted when a process launches
<rdar://problem/8617928>
https://bugs.webkit.org/show_bug.cgi?id=48838

Reviewed by Anders Carlsson.

WebKit2:

Adds WKContextSetInitializationUserDataForInjectedBundle which sets an
APIObject on WebContext to be sent to the InjectedBundle's WKBundleInitialize
function each time a process starts up. This can help in cases where
the WebProcess dies, and will be helpful in the world where there are
multiple WebProcess's per context.

  • Scripts/webkit2/messages.py:
  • UIProcess/API/C/WKContext.cpp:

(WKContextSetInitializationUserDataForInjectedBundle):

  • UIProcess/API/C/WKContext.h:
  • UIProcess/WebContext.cpp:

(WebKit::WebContext::ensureWebProcess):

  • UIProcess/WebContext.h:

(WebKit::WebContext::setInjectedBundleInitializationUserData):
(WebKit::WebContext::injectedBundleInitializationUserData):

  • WebProcess/InjectedBundle/API/c/WKBundleInitialize.h:
  • WebProcess/InjectedBundle/InjectedBundle.h:
  • WebProcess/InjectedBundle/mac/InjectedBundleMac.cpp:

(WebKit::InjectedBundle::load):

  • WebProcess/InjectedBundle/qt/InjectedBundleQt.cpp:

(WebKit::InjectedBundle::load):

  • WebProcess/InjectedBundle/win/InjectedBundleWin.cpp:

(WebKit::InjectedBundle::load):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::initializeWebProcess):

  • WebProcess/WebProcess.h:
  • WebProcess/WebProcess.messages.in:

WebKitTools:

  • MiniBrowser/mac/WebBundle/WebBundleMain.m:

(WKBundleInitialize):

  • TestWebKitAPI/InjectedBundleController.cpp:

(TestWebKitAPI::InjectedBundleController::initialize):
(TestWebKitAPI::InjectedBundleController::didReceiveMessage):

  • TestWebKitAPI/InjectedBundleController.h:
  • TestWebKitAPI/InjectedBundleMain.cpp:

(WKBundleInitialize):

  • TestWebKitAPI/PlatformUtilities.cpp:

(TestWebKitAPI::Util::createContextForInjectedBundleTest):

  • WebKitTestRunner/InjectedBundle/InjectedBundleMain.cpp:

(WKBundleInitialize):
Update implementations of WKBundleInitialize to take an initial userData
argument. Change TestWebKitAPI to use the new initial userData to initialize
each test's bundle.

3:01 PM Changeset in webkit [71174] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

JPEG decoders should understand color profiles
https://bugs.webkit.org/show_bug.cgi?id=48819

This patch is currently a no-op because no one defines USE(ICCJPEG).
We'll enable this for Chromium Mac once we have ICCJPEG landed in
Chromium's third_party directory.

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::readColorProfile): (WebCore::JPEGImageReader::JPEGImageReader): (WebCore::JPEGImageReader::decode): (WebCore::JPEGImageDecoder::outputScanlines):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h: (WebCore::JPEGImageDecoder::setColorProfile):
2:54 PM Changeset in webkit [71173] by andersca@apple.com
  • 5 edits in trunk/WebKit2

Implement PluginProxy::pluginScriptableNPObject
https://bugs.webkit.org/show_bug.cgi?id=48874

Reviewed by Sam Weinig.

  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::getPluginScriptableNPObject):
Ask the plug-in for its scriptable NPObject, register it with the map and return its object ID.

  • PluginProcess/PluginControllerProxy.messages.in:

Add GetPluginScriptableNPObject message.

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::pluginScriptableNPObject):
Send the GetPluginScriptableNPObject message.

2:35 PM Changeset in webkit [71172] by Patrick Gansterer
  • 2 edits in trunk/WebKit/wince

2010-11-02 Patrick Gansterer <Patrick Gansterer>

Unreviewed, build fix after r71041.

  • WebCoreSupport/ChromeClientWinCE.h: (WebKit::ChromeClientWinCE::showContextMenu):
2:31 PM WebKit Team edited by leviw@google.com
Updating my company to Google. (diff)
2:26 PM Changeset in webkit [71171] by andersca@apple.com
  • 4 edits in trunk/WebKit2

Add strings to NPVariantData
https://bugs.webkit.org/show_bug.cgi?id=48871

Reviewed by Dan Bernstein.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::npVariantToNPVariantData):
(WebKit::NPRemoteObjectMap::npVariantDataToNPVariant):

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::makeString):
(WebKit::NPVariantData::encode):
(WebKit::NPVariantData::decode):

  • Shared/Plugins/NPVariantData.h:

(WebKit::NPVariantData::stringValue):

2:23 PM Changeset in webkit [71170] by mihaip@chromium.org
  • 4 edits
    5 adds in trunk

2010-11-02 Mihai Parparita <mihaip@chromium.org>

Reviewed by Adam Barth.

[Chromium] Crash when encountering history.back() call during Page::goToItem execution
https://bugs.webkit.org/show_bug.cgi?id=48477

Add a reduced version of the page that was originally reported at
http://crbug.com/59554.

  • http/tests/history/back-during-onload-triggered-by-back-expected.txt: Added.
  • http/tests/history/back-during-onload-triggered-by-back.html: Added.
  • http/tests/history/resources/back-during-onload-container.html: Added.
  • http/tests/history/resources/back-during-onload-hung-page.php: Added.
  • http/tests/history/resources/back-during-onload-middle.html: Added.

2010-11-02 Mihai Parparita <mihaip@chromium.org>

Reviewed by Adam Barth.

[Chromium] Crash when encountering history.back() call during Page::goToItem execution
https://bugs.webkit.org/show_bug.cgi?id=48477

For the Chromium port, BackForwardList::itemAtIndex synthesizes a
HistoryItem and saves a pointer to it in m_pendingItem. During
Page::goToItem we call FrameLoader::stopAllLoaders, which can trigger
onload handlers (if a subframe was not considered committed by the frame
loader). If one of those handlers calls calls history.back() or another
operation that ends up in NavigationScheduler::scheduleHistoryNavigation,
we would call BackForwardList::itemAtIndex, which means that we would
lose the m_pendingItem RefPtr that pointed to the item being navigated
to, causing its ref count to go to 0*, and thus for the HistoryItem to
be deleted before we were done navigating to it.

This is fixed in two ways:

  • Add a protector RefPtr in Page::goToItem to make sure that the item is still around for when we pass it to HistoryController:goToItem.
  • Change NavigationScheduler::scheduleHistoryNavigation to not use BackForwardList::itemAtIndex and instead look at the forward/backListCount() (since it doesn't actually care about the returned HistoryItem).

Test: http/tests/history/back-during-onload-triggered-by-back.html

  • loader/NavigationScheduler.cpp: (WebCore::NavigationScheduler::scheduleHistoryNavigation):
  • page/Page.cpp: (WebCore::Page::goToItem):
2:18 PM Changeset in webkit [71169] by andersca@apple.com
  • 7 edits in trunk/WebKit2

Add convenience functions for allocating NPAPI memory
https://bugs.webkit.org/show_bug.cgi?id=48870

Reviewed by Dan Bernstein.

Add npnMemAlloc, npnMemFree and createNPString functions and
npnMemNew, npnMemNewArray function templates.

  • WebProcess/Plugins/Netscape/JSNPObject.cpp:

(WebKit::JSNPObject::getOwnPropertyNames):
Use npnMemFree.

  • WebProcess/Plugins/Netscape/NPJSObject.cpp:

(WebKit::NPJSObject::enumerate):
Use npnMemNewArray.

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

(WebKit::NPRuntimeObjectMap::convertJSValueToNPVariant):
Use createNPString.

  • WebProcess/Plugins/Netscape/NPRuntimeUtilities.cpp:

(WebKit::npnMemAlloc):
Call malloc.

(WebKit::npnMemFree):
Call free.

(WebKit::createNPString):
Allocate the characters for a new NPString.

(WebKit::createNPObject):
Call npnMemNew.

(WebKit::deallocateNPObject):
Call npnMemFree.

(WebKit::releaseNPVariantValue):
Call npnMemFree.

  • WebProcess/Plugins/Netscape/NPRuntimeUtilities.h:

(WebKit::npnMemNew):
(WebKit::npnMemNewArray):
Add new functions.

  • WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp:

(WebKit::NPN_MemAlloc):
Call npnMemAlloc.

(WebKit::NPN_MemFree):
Call npnMemFree.

(WebKit::NPN_UTF8FromIdentifier):
Call npnMemNewArray.

(WebKit::copyCString):
Call npnMemNewArray. Correctly initialize the "len" out parameter.

2:14 PM Changeset in webkit [71168] by jberlin@webkit.org
  • 2 edits in trunk/WebKit2

Windows build fix. Unreviewed.

  • win/WebKit2Generated.make:

Add WKInspector.h so that it is copied over to the WebKitBuild directory.

2:13 PM Changeset in webkit [71167] by dimich@chromium.org
  • 1 edit
    1 move
    1 add in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Move the expected file higher up so it's used for all chromium flavors.

  • platform/chromium/security/block-test-no-port-expected.txt: Renamed from LayoutTests/platform/chromium-mac/security/block-test-no-port-expected.txt.
2:05 PM Changeset in webkit [71166] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

2010-11-02 Jenn Braithwaite <jennb@chromium.org>

Reviewed by Andreas Kling.

fast/frames/iframe-reparenting-new-page.html output is ordered wrong
https://bugs.webkit.org/show_bug.cgi?id=48807

  • fast/frames/iframe-reparenting-new-page-expected.txt:
  • fast/frames/script-tests/iframe-reparenting-new-page.js: (finish):
2:02 PM Changeset in webkit [71165] by beidson@apple.com
  • 2 edits in trunk/WebKit/win

<rdar://problem/8346191> and https://bugs.webkit.org/show_bug.cgi?id=48868
Implement IMutableWebRequest::setTimeoutInterval

Reviewed by Anders Carlsson.

  • WebMutableURLRequest.cpp:

(WebMutableURLRequest::setTimeoutInterval):

2:01 PM Changeset in webkit [71164] by zmo@google.com
  • 5 edits in trunk

2010-10-28 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Update getUniform for bvec[234] to return boolean[] rather than Uint8Array
https://bugs.webkit.org/show_bug.cgi?id=47568

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::getUniform):

2010-10-28 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Update getUniform for bvec[234] to return boolean[] rather than Uint8Array
https://bugs.webkit.org/show_bug.cgi?id=47568

  • fast/canvas/webgl/gl-object-get-calls-expected.txt:
  • fast/canvas/webgl/gl-object-get-calls.html:
1:53 PM Changeset in webkit [71163] by crogers@google.com
  • 1 edit
    3 adds in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Add RealtimeAnalyserNode files
https://bugs.webkit.org/show_bug.cgi?id=48798

No new tests since audio API is not yet implemented.

  • webaudio/RealtimeAnalyserNode.cpp: Added. (WebCore::RealtimeAnalyserNode::RealtimeAnalyserNode): (WebCore::RealtimeAnalyserNode::~RealtimeAnalyserNode): (WebCore::RealtimeAnalyserNode::process): (WebCore::RealtimeAnalyserNode::pullInputs): (WebCore::RealtimeAnalyserNode::reset):
  • webaudio/RealtimeAnalyserNode.h: Added. (WebCore::RealtimeAnalyserNode::create): (WebCore::RealtimeAnalyserNode::fftSize): (WebCore::RealtimeAnalyserNode::setFftSize): (WebCore::RealtimeAnalyserNode::frequencyBinCount): (WebCore::RealtimeAnalyserNode::setMinDecibels): (WebCore::RealtimeAnalyserNode::minDecibels): (WebCore::RealtimeAnalyserNode::setMaxDecibels): (WebCore::RealtimeAnalyserNode::maxDecibels): (WebCore::RealtimeAnalyserNode::setSmoothingTimeConstant): (WebCore::RealtimeAnalyserNode::smoothingTimeConstant): (WebCore::RealtimeAnalyserNode::getFloatFrequencyData): (WebCore::RealtimeAnalyserNode::getByteFrequencyData): (WebCore::RealtimeAnalyserNode::getByteTimeDomainData):
  • webaudio/RealtimeAnalyserNode.idl: Added.
1:48 PM Changeset in webkit [71162] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

2010-11-02 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r71152.
http://trac.webkit.org/changeset/71152
https://bugs.webkit.org/show_bug.cgi?id=48866

Causes lots and lots of crashes (Requested by jamesr_ on
#webkit).

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::JPEGImageReader::JPEGImageReader): (WebCore::JPEGImageReader::decode): (WebCore::JPEGImageDecoder::outputScanlines):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h:
1:47 PM Changeset in webkit [71161] by cmarrin@apple.com
  • 3 edits
    2 adds in trunk

2010-11-02 Chris Marrin <cmarrin@apple.com>

Reviewed by Simon Fraser.

When animations are paused, play-state can cause them to be unpaused.
https://bugs.webkit.org/show_bug.cgi?id=46525

I Changed updatePlayState() to only unpause when both play-state is
"running" and not suspended. Likewise I pause animation when either
play-state is "paused" or suspended.

Test: animations/play-state-suspend.html

  • page/animation/AnimationBase.cpp: (WebCore::AnimationBase::updatePlayState):
1:42 PM Changeset in webkit [71160] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-02 Benjamin Kalman <kalman@google.com>

Reviewed by Ojan Vafai.

new-run-webkit-tests doesn't strip "LayoutTests/" from prefix, unlike old-run-webkit-tests
https://bugs.webkit.org/show_bug.cgi?id=48794

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py: Strip the "LayoutTests/" prefix from test argument paths.
1:34 PM Changeset in webkit [71159] by ap@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=48813
<rdar://problem/8602552> REGRESSION (r63622): DNS prefetching for <link> tags doesn't work
in non-HTTP documents

Cannot test DNS prefetching.

  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process): We should be looking at preferences, not at whether this document has prefetching enabled.
1:30 PM Changeset in webkit [71158] by commit-queue@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-11-02 Ilya Sherman <isherman@chromium.org>

Reviewed by Kent Tamura.

Expose the sendChangeEvent parameter in WebInputElement::setValue() API,
primarily so that chromium form autofill can fire the onChange event.
https://bugs.webkit.org/show_bug.cgi?id=48177

  • public/WebInputElement.h:
  • src/WebInputElement.cpp: (WebKit::WebInputElement::setValue):
1:28 PM Changeset in webkit [71157] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Intentional error result to narrow down a regression caused by http://codereview.chromium.org/3976004

  • platform/chromium-mac/security/block-test-no-port-expected.txt:
1:24 PM Changeset in webkit [71156] by andersca@apple.com
  • 4 edits in trunk/WebKit2

Add some support for remote and local NPObjects to NPRemoteObjectMap
https://bugs.webkit.org/show_bug.cgi?id=48864

Reviewed by Adam Roben.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::npVariantToNPVariantData):
If the NPVariant contains an NPObject that isn't an NPObjectProxy, register it
and return its ID.

(WebKit::NPRemoteObjectMap::npVariantDataToNPVariant):
If the NPVariantData contains a remote object, create an NPObjectProxy and return it.

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::NPVariantData):
Initialize m_localNPObjectIDValue and m_remoteNPObjectIDValue.

(WebKit::NPVariantData::makeLocalNPObjectID):
Create an NPVariantData that contains a local NPObject ID.

(WebKit::NPVariantData::encode):
Handle local and remote NPObject IDs.

(WebKit::NPVariantData::decode):
Decode local and remote NPObject IDs and "swap them" so an encoded remote NPObject ID is
decoded as a local NPObjectID (and vice versa).

  • Shared/Plugins/NPVariantData.h:

(WebKit::NPVariantData::localNPObjectIDValue):
(WebKit::NPVariantData::remoteNPObjectIDValue):

12:52 PM Changeset in webkit [71155] by Adam Roben
  • 3 edits
    1 add in trunk/JavaScriptCore

Windows build fix after r71127

MSVC isn't smart enough to figure out that the definition of the global
nullptr variable isn't needed, so we provide one for it.

Fixes <http://webkit.org/b/48862> Windows build is broken due to
undefined symbol nullptr

Reviewed by Anders Carlsson.

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Export nullptr.

resort the files.

  • wtf/NullPtr.cpp: Added.
12:51 PM Changeset in webkit [71154] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: Web Inspector: follow up to r71139. Fixed Mac monospace
height style.

  • inspector/front-end/PropertiesSection.js: (WebInspector.PropertiesSection):
  • inspector/front-end/inspector.css:
12:51 PM Changeset in webkit [71153] by abarth@webkit.org
  • 1 edit
    2 copies in branches/chromium/552

Merge 71147 - 2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

querySelectorAll('body>:last-child') does not work unless HTML file ends with 'new-line'.
https://bugs.webkit.org/show_bug.cgi?id=47166

We need to pop all the open elements when we end the document in the
AfterAfterBody state.

Test: fast/parser/pop-all-after-after-body.html

  • html/parser/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndOfFile):

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

querySelectorAll('body>:last-child') does not work unless HTML file ends with 'new-line'.
https://bugs.webkit.org/show_bug.cgi?id=47166

Test that :last-child works when we end the document in the
AfterAfterBody state.

  • fast/parser/pop-all-after-after-body-expected.txt: Added.
  • fast/parser/pop-all-after-after-body.html: Added.

TBR=abarth@webkit.org

12:45 PM Changeset in webkit [71152] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

JPEG decoders should understand color profiles
https://bugs.webkit.org/show_bug.cgi?id=48819

This patch is currently a no-op because no one defines USE(ICCJPEG).
We'll enable this for Chromium Mac once we have ICCJPEG landed in
Chromium's third_party directory.

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (WebCore::readColorProfile): (WebCore::JPEGImageReader::JPEGImageReader): (WebCore::JPEGImageReader::decode): (WebCore::JPEGImageDecoder::outputScanlines):
  • platform/image-decoders/jpeg/JPEGImageDecoder.h: (WebCore::JPEGImageDecoder::setColorProfile):
12:39 PM Changeset in webkit [71151] by jamesr@google.com
  • 153 edits
    152 copies
    3 adds in trunk/LayoutTests

2010-11-02 James Robinson <jamesr@chromium.org>

Update pixel baselines for Snow Leopard for tests in fast/text/*

12:37 PM Changeset in webkit [71150] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

2010-11-02 Ilya Sherman <isherman@chromium.org>

Reviewed by Simon Fraser.

Marks selectionStart() and selectionEnd() as const.
https://bugs.webkit.org/show_bug.cgi?id=48786

  • html/HTMLFormControlElement.cpp: (WebCore::HTMLTextFormControlElement::selectionStart): (WebCore::HTMLTextFormControlElement::selectionEnd):
  • html/HTMLFormControlElement.h:
12:29 PM Changeset in webkit [71149] by crogers@google.com
  • 13 edits in trunk/WebCore

2010-11-02 Chris Rogers <crogers@google.com>

Reviewed by Kenneth Russell.

Fix license for audio files
https://bugs.webkit.org/show_bug.cgi?id=48859

  • webaudio/AudioChannelSplitter.cpp:
  • webaudio/AudioChannelSplitter.h:
  • webaudio/AudioDestinationNode.cpp:
  • webaudio/AudioDestinationNode.h:
  • webaudio/AudioGainNode.cpp:
  • webaudio/AudioGainNode.h:
  • webaudio/AudioPannerNode.cpp:
  • webaudio/AudioPannerNode.h:
  • webaudio/AudioPannerNode.idl:
12:20 PM Changeset in webkit [71148] by Martin Robinson
  • 7 edits in trunk

2010-11-02 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

Remove special handling of HashTableDeletedValue in PlatformRefPtr and manually manage memory that cannot be controlled by HashTraits
https://bugs.webkit.org/show_bug.cgi?id=48841

Remove special handling of HashTableDeletedValue in PlatformRefPtr.
This is better handled on a case-by-case basis, when HashTraits
cannot account for it.

  • wtf/PlatformRefPtr.h: (WTF::PlatformRefPtr::~PlatformRefPtr): (WTF::PlatformRefPtr::clear): (WTF::::operator):

2010-11-02 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

Remove special handling of HashTableDeletedValue in PlatformRefPtr and manually manage memory that cannot be controlled by HashTraits
https://bugs.webkit.org/show_bug.cgi?id=48841

Switch to manually managing the memory in FontPlatformDataFreeType. This
is necessary because smart pointers do not know how to deal with a pointer
value of -1 (HashTableDeletedValue) and HashTraits can only manage the type
contained in the HashMap.

No new tests as this should not change functionality.

  • platform/graphics/cairo/FontPlatformDataFreeType.cpp: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::operator=): (WebCore::FontPlatformData::~FontPlatformData): (WebCore::FontPlatformData::operator==): (WebCore::FontPlatformData::initializeWithFontFace):
  • platform/graphics/cairo/FontPlatformDataFreeType.h: (WebCore::FontPlatformData::FontPlatformData): (WebCore::FontPlatformData::scaledFont): (WebCore::FontPlatformData::hash): (WebCore::FontPlatformData::isHashTableDeletedValue): (WebCore::FontPlatformData::hashTableDeletedFontValue):
  • platform/graphics/cairo/GlyphPageTreeNodeCairo.cpp: (WebCore::GlyphPage::fill):
  • platform/graphics/cairo/SimpleFontDataCairo.cpp: (WebCore::SimpleFontData::platformInit): (WebCore::SimpleFontData::containsCharacters): (WebCore::SimpleFontData::platformWidthForGlyph):
12:15 PM Changeset in webkit [71147] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

querySelectorAll('body>:last-child') does not work unless HTML file ends with 'new-line'.
https://bugs.webkit.org/show_bug.cgi?id=47166

We need to pop all the open elements when we end the document in the
AfterAfterBody state.

Test: fast/parser/pop-all-after-after-body.html

  • html/parser/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndOfFile):

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

querySelectorAll('body>:last-child') does not work unless HTML file ends with 'new-line'.
https://bugs.webkit.org/show_bug.cgi?id=47166

Test that :last-child works when we end the document in the
AfterAfterBody state.

  • fast/parser/pop-all-after-after-body-expected.txt: Added.
  • fast/parser/pop-all-after-after-body.html: Added.
12:08 PM Changeset in webkit [71146] by Adam Roben
  • 2 edits
    2 adds in trunk/WebKitTools

Skip webkitpy.layout_tests.run_webkit_tests_unittest.MainTest on Cygwin Python 2.5.x

It is known to hang on that version of Python. See
<http://webkit.org/b/48614>.

Reviewed by Adam Barth.

  • Scripts/webkitpy/layout_tests/run_webkit_tests_unittest.py: Use

skip_if to skip MainTest on Cygwin Python 2.5.x.

  • Scripts/webkitpy/test/skip.py: Added.

(skip_if): If the passed-in condition is false, find all the test_*
methods of the given class and replace them with a function that just
logs that we're skipping these tests. This is loosely based on the
unittest.skip_if decorator added in Python 3.1.
(_skipped_method): Returns a function that just logs that the tests are
being skipped. This is used in place of the actual test_* functions.

  • Scripts/webkitpy/test/skip_unittest.py: Added.

(SkipTest.setUp): Create a logger for ourselves and give it a handler
that logs to a private stream.
(SkipTest.tearDown): Reset the logger.
(SkipTest.create_fixture_class): Returns a class that we use to test
skip_if. It has a single test_* method, test_foo, that just calls
through to a callback.
(SkipTest.foo_callback): Record that test_foo was called.
(SkipTest.test_skip_if_false): Pass skip_if a False condition and test
that test_foo does get called.
(SkipTest.test_skip_if_true): Pass skip_if a True condition and test
that test_foo does not get called and the appropriate message gets
logged.

12:07 PM Changeset in webkit [71145] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-11-02 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Roben.

Cleanup createGlobalImageFileDescriptor in ClipboardWin
https://bugs.webkit.org/show_bug.cgi?id=48189

  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::writeURL):
12:00 PM Changeset in webkit [71144] by pfeldman@chromium.org
  • 7 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: display frame names in the resources panel.
https://bugs.webkit.org/show_bug.cgi?id=48839

  • inspector/Inspector.idl:
  • inspector/InspectorResourceAgent.cpp: (WebCore::frameId): (WebCore::buildObjectForDocumentLoader): (WebCore::buildObjectForFrame): (WebCore::buildObjectForFrameTree): (WebCore::InspectorResourceAgent::didCommitLoad): (WebCore::InspectorResourceAgent::frameDetachedFromParent): (WebCore::InspectorResourceAgent::frameForId): (WebCore::InspectorResourceAgent::resourceContent):
  • inspector/InspectorResourceAgent.h:
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceManager.prototype.didCommitLoadForFrame): (WebInspector.ResourceManager.prototype._processCachedResources): (WebInspector.ResourceManager.prototype._addFramesRecursively): (WebInspector.ResourceTreeModel.prototype.addOrUpdateFrame): (WebInspector.ResourceTreeModel.prototype.didCommitLoadForFrame):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel.prototype.addOrUpdateFrame): (WebInspector.BaseStorageTreeElement.prototype.get titleText): (WebInspector.FrameTreeElement): (WebInspector.FrameTreeElement.prototype.onattach): (WebInspector.FrameTreeElement.prototype.get nameForSorting): (WebInspector.FrameTreeElement.prototype.setTitles): (WebInspector.FrameTreeElement.prototype.set hovered):
  • inspector/front-end/inspector.css: (li.selected .base-storage-tree-element-subtitle): (.base-storage-tree-element-subtitle):
11:50 AM Changeset in webkit [71143] by pfeldman@chromium.org
  • 3 edits
    2 deletes in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Follow up to r71139: removing unused images.
https://bugs.webkit.org/show_bug.cgi?id=48827

  • WebCore.gypi:
  • inspector/front-end/Images/grayConnectorPoint.png: Removed.
  • inspector/front-end/Images/whiteConnectorPoint.png: Removed.
  • inspector/front-end/WebKit.qrc:
11:48 AM Changeset in webkit [71142] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2010-11-02 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

webkit-patch should tell check-webkit-style which files were changed so
check-webkit-style doesn't have to stat the whole working copy again
https://bugs.webkit.org/show_bug.cgi?id=48792

  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/checkstyle.py:
11:40 AM Changeset in webkit [71141] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Attempt to fix a GCC release only warning.

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::NPVariantData):

11:39 AM Changeset in webkit [71140] by andersca@apple.com
  • 6 edits in trunk/WebKit2

Invalidate the NPRemoteObjectMap
https://bugs.webkit.org/show_bug.cgi?id=48856

Reviewed by John Sullivan.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::~NPObjectMessageReceiver):
Remove FIXME. The NPRemoteObjectMap always outlives the NPObjectMessageReceiver.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::~NPObjectProxy):
Call npObjectProxyDestroyed here.

(WebKit::NPObjectProxy::invalidate):
Null out m_npRemoteObjectMap.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::~NPRemoteObjectMap):
Add assertions.

(WebKit::NPRemoteObjectMap::createNPObjectProxy):
Keep track of the NPObjectProxy by adding it to the m_npObjectProxies set.

(WebKit::NPRemoteObjectMap::npObjectProxyDestroyed):
Remove the NPObjectProxy from the m_npObjectProxies set.

(WebKit::NPRemoteObjectMap::invalidate):
Delete all NPObjectMessageReceivers. Iterate over all NPObjectProxy objects and invalidate them.

11:31 AM Changeset in webkit [71139] by pfeldman@chromium.org
  • 6 edits in trunk

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make properties and event listeners look consistent with the styles section.
https://bugs.webkit.org/show_bug.cgi?id=48827

  • inspector/front-end/EventListenersSidebarPane.js:
  • inspector/front-end/PropertiesSection.js: (WebInspector.PropertiesSection):
  • inspector/front-end/PropertiesSidebarPane.js: (WebInspector.PropertiesSidebarPane.prototype.update.callback): (WebInspector.PropertiesSidebarPane.prototype.update):
  • inspector/front-end/inspector.css: (.events-pane .section:not(:nth-of-type(1))): (.event-bar:first-child): (.section .header): (.section .header::before): (.section.expanded .header::before): (.section .header .title, .event-bar .header .title): (.section .header .subtitle, .event-bar .header .subtitle): (.section.expanded .properties, .event-bar.expanded .event-properties): (.properties-tree): (.event-bar): (.event-bars .event-bar .header): (.event-bars .event-bar .header .title): (.event-bars .event-bar .header::before):

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: make properties and event listeners look consistent with the styles section.
https://bugs.webkit.org/show_bug.cgi?id=48827

  • src/js/devTools.css:
11:20 AM Changeset in webkit [71138] by mitz@apple.com
  • 3 edits in trunk/WebCore

Remove unused methods and member variables from AutoTableLayout.

Reviewed by Anders Carlsson.

  • rendering/AutoTableLayout.cpp:

(WebCore::AutoTableLayout::AutoTableLayout): Removed initialization of m_percentagesDirty
and m_totalPercent.
(WebCore::AutoTableLayout::fullRecalc): Removed setting of m_percentagesDirty.
(WebCore::AutoTableLayout::calcPercentages): Removed.

  • rendering/AutoTableLayout.h: Removed totalPercent(), m_percentagesDirty, and m_totalPercent.
11:15 AM Changeset in webkit [71137] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-02 Robert Kroeger <rjkroege@chromium.org>

Reviewed by James Robinson.

[Chromium/DRT] Make EventSendingController honour leapForward for touch events.
https://bugs.webkit.org/show_bug.cgi?id=48777

  • DumpRenderTree/chromium/EventSender.cpp: (EventSender::sendCurrentTouchEvent):
11:12 AM Changeset in webkit [71136] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: clear / hide timing information for cached resources.
https://bugs.webkit.org/show_bug.cgi?id=48848

  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.set cached): (WebInspector.Resource.prototype.get timing): (WebInspector.Resource.prototype.set timing):
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceManager.prototype.markResourceAsCached):
11:11 AM Changeset in webkit [71135] by mthole@apple.com
  • 2 edits in trunk/WebKit2

WebProcessManager informed about crashes too late (pages may have already tried to load a new URL)
https://bugs.webkit.org/show_bug.cgi?id=48853

Reviewed by Anders Carlsson.

This also fixes: <rdar://problem/8610451> Webkit2: Crash in WebKit!WebKit::WebProcessProxy::sendMessage opening google.com page

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::~WebProcessProxy):
Removed invalid assertion that is hit when the WebProcess dies unexpectedly.
(WebKit::WebProcessProxy::didClose):
Reordered so that the individual pages are informed that the processDidCrash() after the shared
WebProcessManager has been. This allows the pages to properly trigger a new process to be launched.

11:10 AM Changeset in webkit [71134] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: improve network's 'preview close' usability.
https://bugs.webkit.org/show_bug.cgi?id=48846

  • inspector/front-end/networkPanel.css: (#network-close-button:hover): (#network-close-button:active):
11:08 AM Changeset in webkit [71133] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Image heights are displayed as 0.
https://bugs.webkit.org/show_bug.cgi?id=40817

  • inspector/front-end/ImageView.js: (WebInspector.ImageView.prototype.contentTabSelected): (WebInspector.ImageView.prototype.contentTabSelected.onImageLoad):
11:01 AM Changeset in webkit [71132] by andersca@apple.com
  • 4 edits in trunk/WebKit2

Handle boolean values in NPVariant and NPVariantData
https://bugs.webkit.org/show_bug.cgi?id=48852

Reviewed by John Sullivan.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::npVariantToNPVariantData):
(WebKit::NPRemoteObjectMap::npVariantDataToNPVariant):

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::makeBool):
(WebKit::NPVariantData::encode):
(WebKit::NPVariantData::decode):

  • Shared/Plugins/NPVariantData.h:

(WebKit::NPVariantData::boolValue):

10:59 AM Changeset in webkit [71131] by adele@apple.com
  • 3 edits
    2 adds in trunk

WebCore: Fix for https://bugs.webkit.org/show_bug.cgi?id=48814
<rdar://problem/8546143> Attempting to redo typing in apple.com/startpage search field causes infinite recursion in TextControlInnerTextElement::defaultEventHandler

Reviewed by Kent Tamura.

Test: editing/undo/redo-after-detach.html

  • rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerTextElement::defaultEventHandler):

A TextControlInnerTextElement will almost always have a shadowAncestorNode, the HTMLInputElement. In this case,
after the renderer was destroyed, this shadow node was kept alive by the EditCommand, even though its not hooked up
anymore to the shadow DOM. EditCommands can sometimes operate on stale selections and are expected to fail silently.
So here we prevent the infinite loop during event dispatch, and the rest of the redo operation will fail silently.

LayoutTests: Test for https://bugs.webkit.org/show_bug.cgi?id=48814
<rdar://problem/8546143> Attempting to redo typing in apple.com/startpage search field causes infinite recursion in TextControlInnerTextElement::defaultEventHandler

Reviewed by Kent Tamura.

  • editing/undo/redo-after-detach-expected.txt: Added.
  • editing/undo/redo-after-detach.html: Added.
10:55 AM Changeset in webkit [71130] by enrica@apple.com
  • 2 edits in trunk/WebKit2

WebKit2: Repeated cmd-key presses lead to unusable UI state
https://bugs.webkit.org/show_bug.cgi?id=48811
<rdar://problem/8611924>

Reviewed by Alexey Proskuryakov.

Given the asynchronous nature of every interaction with the WebProcess,
we could re-send a keyDown event to AppKit that maps to an
action that is currently disabled and therefore will not trigger an action
but a call to the keyDown method of the NSView. We need to detect this
situation to avoid an infinite loop.

  • UIProcess/API/mac/WKView.mm:

(-[WKView performKeyEquivalent:]): Delay the release of the last processed event
to the moment we receive another event.
(-[WKView keyDown:]): Added early return in case the event is one we have re-sent.

10:51 AM Changeset in webkit [71129] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Assertion hit in WebProcessProxy::~WebProcessProxy()
https://bugs.webkit.org/show_bug.cgi?id=48849
<rdar://problem/8617676>

Reviewed by John Sullivan.

It is perfectly valid for the page map to contain pages if the web process proxy
goes away, this can happen if the process crashes for example.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::~WebProcessProxy):
Remove the assertion.

10:35 AM Changeset in webkit [71128] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit2

2010-11-02 Christian Sejersen <Christian Sejersen>

Reviewed by Andreas Kling.

The previous patch only took into account arm targets. This one
now checks whether the build is done inside Scratchbox.
https://bugs.webkit.org/show_bug.cgi?id=47895

  • DerivedSources.pro:
10:26 AM Changeset in webkit [71127] by andersca@apple.com
  • 15 edits in trunk/WebKit2

More NPRuntime work
https://bugs.webkit.org/show_bug.cgi?id=48847

Reviewed by Adam Roben.

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::~WebProcessConnection):
Add assertions.

(WebKit::WebProcessConnection::removePluginControllerProxy):
If we have no more plug-in controller proxies, invalidate the remote object map.

  • Shared/Plugins/NPIdentifierData.cpp:

(WebKit::NPIdentifierData::createNPIdentifier):
Create an NPIdentifier from the given NPIdentifierData.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::create):
(WebKit::NPObjectMessageReceiver::NPObjectMessageReceiver):
This now takes the NPRemoteObjectMap as well as the npObjectID of the object.

(WebKit::NPObjectMessageReceiver::~NPObjectMessageReceiver):
Unregister the object.

(WebKit::NPObjectMessageReceiver::deallocate):
Delete the object.

(WebKit::NPObjectMessageReceiver::getProperty):
Ask the NPObject for the property and convert it back to an NPVariantData.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::~NPObjectProxy):
Send a Deallocate message to the corresponding message receiver on the other side.

(WebKit::NPObjectProxy::getProperty):
Convert the NPVariantData back to an NPVariant and return it.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::registerNPObject):
Pass the NPRemoteObjectMap and the npObjectID to NPObjectMessageReceiver::create.

(WebKit::NPRemoteObjectMap::unregisterNPObject):
Remove the given object from the map of registered objects.

(WebKit::NPRemoteObjectMap::npVariantToNPVariantData):
Given an NPVariant, create an NPVariantData. Only supports void and double types right now.

(WebKit::NPRemoteObjectMap::npVariantDataToNPVariant):
Given an NPVariantData, create an NPVariant. Only supports void and double types right now.

(WebKit::NPRemoteObjectMap::invalidate):
Add stub.

  • Shared/Plugins/NPVariantData.cpp:

(WebKit::NPVariantData::NPVariantData):
Set the type to void by default.

(WebKit::NPVariantData::makeVoid):
Return a void NPVariantData.

(WebKit::NPVariantData::makeDouble):
Return an NPVariantData that contains a double value.

(WebKit::NPVariantData::encode):
Encode the NPVariantData.

(WebKit::NPVariantData::decode):
Decode the NPVariantData.

  • Shared/Plugins/NPVariantData.h:

(WebKit::NPVariantData::type):
Return the type of the NPVariantData.

(WebKit::NPVariantData::doubleValue):
Return the double value of the NPVariantData.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::didClose):
Use nullptr instead of 0.

  • WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::~PluginProcessConnection):
Assert that the connection and the remote object map are both null.

(WebKit::PluginProcessConnection::removePluginProxy):
If this was the last plug-in proxy, invalidate the remote object map.

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::shutdown):
Use nullptr instead of 0.

10:25 AM Changeset in webkit [71126] by Adam Roben
  • 2 edits in trunk/LayoutTests

Skip another test that fails in WebKit2

  • platform/mac-wk2/Skipped:
10:19 AM Changeset in webkit [71125] by cmarrin@apple.com
  • 7 edits in trunk/WebCore

2010-11-02 Chris Marrin <cmarrin@apple.com>

Reviewed by Adam Roben.

Make RenderStyle::playState() return typed value and cleanup naming in Animation code
https://bugs.webkit.org/show_bug.cgi?id=48844

playState() function was returning unsigned value, but there is a EAnimPlayState
enum type which has the legal playState values. This type is now used everywhere.
I also changed the naming of the m_isSuspended to m_suspended to match the style
elsewhere in the code.

  • page/animation/AnimationBase.cpp: (WebCore::AnimationBase::updatePlayState):
  • page/animation/AnimationBase.h:
  • page/animation/AnimationController.cpp: (WebCore::AnimationControllerPrivate::clear): (WebCore::AnimationControllerPrivate::updateAnimationTimer):
  • page/animation/CompositeAnimation.cpp: (WebCore::CompositeAnimation::updateKeyframeAnimations): (WebCore::CompositeAnimation::suspendAnimations): (WebCore::CompositeAnimation::resumeAnimations):
  • page/animation/CompositeAnimation.h: (WebCore::CompositeAnimation::suspended): (WebCore::CompositeAnimation::CompositeAnimation):
  • platform/animation/Animation.h: (WebCore::Animation::playState): (WebCore::Animation::setPlayState): (WebCore::Animation::initialAnimationPlayState):
10:08 AM Changeset in webkit [71124] by Adam Roben
  • 13 edits in trunk/LayoutTests

Update Windows results after r71043

  • platform/win/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/win/fast/blockflow/border-vertical-lr-expected.png:
  • platform/win/fast/blockflow/border-vertical-lr-expected.txt:
  • platform/win/fast/blockflow/japanese-lr-text-expected.checksum:
  • platform/win/fast/blockflow/japanese-lr-text-expected.png:
  • platform/win/fast/blockflow/japanese-lr-text-expected.txt:
  • platform/win/fast/blockflow/japanese-rl-text-expected.checksum:
  • platform/win/fast/blockflow/japanese-rl-text-expected.png:
  • platform/win/fast/blockflow/japanese-rl-text-expected.txt:
  • platform/win/fast/text/international/vertical-text-glyph-test-expected.checksum:
  • platform/win/fast/text/international/vertical-text-glyph-test-expected.png:
  • platform/win/fast/text/international/vertical-text-glyph-test-expected.txt:
10:00 AM TriagingTestFailures edited by Adam Roben
More wordsmithing (diff)
9:46 AM TriagingTestFailures edited by Adam Roben
Reword the intro a bit (diff)
9:44 AM Changeset in webkit [71123] by zmo@google.com
  • 3 edits
    2 adds in trunk

2010-11-01 Zhenyao Mo <zmo@google.com>

Reviewed by Andreas Kling.

vertexAttribPointer should generate INVALID_VALUE if the stride exceeds 255
https://bugs.webkit.org/show_bug.cgi?id=48677

Test: fast/canvas/webgl/gl-vertexattribpointer.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::vertexAttribPointer): Generate INVALID_VALUE if stride > 255.

2010-11-01 Zhenyao Mo <zmo@google.com>

Reviewed by Andreas Kling.

vertexAttribPointer should generate INVALID_VALUE if the stride exceeds 255
https://bugs.webkit.org/show_bug.cgi?id=48677

  • fast/canvas/webgl/gl-vertexattribpointer-expected.txt: Added.
  • fast/canvas/webgl/gl-vertexattribpointer.html: Added.
9:44 AM TriagingTestFailures edited by Adam Roben
Flesh out the intro and add a note about migrating into Radar (diff)
9:39 AM Changeset in webkit [71122] by kbalazs@webkit.org
  • 2 edits in trunk/WebKit2

Unreviewed Qt buildfix.

message recievers.

9:38 AM TriagingTestFailures edited by Adam Roben
Address more of John Sullivan's feedback (diff)
9:35 AM TriagingTestFailures edited by Adam Roben
Address more of John Sullivan's feedback (diff)
9:32 AM TriagingTestFailures edited by Adam Roben
Address some of John Sullivan's feedback (diff)
9:25 AM TriagingTestFailures edited by Adam Roben
Fleshed out a little more (diff)
9:11 AM TriagingTestFailures edited by Adam Roben
Slightly more prose-y (diff)
9:10 AM TriagingTestFailures created by Adam Roben
Initial version
9:06 AM Changeset in webkit [71121] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-02 Stephen White <senorblanco@chromium.org>

Unreviewed; test expectations fix.

[Chromium] Update chromium-gpu test_expectations to reflect the
test results from the FYI bots.

  • platform/chromium-gpu/test_expectations.txt:
9:04 AM Changeset in webkit [71120] by dbates@webkit.org
  • 6 edits in trunk/WebKit

2010-11-02 Daniel Bates <dbates@rim.com>

Reviewed by Martin Robinson.

Set frame name before appending it to the frame tree in the Apple Windows,
GTK, and EFL ports
https://bugs.webkit.org/show_bug.cgi?id=48806

Make the frame creation process in the GTK-port consistent with the
Mac, Qt, and Haiku ports. In particular, set the name of the new
frame before it's appended to the frame tree.

At this time we cannot test this change since it is being masked by
HTMLFrameElementBase::setName() <http://trac.webkit.org/browser/trunk/WebCore/html/HTMLFrameElementBase.cpp?rev=70976#L160>.
We'll be able to test this once we fix bug #6751.

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::createFrame):

2010-11-02 Daniel Bates <dbates@rim.com>

Reviewed by Martin Robinson.

Set frame name before appending it to the frame tree in the Apple Windows,
GTK, and EFL ports
https://bugs.webkit.org/show_bug.cgi?id=48806

Make the frame creation process in the Apple Windows-port consistent
with the Mac, Qt, and Haiku ports. In particular, set the name of
the new frame before it's appended to the frame tree.

At this time we cannot test this change since it is being masked by
HTMLFrameElementBase::setName() <http://trac.webkit.org/browser/trunk/WebCore/html/HTMLFrameElementBase.cpp?rev=70976#L160>.
We'll be able to test this once we fix bug #6751.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::createFrame):

2010-11-02 Daniel Bates <dbates@rim.com>

Reviewed by Martin Robinson.

Set frame name before appending it to the frame tree in the Apple Windows,
GTK, and EFL ports
https://bugs.webkit.org/show_bug.cgi?id=48806

Make the frame creation process in the EFL-port consistent with the
Mac, Qt, and Haiku ports. In particular, set the name of the new
frame before it's appended to the frame tree.

At this time we cannot test this change since it is being masked by
HTMLFrameElementBase::setName() <http://trac.webkit.org/browser/trunk/WebCore/html/HTMLFrameElementBase.cpp?rev=70976#L160>.
We'll be able to test this once we fix bug #6751.

  • ewk/ewk_frame.cpp: (ewk_frame_child_add):
8:52 AM Changeset in webkit [71119] by Martin Robinson
  • 3 edits in trunk/WebCore

2010-11-01 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[Soup] Random crashes in http/tests/websocket/tests/workers/worker-handshake-challenge-randomness.html
https://bugs.webkit.org/show_bug.cgi?id=48805

Track active WebSocket handles via a sequential id. This ensures
that when a handle is reallocated into a recently used segment of
memory, it doesn't trigger a false positive in the code which ensures
the original handle is active.

No new tests. This test should stop crashing on the bots, proving the fix.

  • platform/network/soup/SocketStreamHandle.h: (WebCore::SocketStreamHandle::id): Added an m_id member and accessor to SocketStreamHandle.
  • platform/network/soup/SocketStreamHandleSoup.cpp: (WebCore::getHandleFromId): Updated to work with HashMap of handle ids to SocketStreamHandle*. (WebCore::deactivateHandle): Ditto. (WebCore::activateHandle): Ditto. (WebCore::SocketStreamHandle::SocketStreamHandle): Ditto. (WebCore::SocketStreamHandle::connected): Ditto. (WebCore::SocketStreamHandle::readBytes): Ditto. (WebCore::SocketStreamHandle::beginWaitingForSocketWritability): Ditto. (WebCore::connectedCallback): Ditto. (WebCore::readReadyCallback): Ditto. (WebCore::writeReadyCallback): Ditto.
8:47 AM Changeset in webkit [71118] by kbalazs@webkit.org
  • 3 edits in trunk/WebKit2

[Qt][WK2] SharedMemory is broken
https://bugs.webkit.org/show_bug.cgi?id=48520

Reviewed by Andreas Kling.

Implement SharedMemory with QSharedMemory to correctly handle
ownership and to avoid extra allocation for page size requests.

  • Platform/SharedMemory.h:
  • Platform/qt/SharedMemoryQt.cpp:

(WebKit::SharedMemory::Handle::Handle):
(WebKit::SharedMemory::Handle::isNull):
(WebKit::SharedMemory::Handle::encode):
(WebKit::SharedMemory::Handle::decode):
(WebKit::createUniqueKey):
(WebKit::SharedMemory::create):
(WebKit::accessMode):
(WebKit::SharedMemory::~SharedMemory):
(WebKit::SharedMemory::createHandle):

8:41 AM WikiStart edited by Adam Roben
Add a link to a new to-be-written page about triaging test failures (diff)
8:37 AM Changeset in webkit [71117] by Csaba Osztrogonác
  • 2 edits in trunk/WebCore

Unreviewed trivial fix.

  • WebCore.pro: Remove inspector/InspectorResource.h after r71035.
8:36 AM Changeset in webkit [71116] by Adam Roben
  • 2 edits in trunk/WebKitTools

Only track resource identifiers in DRT when dumpResourceLoadCallbacks is on

This reverts Windows to our pre-r71097 behavior. That patch made us
track all resource identifiers, including the main resource, so the
main resource's URL started appearing in test output instead of
"<unknown>". Arguably having the main resource's URL is better, but all
other platforms print "<unknown>" and we want to match.

Fixes <http://webkit.org/b/48837> <rdar://problem/8620351> REGRESSION
(r71097): Many http tests failing on Windows

Reviewed by Anders Carlsson.

  • DumpRenderTree/win/ResourceLoadDelegate.cpp:

(ResourceLoadDelegate::identifierForInitialRequest): Don't add the
identifier to the URL map if we're not supposed to dump resource load
callbacks.
(ResourceLoadDelegate::removeIdentifierForRequest): Always remove the
identifier from the URL map even if we're already "done". There's no
point in keeping out-of-date identifiers around.

8:22 AM Changeset in webkit [71115] by timothy@apple.com
  • 13 edits
    1 add in trunk/WebKit2

Add WKInspector public APIs.

https://webkit.org/b/48718

Reviewed by Pavel Feldman.

  • DerivedSources.make: Added WebInspector to MESSAGE_RECEIVERS section.
  • UIProcess/API/C/WKInspector.cpp:

(WKInspectorIsVisible): Added.
(WKInspectorShow): Added.
(WKInspectorClose): Added.
(WKInspectorShowConsole): Added.
(WKInspectorIsAttached): Added.
(WKInspectorAttach): Added.
(WKInspectorDetach): Added.
(WKInspectorIsDebuggingJavaScript): Added.
(WKInspectorToggleJavaScriptDebugging): Added.
(WKInspectorIsProfilingJavaScript): Added.
(WKInspectorToggleJavaScriptProfiling): Added.
(WKInspectorIsProfilingPage): Added.
(WKInspectorTogglePageProfiling): Added.

  • UIProcess/API/C/WKInspector.h: Added new functions.
  • UIProcess/WebInspectorProxy.cpp:

(WebKit::WebInspectorProxy::WebInspectorProxy): Initialize new booleans.
(WebKit::WebInspectorProxy::invalidate): Reset new booleans.
(WebKit::WebInspectorProxy::show): Added.
(WebKit::WebInspectorProxy::close): Added.
(WebKit::WebInspectorProxy::showConsole): Added.
(WebKit::WebInspectorProxy::attach): Added.
(WebKit::WebInspectorProxy::detach): Added.
(WebKit::WebInspectorProxy::toggleJavaScriptDebugging): Added.
(WebKit::WebInspectorProxy::toggleJavaScriptProfiling): Added.
(WebKit::WebInspectorProxy::togglePageProfiling): Added.

  • UIProcess/WebInspectorProxy.h:

(WebKit::WebInspectorProxy::isVisible): Added.
(WebKit::WebInspectorProxy::isAttached): Added.
(WebKit::WebInspectorProxy::isDebuggingJavaScript): Added.
(WebKit::WebInspectorProxy::isProfilingJavaScript): Added.
(WebKit::WebInspectorProxy::isProfilingPage): Added.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::close): Invalidate the WebInspectorProxy.
(WebKit::WebPageProxy::inspector): Return 0 if closed or invalid.

  • WebKit2.pro: Added new files.
  • WebKit2.xcodeproj/project.pbxproj: Added new Files.
  • WebProcess/WebPage/WebInspector.cpp:

(WebKit::WebInspector::show): Added.
(WebKit::WebInspector::close): Added.
(WebKit::WebInspector::showConsole): Added.
(WebKit::WebInspector::startJavaScriptDebugging): Added.
(WebKit::WebInspector::stopJavaScriptDebugging): Added.
(WebKit::WebInspector::startJavaScriptProfiling): Added.
(WebKit::WebInspector::stopJavaScriptProfiling): Added.
(WebKit::WebInspector::startPageProfiling): Added.
(WebKit::WebInspector::stopPageProfiling): Added.

  • WebProcess/WebPage/WebInspector.h:

(WebKit::WebInspector::page): Added.

  • WebProcess/WebPage/WebInspector.messages.in: Added.
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::close): Destroy the WebInspector.
(WebKit::WebPage::inspector): Return 0 if closed.
(WebKit::WebPage::didReceiveMessage): Send the message to the WebInspector
if the message class is MessageClassWebInspector.

  • win/WebKit2.vcproj: Added new files.
8:22 AM Changeset in webkit [71114] by timothy@apple.com
  • 3 edits in trunk/WebKit2

Make WebInspectorFrontendClient inherit from InspectorFrontendClientLocal.

We can do this since the Web Inspector WebPage will share the same process
with the inspected WebPage.

https://webkit.org/b/48652

Reviewed by Pavel Feldman.

  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:

(WebKit::WebInspectorFrontendClient::WebInspectorFrontendClient):
Call the InspectorFrontendClientLocal constructor.
(WebKit::WebInspectorFrontendClient::attachWindow): Added.
(WebKit::WebInspectorFrontendClient::detachWindow): Added.
(WebKit::WebInspectorFrontendClient::setAttachedWindowHeight): Added.

  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.h:
7:32 AM Changeset in webkit [71113] by podivilov@chromium.org
  • 4 edits in trunk/WebCore

2010-11-02 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: make xhr breakpoints editable
https://bugs.webkit.org/show_bug.cgi?id=48830

  • inspector/front-end/BreakpointManager.js: (WebInspector.Breakpoint.prototype.set sourceText): (WebInspector.Breakpoint.prototype.click): (WebInspector.Breakpoint.prototype.populateLabelElement): (WebInspector.XHRBreakpoint.prototype.populateEditElement):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype.addBreakpointItem): (WebInspector.BreakpointsSidebarPane.prototype._breakpointItemClicked): (WebInspector.BreakpointsSidebarPane.prototype._contextMenuEventFired): (WebInspector.BreakpointsSidebarPane.prototype._addListElement): (WebInspector.BreakpointsSidebarPane.prototype._removeListElement): (WebInspector.XHRBreakpointsSidebarPane.addButtonClicked): (WebInspector.XHRBreakpointsSidebarPane): (WebInspector.XHRBreakpointsSidebarPane.prototype._startEditingBreakpoint): (WebInspector.XHRBreakpointsSidebarPane.prototype._hideEditBreakpointDialog): (WebInspector.XHRBreakpointsSidebarPane.prototype._breakpointItemClicked): (WebInspector.BreakpointItem): (WebInspector.BreakpointItem.prototype.get element): (WebInspector.BreakpointItem.prototype.click): (WebInspector.BreakpointItem.prototype.populateEditElement): (WebInspector.BreakpointItem.prototype.remove): (WebInspector.BreakpointItem.prototype._hitStateChanged): (WebInspector.BreakpointItem.prototype._labelChanged): (WebInspector.BreakpointItem.prototype._createLabelElement):
  • inspector/front-end/inspector.js: (WebInspector.createJSBreakpointsSidebarPane.breakpointAdded): (WebInspector.createJSBreakpointsSidebarPane): (WebInspector.createDOMBreakpointsSidebarPane.breakpointAdded): (WebInspector.createDOMBreakpointsSidebarPane): (WebInspector.createXHRBreakpointsSidebarPane.breakpointAdded): (WebInspector.createXHRBreakpointsSidebarPane):
6:54 AM Changeset in webkit [71112] by podivilov@chromium.org
  • 3 edits in trunk/WebCore

2010-11-02 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: fix for breakpoints not restored on live edit bug
https://bugs.webkit.org/show_bug.cgi?id=48833

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.debuggerPaused):
  • inspector/front-end/inspector.js: (WebInspector.pausedScript):
6:35 AM Changeset in webkit [71111] by apavlov@chromium.org
  • 3 edits in trunk/WebCore

2010-11-02 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [REGRESSION] Rule addition/selector editing result in unusable rules
https://bugs.webkit.org/show_bug.cgi?id=48826

  • inspector/front-end/CSSStyleModel.js: (WebInspector.CSSStyleModel.prototype.setRuleSelector): (WebInspector.CSSStyleModel.prototype.addRule):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted.successCallback): (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted):
5:08 AM Changeset in webkit [71110] by vestbo@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Don't include Objective-C sources in SOURCES

Reviewed by Andreas Kling.

  • WebCore.pro:
3:43 AM Changeset in webkit [71109] by podivilov@chromium.org
  • 17 edits in trunk/WebCore

2010-11-02 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: persist native breakpoints in localStorage
https://bugs.webkit.org/show_bug.cgi?id=48711

  • inspector/InspectorController.cpp: (WebCore::InspectorController::disconnectFrontend): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::clearNativeBreakpoints):
  • inspector/InspectorController.h:
  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager): (WebInspector.DOMBreakpoint): (WebInspector.EventListenerBreakpoint): (WebInspector.EventListenerBreakpoint.eventNameForUI): (WebInspector.EventListenerBreakpoint.prototype._uiEventName): (WebInspector.XHRBreakpoint): (WebInspector.XHRBreakpoint.prototype.populateStatusMessageElement):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype.addBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype._breakpointHit): (WebInspector.BreakpointItem.prototype._hitStateChanged): (WebInspector.EventListenerBreakpointsSidebarPane):
  • inspector/front-end/ConsoleView.js:
  • inspector/front-end/EventListenersSidebarPane.js:
  • inspector/front-end/NetworkPanel.js:
  • inspector/front-end/Panel.js:
  • inspector/front-end/ResourceView.js:
  • inspector/front-end/ScriptsPanel.js:
  • inspector/front-end/Settings.js: (WebInspector.Settings): (WebInspector.Settings.prototype.inspectedURLChanged): (WebInspector.Settings.prototype._formatProjectKey):
  • inspector/front-end/StoragePanel.js:
  • inspector/front-end/StylesSidebarPane.js:
  • inspector/front-end/WatchExpressionsSidebarPane.js:
  • inspector/front-end/inspector.css: (.pane .breakpoint-hit):
  • inspector/front-end/inspector.js: (WebInspector.set currentPanel): (WebInspector.doLoadedDone.onPopulateScriptObjects): (WebInspector.doLoadedDone): (WebInspector.reset): (WebInspector.inspectedURLChanged):
3:31 AM Changeset in webkit [71108] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: use mimeType from cached resource
in case response data contains none.
https://bugs.webkit.org/show_bug.cgi?id=48773

  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::didReceiveResponse):
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel.prototype._createTable):
3:29 AM Changeset in webkit [71107] by caseq@chromium.org
  • 13 edits in trunk

2010-11-01 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [Extensions API] expose API to create properties watch sidebar panes
https://bugs.webkit.org/show_bug.cgi?id=48761

  • inspector/front-end/ExtensionAPI.js: (WebInspector.injectedExtensionAPI.PanelImpl.prototype.createSidebarPane): (WebInspector.injectedExtensionAPI.PanelImpl.prototype.createWatchExpressionSidebarPane.callbackWrapper): (WebInspector.injectedExtensionAPI.PanelImpl.prototype.createWatchExpressionSidebarPane): (WebInspector.injectedExtensionAPI): (WebInspector.injectedExtensionAPI.WatchExpressionSidebarPaneImpl): (WebInspector.injectedExtensionAPI.WatchExpressionSidebarPaneImpl.prototype.setExpression): (WebInspector.injectedExtensionAPI.WatchExpressionSidebarPaneImpl.prototype.setObject):
  • inspector/front-end/ExtensionPanel.js: Added ExtensionWatchSidebarPane (WebInspector.ExtensionWatchSidebarPane): (WebInspector.ExtensionWatchSidebarPane.prototype.setObject): (WebInspector.ExtensionWatchSidebarPane.prototype.setExpression): (WebInspector.ExtensionWatchSidebarPane.prototype._onEvaluate): (WebInspector.ExtensionWatchSidebarPane.prototype._setObject):
  • inspector/front-end/ExtensionServer.js: (WebInspector.ExtensionServer): (WebInspector.ExtensionServer.prototype.notifyExtensionWatchSidebarUpdated): (WebInspector.ExtensionServer.prototype._onCreateSidebar): (WebInspector.ExtensionServer.prototype._onCreateWatchExpressionSidebarPane): (WebInspector.ExtensionServer.prototype._createSidebar): (WebInspector.ExtensionServer.prototype._onSetWatchSidebarContent):
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor.):
  • inspector/front-end/RemoteObject.js: Added a wrapper for local JSON object to simulate RemoteObject interface. (WebInspector.RemoteObject.fromLocalObject): (WebInspector.LocalJSONObject): (WebInspector.LocalJSONObject.prototype.get description): (WebInspector.LocalJSONObject.prototype.get type): (WebInspector.LocalJSONObject.prototype.get hasChildren): (WebInspector.LocalJSONObject.prototype.getOwnProperties): (WebInspector.LocalJSONObject.prototype.getProperties): (WebInspector.LocalJSONObject.prototype.isError):

2010-11-01 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: [Extensions API] expose API to create properties watch sidebar panes
https://bugs.webkit.org/show_bug.cgi?id=48761

  • inspector/extensions-api-expected.txt:
  • inspector/extensions-events-expected.txt:
  • inspector/extensions-expected.txt:
  • inspector/extensions-test.js: Added generic way to dispatch messages to front-end and receive callbacks. (initialize_ExtensionsTest.InspectorTest.dispatchOnMessage): (initialize_ExtensionsTest.InspectorTest.dumpSidebarContent):
  • inspector/extensions.html:
  • inspector/resources/extension-main.js: (dispatchOnFrontend.callbackWrapper): (dispatchOnFrontend):
3:20 AM Changeset in webkit [71106] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2010-11-02 Philippe Normand <pnormand@igalia.com>

Unreviewed, unskip passing media test on the GTK+ port.

  • platform/gtk/Skipped: Unskip media/video-canvas-alpha.html.
3:11 AM Changeset in webkit [71105] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-11-02 Ryuan Choi <ryuan.choi@samsung.com>

Unreviewed build fix for EFL build with libcurl

[EFL] remove multiple definition with libcurl
https://bugs.webkit.org/show_bug.cgi?id=48499

Add macro to isolate setCookieStoragePrivateBrowsingEnabled from EFL
build because it was duplicated.

  • platform/network/curl/CookieJarCurl.cpp:
2:57 AM Changeset in webkit [71104] by pfeldman@chromium.org
  • 6 edits in trunk/WebCore

2010-11-02 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: highlight frame on hover in resources panel.
https://bugs.webkit.org/show_bug.cgi?id=48760

  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::highlightFrame):
  • inspector/InspectorController.h: (WebCore::InspectorController::hideFrameHighlight):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.showView): (WebInspector.StoragePanel.prototype._onmousemove): (WebInspector.StoragePanel.prototype._onmouseout): (WebInspector.FrameTreeElement.prototype.onselect): (WebInspector.FrameTreeElement.prototype.ondeselect): (WebInspector.FrameTreeElement.prototype.set displayName): (WebInspector.FrameTreeElement.prototype.set hovered):
  • inspector/front-end/inspector.css: (.storage.panel .sidebar li.selected .selection): (.storage.panel .sidebar :focus li.selected .selection): (body.inactive .storage.panel .sidebar li.selected .selection):
1:35 AM Changeset in webkit [71103] by Nikolas Zimmermann
  • 28 edits
    7 adds in trunk

2010-11-02 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Convert SVGPreserveAspectRatio to the new SVGPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48822

Convert the next type to use the SVGPropertyTearOffConcept. Just SVGAnimatedTransformList/SVGTransform/SVGStringList are missing.

Tests: svg/dom/SVGAnimatedPreserveAspectRatio.html

svg/dom/SVGPreserveAspectRatio.html

  • GNUmakefile.am: Add SVGAnimatedPreserveAspectRatio.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/scripts/CodeGenerator.pm: Recognize SVGAnimatedPreserveAspectRatio as new style animated type.
  • bindings/scripts/CodeGeneratorJS.pm: Remove special SVGPreserveAspectRatio handling.
  • bindings/scripts/CodeGeneratorObjC.pm: Always include the type wrapped by SVGPropertyTearOff<> - this was missing before, and now become visible as missing include.
  • bindings/scripts/CodeGeneratorV8.pm: Remove special SVGPreserveAspectRatio handling.
  • svg/DeprecatedSVGAnimatedPropertyTraits.h: Remove SVGPreserveAspectRatio handling.
  • svg/DeprecatedSVGAnimatedTemplate.h: Ditto.
  • svg/SVGAnimatedPreserveAspectRatio.h: Added.
  • svg/SVGFEImageElement.h: s/DECLARE_ANIMATED_PROPERTY/DECLARE_ANIMATED_PROPERTY_NEW/ where SVGPreserveAspectRatio is used.
  • svg/SVGFitToViewBox.cpp: (WebCore::SVGFitToViewBox::viewBoxToViewTransform): SVGPreserveAspectRatio::getCTM doesn't need x/y parameters, they were not used anywhere, remove them.
  • svg/SVGFitToViewBox.h: Replace includes by class forwards.
  • svg/SVGImageElement.h: s/DECLARE_ANIMATED_PROPERTY/DECLARE_ANIMATED_PROPERTY_NEW/ where SVGPreserveAspectRatio is used.
  • svg/SVGLocatable.cpp: Add SVGNames.h include, to allow Qt to build (thanks ews!).
  • svg/SVGMarkerElement.h: Ditto.
  • svg/SVGPatternElement.h: Ditto.
  • svg/SVGPreserveAspectRatio.cpp: Modernize code, add error handling for align/meetOrSlice covered by the new tests. (WebCore::SVGPreserveAspectRatio::SVGPreserveAspectRatio): (WebCore::SVGPreserveAspectRatio::setAlign): (WebCore::SVGPreserveAspectRatio::setMeetOrSlice): (WebCore::SVGPreserveAspectRatio::parsePreserveAspectRatio): (WebCore::SVGPreserveAspectRatio::transformRect): (WebCore::SVGPreserveAspectRatio::getCTM): (WebCore::SVGPreserveAspectRatio::valueAsString):
  • svg/SVGPreserveAspectRatio.h: Reindented header. (WebCore::SVGPreserveAspectRatio::align): Inlined. (WebCore::SVGPreserveAspectRatio::meetOrSlice): Inlined.
  • svg/SVGPreserveAspectRatio.idl: Enable StrictTypeChecking for the align/meetOrSlice properties.
  • svg/SVGSVGElement.h: s/DECLARE_ANIMATED_PROPERTY/DECLARE_ANIMATED_PROPERTY_NEW/ where SVGPreserveAspectRatio is used.
  • svg/SVGStyledTransformableElement.h: Add SVGAnimatedPropertyMacros.h include, needed by efl to build (thanks ews!).
  • svg/SVGSymbolElement.h: Ditto.
  • svg/SVGViewElement.h: Ditto.
  • svg/SVGViewSpec.h: Ditto.

2010-11-02 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Convert SVGPreserveAspectRatio to the new SVGPropertyTearOff concept
https://bugs.webkit.org/show_bug.cgi?id=48822

  • svg/dom/SVGAnimatedPreserveAspectRatio-expected.txt: Added.
  • svg/dom/SVGAnimatedPreserveAspectRatio.html: Added.
  • svg/dom/SVGPreserveAspectRatio-expected.txt: Added.
  • svg/dom/SVGPreserveAspectRatio.html: Added.
  • svg/dom/script-tests/SVGAnimatedPreserveAspectRatio.js: Added.
  • svg/dom/script-tests/SVGPreserveAspectRatio.js: Added.

Nov 1, 2010:

11:34 PM Changeset in webkit [71102] by morrita@google.com
  • 4 edits in trunk/LayoutTests

2010-11-01 MORITA Hajime <morrita@google.com>

Unreviewed, added skip entries for platforms where spellchecking
is not supported.

  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
10:30 PM Changeset in webkit [71101] by morrita@google.com
  • 8 edits
    3 adds in trunk

2010-11-01 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

@spellcheck attribute at the child of contentEditable node is ignored.
https://bugs.webkit.org/show_bug.cgi?id=48418

  • editing/spelling/script-tests/spelling-attribute-at-child.js: Added.
  • editing/spelling/spelling-attribute-at-child-expected.txt: Added.
  • editing/spelling/spelling-attribute-at-child.html: Added.

2010-11-01 MORITA Hajime <morrita@google.com>

Reviewed by Kent Tamura.

@spellcheck attribute at the child of contentEditable node is ignored.
https://bugs.webkit.org/show_bug.cgi?id=48418

Changed to check spellcheck availability against a node under the
selection (caret) instead of the focus. For shadow elements, the
check now refers its host node. Note that the original code didn't
care shadows because focus is never set on them.

Test: editing/spelling/spelling-attribute-at-child.html

  • dom/Element.h:
  • editing/Editor.cpp: (WebCore::Editor::markMisspellingsOrBadGrammar): (WebCore::Editor::isSpellCheckingEnabledFor): (WebCore::Editor::isSpellCheckingEnabledInFocusedNode): Now just calling isSpellCheckingEnabledFor() (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges):
  • editing/Editor.h:
  • platform/ContextMenu.cpp: (WebCore::ContextMenu::populate):
  • rendering/TextControlInnerElements.cpp: (WebCore::TextControlInnerElement::isSpellCheckingEnabled): Added.
  • rendering/TextControlInnerElements.h:
10:09 PM Changeset in webkit [71100] by abarth@webkit.org
  • 50 edits
    11 deletes in trunk/LayoutTests

2010-11-01 Adam Barth <abarth@webkit.org>

Update expected results for PNG ICC profile.

  • platform/chromium-mac/fast/borders/svg-as-border-image-2-expected.checksum:
  • platform/chromium-mac/fast/borders/svg-as-border-image-2-expected.png: Removed.
  • platform/chromium-mac/fast/borders/svg-as-border-image-expected.checksum:
  • platform/chromium-mac/fast/borders/svg-as-border-image-expected.png: Removed.
  • platform/chromium-mac/fast/css/background-shorthand-invalid-url-expected.checksum:
  • platform/chromium-mac/fast/css/background-shorthand-invalid-url-expected.png: Removed.
  • platform/chromium-mac/fast/media/mq-min-pixel-ratio-expected.checksum: Removed.
  • platform/chromium-mac/fast/media/mq-min-pixel-ratio-expected.png: Removed.
  • platform/chromium-mac/fast/reflections/reflection-masks-expected.checksum: Removed.
  • platform/chromium-mac/fast/reflections/reflection-masks-expected.png: Removed.
  • platform/chromium-mac/fast/reflections/reflection-masks-opacity-expected.checksum: Removed.
  • platform/chromium-mac/http/tests/misc/slow-loading-image-in-pattern-expected.checksum:
  • platform/chromium-mac/http/tests/misc/slow-loading-image-in-pattern-expected.png:
  • platform/chromium-mac/scrollbars/listbox-scrollbar-combinations-expected.checksum:
  • platform/chromium-mac/scrollbars/overflow-scrollbar-combinations-expected.checksum:
  • platform/chromium-mac/scrollbars/overflow-scrollbar-combinations-expected.png: Removed.
  • platform/chromium-mac/svg/custom/createImageElement-expected.checksum:
  • platform/chromium-mac/svg/custom/createImageElement-expected.png:
  • platform/chromium-mac/svg/custom/focus-ring-expected.checksum:
  • platform/chromium-mac/svg/custom/focus-ring-expected.png: Removed.
  • platform/chromium-mac/svg/custom/group-opacity-expected.checksum:
  • platform/chromium-mac/svg/custom/group-opacity-expected.png: Removed.
  • platform/chromium-mac/svg/custom/image-parent-translation-expected.checksum:
  • platform/chromium-mac/svg/custom/image-parent-translation-expected.png:
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.checksum:
  • platform/chromium-mac/svg/custom/image-small-width-height-expected.png:
  • platform/chromium-mac/svg/custom/image-with-transform-clip-filter-expected.checksum:
  • platform/chromium-mac/svg/custom/js-update-image-and-display-expected.checksum:
  • platform/chromium-mac/svg/custom/js-update-image-and-display-expected.png:
  • platform/chromium-mac/svg/custom/js-update-image-and-display2-expected.checksum:
  • platform/chromium-mac/svg/custom/js-update-image-and-display2-expected.png:
  • platform/chromium-mac/svg/custom/js-update-image-and-display3-expected.checksum:
  • platform/chromium-mac/svg/custom/js-update-image-and-display3-expected.png:
  • platform/chromium-mac/svg/custom/js-update-image-expected.checksum:
  • platform/chromium-mac/svg/custom/js-update-image-expected.png:
  • platform/chromium-mac/svg/custom/pointer-events-image-expected.checksum:
  • platform/chromium-mac/svg/custom/pointer-events-image-expected.png:
  • platform/chromium-mac/svg/custom/text-image-opacity-expected.checksum:
  • platform/chromium-mac/svg/custom/text-image-opacity-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-height-attr-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-preserveAspectRatio-attr-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-width-attr-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-x-attr-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-dom-y-attr-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-height-prop-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-preserveAspectRatio-prop-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-width-prop-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-x-prop-expected.png:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.checksum:
  • platform/chromium-mac/svg/dynamic-updates/SVGImageElement-svgdom-y-prop-expected.png:
  • platform/chromium/test_expectations.txt:
9:34 PM Changeset in webkit [71099] by Martin Robinson
  • 2 edits in trunk/WebCore

2010-11-01 Martin Robinson <mrobinson@igalia.com>

Reviewed by Daniel Bates.

[GTK] PopupMenuGtk has a very confusing if-else block
https://bugs.webkit.org/show_bug.cgi?id=48816

Small code cleanup for PopupMenuGtk.

  • platform/gtk/PopupMenuGtk.cpp: (WebCore::PopupMenuGtk::show):
9:15 PM Changeset in webkit [71098] by abarth@webkit.org
  • 7 edits in trunk/WebCore

2010-11-01 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[Chromium] Add ICC support for PNG on Mac
https://bugs.webkit.org/show_bug.cgi?id=48170

This just pipes the ICC profile from libpng to CoreGraphics. This
patch would have been a lot prettier on Snow Leopard, but we have to
use a somewhat ugly API to get this to work on Leopard.

This is covered by about infinite tests.

  • platform/image-decoders/ImageDecoder.cpp: (WebCore::RGBA32Buffer::setColorProfile):
  • platform/image-decoders/ImageDecoder.h:
  • platform/image-decoders/cg/ImageDecoderCG.cpp: (WebCore::RGBA32Buffer::asNewNativeImage):
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageDecoder::headerAvailable): (WebCore::PNGImageDecoder::rowAvailable):
  • platform/image-decoders/qt/RGBA32BufferQt.cpp: (WebCore::RGBA32Buffer::setColorProfile):
  • platform/image-decoders/skia/ImageDecoderSkia.cpp: (WebCore::RGBA32Buffer::setColorProfile):
7:42 PM Changeset in webkit [71097] by commit-queue@webkit.org
  • 7 edits
    1 add in trunk

2010-11-01 Jenn Braithwaite <jennb@chromium.org>

Reviewed by Adam Roben.

Windows: Update resource tracking when moving a frame between documents
https://bugs.webkit.org/show_bug.cgi?id=48364

  • Interfaces/IWebResourceLoadDelegatePrivate2.idl:Added Added removeIdentifierForRequest.
  • Interfaces/WebKit.idl: Added IWebResourceLoadDelegatePrivate2.idl.
  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::transferLoadingResourceFromPage):

2010-11-01 Jenn Braithwaite <jennb@chromium.org>

Reviewed by Adam Roben.

Windows: Update resource tracking when moving a frame between documents
https://bugs.webkit.org/show_bug.cgi?id=48364

  • DumpRenderTree/win/DumpRenderTree.cpp: (createWebViewAndOffscreenWindow): (main): Give each WebView its own ResourceLoadDelegate instance in order to make assertions about resource ids on a particular WebView.
  • DumpRenderTree/win/ResourceLoadDelegate.cpp: (ResourceLoadDelegate::identifierForInitialRequest): Always add id to the map. (ResourceLoadDelegate::removeIdentifierForRequest): Added. (ResourceLoadDelegate::willSendRequest): (ResourceLoadDelegate::didReceiveAuthenticationChallenge): (ResourceLoadDelegate::didReceiveResponse): (ResourceLoadDelegate::didFinishLoadingFromDataSource): (ResourceLoadDelegate::didFailLoadingWithError): (ResourceLoadDelegate::descriptionSuitableForTestResult): Replace static descriptionSuitableForTestResult with static member function to access identifier map.
  • DumpRenderTree/win/ResourceLoadDelegate.h: (ResourceLoadDelegate::urlMap): Moved within class so that each WebView has its own id map.
7:20 PM Changeset in webkit [71096] by commit-queue@webkit.org
  • 2 edits in trunk

2010-11-01 Viatcheslav Ostapenko <ostapenko.viatcheslav@nokia.com>

Reviewed by Andreas Kling.

[Qt][Symbian] QtTestBrowser runs out of memory with most of javascript benchmarks.
https://bugs.webkit.org/show_bug.cgi?id=48730

Increase the maximum heap size on Symbian to 256 MB.

  • WebKit.pri:
7:10 PM Changeset in webkit [71095] by dimich@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Snap chromium checksum for the newly added baseline (added in http://trac.webkit.org/changeset/71088)

  • platform/chromium-mac/fast/css/text-input-with-webkit-border-radius-expected.checksum: Added.
6:56 PM Changeset in webkit [71094] by chang.shu@nokia.com
  • 5 edits in trunk

2010-11-01 Chang Shu <chang.shu@nokia.com>

Reviewed by Antonio Gomes.

Leave SelectElement key handling without setting event default
handled when the selection reaches the boundary so focus can
move to neighbor nodes in spatial navigation.
https://bugs.webkit.org/show_bug.cgi?id=48145

  • fast/events/spatial-navigation/snav-multiple-select-expected.txt:
  • fast/events/spatial-navigation/snav-multiple-select.html:

2010-11-01 Chang Shu <chang.shu@nokia.com>

Reviewed by Antonio Gomes.

Leave SelectElement key handling without setting event default
handled when the selection reaches the boundary so focus can
move to neighbor nodes in spatial navigation.
https://bugs.webkit.org/show_bug.cgi?id=48145

  • dom/SelectElement.cpp: (WebCore::SelectElement::listBoxDefaultEventHandler):
6:54 PM Changeset in webkit [71093] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Did I run --lint-test-files last time? Probably not.

  • platform/chromium/test_expectations.txt:
6:44 PM Changeset in webkit [71092] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.

  • platform/chromium/test_expectations.txt:
6:40 PM Changeset in webkit [71091] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

2010-11-01 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r71080.
http://trac.webkit.org/changeset/71080
https://bugs.webkit.org/show_bug.cgi?id=48815

This change caused many crashes on the debug bot. (Requested
by mrobinson on #webkit).

  • platform/network/soup/SocketStreamHandle.h:
  • platform/network/soup/SocketStreamHandleSoup.cpp: (WebCore::isActiveHandle): (WebCore::deactivateHandle): (WebCore::SocketStreamHandle::SocketStreamHandle): (WebCore::SocketStreamHandle::connected): (WebCore::SocketStreamHandle::readBytes): (WebCore::SocketStreamHandle::beginWaitingForSocketWritability): (WebCore::connectedCallback): (WebCore::readReadyCallback): (WebCore::writeReadyCallback):
6:36 PM Changeset in webkit [71090] by commit-queue@webkit.org
  • 4 edits in trunk/WebKit/gtk

2010-11-01 Jennifer Braithwaite <jennb@google.com>

Reviewed by Martin Robinson.

GTK: Update resource tracking when moving a frame between documents
https://bugs.webkit.org/show_bug.cgi?id=48362

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::transferLoadingResourceFromPage): Assign resource id to current web view and remove from former web view.
  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (webkit_web_view_remove_resource): Added.
6:15 PM Changeset in webkit [71089] by commit-queue@webkit.org
  • 5 edits in trunk/WebKit2

2010-11-01 Juha Savolainen <juha.savolainen@weego.fi>

Reviewed by Kenneth Rohde Christiansen.

Initial implementation of QWKHistoryItem.
https://bugs.webkit.org/show_bug.cgi?id=48620

The QWkHistoryItem represents one item in the history of QWKPage.

  • UIProcess/API/qt/qwkhistory.cpp: (QWKHistoryItemPrivate::QWKHistoryItemPrivate): (QWKHistoryItem::QWKHistoryItem): (QWKHistoryItem::~QWKHistoryItem): (QWKHistoryItem::title): The page title (QWKHistoryItem::url): The URL of the page. (QWKHistoryItem::originalUrl): The URL used to access the page.
  • UIProcess/API/qt/qwkhistory.h:
  • UIProcess/API/qt/qwkhistory_p.h:
  • WebKit2.pro: Added WKBackForwardListItem.h and WKBackForwardListItem.cpp
5:55 PM Changeset in webkit [71088] by mihaip@chromium.org
  • 311 edits
    310 copies
    3 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for css/ for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

5:47 PM Changeset in webkit [71087] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.

  • platform/chromium/test_expectations.txt:
5:39 PM Changeset in webkit [71086] by hyatt@apple.com
  • 1 add in trunk/LayoutTests/platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.txt

Fix layout tests.

5:38 PM Changeset in webkit [71085] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/platform/mac/fast/text/international/vertical-text-glyph-test-expected.txt

Fix layout tests.

5:31 PM Changeset in webkit [71084] by mihaip@chromium.org
  • 263 edits
    262 copies
    2 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for editing/ for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

Some tests had differences due to spell correction underlining, those
were skipped since it's not clear what the correct dictionary state is.

5:17 PM Changeset in webkit [71083] by ojan@chromium.org
  • 2 edits in trunk/WebKitTools

2010-11-01 Ojan Vafai <ojan@chromium.org>

Reviewed by Dimitri Glazkov.

remove debug code from run_webkit_tests.py
https://bugs.webkit.org/show_bug.cgi?id=48800

Remove temporary debug code and make --master-name required
if --test-results-server is set now that all clients set
--master-name.

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
5:17 PM Changeset in webkit [71082] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Temporarily disable new test added by http://trac.webkit.org/changeset/71066
Need to convert it to dumpAsText().

  • platform/chromium/test_expectations.txt:
5:12 PM Changeset in webkit [71081] by andersca@apple.com
  • 9 edits in trunk/WebKit2

Make NPRemoteObjectMap a ref counted object
https://bugs.webkit.org/show_bug.cgi?id=48808

Reviewed by Oliver Hunt.

  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::windowScriptNPObject):

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::WebProcessConnection):
(WebKit::WebProcessConnection::didReceiveSyncMessage):

  • PluginProcess/WebProcessConnection.h:

(WebKit::WebProcessConnection::npRemoteObjectMap):

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::create):
(WebKit::NPRemoteObjectMap::~NPRemoteObjectMap):
(WebKit::NPRemoteObjectMap::createNPObjectProxy):

  • Shared/Plugins/NPRemoteObjectMap.h:
  • WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::PluginProcessConnection):
(WebKit::PluginProcessConnection::didReceiveSyncMessage):

  • WebProcess/Plugins/PluginProcessConnection.h:

(WebKit::PluginProcessConnection::npRemoteObjectMap):

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::getWindowScriptNPObject):

5:09 PM Changeset in webkit [71080] by Martin Robinson
  • 3 edits in trunk/WebCore

2010-11-01 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[Soup] Random crashes in http/tests/websocket/tests/workers/worker-handshake-challenge-randomness.html
https://bugs.webkit.org/show_bug.cgi?id=48805

Track active WebSocket handles via a sequential id. This ensures
that when a handle is reallocated into a recently used segment of
memory, it doesn't trigger a false positive in the code which ensures
the original handle is active.

No new tests. This test should stop crashing on the bots, proving the fix.

  • platform/network/soup/SocketStreamHandle.h: (WebCore::SocketStreamHandle::id): Added an m_id member and accessor to SocketStreamHandle.
  • platform/network/soup/SocketStreamHandleSoup.cpp: (WebCore::getHandleFromId): Updated to work with HashMap of handle ids to SocketStreamHandle*. (WebCore::deactivateHandle): Ditto. (WebCore::activateHandle): Ditto. (WebCore::SocketStreamHandle::SocketStreamHandle): Ditto. (WebCore::SocketStreamHandle::connected): Ditto. (WebCore::SocketStreamHandle::readBytes): Ditto. (WebCore::SocketStreamHandle::beginWaitingForSocketWritability): Ditto. (WebCore::connectedCallback): Ditto. (WebCore::readReadyCallback): Ditto. (WebCore::writeReadyCallback): Ditto.
5:05 PM Changeset in webkit [71079] by tkent@chromium.org
  • 2 edits in trunk/WebCore

2010-11-01 Kent Tamura <tkent@chromium.org>

Unreviewed. Run sort-Xcode-project-file.

  • WebCore.xcodeproj/project.pbxproj:
5:05 PM Changeset in webkit [71078] by tony@chromium.org
  • 2 edits in trunk/WebKitTools

2010-11-01 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

add plugin event logging for linux
https://bugs.webkit.org/show_bug.cgi?id=48779

This is taken from Chromium's fork of the layout test plugin:
http://git.chromium.org/gitweb/?p=chromium.git/.git;a=blob;f=webkit/tools/npapi_layout_test_plugin/main.cpp;h=3ebdada2f049b3624756438cff852364f86a2ede;hb=HEAD#l348

  • DumpRenderTree/TestNetscapePlugIn/main.cpp: (handleEventX11): (NPP_HandleEvent):
5:00 PM Changeset in webkit [71077] by tony@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-11-01 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] Compile TestNetscapePlugin on chromium win
https://bugs.webkit.org/show_bug.cgi?id=48802

It creates npTestNetscapePlugin.dll in the build directory, so
it's not yet loaded by DRT or test_shell.

  • WebKit.gyp:
5:00 PM Changeset in webkit [71076] by andersca@apple.com
  • 1 edit in trunk/WebKit2/ChangeLog

Remove duplicate ChangeLog entry.

4:58 PM Changeset in webkit [71075] by andersca@apple.com
  • 12 edits
    4 copies in trunk/WebKit2

Respond to NP_GetProperty by sending a GetProperty message
https://bugs.webkit.org/show_bug.cgi?id=48797

Reviewed by John Sullivan.

  • Platform/CoreIPC/HandleMessage.h:

(CoreIPC::callMemberFunction):
Add new sync message overload.

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::didReceiveSyncMessage):
If the message class is MessageClassNPObjectMessageReceiver, pass it to the NPRemoteObjectMap.

  • Shared/Plugins/NPIdentifierData.cpp:
  • Shared/Plugins/NPIdentifierData.h:

Add NPIdentifierData, a CoreIPC representation of an NPIdentifier.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::getProperty):
Add stub.

  • Shared/Plugins/NPObjectMessageReceiver.messages.in:

Add GetProperty.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::create):
(WebKit::NPObjectProxy::NPObjectProxy):
(WebKit::NPObjectProxy::initialize):
NPObjectProxy now takes an NPRemoteObjectMap in its create function.

(WebKit::NPObjectProxy::getProperty):
Send a GetProperty message.

(WebKit::NPObjectProxy::NP_GetProperty):
Call getProperty.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::createNPObjectProxy):
Pass the NPRemoteObjectMap.

(WebKit::NPRemoteObjectMap::didReceiveSyncMessage):
Find the right message receiver and dispatch the message to it.

  • Shared/Plugins/NPVariantData.cpp:
  • Shared/Plugins/NPVariantData.h:

Add NPVariantData, which will be a CoreIPC representation of an NPVariant.

  • WebKit2.xcodeproj/project.pbxproj:

Add new files.

  • WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::didReceiveSyncMessage):
If the message class is MessageClassNPObjectMessageReceiver, pass it to the NPRemoteObjectMap.

4:41 PM Changeset in webkit [71074] by Simon Fraser
  • 3 edits
    3 adds in trunk

2010-11-01 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Assertion failure in FrameView::layout when clicking on a YouTube video on youtube.com front page
https://bugs.webkit.org/show_bug.cgi?id=44287

When removing subframes, it's possible for layout() to get called on a FrameView
for a Frame that has been removed from the frame tree. This can happen if the parent
document happens to do a layout() while RenderView still has a reference to the widget
for the removed frame; updateWidgetPosition() will try to force a layout() on the
subframe.

Fix by checking that the FrameView's Frame still has a reference to the page before
calling layout() on the subframe.

Test: fast/frames/layout-after-destruction.html

  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::updateWidgetPosition):
4:36 PM Changeset in webkit [71073] by mihaip@chromium.org
  • 201 edits
    200 copies
    18 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/dom, fast/dynamic, fast/encoding, fast/events, and fast/gradients for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

4:30 PM Changeset in webkit [71072] by mihaip@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/dom, fast/dynamic, fast/encoding, fast/events, and fast/gradients for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

4:25 PM Changeset in webkit [71071] by dimich@chromium.org
  • 7 edits in trunk/WebCore

2010-11-01 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r71065.
http://trac.webkit.org/changeset/71065
https://bugs.webkit.org/show_bug.cgi?id=48801

Seem to breake a lot of tests on Chromium bots (Requested by
dimich on #webkit).

  • platform/image-decoders/ImageDecoder.cpp:
  • platform/image-decoders/ImageDecoder.h:
  • platform/image-decoders/cg/ImageDecoderCG.cpp: (WebCore::RGBA32Buffer::asNewNativeImage):
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageDecoder::headerAvailable): (WebCore::PNGImageDecoder::rowAvailable):
  • platform/image-decoders/qt/RGBA32BufferQt.cpp:
  • platform/image-decoders/skia/ImageDecoderSkia.cpp:
4:23 PM Changeset in webkit [71070] by jamesr@google.com
  • 4 edits
    9 adds
    2 deletes in trunk/LayoutTests

2010-11-01 James Robinson <jamesr@chromium.org>

Update chromium baselines for a few tests.

  • platform/chromium-linux/fast/repaint/iframe-scroll-repaint-expected.checksum: Added.
  • platform/chromium-linux/fast/repaint/iframe-scroll-repaint-expected.png: Added.
  • platform/chromium-linux/svg/custom/resource-invalidate-on-target-update-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/resource-invalidate-on-target-update-expected.png: Added.
  • platform/chromium-mac/fast/repaint/iframe-scroll-repaint-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/iframe-scroll-repaint-expected.png: Added.
  • platform/chromium-mac/svg/custom/resource-invalidate-on-target-update-expected.checksum: Removed.
  • platform/chromium-mac/svg/custom/resource-invalidate-on-target-update-expected.png: Removed.
  • platform/chromium-win/fast/repaint/iframe-scroll-repaint-expected.checksum: Added.
  • platform/chromium-win/fast/repaint/iframe-scroll-repaint-expected.png: Added.
  • platform/chromium-win/fast/repaint/iframe-scroll-repaint-expected.txt: Added.
  • platform/chromium-win/svg/custom/resource-invalidate-on-target-update-expected.checksum:
  • platform/chromium-win/svg/custom/resource-invalidate-on-target-update-expected.png:
  • platform/chromium/test_expectations.txt:
4:12 PM Changeset in webkit [71069] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Add ArgumentCoder specialization for WTF::CString
https://bugs.webkit.org/show_bug.cgi?id=48796

Reviewed by Sam Weinig.

  • Platform/CoreIPC/ArgumentCoders.h:
4:01 PM Changeset in webkit [71068] by dpranke@chromium.org
  • 3 edits in trunk/WebKitTools

2010-11-01 Dirk Pranke <dpranke@chromium.org>

Reviewed by James Robinson.

new-run-webkit-tests: use DRT, child-processes=1 for GPU tests by default
https://bugs.webkit.org/show_bug.cgi?id=48790

  • Scripts/webkitpy/layout_tests/port/chromium_gpu.py:
  • Scripts/webkitpy/layout_tests/port/chromium_gpu_unittest.py:
3:43 PM Changeset in webkit [71067] by hyatt@apple.com
  • 1 edit in trunk/WebCore/rendering/svg/SVGInlineTextBox.cpp

Fix SVG bustage.

3:40 PM Changeset in webkit [71066] by mdelaney@apple.com
  • 3 edits
    3 adds in trunk

2010-11-01 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Setting attr repeatDur=0 on SVG element causes hang
https://bugs.webkit.org/show_bug.cgi?id=48785

Test: svg/animations/repeatDur-zero.xhtml

  • svg/animation/SVGSMILElement.cpp: Fixing bound checking for legal repeatDur values.

2010-11-01 Matthew Delaney <mdelaney@apple.com>

Reviewed by Simon Fraser.

Setting attr repeatDur=0 on SVG element causes hang
https://bugs.webkit.org/show_bug.cgi?id=48785

  • svg/animations/repeatDur-zero.xhtml: Added. Tests if setting attr repeatDur=0 causes a hang or not.
3:33 PM Changeset in webkit [71065] by abarth@webkit.org
  • 7 edits in trunk/WebCore

2010-10-31 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

[Chromium] Add ICC support for PNG on Mac
https://bugs.webkit.org/show_bug.cgi?id=48170

This just pipes the ICC profile from libpng to CoreGraphics. This
patch would have been a lot prettier on Snow Leopard, but we have to
use a somewhat ugly API to get this to work on Leopard.

This is covered by about infinite tests.

  • platform/image-decoders/ImageDecoder.cpp: (WebCore::RGBA32Buffer::setColorProfile):
  • platform/image-decoders/ImageDecoder.h:
  • platform/image-decoders/cg/ImageDecoderCG.cpp: (WebCore::RGBA32Buffer::asNewNativeImage):
  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageDecoder::headerAvailable): (WebCore::PNGImageDecoder::rowAvailable):
  • platform/image-decoders/qt/RGBA32BufferQt.cpp: (WebCore::RGBA32Buffer::setColorProfile):
  • platform/image-decoders/skia/ImageDecoderSkia.cpp: (WebCore::RGBA32Buffer::setColorProfile):
3:27 PM Changeset in webkit [71064] by abarth@webkit.org
  • 5 edits in trunk/WebKitTools

2010-11-01 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Teach check-webkit-style how to accept a list of files to diff on the
command line
https://bugs.webkit.org/show_bug.cgi?id=48784

In a future patch, webkit-patch will use this option to improve
performance. I'm landing this in two pieces to avoid causing a version
skew problem for the style-bot.

  • Scripts/check-webkit-style:
  • Scripts/webkitpy/style/optparser.py:
  • Scripts/webkitpy/style/optparser_unittest.py:
  • Scripts/webkitpy/style_references.py:
3:20 PM Changeset in webkit [71063] by dimich@chromium.org
  • 3 edits
    4 adds
    2 deletes in trunk/LayoutTests

[Chromium] Unreviewed test rebaseline.

  • platform/chromium-linux/svg/custom/gradient-userSpaceOnUse-with-percentage-expected.checksum: Removed.
  • platform/chromium-linux/svg/custom/gradient-userSpaceOnUse-with-percentage-expected.png: Removed.
  • platform/chromium-mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/chromium-mac/fast/repaint/inline-horizontal-bt-overflow-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/inline-horizontal-bt-overflow-expected.png: Added.
  • platform/chromium-mac/fast/repaint/inline-vertical-lr-overflow-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/inline-vertical-lr-overflow-expected.png: Added.
  • platform/chromium/test_expectations.txt:
3:16 PM Changeset in webkit [71062] by mihaip@chromium.org
  • 204 edits
    202 copies
    8 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/body-propagation, fast/borders, fast/box-shadow, fast/box-sizing, fast/clip, and fast/compact for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

platform/mac/test_expectations.txt was also modified to remove two
now-passing tests.

3:10 PM Changeset in webkit [71061] by senorblanco@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-01 Stephen White <senorblanco@chromium.org>

Unreviewed; test expectations update.

Update chromium-gpu test expectations to reflect test_shell
timeouts on ~35 tests.

  • platform/chromium-gpu/test_expectations.txt:
2:53 PM Changeset in webkit [71060] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Fix build.

  • WebKit2.xcodeproj/project.pbxproj:
2:51 PM Changeset in webkit [71059] by crogers@google.com
  • 11 edits in branches/audio/WebCore

audio branch: bracket some JS and V8 wrappper and Event code with #if ENABLE(WEB_AUDIO)

2:30 PM Changeset in webkit [71058] by Simon Fraser
  • 3 edits in trunk/LayoutTests

2010-11-01 Simon Fraser <Simon Fraser>

Fix test failures on Qt by avoiding references to external image files.

  • fast/backgrounds/multiple-backgrounds-computed-style-expected.txt:
  • fast/backgrounds/script-tests/multiple-backgrounds-computed-style.js:
2:29 PM Changeset in webkit [71057] by andersca@apple.com
  • 6 edits
    1 add in trunk/WebKit2

Add NPObjectMessageReceiver.messages.in
https://bugs.webkit.org/show_bug.cgi?id=48782

Reviewed by Adam Roben.

  • DerivedSources.make:

Add NPObjectMessageReceiver.

  • Platform/CoreIPC/MessageID.h:

Add MessageClassNPObjectMessageReceiver message class.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::deallocate):
Add stubbed out deallocate member function.

  • Shared/Plugins/NPObjectMessageReceiver.messages.in: Added.
  • WebKit2.xcodeproj/project.pbxproj:

Add NPObjectMessageReceiver.messages.in.

2:25 PM Changeset in webkit [71056] by mihaip@chromium.org
  • 203 edits
    202 copies
    1 add in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/selectors for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

2:13 PM Changeset in webkit [71055] by hyatt@apple.com
  • 14 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=48776

Reviewed by Dan Bernstein.

Change isVertical() on InlineBox to isHorizontal() instead. This makes the horizontal code path come
first if you branch on isHorizontal() and makes the code read better (since that's the more common code
path).

  • rendering/InlineBox.cpp:

(WebCore::InlineBox::logicalHeight):

  • rendering/InlineBox.h:

(WebCore::InlineBox::InlineBox):
(WebCore::InlineBox::isHorizontal):
(WebCore::InlineBox::setIsHorizontal):
(WebCore::InlineBox::width):
(WebCore::InlineBox::height):
(WebCore::InlineBox::logicalLeft):
(WebCore::InlineBox::setLogicalLeft):
(WebCore::InlineBox::logicalTop):
(WebCore::InlineBox::setLogicalTop):
(WebCore::InlineBox::baselinePosition):
(WebCore::InlineBox::lineHeight):

  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::addToLine):
(WebCore::InlineFlowBox::placeBoxesInInlineDirection):
(WebCore::InlineFlowBox::computeBlockDirectionOverflow):
(WebCore::InlineFlowBox::paintFillLayer):
(WebCore::InlineFlowBox::paintBoxDecorations):
(WebCore::InlineFlowBox::paintMask):

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::marginLogicalLeft):
(WebCore::InlineFlowBox::marginLogicalRight):
(WebCore::InlineFlowBox::borderLogicalLeft):
(WebCore::InlineFlowBox::borderLogicalRight):
(WebCore::InlineFlowBox::paddingLogicalLeft):
(WebCore::InlineFlowBox::paddingLogicalRight):
(WebCore::InlineFlowBox::setInlineDirectionOverflowPositions):
(WebCore::InlineFlowBox::setBlockDirectionOverflowPositions):

  • rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::selectionRect):
(WebCore::InlineTextBox::applyShadowToGraphicsContext):
(WebCore::paintTextWithShadows):
(WebCore::InlineTextBox::paint):
(WebCore::InlineTextBox::paintDecoration):

  • rendering/InlineTextBox.h:
  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::createLineBoxes):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::blockDirectionOverflow):

  • rendering/RenderBox.h:
  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

  • rendering/RenderLineBoxList.cpp:

(WebCore::RenderLineBoxList::hitTest):

  • rendering/RootInlineBox.cpp:

(WebCore::RootInlineBox::RootInlineBox):
(WebCore::RootInlineBox::placeEllipsis):

  • rendering/RootInlineBox.h:

(WebCore::RootInlineBox::baselinePosition):
(WebCore::RootInlineBox::lineHeight):

2:10 PM Changeset in webkit [71054] by andersca@apple.com
  • 7 edits in trunk/WebKit2

Start stubbing out NPClass functions in NPObjectProxy
https://bugs.webkit.org/show_bug.cgi?id=48778

Reviewed by Adam Roben.

  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::windowScriptNPObject):
getOrCreateNPObjectProxy is now createNPObjectProxy.

  • Shared/Plugins/NPObjectProxy.cpp:

(WebKit::NPObjectProxy::create):
Allocate and initialize an NPObjectProxy object.

(WebKit::NPObjectProxy::NPObjectProxy):
Initialize m_npObjectID to 0.

(WebKit::NPObjectProxy::isNPObjectProxy):
Check the class.

(WebKit::NPObjectProxy::initialize):
Set m_npObjectID.

(WebKit::NPObjectProxy::npClass):
Return the NPClass with all functions filled in.

(WebKit::NPObjectProxy::NP_Allocate):
Create a new NPObjectProxy object.

(WebKit::NPObjectProxy::NP_Deallocate):
Deallocate the NPObjectProxy object.

(WebKit::NPObjectProxy::NP_HasMethod):
(WebKit::NPObjectProxy::NP_Invoke):
(WebKit::NPObjectProxy::NP_InvokeDefault):
(WebKit::NPObjectProxy::NP_HasProperty):
(WebKit::NPObjectProxy::NP_GetProperty):
(WebKit::NPObjectProxy::NP_SetProperty):
(WebKit::NPObjectProxy::NP_RemoveProperty):
(WebKit::NPObjectProxy::NP_Enumerate):
(WebKit::NPObjectProxy::NP_Construct):
Add stubs.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::NPRemoteObjectMap::createNPObjectProxy):

  • Shared/Plugins/NPRemoteObjectMap.h:

Rename getOrCreateNPObjectProxy to createNPObjectProxy.

  • WebProcess/Plugins/Netscape/NPJSObject.cpp:

(WebKit::NPJSObject::NP_Allocate):
Remove unused parameter.

2:01 PM Changeset in webkit [71053] by dimich@chromium.org
  • 1 edit
    4 adds in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
Add baseline for a new test added in http://trac.webkit.org/changeset/71043

  • platform/chromium-mac/fast/repaint/inline-vertical-rl-overflow-expected.checksum: Added.
  • platform/chromium-mac/fast/repaint/inline-vertical-rl-overflow-expected.png: Added.
  • platform/chromium-mac/fast/repaint/inline-vertical-rl-overflow-expected.txt: Added.
  • platform/chromium-win/fast/repaint/inline-vertical-rl-overflow-expected.txt: Added.
2:00 PM Changeset in webkit [71052] by mario@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-11-01 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Martin Robinson.

[GTK] Cleanup needed in testatk.c file
https://bugs.webkit.org/show_bug.cgi?id=48759

Cleanup done in testatk.c, mainly addressing the following issues:

  • Removed any trace of extra main loops in tests, so now we just spin the original main loop in all of them to get the a11y objects.
  • As we no longer use the extra main loops, removed the bail_out function, used along with g_idle_add() to stop those extra loops.
  • As we now spin the loop in every test, created a new function waitForAccessibilityObjects() to write the loop just once.
  • Fixed name in incorrectly named test testWekitAtkTextSelections.
  • Alphabetically sorted the list of includes.
  • Renamed all identifiers to camel Case notation.
  • Declare variables when needed only (instead of declaring all of them at the beginning of the function).
  • Don't declare two variables of the same type in the same line.
  • No more short names like 'obj', 'textObj' or 'alloc'. Use names like 'object', 'textObject' and 'allocation' instead.
  • Fixed indentation.
  • Make sure all comments finish with '.'
  • Replace NULL's with 0's.
  • tests/testatk.c: (waitForAccessibleObjects): New, manually spins the main context to make sure accessible objects are created before continuing. (testGetTextFunction): Fixed this function to be compliant with the WebKit's coding style. (runGetTextTests): Ditto. (testWebkitAtkGetTextAtOffsetForms): Ditto. (testWebkitAtkGetTextAtOffset): Ditto. (testWebkitAtkGetTextAtOffsetNewlines): Ditto. (testWebkitAtkGetTextAtOffsetTextarea): Ditto. (testWebkitAtkGetTextAtOffsetTextInput): Ditto. (testWebkitAtkGetTextInParagraphAndBodySimple): Ditto. (testWebkitAtkGetTextInParagraphAndBodyModerate): Ditto. (testWebkitAtkGetTextInTable): Ditto. (testWebkitAtkGetHeadersInTable): Ditto. (compAtkAttribute): Ditto. (atkAttributeSetAttributeNameHasValue): Ditto. (testWebkitAtkTextAttributes): Ditto. (testWebkitAtkTextSelections): Ditto. (testWebkitAtkGetExtents): Ditto. (testWebkitAtkLayoutAndDataTables): Ditto. (testWebkitAtkLinksWithInlineImages): Ditto. (testWebkitAtkHypertextAndHyperlinks): Ditto. (testWebkitAtkListsOfItems): Ditto. (testWebkitAtkTextChangedNotifications): Ditto. (main): Ditto.
1:53 PM UpdatingBugzilla edited by ddkilzer@apple.com
(diff)
1:47 PM Changeset in webkit [71051] by andersca@apple.com
  • 12 edits in trunk/WebKit2

Both the WebProcessConnection and PluginProcessConnection should have NPRemoteObjectMaps
https://bugs.webkit.org/show_bug.cgi?id=48775

Reviewed by Adam Roben.

  • PluginProcess/PluginControllerProxy.cpp:

(WebKit::PluginControllerProxy::windowScriptNPObject):
Ask for an NPObjectProxy for the window script NPObject.

  • PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::WebProcessConnection):
Initialize m_npRemoteObjectMap.

  • Shared/Plugins/NPObjectMessageReceiver.cpp:

(WebKit::NPObjectMessageReceiver::create):
Add a create member function that takes an NPObject.

(WebKit::NPObjectMessageReceiver::NPObjectMessageReceiver):
Retain the NPObject.

(WebKit::NPObjectMessageReceiver::~NPObjectMessageReceiver):
Release the NPObject.

  • Shared/Plugins/NPRemoteObjectMap.cpp:

(WebKit::generateNPObjectID):
Generate a unique 64-bit NPObject ID.

(WebKit::NPRemoteObjectMap::getOrCreateNPObjectProxy):
Add stub.

(WebKit::NPRemoteObjectMap::registerNPObject):
Create a NPObjectMessageReceiver and add it to the map.

  • WebKit2.xcodeproj/project.pbxproj:

Set the correct target for NPRuntime related files.

  • WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::PluginProcessConnection):
Initialize m_npRemoteObjectMap

  • WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::getWindowScriptNPObject):
Ask the real plug-in controller for the window script NPObject and register it.

1:44 PM Changeset in webkit [71050] by dimich@chromium.org
  • 11 edits in trunk/LayoutTests

[Chromium] Unreviewed, update test expectations.
Updated render tree dumps after http://trac.webkit.org/changeset/71043.

  • platform/chromium-linux/fast/blockflow/japanese-lr-text-expected.txt:
  • platform/chromium-linux/fast/blockflow/japanese-rl-text-expected.txt:
  • platform/chromium-win/fast/blockflow/basic-vertical-line-expected.txt:
  • platform/chromium-win/fast/blockflow/border-radius-clipping-vertical-lr-expected.txt:
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-lr-expected.txt:
  • platform/chromium-win/fast/blockflow/box-shadow-vertical-rl-expected.txt:
  • platform/chromium-win/fast/blockflow/english-lr-text-expected.txt:
  • platform/chromium-win/fast/blockflow/english-rl-text-expected.txt:
  • platform/chromium-win/fast/blockflow/japanese-lr-text-expected.txt:
  • platform/chromium-win/fast/blockflow/japanese-rl-text-expected.txt:
1:39 PM Changeset in webkit [71049] by Simon Fraser
  • 3 edits
    5 adds in trunk

2010-11-01 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt.

Multiple background properties are not retrievable via DOM method getComputedStyle
https://bugs.webkit.org/show_bug.cgi?id=23203

Return lists of values from getComputedStyle() for multiple backgrounds,
and multiple masks.

Test: fast/backgrounds/multiple-backgrounds-computed-style.html

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::fillSizeToCSSValue): (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
1:37 PM Changeset in webkit [71048] by sullivan@apple.com
  • 5 edits in trunk/WebKit2

https://bugs.webkit.org/show_bug.cgi?id=48774
minimFontSize preference needs to be exposed in WebKit2

Reviewed by Anders Carlsson.

  • UIProcess/WebPreferences.cpp:

(WebKit::WebPreferences::setMinimumFontSize):
New function, parallel to existing functions for other preferences.
(WebKit::WebPreferences::minimumFontSize):
Ditto.

  • UIProcess/WebPreferences.h:

Declared new function.

  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetMinimumFontSize):
New UI process wrapper for new function.
(WKPreferencesGetMinimumFontSize):
Ditto.

  • UIProcess/API/C/WKPreferences.h:

Declared new functions.

1:24 PM Changeset in webkit [71047] by mihaip@chromium.org
  • 101 edits
    100 copies
    2 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/block/positioning for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

1:20 PM Changeset in webkit [71046] by dimich@chromium.org
  • 2 edits in trunk/LayoutTests

[Chromium] Unreviewed update of test expectations.
svg-as-background.html now fails on debug mac 10.5 bots.

  • platform/chromium/test_expectations.txt:
12:30 PM Changeset in webkit [71045] by crogers@google.com
  • 13 edits
    1 move
    1 add
    1 delete in branches/audio/WebCore

audio branch: changes in preparation for landing in trunk

  • change name from Accelerate to VectorMath
  • AudioResamplerKernel cleanup
  • JSDOMWindowCustom.cpp: guard include and audioContext() method with ENABLE_WEB_AUDIO
  • V8Binding.cpp: remove AdjustAmountOfExternalAllocatedMemory() hack
12:16 PM Changeset in webkit [71044] by andersca@apple.com
  • 7 edits in trunk

Tear down the related WebProcessProxy when a WebContext is deallocated
https://bugs.webkit.org/show_bug.cgi?id=48769

Reviewed by John Sullivan.

WebKit2:

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::~WebContext):
Call WebProcessManager::contextWasDestroyed.

(WebKit::WebContext::didNavigateWithNavigationData):
(WebKit::WebContext::didPerformClientRedirect):
(WebKit::WebContext::didPerformServerRedirect):
(WebKit::WebContext::didUpdateHistoryTitle):
It is valid for a frame to have a null page here, if the frame has outlived
its page.

  • UIProcess/WebProcessManager.cpp:

(WebKit::WebProcessManager::contextWasDestroyed):
Remove the context from the map.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::~WebProcessProxy):
It's OK for the connection to be non-null here if the process goes away because
the context has been deallocated.

WebKitTools:

  • TestWebKitAPI/Tests/WebKit2/FailedLoad.cpp:

(TestWebKitAPI::didFailProvisionalLoadWithErrorForFrame):
We don't support empty URLs anymore, update test to expect a null URL instead.

12:07 PM Changeset in webkit [71043] by hyatt@apple.com
  • 26 edits
    3 copies
    8 adds in trunk

https://bugs.webkit.org/show_bug.cgi?id=48663

Reviewed by Dan Bernstein.

Make repaint invalidation work with vertical lines. Rewrite linesBoundingBox for RenderText and
RenderInline to give the correct rectangle back for vertical lines. Also patch linesVisibleOverflowBoundingBox
to give back the correct rectangle for vertical lines.

Fix bugs in the overflow accessors of InlineFlowBox when m_overflow was 0.

Patch hit testing to also account for "rt" and "bt" blocks.

WebCore:

  • rendering/InlineBox.cpp:

(WebCore::InlineBox::adjustForFlippedBlocksWritingMode):

  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::nodeAtPoint):

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::logicalLeftVisibleOverflow):
(WebCore::InlineFlowBox::logicalRightVisibleOverflow):
(WebCore::InlineFlowBox::bottomLayoutOverflow):
(WebCore::InlineFlowBox::rightLayoutOverflow):
(WebCore::InlineFlowBox::bottomVisualOverflow):
(WebCore::InlineFlowBox::rightVisualOverflow):

  • rendering/InlineTextBox.cpp:

(WebCore::InlineTextBox::nodeAtPoint):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::hitTestFloats):
(WebCore::RenderBlock::hitTestContents):

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computeRectForRepaint):
(WebCore::RenderBox::adjustForFlippedBlocksWritingMode):

  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp:

(WebCore::RenderInline::linesBoundingBox):
(WebCore::RenderInline::linesVisibleOverflowBoundingBox):
(WebCore::RenderInline::clippedOverflowRectForRepaint):

  • rendering/RenderText.cpp:

(WebCore::RenderText::linesBoundingBox):

LayoutTests:

  • fast/repaint/inline-horizontal-bt-overflow.html: Added.
  • fast/repaint/inline-vertical-lr-overflow.html: Added.
  • fast/repaint/inline-vertical-rl-overflow.html: Copied from fast/repaint/inline-vertical-rl-overflow.html.
  • platform/mac/fast/blockflow/background-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/background-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/basic-vertical-line-expected.txt:
  • platform/mac/fast/blockflow/border-image-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/border-image-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/border-radius-clipping-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.checksum:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.png:
  • platform/mac/fast/blockflow/border-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/box-shadow-vertical-lr-expected.txt:
  • platform/mac/fast/blockflow/box-shadow-vertical-rl-expected.txt:
  • platform/mac/fast/blockflow/english-lr-text-expected.txt:
  • platform/mac/fast/blockflow/english-rl-text-expected.txt:
  • platform/mac/fast/blockflow/japanese-lr-text-expected.txt:
  • platform/mac/fast/blockflow/japanese-rl-text-expected.txt:
  • platform/mac/fast/repaint/inline-horizontal-bt-overflow-expected.checksum: Added.
  • platform/mac/fast/repaint/inline-horizontal-bt-overflow-expected.png: Added.
  • platform/mac/fast/repaint/inline-horizontal-bt-overflow-expected.txt: Added.
  • platform/mac/fast/repaint/inline-vertical-lr-overflow-expected.checksum: Added.
  • platform/mac/fast/repaint/inline-vertical-lr-overflow-expected.png: Added.
  • platform/mac/fast/repaint/inline-vertical-lr-overflow-expected.txt: Added.
  • platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.checksum: Copied from platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.checksum.
  • platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.png: Copied from platform/mac/fast/repaint/inline-vertical-rl-overflow-expected.png.
11:53 AM Changeset in webkit [71042] by beidson@apple.com
  • 2 edits in trunk/WebKit2

Windows build fix.

  • UIProcess/win/WebView.cpp:
11:34 AM Changeset in webkit [71041] by beidson@apple.com
  • 50 edits
    8 adds in trunk

<rdar://problem/7660547> and https://bugs.webkit.org/show_bug.cgi?id=48699
Context menu support for WebKit 2.

Reviewed by Anders Carlsson.

WebCore:

Add a "showContextMenu" call to Chrome/ChromeClient:

  • loader/EmptyClients.h:

(WebCore::EmptyChromeClient::showContextMenu):

  • page/Chrome.cpp:

(WebCore::Chrome::showContextMenu):

  • page/Chrome.h:
  • page/ChromeClient.h:

Add a "platform description to ContextMenuItem" creator, will be needed by WebKit2

  • platform/ContextMenu.h:
  • platform/mac/ContextMenuMac.mm:

(WebCore::contextMenuItemVector):

  • platform/mac/ContextMenuItemMac.mm:

(WebCore::ContextMenuItem::checked): WebKit2 ports now need this implemented.

Stub these out to keep their WK2 ports building:

  • platform/qt/ContextMenuItemQt.cpp:

(WebCore::ContextMenuItem::checked):

  • platform/qt/ContextMenuQt.cpp:

(WebCore::contextMenuItemVector):

  • platform/win/ContextMenuItemWin.cpp:

(WebCore::ContextMenuItem::checked):

  • platform/win/ContextMenuWin.cpp:

(WebCore::contextMenuItemVector):

WebKit/chromium:

  • src/ChromeClientImpl.h:

(WebKit::ChromeClientImpl::showContextMenu):

WebKit/efl:

  • WebCoreSupport/ChromeClientEfl.h:

(WebCore::ChromeClientEfl::showContextMenu):

WebKit/gtk:

  • WebCoreSupport/ChromeClientGtk.h:

(WebKit::ChromeClient::showContextMenu):

WebKit/haiku:

  • WebCoreSupport/ChromeClientHaiku.h:

(WebCore::ChromeClientHaiku::showContextMenu):

WebKit/mac:

  • WebCoreSupport/WebChromeClient.h:

(WebChromeClient::showContextMenu):

WebKit/qt:

  • WebCoreSupport/ChromeClientQt.h:

(WebCore::ChromeClientQt::showContextMenu):

WebKit/win:

  • WebCoreSupport/WebChromeClient.h:

(WebChromeClient::showContextMenu):

WebKit/wx:

  • WebKitSupport/ChromeClientWx.h:

(WebCore::ChromeClientWx::showContextMenu):

WebKit2:

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • win/WebKit2.vcproj:
  • Shared/WebContextMenuItem.cpp: Added.

(WebKit::WebContextMenuItem::WebContextMenuItem):
(WebKit::WebContextMenuItem::encode):
(WebKit::WebContextMenuItem::decode):
(WebKit::kitItems):

  • Shared/WebContextMenuItem.h: Added.
  • UIProcess/API/mac/PageClientImpl.h:
  • UIProcess/API/mac/PageClientImpl.mm:

(WebKit::PageClientImpl::createContextMenuProxy):

  • UIProcess/API/qt/qwkpage.cpp:

(QWKPagePrivate::createContextMenuProxy): Stub to keep the build working.

  • UIProcess/API/qt/qwkpage_p.h:
  • UIProcess/PageClient.h:
  • UIProcess/WebContextMenuProxy.cpp: Added.

(WebKit::WebContextMenuProxy::~WebContextMenuProxy):
(WebKit::WebContextMenuProxy::WebContextMenuProxy):

  • UIProcess/WebContextMenuProxy.h: Added.
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::showContextMenu): Shows the menu after receiving the message from the WebProcess.
(WebKit::WebPageProxy::contextMenuItemSelected): Sends info about the selected menu item to the WebProcess

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/mac/WebContextMenuProxyMac.h: Added.

(WebKit::WebContextMenuProxyMac::create):

  • UIProcess/mac/WebContextMenuProxyMac.mm: Added.

(+[WebMenuTarget sharedMenuTarget]):
(-[WebMenuTarget WebKit::]):
(-[WebMenuTarget setMenuProxy:WebKit::]):
(-[WebMenuTarget forwardContextMenuAction:]):
(WebKit::WebContextMenuProxyMac::WebContextMenuProxyMac):
(WebKit::WebContextMenuProxyMac::~WebContextMenuProxyMac):
(WebKit::WebContextMenuProxyMac::contextMenuItemSelected):
(WebKit::populateNSMenu):
(WebKit::nsMenuItemVector):
(WebKit::WebContextMenuProxyMac::populate):
(WebKit::WebContextMenuProxyMac::showContextMenu):
(WebKit::WebContextMenuProxyMac::hideContextMenu):

  • UIProcess/win/WebView.cpp:

(WebKit::WebView::createContextMenuProxy): Stub for now.

  • UIProcess/win/WebView.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::showContextMenu):

  • WebProcess/WebCoreSupport/WebChromeClient.h:
  • WebProcess/WebCoreSupport/WebContextMenuClient.cpp:

(WebKit::WebContextMenuClient::getCustomMenuFromDefaultItems): Return the WebCore-provided default

menu for now. Embedding app customization will come later (see bug 48720)

  • WebProcess/WebPage/WebContextMenu.cpp: Added.

(WebKit::WebContextMenu::WebContextMenu):
(WebKit::WebContextMenu::~WebContextMenu):
(WebKit::WebContextMenu::show):
(WebKit::WebContextMenu::itemSelected):

  • WebProcess/WebPage/WebContextMenu.h: Added.

(WebKit::WebContextMenu::create):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::contextMenu):
(WebKit::handleMouseEvent): Pass right mouse presses along to WebCore as both a mouse event and context menu event,

the same way WebKit1 does.

(WebKit::WebPage::didSelectItemFromActiveContextMenu):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:
11:30 AM Changeset in webkit [71040] by pfeldman@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: Chromium build fix.

  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::WebDevToolsAgentImpl):
11:18 AM Changeset in webkit [71039] by eric.carlson@apple.com
  • 13 edits
    2 adds in trunk

2010-11-01 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Seeking by very small increment doesn't generate 'seeked' event
https://bugs.webkit.org/show_bug.cgi?id=48530

Test: media/video-seek-by-small-increment.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::seek): Ask the media engine for its closest time value so we can avoid asking it to seek to the current time.
  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::mediaTimeForTimeValue): New.
  • platform/graphics/MediaPlayer.h:
  • platform/graphics/MediaPlayerPrivate.h: (WebCore::MediaPlayerPrivateInterface::mediaTimeForTimeValue): Ditto.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::mediaTimeForTimeValue): Return the closest value in the movie's time scale.
  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp: (WebCore::MediaPlayerPrivateQuickTimeVisualContext::mediaTimeForTimeValue): Ditto
  • platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.h:
  • platform/graphics/win/QTMovie.cpp: (QTMovie::timeScale): Return the movie's time scale.
  • platform/graphics/win/QTMovie.h:

2010-11-01 Eric Carlson <eric.carlson@apple.com>

Reviewed by Adam Roben.

Seeking by very small increment doesn't generate 'seeked' event
https://bugs.webkit.org/show_bug.cgi?id=48530

  • media/video-seek-by-small-increment-expected.txt: Added.
  • media/video-seek-by-small-increment.html: Added.
  • platform/qt/Skipped: Skip new test as it fails.
11:14 AM Changeset in webkit [71038] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: Chromium build fix. Adding missing transitive dependency.

  • inspector/InspectorResourceAgent.cpp:
11:09 AM Changeset in webkit [71037] by mihaip@chromium.org
  • 139 edits
    138 copies
    2 adds in trunk/LayoutTests

2010-11-01 Mihai Parparita <mihaip@chromium.org>

Update the pixel expectations for fast/block/margin-collapse for the Mac port.

For the tests that currently fail on Snow Leopard with --pixel-tests
--tolerance 0, this moves the current pixel results to mac-leopard and
adds new Snow Leopard-specific results to platform/mac.

List of files omitted due to length but composed entirely of moves from
platform/mac to platform/mac-leopard and newly added files in
platform/mac.

11:02 AM Changeset in webkit [71036] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Removing references to removed file from project files.

  • WebCore.gypi:
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
10:56 AM Changeset in webkit [71035] by pfeldman@chromium.org
  • 38 edits
    4 deletes in trunk

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: remove old resources panel.
https://bugs.webkit.org/show_bug.cgi?id=45657

  • CMakeLists.txt:
  • English.lproj/localizedStrings.js:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSInjectedScriptHostCustom.cpp:
  • inspector/InjectedScriptHost.cpp:
  • inspector/Inspector.idl:
  • inspector/InspectorCSSStore.cpp:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::releaseFrontendLifetimeAgents): (WebCore::InspectorController::populateScriptObjects): (WebCore::InspectorController::unbindAllResources): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::frameDetachedFromParent): (WebCore::InspectorController::didLoadResourceFromMemoryCache): (WebCore::InspectorController::identifierForInitialRequest): (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::markResourceAsCached): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didReceiveContentLength): (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading): (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported): (WebCore::InspectorController::didCreateWebSocket): (WebCore::InspectorController::willSendWebSocketHandshakeRequest): (WebCore::InspectorController::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorController::didCloseWebSocket):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontendHost.cpp:
  • inspector/InspectorResource.cpp: Removed.
  • inspector/InspectorResource.h: Removed.
  • inspector/InspectorResourceAgent.cpp: (WebCore::InspectorResourceAgent::setOverrideContent):
  • inspector/InspectorResourceAgent.h:
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.addMessage): (WebInspector.ConsoleView.prototype.clearMessages):
  • inspector/front-end/ExtensionServer.js: (WebInspector.ExtensionServer.prototype._onRevealAndSelectResource):
  • inspector/front-end/Images/resourcesSilhouette.png: Removed.
  • inspector/front-end/Resource.js:
  • inspector/front-end/ResourceManager.js: (WebInspector.ResourceManager.getContent):
  • inspector/front-end/ResourcesPanel.js: Removed.
  • inspector/front-end/Settings.js:
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.get toolbarItemLabel):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector._createPanels): (WebInspector.get networkResources): (WebInspector.forAllResources): (WebInspector.resourceForURL): (WebInspector.doLoadedDone.populateInspectorState): (WebInspector.openResource): (WebInspector.domContentEventFired): (WebInspector.loadEventFired): (WebInspector.reset): (WebInspector._choosePanelToShowSourceLine):

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: remove old resources panel.
https://bugs.webkit.org/show_bug.cgi?id=45657

  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::setApuAgentEnabled):
  • src/WebDevToolsAgentImpl.h:

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: remove old resources panel.
https://bugs.webkit.org/show_bug.cgi?id=45657

  • http/tests/inspector/inspector-test2.js:
  • http/tests/inspector/resource-har-conversion.html:
  • http/tests/inspector/resource-parameters.html:
  • inspector/audits-panel-functional.html:
  • inspector/extensions-test.js:
  • inspector/report-protocol-errors-expected.txt:
  • inspector/report-protocol-errors.html:
  • inspector/styles-source-offsets.html:
10:35 AM Changeset in webkit [71034] by zmo@google.com
  • 5 edits in trunk/WebCore

2010-10-28 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Fix enum naming style violation in GraphicsContext3D
https://bugs.webkit.org/show_bug.cgi?id=48441

  • platform/graphics/GraphicsContext3D.cpp: (WebCore::GraphicsContext3D::extractImageData): (WebCore::GraphicsContext3D::extractTextureData): (WebCore::doPacking): (WebCore::GraphicsContext3D::packPixels):
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/cg/GraphicsContext3DCG.cpp: (WebCore::getSourceDataFormat): (WebCore::GraphicsContext3D::getImageData):
  • platform/graphics/skia/GraphicsContext3DSkia.cpp: (WebCore::GraphicsContext3D::getImageData):
9:51 AM Changeset in webkit [71033] by Adam Roben
  • 6 edits in trunk/WebKit2

Cancel main resource loads after we hand them off to the media engine

This is the WebKit2 equivalent of r51104. Clearly this code should be
moved to a cross-platform location someday.

Fixes <http://webkit.org/b/48561> <rdar://problem/8606679> Assertion
failure in DocumentLoader::commitData when loading a media document in
WebKit2

Reviewed by Eric Carlson.

  • WebProcess/WebCoreSupport/WebErrors.h: Added pluginWillHandleLoadError.
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::committedLoad): Cancel the main resource load
after handing off the load to the media engine. This code originally
came from -[WebHTMLRepresentation receivedData:withDataSource:].
(WebKit::WebFrameLoaderClient::pluginWillHandleLoadError): Call through to WebErrors.
(WebKit::WebFrameLoaderClient::shouldFallBack): Implemented. We fall
back for all errors except when the load was cancelled or we handed it
off to the media engine or a plugin.

  • WebProcess/WebCoreSupport/mac/WebErrorsMac.mm:

(WebKit::pluginWillHandleLoadError): Implemented.

  • WebProcess/WebCoreSupport/qt/WebErrorsQt.cpp:

(WebKit::pluginWillHandleLoadError): Stubbed out.

  • WebProcess/WebCoreSupport/win/WebErrorsWin.cpp:

(WebKit::pluginWillHandleLoadError): Implemented.

9:50 AM Changeset in webkit [71032] by Adam Roben
  • 3 edits in trunk/WebKit/win

Cancel main resource loads after we hand them off to the media engine

This is the Windows equivalent of r51104. Clearly this code should be
moved to a cross-platform location someday.

Fixes <http://webkit.org/b/48531> <rdar://problem/8606635> Assertion
failure in DocumentLoader::commitData when loading a media document in
WebKit1 on Windows

Reviewed by Dan Bernstein.

  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::committedLoad): Cancel the main resource load
after handing off the load to the media engine. This code originally
came from -[WebHTMLRepresentation receivedData:withDataSource:].

  • WebFrame.cpp:

(WebFrame::shouldFallBack): Don't fall back when handing the resource
load off to the media engine or a plugin. Added error domain checking
so that we don't rely on error codes being unique.

9:00 AM Changeset in webkit [71031] by Martin Robinson
  • 4 edits in trunk

2010-11-01 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] Pasting markup into Thunderbird compose window produces no text
https://bugs.webkit.org/show_bug.cgi?id=43737

Include a content-type meta tag prefix on all clipboard markup. Programs like
Thunderbird expect this meta tag and will not paste anything unless it is there.

This is covered by changes to WebKit/gtk/tests/testcopyandpaste.c. This patch was
written in such a way as to not affect layout test results, otherwise there
would be many new GTK+-specific results that say "FAIL").

  • platform/gtk/PasteboardHelper.cpp: (WebCore::removeMarkupPrefix): Added this helper which removes the prefix when found on incoming clipboard and drag-and-drop text. (WebCore::PasteboardHelper::getClipboardContents): Remove the meta tag prefix. (WebCore::PasteboardHelper::fillSelectionData): Add the meta tag prefix. (WebCore::PasteboardHelper::fillDataObjectFromDropData): Remove the meta tag prefix

2010-11-01 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] Pasting markup into Thunderbird compose window produces no text
https://bugs.webkit.org/show_bug.cgi?id=43737

Added a test verifying that the meta tag prefix on markup data exists.

  • tests/testcopyandpaste.c: (load_status_cb): Add a check for the meta tag prefix.
8:50 AM Changeset in webkit [71030] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-01 Søren Gjesse <sgjesse@chromium.org>

Reviewed by Andreas Kling.

Fix warning when compiling the chromium port of DumpRenderShell
with clang.
https://bugs.webkit.org/show_bug.cgi?id=48414

  • DumpRenderTree/chromium/TestShell.h: (TestShell::javaScriptFlagsForLoad):
8:33 AM Changeset in webkit [71029] by andreas.kling@nokia.com
  • 3 edits in trunk/WebKit2

2010-11-01 Andreas Kling <kling@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt][WK2] Cleanup viewportAttributesForSize API

Change QSize argument to a const QSize&.

  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::viewportAttributesForSize):
  • UIProcess/API/qt/qwkpage.h:
8:27 AM Changeset in webkit [71028] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] DRT sideeffect revealed by r71004
https://bugs.webkit.org/show_bug.cgi?id=48758

  • platform/qt/Skipped: fast/dom/nodesFromRect-basic.html added until fix.
8:23 AM Changeset in webkit [71027] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

2010-11-01 Justin Schuh <jschuh@chromium.org>

Reviewed by Dirk Schulze.

Check for NULL node in SVGUseElement::associateInstancesWithShadowTreeElements loop
https://bugs.webkit.org/show_bug.cgi?id=48741

  • svg/custom/use-on-use-with-child-and-empty-target-expected.txt: Added.
  • svg/custom/use-on-use-with-child-and-empty-target.svg: Added.

2010-11-01 Justin Schuh <jschuh@chromium.org>

Reviewed by Dirk Schulze.

Check for NULL node in SVGUseElement::associateInstancesWithShadowTreeElements loop
https://bugs.webkit.org/show_bug.cgi?id=48741

Test: svg/custom/use-on-use-with-child-and-empty-target.svg

  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::associateInstancesWithShadowTreeElements):
8:11 AM BuildingOnWindows edited by Adam Roben
Added some more DLLs that need to be rebased (diff)
8:08 AM BuildingOnWindows edited by Adam Roben
Update instructions for rebasing libphp4.dll (diff)
8:05 AM Changeset in webkit [71026] by mario@webkit.org
  • 5 edits
    2 adds in trunk

2010-11-01 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Martin Robinson.

[Gtk] AtkHyperlink needs to be implemented
https://bugs.webkit.org/show_bug.cgi?id=33785

Implemented the AtkHyperlink/AtkHypertext stuff in the GTK port.

Even thought these kind of tasks are usually about implementing an
interface, in this case it was needed to implement some more
things due to the fact that AtkHyperlink is neither an interface
nor an AtkObject, but just an abstract class child of GObject that
needs to be redefined in a specific subclass of that one. On top
of that, it was needed to implement the AtkAction interface as
well for that new class, so exposed hyperlinks can work as
expected with Assistive Technologies based on ATK.

Furthermore, as there's no mechanism to get an instance of that
AtkHyperlink other than doing it through an AtkObject implementing
the AtkHypertext interface, it was needed to also add the code to
implement that one, so it makes possible to ask for the
'hyperlinks' under an 'hypertext' accessible object.

Finally, to complete the implementation of all this stuff and make
it all consistent, it was needed as well to implement the
AtkHyperlinkImpl interface (providing just one method only) so
that allows retrieving the AtkHyperlink object associated to an
AtkObject implementing such an interface.

  • GNUmakefile.am: Add the new WebKitAccessibleHyperlink.[h|cpp] files to the sources list for the GTK port.
  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (core): New function to return the core accessibility object related to an AtkObject implementing the AtkHypertext interface. (webkitAccessibleHypertextGetLink): New, part of the implementation of the AtkHypertext interface. (webkitAccessibleHypertextGetNLinks): New, likewise. (webkitAccessibleHypertextGetLinkIndex): New, likewise. (atkHypertextInterfaceInit): New, initialize the AtkHypertext interface. (webkitAccessibleHyperlinkImplGetHyperlink): New, part of the implementation of the AtkHyperlinkImpl interface. (atkHyperlinkImplInterfaceInit): New, initialize the AtkHyperlinkImpl interface. (GetAtkInterfaceTypeFromWAIType): Add ATK_TYPE_HYPERTEXT and ATK_TYPE_HYPERLINK_IMPL to the list of recognized types. (getInterfaceMaskFromObject): Decide when it's needed to add WAI_HYPERTEXT and WAI_HYPERLINK to the interface mask.
  • accessibility/gtk/WebKitAccessibleHyperlink.h: Added.
  • accessibility/gtk/WebKitAccessibleHyperlink.cpp: Added. (returnString): Return (const char*) from String variables. (core): Return the associated AccessibilityObject. (webkitAccessibleHyperlinkActionDoAction): Part of the implementation of the AtkAction interface. (webkitAccessibleHyperlinkActionGetNActions): Likewise. (webkitAccessibleHyperlinkActionGetDescription): Likewise. (webkitAccessibleHyperlinkActionGetKeybinding): Likewise. (webkitAccessibleHyperlinkActionGetName): Likewise. (atkActionInterfaceInit): Initialize the AtkAction interface. (getRangeLengthForObject): Returns the length for a given Range, considering special cases (e.g. List item markers). (webkitAccessibleHyperlinkGetURI): Implementation of one of the methods in the AtkHyperlink abstract class. (webkitAccessibleHyperlinkGetObject): Likewise. (webkitAccessibleHyperlinkGetStartIndex): Likewise. (webkitAccessibleHyperlinkGetEndIndex): Likewise. (webkitAccessibleHyperlinkIsValid): Likewise. (webkitAccessibleHyperlinkGetNAnchors): Likewise. (webkitAccessibleHyperlinkIsSelectedLink): Likewise. (webkitAccessibleHyperlinkGetProperty): Getter method. (webkitAccessibleHyperlinkSetProperty): Setter method. (webkitAccessibleHyperlinkFinalize): Finalize method. (webkitAccessibleHyperlinkClassInit): Initialize the WebKitAccessibleHyperlink class. (webkitAccessibleHyperlinkInit): Initialize the WebKitAccessibleHyperlink instance. (webkitAccessibleHyperlinkGetType): Implementation of the get_type() function for the WebKitAccessibleHyperlink class. (webkitAccessibleHyperlinkNew): Returns an instance of the class associated to a given AtkHyperlinkImpl object. (webkitAccessibleHyperlinkGetAccessibilityObject): Public function returning the AccessibilityObject associated to the instance of the WebKitAccessibleHyperlink class.

2010-11-01 Mario Sanchez Prada <msanchez@igalia.com>

Reviewed by Martin Robinson.

[Gtk] AtkHyperlink needs to be implemented
https://bugs.webkit.org/show_bug.cgi?id=33785

New test to ensure the AtkHypertext/AtkHyperlink stuff works.

  • tests/testatk.c: (testWebkitAtkHypertextAndHyperlinks): New test, checking both the implementation of the AtkHypertext interface, the subclass of the AtkHyperlink abstract class, and the implementation of the AtkAction interface in that subclass. (main): Added the new unit test.
7:55 AM Changeset in webkit [71025] by Adam Roben
  • 2 edits in trunk/LayoutTests

Add two more failing tests to the WebKit2 Skipped file

  • platform/mac-wk2/Skipped:
7:38 AM Changeset in webkit [71024] by Adam Roben
  • 2 edits in trunk/LayoutTests

Add some failing/timing out tests to the Windows Skipped file

  • platform/win/Skipped:
7:32 AM Changeset in webkit [71023] by Adam Roben
  • 2 edits in trunk/WebKitTools

Fix typo from r71022

  • BuildSlaveSupport/build.webkit.org-config/config.json:
7:29 AM Changeset in webkit [71022] by Adam Roben
  • 2 edits in trunk/WebKitTools

Trigger the Windows Release WebKit2 tests when a Release build finishes, not when a Debug build finishes

Fixes <http://webkit.org/b/48754> Windows Release WebKit2 tests are
triggered at the wrong time

Reviewed by Sam Weinig.

  • BuildSlaveSupport/build.webkit.org-config/config.json: Fixed

triggerable name and trigger.

7:28 AM Changeset in webkit [71021] by jorlow@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-01 Jeremy Orlow <jorlow@chromium.org>

Remove test previous patch fixed.

  • platform/chromium/test_expectations.txt:
7:26 AM Changeset in webkit [71020] by jorlow@chromium.org
  • 3 edits in trunk/LayoutTests

2010-11-01 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Fix IndexedDB layout test
https://bugs.webkit.org/show_bug.cgi?id=48644

This test needs to be async. Also clean up a few other things like
using a logger div rather than document.write.

  • storage/indexeddb/tutorial-expected.txt:
  • storage/indexeddb/tutorial.html:
5:56 AM Changeset in webkit [71019] by satish@chromium.org
  • 6 edits in trunk

2010-11-01 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

This is the last part of a 4-sided patch for the language attribute
in speech text buttons. This patch removes what now is dead code
and re-enables the layout test introduced by bug 47089.
https://bugs.webkit.org/show_bug.cgi?id=47420

  • platform/chromium/test_expectations.txt:

2010-11-01 Leandro Gracia Gil <leandrogracia@google.com>

Reviewed by Jeremy Orlow.

This is the last part of a 4-sided patch for the language attribute
in speech text buttons. This patch removes what now is dead code
and re-enables the layout test introduced by bug 47089.
https://bugs.webkit.org/show_bug.cgi?id=47420

  • public/WebSpeechInputControllerMock.h:
  • src/WebSpeechInputControllerMockImpl.cpp:
  • src/WebSpeechInputControllerMockImpl.h:
5:21 AM WinCE edited by Patrick Gansterer
(diff)
4:28 AM Changeset in webkit [71018] by mario@webkit.org
  • 2 edits in trunk/WebKitTools

2010-11-01 Mario Sanchez Prada <msanchez@igalia.com>

Unreviewed. Adding my IRC nickname to the list of committers.

  • Scripts/webkitpy/common/config/committers.py:
4:09 AM Changeset in webkit [71017] by pfeldman@chromium.org
  • 6 edits in trunk

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: simplify inspector styles test to make all bots happy.
(drive-by fix for overriden inspector styles in network panel).

  • inspector/InspectorController.h:
  • inspector/front-end/networkPanel.css:

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: simplify styles test to make all bots happy.

  • inspector/styles-source-lines-inline-expected.txt:
  • inspector/styles-source-lines-inline.html:
3:25 AM Changeset in webkit [71016] by pfeldman@chromium.org
  • 3 edits in trunk

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Updating inspector test expectation as a follow up to r71010.

  • inspector/styles-source-lines-inline-expected.txt:
2:12 AM Changeset in webkit [71015] by Nikolas Zimmermann
  • 12 edits in trunk/LayoutTests

2010-11-01 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed.

Enable StrictTypeChecking for all types using SVG(Animated)PropertyTearOff
https://bugs.webkit.org/show_bug.cgi?id=48715

Forgot to land some changes in my previous commit.

  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-cx-prop.js:
  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-cy-prop.js:
  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-r-prop.js:
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-cx-prop.js:
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-cy-prop.js:
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-rx-prop.js:
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-ry-prop.js:
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-height-prop.js:
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-width-prop.js:
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-x-prop.js:
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-y-prop.js:
1:57 AM Changeset in webkit [71014] by Nikolas Zimmermann
  • 18 edits
    3 moves
    27 adds
    2 deletes in trunk

2010-10-31 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Enable StrictTypeChecking for all types using SVG(Animated)PropertyTearOff
https://bugs.webkit.org/show_bug.cgi?id=48715

Enable StrictTypeChecking for all readwrite attributes and function calls of all types using the new SVG(Animated)PropertyTearOff concept.
Also enable RequiresAllArguments=Raise for all function calls of these new types. Cover this functionality by tests for each of them.

When testing SVGLength it became obvious that quite a lot of functionality was missing.
Conversion from relative arguments (percentage, ems, exs) to user specified units was missing, added that functionality and tested it.
User units to pt conversion was wrong, fixed that.

Tests: svg/dom/SVGAnimatedAngle.html

svg/dom/SVGAnimatedBoolean.html
svg/dom/SVGAnimatedEnumeration.html
svg/dom/SVGAnimatedInteger.html
svg/dom/SVGAnimatedLength.html
svg/dom/SVGAnimatedLengthList.html
svg/dom/SVGAnimatedRect.html
svg/dom/SVGLength-px-with-context.html
svg/dom/SVGLength-px.html
svg/dom/SVGLength.html

  • bindings/js/JSSVGLengthCustom.cpp: These SVGLength methods now take an ExceptionCode parameter, adapt to that. (WebCore::JSSVGLength::value): (WebCore::JSSVGLength::setValue): (WebCore::JSSVGLength::convertToSpecifiedUnits):
  • bindings/scripts/CodeGeneratorObjC.pm: Remove FIXMEs, pass real context instead off null pointer. DOMSVGLength::value is functional now.
  • bindings/v8/custom/V8SVGLengthCustom.cpp: These SVGLength methods now take an ExceptionCode parameter, adapt to that. (WebCore::V8SVGLength::valueAccessorGetter): (WebCore::V8SVGLength::valueAccessorSetter): (WebCore::V8SVGLength::convertToSpecifiedUnitsCallback):
  • rendering/style/SVGRenderStyle.h: Adapt to SVGLength API changes. (WebCore::SVGRenderStyle::initialBaselineShiftValue): (WebCore::SVGRenderStyle::initialKerning): (WebCore::SVGRenderStyle::initialStrokeDashOffset): (WebCore::SVGRenderStyle::initialStrokeWidth):
  • svg/SVGAngle.idl: Add StrictTypeChecking, RequiresAllArguments=Raise to all methods.
  • svg/SVGAnimatedBoolean.idl: Remove StrictTypeChecking from animVal, it's readonly so this is useless.
  • svg/SVGAnimatedEnumeration.idl: Add StrictTypeChecking to all attributes.
  • svg/SVGAnimatedInteger.idl: Ditto.
  • svg/SVGLength.cpp: Implement all missing features of SVGLength. Conform to our style guide (s/.0f etc) (WebCore::lengthTypeToString): (WebCore::SVGLength::SVGLength): (WebCore::SVGLength::value): Throw NOT_SUPPORTED_ERR when trying to obtain to relative units, when no context is given (SVGLength created by createSVGLength()). (WebCore::SVGLength::setValue): Ditto, for the setter. (WebCore::SVGLength::valueAsPercentage): Use m_valueInSpecifiedUnits instead of valueInSpecifiedUnits(). (WebCore::SVGLength::setValueAsString): Throw SYNTAX_ERR if parsing failed. (WebCore::SVGLength::valueAsString): Use makeString() instead of String and operator+. (WebCore::SVGLength::newValueSpecifiedUnits): Add ExceptionCode parameter, throw NOT_SUPPORTED_ERR, if the given unit type is invalid. (WebCore::SVGLength::convertToSpecifiedUnits): Ditto. (WebCore::SVGLength::determineViewport): Added helper function used by convertValue*Percentage*. (WebCore::SVGLength::convertValueFromUserUnitsToPercentage): Add new helper functions, used by value/setValue. (WebCore::SVGLength::convertValueFromPercentageToUserUnits): Ditto. (WebCore::SVGLength::convertValueFromUserUnitsToEMS): Ditto. (WebCore::SVGLength::convertValueFromEMSToUserUnits): Ditto. (WebCore::SVGLength::convertValueFromUserUnitsToEXS): Ditto. (WebCore::SVGLength::convertValueFromEXSToUserUnits): Ditto. (WebCore::SVGLength::fromCSSPrimitiveValue): Adapt to newValueSpecifiedUnits API change. (WebCore::SVGLength::toCSSPrimitiveValue): Remove default switch case.
  • svg/SVGLength.h: (WebCore::SVGLength::valueInSpecifiedUnits): Inlined. (WebCore::SVGLength::setValueInSpecifiedUnits): Ditto. (WebCore::SVGLength::isRelative):
  • svg/SVGLength.idl: Add StrictTypeChecking for all attributes/functions and RequiresAllArguments=Raise for all functions.
  • svg/SVGLengthList.cpp: Adapt to SVGLength API changes. (WebCore::SVGLengthList::parse):

2010-10-31 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Rob Buis.

Enable StrictTypeChecking for all types using SVG(Animated)PropertyTearOff
https://bugs.webkit.org/show_bug.cgi?id=48715

  • svg/dom/SVGAngle-expected.txt:
  • svg/dom/SVGAnimatedAngle-expected.txt: Added.
  • svg/dom/SVGAnimatedAngle.html: Added.
  • svg/dom/SVGAnimatedBoolean-expected.txt: Copied from LayoutTests/svg/dom/SVGExternalResourcesRequired-expected.txt.
  • svg/dom/SVGAnimatedBoolean.html: Copied from LayoutTests/svg/dom/SVGExternalResourcesRequired.html.
  • svg/dom/SVGAnimatedEnumeration-expected.txt: Added.
  • svg/dom/SVGAnimatedEnumeration.html: Added.
  • svg/dom/SVGAnimatedInteger-expected.txt: Added.
  • svg/dom/SVGAnimatedInteger.html: Added.
  • svg/dom/SVGAnimatedLength-expected.txt: Added.
  • svg/dom/SVGAnimatedLength.html: Added.
  • svg/dom/SVGAnimatedLengthList-expected.txt: Added.
  • svg/dom/SVGAnimatedLengthList.html: Added.
  • svg/dom/SVGAnimatedRect-expected.txt: Added.
  • svg/dom/SVGAnimatedRect.html: Added.
  • svg/dom/SVGExternalResourcesRequired-expected.txt: Removed.
  • svg/dom/SVGExternalResourcesRequired.html: Removed.
  • svg/dom/SVGLength-expected.txt: Added.
  • svg/dom/SVGLength-px-expected.txt: Added.
  • svg/dom/SVGLength-px-with-context-expected.txt: Added.
  • svg/dom/SVGLength-px-with-context.html: Added.
  • svg/dom/SVGLength-px.html: Added.
  • svg/dom/SVGLength.html: Added.
  • svg/dom/script-tests/SVGAngle.js:
  • svg/dom/script-tests/SVGAnimatedAngle.js: Added.
  • svg/dom/script-tests/SVGAnimatedBoolean.js: Copied from LayoutTests/svg/dom/script-tests/SVGExternalResourcesRequired.js.
  • svg/dom/script-tests/SVGAnimatedEnumeration.js: Added.
  • svg/dom/script-tests/SVGAnimatedInteger.js: Added.
  • svg/dom/script-tests/SVGAnimatedLength.js: Added.
  • svg/dom/script-tests/SVGAnimatedLengthList.js: Added.
  • svg/dom/script-tests/SVGAnimatedRect.js: Added.
  • svg/dom/script-tests/SVGExternalResourcesRequired.js: Removed.
  • svg/dom/script-tests/SVGLength-px-with-context.js: Added. Tests various operations on a SVGLength object obtained via rectElement.x.baseVal.
  • svg/dom/script-tests/SVGLength-px.js: Added. Tests various operations on a SVGLength object created via svgElement.createSVGLength().
  • svg/dom/script-tests/SVGLength.js: Added.
  • svg/dom/svg-length-units-expected.txt: Removed.
  • svg/dom/svg-length-units.html: Removed. Test is obsolete now, as SVGLength* covers this in detail.
  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-cx-prop.js: (executeTest): Fixed StrictTypeChecking correctness, don't assign strings as SVGLength value.
  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-cy-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-r-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-cx-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-cy-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-rx-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-ry-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-height-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-width-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-x-prop.js: (executeTest): Ditto.
  • svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-y-prop.js: (executeTest): Ditto.
12:57 AM Changeset in webkit [71013] by rolandsteiner@chromium.org
  • 2 edits in trunk/LayoutTests

2010-10-31 Roland Steiner <rolandsteiner@chromium.org>

Unreviewed: add failing SVG tests to Chromium test_expectations.txt.

WebKit roll 70949:71000 caused failures within 4 SVG tests on all platforms.
Bot and server outages make analysis difficult, thus adding to
Chromium test_expectations.txt for the time being.

  • platform/chromium/test_expectations.txt:
12:30 AM Changeset in webkit [71012] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

2010-11-01 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Fixing test expectation for r71010.

  • inspector/console-log-syntax-error-expected.txt:
12:19 AM Changeset in webkit [71011] by commit-queue@webkit.org
  • 5 edits in trunk/LayoutTests

2010-11-01 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r71002.
http://trac.webkit.org/changeset/71002
https://bugs.webkit.org/show_bug.cgi?id=48743

Makes Leopard and GTK bots red. (Requested by pfeldman on
#webkit).

  • platform/chromium-gpu/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac-leopard/Skipped:
  • platform/qt/Skipped:
12:07 AM Changeset in webkit [71010] by pfeldman@chromium.org
  • 16 edits in trunk

2010-10-31 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: unhide new network and resources panels.
https://bugs.webkit.org/show_bug.cgi?id=48725

  • English.lproj/localizedStrings.js:
  • inspector/InspectorController.h:
  • inspector/front-end/NetworkPanel.js: (WebInspector.NetworkPanel):
  • inspector/front-end/Settings.js:
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.reset): (WebInspector.StoragePanel.prototype.addResourceToFrame): (WebInspector.StoragePanel.prototype.addDocumentURL): (WebInspector.StoragePanel.prototype.showFileSystem): (WebInspector.FileSystemTreeElement): (WebInspector.FileSystemTreeElement.prototype.get itemURL): (WebInspector.FileSystemTreeElement.prototype.onselect):
  • inspector/front-end/inspector.js: (WebInspector.updateResource):
  • inspector/front-end/networkPanel.css: (#network-close-button): (.network.panel.viewing-resource #network-close-button):

2010-10-31 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: unhide new network and resources panels.
https://bugs.webkit.org/show_bug.cgi?id=48725

  • http/tests/inspector/console-xhr-logging-expected.txt:
  • http/tests/inspector/resource-parameters-expected.txt:
  • inspector/console-log-syntax-error-expected.txt:
  • inspector/elements-panel-styles-expected.txt:

2010-10-31 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: unhide new network and resources panels.
https://bugs.webkit.org/show_bug.cgi?id=48725

  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::setApuAgentEnabled):
Note: See TracTimeline for information about the timeline view.