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

Timeline



Nov 26, 2011:

7:05 PM Changeset in webkit [101199] by kevino@webkit.org
  • 4 edits in trunk

[wx] Unreviewed build fixes for Windows.

5:26 PM Changeset in webkit [101198] by commit-queue@webkit.org
  • 3 edits in trunk/Source/JavaScriptCore

Needs WTF_INLINE and JS_INLINE
​https://bugs.webkit.org/show_bug.cgi?id=72853

Patch by Hajime Morrita <​morrita@chromium.org> on 2011-11-26
Reviewed by Kevin Ollivier.

Added WTF_HIDDEN, WTF_INLINE and JS_INLINE which
indirect attribute((visibility("hidden"))

  • config.h:
  • wtf/ExportMacros.h:
11:31 AM Changeset in webkit [101197] by alex
  • 2 edits in trunk/Source/WebCore

2011-11-26 Alejandro G. Castro <​alex@igalia.com>

Fix compilation after r101157.

Reviewed by Martin Robinson.

  • bindings/scripts/preprocessor.pm: (applyPreprocessor): In case we have a string with double-quotations (") we replace it with the empty string.
10:00 AM Changeset in webkit [101196] by igor.oliveira@openbossa.org
  • 2 edits in trunk/Source/WebCore

[TexMapper][WK2][Qt] Simple opacity animations are not working
​https://bugs.webkit.org/show_bug.cgi?id=73157

LayerTreeHostQt needs to know what nodes in the TextureMapper are visible, to associate
a tile to it. It is done using TextureMapperNode::collectVisibleContentsRects.

However TextureMapperNode::collectVisibleContentsRects has an optimization to check if
the current tile has opacity greater than 0.01, otherwise it will not create a tile for
the current node.

For opacity animations from 0 to 1, it is a problem because the tile is never created
and the animation never happens.

This patch fixes opacity animations.

Reviewed by Noam Rosenthal.

  • platform/graphics/texmap/TextureMapperNode.cpp:

(WebCore::TextureMapperNode::countDescendantsWithContent):
(WebCore::TextureMapperNode::collectVisibleContentsRects):

8:09 AM Changeset in webkit [101195] by caio.oliveira@openbossa.org
  • 5 edits in trunk/Source/WebKit2

[Qt] [WK2] QtDownloadManager doesn't need to be RefCounted
​https://bugs.webkit.org/show_bug.cgi?id=73151

Reviewed by Kenneth Rohde Christiansen.

QtDownloadManager is owned by QtWebContext, and doesn't need to be a RefCounted
class. This commit also makes the callback handlers private -- following the similar
approach we take in other clients of the C API.

  • UIProcess/qt/QtDownloadManager.cpp:

(WebKit::QtDownloadManager::QtDownloadManager):
(WebKit::QtDownloadManager::addDownload):
(WebKit::toQtDownloadManager):
(WebKit::QtDownloadManager::didReceiveResponse):
(WebKit::QtDownloadManager::didCreateDestination):
(WebKit::QtDownloadManager::didFinishDownload): remove unused failingUrl local variable.
(WebKit::QtDownloadManager::didFailDownload):
(WebKit::QtDownloadManager::didReceiveDataForDownload):

  • UIProcess/qt/QtDownloadManager.h:
  • UIProcess/qt/QtWebContext.cpp:

(WebKit::QtWebContext::initialize):

  • UIProcess/qt/QtWebContext.h:
7:51 AM Changeset in webkit [101194] by commit-queue@webkit.org
  • 45 edits in trunk/Source

Unreviewed, rolling out r101193.
​http://trac.webkit.org/changeset/101193
​https://bugs.webkit.org/show_bug.cgi?id=73158

Breaks Windows and Qt minimal. (Requested by pfeldman on
#webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-11-26

Source/WebCore:

  • inspector/Inspector.json:
  • inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::inspectedPageDestroyed):

  • inspector/InspectorClient.h:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::close):

  • inspector/InspectorFrontendClient.h:
  • inspector/InspectorFrontendHost.cpp:

(WebCore::InspectorFrontendHost::disconnectFromBackend):

  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorFrontendHostStub.js:

(.WebInspector.InspectorFrontendHostStub.prototype.disconnectFromBackend):

  • inspector/front-end/inspector.js:

(WebInspector.disconnectFromBackend):

  • loader/EmptyClients.h:

Source/WebKit/chromium:

  • src/InspectorClientImpl.cpp:
  • src/InspectorClientImpl.h:
  • src/InspectorFrontendClientImpl.cpp:

(WebKit::InspectorFrontendClientImpl::disconnectFromBackend):

  • src/InspectorFrontendClientImpl.h:
  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgent::disconnectEventAsText):

  • src/WebDevToolsAgentImpl.h:

Source/WebKit/efl:

  • WebCoreSupport/InspectorClientEfl.cpp:
  • WebCoreSupport/InspectorClientEfl.h:

Source/WebKit/gtk:

  • WebCoreSupport/InspectorClientGtk.cpp:

(WebKit::InspectorClient::inspectorDestroyed):
(WebKit::InspectorFrontendClient::disconnectFromBackend):

  • WebCoreSupport/InspectorClientGtk.h:

Source/WebKit/mac:

  • WebCoreSupport/WebInspectorClient.h:
  • WebCoreSupport/WebInspectorClient.mm:

(WebInspectorClient::inspectorDestroyed):

Source/WebKit/qt:

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorClientQt::inspectorDestroyed):
(WebCore::InspectorFrontendClientQt::disconnectFromBackend):
(WebCore::InspectorFrontendClientQt::inspectorClientDestroyed):

  • WebCoreSupport/InspectorClientQt.h:

Source/WebKit/win:

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::inspectorDestroyed):
(WebInspectorFrontendClient::disconnectFromBackend):

  • WebCoreSupport/WebInspectorClient.h:

Source/WebKit/wince:

  • WebCoreSupport/InspectorClientWinCE.cpp:
  • WebCoreSupport/InspectorClientWinCE.h:

Source/WebKit/wx:

  • WebKitSupport/InspectorClientWx.cpp:
  • WebKitSupport/InspectorClientWx.h:

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebInspectorClient.cpp:

(WebKit::WebInspectorClient::inspectorDestroyed):

  • WebProcess/WebCoreSupport/WebInspectorClient.h:
  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:

(WebKit::WebInspectorFrontendClient::disconnectFromBackend):

  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.h:
7:30 AM Changeset in webkit [101193] by pfeldman@chromium.org
  • 42 edits in trunk/Source

Web Inspector: remove disconnectFromBackend from the protocol.
​https://bugs.webkit.org/show_bug.cgi?id=73127

Source/WebCore:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • inspector/Inspector.json:
  • inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::inspectedPageDestroyed):

  • inspector/InspectorClient.h:
  • inspector/InspectorController.cpp:

