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

Timeline



Mar 2, 2009:

10:15 PM Changeset in webkit [41379] by Simon Fraser
  • 3 edits
    2 adds in trunk

2009-03-02 Simon Fraser <Simon Fraser>

Reviewed by Adele Peterson

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

If RenderTextControlSingleLine::nodeAtPoint() pokes a node into the
HitTestResult, then it had better well return |true|. And m_innerBlock
could never have been set as the hit node by the superclass, because
the superclass knows nothing about m_innerBlock.

Test: fast/forms/input-hit-test-border.html

  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::nodeAtPoint):
6:24 PM Changeset in webkit [41378] by weinig@apple.com
  • 8 edits in trunk

JavaScriptCore:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Mark Rowe.

Enable Geolocation (except on Tiger and Leopard).

  • Configurations/JavaScriptCore.xcconfig:

WebCore:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Mark Rowe.

Enable Geolocation (except on Tiger and Leopard).

  • Configurations/WebCore.xcconfig:

WebKit/mac:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Mark Rowe.

Enable Geolocation (except on Tiger and Leopard).

  • Configurations/WebKit.xcconfig:

WebKitTools:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Mark Rowe.

Enable Geolocation (except on Tiger and Leopard).

  • Scripts/build-webkit:
6:12 PM Changeset in webkit [41377] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-03-02 Ojan Vafai <​ojan@chromium.org>

Reviewed by Adele Peterson.

Fix ​https://bugs.webkit.org/show_bug.cgi?id=24307
Null out m_highlightedNode on hideHighlight().
Ran WebCore/manual-tests/inspector/highlight-nodes.html.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::hideHighlight):
5:51 PM Changeset in webkit [41376] by andersca@apple.com
  • 5 edits in trunk/WebKit/mac

2009-03-02 Anders Carlsson <​andersca@apple.com>

Reviewed by Dan Bernstein.

WebKit part of <rdar://problem/6638658>.


  • Plugins/Hosted/NetscapePluginHostManager.h:
  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::createPropertyListFile): Spawn the plug-in host and wait for it to create the property list.


  • Plugins/WebBasePluginPackage.mm: (-[WebBasePluginPackage createPropertyListFile]): Factor code out into a new method.


(-[WebBasePluginPackage pListForPath:createFile:]):
Call the newly added createPropertyListFile method.


  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage createPropertyListFile]): Tell the plug-in host manager to create a property list file for us.
5:47 PM Changeset in webkit [41375] by weinig@apple.com
  • 13 edits
    2 adds in trunk

WebCore:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6507404> Add Geolocation support.

This is not yet turned on for any Mac platform.

  • WebCore.xcodeproj/project.pbxproj:
  • page/Chrome.cpp: (WebCore::Chrome::shouldAllowGeolocationForFrame):
  • page/Chrome.h:
  • page/ChromeClient.h: (WebCore::ChromeClient::shouldAllowGeolocationForFrame):
  • page/Geolocation.cpp: (WebCore::Geolocation::Geolocation): (WebCore::Geolocation::disconnectFrame): Remove call to setUsingGeolocation as the document will not be alive at this point. (WebCore::Geolocation::getCurrentPosition): Check if the embedding app allows geolocation and return a PERMISSION_DENIED if not. (WebCore::Geolocation::watchPosition): Ditto. (WebCore::Geolocation::shouldAllowGeolocation): Perform request to embedding layer of whether to allow geolocation and cache the result.
  • page/Geolocation.h: (WebCore::Geolocation::):
  • platform/GeolocationService.cpp:
  • platform/GeolocationService.h: (WebCore::GeolocationService::~GeolocationService): (WebCore::GeolocationService::stopUpdating):
  • platform/mac/GeolocationServiceMac.h: Added. (WebCore::GeolocationServiceMac::lastPosition): (WebCore::GeolocationServiceMac::lastError):
  • platform/mac/GeolocationServiceMac.mm: Added. (WebCore::GeolocationService::create): (WebCore::GeolocationServiceMac::GeolocationServiceMac): (WebCore::GeolocationServiceMac::~GeolocationServiceMac): (WebCore::GeolocationServiceMac::startUpdating): (WebCore::GeolocationServiceMac::stopUpdating): (WebCore::GeolocationServiceMac::suspend): (WebCore::GeolocationServiceMac::resume): (WebCore::GeolocationServiceMac::positionChanged): (WebCore::GeolocationServiceMac::errorOccurred): (-[WebCoreCoreLocationObserver initWithCallback:]): (-[WebCoreCoreLocationObserver locationManager:didUpdateToLocation:fromLocation:]): (-[WebCoreCoreLocationObserver locationManager:didFailWithError:]):

