Timeline



Sep 23, 2007:

6:23 PM Changeset in webkit [25708] by mjs
  • 9 edits in trunk/WebKitSite

Reviewed by Mark Rowe.


  • Limit custom list bullets to ordered lists only
  • css/blue.css:
  • css/gray.css:
  • css/green.css:
  • css/ie.css:
  • css/main.css:
  • css/pink.css:
  • css/purple.css:
  • css/yellow.css:
5:21 PM Changeset in webkit [25707] by bdash
  • 4 edits in trunk

Versioning.

5:21 PM Changeset in webkit [25706] by bdash
  • 1 copy in tags/Safari-4523.8

New tag.

5:20 PM Changeset in webkit [25705] by bdash
  • 1 copy in tags/Safari-5523.8

New tag.

11:24 AM BuildingGtk edited by freyther@gmit-gmbh.de
(diff)
9:38 AM Changeset in webkit [25704] by ddkilzer
  • 2 edits in trunk/WebKitSite

WebKitSite:

Reviewed by Adam.

  • coding/coding-style.html:
    • in section Other Punctuation
      • syntax instead of synatax
      • fix sample code alignment
8:11 AM BuildingOnLinux edited by freyther@gmit-gmbh.de
(diff)
8:10 AM WikiStart edited by freyther@gmit-gmbh.de
(diff)
8:09 AM BuildingGdk edited by freyther@gmit-gmbh.de
(diff)
8:05 AM BuildingGtk created by freyther@gmit-gmbh.de
2:50 AM Changeset in webkit [25703] by zecke
  • 37 edits
    38 copies
    5 moves
    38 deletes in trunk

/:
2007-09-20 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Adam.