(WebCore::InspectorController::close):

  • inspector/InspectorFrontendClient.h:
  • inspector/InspectorFrontendHost.cpp:
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorFrontendHostStub.js:
  • inspector/front-end/inspector.js:

(WebInspector.disconnectFromWorker):

  • loader/EmptyClients.h:

(WebCore::EmptyInspectorClient::closeInspectorFrontend):

Source/WebKit/chromium:

We should instead use InspectorClient::closeInspectorFrontend that would
be closing the front-end window.

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • src/InspectorClientImpl.cpp:

(WebKit::InspectorClientImpl::closeInspectorFrontend):

  • src/InspectorClientImpl.h:
  • src/InspectorFrontendClientImpl.cpp:
  • src/InspectorFrontendClientImpl.h:
  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::closeInspectorFrontend):
(WebKit::WebDevToolsAgent::disconnectEventAsText):

  • src/WebDevToolsAgentImpl.h:

Source/WebKit/gtk:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • WebCoreSupport/InspectorClientGtk.cpp:

(WebKit::InspectorClient::inspectorDestroyed):
(WebKit::InspectorClient::closeInspectorFrontend):

  • WebCoreSupport/InspectorClientGtk.h:

Source/WebKit/mac:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • WebCoreSupport/WebInspectorClient.h:
  • WebCoreSupport/WebInspectorClient.mm:

(WebInspectorClient::inspectorDestroyed):
(WebInspectorClient::closeInspectorFrontend):

Source/WebKit/qt:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorClientQt::inspectorDestroyed):
(WebCore::InspectorClientQt::closeInspectorFrontend):
(WebCore::InspectorFrontendClientQt::inspectorClientDestroyed):

  • WebCoreSupport/InspectorClientQt.h:

Source/WebKit/win:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • WebCoreSupport/WebInspectorClient.cpp:

(WebInspectorClient::inspectorDestroyed):
(WebInspectorClient::closeInspectorFrontend):

  • WebCoreSupport/WebInspectorClient.h:

Source/WebKit2:

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-11-26
Reviewed by Yury Semikhatsky.

  • WebProcess/WebCoreSupport/WebInspectorClient.cpp:

(WebKit::WebInspectorClient::inspectorDestroyed):

  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.cpp:
  • WebProcess/WebCoreSupport/WebInspectorFrontendClient.h:
4:48 AM Changeset in webkit [101192] by timothy@apple.com
  • 2 edits in trunk/Source/WebCore

Fix a bug in TreeOutline which would cause parent elements to show up as expanded and empty
when removed and added back to the tree.

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

Reviewed by Pavel Feldman.

  • inspector/front-end/treeoutline.js:

(TreeElement.prototype.collapse): Set _treeElementsExpandedState to false instead of true.
Only TreeElement.prototype.expand should set it to true, which it does.

2:47 AM Changeset in webkit [101191] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit

[CMake] Clean up Web Inspector target in Source/WebKit/blackberry/CMakeListsBlackBerry.txt
​https://bugs.webkit.org/show_bug.cgi?id=72784

Patch by Jonathan Dong <Jonathan Dong> on 2011-11-26
Reviewed by Nikolas Zimmermann.

Abstracted the JavaScript file names from inspector/front-end/inspector.html
to keep them in-sync with the changes of inspector.html.

  • blackberry/CMakeListsBlackBerry.txt:
2:16 AM Changeset in webkit [101190] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[GTK] Fix some warnings in WebCore GTK+
​https://bugs.webkit.org/show_bug.cgi?id=73137

Patch by Martin Robinson <​mrobinson@igalia.com> on 2011-11-26
Reviewed by Philippe Normand.

No new tests. This patch just fixes compilation warnings.

  • platform/gtk/CursorGtk.cpp:

(WebCore::createNamedCursor): Remove unused variable.

  • platform/gtk/RenderThemeGtk3.cpp:

(WebCore::RenderThemeGtk::paintSliderTrack): ASSERT_UNUSED instead of ASSERT.

  • platform/gtk/SharedTimerGtk.cpp: Ditto.

(WebCore::stopSharedTimer):

12:59 AM Changeset in webkit [101189] by Martin Robinson
  • 2 edits in trunk/Tools

Try to fix the build after r101174.

  • gtk/generate-gtkdoc: Properly handle empty PKG_CONFIG_PATHs.

Nov 25, 2011:

11:17 PM Changeset in webkit [101188] by caio.oliveira@openbossa.org
  • 5 edits
    2 adds
    2 deletes in trunk/Source/WebKit2

[Qt] [WK2] Remove WebContext related code from QtWebPageProxy
​https://bugs.webkit.org/show_bug.cgi?id=73150

Reviewed by Kenneth Rohde Christiansen.

Creates a QtWebContext class that abstracts away from QtWebPageProxy the
handling of creating / maintaining the default WebContext, as well as the
code for the WKContextRef clients.

Another motivation for having this entity is to have a proper place to add
objects are bound to the context, like the QtDownloadManager.

  • Target.pri:
  • UIProcess/qt/ClientImpl.cpp: Removed.
  • UIProcess/qt/ClientImpl.h: Removed.
  • UIProcess/qt/QtDownloadManager.cpp:
  • UIProcess/qt/QtWebContext.cpp: Added.

(WebKit::QtWebContext::QtWebContext):
(WebKit::QtWebContext::~QtWebContext):
(WebKit::QtWebContext::create):
(WebKit::QtWebContext::defaultContext):
(WebKit::QtWebContext::createWebPage):
(WebKit::QtWebContext::setNavigatorQtObjectEnabled):
(WebKit::QtWebContext::postMessageToNavigatorQtObject):
(WebKit::QtWebContext::initialize):
(WebKit::QtWebContext::initializeContextInjectedBundleClient):
(WebKit::toQtWebContext):
(WebKit::QtWebContext::didReceiveMessageFromInjectedBundle):

  • UIProcess/qt/QtWebContext.h: Added.

(WebKit::QtWebContext::downloadManager):

  • UIProcess/qt/QtWebPageProxy.cpp:

(QtWebPageProxy::QtWebPageProxy):
(QtWebPageProxy::~QtWebPageProxy):
(QtWebPageProxy::setNavigatorQtObjectEnabled):
(QtWebPageProxy::postMessageToNavigatorQtObject):
(QtWebPageProxy::handleDownloadRequest):

  • UIProcess/qt/QtWebPageProxy.h:
10:15 PM Changeset in webkit [101187] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

String.prototype.toLower should be optimized for 8 bit strings
​https://bugs.webkit.org/show_bug.cgi?id=73154

Changed stringProtoFuncToLowerCase to use StringImpl::lower() which has
been optimized for 8 bit strings.

This is worth ~7% to sunspider string.tagcloud.

Reviewed by Filip Pizlo.

  • runtime/StringPrototype.cpp:

(JSC::stringProtoFuncToLowerCase):