WebKit/mac:

2009-03-02 Sam Weinig <​sam@webkit.org>

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6507404> Add Geolocation support.

This is not yet turned on for any Mac platform.

Add SPI to ask the embedding application whether to allow
Geolocation for an origin.

  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm: (WebChromeClient::shouldAllowGeolocationForFrame):
  • WebView/WebUIDelegatePrivate.h:
5:36 PM Changeset in webkit [41374] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-02 Anders Carlsson <​andersca@apple.com>

Fix PowerPC build.

  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage _tryLoad]):
5:36 PM UsingGitWithWebKit edited by ojan@chromium.org
(diff)
4:20 PM Changeset in webkit [41373] by andersca@apple.com
  • 1 edit in trunk/WebKit/mac/ChangeLog

Fix tyop

4:14 PM Changeset in webkit [41372] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-02 Anders Carlsson <​andersca@apple.com>

Reviewed by John Sullivan, Ada Chan.

Factor loading code out into its own method and get rid of a bunch of goots.


  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage _tryLoad]): (-[WebNetscapePluginPackage load]):
4:00 PM Changeset in webkit [41371] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-02 Anders Carlsson <​andersca@apple.com>

Build fix.

  • Plugins/WebNetscapeDeprecatedFunctions.h:
3:58 PM Changeset in webkit [41370] by andersca@apple.com
  • 2 edits in trunk/WebKit

2009-03-02 Anders Carlsson <​andersca@apple.com>

WebNetscapePluginPackage.mm is an Objective-C++ file.

  • WebKit.xcodeproj/project.pbxproj:
3:53 PM Changeset in webkit [41369] by andersca@apple.com
  • 3 edits
    1 move in trunk/WebKit

WebKit:

2009-03-02 Anders Carlsson <​andersca@apple.com>

Reviewed by John Sullivan.

Rename WebNetscapePluginPackage.m to WebNetscapePluginPackage.mm


  • WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

2009-03-02 Anders Carlsson <​andersca@apple.com>

Reviewed by John Sullivan.

Rename WebNetscapePluginPackage.m to WebNetscapePluginPackage.mm

  • Plugins/WebNetscapePluginPackage.m: Removed.
  • Plugins/WebNetscapePluginPackage.mm: Copied from mac/Plugins/WebNetscapePluginPackage.m.
3:29 PM Changeset in webkit [41368] by kevino@webkit.org
  • 11 edits in trunk

Build fixes for wxWidgets Mac trunk build.

3:25 PM Changeset in webkit [41367] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wxGTK missing header build fix.

1:55 PM Changeset in webkit [41366] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

  • platform/graphics/win/FontCacheWin.cpp: (WebCore::createGDIFont): Added back code to verify that the created HFONT has the requested face name.
1:38 PM Changeset in webkit [41365] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fixes a regression caused by splitting the input and output of console commands.

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

Reviewed by Kevin McCullough.

  • inspector/front-end/Console.js: (WebInspector.Console.prototype.addMessage): Don't treat ConsoleCommandResult as a normal ConsoleMessage when appending. This prevents resetting repeatCounts incorrectly.
1:38 PM Changeset in webkit [41364] by timothy@apple.com
  • 4 edits in trunk/WebCore

Make exception messages and logged Error objects display consistently.

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

Reviewed by Adam Roben.

  • English.lproj/localizedStrings.js: Remove a string for line numbers.
  • inspector/front-end/Console.js: (WebInspector.Console.prototype._enterKeyPressed): Pass the exception boolean to the ConsoleCommandResult constructor and don't pass level. (WebInspector.Console.prototype._formatnode): Fix a className typo. (WebInspector.Console.prototype._formaterror): Add an error-message classed span around the message to show in red and remove the console-message-url class from the link so it doesn't float right. Instead enclose the link in parentheses and use displayNameForURL to shorten the URL in the link. (WebInspector.ConsoleMessage.prototype.toMessageElement): Simplify how line numbers are shown since link underlining a string with parentheses looks bad. (WebInspector.ConsoleCommandResult): Take an exception boolean flag instead of a level and get line and URL based on that flag.
  • inspector/front-end/inspector.css: Tweaks to the styles.