Renamed files from *Gdk to *Gtk (see #14732) using the
work of Juan A. Suarez Romero as a base.

GDK -> GTK

  • WebKit.pri:
  • WebKit.pro:

JavaScriptCore:
2007-09-20 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Adam.

Renamed files from *Gdk to *Gtk (see #14732) using the
work of Juan A. Suarez Romero as a base.

GDK -> GTK

  • JavaScriptCore.pri:
  • kjs/testkjs.pro:
  • pcre/dftables.pro:
  • wtf/Platform.h: PLATFORM(GDK) to PLATFORM(GTK)

WebCore:
2007-09-20 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Adam.

Renamed files from *Gdk to *Gtk (see #14732) using the
work of Juan A. Suarez Romero as a base.

GDK -> GTK

  • Projects/gdk/webcore-gdk.bkl:
  • WebCore.pro:
  • WebCoreSources.bkl:
  • loader/gdk: Removed.
  • loader/gdk/DocumentLoaderGdk.cpp: Removed.
  • loader/gtk: Added.
  • loader/gtk/DocumentLoaderGdk.cpp: Removed.
  • loader/gtk/DocumentLoaderGtk.cpp: Added.
  • page/FrameView.cpp:
  • page/FrameView.h:
  • page/gdk: Removed.
  • page/gdk/DragControllerGdk.cpp: Removed.
  • page/gdk/EventHandlerGdk.cpp: Removed.
  • page/gdk/FrameGdk.cpp: Removed.
  • page/gtk: Added.
  • page/gtk/DragControllerGdk.cpp: Removed.
  • page/gtk/DragControllerGtk.cpp: Added.
  • page/gtk/EventHandlerGdk.cpp: Removed.
  • page/gtk/EventHandlerGtk.cpp: Added. (WebCore::EventHandler::createDraggingClipboard):
  • page/gtk/FrameGdk.cpp: Removed.
  • page/gtk/FrameGtk.cpp: Added.
  • platform/ContextMenuItem.h:
  • platform/Cursor.h:
  • platform/DragData.h:
  • platform/DragImage.h:
  • platform/FontData.h:
  • platform/NotImplemented.h:
  • platform/PlatformKeyboardEvent.h:
  • platform/PlatformMenuDescription.h:
  • platform/PlatformMouseEvent.h:
  • platform/PlatformWheelEvent.h:
  • platform/ScrollView.h:
  • platform/Widget.h:
  • platform/gdk: Removed.
  • platform/gdk/ClipboardGdk.cpp: Removed.
  • platform/gdk/ClipboardGdk.h: Removed.
  • platform/gdk/ContextMenuGdk.cpp: Removed.
  • platform/gdk/ContextMenuItemGdk.cpp: Removed.
  • platform/gdk/CookieJarGdk.cpp: Removed.
  • platform/gdk/CursorGdk.cpp: Removed.
  • platform/gdk/DragDataGdk.cpp: Removed.
  • platform/gdk/DragImageGdk.cpp: Removed.
  • platform/gdk/FileChooserGdk.cpp: Removed.
  • platform/gdk/FileSystemGdk.cpp: Removed.
  • platform/gdk/FontCacheGdk.cpp: Removed.
  • platform/gdk/FontDataGdk.cpp: Removed.
  • platform/gdk/FontGdk.cpp: Removed.
  • platform/gdk/FontPlatformData.h: Removed.
  • platform/gdk/FontPlatformDataGdk.cpp: Removed.
  • platform/gdk/GlyphPageTreeNodeGdk.cpp: Removed.
  • platform/gdk/KeyEventGdk.cpp: Removed.
  • platform/gdk/KeyboardCodes.h: Removed.
  • platform/gdk/LocalizedStringsGdk.cpp: Removed.
  • platform/gdk/LoggingGdk.cpp: Removed.
  • platform/gdk/MIMETypeRegistryGdk.cpp: Removed.
  • platform/gdk/MouseEventGdk.cpp: Removed.
  • platform/gdk/PasteboardGdk.cpp: Removed.
  • platform/gdk/PlatformScreenGdk.cpp: Removed.
  • platform/gdk/PlatformScrollBar.h: Removed.
  • platform/gdk/PlatformScrollBarGdk.cpp: Removed.
  • platform/gdk/PopupMenuGdk.cpp: Removed.
  • platform/gdk/RenderThemeGdk.cpp: Removed.
  • platform/gdk/RenderThemeGdk.h: Removed.
  • platform/gdk/ScrollViewGdk.cpp: Removed.
  • platform/gdk/SearchPopupMenuGdk.cpp: Removed.
  • platform/gdk/SharedTimerLinux.cpp: Removed.
  • platform/gdk/SoundGdk.cpp: Removed.
  • platform/gdk/SystemTimeLinux.cpp: Removed.
  • platform/gdk/TemporaryLinkStubs.cpp: Removed.
  • platform/gdk/WheelEventGdk.cpp: Removed.
  • platform/gdk/WidgetGdk.cpp: Removed.
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::setPlatformFont):
  • platform/graphics/gdk: Removed.
  • platform/graphics/gdk/IconGdk.cpp: Removed.
  • platform/graphics/gdk/ImageGdk.cpp: Removed.
  • platform/graphics/gtk: Added.
  • platform/graphics/gtk/IconGdk.cpp: Removed.
  • platform/graphics/gtk/IconGtk.cpp: Added.
  • platform/graphics/gtk/ImageGdk.cpp: Removed.
  • platform/graphics/gtk/ImageGtk.cpp: Added.
  • platform/gtk: Added.
  • platform/gtk/ClipboardGdk.cpp: Removed.
  • platform/gtk/ClipboardGdk.h: Removed.
  • platform/gtk/ClipboardGtk.cpp: Added. (WebCore::ClipboardGtk::ClipboardGtk): (WebCore::ClipboardGtk::~ClipboardGtk): (WebCore::ClipboardGtk::clearData): (WebCore::ClipboardGtk::clearAllData): (WebCore::ClipboardGtk::getData): (WebCore::ClipboardGtk::setData): (WebCore::ClipboardGtk::types): (WebCore::ClipboardGtk::dragLocation): (WebCore::ClipboardGtk::dragImage): (WebCore::ClipboardGtk::setDragImage): (WebCore::ClipboardGtk::dragImageElement): (WebCore::ClipboardGtk::setDragImageElement): (WebCore::ClipboardGtk::createDragImage): (WebCore::ClipboardGtk::declareAndWriteDragImage): (WebCore::ClipboardGtk::writeURL): (WebCore::ClipboardGtk::writeRange): (WebCore::ClipboardGtk::hasData):
  • platform/gtk/ClipboardGtk.h: Added.
  • platform/gtk/ContextMenuGdk.cpp: Removed.
  • platform/gtk/ContextMenuGtk.cpp: Added.
  • platform/gtk/ContextMenuItemGdk.cpp: Removed.
  • platform/gtk/ContextMenuItemGtk.cpp: Added.
  • platform/gtk/CookieJarGdk.cpp: Removed.
  • platform/gtk/CookieJarGtk.cpp: Added.
  • platform/gtk/CursorGdk.cpp: Removed.
  • platform/gtk/CursorGtk.cpp: Added.
  • platform/gtk/DragDataGdk.cpp: Removed.
  • platform/gtk/DragDataGtk.cpp: Added.
  • platform/gtk/DragImageGdk.cpp: Removed.
  • platform/gtk/DragImageGtk.cpp: Added.
  • platform/gtk/FileChooserGdk.cpp: Removed.
  • platform/gtk/FileChooserGtk.cpp: Added.
  • platform/gtk/FileSystemGdk.cpp: Removed.
  • platform/gtk/FileSystemGtk.cpp: Added.
  • platform/gtk/FontCacheGdk.cpp: Removed.
  • platform/gtk/FontCacheGtk.cpp: Added.
  • platform/gtk/FontDataGdk.cpp: Removed.
  • platform/gtk/FontDataGtk.cpp: Added.
  • platform/gtk/FontGdk.cpp: Removed.
  • platform/gtk/FontGtk.cpp: Added.
  • platform/gtk/FontPlatformDataGdk.cpp: Removed.
  • platform/gtk/FontPlatformDataGtk.cpp: Added.
  • platform/gtk/GlyphPageTreeNodeGdk.cpp: Removed.
  • platform/gtk/GlyphPageTreeNodeGtk.cpp: Added.
  • platform/gtk/KeyEventGdk.cpp: Removed.
  • platform/gtk/KeyEventGtk.cpp: Added.
  • platform/gtk/LocalizedStringsGdk.cpp: Removed.
  • platform/gtk/LocalizedStringsGtk.cpp: Added.
  • platform/gtk/LoggingGdk.cpp: Removed.
  • platform/gtk/LoggingGtk.cpp: Added.
  • platform/gtk/MIMETypeRegistryGdk.cpp: Removed.
  • platform/gtk/MIMETypeRegistryGtk.cpp: Added.
  • platform/gtk/MouseEventGdk.cpp: Removed.
  • platform/gtk/MouseEventGtk.cpp: Added.
  • platform/gtk/PasteboardGdk.cpp: Removed.
  • platform/gtk/PasteboardGtk.cpp: Added.
  • platform/gtk/PlatformScreenGdk.cpp: Removed.
  • platform/gtk/PlatformScreenGtk.cpp: Added.
  • platform/gtk/PlatformScrollBarGdk.cpp: Removed.
  • platform/gtk/PlatformScrollBarGtk.cpp: Added.
  • platform/gtk/PopupMenuGdk.cpp: Removed.
  • platform/gtk/PopupMenuGtk.cpp: Added.
  • platform/gtk/RenderThemeGdk.cpp: Removed.
  • platform/gtk/RenderThemeGdk.h: Removed.
  • platform/gtk/RenderThemeGtk.cpp: Added. (WebCore::theme): (WebCore::RenderThemeGtk::RenderThemeGtk): (WebCore::RenderThemeGtk::close): (WebCore::RenderThemeGtk::addIntrinsicMargins): (WebCore::RenderThemeGtk::supportsFocus): (WebCore::RenderThemeGtk::determineState): (WebCore::RenderThemeGtk::determineShadow): (WebCore::RenderThemeGtk::getThemeData): (WebCore::RenderThemeGtk::setCheckboxSize): (WebCore::RenderThemeGtk::paintCheckbox): (WebCore::RenderThemeGtk::setRadioSize): (WebCore::RenderThemeGtk::paintRadio): (WebCore::RenderThemeGtk::paintButton): (WebCore::RenderThemeGtk::adjustTextFieldStyle): (WebCore::RenderThemeGtk::paintTextField): (WebCore::RenderThemeGtk::paintTextArea): (WebCore::RenderThemeGtk::adjustButtonStyle): (WebCore::RenderThemeGtk::systemFont): (WebCore::RenderThemeGtk::gtkButton): (WebCore::RenderThemeGtk::gtkCheckbox): (WebCore::RenderThemeGtk::gtkRadioButton): (WebCore::RenderThemeGtk::gtkWindowContainer):
  • platform/gtk/RenderThemeGtk.h: Added.
  • platform/gtk/ScrollViewGdk.cpp: Removed.
  • platform/gtk/ScrollViewGtk.cpp: Added.
  • platform/gtk/SearchPopupMenuGdk.cpp: Removed.
  • platform/gtk/SearchPopupMenuGtk.cpp: Added.
  • platform/gtk/SoundGdk.cpp: Removed.
  • platform/gtk/SoundGtk.cpp: Added.
  • platform/gtk/WheelEventGdk.cpp: Removed.
  • platform/gtk/WheelEventGtk.cpp: Added.
  • platform/gtk/WidgetGdk.cpp: Removed.
  • platform/gtk/WidgetGtk.cpp: Added.
  • platform/network/ResourceHandle.h:

WebKitTools:
2007-09-20 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Adam.

  • GdkLauncher: Removed.
  • GdkLauncher/Bakefiles.bkgen: Removed.
  • GdkLauncher/ENV: Removed.
  • GdkLauncher/GdkLauncher.pro: Removed.
  • GdkLauncher/gdklauncher.bkl: Removed.
  • GdkLauncher/main.cpp: Removed.
  • GdkLauncher/mk: Removed.
  • GdkLauncher/simple.svg: Removed.
  • GdkLauncher/text.html: Removed.
  • GtkLauncher: Added.
  • GtkLauncher/GdkLauncher.pro: Removed.
  • GtkLauncher/GtkLauncher.pro: Added.
  • GtkLauncher/main.cpp: (updateWindowTitle): (main):
  • Scripts/build-webkit:
  • Scripts/regenerate-makefiles:
  • Scripts/run-javascriptcore-tests:
  • Scripts/run-launcher:
  • Scripts/run-webkit-tests:
  • Scripts/webkitdirs.pm:
1:55 AM Changeset in webkit [25702] by rwlbuis
  • 4 edits
    4 adds in trunk

Reviewed by Hyatt.

http://bugs.webkit.org/show_bug.cgi?id=15025
Background repeat css property is not inherited from parent if unspecified

Don't let invalid url specification in background shorthand make
the entire shorthand invalid.

Sep 22, 2007:

9:03 PM Changeset in webkit [25701] by aroben
  • 3 edits
    2 adds in trunk

Fix Bug 15251: REGRESSION: <input type=range> doesn't respond to form.reset() or setting input.value

WebCore:

Fix Bug 15251: REGRESSION: <input type=range> doesn't respond to form.reset() or setting input.value

http://bugs.webkit.org/show_bug.cgi?id=15251
<rdar://5498169>

Reviewed by Maciej.

Test: fast/forms/range-reset.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::storesValueSeparateFromAttribute): <input type=range> should be storing its value separately from the value attribute, as do all of our variable-input input types.

LayoutTests:

Test for Bug 15251: REGRESSION: <input type=range> doesn't respond to form.reset() or setting input.value

http://bugs.webkit.org/show_bug.cgi?id=15251
<rdar://5498169>

Reviewed by Maciej.

  • fast/forms/range-reset-expected.txt: Added.
  • fast/forms/range-reset.html: Added.
2:21 PM Changeset in webkit [25700] by oliver
  • 3 edits in branches/feature-branch/WebCore

Reviewed by NOBODY (Build fix).

  • platform/graphics/svg/cg/SVGPaintServerGradientCg.cpp: (WebCore::SVGPaintServerGradient::teardown): (WebCore::SVGPaintServerGradient::renderPath):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBoxPaintWalker::chunkPortionCallback):