10:01 PM Changeset in webkit [101186] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Array.toString always uses StringImpl::characters()
​https://bugs.webkit.org/show_bug.cgi?id=72969

If all component strings are 8 bit, create an 8 bit result string for toString().

This appears to be performance neutral to sunspider and v8.

Reviewed by Filip Pizlo.

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncToString):

8:28 PM Changeset in webkit [101185] by eric.carlson@apple.com
  • 11 edits
    2 adds
    2 deletes in trunk

Implement addCue and removeCue in TextTrack
​https://bugs.webkit.org/show_bug.cgi?id=72554

Reviewed by Darin Adler.

Source/WebCore:

Test: media/track/track-add-remove-cue.html

  • html/HTMLTrackElement.cpp:

(WebCore::HTMLTrackElement::ensureTrack): Go ahead and allocate a Track even if the feature

is disabled, it just won't load anything.

(WebCore::HTMLTrackElement::scheduleLoad): Early return if the featue is disabled.
(WebCore::HTMLTrackElement::canLoadUrl): Ditto.

  • html/LoadableTextTrack.cpp:

(WebCore::LoadableTextTrack::newCuesAvailable): Add new cues one at a time because

cues->add(Vector<TextTrackCue*>&) is gone.

  • html/TextTrack.cpp:

(WebCore::TextTrack::addCue): Implement.
(WebCore::TextTrack::removeCue): Ditto.

  • html/TextTrack.h:

(WebCore::TextTrackCue::TextTrackCue): Initialize every member variable.
(WebCore::TextTrackCue::track): m_track is now a RefPtr.
(WebCore::TextTrackCue::setTrack): Ditto.

  • html/TextTrackCue.h:
  • html/TextTrackCueList.cpp:

(WebCore::TextTrackCueList::add): Don't ignore out of order cues, the spec text is not

a conformance requirement. Return bool to indicate success or failure.

(WebCore::TextTrackCueList::remove): Return bool to indicate success or failure.

  • html/TextTrackCueList.h:
  • loader/TextTrackLoader.cpp:

(WebCore::TextTrackLoader::notifyFinished): Don't change m_state once it is set to Failed.

LayoutTests:

  • media/track/track-add-remove-cue-expected.txt: Added.
  • media/track/track-add-remove-cue.html: Added.
  • media/track/track-webvtt-tc012-out-of-order-expected.txt: Removed.
  • media/track/track-webvtt-tc012-out-of-order.html: Removed.
4:34 PM Changeset in webkit [101184] by haraken@chromium.org
  • 4 edits in trunk/Source/WebCore

Refactoring CodeGenerator*.pm for bug 72138
​https://bugs.webkit.org/show_bug.cgi?id=73115

Reviewed by Adam Barth.

Tests: bindings/scripts/test/TestObj.idl

  • bindings/scripts/CodeGeneratorGObject.pm: Added "1;" at the end since this perl script is loaded as a package.
  • bindings/scripts/CodeGeneratorV8.pm: Ditto. Removed unnecessary variable names from method declarations in order to supress style check errors when a new run-bindings-tests IDL is added.

(GenerateHeaderCustomCall):

  • bindings/scripts/test/V8/V8TestObj.h: Updated a run-bindings-tests result.
3:34 PM Changeset in webkit [101183] by haraken@chromium.org
  • 6 edits
    2 adds in trunk

Implement the WebGLContextEvent constructor
​https://bugs.webkit.org/show_bug.cgi?id=72856

Reviewed by Adam Barth.

Source/WebCore:

This patch makes WebGLContextEvent constructable.
The spec: ​http://www.khronos.org/registry/webgl/specs/latest/#5.14

Test: fast/events/constructors/webgl-context-event-constructor.html

  • html/canvas/WebGLContextEvent.cpp: Added an implementation of the WebGLContextEvent constructor.

(WebCore::WebGLContextEventInit::WebGLContextEventInit):
(WebCore::WebGLContextEvent::WebGLContextEvent):

  • html/canvas/WebGLContextEvent.h: Added a definition of WebGLContextEventInit.

(WebCore::WebGLContextEvent::create):

  • html/canvas/WebGLContextEvent.idl: Added [ConstructorTemplate=Event] IDL.

LayoutTests:

webgl-context-event-constructor.html checks the behavior of the WebGLContextEvent constructor.

  • fast/dom/constructed-objects-prototypes-expected.txt: Added window.WebGLContextEvent.
  • fast/events/constructors/webgl-context-event-constructor-expected.txt: Added.
  • fast/events/constructors/webgl-context-event-constructor.html: Added.
2:59 PM Changeset in webkit [101182] by rniwa@webkit.org
  • 1 edit
    9 adds in trunk/LayoutTests

Add missing results for Mac after r100949 and r100036.
svg/repaint/filter-child-repaint.svg is failing. The failure is tracked by the bug 73146.

  • platform/mac/css2.1/20110323/empty-inline-001-expected.png: Added.
  • platform/mac/css2.1/20110323/empty-inline-001-expected.txt: Added.
  • platform/mac/css2.1/20110323/empty-inline-002-expected.png: Added.
  • platform/mac/css2.1/20110323/empty-inline-002-expected.txt: Added.
  • platform/mac/css2.1/20110323/empty-inline-003-expected.png: Added.
  • platform/mac/css2.1/20110323/empty-inline-003-expected.txt: Added.
  • platform/mac/media/controls-without-preload-expected.png: Added.
  • platform/mac/svg/repaint/filter-child-repaint-expected.png: Added.
  • platform/mac/svg/repaint/filter-child-repaint-expected.txt: Added.
1:52 PM Changeset in webkit [101181] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[Chromium] The DrawingBuffer::bind method was incorrectly resetting the
GL viewport parameters. This was unnecessary, and resulted in the corruption
of the WebGL context's state.
​https://bugs.webkit.org/show_bug.cgi?id=73091

Patch by Jeff Timanus <​twiz@chromium.org> on 2011-11-25
Reviewed by Stephen White.

Tests: fast/canvas/webgl/webgl-viewport-parameters-preserved.html

Source/WebCore:

  • platform/graphics/gpu/DrawingBuffer.cpp:

(WebCore::DrawingBuffer::bind):

LayoutTests:

  • fast/canvas/webgl/webgl-viewport-parameters-preserved-expected.txt: Added.
  • fast/canvas/webgl/webgl-viewport-parameters-preserved.html: Added.
12:21 PM Changeset in webkit [101180] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

Crash in BidiRunList<Run>::replaceRunWithRuns with an empty bdi element
​https://bugs.webkit.org/show_bug.cgi?id=73116

Reviewed by Eric Seidel.

Source/WebCore:

The assertion failure was caused because isolatedResolver is initialized with
null start object due to the isolated run being empty.

Fixed the failure by skipping empty isolated runs.

Test: fast/text/empty-bdi-crash.html

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::constructBidiRuns):

LayoutTests:

Added a regression test.

  • fast/text/empty-bdi-crash-expected.txt: Added.
  • fast/text/empty-bdi-crash.html: Added.
11:57 AM Changeset in webkit [101179] by jesus@webkit.org
  • 11 edits in trunk/Source/WebKit2

[Qt][WK2] Refactor GestureRecognizers to use QtWebPageEventHandler
​https://bugs.webkit.org/show_bug.cgi?id=73096

Reviewed by Kenneth Rohde Christiansen.

Move all the interaction engine usage to QtWebPageEventHandler,
making all GestureRecognizers (Tap, Pinch and Pan) use it directly.

  • UIProcess/qt/QtGestureRecognizer.cpp:

(WebKit::QtGestureRecognizer::QtGestureRecognizer):

  • UIProcess/qt/QtGestureRecognizer.h:
  • UIProcess/qt/QtPanGestureRecognizer.cpp:

(WebKit::QtPanGestureRecognizer::QtPanGestureRecognizer):
(WebKit::QtPanGestureRecognizer::recognize):

  • UIProcess/qt/QtPanGestureRecognizer.h:
  • UIProcess/qt/QtPinchGestureRecognizer.cpp:

(WebKit::QtPinchGestureRecognizer::QtPinchGestureRecognizer):
(WebKit::QtPinchGestureRecognizer::recognize):

  • UIProcess/qt/QtPinchGestureRecognizer.h:
  • UIProcess/qt/QtTapGestureRecognizer.cpp:

(WebKit::QtTapGestureRecognizer::QtTapGestureRecognizer):
(WebKit::QtTapGestureRecognizer::recognize):
(WebKit::QtTapGestureRecognizer::tapTimeout):

  • UIProcess/qt/QtTapGestureRecognizer.h:
  • UIProcess/qt/QtWebPageEventHandler.cpp:

(QtWebPageEventHandler::QtWebPageEventHandler):
(QtWebPageEventHandler::setViewportInteractionEngine):

  • UIProcess/qt/QtWebPageEventHandler.h:

(QtWebPageEventHandler::interactionEngine):

11:26 AM Changeset in webkit [101178] by caio.oliveira@openbossa.org
  • 2 edits in trunk/Source/WebKit2

[Qt] Remove unneeded declaration from QQuickWebPagePrivate
​https://bugs.webkit.org/show_bug.cgi?id=73141

Reviewed by Antonio Gomes.

Left-over from r101165.

  • UIProcess/API/qt/qquickwebpage_p_p.h:
10:08 AM Changeset in webkit [101177] by Antti Koivisto
  • 10 edits in trunk/Source/WebCore

StyleGeneratedImage should ref CSSImageGeneratorValue
​https://bugs.webkit.org/show_bug.cgi?id=73074

Reviewed by Andreas Kling and Nikolas Zimmermann.

RenderStyle owns a bunch of StyleImage objects. However StyleGeneratedImage does not ref the
CSSImageGeneratorValue it holds so we currently rely on the stylesheet to keep the CSSImageGeneratorValues
alive as long as RenderStyle stays alive. While this works (RenderStyles are thrown away if stylesheets
change) it is not particularly robust or nice.

  • Use RefPtr<CSSImageGeneratorValue> in StyleGeneratedImage
  • Remove the RefPtr<StyleGeneratedImage> from CSSImageGeneratorValue. There is no good reason to cache StyleGeneratedImage as it is a small and uncommon object. With that the whole back-reference becomes unnecessary.
  • Switch more places to use (Pass)RefPtr<StyleImage> for consistency.
  • css/CSSImageGeneratorValue.cpp:

(WebCore::CSSImageGeneratorValue::CSSImageGeneratorValue):

  • css/CSSImageGeneratorValue.h:
  • css/CSSStyleApplyProperty.cpp:
  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
(WebCore::CSSStyleSelector::styleImage):
(WebCore::CSSStyleSelector::cachedOrPendingFromValue):
(WebCore::CSSStyleSelector::generatedOrPendingFromValue):
(WebCore::CSSStyleSelector::loadPendingImage):
(WebCore::CSSStyleSelector::loadPendingImages):

  • css/CSSStyleSelector.h:
  • rendering/style/FillLayer.h:

(WebCore::FillLayer::setImage):

  • rendering/style/NinePieceImage.h:

(WebCore::NinePieceImage::NinePieceImage):

  • rendering/style/StyleGeneratedImage.cpp:

(WebCore::StyleGeneratedImage::StyleGeneratedImage):
(WebCore::StyleGeneratedImage::cssValue):
(WebCore::StyleGeneratedImage::imageSize):
(WebCore::StyleGeneratedImage::addClient):
(WebCore::StyleGeneratedImage::removeClient):
(WebCore::StyleGeneratedImage::image):

  • rendering/style/StyleGeneratedImage.h:

(WebCore::StyleGeneratedImage::create):
(WebCore::StyleGeneratedImage::data):

9:47 AM Changeset in webkit [101176] by yurys@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] Web Inspector: assertion failure in InspectorController::restoreInspectorStateFromCookie
​https://bugs.webkit.org/show_bug.cgi?id=73136

Don't call connectFrontend twice on reattach.

Reviewed by Pavel Feldman.

  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::reattach):

8:46 AM Changeset in webkit [101175] by andreas.kling@nokia.com
  • 3 edits in trunk/Source/WebCore

Remove redundant setStrictParsing(false) calls on CSSMappedAttributeDeclarations.
<​http://webkit.org/b/73134>

Reviewed by Antti Koivisto.

CSSMappedAttributeDeclarations always use non-strict parsing since they use
the argument-less CSSMutableStyleDeclaration constructor.

  • dom/StyledElement.cpp:

(WebCore::StyledElement::createMappedDecl):

  • html/HTMLTableElement.cpp:

(WebCore::HTMLTableElement::additionalAttributeStyleDecls):
(WebCore::HTMLTableElement::addSharedCellBordersDecl):
(WebCore::HTMLTableElement::addSharedCellPaddingDecl):
(WebCore::HTMLTableElement::addSharedGroupDecls):

8:19 AM Changeset in webkit [101174] by Martin Robinson
  • 10 edits
    1 move
    3 adds
    4 deletes in trunk

[GTK] Integrate build-gtkdoc into build-webkit and make
​https://bugs.webkit.org/show_bug.cgi?id=72626

Reviewed by Philippe Normand.

.:

  • GNUmakefile.am: Remove references to old gtkdoc files. Update

the dist-hook to include copying documentation into the distribution.
This also means that the build will fail if documentation hasn't been
generated before 'make dist.'

  • configure.ac: Ditto.

Source/WebKit/gtk:

  • docs/GNUmakefile.am: Removed.
  • docs/version.xml.in: Removed.

Source/WebKit2:

  • GNUmakefile.am: Removed references to build-gtkdoc and gtkdoc.py.
  • UIProcess/API/gtk/docs/build-gtkdoc: Removed.

