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

Timeline



Dec 8, 2007:

11:56 PM Changeset in webkit [28572] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Oliver.

Cleanup kjs_window.h/cpp.

  • bindings/js/kjs_window.cpp: (KJS::WindowPrivate::WindowPrivate): (KJS::DOMWindowTimer::DOMWindowTimer): (KJS::Window::Window): (KJS::Window::retrieveWindow): (KJS::Window::retrieveActive): (KJS::Window::retrieve): (KJS::Window::location): (KJS::Window::mark): (KJS::allowPopUp): (KJS::parseModalDialogFeatures): (KJS::floatFeature): (KJS::canShowModalDialog): (KJS::canShowModalDialogNow): (KJS::showModalDialog): (KJS::Window::getValueProperty): (KJS::Window::getOwnPropertySlot): (KJS::Window::globalExec): (KJS::Window::setListener): (KJS::Window::getListener): (KJS::Window::findOrCreateJSEventListener): (KJS::Window::findOrCreateJSUnprotectedEventListener): (KJS::Window::clearHelperObjectProperties): (KJS::Window::setCurrentEvent): (KJS::WindowProtoFuncAToB::callAsFunction): (KJS::WindowProtoFuncBToA::callAsFunction): (KJS::WindowProtoFuncOpen::callAsFunction): (KJS::WindowProtoFuncSetTimeout::callAsFunction): (KJS::WindowProtoFuncClearTimeout::callAsFunction): (KJS::WindowProtoFuncSetInterval::callAsFunction): (KJS::WindowProtoFuncAddEventListener::callAsFunction): (KJS::WindowProtoFuncRemoveEventListener::callAsFunction): (KJS::WindowProtoFuncShowModalDialog::callAsFunction): (KJS::WindowProtoFuncNotImplemented::callAsFunction):
  • bindings/js/kjs_window.h: (KJS::PausedTimeouts::PausedTimeouts): (KJS::PausedTimeouts::takeTimeouts): (KJS::ScheduledAction::ScheduledAction):
11:48 PM Changeset in webkit [28571] by weinig@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Adam Roben.

  • fast/dom/Window/window-resize-and-move-arguments.html: Make the window small enough that this test won't fail on systems with lower resolutions.
4:47 PM Changeset in webkit [28570] by weinig@apple.com
  • 10 edits
    4 adds in trunk

WebCore:

Reviewed by Mitz.

Move window scrolling, moving and resizing methods from KJS::Window
to WebCore::DOMWindow so that there bindings can be autogenerated.

Tests: fast/dom/Window/window-resize-and-move-arguments.html

fast/dom/Window/window-scroll-arguments.html

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/kjs_window.cpp: (KJS::Window::getValueProperty): Remove extraneous allowsAccessFrom check. (KJS::WindowProtoFuncOpen::callAsFunction): (KJS::WindowProtoFuncNotImplemented::callAsFunction): Remove extraneous allowsAccessFrom check.
  • bindings/js/kjs_window.h:
  • bindings/scripts/CodeGeneratorJS.pm: Add new extended attribute to ensure that the no less than the declared number of attributes is allowed.
  • page/DOMWindow.cpp: (WebCore::DOMWindow::adjustWindowRect): Moved from kjs_window. (WebCore::DOMWindow::scrollBy): (WebCore::DOMWindow::scrollTo): (WebCore::DOMWindow::moveBy): (WebCore::DOMWindow::moveTo): (WebCore::DOMWindow::resizeBy): (WebCore::DOMWindow::resizeTo):
  • page/DOMWindow.h: (WebCore::DOMWindow::scroll):
  • page/DOMWindow.idl:

LayoutTests:

Reviewed by Sam W.

Split the ENABLE_SVG_EXPERIMENTAL_FEATURES flag into separate flags.


Fixes <rdar://problem/5620249> Must disable SVG animation
<rdar://problem/5612772> Disable SVG filters on Mac to match Windows behavior

These layout test changes are caused by some of the experimental
features no longer being enabled by default.

  • fast/dom/Window/window-properties-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-blend-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-color-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-composite-02-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-comptran-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-conv-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-diffuse-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-displace-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-example-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-felem-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-gauss-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-image-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-light-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-morph-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-offset-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-specular-01-f-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-tile-01-b-expected.txt:
  • platform/mac/svg/W3C-SVG-1.1/filters-turb-01-f-expected.txt:
  • platform/mac/svg/custom/empty-merge-expected.txt:
  • platform/mac/svg/custom/feComponentTransfer-Discrete-expected.txt:
  • platform/mac/svg/custom/feComponentTransfer-Gamma-expected.txt:
  • platform/mac/svg/custom/feComponentTransfer-Linear-expected.txt:
  • platform/mac/svg/custom/feComponentTransfer-Table-expected.txt:
  • platform/mac/svg/custom/feDisplacementMap-01-expected.txt:
  • platform/mac/svg/custom/filter-source-alpha-expected.txt:
  • platform/mac/svg/custom/image-with-transform-clip-filter-expected.txt:
  • platform/mac/svg/custom/invalid-css-expected.txt:
  • platform/mac/svg/custom/non-opaque-filters-expected.txt:
  • platform/mac/svg/custom/text-filter-expected.txt:
  • platform/mac/svg/custom/visibility-override-filter-expected.txt:
  • svg/batik/filters/filterRegions-expected.txt:
  • svg/batik/text/textEffect-expected.txt:
  • svg/batik/text/textEffect3-expected.txt:
  • svg/batik/text/textFeatures-expected.txt:
  • svg/batik/text/textProperties-expected.txt:
  • svg/custom/js-svg-constructors-expected.txt:
  • svg/dom/font-face-elements-expected.txt:
  • svg/webarchive/svg-feimage-subresources-expected.txt:
4:32 PM Changeset in webkit [28569] by alp@webkit.org
  • 3 edits in trunk/WebCore

2007-12-08 Kevin Ollivier <​kevino@theolliviers.com>

Reviewed by Alp Toker.

​http://bugs.webkit.org/show_bug.cgi?id=14651
[CURL] didReceiveResponse() only called for HTTP loads

​http://bugs.webkit.org/show_bug.cgi?id=14583
[GDK] ​file:// relative CSS include URLs handled incorrectly

Make sure CURL sets the ResourceResponse URL and calls
didReceiveResponse for local files too.

  • platform/network/curl/ResourceHandleManager.cpp: (WebCore::writeCallback):
4:27 PM Changeset in webkit [28568] by oliver@apple.com
  • 241 edits
    2 adds in trunk

Split the ENABLE_SVG_EXPERIMENTAL_FEATURES flag into separate flags.

Reviewed by Sam W.

Fixes <rdar://problem/5620249> Must disable SVG animation

<rdar://problem/5612772> Disable SVG filters on Mac to match Windows behavior

In order to allow finer grained control over the set of SVG features
this patch splits ENABLE_SVG_EXPERIMENTAL_FEATURES into the following
distinct flags:

ENABLE_SVG_ANIMATION
ENABLE_SVG_FILTERS
ENABLE_SVG_FONTS
ENABLE_SVG_AS_IMAGE
ENABLE_SVG_USE

by default only ENABLE_SVG_AS_IMAGE and ENABLE_SVG_USE are set.

4:19 PM BuildingGtk edited by andi5.py@gmx.net
apt-get typo (diff)
3:45 PM Changeset in webkit [28567] by mitz@apple.com
  • 4 edits
    6 adds in trunk

WebCore:

Reviewed by Adele Peterson.

  • fix two bugs in parsing of stylesheets in <style> elements created by the parser:
    1. each such stylesheet is parsed twice, once when the text node is added and again when the </style> tag is reached
    2. re-inserting such a <style> element into the document fails to re-parse and apply its stylesheet.

Test for bug #2: fast/dom/HTMLStyleElement/insert-parser-generated.html

  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::finishedParsing):
  • svg/SVGStyleElement.cpp: (WebCore::SVGStyleElement::finishedParsing):

LayoutTests:

Reviewed by Adele Peterson.

  • test that re-inserting a parser-generated <style> element applies its sylesheet.
  • fast/dom/HTMLStyleElement: Added.
  • fast/dom/HTMLStyleElement/insert-parser-generated.html: Added.
  • platform/mac/fast/dom/HTMLStyleElement: Added.
  • platform/mac/fast/dom/HTMLStyleElement/insert-parser-generated-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLStyleElement/insert-parser-generated-expected.png: Added.
  • platform/mac/fast/dom/HTMLStyleElement/insert-parser-generated-expected.txt: Added.
1:02 PM Changeset in webkit [28566] by weinig@apple.com
  • 1 edit in trunk/JavaScriptCore/ChangeLog

Fix ChangeLog

10:03 AM Changeset in webkit [28565] by weinig@apple.com
  • 10 edits in trunk

JavaScriptCore:

Reviewed by Darin.

  • Rename isSafeScript to allowsAccess.
  • bindings/NP_jsobject.cpp: (_isSafeScript):
  • kjs/JSGlobalObject.h: (KJS::JSGlobalObject::allowsAccessFrom): Reverse caller/argument of allowsAccessFrom to match the new call.

WebCore:

Reviewed by Darin.

  • Removes the faulty isSafeScript implementation that was only used for plugins.
  • Renames isSafeScript to allowsAccessFrom.
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::customGetOwnPropertySlot): (WebCore::JSDOMWindow::customPut):
  • bindings/js/kjs_dom.cpp: (WebCore::checkNodeSecurity):
  • bindings/js/kjs_window.cpp: (KJS::createWindow): (KJS::Window::getValueProperty): (KJS::Window::namedItemGetter): (KJS::Window::getOwnPropertySlot): (KJS::Window::put): (KJS::Window::allowsAccessFrom): (KJS::Window::setListener): (KJS::Window::getListener): (KJS::WindowProtoFuncOpen::callAsFunction): (KJS::WindowProtoFuncSetTimeout::callAsFunction): (KJS::WindowProtoFuncClearTimeout::callAsFunction): (KJS::WindowProtoFuncSetInterval::callAsFunction): (KJS::WindowProtoFuncAddEventListener::callAsFunction): (KJS::WindowProtoFuncRemoveEventListener::callAsFunction): (KJS::WindowProtoFuncNotImplemented::callAsFunction): (KJS::Location::getOwnPropertySlot): (KJS::Location::put): (KJS::LocationProtoFuncReplace::callAsFunction): (KJS::LocationProtoFuncReload::callAsFunction): (KJS::LocationProtoFuncAssign::callAsFunction): (KJS::LocationProtoFuncToString::callAsFunction):
  • bindings/js/kjs_window.h: (KJS::Window::allowsAccessFrom):
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject _isSafeScript]): Reverse caller/argument of allowsAccessFrom to match the new call.
  • bindings/scripts/CodeGeneratorJS.pm:
6:39 AM Changeset in webkit [28564] by alp@webkit.org
  • 5 edits in trunk/WebKit/gtk

2007-12-08 Luca Bruno <​lethalman88@gmail.com>

Reviewed by Alp Toker.

​http://bugs.webkit.org/show_bug.cgi?id=16346
[GTK] Some webview parts must use the focused frame.

Also update some old code.

  • WebView/webkitprivate.cpp: (getFrameFromView): removed
  • WebView/webkitprivate.h: (getFrameFromView): removed


  • WebView/webkitwebview.cpp: (webkit_web_view_expose_event): (webkit_web_view_key_press_event): use focused frame (webkit_web_view_key_release_event): use focused frame (webkit_web_view_button_press_event): (webkit_web_view_button_release_event): (webkit_web_view_motion_event): (webkit_web_view_scroll_event): (webkit_web_view_size_allocate): (webkit_web_view_set_scroll_adjustments): (webkit_web_view_execute_script): (webkit_web_view_stop_loading): (webkit_web_view_load_string): (webkit_web_view_reload): (webkit_web_view_open): (webkit_web_view_can_go_forward): (webkit_web_view_can_go_backward): (webkit_web_view_go_forward): (webkit_web_view_go_backward):
  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::handleKeypress): fix for page up and page down keys for editable contents
4:27 AM Changeset in webkit [28563] by rwlbuis@webkit.org
  • 4 edits
    4 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=15464
SVGLengthList allows bad values

Be more strict with svg lengths without a unit identifier.

Test: svg/custom/invalid-lengthlist.svg

4:11 AM Changeset in webkit [28562] by rwlbuis@webkit.org
  • 2 edits in trunk/WebCore

Mac Tiger build fix.

Use the wtf prefix when including MathExtras.h.

3:35 AM Changeset in webkit [28561] by alp@webkit.org
  • 1 edit in trunk/WebCore/ChangeLog

Remove 'Reviewed by' from build fix ChangeLog entry

3:32 AM Changeset in webkit [28560] by alp@webkit.org
  • 2 edits in trunk/WebCore

2007-12-08 Alp Toker <​alp@atoker.com>

Reviewed by Alp Toker.

GTK+ build fix (for ENABLE_VIDEO builds):

Include MathExtras.h to get isfinite().

  • rendering/RenderMedia.cpp:
12:53 AM Changeset in webkit [28559] by rwlbuis@webkit.org
  • 12 edits in trunk/WebCore

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=15528
svg_dynamic_cast should be removed

Replace svg_dynamic_cast with a combination of
isSVGElement and static_cast.

Dec 7, 2007:

10:49 PM Changeset in webkit [28558] by Antti Koivisto
  • 3 edits
    5 adds in trunk

WebCore:

Reviewed by Adele.


Partial fix for <rdar://problem/5633400>
Transformed <video>, <img>, <embed> are not clipped correctly until a repaint is forced


Fix video painting when transform is applied.

Test: media/video-transformed.html

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::paint):

LayoutTests:

Reviewed by Adele.


Test video painting when transform is applied.

  • media/video-paint-test.js: Added.
  • media/video-transformed.html: Added.
  • platform/mac/media/video-transformed-expected.checksum: Added.
  • platform/mac/media/video-transformed-expected.png: Added.
  • platform/mac/media/video-transformed-expected.txt: Added.
9:44 PM Changeset in webkit [28557] by sfalken@apple.com
  • 4 edits in trunk

WebKit/win:

Fix version parsing.

Rubber-stamped by Oliver.

  • WebKit.vcproj/auto-version.sh:

WebKitTools:

Fix version parsing.

Rubber-stamped by Oliver.

  • Drosera/win/Drosera.vcproj/auto-version.sh:
8:26 PM Changeset in webkit [28556] by sfalken@apple.com
  • 3 edits
    4 adds in trunk/WebKitTools

Build modifications for Drosera.

Reviewed by Adam.

  • Drosera/DroseraWin.make: Added.
  • Drosera/win/Drosera.vcproj/Drosera.rc:
  • Drosera/win/Drosera.vcproj/Drosera.vcproj:
  • Drosera/win/Drosera.vcproj/PRODUCTVERSION: Added.
  • Drosera/win/Drosera.vcproj/VERSION: Added.
  • Drosera/win/Drosera.vcproj/auto-version.sh: Added.