Sep 21, 2007:

9:20 PM Changeset in webkit [25699] by bdash
  • 5 edits in trunk/WebCore

2007-09-21 Mark Rowe <mrowe@apple.com>

Build fix for non-Mac platforms after r25697.

Rename Frame::cleanupPlatformScriptObjects to Frame::clearPlatformScriptObjects.

  • bridge/win/FrameWin.cpp:
  • page/gdk/FrameGdk.cpp:
  • page/qt/FrameQt.cpp:
  • platform/wx/TemporaryLinkStubs.cpp:
8:56 PM Changeset in webkit [25698] by adele
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Sam.

Fix for <rdar://problem/5497357> REGRESSION: Safari doesn't show the month days on AirFrance reservation page

Test: fast/dom/HTMLSelectElement/options-collection-set-string-length.html

  • bindings/js/JSHTMLOptionsCollectionCustom.cpp: (WebCore::JSHTMLOptionsCollection::setLength): Convert the value to a number. We already have code here to check if the value is not a number. This matches other browsers, which allow a string value to be used to set the length.

LayoutTests:

Reviewed by Sam.

Test for <rdar://problem/5497357> REGRESSION: Safari doesn't show the month days on AirFrance reservation page

  • fast/dom/HTMLSelectElement/options-collection-set-string-length-expected.txt: Added.
  • fast/dom/HTMLSelectElement/options-collection-set-string-length.html: Added.
5:49 PM Changeset in webkit [25697] by thatcher
  • 12 edits in trunk

WebCore:

Reviewed by Darin.

<rdar://problem/5491013> REGRESSION: -[WebView windowScriptObject] returns a dummy object or nil if a page hasn't loaded

Calling -[WebView windowScriptObject] before the page loads would give you nil. This behavior didn't match Tiger.
The API behavior in Tiger let you get the window script object once and keep ahold of it as long as you needed it.
The window object would remain valid even after page loads. This change restores the Tiger behavior.

<rdar://problem/5495790> NULL dereference crash beneath Bindings::RootObject::interpreter when saving Dashcode document

The changes to WebScriptObject's _isSafeScript call also fixed the crash in Dashcode.

  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject _setOriginRootObject:andRootObject:]): New method used to update the the root objects, so the WebScriptObject can still be used after a page load. (-[WebScriptObject _isSafeScript]): Call [self _rootObject] instead of accessing the data member directly. DOMNode has an override for the _rootObject method, and it can return 0 when _private->_rootObject is non-zero. We would return YES here when it wasn't safe and later crash with my modified layout tests. Checking _rootObject first prevents other calls sites from needing to check for a valid root object, this fixed the Dashcode crash. (-[WebScriptObject _imp]): Ditto.
  • bindings/objc/WebScriptObjectPrivate.h: Add _setOriginRootObject:andRootObject:.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::clear): Call the renamed clearScriptObjects function. (WebCore::FrameLoader::dispatchWindowObjectAvailable): Some gratuitous code cleanup.
  • page/Frame.cpp: (WebCore::Frame::clearScriptObjects): Renamed cleanupScriptObject to clearScriptObjects. Call clearPlatformScriptObjects last so m_bindingRootObject is already NULL. (WebCore::Frame::windowScriptNPObject): Hold a JSLock before accessing the window. This change is unrelated to the bug, but should be fixed.
  • page/mac/FrameMac.mm: (WebCore::Frame::windowScriptObject): Return a script object even if the interpreter is NULL. This resotres the Tiger behavior of always being able to access the window object. (WebCore::Frame::clearPlatformScriptObjects): Keep the window script object around, and update the root objects for the window script object.
  • page/Frame.h: Rename cleanupScriptObject to clearScriptObjects.
  • page/FramePrivate.h: Use a RetainPtr for m_windowScriptObject.

WebKitTools:

Reviewed by Darin.

<rdar://problem/5491013> REGRESSION: -[WebView windowScriptObject] returns a dummy object or nil if a page hasn't loaded (breaks EA Sports Online)

Assert that the -[WebScriptObject JSObject] return value is only NULL for non-window objects.
This is tested by plugins/root-object-premature-delete-crash.html.

  • DumpRenderTree/mac/ObjCController.m: (-[ObjCController accessStoredWebScriptObject]):