1:38 PM Changeset in webkit [41363] by timothy@apple.com
  • 2 edits in trunk/WebKitTools

Allow for multiline quoted text in JavaScript files when looking for function names.

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

Reviewed by David Kilzer.

  • Scripts/prepare-ChangeLog:
11:16 AM Changeset in webkit [41362] by levin@chromium.org
  • 10 edits in trunk/WebCore

2009-03-02 Ojan Vafai <​ojan@chromium.org>

Reviewed by Adele Peterson.

Bug 24048: extra windows button padding doesn't apply when there's no appearance
<​https://bugs.webkit.org/show_bug.cgi?id=24048>

Replace adjustButtonInnerStyle with buttonInternalPaddingLeft/Right/Top/Bottom.

No tests added as existing tests already cover this behavior.

  • rendering/RenderButton.cpp: (WebCore::RenderButton::setupInnerStyle):
  • rendering/RenderTheme.cpp:
  • rendering/RenderTheme.h: (WebCore::RenderTheme::buttonInternalPaddingLeft): (WebCore::RenderTheme::buttonInternalPaddingRight): (WebCore::RenderTheme::buttonInternalPaddingTop):
  • rendering/RenderThemeChromiumGtk.cpp: (WebCore::RenderThemeWin::buttonInternalPaddingLeft): (WebCore::RenderThemeWin::buttonInternalPaddingRight): (WebCore::RenderThemeWin::buttonInternalPaddingTop): (WebCore::RenderThemeWin::buttonInternalPaddingBottom):
  • rendering/RenderThemeChromiumGtk.h:
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::buttonInternalPaddingLeft): (WebCore::RenderThemeChromiumWin::buttonInternalPaddingRight): (WebCore::RenderThemeChromiumWin::buttonInternalPaddingTop): (WebCore::RenderThemeChromiumWin::buttonInternalPaddingBottom):
  • rendering/RenderThemeChromiumWin.h:
  • rendering/RenderThemeWin.cpp: (WebCore::RenderThemeWin::buttonInternalPaddingLeft): (WebCore::RenderThemeWin::buttonInternalPaddingRight): (WebCore::RenderThemeWin::buttonInternalPaddingTop): (WebCore::RenderThemeWin::buttonInternalPaddingBottom):
  • rendering/RenderThemeWin.h:
10:26 AM Changeset in webkit [41361] by treat@webkit.org
  • 5 edits in trunk

2009-03-02 Adam Treat <​adam.treat@torchmobile.com>

Reviewed by Eric Seidel.