5:48 PM Changeset in webkit [28555] by mitz@apple.com
  • 3 edits
    5 adds in trunk

WebCore:

Reviewed by Sam Weinig.

Test: fast/css/font-face-default-font.html

  • dom/Document.cpp: (WebCore::Document::recalcStyle): Pass our font selector to Font::update() if we already have one.

LayoutTests:

Reviewed by Sam Weinig.

  • fast/css/font-face-default-font.html: Added.
  • platform/mac-leopard/fast/css: Added.
  • platform/mac-leopard/fast/css/font-face-default-font-expected.checksum: Added.
  • platform/mac-leopard/fast/css/font-face-default-font-expected.png: Added.
  • platform/mac/fast/css/font-face-default-font-expected.txt: Added.
5:37 PM Changeset in webkit [28554] by Darin Adler
  • 2 edits in trunk/WebKit/mac

Reviewed by Kevin Decker and Tim Hatcher.

  • speculative fix for <rdar://problem/5400159> CrashTracer: [USER] 726 crashes in Safari at com.apple.WebKit: -[WebHTMLView(WebPrivate) _updateMouseoverWithFakeEvent] + 389
  • WebView/WebHTMLView.mm: (-[WebHTMLView _frameOrBoundsChanged]): Only schedule the mouseover timer if we are in a window and not closed. That's because viewDidMoveToWindow and close are the entry points for cancelling. (-[WebHTMLView close]): Add code to cancel both timers. Needed for the case where the entire window goes away, and the view is never removed from the window. (-[WebHTMLView viewDidMoveToWindow]): Don't do work if the view is closed.
5:06 PM Changeset in webkit [28553] by sfalken@apple.com
  • 3 edits in trunk/WebKit/win

Fix build.

  • WebKit.vcproj/WebKit.vcproj:
  • WebScriptCallFrame.cpp:
5:00 PM Changeset in webkit [28552] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

Fix build.

  • WebHistory.cpp: (_WebCoreHistoryProvider::containsURL):
4:59 PM Changeset in webkit [28551] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

Reviewed by Sam Weinig.


Refactored variable access optimization: Removed the assumption that
the FunctionBodyNode holds the symbol table.

4:56 PM Changeset in webkit [28550] by Darin Adler
  • 2 edits in trunk/WebKit/qt
  • try to fix build
  • Api/qwebhistoryinterface.cpp: (WebCore::historyContains): There's a WebCore function here in WebKit! Needs to be updated, since WebCore changed, but this should not be here.
4:39 PM Changeset in webkit [28549] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added #include.

  • kjs/nodes.cpp:
4:35 PM Changeset in webkit [28548] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added #include.

  • kjs/interpreter.cpp:
4:30 PM Changeset in webkit [28547] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added #include.

  • kjs/grammar.y:
4:24 PM Changeset in webkit [28546] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added #include.

  • kjs/function_object.cpp:
4:23 PM Changeset in webkit [28545] by ggaren@apple.com
  • 3 edits in trunk/JavaScriptCore

Reviewed by Sam Weinig.


Fixed crash seen running layout tests.


Reverted a change I made earlier today. Added a comment to try to
discourage myself from making this mistake a third time.

  • kjs/function.cpp: (KJS::ActivationImp::mark):
  • kjs/function.h: (KJS::ActivationImp::ActivationImpData::ActivationImpData):
4:22 PM Changeset in webkit [28544] by Darin Adler
  • 2 edits in trunk/WebCore
  • fix Windows build
  • bridge/win/GlobalHistoryWin.cpp: (WebCore::historyContains): Missed a rename.
4:05 PM Changeset in webkit [28543] by beidson@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Anders and Darin

When a statement bumps up against the quota and the UI Delegate grants more space, we need to
actually set the new maximum size on the SQLiteDatabase (in addition to storing the new max quota
in the DatabaseTracker, which was already done)

  • storage/SQLTransaction.cpp: (WebCore::SQLTransaction::runStatements): If a statement is being retried, set the maximum size on the SQLiteDatabase to the new maximum size
4:02 PM Changeset in webkit [28542] by Darin Adler
  • 2 edits in trunk/WebCore
  • fix Tiger build
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: Tiger didn't even have QTKIT_VERSION_MAX_ALLOWED, so add logic to work without that.
4:02 PM Changeset in webkit [28541] by beidson@apple.com
  • 1 edit
    1 add in trunk/WebKitSite

Reviewed by Darin's rubberstamp

When developing the Database feature and dealing with UI Delegate issues, it is important to have
live examples on the web to help reproduce certain situations.

This file is a dumping ground for such examples, and starts out with one function - the ability to add
arbitrary amounts of data to a database to test the quota-related mechanisms of the UI

  • misc/DatabaseTester.html: Added.
4:01 PM Changeset in webkit [28540] by ggaren@apple.com
  • 7 edits in trunk/JavaScriptCore

Reviewed by Sam Weinig.