LayoutTests:

Reviewed by Darin.

<rdar://problem/5491013> REGRESSION: -[WebView windowScriptObject] returns a dummy object or nil if a page hasn't loaded (breaks EA Sports Online)

Updated results to show the didClearWindowScriptObject delegate call. Tweaked plugins/root-object-premature-delete-crash.html
to test a non-window script object half the time. This change works with a change to DumpRenderTree.

  • http/tests/loading/empty-subframe-expected.txt:
  • http/tests/loading/simple-subframe-expected.txt:
  • plugins/root-object-premature-delete-crash.html:
  • webarchive/loading/test-loading-archive-expected.txt:
5:30 PM Changeset in webkit [25696] by staikos
  • 2 edits in trunk/WebCore

2007-09-21 Mike Fenton <mike@staikos.net>

Reviewed by George Staikos.

Fix styled pen support by not clobbering QPen properties.

  • html/CanvasStyle.cpp: (WebCore::CanvasStyle::applyStrokeColor): (WebCore::CanvasStyle::applyFillColor):
5:00 PM Changeset in webkit [25695] by oliver
  • 1 edit in trunk/WebCore/ChangeLog

Darin reviewed as well, and pointed out mistake in ChangeLog

4:51 PM Changeset in webkit [25694] by oliver
  • 2 edits in trunk/WebCore

Reviewed by Tristan.

http://bugs.webkit.org/show_bug.cgi?id=15239
<rdar://problem/5491955> REGRESSION (r25547): With Hanin IME, one cannot input characters into forms (15239)


This problem was caused by the setComposition method failing to clearing
the composition markers in the presence of an empty string. This would
result in the caret being locked within the 0 length bounds of the "composition"
and so preventing basic keyboard interaction.

  • editing/Editor.cpp: (WebCore::Editor::setComposition):
4:29 PM Changeset in webkit [25693] by adele
  • 3 edits in trunk/WebCore

Reviewed by Tristan.

Fix for <rdar://problem/5497346> "Check spelling when I click Send" crashes on stationery messages every time

No test. I wasn't able to find a way to get this to occur with Safari or DumpRenderTree.

  • editing/Editor.cpp: (WebCore::Editor::advanceToNextMisspelling): Updated comment.
  • editing/htmlediting.cpp: (WebCore::firstEditablePositionAfterPositionInRoot): Only return the root position if its editable. (WebCore::maxDeepOffset): Added nil check & assert.
4:18 PM Changeset in webkit [25692] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Steve.

<rdar://problem/5404339>
Site redirects continuosly, starving UI thread, causing app to hang.


Use GetQueueStatus to see if there are input messages in the queue that aren't being processed.
If so, use a low-priority timer instead of PostMessage.


  • platform/win/SharedTimerWin.cpp: (WebCore::setSharedTimerFireTime):
4:11 PM Changeset in webkit [25691] by tristan
  • 8 edits in branches/Safari-2.0-dashboard/WebCore

Merge r24938 and r24946 from ToT to Safari-2.0-dashboard.


2007-08-08 Sam Weinig <sam@webkit.org>


Reviewed by Geoff Garen.

Fix for <rdar://problem/5354635>

Match Firefox's model for data: URLs by not allowing them script access
to any frames other then itself.

  • bindings/js/kjs_window.cpp: (KJS::Window::isSafeScript):
  • dom/Document.cpp: (WebCore::Document::initSecurityPolicyURL):


2007-08-08 Sam Weinig <sam@webkit.org>

Reviewed by Brady.

Make protocol and host compares case-insensitive.

  • bindings/js/kjs_window.cpp: (KJS::Window::isSafeScript):
  • dom/Document.cpp: (WebCore::Document::initSecurityPolicyURL):
  • platform/DeprecatedString.cpp: (WebCore::equalIgnoringCase):
  • platform/DeprecatedString.h: (WebCore::equalIgnoringCase):
3:47 PM Changeset in webkit [25690] by darin
  • 4 edits in trunk/WebCore

Reviewed by Anders.

  • add QueryInterface capabilities to COMPtr
  • platform/win/COMPtr.h: (COMPtr::COMPtr): Added constructor that takes a Query tag and does an appropriate QueryInterface. (COMPtr::query): Added function to do a queryInterface. (COMPtr::copyQueryInterfaceRef): Added private helper used by both of the above. Note that when the query fails you get a 0, which is the same thing you get if a 0 pointer is passed in.
  • platform/win/FontCacheWin.cpp: (WebCore::FontCache::getFontLinkInterface): Convert to using the new query as a test case. Also eliminate the unnecessary second global variable and use COMPtr objects rather than just leaking references.
  • platform/win/WCDataObject.h: Removed unneeded include and using statements.
2:58 PM Changeset in webkit [25689] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Adam.

<rdar://problem/5494790>
clicking any link in the Flash regions at bananarepublic.com results in bad page

Don't replace the frame contents, the Mac version doesn't do this.


  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::performRequest):
1:15 PM Changeset in webkit [25688] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Darin.

<rdar://problem/5496700>
Repro crash loading http://www.tivo.com/whatistivo/tivohd/index.html

Don't continue if the plug-in cancelled the stream.


  • loader/win/NetscapePlugInStreamLoaderWin.cpp: (WebCore::NetscapePlugInStreamLoader::didReceiveResponse):
1:12 PM Changeset in webkit [25687] by kmccullo
  • 2 edits in trunk/WebKitTools

Reviewed by Oliver.

  • We need to initialize the ref count to 1 at initialization so that objects are not garbage collected. (actual patch written by Oliver)
  • DumpRenderTree/win/FrameLoaderDelegate.cpp: (FrameLoadDelegate::FrameLoadDelegate):
12:48 PM Changeset in webkit [25686] by kdecker
  • 2 edits in trunk/WebKit
  • Plugins/WebBaseNetscapePluginView.mm: Build fix. The first argument of aglChoosePixelFormat() has changed from const AGLDevice *gdevs on Tiger to const void *gdevs on Leopard.
12:15 PM Changeset in webkit [25685] by kdecker
  • 3 edits in trunk/WebKit

Reviewed by Darin Adler.

<rdar://problem/5493093>

Reduced launch time by lazily linking both the AGL and OpenGL frameworks until they are really needed.

  • Plugins/WebBaseNetscapePluginView.mm: Soft link all AGL and OpenGL functions used by WebBaseNetscapePluginView.
  • WebKit.xcodeproj/project.pbxproj: Removed AGL and OpenGL from the project.
12:14 PM Changeset in webkit [25684] by kdecker
  • 2 edits in trunk/WebCore
  • platform/mac/SoftLinking.h: Build fix; added necessary #imports.
12:06 PM Changeset in webkit [25683] by kdecker
  • 2 edits
    1 add in trunk/WebCore