Add three new drt helper functions that enable all of the tests in
LayoutTests/animation/* and LayoutTests/transitions/* to now pass.

  • Api/qwebframe.cpp: (qt_drt_pauseAnimation): (qt_drt_pauseTransitionOfProperty): (qt_drt_numberOfActiveAnimations):
  • DumpRenderTree/qt/jsobjects.cpp: (LayoutTestController::pauseAnimationAtTimeOnElementWithId): (LayoutTestController::pauseTransitionAtTimeOnElementWithId): (LayoutTestController::numberOfActiveAnimations):
  • DumpRenderTree/qt/jsobjects.h:
8:55 AM Changeset in webkit [41360] by treat@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-03-02 Benjamin C Meyer <​benjamin.meyer@torchmobile.com>

Reviewed by George Staikos.

​https://bugs.webkit.org/show_bug.cgi?id=21230
On X11 match the behavior of Firefox and also copy the url to the
clipboard selection when the action Copy Link Location is executed.

  • Api/qwebpage.cpp: (QWebPage::triggerAction):
8:30 AM Changeset in webkit [41359] by treat@webkit.org
  • 2 edits in trunk/WebCore

2009-03-02 Adam Treat <​adam.treat@torchmobile.com>

Build fix for Qt port. Remove unnecessary assert.

  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::ImageBuffer::putImageData):
7:37 AM Changeset in webkit [41358] by treat@webkit.org
  • 9 edits in trunk/WebCore

2009-03-02 Adam Treat <​adam.treat@torchmobile.com>

Reviewed by Adam Roben and previously by Eric Seidel and Simon Fraser.

​https://bugs.webkit.org/show_bug.cgi?id=24227
Ensure that the checkForSolidColor() optimization is correctly triggered
for all cases of drawPattern. Currently, the optimization was not triggered
when the check had not been previously performed via a request for the
image's NativeImagePtr.

Implement the Qt version of the checkForSolidColor() method. Combined with
the bug fix this reduces the time it takes to draw a repeating background
of a 1x1 image from ~50msecs to ~0msecs on my machine.

  • platform/graphics/BitmapImage.cpp: (WebCore::BitmapImage::BitmapImage):
  • platform/graphics/BitmapImage.h: (WebCore::BitmapImage::mayFillWithSolidColor):
  • platform/graphics/Image.h: (WebCore::Image::mayFillWithSolidColor):
  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::BitmapImage): (WebCore::BitmapImage::checkForSolidColor):
  • platform/graphics/cg/ImageCG.cpp: (WebCore::BitmapImage::BitmapImage): (WebCore::BitmapImage::checkForSolidColor):
  • platform/graphics/qt/ImageQt.cpp: (WebCore::BitmapImage::checkForSolidColor):
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImage::checkForSolidColor):
  • platform/graphics/wx/ImageWx.cpp: (WebCore::BitmapImage::checkForSolidColor):
6:56 AM Changeset in webkit [41357] by kov@webkit.org
  • 4 edits in trunk

2009-03-02 Gustavo Noronha Silva <Gustavo Noronha Silva>

Unreviewed build fix; adding missing files to EXTRA_DIST, so that
they show up in the tarball.

  • GNUmakefile.am:
6:38 AM QtWebKitTodo edited by Simon Hausmann
(diff)
6:38 AM Changeset in webkit [41356] by krit@webkit.org
  • 1 edit in trunk/WebCore/ChangeLog

2009-03-02 Dirk Schulze <​krit@webkit.org>

Just a correction of the ChangeLog. Removed Ariya Hidayat
from the reviewing process.

6:22 AM Changeset in webkit [41355] by krit@webkit.org
  • 2 edits in trunk/WebCore

2009-03-02 Dirk Schulze <​krit@webkit.org>

Reviewed by Holger Freyther, rubber-stamped by Ariya Hidayat.

Added putImageData to Qt.

[Qt] lacks putImageData support in Canvas
​https://bugs.webkit.org/show_bug.cgi?id=22186

  • platform/graphics/qt/ImageBufferQt.cpp: (WebCore::ImageBuffer::putImageData):
5:26 AM Changeset in webkit [41354] by Adam Roben
  • 2 edits in trunk/WebKitTools

Windows build fix after r41349

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp: (AccessibilityUIElement::getChildrenWithRange):
12:22 AM Changeset in webkit [41353] by timothy@apple.com
  • 2 edits in trunk/WebCore

Show exception messages again when evaluating bad expressions in the
Web Inspector's console.

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

Reviewed by Oliver Hunt.

  • bindings/js/JSQuarantinedObjectWrapper.cpp: (WebCore::JSQuarantinedObjectWrapper::transferExceptionToExecState): Fix the order in which the exception is set and cleared now that the exception is stored in the shared GlobalData, not per ExecState.
12:22 AM Changeset in webkit [41352] by timothy@apple.com
  • 3 edits
    1 add in trunk/WebCore

Make input in the Web Inspector console print before any output
that might be added by the called function.

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

Reviewed by Oliver Hunt.

  • inspector/front-end/Console.js: (WebInspector.Console.prototype._enterKeyPressed): Add the command message before evaluating the result. Associate the originating command to the result, so if they are adjacent there is no divider. (WebInspector.ConsoleCommand): No longer take a result. (WebInspector.ConsoleCommand.prototype.toMessageElement): Result code removed since it isn't used now. (WebInspector.ConsoleCommandResult): Subclass ConsoleMessage. (WebInspector.ConsoleCommandResult.prototype.toMessageElement): Call the ConsoleMessage toMessageElement and add a style class.
  • inspector/front-end/inspector.css: Add a new style class for adjacent results to hide the divider. Also tweak the position of the disclosure triangle for objects to not use the left margin.

Mar 1, 2009:

9:46 PM Changeset in webkit [41351] by timothy@apple.com
  • 1 edit in trunk/WebKitTools/DumpRenderTree/mac/AccessibilityUIElementMac.mm

Fix the Mac build.

8:18 PM Changeset in webkit [41350] by ddkilzer@apple.com
  • 4 edits in trunk/JavaScriptCore

<rdar://problem/6635688> Move HAVE_DTRACE check to Base.xcconfig

Reviewed by Mark Rowe.

  • Configurations/Base.xcconfig: Set HAVE_DTRACE Xcode variable based on PLATFORM_NAME and MAC_OS_X_VERSION_MAJOR. Also define it as a preprocessor macro by modifying GCC_PREPROCESSOR_DEFINITIONS.
  • JavaScriptCore.xcodeproj/project.pbxproj: Changed "Generate DTrace header" script phase to check for HAVE_DTRACE instead of MACOSX_DEPLOYMENT_TARGET.
  • wtf/Platform.h: Removed definition of HAVE_DTRACE macro since it's defined in Base.xcconfig now.
6:55 PM Changeset in webkit [41349] by Chris Fleizach
  • 6 edits in trunk

Bug 24282: AX Palindrome error when asking for a specific index of the AXChildren array
<​https://bugs.webkit.org/show_bug.cgi?id=24282>

4:45 PM Changeset in webkit [41348] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-01 Anders Carlsson <​andersca@apple.com>

Reviewed by Sam Weinig.

WebKit side of <rdar://problem/6449689>

Pass the visible name to the plug-in host.


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost):
4:41 PM Changeset in webkit [41347] by levin@chromium.org
  • 19 edits in trunk

2009-03-01 Ojan Vafai <​ojan@chromium.org>

Reviewed by Eric Seidel.

Bug 24251: Cleanup: replace positionForCoordinates with positionForPoint
<​https://bugs.webkit.org/show_bug.cgi?id=24251>

Make RenderObject::positionForCoordinate non-virtual and
RenderObject::positionForPoint virtual in preparation for
removing positionFor* from RenderObject/RenderText.

  • page/Frame.cpp: (WebCore::Frame::visiblePositionForPoint):
  • rendering/RenderBR.cpp: (WebCore::RenderBR::positionForPoint):
  • rendering/RenderBR.h:
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::positionForPoint):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::positionForPoint):
  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::positionForPoint):
  • rendering/RenderInline.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::positionForCoordinates): (WebCore::RenderObject::positionForPoint):
  • rendering/RenderObject.h:
  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::positionForPoint):
  • rendering/RenderReplaced.h:
  • rendering/RenderSVGInlineText.cpp: (WebCore::RenderSVGInlineText::positionForPoint):
  • rendering/RenderSVGInlineText.h:
  • rendering/RenderText.cpp: (WebCore::RenderText::positionForPoint):
  • rendering/RenderText.h:
4:41 PM Changeset in webkit [41346] by levin@chromium.org
  • 4 edits in trunk/WebCore

2009-03-01 Larry Ewing <​lewing@novell.com>

Reviewed by Alexey Proskuryakov.

​https://bugs.webkit.org/show_bug.cgi?id=24080
NPN_GetValue casting to the wrong type and writing outside bounds

Make sure to cast the value to the correct type so that only
memory owned by the value is written to.

  • plugins/gtk/PluginViewGtk.cpp (PluginView::getValueStatic):
  • plugins/qt/PluginViewQt.cpp (PluginView::getValueStatic):
  • plugins/mac/PluginViewMac.cpp (PluginView::getValueStatic): (PluginView::getValue):
4:41 PM Changeset in webkit [41345] by levin@chromium.org
  • 9 edits in trunk/WebCore

2009-03-01 Jeremy Moskovich <​jeremy@chromium.org>

Reviewed by Simon Fraser.

​https://bugs.webkit.org/show_bug.cgi?id=24197 and
​https://bugs.webkit.org/show_bug.cgi?id=23512

The Windows version of cgColor() is conditionally compiled using
!PLATFORM(MAC) change this to PLATFORM(WIN_OS) so that Chrome
can use the OS X version instead of the Windows version.

Also rename cgColor() -> createCGColor()

No observable change in behavior, so no test.

  • platform/graphics/Color.h:
  • platform/graphics/cg/ColorCG.cpp: (WebCore::createCGColor):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::setPlatformShadow):
  • platform/graphics/mac/ColorMac.mm: (WebCore::createCGColor):
  • platform/graphics/mac/GraphicsContextMac.mm: (WebCore::GraphicsContext::drawFocusRing):
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::setLayerBorderColor): (WebCore::setLayerBackgroundColor): (WebCore::GraphicsLayerCA::setBackgroundColor):
  • platform/graphics/win/GraphicsContextCGWin.cpp: (WebCore::GraphicsContext::drawFocusRing):
  • svg/graphics/filters/cg/SVGFEHelpersCg.mm: (WebCore::ciColor):
1:09 PM Changeset in webkit [41344] by levin@chromium.org
  • 17 edits
    9 adds in trunk

WebCore:

2009-03-01 David Levin <​levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Bug 24271: XMLHttpRequest needs js bindings for Workers.
<​https://bugs.webkit.org/show_bug.cgi?id=24271>

Tests: http/tests/xmlhttprequest/workers/methods-async.html

http/tests/xmlhttprequest/workers/methods.html

  • bindings/js/JSWorkerContextBase.cpp: (WebCore::JSWorkerContextBase::getOwnPropertySlot): (jsWorkerContextBaseXMLHttpRequest): (setJSWorkerContextBaseXMLHttpRequest): Expose XMLHttpRequest to Workers.
  • bindings/js/JSWorkerContextBase.h:
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor): (WebCore::constructXMLHttpRequest): (WebCore::JSXMLHttpRequestConstructor::mark): Removed the dependencies on Document.
  • bindings/js/JSXMLHttpRequestConstructor.h: (WebCore::JSXMLHttpRequestConstructor::scriptExecutionContext):
  • dom/EventException.idl:
  • dom/EventListener.idl: Chnage to have the NoStaticTables attribute since they are used from XHR.
  • loader/WorkerThreadableLoader.cpp: (WebCore::workerContextDidFinishLoading): (WebCore::WorkerThreadableLoader::MainThreadBridge::didFinishLoading): Fixed identifier to be unsigned long.
  • loader/WorkerThreadableLoader.h:
  • workers/WorkerThread.cpp: (WebCore::WorkerThread::workerThread): Stop active objects when the thread is getting shutdown so that xhr gets properly shutdown.
  • xml/XMLHttpRequestUpload.cpp:
  • xml/XMLHttpRequest.idl:
  • xml/XMLHttpRequestException.idl:
  • xml/XMLHttpRequestProgressEvent.idl:
  • xml/XMLHttpRequestUpload.idl: Chnage to have the NoStaticTables attribute since they are used from XHR.

LayoutTests:

2009-03-01 David Levin <​levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Bug 24271: XMLHttpRequest needs js bindings for Workers.
<​https://bugs.webkit.org/show_bug.cgi?id=24271>

Some sanity checks for xhr done from workers. These are the same as
tests done for normal workers.

  • http/tests/xmlhttprequest/workers/methods-async-expected.txt: Added.
  • http/tests/xmlhttprequest/workers/methods-async.html: Added.
  • http/tests/xmlhttprequest/workers/methods-expected.txt: Added.
  • http/tests/xmlhttprequest/workers/methods.html: Added.
  • http/tests/xmlhttprequest/workers/resources/methods-async.js: Added. (onmessage): (log): (done): (onReqAbort): (onSyncReqError): (start): (processStateChange):
  • http/tests/xmlhttprequest/workers/resources/methods.cgi: Copied from LayoutTests/http/tests/xmlhttprequest/methods.cgi.
  • http/tests/xmlhttprequest/workers/resources/methods.js: Added. (log): (done):
12:14 PM Changeset in webkit [41343] by timothy@apple.com
  • 1 edit in trunk/JavaScriptCore/parser/Nodes.h

Build fix. Add isBlock() back since there was a new client since the patch was made.

11:37 AM Applications using WebKit edited by todd.ditchendorf@gmail.com
(diff)
11:35 AM Changeset in webkit [41342] by timothy@apple.com
  • 4 edits
    5 adds in trunk

JavaScriptCore:

2009-03-01 Horia Olaru <​olaru@adobe.com>

By looking in grammar.y there are only a few types of statement nodes
on which the debugger should stop.

Removed isBlock and isLoop virtual calls. No need to emit debug hooks in
the "statementListEmitCode" method as long as the necessary hooks can be
added in each "emitCode".

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

Reviewed by Kevin McCullough.

  • parser/Nodes.cpp: (JSC::ConstStatementNode::emitBytecode): (JSC::statementListEmitCode): (JSC::EmptyStatementNode::emitBytecode): (JSC::ExprStatementNode::emitBytecode): (JSC::VarStatementNode::emitBytecode): (JSC::IfNode::emitBytecode): (JSC::IfElseNode::emitBytecode): (JSC::DoWhileNode::emitBytecode): (JSC::WhileNode::emitBytecode): (JSC::ForNode::emitBytecode): (JSC::ForInNode::emitBytecode): (JSC::ContinueNode::emitBytecode): (JSC::BreakNode::emitBytecode): (JSC::ReturnNode::emitBytecode): (JSC::WithNode::emitBytecode): (JSC::SwitchNode::emitBytecode): (JSC::LabelNode::emitBytecode): (JSC::ThrowNode::emitBytecode): (JSC::TryNode::emitBytecode):
  • parser/Nodes.h:

WebCore:

2009-03-01 Horia Olaru <​olaru@adobe.com>

Adding manual debugger test cases for bug.

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

Reviewed by Kevin McCullough.

  • manual-tests/inspector/debugger-step-on-do-while-statements.html: Added.
  • manual-tests/inspector/debugger-step-on-for-in-statements.html: Added.
  • manual-tests/inspector/debugger-step-on-for-statements.html: Added.
  • manual-tests/inspector/debugger-step-on-while-statements.html: Added.
  • manual-tests/inspector/resources/loop-statements.js: Added.
11:22 AM Changeset in webkit [41341] by jmalonzo@webkit.org
  • 7 edits in trunk

2009-03-01 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] get the HTTP layout tests going
​https://bugs.webkit.org/show_bug.cgi?id=24259

Added API to get the response mime type from a frame. We need this
so we can decide if we need to dump the frame as text or its
render tree

  • webkit/webkitprivate.h:
  • webkit/webkitwebframe.cpp:

2009-03-01 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] get the HTTP layout tests going
​https://bugs.webkit.org/show_bug.cgi?id=24259

Determine the frame's response and decide whether to dump as text
or the render tree

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (dump):

2009-02-28 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] get the HTTP layout tests going
​https://bugs.webkit.org/show_bug.cgi?id=24259

Enable http tests, excluding tests/folders that fail or untested

  • platform/gtk/Skipped:
11:21 AM Changeset in webkit [41340] by jmalonzo@webkit.org
  • 4 edits in trunk/WebKitTools

2009-03-01 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] get the HTTP layout tests going
​https://bugs.webkit.org/show_bug.cgi?id=24259

Implement dumping of WebKitWebBackForwardList and its history
items.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (compareHistoryItems): (dumpHistoryItem): (dumpBackForwardListForWebView): (dump): (runTest):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::clearBackForwardList):
  • DumpRenderTree/gtk/WorkQueueItemGtk.cpp: (BackForwardItem::invoke):
11:21 AM Changeset in webkit [41339] by jmalonzo@webkit.org
  • 4 edits in trunk/WebKit/gtk

2009-03-01 Jan Michael Alonzo <​jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] get the HTTP layout tests going
​https://bugs.webkit.org/show_bug.cgi?id=24259

Create a WebKitWebHistoryItem for each WebCore::HistoryItem when
necessary.
Add necessary API additions for us to be able to dump a WebKitWebHistoryItem

  • webkit/webkitprivate.h:
  • webkit/webkitwebbackforwardlist.cpp: (_WebKitWebBackForwardListPrivate::webkit_web_back_forward_list_get_forward_list_with_limit): (_WebKitWebBackForwardListPrivate::webkit_web_back_forward_list_get_back_list_with_limit):
  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_new_with_core_item): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_new): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_new_with_data): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_target): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_is_target_item): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_children): (WebKit::core): (WebKit::kit):
11:02 AM Changeset in webkit [41338] by mitz@apple.com
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • fix <rdar://problem/6634768> Reproducible crash in RenderLayer::updateScrollCornerStyle() using full-page zoom at MobileMe Contacts

Test: fast/events/scroll-during-zoom-change.html

  • dom/Document.cpp: (WebCore::Document::recalcStyle): Pause scheduled events during style reclaculation. Changes to the zoom property result in scroll events being generated during style recalc, and those events need to be deferred until the render tree is consistent.
  • page/FrameView.cpp: (WebCore::FrameView::layout): Changed to account for a case in which the post-layout tasks trigger a nested layout, which reactivates the post-layout task timer.

LayoutTests:

Reviewed by Darin Adler.

  • test for <rdar://problem/6634768> Reproducible crash in RenderLayer::updateScrollCornerStyle() using full-page zoom at MobileMe Contacts
  • fast/events/scroll-during-zoom-change-expected.txt: Added.
  • fast/events/scroll-during-zoom-change.html: Added.
11:00 AM Changeset in webkit [41337] by christian@webkit.org
  • 4 edits in trunk/WebKit/gtk

2009-03-01 Christian Dywan <​christian@twotoasts.de>

Reviewed by Holger Freyther.

  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (webkit_web_view_get_encoding):
  • webkit/webkitwebview.h: Implement 'encoding' and 'custom-encoding' properties as well as webkit_web_view_get_encoding.
10:47 AM Changeset in webkit [41336] by christian@webkit.org
  • 2 edits in trunk/WebKitTools

2009-02-28 Zan Dobersek <​zandobersek@gmail.com>

Reviewed by Holger Freyther.

Set the resolution for the default screen to 72.0.
This way, setting font sizes results in expected values.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (setDefaultsToConsistentStateValuesForTesting):
8:53 AM Changeset in webkit [41335] by timothy@apple.com
  • 3 edits in trunk/WebCore

Alphabetically sort scripts in the Script's panel file popup menu.

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

Reviewed by Timothy Hatcher.

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu):
  • inspector/front-end/utilities.js: (insertionIndexForObjectInListSortedByFunction): (indexOfObjectInListSortedByFunction):
8:42 AM Changeset in webkit [41334] by timothy@apple.com
  • 3 edits in trunk/WebCore

Pass securityOrigin->host() instead of securityOrigin->domain() to WebInspector
when creating databases, localStorage or sessionStorage. Changing document.domain
should have no affect on databases, localStorage and sessionStorage in Web Inspector.

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

Reviewed by Timothy Hatcher.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::addDOMStorageScriptResource): (WebCore::InspectorController::didUseDOMStorage):
  • storage/Database.cpp: (WebCore::Database::openDatabase):
8:22 AM Changeset in webkit [41333] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-03-01 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Holger Freyther.

Avoid leaking errors when reporting GIO errors.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::readCallback): (WebCore::openCallback): (WebCore::queryInfoCallback):
8:21 AM Changeset in webkit [41332] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-03-01 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Holger Freyther.

Fixes crash when the GIO backend is given the URI for a directory.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::networkErrorForFile):
8:04 AM Changeset in webkit [41331] by kov@webkit.org
  • 2 edits in trunk/WebKit/gtk

Simple wording fix for the NEWS file.

7:39 AM Changeset in webkit [41330] by christian@webkit.org
  • 2 edits in trunk

2009-03-01 Christian Dywan <​christian@twotoasts.de>

  • configure.ac: Bump GTK port version to 1.1.1.
7:38 AM Changeset in webkit [41329] by kov@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk

2009-03-01 Xan Lopez <​xan@gnome.org>

Add NEWS file to track progress between releases.

  • NEWS: Added.
7:29 AM Changeset in webkit [41328] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-02-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Holger Freyther.

Adds a WebKitMakeArguments environment variable to enable passing
of arguments such as '-j2' to make for the autotools build.

  • Scripts/webkitdirs.pm:
7:28 AM Changeset in webkit [41327] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-02-28 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Holger Freyther.

Add a GTK+-only option to enable GNOME Keyring when building.

  • Scripts/build-webkit:
Note: See TracTimeline for information about the timeline view.