Refactored parsing of global code: Removed the assumption that
ProgramNode inherits from FunctionBodyNode from the parser.

  • kjs/Parser.cpp: (KJS::Parser::parseProgram): (KJS::Parser::parseFunctionBody): (KJS::Parser::parse):
  • kjs/Parser.h: (KJS::Parser::didFinishParsing):
  • kjs/function.cpp:
  • kjs/grammar.y:
  • kjs/nodes.h:
3:53 PM Changeset in webkit [28539] by Darin Adler
  • 2 edits in trunk/WebCore
  • fix 64-bit build, hopefully without breaking Tiger build
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::updateStates): Don't use Movies.h constants that are nonexistent in 64-bit. Define the new QTMovie.h constants, though, when using an older QTKit.
3:22 PM Changeset in webkit [28538] by ggaren@apple.com
  • 2 edits in trunk/WebKit/win

Tried to fix build by suppressing warning.

  • WebScriptCallFrame.cpp:
3:00 PM Changeset in webkit [28537] by beidson@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Anders

Fix <rdar://problem/5636115> - Prompted for quota increase to create database when it already existed

  • storage/DatabaseTracker.cpp: (WebCore::DatabaseTracker::canEstablishDatabase): Check hasEntryForDatabase before doing any prompting (WebCore::DatabaseTracker::hasEntryForDatabase): Check and see if this database already exists
  • storage/DatabaseTracker.h:
2:58 PM Changeset in webkit [28536] by Darin Adler
  • 15 edits in trunk

WebCore:

Reviewed by Mitz.

  • bridge/GlobalHistory.h: Change historyContains to take a character pointer plus length instead of requiring a DeprecatedString.
  • bridge/mac/GlobalHistoryMac.mm: (WebCore::historyContains): Updated for above change. Also removes pointless "fast Latin-1" case that was never used.
  • bridge/win/GlobalHistoryWin.cpp: (WebCore::historyContains): Ditto.
  • platform/gtk/TemporaryLinkStubs.cpp: (WebCore::historyContains): Ditto.
  • platform/wx/TemporaryLinkStubs.cpp: (WebCore::historyContains): Ditto.
  • css/CSSStyleSelector.cpp: (WebCore::findHash): Added. Helper for cleanpath. (WebCore::findSlashDotDotSlash): Ditto. (WebCore::findSlashSlash): Ditto. (WebCore::findSlashDotSlash): Ditto. (WebCore::cleanpath): Changed to use fast helper functions instead of slower general-purpose DeprecatedString find function. (WebCore::containsColonSlashSlash): Added. Helper for checkPseudoState. (WebCore::checkPseudoState): Got rid of reference count churn by using an AtomicString* instead of an AtomicString for the attribute value. Changed to use fast helper function instead of slower DeprecatedString::contains function, and also made the fast case not bother allocating a DeprecatedConstString.
  • unrelated tiny cleanup
  • platform/graphics/svg/cg/SVGPaintServerGradientCg.cpp: (WebCore::releaseCachedStops): Use static_cast instead of reinterpret_cast. (WebCore::cgGradientCallback): Ditto.

WebKit/mac:

Reviewed by Mitz.

  • History/WebHistory.mm: Removed unused Latin-1 code path. (-[_WebCoreHistoryProvider containsURL:length:]): Updated for method name change.

WebKit/win:

Reviewed by Mitz.

  • WebHistory.cpp: Removed unused Latin-1 code path. (_WebCoreHistoryProvider::containsItem): Updated for function name change.
2:45 PM Changeset in webkit [28535] by Darin Adler
  • 2 edits in trunk/WebCore

Fix build on Tiger (Mark Rowe told me how).

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: Include <objc/objc-runtime.h>, which existed back on Tiger, rather than <objc/runtime.h>, which did not.
2:45 PM Changeset in webkit [28534] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added JSVariableObject.cpp to the .pri file.

2:37 PM Changeset in webkit [28533] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: added #include.

  • kjs/function.cpp:
2:33 PM Changeset in webkit [28532] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Build fix: rolling out last build fix to change #include path.

WARNING: NO TEST CASES ADDED OR CHANGED

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm:
2:30 PM Changeset in webkit [28531] by sfalken@apple.com
  • 15 edits in trunk

JavaScriptCore:

Re-named our B&I flag from BUILDBOT to PRODUCTION.

Reviewed by Sam Weinig.

WebCore:

Re-named our B&I flag from BUILDBOT to PRODUCTION.

Reviewed by Sam Weinig.

  • WebCore.vcproj/WebCore.make:

WebKit/win:

Re-named our B&I flag from BUILDBOT to PRODUCTION.
Fix our tiny versioning.
Bump our product version.
Use just the major version in the fallback VERSION files.
Pick up the version numbers from the production builder.

Reviewed by Sam Weinig.

  • Interfaces/WebKit.idl:
  • WebKit.vcproj/Interfaces.vcproj:
  • WebKit.vcproj/PRODUCTVERSION:
  • WebKit.vcproj/VERSION:
  • WebKit.vcproj/WebKit.make:
  • WebKit.vcproj/WebKit.rc:
  • WebKit.vcproj/WebKit.vcproj:
  • WebKit.vcproj/auto-version.sh:
  • WebKitDLL.cpp: (DllUnregisterServer):