Reviewed by Darin Adler.

  • WebCore.xcodeproj/project.pbxproj: Added SoftLinking.h to the project.
  • platform/mac/SoftLinking.h: Contains a new macro; useful in helping to lazily load frameworks.
10:15 AM Changeset in webkit [25682] by kmccullo
  • 2 edits in trunk/WebKit/win
  • Fixed a syntax guideline mistake.
  • WebChromeClient.cpp: (WebChromeClient::scrollbarsVisible):
4:06 AM Changeset in webkit [25681] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-09-21 Mark Rowe <mrowe@apple.com>

Reviewed by Antti Koivisto.

http://bugs.webkit.org/show_bug.cgi?id=15250
<rdar://problem/5496942> REGRESSION: Reproducible crash in Safari when evaluating script in Drosera console (15250)

  • kjs/function.cpp: (KJS::GlobalFuncImp::callAsFunction): Null-check thisObj before passing it to interpreterForGlobalObject.
12:02 AM Changeset in webkit [25680] by oliver
  • 3 edits in trunk/WebKit/win

2007-09-20 Oliver Hunt <oliver@apple.com>

Reviewed by Steve.

<rdar://problem/5487153> Korean characters are displayed as garbage with default encoding in some Korean web sites

Adding support for per-localisation default character encodings on
windows.

  • English.lproj/Localizable.strings:
  • WebPreferences.cpp: (WebPreferences::initializeDefaultSettings):

Sep 20, 2007:

5:48 PM Changeset in webkit [25679] by kmccullo
  • 2 edits in trunk/WebKit/win

Reviewed by Oliver.

  • Added the ability to ask if scrollbars are visible
  • <rdar://problem/5496211> scrollbarsVisible in WebChromeClient is not implemented
  • WebChromeClient.cpp: (WebChromeClient::scrollbarsVisible):
5:28 PM Changeset in webkit [25678] by darin
  • 1 edit in trunk/WebKitLibraries/win/lib

Ignore WebKitAuxiliaryLibraries files put here by update-webkit.

5:27 PM Changeset in webkit [25677] by darin
  • 1 edit in trunk/WebKitLibraries/win/include

Ignore WebKitAuxiliaryLibraries files put here by the update-webkit script.

5:19 PM Changeset in webkit [25676] by adachan
  • 6 edits in trunk

<rdar://problem/5477240> Regression: Footer is too high in print preview


Reviewed by Steve.

WebCore:

Make computePageRectsForFrame() return the height of the page adjusted for margins.

  • bridge/win/FrameWin.cpp: (WebCore::computePageRectsForFrame):
  • bridge/win/FrameWin.h:


WebKit/win:

  • WebFrame.cpp: (WebFrame::WebFrame): initialize new data member m_pageHeight, which is the height of the page adjusted for margins. (WebFrame::computePageRects): get the height of the page adjusted for margins by passing m_pageHeight to computePageRectsForFrame(). (WebFrame::spoolPages): footer rect's top is the max of the bottom of the page content and the bottom of the page minus footer height.
  • WebFrame.h:
4:26 PM Changeset in webkit [25675] by kmccullo
  • 8 edits in trunk

WebCore:

Reviewed by Hyatt.

  • <rdar://problem/5101991> Avril Lavigne music player comes up in a window with scrollbars
  • Implementing missing scrollbar functionality to allow turning off scrollbars correctly.
  • platform/ScrollView.h:
  • platform/win/ScrollViewWin.cpp: (WebCore::ScrollView::ScrollViewPrivate::setAllowsScrolling): (WebCore::ScrollView::ScrollViewPrivate::allowsScrolling): (WebCore::ScrollView::updateScrollbars): (WebCore::ScrollView::setAllowsScrolling): (WebCore::ScrollView::allowsScrolling):

win:

Reviewed by Hyatt.

  • <rdar://problem/5101991> Avril Lavigne music player comes up in a window with scrollbars
  • Implementing missing scrollbar functionality to allow turning off scrollbars correctly.
  • Interfaces/IWebFrame.idl:
  • Interfaces/IWebFrameView.idl:
  • WebChromeClient.cpp: (WebChromeClient::setScrollbarsVisible):
  • WebFrame.cpp: (WebFrame::setAllowsScrolling): (WebFrame::allowsScrolling): (WebFrame::frameView):
  • WebFrame.h:
4:13 PM Changeset in webkit [25674] by beidson
  • 6 edits in trunk

WebCore:

Reviewed by Dave Hyatt

<rdar://problem/5245981> - No favicon shows up for cnet.com


CNet's favicon has two images in it - a 32x32 white square, and a 16x16 image that is their actual icon
Till now on Windows we've always gone straight for the first image in an icon for the favicon, now we
actually do proper size matching

  • platform/graphics/BitmapImage.h:
  • platform/graphics/Image.h: (WebCore::Image::getHBITMAPOfSize): (WebCore::Image::drawFrameMatchingSourceSize): Search through the frames of the image for the correct size
  • platform/graphics/win/ImageWin.cpp: (WebCore::BitmapImage::getHBITMAP): (WebCore::BitmapImage::getHBITMAPOfSize): (WebCore::BitmapImage::drawFrameMatchingSourceSize):

WebKit/win:

Reviewed by Dave Hyatt

<rdar://problem/5245981> - No favicon shows up for cnet.com


  • WebIconDatabase.cpp: (WebIconDatabase::iconForURL): Call getHBITMAPOfSize (WebIconDatabase::getOrCreateDefaultIconBitmap): Ditto
2:58 PM Changeset in webkit [25673] by sfalken
  • 2 edits in trunk/WebCore

Reviewed by Brady.

Call ascii() instead of utf8() from debug logging code to avoid threading
issues associated w/ converting to UTF8.

11:53 AM Changeset in webkit [25672] by staikos
  • 4 edits in trunk/WebCore

2007-09-20 Mike Fenton <mike@staikos.net>

Reviewed by George Staikos.

Implement gradients and more of canvas for Qt. Style changes and some
of the stroking fixes by George.

  • html/CanvasGradient.cpp: (WebCore::CanvasGradient::CanvasGradient): (WebCore::CanvasGradient::~CanvasGradient): (WebCore::CanvasGradient::addColorStop): (WebCore::CanvasGradient::platformShading):
  • html/CanvasGradient.h:
  • html/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::fill): (WebCore::CanvasRenderingContext2D::stroke): (WebCore::CanvasRenderingContext2D::fillRect): (WebCore::CanvasRenderingContext2D::applyFillPattern):
10:26 AM Changeset in webkit [25671] by sullivan
  • 2 edits in trunk/WebKit

Build fix for stoooopid old PPC gcc compiler

  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::checkSpellingOfString): replace perfectly valid ?: syntax with if/else
4:05 AM Changeset in webkit [25670] by ddkilzer
  • 3 edits in trunk/WebKitTools