Tools:

  • GNUmakefile.am: Added support for new scripts and "make docs".
  • Scripts/webkitdirs.pm: Call generate-gtkdoc when building.

(buildAutotoolsProject):

  • gtk/generate-gtkdoc: Added.
  • gtk/gtkdoc.py: Renamed from Source/WebKit2/UIProcess/API/gtk/docs/gtkdoc.py.
7:14 AM Changeset in webkit [101173] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2011-11-25 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Mark inspector/elements/remove-node.html test as crashing sometimes on Linux.

  • platform/chromium/test_expectations.txt:
7:00 AM QtWebKitBuildBots edited by Csaba Osztrogonác
Update Qt5 on the bots (diff)
6:58 AM BuildingQt5OnHarmattan edited by Csaba Osztrogonác
Update Qt5 revision (diff)
6:44 AM Changeset in webkit [101172] by andreas.kling@nokia.com
  • 16 edits
    1 add in trunk/Source/WebCore

Factor element pointer out of CSSMutableStyleDeclaration.
<​http://webkit.org/b/73121>

Reviewed by Antti Koivisto.

Move the StyledElement pointer out of CSSMutableStyleDeclaration and into a subclass
for the cases where it's needed. This reduces the size of vanilla style declarations
by one CPU word (4 or 8 bytes, depending on architecture.)

  • css/CSSMutableStyleDeclaration.h:

(WebCore::CSSElementStyleDeclaration::element):
(WebCore::CSSElementStyleDeclaration::setElement):
(WebCore::CSSElementStyleDeclaration::CSSElementStyleDeclaration):
(WebCore::CSSElementStyleDeclaration::~CSSElementStyleDeclaration):

Added CSSElementStyleDeclaration, a common parent class for inline styles
and mapped attributes. It extends CSSMutableStyleDeclaration by adding a backpointer
to the StyledElement that owns it.

  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSInlineStyleDeclaration.h: Added.

(WebCore::CSSInlineStyleDeclaration::~CSSInlineStyleDeclaration):
(WebCore::CSSInlineStyleDeclaration::create):
(WebCore::CSSInlineStyleDeclaration::CSSInlineStyleDeclaration):

Added CSSInlineStyleDeclaration.

  • bindings/js/JSDOMBinding.h:

(WebCore::root):

  • css/CSSMutableStyleDeclaration.cpp:

(WebCore::CSSMutableStyleDeclaration::setNeedsStyleRecalc):

  • css/CSSStyleRule.cpp:

(WebCore::CSSStyleRule::setSelectorText):

Only follow the associated element on CSSElementStyleDeclarations.

  • css/CSSMutableStyleDeclaration.cpp:

(WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration):
(WebCore::CSSMutableStyleDeclaration::operator=):

Remove comment about m_element since the field was moved to a subclass.

  • css/CSSStyleDeclaration.cpp:

(WebCore::CSSStyleDeclaration::CSSStyleDeclaration):

  • css/CSSStyleDeclaration.h:

(WebCore::CSSStyleDeclaration::isElementStyleDeclaration):
(WebCore::CSSStyleDeclaration::isInlineStyleDeclaration):

We no longer need to track whether a style declaration is mutable or not,
so the constructors are slightly simplified.
Added two bits (isElementStyleDeclaration and isInlineStyleDeclaration) for
runtime type checking. These are protected and set by the CSSElementStyleDeclaration
constructor.

  • dom/CSSMappedAttributeDeclaration.h:

(WebCore::CSSMappedAttributeDeclaration::create):
(WebCore::CSSMappedAttributeDeclaration::CSSMappedAttributeDeclaration):

Remove pointless CSSRule* argument to constructor since we always passed 0.

  • dom/StyledElement.h:

(WebCore::StyledElement::inlineStyleDecl):

  • dom/StyledElement.cpp:

(WebCore::StyledElement::createInlineStyleDecl):
(WebCore::StyledElement::getInlineStyleDecl):
(WebCore::StyledElement::addSubresourceAttributeURLs):

Make the inline style on StyledElement a CSSInlineStyleDeclaration.

  • inspector/InspectorCSSAgent.cpp:

(WebCore::InspectorCSSAgent::inlineStyleElement):

Slightly simplified by having CSSInlineStyleDeclaration.

6:35 AM Changeset in webkit [101171] by alexis.menard@openbossa.org
  • 3 edits in trunk/Source/WebKit/qt

[Qt][WK2] QWebDownloadItem should be registered in the experimental plugin.
​https://bugs.webkit.org/show_bug.cgi?id=73132

Reviewed by Simon Hausmann.

Make sure QWebDownloadItem is registered in the right plugin.

  • declarative/experimental/plugin.cpp:
  • declarative/plugin.cpp:

(WebKitQmlPlugin::registerTypes):

6:32 AM Changeset in webkit [101170] by pfeldman@chromium.org
  • 7 edits in trunk

Web Inspector: unflake timeline-network tests
​https://bugs.webkit.org/show_bug.cgi?id=73123

Source/WebCore:

We should capture stack only when it makes sense. Otherwise loading from cache
results in different data sets (stacks) than regular load.

Reviewed by Yury Semikhatsky.

  • inspector/InspectorTimelineAgent.cpp:

(WebCore::InspectorTimelineAgent::willCallFunction):
(WebCore::InspectorTimelineAgent::willDispatchEvent):
(WebCore::InspectorTimelineAgent::willLayout):
(WebCore::InspectorTimelineAgent::willRecalculateStyle):
(WebCore::InspectorTimelineAgent::willPaint):
(WebCore::InspectorTimelineAgent::willWriteHTML):
(WebCore::InspectorTimelineAgent::didInstallTimer):
(WebCore::InspectorTimelineAgent::didRemoveTimer):
(WebCore::InspectorTimelineAgent::willFireTimer):
(WebCore::InspectorTimelineAgent::willChangeXHRReadyState):
(WebCore::InspectorTimelineAgent::willLoadXHR):
(WebCore::InspectorTimelineAgent::willEvaluateScript):
(WebCore::InspectorTimelineAgent::didScheduleResourceRequest):
(WebCore::InspectorTimelineAgent::willReceiveResourceData):
(WebCore::InspectorTimelineAgent::willReceiveResourceResponse):
(WebCore::InspectorTimelineAgent::didFinishLoadingResource):
(WebCore::InspectorTimelineAgent::didTimeStamp):
(WebCore::InspectorTimelineAgent::didMarkDOMContentEvent):
(WebCore::InspectorTimelineAgent::didMarkLoadEvent):
(WebCore::InspectorTimelineAgent::didRegisterAnimationFrameCallback):
(WebCore::InspectorTimelineAgent::didCancelAnimationFrameCallback):
(WebCore::InspectorTimelineAgent::willFireAnimationFrameEvent):
(WebCore::InspectorTimelineAgent::appendRecord):
(WebCore::InspectorTimelineAgent::pushCurrentRecord):

  • inspector/InspectorTimelineAgent.h:
  • inspector/TimelineRecordFactory.cpp:

(WebCore::TimelineRecordFactory::createGenericRecord):

LayoutTests:

Reviewed by Yury Semikhatsky.

  • platform/chromium/inspector/timeline/timeline-animation-frame-expected.txt:
  • platform/chromium/inspector/timeline/timeline-event-dispatch-expected.txt:
6:19 AM Changeset in webkit [101169] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

[Qt] Add support for using OWRT with Qt5
​https://bugs.webkit.org/show_bug.cgi?id=72947

Add qt-5.0 platform and don't pass -graphicssystem to QtTestBrowser if using Qt5.

Patch by Ádám Kallai <​Kallai.Adam@stud.u-szeged.hu> on 2011-11-25
Reviewed by Csaba Osztrogonác.

  • Scripts/old-run-webkit-tests:
6:17 AM Changeset in webkit [101168] by yurys@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-11-25 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Chromium build fix.

  • public/WebDevToolsFrontendClient.h: (WebKit::WebDevToolsFrontendClient::sendFrontendLoaded): dowsntream implementation with OVERRIDE modifier fails to compile without this stub.
5:56 AM WebKit Team edited by Alexandru Chiculita
(diff)
5:53 AM Changeset in webkit [101167] by kenneth@webkit.org
  • 2 edits in trunk/Source/WebKit2

Clean up code in WK2/ChromeClientClient related to viewport handling
​https://bugs.webkit.org/show_bug.cgi?id=73128

Reviewed by Simon Hausmann.

  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::contentsSizeChanged):

Make sure that we only call resizeToContentsIfNeeded when we are
using the tiled backing store and fixed layout as well.

Guard the scrollbar code so that it is only called when scroll
delegation is not used. Do similarily for the frameset code,
which makes no sense with frame flattening.

(WebKit::WebChromeClient::dispatchViewportPropertiesDidChange):

This method is only called for the main frame, so the main frame
check has been removed.

5:39 AM Changeset in webkit [101166] by yurys@chromium.org
  • 12 edits in trunk

[Chromium] Web Inspector: get rid of WebDevToolsFrontendClient::sendFrontendLoaded method
​https://bugs.webkit.org/show_bug.cgi?id=73126

Source/WebCore:

Notify embedder that front-end is loadded only when all dispatchers, panels and models
have been created so that it could dispatch messages synchronously in the loaded handler.

Reviewed by Pavel Feldman.

  • inspector/front-end/inspector.js:

Source/WebKit/chromium:

No need to have a separate client call for this. The front-end can store messages
which arrive before front-end loaded and dispatch them later. This happens in layout
tests only by the way as all other messages should be initiated by the front-end code.

Reviewed by Pavel Feldman.

  • public/WebDevToolsFrontendClient.h:
  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::attach):
(WebKit::WebDevToolsAgentImpl::frontendLoaded):

  • src/WebDevToolsFrontendImpl.cpp:

(WebKit::WebDevToolsFrontendImpl::WebDevToolsFrontendImpl):
(WebKit::WebDevToolsFrontendImpl::dispatchOnInspectorFrontend):
(WebKit::WebDevToolsFrontendImpl::frontendLoaded):
(WebKit::WebDevToolsFrontendImpl::doDispatchOnInspectorFrontend):

  • src/WebDevToolsFrontendImpl.h:

Tools:

No need to have a separate client call for this. The front-end can store messages
which arrive before front-end loaded and dispatch them later. This happens in layout
tests only by the way as all other messages should be initiated by the front-end code.

Reviewed by Pavel Feldman.

  • DumpRenderTree/chromium/DRTDevToolsAgent.cpp:
  • DumpRenderTree/chromium/DRTDevToolsAgent.h:
  • DumpRenderTree/chromium/DRTDevToolsClient.cpp:
  • DumpRenderTree/chromium/DRTDevToolsClient.h:
4:54 AM Changeset in webkit [101165] by Simon Hausmann
  • 3 edits in trunk/Source/WebKit2

[Qt] Eliminate warnings about the deprecated QQuickCanvas::sceneGraphEngine() function.

Reviewed by Andreas Kling.

We use the function in question to find out if we can connect to the scene graph's
afterRendering signal. The signal has been moved into QQuickCanvas right away, so
we can connect to it as soon as we have a canvas.

  • UIProcess/API/qt/qquickwebpage.cpp:

(QQuickWebPagePrivate::initializeSceneGraphConnections):

  • UIProcess/API/qt/qquickwebpage_p.h:
4:48 AM Changeset in webkit [101164] by jesus@webkit.org
  • 9 edits
    2 adds in trunk/Source/WebKit2

[Qt][WK2] Move event handling out of QtWebPageProxy
​https://bugs.webkit.org/show_bug.cgi?id=73089

Reviewed by Kenneth Rohde Christiansen.

Move all logic related to event handling, gestures and the
interaction engine out of QtWebPageProxy, creating a separate
class called QtWebPageEventHandler. This handler will be created
by the QQuickWebViewPrivate and will be registered into QtWebPageProxy.
QQuickWebPage will now call handleEvent through this entity.

  • Target.pri:
  • UIProcess/API/qt/qquickwebpage.cpp:

(QQuickWebPage::event):

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::QQuickWebViewPrivate):
(QQuickWebViewPrivate::initializeDesktop):
(QQuickWebViewPrivate::initializeTouch):

  • UIProcess/API/qt/qquickwebview_p_p.h:
  • UIProcess/qt/QtTapGestureRecognizer.cpp:

(WebKit::QtTapGestureRecognizer::QtTapGestureRecognizer):
(WebKit::QtTapGestureRecognizer::recognize):
(WebKit::QtTapGestureRecognizer::tapTimeout):

  • UIProcess/qt/QtTapGestureRecognizer.h:
  • UIProcess/qt/QtWebPageEventHandler.cpp: Added.