2:26 PM Changeset in webkit [28530] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Build fix: corrected #include path.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm:
2:23 PM Changeset in webkit [28529] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: removed stray name qualification.

  • kjs/function.h: (KJS::ActivationImp::ActivationImp):
2:14 PM Changeset in webkit [28528] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: moved functions with qualified names outside of class
declaration.

  • kjs/JSVariableObject.h: (KJS::JSVariableObject::symbolTableGet): (KJS::JSVariableObject::symbolTablePut):
2:05 PM Changeset in webkit [28527] by ggaren@apple.com
  • 17 edits
    4 adds in trunk

JavaScriptCore:

Reviewed by Sam Weinig.

Next step in refactoring JSGlobalObject: Added JSVariableObject class,
and factored symbol-table-related code into it. (JSGlobalObject doesn't
use the symbol table code yet, though.)


Layout and JS tests, and testapi, pass. SunSpider reports no regression.

WebCore:

Reviewed by Sam Weinig.

Added some namespace qualifications and a forwarding header, now that
KJS::Node is sometimes #included in WebCore by JavaScriptCore headers.

  • ForwardingHeaders/wtf/ListRefPtr.h: Added.
  • bindings/js/JSXSLTProcessor.cpp: (KJS::JSXSLTProcessorPrototypeFunctionTransformToFragment::callAsFunction):
  • bindings/js/kjs_binding.cpp: (KJS::ScriptInterpreter::getDOMNodeForDocument): (KJS::ScriptInterpreter::forgetDOMNodeForDocument): (KJS::ScriptInterpreter::putDOMNodeForDocument): (KJS::ScriptInterpreter::markDOMNodesForDocument): (KJS::ScriptInterpreter::updateDOMNodeDocument):

WebKit/mac:

Reviewed by Sam Weinig.


Added a forwarding header, since we now #include nodes.h through some
JavaScriptCore headers.

  • ForwardingHeaders/wtf/ListRefPtr.h: Added.
12:21 PM Changeset in webkit [28526] by Adam Roben
  • 7 edits
    1 add in trunk

Add SoftLinking.h for Windows and use it in a few places

WebCore:

Add SoftLinking.h for Windows and use it in a few places

Reviewed by Oliver.

  • WebCore.vcproj/WebCore.vcproj: Added new file to project.
  • platform/win/PlatformScrollBarSafari.cpp: Use SoftLinking.h. (WebCore::PlatformScrollbar::PlatformScrollbar): Removed manual soft-linking calls. (WebCore::PlatformScrollbar::paintButton): Check for the presence of the SafariTheme library now that we can't check for the presence of paintThemePart directly. (WebCore::PlatformScrollbar::paintTrack): Ditto. (WebCore::PlatformScrollbar::paintThumb): Ditto.
  • platform/win/SoftLinking.h: Copied from WebCore/platform/mac/SoftLinking.h.
  • rendering/RenderThemeSafari.cpp: Same basic changes as to PlatformScrollBarSafari.cpp. (WebCore::RenderThemeSafari::RenderThemeSafari): (WebCore::RenderThemeSafari::isControlStyled): (WebCore::RenderThemeSafari::paintCapsLockIndicator):
  • rendering/RenderThemeSafari.h: Removed m_themeDLL member.

WebKit/win:

Use SoftLinking.h in WebKitClassFactory.

Reviewed by Oliver.

  • WebKitClassFactory.cpp: (WebKitClassFactory::WebKitClassFactory): We don't have to worry about not finding STInitialize anymore now that a version of SafariTheme containing this function has shipped.
11:59 AM Changeset in webkit [28525] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

Reviewed by Geoff.

While this does not make SunSpider faster, it will make many regular
expressions a bit faster.

  • pcre/pcre_compile.cpp: Moved CompileData struct in here from the header since it's private to this file. (compile_branch): Updated for function name change. (compile_bracket): Renamed from compile_regex, since, for one thing, this does not compile an entire regular expression. (calculateCompiledPatternLengthAndFlags): Removed unused item_count local variable. Renamed CompileData to cd instead of compile_block to be consistent with other functions. Added code to set the needOuterBracket flag if there's at least one "|" at the outer level. (jsRegExpCompile): Renamed CompileData to cd instead of compile_block to be consistent with other functions. Removed unneeded "size" field from the compiled regular expression. If no outer bracket is needed, then use compile_branch to compile the regular expression.
  • pcre/pcre_internal.h: Removed the CompileData struct, which is now private to pcre_compile.cpp. Removed the size member from JSRegExp.
11:44 AM Changeset in webkit [28524] by Darin Adler
  • 8 edits
    2 adds in trunk

WebCore:

Reviewed by Adele.

  • fix <rdar://problem/5608795> CrashTracer: 481 crashes in Safari at WebCore::HTMLSelectElement::saveState const + 152