WebKitTools:

Reviewed by Adam.

Fix: http://bugs.webkit.org/show_bug.cgi?id=15223
webkitdir.pm::isQt() is not working properly in run-webkit-tests under Linux/Qt

Added determineIsQt() and determineIsGdk() to webkitdirs.pm.
Now isQt() and isGdk() use global variables that are only set once.

An array of all the parameters is now given to build-dumprender which includes
the platform on linux.

  • Scripts/build-dumprendertree:
  • Scripts/run-webkit-tests:
  • Scripts/webkitdirs.pm:
3:53 AM Changeset in webkit [25669] by ddkilzer
  • 1 edit in trunk/WebCore/ChangeLog

Added radar.

1:52 AM Changeset in webkit [25668] by rwlbuis
  • 3 edits
    2 adds in trunk

Reviewed by Mitz.

http://bugs.webkit.org/show_bug.cgi?id=15235
Options can not be removed from a Select element with OptGroup's using JavaScript

Remove HTMLOptionElement children against the proper parent node.

Sep 19, 2007:

11:44 PM Changeset in webkit [25667] by sfalken
  • 1 copy in tags/Safari-523.7b

New tag.

11:44 PM Changeset in webkit [25666] by sfalken
  • 1 edit in trunk/WebKit/win/WebKit.vcproj/VERSION

Bump version for submit

10:56 PM Changeset in webkit [25665] by weinig
  • 2 edits in trunk/WebKit/win

Rubber stamped by Adam Roben.

  • WebKit.vcproj/WebKit.sln: Update location of DumpRenderTree and TestNetscapePlugin to point to their new locations.
7:03 PM Changeset in webkit [25664] by bdash
  • 4 edits in trunk

Versioning.

7:03 PM Changeset in webkit [25663] by bdash
  • 1 copy in tags/Safari-4523.7

New tag.

7:02 PM Changeset in webkit [25662] by bdash
  • 1 copy in tags/Safari-5523.7

New tag.

5:43 PM Changeset in webkit [25661] by sullivan
  • 4 edits in trunk

WebCore:

Reviewed by Darin Adler.


  • WebCore part of speculative fix for <rdar://problem/5490627>, about crashes constructing a String using the values filled in by checkSpellingOfString()
  • editing/Editor.cpp: (WebCore::findFirstMisspellingInRange): initialize out parameters the way we do elsewhere; add many assertions for improper results from checkSpellingOfString(); don't create a String at all until we've checked all the ways that checkSpellingOfString() results could indicate invalid string, even the unexpected ones; as an optimization, don't construct a String at all when looping through to mark all instances.

WebKit:

Reviewed by Darin Adler.

  • WebKit part of speculative fix for <rdar://problem/5490627>, about crashes constructing a String using the values filled in by checkSpellingOfString()
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::checkSpellingOfString): convert NSNotFound to -1, since WebCore code expects -1 for this purpose. We already do this in checkGrammarOfString.
5:17 PM Changeset in webkit [25660] by aroben
  • 1 edit
    1 delete in trunk/WebKitTools

Rubberstamped by Hyatt.

  • Spinneret: Removed.
5:13 PM Changeset in webkit [25659] by bdash
  • 2 edits in branches/Safari-2.0-dashboard/WebKit

2007-09-19 Mark Rowe <mrowe@apple.com>

Merge r18567 from trunk to Safari-2.0-dashboard.

2007-01-03 John Sullivan <sullivan@apple.com>

  • WebView/WebPDFView.mm: (-[WebPDFView _openWithFinder:]): Tiger build fix: use [NSNumber initWithInt:] rather than the new [NSNumber initWithInteger:]
4:55 PM Changeset in webkit [25658] by zecke
  • 2 edits in trunk/JavaScriptCore

2007-09-19 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber stamped by Adam.

Make the guard/#if use the same name (ENABLE_FTPDIR) as the #define. This follows
the ENABLE_ICONDATABASE example from a couple of lines above.

  • wtf/Platform.h:
4:14 PM Changeset in webkit [25657] by zecke
  • 1 edit in trunk/WebCore/ChangeLog

Restore the ChangeLog entries accidently removed by r25646 which was spotted by Mark.

4:00 PM Changeset in webkit [25656] by zecke
  • 4 edits in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

There is no need to have FTPDirectoryDocument stubs. We can
compile FTPDirectoryDocument.cpp and FTPDirectoryTokenizer.cpp
instead.

  • WebCore.pro:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/qt/TemporaryLinkStubs.cpp:
4:00 PM Changeset in webkit [25655] by zecke
  • 2 edits in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Remove includes that are not needed.

  • platform/gdk/TemporaryLinkStubs.cpp:
3:59 PM Changeset in webkit [25654] by zecke
  • 3 edits
    1 add in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the Pasteboard stubs to PasteboardGdk.cpp.

  • WebCore.pro:
  • platform/gdk/PasteboardGdk.cpp: Added. (WebCore::Pasteboard::generalPasteboard): (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::writeURL): (WebCore::Pasteboard::writeImage): (WebCore::Pasteboard::clear): (WebCore::Pasteboard::canSmartReplace): (WebCore::Pasteboard::documentFragment): (WebCore::Pasteboard::plainText): (WebCore::Pasteboard::Pasteboard):
  • platform/gdk/TemporaryLinkStubs.cpp:
3:57 PM Changeset in webkit [25653] by zecke
  • 3 edits
    1 add in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the Icon stubs to IconGdk.cpp

  • WebCore.pro:
  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/graphics/gdk/IconGdk.cpp: Added. (WebCore::Icon::Icon): (WebCore::Icon::~Icon): (WebCore::Icon::newIconForFile):
3:48 PM Changeset in webkit [25652] by zecke
  • 3 edits in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the Font stubs to FontGdk.cpp

  • platform/gdk/FontGdk.cpp: (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForComplexText): (WebCore::Font::selectionRectForComplexText):
  • platform/gdk/TemporaryLinkStubs.cpp:
3:45 PM Changeset in webkit [25651] by zecke
  • 3 edits
    1 add in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the SearchPopupMenu stubs to SearchPopupMenuGdk.cpp

  • WebCore.pro:
  • platform/gdk/SearchPopupMenuGdk.cpp: Added. (WebCore::SearchPopupMenu::SearchPopupMenu): (WebCore::SearchPopupMenu::saveRecentSearches): (WebCore::SearchPopupMenu::loadRecentSearches): (WebCore::SearchPopupMenu::enabled):
  • platform/gdk/TemporaryLinkStubs.cpp:
3:39 PM Changeset in webkit [25650] by zecke
  • 3 edits
    2 adds in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the ContextMenu and ContextMenuItem stubs to