(QtWebPageEventHandler::dragOperationToDropAction):
(QtWebPageEventHandler::dragOperationToDropActions):
(QtWebPageEventHandler::dropActionToDragOperation):
(QtWebPageEventHandler::QtWebPageEventHandler):
(QtWebPageEventHandler::~QtWebPageEventHandler):
(QtWebPageEventHandler::handleEvent):
(QtWebPageEventHandler::handleMouseMoveEvent):
(QtWebPageEventHandler::handleMousePressEvent):
(QtWebPageEventHandler::handleMouseReleaseEvent):
(QtWebPageEventHandler::handleMouseDoubleClickEvent):
(QtWebPageEventHandler::handleWheelEvent):
(QtWebPageEventHandler::handleHoverLeaveEvent):
(QtWebPageEventHandler::handleHoverMoveEvent):
(QtWebPageEventHandler::handleDragEnterEvent):
(QtWebPageEventHandler::handleDragLeaveEvent):
(QtWebPageEventHandler::handleDragMoveEvent):
(QtWebPageEventHandler::handleDropEvent):
(QtWebPageEventHandler::handleSingleTapEvent):
(QtWebPageEventHandler::handleDoubleTapEvent):
(QtWebPageEventHandler::timerEvent):
(QtWebPageEventHandler::handleKeyPressEvent):
(QtWebPageEventHandler::handleKeyReleaseEvent):
(QtWebPageEventHandler::handleFocusInEvent):
(QtWebPageEventHandler::handleFocusOutEvent):
(QtWebPageEventHandler::setViewportInteractionEngine):
(QtWebPageEventHandler::touchEvent):
(QtWebPageEventHandler::resetGestureRecognizers):
(QtWebPageEventHandler::doneWithTouchEvent):
(QtWebPageEventHandler::didFindZoomableArea):

  • UIProcess/qt/QtWebPageEventHandler.h: Added.
  • UIProcess/qt/QtWebPageProxy.cpp:

(QtWebPageProxy::QtWebPageProxy):
(QtWebPageProxy::init):
(QtWebPageProxy::didFindZoomableArea):
(QtWebPageProxy::processDidCrash):
(QtWebPageProxy::startDrag):
(QtWebPageProxy::doneWithTouchEvent):

  • UIProcess/qt/QtWebPageProxy.h:

(QtWebPageProxy::eventHandler):

4:12 AM Changeset in webkit [101163] by yurys@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: don't send Console.messagesCleared on front-end connection if console agent is not enabled
​https://bugs.webkit.org/show_bug.cgi?id=73122

Console agent should stay silent if it is not enabled.

Reviewed by Pavel Feldman.

  • inspector/InspectorConsoleAgent.cpp:

(WebCore::InspectorConsoleAgent::clearMessages):
(WebCore::InspectorConsoleAgent::addConsoleMessage):

3:39 AM Changeset in webkit [101162] by Simon Hausmann
  • 9 edits
    1 add in trunk/Source/WebKit2

[Qt] Cleanup in QML tests.

Reviewed by Tor Arne Vestbø.

Use a simple QML component for the "DesktopWebView" instead of a new QML type.
This will allow the use of the experimental extension in the future.

  • UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewExperimental::useTraditionalDesktopBehaviour):

  • UIProcess/API/qt/qquickwebview_p.h:
  • UIProcess/API/qt/tests/qmltests/DesktopBehavior/DesktopWebView.qml: Added.
  • UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_linkHovered.qml:
  • UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_loadHtml.qml:
  • UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_messaging.qml:
  • UIProcess/API/qt/tests/qmltests/DesktopBehavior/tst_navigationRequested.qml:
  • UIProcess/API/qt/tests/qmltests/qmltests.pro:
  • UIProcess/API/qt/tests/qmltests/tst_qmltests.cpp:

(main):

2:57 AM Changeset in webkit [101161] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/Source/WebCore

Upstream the QNX/RIM specific HistoryItemViewState
​https://bugs.webkit.org/show_bug.cgi?id=73114

Patch by Wei Charles <​charles.wei@torchmobile.com.cn> on 2011-11-25
Reviewed by Daniel Bates.

This patch is to upstream RIM/QNX specific HistoryItemViewState extension
to store some extra view state.

No new tests. It's not built yet.

  • history/blackberry: Added.
  • history/blackberry/HistoryItemViewState.h: Added.

(WebCore::HistoryItemViewState::HistoryItemViewState):

2:33 AM Changeset in webkit [101160] by vsevik@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: Split scripts panel top status bar into debug toolbar and editor toolbar.
​https://bugs.webkit.org/show_bug.cgi?id=73087

Reviewed by Pavel Feldman.

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel.prototype._reset):
(WebInspector.ScriptsPanel.prototype._createEditorToolbar):
(WebInspector.ScriptsPanel.prototype._createDebugToolbar):
(WebInspector.ScriptsPanel.prototype._createButtonAndRegisterShortcuts):

  • inspector/front-end/scriptsPanel.css:

(#scripts-debug-toolbar):
(#scripts-editor-toolbar):
(#scripts-debug-sidebar-resizer-widget):
(.script-view):

2:06 AM Changeset in webkit [101159] by leo.yang@torchmobile.com.cn
  • 1 edit
    3 adds in trunk/Source/WebCore

Upstream BlackBerry porting of platform/text
​https://bugs.webkit.org/show_bug.cgi?id=73117

Reviewed by Daniel Bates.

Initial upstream, can't be built yet, no test cases.

  • platform/text/blackberry/StringBlackBerry.cpp: Added.

(WTF::String::String):
(WTF::String::operator WebString):

  • platform/text/blackberry/TextBreakIteratorInternalICUBlackBerry.cpp: Added.

(WebCore::currentSearchLocaleID):
(WebCore::currentTextBreakLocaleID):

1:41 AM Changeset in webkit [101158] by pfeldman@chromium.org
  • 2 edits in trunk/LayoutTests

Web Inspector: resource-tree-errors-reload.html is flaky.
​https://bugs.webkit.org/show_bug.cgi?id=66217

Reviewed by Yury Semikhatsky.

  • platform/chromium/test_expectations.txt:
1:31 AM Changeset in webkit [101157] by haraken@chromium.org
  • 3 edits
    1 delete in trunk/Source/WebCore

Remove WebCore.gyp/scripts/rule_binding.py
​https://bugs.webkit.org/show_bug.cgi?id=73109

Reviewed by Adam Barth.

rule_binding.py is used by WebCore.gyp only. rule_binding.py is just a wrapper
of generate-bindings.pl and thus we can remove it.

No new tests. No change in behavior.

  • WebCore.gyp/WebCore.gyp: Replaces rule_binding.py with generate-bindings.pl.
  • WebCore.gyp/scripts/rule_binding.py: Removed.
  • bindings/scripts/preprocessor.pm: When '"ENABLE_SOMETHING=1" "ENABLE_OTHERS=0"' is passed as a define macro, the string is passed to our perl scripts without the double-quotations (") omitted. We need to omit the double quatations here.

(applyPreprocessor):

1:29 AM Changeset in webkit [101156] by yurys@chromium.org
  • 15 edits
    4 deletes in trunk

[Chromium] Web Inspector: remove legacy debugger support
​https://bugs.webkit.org/show_bug.cgi?id=73081

Removed classed providing access to V8 debugging protocol. Clients should
use Web Inspector protocol instead.

Reviewed by Pavel Feldman.

Source/WebCore:

  • bindings/v8/PageScriptDebugServer.cpp:

(WebCore::PageScriptDebugServer::PageScriptDebugServer):
(WebCore::PageScriptDebugServer::addListener):

  • bindings/v8/PageScriptDebugServer.h:

Source/WebKit/chromium:

  • WebKit.gyp:
  • public/WebDevToolsAgent.h:
  • public/WebDevToolsAgentClient.h:
  • src/DebuggerAgentImpl.cpp: Removed.
  • src/DebuggerAgentImpl.h: Removed.
  • src/DebuggerAgentManager.cpp: Removed.
  • src/DebuggerAgentManager.h: Removed.
  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::WebDevToolsAgentImpl):
(WebKit::WebDevToolsAgentImpl::~WebDevToolsAgentImpl):
(WebKit::WebDevToolsAgentImpl::attach):
(WebKit::WebDevToolsAgentImpl::detach):
(WebKit::WebDevToolsAgentImpl::didNavigate):
(WebKit::WebDevToolsAgentImpl::didClearWindowObject):

  • src/WebDevToolsAgentImpl.h:
1:25 AM Changeset in webkit [101155] by yurys@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-11-25 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Update Chromium deps revision to 111575

  • DEPS:
1:23 AM Changeset in webkit [101154] by pfeldman@chromium.org
  • 26 edits in trunk

Web Inspector: refactor resource tree model and introduce explicit class for ResourceTreeFrame.
​https://bugs.webkit.org/show_bug.cgi?id=73078

Otherwise we are managing frames and their resources using too many maps.

Reviewed by Yury Semikhatsky.

Source/WebCore:

  • inspector/Inspector.json:
  • inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::restore):

  • inspector/InspectorResourceAgent.cpp:

(WebCore::InspectorResourceAgent::didReceiveResponse):

  • inspector/front-end/ApplicationCacheModel.js:

(WebInspector.ApplicationCacheModel.prototype._frameNavigated):
(WebInspector.ApplicationCacheModel.prototype._frameDetached):

  • inspector/front-end/JavaScriptContextManager.js:

(WebInspector.JavaScriptContextManager.prototype._frameAdded):
(WebInspector.JavaScriptContextManager.prototype._frameNavigated):
(WebInspector.FrameEvaluationContext.prototype.get displayName):

  • inspector/front-end/NetworkManager.js:

(WebInspector.NetworkDispatcher.prototype.requestWillBeSent):
(WebInspector.NetworkDispatcher.prototype.responseReceived):
(WebInspector.NetworkDispatcher.prototype.loadingFinished):

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkLogView.prototype._mainFrameNavigated):

  • inspector/front-end/ResourceTreeModel.js:

(WebInspector.ResourceTreeModel):
(WebInspector.ResourceTreeModel.prototype._fetchResourceTree):
(WebInspector.ResourceTreeModel.prototype._processCachedResources):
(WebInspector.ResourceTreeModel.prototype._addFrame):
(WebInspector.ResourceTreeModel.prototype._frameNavigated):
(WebInspector.ResourceTreeModel.prototype._frontendReused):
(WebInspector.ResourceTreeModel.prototype._frameDetached):
(WebInspector.ResourceTreeModel.prototype._onResourceUpdated):
(WebInspector.ResourceTreeModel.prototype._onResourceUpdateDropped):
(WebInspector.ResourceTreeModel.prototype.frameForId):
(WebInspector.ResourceTreeModel.prototype.forAllResources):
(WebInspector.ResourceTreeModel.prototype.resourceForURL):
(WebInspector.ResourceTreeModel.prototype._addFramesRecursively):
(WebInspector.ResourceTreeModel.prototype._createResourceFromFramePayload):
(WebInspector.ResourceTreeModel.prototype._createResource):
(WebInspector.ResourceTreeFrame):
(WebInspector.ResourceTreeFrame.prototype.get id):
(WebInspector.ResourceTreeFrame.prototype.get name):
(WebInspector.ResourceTreeFrame.prototype.get url):
(WebInspector.ResourceTreeFrame.prototype.get loaderId):
(WebInspector.ResourceTreeFrame.prototype.get parentFrame):
(WebInspector.ResourceTreeFrame.prototype.get childFrames):
(WebInspector.ResourceTreeFrame.prototype.isMainFrame):
(WebInspector.ResourceTreeFrame.prototype.navigate):
(WebInspector.ResourceTreeFrame.prototype.get mainResource):
(WebInspector.ResourceTreeFrame.prototype.addChildFrame):
(WebInspector.ResourceTreeFrame.prototype.removeChildFrame):
(WebInspector.ResourceTreeFrame.prototype._addResource):
(WebInspector.ResourceTreeFrame.prototype.resources):
(WebInspector.ResourceTreeFrame.prototype.resourceForURL.filter):
(WebInspector.ResourceTreeFrame.prototype.resourceForURL):
(WebInspector.ResourceTreeFrame.prototype._callForFrameResources):
(WebInspector.PageDispatcher.prototype.frameNavigated):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel):
(WebInspector.ResourcesPanel.prototype.wasShown):
(WebInspector.ResourcesPanel.prototype._initialize):
(WebInspector.ResourcesPanel.prototype._resetWithFrames):
(WebInspector.ResourcesPanel.prototype._reset):
(WebInspector.ResourcesPanel.prototype._populateResourceTree.populateFrame):
(WebInspector.ResourcesPanel.prototype._populateResourceTree):
(WebInspector.ResourcesPanel.prototype._frameAdded):
(WebInspector.ResourcesPanel.prototype._frameDetached):
(WebInspector.ResourcesPanel.prototype._frameNavigated):
(WebInspector.ResourcesPanel.prototype._cachedResourcesLoaded):

  • inspector/front-end/ScriptFormatter.js:

(WebInspector.ScriptFormatter.prototype.formatContent):

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::requestResource):

LayoutTests:

  • http/tests/inspector/inspector-test.js:
  • http/tests/inspector/resource-tree/resource-tree-document-url.html:
  • http/tests/inspector/resource-tree/resource-tree-errors-reload.html:
  • http/tests/inspector/resource-tree/resource-tree-frame-add.html:
  • http/tests/inspector/resource-tree/resource-tree-mimetype.html:
  • http/tests/inspector/resource-tree/resource-tree-test.js:

(initialize_ResourceTreeTest.InspectorTest.dumpResourcesURLMap.collect):

  • http/tests/inspector/resources-test.js:

(initialize_ResourceTest.InspectorTest.runAfterCachedResourcesProcessed):
(initialize_ResourceTest.InspectorTest.runAfterResourcesAreFinished.checkResources.visit):
(initialize_ResourceTest.InspectorTest.runAfterResourcesAreFinished.checkResources):
(initialize_ResourceTest.InspectorTest.runAfterResourcesAreFinished):
(initialize_ResourceTest.InspectorTest.showResource.showResourceCallback.visit):
(initialize_ResourceTest.InspectorTest.showResource):
(initialize_ResourceTest):

  • inspector/debugger/source-frame.html:
  • inspector/styles/css-live-edit.html:
  • inspector/styles/styles-history.html:
  • platform/qt/Skipped:
Note: See TracTimeline for information about the timeline view.