Test: fast/forms/select-set-inner.html

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::removeChildren): Added a return value, as with other calls that change children, so we can optimize for the case where it does nothing. (WebCore::ContainerNode::cloneChildNodes): Changed parameter type to ContainerNode.
  • dom/ContainerNode.h: See above.
  • html/HTMLOptGroupElement.cpp: (WebCore::HTMLOptGroupElement::removeChildren): Override removeChildren and call recalcSelectOptions in that case. (WebCore::HTMLOptGroupElement::childrenChanged): Override childrenChanged instead of addChild, for consistency with HTMLSelectElement; no need to override both. (WebCore::HTMLOptGroupElement::groupLabelText): Made const.
  • html/HTMLOptGroupElement.h: See above.
  • html/HTMLSelectElement.cpp: Don't override addChild any more, because we already override childrenChanged, and addChild calls that. (WebCore::HTMLSelectElement::removeChildren): Override removeChildren and call recalcSelectOptions in that case. (WebCore::HTMLSelectElement::recalcListItems): Tightened up the code a little bit by using a for loop and traverseNextSibling. Also added some new comments and removed some obsolete ones. (WebCore::HTMLSelectElement::checkListItems): Added. Debug-only check to make sure we don't have a stale list items vector.
  • html/HTMLSelectElement.h: Changed listItems() to invoke checkListItems(). This will help us catch cases where we have too few calls to setRecalcListItems.

LayoutTests:

Reviewed by Adele.

  • test for <rdar://problem/5608795> CrashTracer: 481 crashes in Safari at WebCore::HTMLSelectElement::saveState const + 152
  • fast/forms/select-set-inner-expected.txt: Added.
  • fast/forms/select-set-inner.html: Added.
11:24 AM Changeset in webkit [28523] by mitz@apple.com
  • 17 edits
    4 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • WebCore part of fixing <rdar://problem/5499918> REGRESSION: insertion point in input field with custom border cuts holes in focus ring interior edges

Test: fast/repaint/focus-ring.html

  • editing/SelectionController.cpp: (WebCore::SelectionController::caretRepaintRect): Changed to return just the caret rect without any padding. (WebCore::SelectionController::recomputeCaretRect): Changed to repaint just the caret rect without any padding.
  • platform/graphics/GraphicsContext.h: Removed setFocusRingClip() and clearFocusRingClip().
  • platform/graphics/cairo/GraphicsContextCairo.cpp: Ditto.
  • platform/graphics/cg/GraphicsContextCG.cpp: Ditto.
  • platform/graphics/cg/GraphicsContextPlatformPrivate.h: Removed m_focusRingClip member.
  • platform/graphics/mac/GraphicsContextMac.mm: (WebCore::GraphicsContext::drawFocusRing): Changed to call wkDrawFocusRing() once without setting up additional clip. On Leopard, wkDrawFocusRing() respects the context clip now. On Tiger, a transparency layer is used to apply clipping to the focus ring.
  • platform/graphics/qt/GraphicsContextQt.cpp: Removed focus ring clip methods and member.
  • platform/graphics/wx/GraphicsContextWx.cpp: Ditto.
  • platform/mac/WebCoreSystemInterface.h: Removed the clipRect argument to wkDrawFocusRing().
  • platform/mac/WebCoreSystemInterface.mm: Ditto.
  • rendering/RenderLayer.cpp: (WebCore::setClip): Removed call to set the focus ring clip. (WebCore::restoreClip): Removed call to reset the focus ring clip.

WebKitLibraries:

Reviewed by Darin Adler.

  • updated system interface for fixing <rdar://problem/5499918> REGRESSION: insertion point in input field with custom border cuts holes in focus ring interior edges
  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceTiger.a:

LayoutTests:

Reviewed by Darin Adler.

  • repaint test for <rdar://problem/5499918> REGRESSION: insertion point in input field with custom border cuts holes in focus ring interior edges
  • fast/repaint/focus-ring.html: Added.
  • platform/mac/fast/repaint/focus-ring-expected.checksum: Added.
  • platform/mac/fast/repaint/focus-ring-expected.png: Added.
  • platform/mac/fast/repaint/focus-ring-expected.txt: Added.
11:09 AM Changeset in webkit [28522] by beidson@apple.com
  • 2 edits in trunk/WebKitSite

Reviewed by Adam Roben

  • misc/DatabaseExample.html: Changed to differentiate failure between running with an unsupported browser or some other error that prevented the database from opening. In the future we can use ExceptionCodes returned from openDatabase(), but a good list of those don't exist yet
10:44 AM Changeset in webkit [28521] by Darin Adler
  • 11 edits in trunk/WebCore

Reviewed by Antti Koivisto and Kevin Decker.

  • fix <rdar://problem/5601586> QtKit should be dynamically loaded upon need, not linked at startup