ContextMenuGdk.cpp and ContextMenuItemGdk.cpp.

  • WebCore.pro:
  • platform/gdk/ContextMenuGdk.cpp: Added. (WebCore::ContextMenu::ContextMenu): (WebCore::ContextMenu::~ContextMenu): (WebCore::ContextMenu::appendItem): (WebCore::ContextMenu::setPlatformDescription): (WebCore::ContextMenu::platformDescription): (WebCore::ContextMenu::releasePlatformDescription):
  • platform/gdk/ContextMenuItemGdk.cpp: Added. (WebCore::ContextMenuItem::ContextMenuItem): (WebCore::ContextMenuItem::~ContextMenuItem): (WebCore::ContextMenuItem::releasePlatformDescription): (WebCore::ContextMenuItem::type): (WebCore::ContextMenuItem::setType): (WebCore::ContextMenuItem::action): (WebCore::ContextMenuItem::setAction): (WebCore::ContextMenuItem::title): (WebCore::ContextMenuItem::setTitle): (WebCore::ContextMenuItem::platformSubMenu): (WebCore::ContextMenuItem::setSubMenu): (WebCore::ContextMenuItem::setChecked):
  • platform/gdk/TemporaryLinkStubs.cpp:
3:36 PM Changeset in webkit [25649] by zecke
  • 3 edits in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Implement the WebCore::fileSize function using g_stat.

  • platform/gdk/FileSystemGdk.cpp: (WebCore::fileSize):
  • platform/gdk/TemporaryLinkStubs.cpp:
3:33 PM Changeset in webkit [25648] by zecke
  • 4 edits in trunk/WebCore

2007-09-18 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

Move the ResourceHandle stubs from TemporaryLinkStubs
to the ResourceHandleCurl.cpp file. This affects the
Gtk+ and wx port as they share the curl implementation.

  • platform/gdk/TemporaryLinkStubs.cpp:
  • platform/network/curl/ResourceHandleCurl.cpp: (WebCore::ResourceHandle::willLoadFromCache): (WebCore::ResourceHandle::loadsBlocked):
  • platform/wx/TemporaryLinkStubs.cpp:
3:26 PM Changeset in webkit [25647] by zecke
  • 4 edits in trunk/WebKit/gtk

2007-09-17 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Adam.

WebKitGtkPage and WebKitGtkFrame have a dependency on each other. To
allow client code to include both headers the typedef's for
WebKitGtkFrame, WebKitGtkPage, WebKitGtkFrameData and WebKitGtkNetworkRequest
are moved into webkitgtkdefines.h and included by both webkitgtkpage.h and
webkitgtkframe.h

  • Api/webkitgtkdefines.h:
  • Api/webkitgtkframe.h:
  • Api/webkitgtkpage.h:
3:17 PM Changeset in webkit [25646] by zecke
  • 2 edits in trunk/WebCore

2007-09-17 Holger Hans Peter Freyther <zecke@selfish.org>

Reviewed by Mark.

Take http://bugs.webkit.org/show_bug.cgi?id=15221 into account
and change the variable names from nameSystem to systemName and
change the name of the function to stringByAdoptingFileSystemRepresentation.

  • platform/gdk/FileChooserGdk.cpp: (WebCore::stringByAdoptingFileSystemRepresentation): (WebCore::FileChooser::basenameForWidth):
3:02 PM Changeset in webkit [25645] by kmccullo
  • 2 edits in trunk/LayoutTests

Reviewed by.

  • Layout Test fixe
  • fast/dom/Window/new-window-opener.html:
2:25 PM Changeset in webkit [25644] by kmccullo
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by Geoff and Sam.

  • Added test cases for the different features of a new opened window.
  • fast/dom/Window/new-window-opener-expected.txt: Added.
  • fast/dom/Window/new-window-opener.html: Added.
1:51 PM Changeset in webkit [25643] by weinig
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Geoff Garen.

Fix <rdar://problem/5492141> Assertion in isdigit() on windows Safari

Test: fast/css/hexColor-isDigit-assert.html

  • platform/DeprecatedString.cpp: (WebCore::isCharacterAllowedInBase): Check that the character is ascii to avoid an assertions on Windows.

LayoutTests:

Reviewed by Geoff Garen.

Test for <rdar://problem/5492141> Assertion in isdigit() on windows Safari

  • fast/css/hexColor-isDigit-assert-expected.txt: Added.
  • fast/css/hexColor-isDigit-assert.html: Added.
1:17 PM Changeset in webkit [25642] by andersca
  • 3 edits in trunk/WebCore

Reviewed by Geoff.

<rdar://problem/5489879>
Combination of real player with high connection quality in selecting player makes Safari quit.


Prevent the Real Player plug-in from calling the window proc recursively.


  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::wndProc): (WebCore::PluginViewWin::setNPWindowRect): (WebCore::PluginViewWin::determineQuirks): (WebCore::PluginViewWin::PluginViewWin):
  • plugins/win/PluginViewWin.h: (WebCore::):
11:04 AM Changeset in webkit [25641] by adele
  • 25 edits
    1 move
    2 adds
    2 deletes in trunk

WebCore:

Reviewed by Maciej.

Fix for <rdar://problem/5472062> -webkit-user-select: none makes selection difficult
and for <rdar://problem/5472056> REGRESSION: Cannot type into edit fields on a form (sccsheriff.org)

Tests:

This fix has a few steps:
1) Removes the ignore value for the -webkit-user-select property. We now decide when to prevent selection from starting by calling canStartSelection on the node.
2) Implements canStartSelection which returns false for all elements that used to have -webkit-user-select:ignore set, true for content editable nodes,

and walks up the tree to ask the parent before allowing selection to be started on any other nodes.

3) We used to disallow selection from starting within -webkit-user-select:none blocks, but now we only use canStartSelection for that. This will allow easy selections

in cases like iChat where there's a mix of selectable and non-selectable content.