Also did a lot of small tweaks to MediaPlayerPrivateQTKit.

  • WebCore.xcodeproj/project.pbxproj: Don't link to QTKit.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h: Omit unneeded includes and declarations. Made a lot more functions const. Made a few more members private and a couple inline.
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: Added soft linking machinery for all the things we currently use in QTKit. It's a little more awkward for classes and other data objects than it is for functions, but still relatively straightforward, with no changes needed to the client code. Added using namespace directives. Made a cuePointTimerInterval constant and put it at the top of the file. Use 0 consistently instead of sometimes 0 and sometimes 0.0f. (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): Removed unneeded initialization of RetainPtr members to nil. (WebCore::MediaPlayerPrivate::createQTMovie): Use adoptNS instead of autorelease. (WebCore::MediaPlayerPrivate::createQTMovieView): Ditto. Also use -[NSColor clearColor]. (WebCore::MediaPlayerPrivate::createQTTime): Remove an unneeded type cast that had no effect. Changed to use long instead of int because that's the type for a QTTime time scale anyway. (WebCore::MediaPlayerPrivate::duration): Use a static_cast instead of a C-style cast. (WebCore::MediaPlayerPrivate::currentTime): Ditto. Also merged into a single expression. (WebCore::MediaPlayerPrivate::cuePointTimerFired): Added code to make a copy of the cue points set to avoid a potential problem with a set being modified as we iterate it. (WebCore::MediaPlayerPrivate::bytesLoaded): Removed unneeded null check of m_qtMovie. (WebCore::MediaPlayerPrivate::updateStates): Instead of comments explaining the numeric values, used the constants from the headers directly. (WebCore::MediaPlayerPrivate::getSupportedTypes): Instead of (QTMovieFileTypeOptions)0, pass the named constant with value 0, QTIncludeCommonTypes. Skipped the intermediate type of NSString to remove one cast. Replaced C-style cast with reinterpret_cast (arguably no better). Used RetainPtr instead of explicit CFRelease calls.
  • platform/mac/SoftLinking.h: Added macros to do soft linking for classes and for pointers. It's not quite as automatic as the soft linking we can do for functions, since these define functions to get the values, so you need to define macros to make what look like variable accesses turn into function calls. See MediaPlayerPrivateQTKit for the details.
  • html/HTMLMediaElement.h:
  • html/TimeRanges.h:
  • html/VoidCallback.h:
  • platform/graphics/MediaPlayer.h: Use angle brackets for wtf includes. Omit unneeded includes.
10:43 AM Changeset in webkit [28520] by adachan@apple.com
  • 3 edits in trunk/WebKit/win

<rdar://problem/5556378> Implemented database related UI delegate methods (prompts for new/enlarged databases)

Reviewed by Darin.

  • Interfaces/IWebUIDelegatePrivate.idl:
  • WebChromeClient.cpp: (WebChromeClient::requestQuotaIncreaseForNewDatabase): (WebChromeClient::requestQuotaIncreaseForDatabaseOperation):
10:26 AM Changeset in webkit [28519] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Darin Adler.

Test: fast/dynamic/subtree-no-common-root-static-y.html

  • rendering/RenderObject.cpp: (WebCore::RenderObject::markContainingBlocksForLayout): Changed the call to setChildNeedsLayout() to not mark containing blocks and added a separate call to markContainingBlocksForLayout() that will not schedule a layout if we are already in the middle of scheduleRelayoutOfSubtree().

LayoutTests:

Reviewed by Darin Adler.

  • fast/dynamic/subtree-no-common-root-static-y.html: Added.
  • platform/mac/fast/dynamic/subtree-no-common-root-static-y-expected.checksum: Added.
  • platform/mac/fast/dynamic/subtree-no-common-root-static-y-expected.png: Added.
  • platform/mac/fast/dynamic/subtree-no-common-root-static-y-expected.txt: Added.
9:24 AM Changeset in webkit [28518] by kevino@webkit.org
  • 3 edits in trunk/WebKit/wx

Fix wxWebKit page leaks.

Reviewed by Darin Adler.

9:18 AM Changeset in webkit [28517] by kevino@webkit.org
  • 2 edits in trunk/JavaScriptCore

MSVC7 build fix due to a compiler bug with placement new and/or templates and casting.

Reviewed by Darin Adler.

4:26 AM HackingGtk edited by alp@atoker.com
Movie -> MediaPlayer in SVN link (diff)
3:47 AM Changeset in webkit [28516] by alp@webkit.org
  • 2 edits in trunk/WebKit/gtk

2007-12-07 Luca Bruno <​lethalman88@gmail.com>

Reviewed by Alp Toker.

​http://bugs.webkit.org/show_bug.cgi?id=16333
[GTK] Key bindings must work with focused frames.

There are still more cases where getFrameFromView() is mis-used that
need to be fixed, but this is a good start.

  • WebView/webkitwebview.cpp: (webkit_web_view_real_select_all): (webkit_web_view_real_cut_clipboard): (webkit_web_view_real_copy_clipboard): (webkit_web_view_real_paste_clipboard):
2:17 AM Changeset in webkit [28515] by ap@webkit.org
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=16325
<rdar://problem/5632997> REGRESSION: www.xerox.ru doesn't work


Test: http/tests/misc/empty-cookie.html

  • platform/mac/CookieJar.mm: (WebCore::setCookies): Don't store empty cookies. (WebCore::cookies): Filter out empty cookies if we have them, as they could have been set with an earlier version of Leopard!
Note: See TracTimeline for information about the timeline view.