4) Makes -webkit-user-select inherited. After removing the "ignore" value, -webkit-user-select would behave like it was inherited anyways since we already allowed the text value to override the none value.

  • css/html4.css: Removed all usage of -webkit-user-select: ignore. This is now handled internally with canStartSelection(). Now -webkit-user-select won't be unintentionally overridden by nodes that don't want to allow selection to be started, but do want to honor the -webkit-user-select to determine whether or not selection is allowed at all.
  • editing/SelectionController.cpp: (WebCore::SelectionController::selectAll): Don't try to determine whether selection is allowed inside the root node here. VisiblePosition and Selection creation will keep Selection endpoints out of -webkit-user-select:none regions.


  • dom/Node.h:
  • dom/Node.cpp: (WebCore::Node::canStartSelection): Added. Always allows selection to be started in a content editable node. If there's a parent, ask the parent if starting a selection is allowed. If there's no parent, default to allowing selection.
  • html/HTMLAnchorElement.h:
  • html/HTMLAnchorElement.cpp: (WebCore::HTMLAnchorElement::canStartSelection): Added. If its not a link, then calls the base class. Disallows selection for non-editable links. Allows selection for editable links. This logic used to be done in CSSStyleSelector by changing the user-select property.
  • html/HTMLButtonElement.h: (WebCore::HTMLButtonElement::canStartSelection): Added. Disallows selection from starting in buttons.
  • html/HTMLImageElement.h: (WebCore::HTMLImageElement::canStartSelection): Added. Disallows selection from starting in images.
  • html/HTMLInputElement.h:
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::canStartSelection): Added. Disallows selection from anything that's not a text field. Text fields call up to the base class. When they're editable, text controls will always be selectable because of the case for content editable content. Otherwise, they'll respect their parent element's decision about starting a selection.
  • html/HTMLSelectElement.h: (WebCore::HTMLSelectElement::canStartSelection): Added. Disallows selection from starting in popup buttons.
  • page/EventHandler.cpp: (WebCore::EventHandler::selectClosestWordFromMouseEvent): Use canMouseDownStartSelect instead of the ambiguously named shouldSelect(). (WebCore::EventHandler::handleMousePressEventTripleClick): ditto. (WebCore::EventHandler::handleMousePressEventSingleClick): ditto. (WebCore::EventHandler::updateSelectionForMouseDrag): Use canMouseDragExtendSelect. (WebCore::EventHandler::selectCursor): Paint an ibeam in any region that allows you to click to create a selection. (WebCore::EventHandler::canMouseDownStartSelect): Now fires the selectStart event, and calls canStartSelection instead of checking the user-select property. (WebCore::EventHandler::canMouseDragExtendSelect): This is identical to canMouseDownStartSelect because of 12823, even though it seems strange that we would fire the selectStart event here.
  • page/EventHandler.h:
  • page/FocusController.cpp: (WebCore::clearSelectionIfNeeded): Calls canStartSelection instead of checking the user-select property to decide when to prevent selection creation.
  • rendering/RenderObject.cpp: (WebCore::RenderObject::draggableNode): ditto.
  • rendering/RenderObject.h: Removed helper methods that are no longer needed.
  • rendering/RenderStyle.h: Made userSelect inherited. After removing the "ignore" value, -webkit-user-select would behave like it was inherited anyways since we already allowed the text value to override the none value. Removed the SELECT_AUTO and SELECT_IGNORE values since they're no longer used. (WebCore::): (WebCore::RenderStyle::userSelect): (WebCore::RenderStyle::setUserSelect): (WebCore::RenderStyle::initialUserSelect):
  • rendering/RenderStyle.cpp: (WebCore::StyleRareNonInheritedData::StyleRareNonInheritedData): (WebCore::StyleRareNonInheritedData::operator==): (WebCore::StyleRareInheritedData::StyleRareInheritedData): (WebCore::StyleRareInheritedData::operator==): (WebCore::RenderStyle::diff):
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Remove case for SELECT_AUTO and SELECT_IGNORE
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::adjustRenderStyle): Remove adjustment for links. This is now handled in HTMLAnchorElement::canStartSelection. (WebCore::CSSStyleSelector::applyProperty): Map CSS_VAL_AUTO to SELECT_TEXT. Remove CSS_VAL_IGNORE.

LayoutTests:

Reviewed by Maciej.

Tests for <rdar://problem/5472062> -webkit-user-select: none makes selection difficult
and for <rdar://problem/5472056> REGRESSION: Cannot type into edit fields on a form (sccsheriff.org)

  • fast/css/computed-style-expected.txt: Updated for default -webkit-user-select value.
  • fast/css/computed-style-without-renderer-expected.txt: ditto.
  • editing/selection/5333725.html: Re-enabled.
  • editing/selection/5333725.html-disabled: Removed.
  • editing/selection/user-select-text-inside-user-select-ignore.html-disabled: Removed. Adjusted this test to be more specific and renamed it to textfield-inside-anchor.
  • editing/selection/user-select-text-inside-user-select-ignore-expected.txt: Removed.
  • fast/forms/textfield-inside-anchor-expected.txt: Added.
  • fast/forms/textfield-inside-anchor.html: Added.
10:59 AM Changeset in webkit [25640] by kdecker
  • 3 edits in trunk/WebKit

Reviewed by Darin Adler.

<rdar://problem/5491066> soft link Accelerate.framework


  • Misc/WebGraphicsExtras.c: (WebConvertBGRAToARGB): Improve launch time performance and reduce vsize footprint by soft linking the Accelerate.framework.
  • WebKit.xcodeproj/project.pbxproj: Remove no longer needed frameworks.
9:11 AM Changeset in webkit [25639] by ddkilzer
  • 2 edits in trunk/WebKitTools

WebKitTools:

Reviewed by David Kilzer.

  • Scripts/webkitdirs.pm: On Gdk and Qt, check that flex, bison, gperf are available.
7:40 AM Changeset in webkit [25638] by staikos
  • 2 edits in trunk/WebCore

2007-09-19 George Staikos <staikos@kde.org>

As discussed with Brady and Maciej, on platforms without pthreads, use
IconDatabaseNone for now and don't use pthreads. Fixes Qt Win32 build
for now.

  • loader/icon/IconDatabase.h:
3:07 AM Changeset in webkit [25637] by bdash
  • 2 edits in trunk/JavaScriptCore

2007-09-19 Mark Rowe <mrowe@apple.com>

Reviewed by Maciej.

<rdar://problem/5487107> NULL dereference crash in FastMallocZone::enumerate when running leaks against Safari

Storing remote pointers to their local equivalents in mapped memory was leading to the local pointer being
interpreted as a remote pointer. This caused a crash when using the result of mapping this invalid remote pointer.
The fix is to follow the pattern used elsewhere in FastMallocZone by always doing the mapping after reading and
never storing the mapped pointer.

  • wtf/FastMalloc.cpp: (WTF::FastMallocZone::enumerate):
2:45 AM Changeset in webkit [25636] by antti
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Maciej.

Fix <rdar://problem/5488539>
(9A550) iPhone Banner on O2.UK site not clickable with Safari due to styled noscript element


Don't create renderer for noscript if scripts are enabled.


Test: fast/dom/noscript-style.html

  • html/HTMLElement.cpp: (WebCore::HTMLElement::mapToEntry): (WebCore::HTMLElement::parseMappedAttribute): (WebCore::HTMLElement::checkDTD): (WebCore::HTMLElement::rendererIsNeeded): (WebCore::HTMLElement::createRenderer):
  • html/HTMLElement.h:

LayoutTests:

Reviewed by Maciej.


Test for <rdar://problem/5488539>
(9A550) iPhone Banner on O2.UK site not clickable with Safari due to styled noscript element

  • fast/dom/noscript-style-expected.txt: Added.
  • fast/dom/noscript-style.html: Added.
Note: See TracTimeline for information about the timeline view.