Timeline


and

02/02/08:

23:01 Changeset [29946] by dsmith@webkit.org
  • 3 edits in trunk/WebKitSite

Reviewed by Oliver.


Update the selector benchmark/test now that WebKit supports :only-child and :last-child;
Unfortunately, the javascript libraries don't appear to implement :*-of-type correctly, so that one remains disabled for now.

  • perf/slickspeed/header.html: Typo fix, and updated the note to clarify that WebKit is no longer the only implementation causing us to leave some selectors disabled.
  • perf/slickspeed/selectors.list: Added the new selectors
18:29 Changeset [29945] by hyatt@apple.com
  • 2 adds in trunk/LayoutTests/fast/css

Add the test files. Oops.

18:28 Changeset [29944] by hyatt@apple.com
  • 5 edits
    6 adds in trunk

WebCore:

Fix for bug 5468, support CSS3 :only-child and :only-of-type selectors.

Reviewed by olliej

Added fast/css/only-child-pseudo-class.html, fast/css/only-of-type-pseudo-class.html

  • css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType):
  • css/CSSSelector.h: (WebCore::CSSSelector::):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::checkOneSelector):

LayoutTests:

Add test for bug 5468, supporting :only-child and :only-of-type in CSS3.

Reviewed by olliej

  • platform/mac/fast/css/only-child-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/only-child-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/only-child-pseudo-class-expected.txt: Added.
  • platform/mac/fast/css/only-of-type-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/only-of-type-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/only-of-type-pseudo-class-expected.txt: Added.
16:20 Changeset [29943] by ggaren@apple.com
  • 10 edits in trunk

JavaScriptCore:

Reviewed by Geoff Garen.

PLT speedup related to <rdar://problem/5659272> REGRESSION: PLT .4%
slower due to r28884 (global variable symbol table optimization)

Geoff's theory is that the slowdown was due to copying hash tables when
putting things into the back/forward cache. If that's true, then this
should fix the problem.


(According to Geoff's measurements, in a PLT that exaggerates the
importance of symbol table saving during cached page creation, this
patch is a ~3X speedup in cached page creation, and a 9% speedup overall.)

  • kjs/JSVariableObject.cpp: (KJS::JSVariableObject::saveLocalStorage): Updated for changes to SavedProperty, which has been revised to avoid initializing each SavedProperty twice when building the array. Store the property names too, so we don't have to store the symbol table separately. Do this by iterating the symbol table instead of the local storage vector. (KJS::JSVariableObject::restoreLocalStorage): Ditto. Restore the symbol table as well as the local storage vector.
  • kjs/JSVariableObject.h: Removed save/restoreSymbolTable and do that work inside save/restoreLocalStorage instead. Made restoreLocalStorage a non-const member function that takes a const reference to a SavedProperties object.
  • kjs/LocalStorage.h: Changed attributes to be unsigned instead of int to match other declarations of attributes elsewhere.
  • kjs/property_map.cpp: (KJS::SavedProperties::SavedProperties): Updated for data member name change. (KJS::PropertyMap::save): Updated for data member name change and to use the new inline init function instead of setting the fields directly. This allows us to skip initializing the SavedProperty objects when first allocating the array, and just do it when we're actually setting up the individual elements. (KJS::PropertyMap::restore): Updated for SavedProperty changes.
  • kjs/property_map.h: Changed SavedProperty from a struct to a class. Set it up so it does not get initialized at construction time to avoid initializing twice when creating an array of SavedProperty. Removed the m_ prefixes from the members of the SavedProperties struct. Generally we use m_ for class members and not struct.

WebCore:

Reviewed by Geoff Garen.

PLT speedup related to <rdar://problem/5659272> REGRESSION: PLT .4%
slower due to r28884 (global variable symbol table optimization)

  • history/CachedPage.cpp: (WebCore::CachedPage::CachedPage): Removed saveSymbolTable call. (WebCore::CachedPage::restore): Removed restoreSymbolTable call. (WebCore::CachedPage::clear): Removed clear of m_windowSymbolTable.
  • history/CachedPage.h: Removed m_windowSymbolTable, since save/restoreLocalStorage now takes care of the symbol table. Also removed many unnecessary includes.
15:18 Changeset [29942] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by eseidel.

Add an include for for <objidl.h> which isn't included
by default with WIN32_LEAN_AND_MEAN.

  • platform/graphics/FontCache.h:
15:13 Changeset [29941] by eric@webkit.org
  • 4 edits in trunk/JavaScriptCore

Reviewed by darin. Landed by eseidel.

Add #define guards for WIN32_LEAN_AND_MEAN and _CRT_RAND_S.

  • kjs/config.h:
  • wtf/FastMalloc.cpp:
  • wtf/TCSpinLock.h:
12:02 Changeset [29940] by kevino@webkit.org
  • 8 edits in trunk/WebCore

wxFont is reference counted and meant to be used as as a value object; when using wxFont*, ref-counting was not happening properly and the font object would be destroyed while still in use. Also, fix memory leak in wx's FrameData::clear() method.

11:19 Changeset [29939] by darin@apple.com
  • 1 edit
    10 moves in trunk/LayoutTests
  • updated some more last-child and last-of-type tests to expect success (failures seen on buildbot)

The tests were now succeeding, but the results and placement reflected failure.
I'm not sure why we have such a tiny set of tests in the css3 directory.
The expected results are updated as well as moved.

  • css3/css3-modsel-33.html: Copied from css3/expected_failures/css3-modsel-33.html.
  • css3/css3-modsel-35.html: Copied from css3/expected_failures/css3-modsel-35.html.
  • css3/expected_failures/css3-modsel-33.html: Removed.
  • css3/expected_failures/css3-modsel-35.html: Removed.
  • platform/mac/css3/css3-modsel-33-expected.checksum: Copied from platform/mac/css3/expected_failures/css3-modsel-33-expected.checksum.
  • platform/mac/css3/css3-modsel-33-expected.png: Copied from platform/mac/css3/expected_failures/css3-modsel-33-expected.png.
  • platform/mac/css3/css3-modsel-33-expected.txt: Copied from platform/mac/css3/expected_failures/css3-modsel-33-expected.txt.
  • platform/mac/css3/css3-modsel-35-expected.checksum: Copied from platform/mac/css3/expected_failures/css3-modsel-35-expected.checksum.
  • platform/mac/css3/css3-modsel-35-expected.png: Copied from platform/mac/css3/expected_failures/css3-modsel-35-expected.png.
  • platform/mac/css3/css3-modsel-35-expected.txt: Copied from platform/mac/css3/expected_failures/css3-modsel-35-expected.txt.
  • platform/mac/css3/expected_failures/css3-modsel-33-expected.checksum: Removed.
  • platform/mac/css3/expected_failures/css3-modsel-33-expected.png: Removed.
  • platform/mac/css3/expected_failures/css3-modsel-33-expected.txt: Removed.
  • platform/mac/css3/expected_failures/css3-modsel-35-expected.checksum: Removed.
  • platform/mac/css3/expected_failures/css3-modsel-35-expected.png: Removed.
  • platform/mac/css3/expected_failures/css3-modsel-35-expected.txt: Removed.
  • platform/qt/css3/css3-modsel-33-expected.txt: Copied from platform/qt/css3/expected_failures/css3-modsel-33-expected.txt.
  • platform/qt/css3/css3-modsel-35-expected.txt: Copied from platform/qt/css3/expected_failures/css3-modsel-35-expected.txt.
  • platform/qt/css3/expected_failures/css3-modsel-33-expected.txt: Removed.
  • platform/qt/css3/expected_failures/css3-modsel-35-expected.txt: Removed.
08:57 Changeset [29938] by mrowe@apple.com
  • 1 edit
    1 add in trunk/WebKitSite

Add WordPress plugin to give more control over blog comment settings.

08:52 Changeset [29937] by kevino@webkit.org
  • 6 edits in trunk

Don't use callback-based timers on Windows, they eat crashes and put the application in an undefined state (and since the crash was eaten, it's impossible to determine the original cause). This is a version of r28500 for the wx port.

Reviewed by Adam Roben.

08:45 Changeset [29936] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Qt build fix. Fix typo.

08:31 Changeset [29935] by kevino@webkit.org
  • 2 edits in trunk/WebCore

Blind Qt build fix. Add Template.h include.

08:10 Changeset [29934] by kevino@webkit.org
  • 2 edits in trunk/WebCore

wx build fix - add missing include file.

02:46 Changeset [29933] by hyatt@apple.com
  • 6 edits
    8 adds in trunk

WebCore:

Fix for bug 4812. Support last-child and last-of-type CSS3 selectors. Brings Acid3 score up to 68/100.

Reviewed by olliej

Added fast/css/last-child-pseudo-class.html, fast/css/last-of-type-pseudo-class.html

  • css/CSSGrammar.y:
  • css/CSSSelector.cpp: (WebCore::CSSSelector::extractPseudoType):
  • css/CSSSelector.h: (WebCore::CSSSelector::):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::checkOneSelector):

LayoutTests:

Fix for bug 4812. Support last-child and last-of-type (and make sure they are properly dynamic).

Reviewed by olliej

  • fast/css/last-child-pseudo-class.html: Added.
  • fast/css/last-of-type-pseudo-class.html: Added.
  • platform/mac/fast/css/last-child-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/last-child-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/last-child-pseudo-class-expected.txt: Added.
  • platform/mac/fast/css/last-of-type-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/last-of-type-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/last-of-type-pseudo-class-expected.txt: Added.
02:06 Changeset [29932] by hyatt@apple.com
  • 7 edits
    12 adds in trunk

WebCore:

Make :first-child and :first-of-type properly dynamic when the DOM changes. Brings the Acid3 score up
to 66/100.

Reviewed by olliej

Added fast/css/first-child-pseudo-class.html, fast/css/first-of-type-pseudo-class.html, fast/css/empty-body-test.html

  • css/CSSGrammar.y:
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::checkOneSelector):
  • dom/Element.cpp: (WebCore::Element::recalcStyle): (WebCore::Element::childrenChanged):
  • rendering/RenderStyle.cpp: (WebCore::RenderStyle::RenderStyle):
  • rendering/RenderStyle.h: (WebCore::RenderStyle::childrenAffectedByFirstChildRules): (WebCore::RenderStyle::setChildrenAffectedByFirstChildRules): (WebCore::RenderStyle::childrenAffectedByLastChildRules): (WebCore::RenderStyle::setChildrenAffectedByLastChildRules): (WebCore::RenderStyle::childrenAffectedByPositionalRules): (WebCore::RenderStyle::setChildrenAffectedByPositionalRules): (WebCore::RenderStyle::firstChildState): (WebCore::RenderStyle::setFirstChildState): (WebCore::RenderStyle::lastChildState): (WebCore::RenderStyle::setLastChildState):

LayoutTests:

Added tests for dynamic :first-child support (Acid3).

Reviewed by olliej

  • fast/css/empty-body-test.html: Added.
  • fast/css/first-child-pseudo-class.html: Added.
  • fast/css/first-of-type-pseudo-class.html: Added.
  • platform/mac/fast/css/empty-body-test-expected.checksum: Added.
  • platform/mac/fast/css/empty-body-test-expected.png: Added.
  • platform/mac/fast/css/empty-body-test-expected.txt: Added.
  • platform/mac/fast/css/first-child-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/first-child-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/first-child-pseudo-class-expected.txt: Added.
  • platform/mac/fast/css/first-of-type-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/first-of-type-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/first-of-type-pseudo-class-expected.txt: Added.
01:37 Changeset [29931] by mitz@apple.com
  • 2 edits
    1 copy in trunk/LayoutTests
  • added Tiger-only results for a test
  • platform/mac-leopard/fast/text/cg-fallback-bolding-expected.txt: Copied from LayoutTests/platform/mac/fast/text/cg-fallback-bolding-expected.txt.
  • platform/mac/fast/text/cg-fallback-bolding-expected.txt:
00:25 Changeset [29930] by mitz@apple.com
  • 5 edits in trunk

WebCore:

Reviewed by Oliver Hunt.

  • fix <rdar://problem/5720637> Missing characters in right-to-left complex text where different fonts are used in the same run
  • platform/graphics/win/UniscribeController.cpp: (WebCore::UniscribeController::advance): Fixed an off-by-1 error in the start offset of sub-runs of RTL runs. Changed to update m_currentCharacter to the first character of the sub-run before calling itemizeShapeAndPlace. In RTL runs, m_currentCharacter decreases as sub-runs are processed from left to right. (WebCore::UniscribeController::itemizeShapeAndPlace): Removed the updating of m_currentCharacter because advance() does it now.

LayoutTests:

Reviewed by Oliver Hunt.

  • changed a test to use the same fonts on Windows as on Mac OS X
  • fast/text/atsui-spacing-features.html: Changed to use Lucida Grande for Hebrew.
  • platform/win/Skipped: Re-categorized the above test. It is still failing on Windows, but due to a different bug.

02/01/08:

16:19 Changeset [29929] by beidson@apple.com
  • 3 edits
    1 add in trunk/WebCore

Reviewed by Darin's rubberstamp

Move the SecurityOriginHash out into a separate header as it will soon be used in different places

  • WebCore.xcodeproj/project.pbxproj:
  • platform/SecurityOriginHash.h: Added. (WebCore::SecurityOriginHash::hash): (WebCore::SecurityOriginHash::equal): (WebCore::SecurityOriginTraits::deletedValue): (WebCore::SecurityOriginTraits::emptyValue):
  • storage/DatabaseTracker.cpp:
16:02 Changeset [29928] by mitz@apple.com
  • 7 edits
    1 move
    5 adds in trunk/LayoutTests

Reviewed by Dave Hyatt.

  • address the remaining tests listed under <rdar://problem/5075790> Layout tests that depend on certain fonts don't match Mac results
  • fast/text/cg-fallback-bolding.html: Changed to use Ahem instead of Geneva.
  • fast/text/international/bidi-fallback-font-weight.html: Made Mac-only.
  • platform/mac/fast/text/cg-fallback-bolding-expected.checksum: Updated.
  • platform/mac/fast/text/cg-fallback-bolding-expected.png: Updated
  • platform/mac/fast/text/cg-fallback-bolding-expected.txt: Updated.
  • platform/mac/fast/text/international/bidi-fallback-font-weight.html: Copied from LayoutTests/fast/text/international/bidi-fallback-font-weight.html.
  • platform/win/Skipped: Removed passing tests and categorized remaining failures.
  • platform/win/svg/W3C-SVG-1.1: Added.
  • platform/win/svg/W3C-SVG-1.1/resources: Added.
  • platform/win/svg/W3C-SVG-1.1/resources/Mac-compatible-font-fallback.css: Added.
  • platform/win/svg/W3C-SVG-1.1/resources/run-webkit-tests-epilogue.html: Added.
  • platform/win/svg/W3C-SVG-1.1/resources/run-webkit-tests-prologue.html: Added.
  • svg/text/text-tselect-02-f.svg: Added @font-face rule to use Lucida Grande for Hebrew.
14:18 Changeset [29927] by darin@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Tim.

  • fixed mistake in the JSLock fix that caused an assertion every time
  • bindings/js/JSCustomSQLTransactionCallback.cpp: (WebCore::JSCustomSQLTransactionCallback::JSCustomSQLTransactionCallback): Added JSLock. (WebCore::unprotectOnMainThread): Ditto.
13:51 Changeset [29926] by andersca@apple.com
  • 5 edits in trunk/WebCore

Reviewed by Antti and Darin.

Change Text::createWithLengthLimit to take a UChar pointer instead of a string. This
lets us avoid making a copy of the buffer in TextDocument.cpp.


  • dom/Text.cpp: (WebCore::Text::createWithLengthLimit):
  • dom/Text.h:
  • html/HTMLParser.cpp: (WebCore::HTMLParser::parseToken):
  • loader/TextDocument.cpp: (WebCore::TextTokenizer::write):
13:35 Changeset [29925] by darin@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam.

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::loadsBlocked): Fix grammar and word typo.
13:33 Changeset [29924] by darin@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam.

  • fix <rdar://problem/4527931> showModalDialog calls from onload functions fail (Aspect)

No automated regression test because showModalDialog doesn't work in DumpRenderTree.

  • platform/network/mac/ResourceHandleMac.mm: (WebCore::CallbackGuard::CallbackGuard): Added. (WebCore::CallbackGuard::~CallbackGuard): Added. (WebCore::ResourceHandle::supportsBufferedData): Simplified to take advantage of how static initialization works in C++. (WebCore::ResourceHandle::loadsBlocked): Always return false on Leopard. The problem in NSURLConnection that created the need to block loads is fixed in Leoaprd. This is the bug fix. (-[WebCoreResourceHandleAsDelegate connection:willSendRequest:redirectResponse:]): Use CallbackGuard instead of directly incrementing the count; allows us to omit the code entirely on Leopard. (-[WebCoreResourceHandleAsDelegate connection:didReceiveAuthenticationChallenge:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:didCancelAuthenticationChallenge:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:didReceiveResponse:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:didReceiveData:lengthReceived:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:willStopBufferingData:]): Ditto. (-[WebCoreResourceHandleAsDelegate connectionDidFinishLoading:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:didFailWithError:]): Ditto. (-[WebCoreResourceHandleAsDelegate connection:willCacheResponse:]): Ditto. Fixes a problem where this could leave inNSURLConnectionCallback set permanently in one of the code paths; this would break showModalDialog from that point on. This problem doesn't affect Safari.
12:30 Changeset [29923] by darin@apple.com
  • 2 edits in trunk/WebCore
  • platform/network/mac/ResourceHandleMac.mm: (WebCore::ResourceHandle::loadsBlocked): Roll out accidentally checked in file.
12:28 Changeset [29922] by darin@apple.com
  • 19 edits in trunk

WebCore:

Reviewed by Geoff and Brady.

  • fix <rdar://problem/5680469> Each database worker thread takes a JSLock, which slows down all JavaScript execution until the thread terminates
  • bindings/js/JSCustomSQLTransactionCallback.cpp: (WebCore::JSCustomSQLTransactionCallback::JSCustomSQLTransactionCallback): Explicitly gcProtect, since we aren't using ProtectedPtr any more. (WebCore::unprotectOnMainThread): Added. (WebCore::JSCustomSQLTransactionCallback::~JSCustomSQLTransactionCallback): Use unprotectOnMainThread rather than doing a gcUnprotect here.
  • bindings/js/JSCustomSQLTransactionCallback.h: Changed m_callback from a ProtectedPtr to a plain old JSObject*; we gcProtect by hand now.
  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::notifyPendingLoadDecisionsOnMainThread): Renamed. Changed to takea a void* with the icon database pointer. (WebCore::IconDatabase::notifyPendingLoadDecisions): Renamed. (WebCore::IconDatabase::performURLImport): Updated name, and pass the icon database pointer in.
  • loader/icon/IconDatabase.h: Renamed and added void* parameter.
  • platform/Threading.h: Changed callOnMainThread to take a function with a single parameter rather than a function with no parameters. Added a typedef.
  • platform/gtk/ThreadingGtk.cpp: (WebCore::callFunctionOnMainThread): Changed to use a structure with both a function pointer and a context pointer, and delete it after calling. (WebCore::callOnMainThread): Changed to create the structure with both the function and context pointer.
  • platform/mac/Threading.mm: (-[WebCoreFunctionWrapper initWithFunction:context:]): Added context. (-[WebCoreFunctionWrapper invoke]): Updated to do both function and context. Renamed from _call. (WebCore::callOnMainThread): Updated to take both function and context.
  • platform/qt/ThreadingQt.cpp: (WebCore::PerformFunctionEvent::PerformFunctionEvent): Added context. (WebCore::PerformFunctionEvent::invoke): Added. Calls function with context pointer. (WebCore::MainThreadInvoker::event): Changed to call invoke. (WebCore::callOnMainThread): Updated to take both function and context.
  • platform/win/ThreadingWin.cpp: Added FunctionWithContext. (WebCore::callFunctionsOnMainThread): Changed to pass call function with context. (WebCore::callOnMainThread): Updated to queue both function and context.
  • platform/wx/ThreadingWx.cpp: (WebCore::callOnMainThread): Updated to take both function and context.
  • storage/Database.cpp: (WebCore::Database::scheduleTransactionCallback): Pass 0 for context. (WebCore::Database::deliverAllPendingCallbacks): Changed to take ignored context pointer.
  • storage/Database.h: Added void* parameter to deliverAllPendingCallbacks.
  • storage/DatabaseTracker.cpp: (WebCore::DatabaseTracker::scheduleForNotification): Pass 0 for context. (WebCore::DatabaseTracker::notifyDatabasesChanged): Changed to take ignored context pointer.
  • storage/DatabaseTracker.h: Added void* parameter to notifyDatabasesChanged.

WebKit/win:

  • fix <rdar://problem/5680469> Each database worker thread takes a JSLock, which slows down all JavaScript execution until the thread terminates
  • WebIconDatabase.cpp: (WebIconDatabase::scheduleNotificationDelivery): Updated for new callOnMainThread. (WebIconDatabase::deliverNotifications): Ditto.
  • WebIconDatabase.h: Ditto.
11:42 Changeset [29921] by jhoneycutt@apple.com
  • 2 edits in trunk/WebCore

2008-02-01 Jon Honeycutt <jhoneycutt@apple.com>

Rubber-stamped by Ollie.

  • plugins/win/PluginPackageWin.cpp: (WebCore::PluginPackageWin::load): Fix two function pointers
11:39 Changeset [29920] by mitz@apple.com
  • 3 edits
    2 moves
    2 adds
    3 deletes in trunk/LayoutTests

Reviewed by Alexey Proskuryakov.

  • change fast/xsl/xslt-text to dump as text
  • fast/xsl/resources/xslt-text.xml: Copied from LayoutTests/fast/xsl/xslt-text.xml.
  • fast/xsl/resources/xslt-text.xsl: Copied from LayoutTests/fast/xsl/xslt-text.xsl.
  • fast/xsl/xslt-processor.html:
  • fast/xsl/xslt-text-expected.txt: Added.
  • fast/xsl/xslt-text.html: Added.
  • fast/xsl/xslt-text.xml: Removed.
  • fast/xsl/xslt-text.xsl: Removed.
  • platform/mac/fast/xsl/xslt-text-expected.checksum: Removed.
  • platform/mac/fast/xsl/xslt-text-expected.png: Removed.
  • platform/mac/fast/xsl/xslt-text-expected.txt: Removed.
  • platform/win/Skipped:
11:35 Changeset [29919] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Reviewed by John Sullivan, Oliver Hunt.

Fixed <rdar://problem/5688039> REGRESSION (r29428): Weather widget
fails to load due to reliance on "var location" quirk


Added a Dashboard quirk.

  • bindings/js/kjs_window.cpp: (KJS::Window::put): In DB backwards compatibility mode, make assignment to window.location in top-level windows shadow the DOM location API instead of performing a navigation. This makes a reasonable amount of sense, because widgets can't navigate their top-level frames, anyway.
11:21 Changeset [29918] by hyatt@apple.com
  • 13 edits
    4 adds in trunk

WebCore:

Fix for bug 11387, CSS3 :empty selector is not dynamic.

Add support for dynamically updating elements whose style is affected by :empty when the child count changes.

Reviewed by olliej, mitzpettel

fast/css/empty-pseudo-class.html

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::locateSharedStyle): (WebCore::CSSStyleSelector::checkOneSelector):
  • dom/Element.cpp: (WebCore::Element::childrenChanged):
  • dom/Element.h:
  • rendering/RenderStyle.h: (WebCore::RenderStyle::affectedByEmpty): (WebCore::RenderStyle::emptyState): (WebCore::RenderStyle::setEmptyState):

LayoutTests:

Add a new layout test for dynamic :empty selectors.

Reviewed by olliej

  • fast/css/empty-pseudo-class.html: Added.
  • platform/mac/fast/css/empty-pseudo-class-expected.checksum: Added.
  • platform/mac/fast/css/empty-pseudo-class-expected.png: Added.
  • platform/mac/fast/css/empty-pseudo-class-expected.txt: Added.
10:38 Changeset [29917] by sfalken@apple.com
  • 1 edit in trunk/WebKitLibraries/ChangeLog

Fix typo.

10:36 Changeset [29916] by sfalken@apple.com
  • 2 edits in trunk/WebKitLibraries

<rdar://problem/5717523> Don't set DEP opt-in flag (data execution prevention) since it is incompaible with the video plugin used on CNN.com


Rubber-stamped by John Honeycutt.

  • win/tools/vsprops/common.vsprops:
10:06 Changeset [29915] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

fast/text/international/thai-line-breaks.html results do not change.

  • rendering/break_lines.cpp: (WebCore::nextBreakablePosition): Changed to use Carbon only on Tiger.
09:45 Changeset [29914] by pewtermoose@webkit.org
  • 2 edits in trunk/WebCore

2008-02-01 Rodney Dawes <dobey@wayofthemonkey.com>

Reviewed by Darin.

Update npfunctions.h to export the proper function signatures for UNIX and add
the appropriate #if around the Windows versions

  • plugins/npfunctions.h:
06:32 UsingWine edited by davidf@sjsoft.com
fixed link to nightly builds (diff)
05:20 UsingWine edited by davidf@sjsoft.com
rearranged scripts into separate ones per program (diff)
04:57 UsingWine created by davidf@sjsoft.com
added notes on running Safari with custom webkit under wine
01:35 Changeset [29913] by eric@webkit.org
  • 12 edits
    3 adds in trunk

Reviewed by Oliver.

Add getSVGDocument to frame and iframe.

Test: svg/custom/frame-getSVGDocument.html

  • html/HTMLEmbedElement.cpp:
  • html/HTMLEmbedElement.h:
  • html/HTMLFrameElement.idl:
  • html/HTMLFrameOwnerElement.cpp: (WebCore::HTMLFrameOwnerElement::getSVGDocument):
  • html/HTMLFrameOwnerElement.h:
  • html/HTMLIFrameElement.idl:
  • html/HTMLObjectElement.cpp:
  • html/HTMLObjectElement.h:
01:14 Changeset [29912] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

01:13 Changeset [29911] by mrowe@apple.com
  • 1 copy in tags/Safari-5525.7

New tag.

00:00 Changeset [29910] by eric@webkit.org
  • 5 edits in trunk/WebCore

Reviewed by darin.

Beat CSSParser with the RefPtr stick. Hopefully squashing any leaks in the process.
(Deploy RefPtr and PassRefPtr throughout CSSParser)
http://bugs.webkit.org/show_bug.cgi?id=17108

No functional changes, thus no tests.

  • css/CSSParser.cpp: (WebCore::CSSParser::parseColor): (WebCore::CSSParser::parseValue): (WebCore::CSSParser::addBackgroundValue): (WebCore::CSSParser::parseBackgroundShorthand): (WebCore::CSSParser::addTransitionValue): (WebCore::CSSParser::parseTransitionShorthand): (WebCore::CSSParser::parseBackgroundPosition): (WebCore::CSSParser::parseBackgroundProperty): (WebCore::CSSParser::parseTransitionProperty): (WebCore::CSSParser::parseFontFaceSrc): (WebCore::ShadowParseContext::ShadowParseContext): (WebCore::CSSParser::parseShadow): (WebCore::BorderImageParseContext::BorderImageParseContext): (WebCore::BorderImageParseContext::commitBorderImage):
  • css/CSSParser.h:
  • css/SVGCSSParser.cpp: (WebCore::CSSParser::parseSVGValue):

01/31/08:

23:01 Changeset [29909] by alp@webkit.org
  • 4 edits
    1 move in trunk

2008-01-31 Alp Toker <alp@atoker.com>

Reviewed by Mark Rowe.

Fix the pkg-config file to follow GTK+ package naming and versioning
conventions.

Remove unneeded dependency listings.

22:08 Changeset [29908] by oliver@apple.com
  • 2 edits in trunk/WebCore

Add platform/Locker.h to vcproj

RS=Brady

  • WebCore.vcproj/WebCore.vcproj:
20:31 Changeset [29907] by beidson@apple.com
  • 2 edits in trunk/WebCore

Build fix

  • WebCore.xcodeproj/project.pbxproj: Header -> WebKit
20:02 Changeset [29906] by beidson@apple.com
  • 3 edits
    1 add in trunk/WebCore

Reviewed by Maciej

Abstracted the concept of a "Locker" as upcoming work will rely on it.

  • platform/Locker.h: Added. Template class to "lock()" and "unlock()" some arbitrary object (WebCore::Locker::Locker): Lock the object (WebCore::Locker::~Locker): Unlock it
  • platform/Threading.h: class MutexLocker is now typedef Locker<Mutex>
  • WebCore.xcodeproj/project.pbxproj:
17:53 Changeset [29905] by kmccullough@apple.com
  • 15 edits
    10 deletes in trunk

WebCore:

<rdar://problem/5716912> REGRESSION (r29816): Can't navigate back to
Gmail Inbox from Gmail Compose page
Rolling out the change that caused the regression.

  • WebCore.base.exp:
  • bindings/js/JSHTMLDocumentCustom.cpp: (WebCore::JSHTMLDocument::open):
  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createHTMLDocument):
  • dom/Document.cpp: (WebCore::Document::recalcStyle): (WebCore::Document::open): (WebCore::Document::write): (WebCore::Document::clear):
  • dom/Document.h:
  • history/HistoryItem.cpp: (WebCore::HistoryItem::HistoryItem):
  • history/HistoryItem.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen): (WebCore::FrameLoader::load): (WebCore::FrameLoader::reloadAllowingStaleData): (WebCore::FrameLoader::reload): (WebCore::FrameLoader::tokenizerProcessedData): (WebCore::FrameLoader::post): (WebCore::FrameLoader::continueLoadAfterNewWindowPolicy): (WebCore::FrameLoader::shouldTreatURLAsSameAsCurrent): (WebCore::FrameLoader::loadItem):
  • loader/FrameLoader.h:
  • platform/text/CharacterNames.h:
  • xml/DOMParser.cpp: (WebCore::DOMParser::parseFromString):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::getResponseXML):
  • xml/XSLTProcessor.cpp: (WebCore::XSLTProcessor::createDocumentFromSource):

LayoutTests:

<rdar://problem/5716912> REGRESSION (r29816): Can't navigate back to
Gmail Inbox from Gmail Compose page
Rolling out the change that caused the regression.

  • http/tests/navigation/document-open-adds-history-item-expected.txt: Removed.
  • http/tests/navigation/document-open-adds-history-item.html: Removed.
  • http/tests/navigation/document-open-delayed-adds-history-item-expected.txt: Removed.
  • http/tests/navigation/document-open-delayed-adds-history-item.html: Removed.
  • http/tests/navigation/document-open-new-window-adds-history-item-expected.txt: Removed.
  • http/tests/navigation/document-open-new-window-adds-history-item.html: Removed.
  • http/tests/navigation/document-open-replace-no-history-item-expected.txt: Removed.
  • http/tests/navigation/document-open-replace-no-history-item.html: Removed.
  • http/tests/navigation/resources/document-open-page-2.html: Removed.
  • http/tests/navigation/resources/document-open.js: Removed.
17:03 Changeset [29904] by mitz@apple.com
  • 2 edits in trunk/LayoutTests
  • re-categorize three skipped tests
  • platform/win/Skipped:
17:01 Changeset [29903] by aroben@apple.com
  • 2 edits in trunk/WebKit/win

Fix <rdar://5713172> Focus lost from Gmail message body after toggling
tabs

This regressed in r29583. Prior to r29583, the code path that resulted
in calling focusController->setFocusedFrame(0) was never getting hit.
r29583 changed the code to adhere to the comment above the code and
actually call setFocusedFrame(0) when losing focus to another part of
the window. It turns out this is wrong behavior (and not what
WebKit/mac does). The only time we should clear the focused frame is
when we tab out of the WebView, and that's already taken care of by
FocusController::advanceFocus. The code to clear the focused frame in
WebView was written before FocusController::advanceFocus, so it was
correct at one point but no longer is.

We now never call setFocusedFrame(0) when the WebView receives a
WM_KILLFOCUS message. Instead we just call setFocused(false) on the
focused frame so that blur events will be sent. This matches what
WebKit/mac does.

Reviewed by Darin and Adele.

  • WebView.cpp: (WebViewWndProc): Also reworded a couple of comments to make them clearer.
16:50 Changeset [29902] by pewtermoose@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Tim Hatcher.

Bug 14960: Cannot copy text in Web Inspector Network panel
http://bugs.webkit.org/show_bug.cgi?id=14960

  • page/inspector/inspector.css:
16:48 Changeset [29901] by pewtermoose@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

Bug 14514: Add full URL view on mouseOver in the network page
http://bugs.webkit.org/show_bug.cgi?id=14514
<rdar://problem/5712841>

  • page/inspector/inspector.js:
16:35 Changeset [29900] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

Reviewed by Geoffrey Garen.

  • change the font used in fast/text/monospace-width-cache.html again to a font that contains the U+2500 character in Tiger too
  • fast/text/monospace-width-cache.html:
  • platform/mac/fast/text/monospace-width-cache-expected.checksum:
  • platform/mac/fast/text/monospace-width-cache-expected.png:
  • platform/mac/fast/text/monospace-width-cache-expected.txt:
16:00 Changeset [29899] by alp@webkit.org
  • 4 edits in trunk

Fix r29898 ChangeLog entries: rubber-stamped, not reviewed

15:51 Changeset [29898] by alp@webkit.org
  • 13 edits
    18 copies
    1 move in trunk

2008-01-31 Alp Toker <alp@atoker.com>

Reviewed by Adam Roben.

http://bugs.webkit.org/show_bug.cgi?id=17006
[GTK] Header path should be webkit/webkit.h

Move the GTK+ API sources as needed and update the build systems.

15:09 Changeset [29897] by weinig@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Geoff Garen.

No need to get the prototype just to compare compare property names.

  • bindings/js/JSHistoryCustom.cpp: (WebCore::JSHistory::customGetOwnPropertySlot):
13:37 Changeset [29896] by weinig@apple.com
  • 6 edits in trunk

WebCore:

Fix non-mac builds.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCoreSources.bkl:
13:06 Changeset [29895] by hyatt@apple.com
  • 3 edits in trunk/LayoutTests/platform/mac/fast/lists
13:04 Changeset [29894] by hyatt@apple.com
  • 3 edits in trunk/LayoutTests/platform/mac/tables/mozilla/bugs
13:03 Changeset [29893] by hyatt@apple.com
  • 4 adds in trunk/LayoutTests
13:03 Changeset [29892] by hyatt@apple.com
  • 2 edits in trunk/WebCore

Fix for bug 17101.

List items need to ignore the line box shrinking quirk, since IE and Firefox both do.

Reviewed by Eric

fast/lists/list-item-line-height.html

  • rendering/InlineFlowBox.h: (WebCore::InlineFlowBox::InlineFlowBox):
12:11 Changeset [29891] by mitz@apple.com
  • 26 edits in trunk/LayoutTests

Reviewed by Adam Roben.

  • change several tests that use Mac-only fonts to use fonts that are available on both Windows and Mac OS X
  • fast/text/monospace-width-cache.html:
  • platform/mac/fast/text/monospace-width-cache-expected.checksum:
  • platform/mac/fast/text/monospace-width-cache-expected.png:
  • platform/mac/fast/text/monospace-width-cache-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug10633-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug10633-expected.png:
  • platform/mac/tables/mozilla/bugs/bug10633-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug113424-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug113424-expected.png:
  • platform/mac/tables/mozilla/bugs/bug113424-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug23151-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug23151-expected.png:
  • platform/mac/tables/mozilla/bugs/bug23151-expected.txt:
  • platform/mac/tables/mozilla/core/bloomberg-expected.checksum:
  • platform/mac/tables/mozilla/core/bloomberg-expected.png:
  • platform/mac/tables/mozilla/core/bloomberg-expected.txt:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug11331-expected.checksum:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug11331-expected.png:
  • platform/mac/tables/mozilla_expected_failures/bugs/bug11331-expected.txt:
  • platform/win/Skipped:
  • tables/mozilla/bugs/bug10633.html:
  • tables/mozilla/bugs/bug113424.html:
  • tables/mozilla/bugs/bug23151.html:
  • tables/mozilla/core/bloomberg.html:
  • tables/mozilla_expected_failures/bugs/bug11331.html:
11:52 Changeset [29890] by weinig@apple.com
  • 15 edits
    2 moves
    8 adds in trunk

WebCore:

Reviewed by Darin Adler.

Fix for <rdar://problem/5708993> Mutability of the History object

  • Don't allow cross-domain get access to any of the history objects properties except the back(), forward() and go() methods.
  • Don't allow cross-domain put access to any of the history objects properties.
  • Don't allow cross-domain enumeration of the History or Location objects.

Tests: http/tests/security/cross-frame-access-history-get-override.html

http/tests/security/cross-frame-access-history-get.html
http/tests/security/cross-frame-access-history-put.html

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp: Remove unnessary KJS::'s (WebCore::JSDOMWindow::customGetOwnPropertySlot): (WebCore::JSDOMWindow::customPut): (WebCore::JSDOMWindow::getPropertyNames): Moved implementation from KJS::Window now that the declaration is autogenerated using the new CustomGetPropertyNames. (WebCore::JSDOMWindow::postMessage):
  • bindings/js/JSHistoryCustom.cpp: Added. (WebCore::allowsAccessFromFrame): (WebCore::JSHistory::customGetOwnPropertySlot): Only allow getting the declared functions back(), forward() and go() from cross-domain. Deny all other gets. (WebCore::JSHistory::customPut): Don't allow putting cross-domain. (WebCore::JSHistory::getPropertyNames): Don't allow enumeration cross-domain.
  • bindings/js/JSLocation.cpp: (WebCore::allowsAccessFromFrame): (WebCore::JSLocation::getPropertyNames): Don't allow enumeration cross-domain.
  • bindings/js/JSLocation.h:
  • bindings/js/kjs_window.cpp:
  • bindings/js/kjs_window.h:
  • bindings/scripts/CodeGeneratorJS.pm: Add support for new CustomGetPropertNames extended attribute and changed the logic of CustomPutFunction to create an overrided put() function even if no read-write properties exist.
  • page/DOMWindow.idl: Added CustomGetPropertNames
  • page/History.idl: Added CustomGetPropertNames

LayoutTests:

Reviewed by Darin Adler.

Tests for <rdar://problem/5708993> Mutability of the History object

  • http/tests/security/cross-frame-access-enumeration-expected.txt:
  • http/tests/security/cross-frame-access-enumeration.html:
  • http/tests/security/cross-frame-access-history-expected.txt: Removed.
  • http/tests/security/cross-frame-access-history-get-expected.txt: Renamed from LayoutTests/http/tests/security/cross-frame-access-history-expected.txt.
  • http/tests/security/cross-frame-access-history-get-override-expected.txt: Added.
  • http/tests/security/cross-frame-access-history-get-override.html: Added.
  • http/tests/security/cross-frame-access-history-get.html: Renamed from LayoutTests/http/tests/security/cross-frame-access-history.html.
  • http/tests/security/cross-frame-access-history-put-expected.txt: Added.
  • http/tests/security/cross-frame-access-history-put.html: Added.
  • http/tests/security/cross-frame-access-history.html: Removed.
  • http/tests/security/resources/cross-frame-access.js:
  • http/tests/security/resources/cross-frame-iframe-for-enumeration-test.html:
  • http/tests/security/resources/cross-frame-iframe-for-history-get-override-test.html: Added.
  • http/tests/security/resources/cross-frame-iframe-for-history-get-test.html: Added.
  • http/tests/security/resources/cross-frame-iframe-for-history-put-test.html: Added.
11:01 Changeset [29889] by justin.garcia@apple.com
  • 5 edits in trunk

WebCore:

Reviewed by Darin Adler.


<rdar://problem/5708115> REGRESSION: Words selected with a double click and copied won't paste into Mail

  • page/mac/WebCoreFrameBridge.h: Re-exposed smartInsertForString:, it's used by a WebKit method used by Mail.

WebKit/mac:

Reviewed by Darin Adler.


<rdar://problem/5708115> REGRESSION: Words selected with a double click and copied won't paste into Mail

  • WebView/WebHTMLView.mm: (-[WebHTMLView _smartInsertForString:replacingRange:beforeString:afterString:]): Brought this back, it's used by Mail. (-[WebHTMLView _canSmartReplaceWithPasteboard:]): This WebInternal method is also used by Mail. Moved to WebPrivate.
  • WebView/WebHTMLViewPrivate.h: Expose two methods that Mail uses here, so that we don't accidently remove them in the future.
10:50 Changeset [29888] by aroben@apple.com
  • 3 edits in trunk/WebCore

Add line box drawing code to InspectorController::drawNodeHighlight

This makes drawNodeHighlight a complete replacement for the painting
code in WebKit/mac/WebNodeHighlightView.mm, and also brings line box
rects to Windows for the first time.

Reviewed by Darin.

  • dom/Node.h: Changed isSVGElement to always exist, but to only be virtual when ENABLE(SVG) is true. This way you can always call node->isSVGElement() without checking ENABLE(SVG).
  • page/InspectorController.cpp: (WebCore::InspectorController::drawNodeHighlight): Ported line box rect code from the Mac implementation in WebNodeHighlightView.mm.
10:47 Changeset [29887] by aroben@apple.com
  • 7 edits in trunk

Let WebCore take care of the highlight drawing entirely

WebCore:

Put more knowledge about the node highlight in WebCore

InspectorController now calculates the overlay rect and node rect when
drawing the node highlight instead of having them be passed in.
InspectorController now holds onto the highlighted node so that it can
determine these rects.

Once all platforms are calling down to drawNodeHighlight instead of
drawing the highlight themselves, we can change
InspectorClient::highlight(Node*) to something like
InspectorClient::updateAndShowHighlight().

This also fixes Bug 14264: Node highlight makes it impossible to
scroll the page
<http://bugs.webkit.org/show_bug.cgi?id=14264>
<rdar://5712788>

Reviewed by Darin.

  • page/InspectorController.cpp: (WebCore::InspectorController::highlight): Store the node for use in drawNodeHighlight. (WebCore::InspectorController::drawNodeHighlight): Changed to be a const instance method. Now calculates the overlay rect and node rect instead of having them passed in.
  • page/InspectorController.h:

WebKit/win:

Let WebCore take care of the highlight drawing entirely

Reviewed by Darin.

  • WebInspectorClient.cpp: (WebInspectorClient::highlight): We now just show our highlight window and let WebCore figure out what/where to paint. Once all ports follow suit the Node* parameter to this method should be removed entirely, and the name should probably change to updateAndShowHighlight or something similar.
  • WebNodeHighlight.cpp: Renamed m_webView to m_inspectedWebViewWindow. (WebNodeHighlight::WebNodeHighlight): Now takes a WebView* parameter. (WebNodeHighlight::show): Renamed from highlight(). Now gets the WebView's HWND. Updated for member rename/removal. (WebNodeHighlight::updateWindow): Updated for member rename and for InspectorController changes.
  • WebNodeHighlight.h: Added m_inspectedWebView member, removed m_rect member, renamed highlight(Node*) -> show().
10:44 Changeset [29886] by aroben@apple.com
  • 5 edits in trunk

Move node highlight drawing code to WebCore

WebCore:

Add node highlight drawing code to InspectorController

The code came from WebKit/win/WebNodeHighlight.cpp. It's not quite as
complete as the Mac implementation (in particular, it doesn't handle
line-box rects), but it's a start.

Reviewed by Darin.

  • page/InspectorController.cpp: (WebCore::InspectorController::drawNodeHighlight): Added.
  • page/InspectorController.h:

WebKit/win:

Move node highlight drawing code to WebCore

Reviewed by Darin.

  • WebNodeHighlight.cpp: (WebNodeHighlight::updateWindow): Call into WebCore to do the node highlight drawing.
10:34 Changeset [29885] by mitz@apple.com
  • 4 edits
    4 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

Test: fast/block/float/intruding-painted-twice.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock): Pass 'false' for the new makeChildPaintOtherFloats parameter to addOverhangingFloats() because at this point we are only taking away floats from the child. (WebCore::RenderBlock::layoutBlockChildren): Pass 'true' for the new makeChildPaintOtherFloats parameter to addOverhangingFloats() iff the child was not laid out again. Only in that case, it may have overhanging floats that it does not paint because they used to be overhanging from the parent, but now they are not. (WebCore::RenderBlock::addOverhangingFloats): Refined the conditions for making the child paint the float: require that the float be a descendant of the child (the other case is when it intrudes into the child from another sibling) and that it does not have a layer (in which case it paints itself). In addition, do the check only if the caller passed 'true' for the makeChildPaintOtherFloats parameter.
  • rendering/RenderBlock.h:

LayoutTests:

Reviewed by Dave Hyatt.

  • fast/block/float/intruding-painted-twice.html: Added.
  • platform/mac-leopard/fast/block/float/intruding-painted-twice-expected.checksum: Added.
  • platform/mac-leopard/fast/block/float/intruding-painted-twice-expected.png: Added.
  • platform/mac/fast/block/float/intruding-painted-twice-expected.txt: Added.

01/30/08:

21:39 Changeset [29884] by mitz@apple.com
  • 9 edits
    4 adds in trunk

WebCore:

Reviewed by Sam Weinig.

  • change the interpretation of unicode-range values in "from-to" form to include the "to" character.

Test: fast/css/font-face-unicode-range.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseFontFaceUnicodeRange):
  • css/CSSSegmentedFontFace.cpp: (WebCore::CSSSegmentedFontFace::overlayRange):
  • platform/graphics/GlyphPageTreeNode.cpp: (WebCore::GlyphPageTreeNode::initializePage):
  • platform/graphics/SegmentedFontData.cpp: (WebCore::SegmentedFontData::fontDataForCharacter): (WebCore::SegmentedFontData::containsCharacters):

LayoutTests:

Reviewed by Sam Weinig.

  • add a test for the unicode-range @font-face property
  • update @font-face rules in existing tests after changing unicode-range values in "from-to" form to include the "to" character
  • fast/css/font-face-unicode-range.html: Added.
  • fast/css/rtl-ordering.html:
  • fast/text/international/resources/Mac-compatible-font-fallback.css:
  • platform/mac-leopard/fast/css/font-face-unicode-range-expected.checksum: Added.
  • platform/mac-leopard/fast/css/font-face-unicode-range-expected.png: Added.
  • platform/mac/fast/css/font-face-unicode-range-expected.txt: Added.
  • platform/win/css2.1/resources/Mac-compatible-font-fallback.css:
21:36 Changeset [29883] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

  • prune references to custom fonts' SimpleFontData from the glyph page tree when they are destroyed
  • css/CSSFontFaceSource.cpp: (WebCore::CSSFontFaceSource::pruneTable):
18:45 Changeset [29882] by justin.garcia@apple.com
  • 1 add
    1 delete in trunk/LayoutTests/editing/style

This test started failed with my last checkin. The new results are *correct* but I don't know why my changes would have fixed the test, so I want to investigate a bit before I check in new results.

17:45 Changeset [29881] by timothy@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Tim Hatcher.

New fix for <rdar://problem/5688428> Reproducible assertion failure
in SQLTransaction::performNextStep() (16876)

Fixes a deadlock that was happening for all Database use. All
storage layout tests pass.

  • storage/Database.cpp: (WebCore::CurrentThreadSetter::CurrentThreadSetter): New helper class to set the current thread and zero it on destruction. (WebCore::CurrentThreadSetter::~CurrentThreadSetter): Set threadIdentifierStorage to 0. (WebCore::Database::Database): Set m_transactionStepThread to 0. (WebCore::Database::performTransactionStep): Set m_transactionStepThread to currentThread(). (WebCore::Database::scheduleTransactionCallback): Assert m_transactionStepThread is currentThread().
  • storage/Database.h: Add m_transactionStepThread for debug builds to track which thread performTransactionStep() was called on.
17:23 Changeset [29880] by slewis@apple.com
  • 2 edits in trunk/WebKitTools

2008-01-30 Stephanie <slewis@apple.com>

Reviewed by Geoff.

Add an option to run-webkit-tests to merge leaks results and print the number of unique leaks found under a certain call depth. This should make it easier for the buildbots to track new leaks.

  • Scripts/run-webkit-tests:
16:55 Changeset [29879] by timothy@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

ASSERTION FAILED: JSLock::lockCount() > 0 when opening the
Web Inspector on a page with a Database.

Fix the ASSERT by taking a JSLock before calling toJS().

  • page/InspectorController.cpp: (WebCore::InspectorController::addDatabaseScriptResource):
16:43 Changeset [29878] by bdakin@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin.

Fix for <rdar://problem/5598609> CrashTracer: [USER] 626 crashes in
Safari at com.apple.WebCore: WebCore::FrameView::needsFullRepaint
const + 6

The real problem here is tracked by rdar://5598072, which is that
frames can lose sync between their view and their document when a
non-HTML view is loaded (such as bookmarks view or a PDF). That can
cause this crash if the layout timer fires before things have fixed
themselves. This fix turns an ASSERT in FrameView::layout() into an
early return to cause graceful failure until the root of the
problem is addressed.

  • page/FrameView.cpp: (WebCore::FrameView::layout):
16:16 Changeset [29877] by justin.garcia@apple.com
  • 3 edits
    8 adds in trunk

WebCore:

Reviewed by Darin Adler.

<rdar://problem/5700414> REGRESSION (Adama-ToT): Selecting "Header 1" style in Leopard Server wiki inserts newline

  • editing/FormatBlockCommand.cpp: (WebCore::FormatBlockCommand::doApply): If the selected paragraph was empty, we may still need to call moveParagrah to remove the line break that holds that paragraph open because the new block of the requested type needs to to replace it.

LayoutTests:

Reviewed by Darin Adler.


<rdar://problem/5700414> REGRESSION (Adama-ToT): Selecting "Header 1" style in Leopard Server wiki inserts newline

  • editing/execCommand/5700414-1.html: Added.
  • editing/execCommand/5700414-2.html: Added.
  • platform/mac/editing/execCommand/5700414-1-expected.checksum: Added.
  • platform/mac/editing/execCommand/5700414-1-expected.png: Added.
  • platform/mac/editing/execCommand/5700414-1-expected.txt: Added.
  • platform/mac/editing/execCommand/5700414-2-expected.checksum: Added.
  • platform/mac/editing/execCommand/5700414-2-expected.png: Added.
  • platform/mac/editing/execCommand/5700414-2-expected.txt: Added.
14:26 Changeset [29876] by timothy@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

<rdar://problem/5688428> Reproducible assertion failure in SQLTransaction::performNextStep() (16876)

Revised fix to use the globalCallbackMutex() mutex and hold the mutex
for the entire function scope.

  • storage/Database.cpp: (WebCore::Database::performTransactionStep):
14:06 Changeset [29875] by timothy@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

Make ResourceTreeElement inherit the prototype of TreeElement.

  • page/inspector/Resource.js:
14:01 Changeset [29874] by jess@apple.com
  • 2 edits in trunk/WebKitSite

Reviewed by Tim Hatcher.

  • coding/coding-style.html: Adding a note about avoiding explicit nil/NO initialization of Objective-C instance variables.
13:56 Changeset [29873] by weinig@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Adam Roben.

Don't include SecurityOrigin.h in Document.h so that we can avoid massive
recompilation when changing the SecurityOrigin.

  • dom/Document.cpp: (WebCore::Document::setSecurityOrigin):
  • dom/Document.h:
12:38 Changeset [29872] by timothy@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Tim Hatcher and Brady.

<rdar://problem/5688428> Reproducible assertion failure in SQLTransaction::performNextStep() (16876)

  • storage/Database.cpp: (WebCore::Database::performTransactionStep): Do not perform the next transaction if a global callback is already scheduled.
11:40 Changeset [29871] by jhoneycutt@apple.com
  • 4 edits in trunk/WebCore

2008-01-30 Jon Honeycutt <jhoneycutt@apple.com>

Reviewed by Darin.

<rdar://problem/5669317> Crash closing pop up window with Real Player
content

Some RealPlayer versions hang on unload. To prevent this hang, don't
unload versions since the last known unloadable version.
<rdar://problem/5713147> tracks revisiting this when the bug in
RealPlayer is fixed and we can again unload the plug-in.

  • plugins/win/PluginPackageWin.cpp: (WebCore::PluginPackageWin::compareFileVersion): Compare the plug-in's file version to the passed file version, returning -1, 0, or 1 if the plug-in's version is less than, equal to, or greater than the version passed. (WebCore::PluginPackageWin::isPluginBlacklisted): Use compareFileVersion()
  • plugins/win/PluginPackageWin.h:
  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::determineQuirks): If the plug-in version is newer than the last-known unloadable version, add the DontUnloadPlugin quirk
09:55 Changeset [29870] by ap@webkit.org
  • 2 edits in trunk/WebKitTools

Reviewed by Adam Roben.

One more debug (external) fix.

  • DumpRenderTree/win/DumpRenderTree.cpp: TestNetscapePlugin is built with a _debug suffix in both DebugInternal and Debug configurations.
04:14 Changeset [29869] by mrowe@apple.com
  • 2 edits in trunk/WebKit/mac

Move off deprecated NSTableView methods.

Reviewed by Oliver Hunt.

  • WebView/WebHTMLView.mm:

(-[WebTextCompleteController _buildUI]): Switch from -setDrawsGrid: to -setGridStyleMask:.
(-[WebTextCompleteController _placePopupWindow:]): Switch from -selectRow:byExtendingSelection: to -selectRowIndexes:byExtendingSelection:.
(-[WebTextCompleteController filterKeyDown:]): Ditto.

01/29/08:

21:26 Changeset [29868] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Look for headers relative to NEXT_ROOT so that build-root picks up the WebCoreSQLite version of sqlite3.h rather than the older system version.

Reviewed by Tim Hatcher.

21:10 Changeset [29867] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Reviewed by Dave Hyatt.

  • try to make this test more robust on slower machines
  • fast/css/font-face-remote.html:
20:32 Changeset [29866] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • build fix
  • loader/CachedFont.cpp: (WebCore::CachedFont::CachedFont):
20:03 Changeset [29865] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Add the correct directory to the header search paths in Production builds. Reviewed by Brady Eidson.

19:50 Changeset [29864] by mitz@apple.com
  • 6 edits
    5 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

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

Added an isSVGFont method to CachedFont and use it to decide how to
process the font data. Currently whether the data is SVG is determined
based on the format property in the @font-face rule, but in the future
the MIME type or other metadata may be used when the format is
unspecified.

  • css/CSSFontFaceSource.cpp: (WebCore::CSSFontFaceSource::getFontData): Changed to parse the data as SVG only if the CachedFont is an SVG font.
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule): Mark the CachedFont as an SVG font based on format.
  • loader/CachedFont.cpp: (WebCore::CachedFont::CachedFont): Initialize m_isSVGFont to false. (WebCore::CachedFont::ensureCustomFontData): (WebCore::CachedFont::ensureSVGFontData): (WebCore::CachedFont::getSVGFontById):
  • loader/CachedFont.h: (WebCore::CachedFont::isSVGFont): Added. (WebCore::CachedFont::setSVGFont): Added.

LayoutTests:

Reviewed by Dave Hyatt.

This is the first @font-face test that actually tests loading font data
from a URL.

  • fast/css/font-face-remote.html: Added.
  • fast/css/resources/Ahem.ttf: Added.
  • platform/mac/fast/css/font-face-remote-expected.checksum: Added.
  • platform/mac/fast/css/font-face-remote-expected.png: Added.
  • platform/mac/fast/css/font-face-remote-expected.txt: Added.
19:43 Changeset [29863] by aroben@apple.com
  • 2 edits in trunk/WebKit/win

Fix a crash when opening the Inspector

Reviewed by Anders.

  • WebInspectorClient.cpp: (WebInspectorClient::createPage): Call WebPreferences::initWithIdentifier so our custom WebPreferences will have all its members initialized.
18:45 Changeset [29862] by aroben@apple.com
  • 2 edits in trunk/WebKit/win

Fix Bug 16786: "Display images when page opens" preference affects Inspector

<http://bugs.webkit.org/show_bug.cgi?id=16786>
<rdar://5712914>

Reviewed by Anders and Darin.

  • WebInspectorClient.cpp: (WebInspectorClient::createPage): Create our own custom WebPreferences so that we won't be affected by global preference changes. This matches what Mac does.
18:38 Changeset [29861] by antti@apple.com
  • 3 edits in trunk/WebCore

Some comment cleanup.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::disableUnsupportedTracks):
  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWin::load): (QTMovieWin::disableUnsupportedTracks):
18:00 Changeset [29860] by adele@apple.com
  • 1 edit
    108 deletes in trunk/LayoutTests

Reviewed by Oliver.

Remove pixel results for text-only tests that I mistakingly added.

  • platform/mac/media/audio-constructor-expected.checksum: Removed.
  • platform/mac/media/audio-constructor-expected.png: Removed.
  • platform/mac/media/audio-constructor-src-expected.checksum: Removed.
  • platform/mac/media/audio-constructor-src-expected.png: Removed.
  • platform/mac/media/broken-video-expected.checksum: Removed.
  • platform/mac/media/broken-video-expected.png: Removed.
  • platform/mac/media/constructors-expected.checksum: Removed.
  • platform/mac/media/constructors-expected.png: Removed.
  • platform/mac/media/fallback-expected.checksum: Removed.
  • platform/mac/media/fallback-expected.png: Removed.
  • platform/mac/media/progress-event-expected.checksum: Removed.
  • platform/mac/media/progress-event-expected.png: Removed.
  • platform/mac/media/remove-from-document-expected.checksum: Removed.
  • platform/mac/media/remove-from-document-expected.png: Removed.
  • platform/mac/media/unsupported-rtsp-expected.checksum: Removed.
  • platform/mac/media/unsupported-rtsp-expected.png: Removed.
  • platform/mac/media/unsupported-tracks-expected.checksum: Removed.
  • platform/mac/media/unsupported-tracks-expected.png: Removed.
  • platform/mac/media/video-append-source-expected.checksum: Removed.
  • platform/mac/media/video-append-source-expected.png: Removed.
  • platform/mac/media/video-autoplay-expected.checksum: Removed.
  • platform/mac/media/video-autoplay-expected.png: Removed.
  • platform/mac/media/video-buffered-expected.checksum: Removed.
  • platform/mac/media/video-buffered-expected.png: Removed.
  • platform/mac/media/video-controls-expected.checksum: Removed.
  • platform/mac/media/video-controls-expected.png: Removed.
  • platform/mac/media/video-currentTime-expected.checksum: Removed.
  • platform/mac/media/video-currentTime-expected.png: Removed.
  • platform/mac/media/video-currentTime-set-expected.checksum: Removed.
  • platform/mac/media/video-currentTime-set-expected.png: Removed.
  • platform/mac/media/video-currentTime-set2-expected.checksum: Removed.
  • platform/mac/media/video-currentTime-set2-expected.png: Removed.
  • platform/mac/media/video-dom-autoplay-expected.checksum: Removed.
  • platform/mac/media/video-dom-autoplay-expected.png: Removed.
  • platform/mac/media/video-dom-end-expected.checksum: Removed.
  • platform/mac/media/video-dom-end-expected.png: Removed.
  • platform/mac/media/video-dom-loopcount-expected.checksum: Removed.
  • platform/mac/media/video-dom-loopcount-expected.png: Removed.
  • platform/mac/media/video-dom-loopend-expected.checksum: Removed.
  • platform/mac/media/video-dom-loopend-expected.png: Removed.
  • platform/mac/media/video-dom-loopstart-expected.checksum: Removed.
  • platform/mac/media/video-dom-loopstart-expected.png: Removed.
  • platform/mac/media/video-dom-src-expected.checksum: Removed.
  • platform/mac/media/video-dom-src-expected.png: Removed.
  • platform/mac/media/video-dom-start-expected.checksum: Removed.
  • platform/mac/media/video-dom-start-expected.png: Removed.
  • platform/mac/media/video-end-expected.checksum: Removed.
  • platform/mac/media/video-end-expected.png: Removed.
  • platform/mac/media/video-error-abort-expected.checksum: Removed.
  • platform/mac/media/video-error-abort-expected.png: Removed.
  • platform/mac/media/video-error-does-not-exist-expected.checksum: Removed.
  • platform/mac/media/video-error-does-not-exist-expected.png: Removed.
  • platform/mac/media/video-load-networkState-expected.checksum: Removed.
  • platform/mac/media/video-load-networkState-expected.png: Removed.
  • platform/mac/media/video-load-readyState-expected.checksum: Removed.
  • platform/mac/media/video-load-readyState-expected.png: Removed.
  • platform/mac/media/video-loopcount-expected.checksum: Removed.
  • platform/mac/media/video-loopcount-expected.png: Removed.
  • platform/mac/media/video-loopend-expected.checksum: Removed.
  • platform/mac/media/video-loopend-expected.png: Removed.
  • platform/mac/media/video-loopstart-expected.checksum: Removed.
  • platform/mac/media/video-loopstart-expected.png: Removed.
  • platform/mac/media/video-muted-expected.checksum: Removed.
  • platform/mac/media/video-muted-expected.png: Removed.
  • platform/mac/media/video-no-autoplay-expected.checksum: Removed.
  • platform/mac/media/video-no-autoplay-expected.png: Removed.
  • platform/mac/media/video-pause-empty-events-expected.checksum: Removed.
  • platform/mac/media/video-pause-empty-events-expected.png: Removed.
  • platform/mac/media/video-play-empty-events-expected.checksum: Removed.
  • platform/mac/media/video-play-empty-events-expected.png: Removed.
  • platform/mac/media/video-play-pause-events-expected.checksum: Removed.
  • platform/mac/media/video-play-pause-events-expected.png: Removed.
  • platform/mac/media/video-play-pause-exception-expected.checksum: Removed.
  • platform/mac/media/video-play-pause-exception-expected.png: Removed.
  • platform/mac/media/video-poster-expected.checksum: Removed.
  • platform/mac/media/video-poster-expected.png: Removed.
  • platform/mac/media/video-seek-past-end-paused-expected.checksum: Removed.
  • platform/mac/media/video-seek-past-end-paused-expected.png: Removed.
  • platform/mac/media/video-seek-past-end-playing-expected.checksum: Removed.
  • platform/mac/media/video-seek-past-end-playing-expected.png: Removed.
  • platform/mac/media/video-seekable-expected.checksum: Removed.
  • platform/mac/media/video-seekable-expected.png: Removed.
  • platform/mac/media/video-seeking-expected.checksum: Removed.
  • platform/mac/media/video-seeking-expected.png: Removed.
  • platform/mac/media/video-size-expected.checksum: Removed.
  • platform/mac/media/video-size-expected.png: Removed.
  • platform/mac/media/video-source-expected.checksum: Removed.
  • platform/mac/media/video-source-expected.png: Removed.
  • platform/mac/media/video-source-media-expected.checksum: Removed.
  • platform/mac/media/video-source-media-expected.png: Removed.
  • platform/mac/media/video-source-type-expected.checksum: Removed.
  • platform/mac/media/video-source-type-expected.png: Removed.
  • platform/mac/media/video-src-change-expected.checksum: Removed.
  • platform/mac/media/video-src-change-expected.png: Removed.
  • platform/mac/media/video-src-expected.checksum: Removed.
  • platform/mac/media/video-src-expected.png: Removed.
  • platform/mac/media/video-src-remove-expected.checksum: Removed.
  • platform/mac/media/video-src-remove-expected.png: Removed.
  • platform/mac/media/video-src-set-expected.checksum: Removed.
  • platform/mac/media/video-src-set-expected.png: Removed.
  • platform/mac/media/video-src-source-expected.checksum: Removed.
  • platform/mac/media/video-src-source-expected.png: Removed.
  • platform/mac/media/video-start-expected.checksum: Removed.
  • platform/mac/media/video-start-expected.png: Removed.
  • platform/mac/media/video-volume-expected.checksum: Removed.
  • platform/mac/media/video-volume-expected.png: Removed.
  • platform/mac/media/video-width-height-expected.checksum: Removed.
  • platform/mac/media/video-width-height-expected.png: Removed.
17:40 Changeset [29859] by adele@apple.com
  • 4 edits in trunk

WebCore:

Reviewed by Antti.

Fix for <rdar://5683527> media/{audio,videocontrols-rendering.html failing


  • rendering/RenderThemeSafari.cpp: (WebCore::RenderThemeSafari::isControlStyled): Don't treat all media controls as styled. This causes builds with an old version of SafariTheme to have different metrics for these controls.

LayoutTests:

Reviewed by Antti.

Removing test from Skipped list for <rdar://5683527> media/{audio,videocontrols-rendering.html failing


  • platform/win/Skipped:
16:52 Changeset [29858] by mitz@apple.com
  • 2 edits in trunk/LayoutTests
  • re-add a couple of tests that are still failing back to the list
  • platform/win/Skipped:
16:28 Changeset [29857] by mitz@apple.com
  • 4 edits in trunk/WebCore

Reviewed by Dave Hyatt.

  • make Font instances cache their primary font
  • WebCore.base.exp:
  • platform/graphics/Font.cpp: (WebCore::Font::Font): (WebCore::Font::operator=): (WebCore::Font::cachePrimaryFont): (WebCore::Font::update):
  • platform/graphics/Font.h: (WebCore::Font::primaryFont):
16:18 Changeset [29856] by ggaren@apple.com
  • 25 edits
    2 adds in trunk

WebKitTools:

Reviewed by Sam Weinig. Sam Weinig.


Fixed <rdar://problem/5692566> fast/encoding/mailto-always-utf-8.html
fails when run after fast/dom/Window/window-property-shadowing.html (16902)

Modified our policy delegates not to output window.name. Since a
previous test may have explicitly set window.name, we can't rely on its
implicit value in test results.

  • DumpRenderTree/mac/PolicyDelegate.mm: (-[PolicyDelegate webView:decidePolicyForNavigationAction:request:frame:decisionListener:]):
  • DumpRenderTree/win/PolicyDelegate.cpp: (PolicyDelegate::decidePolicyForNavigationAction):

LayoutTests:

Reviewed by Sam Weinig. Sam Weinig.


Fixed <rdar://problem/5692566> fast/encoding/mailto-always-utf-8.html
fails when run after fast/dom/Window/window-property-shadowing.html (16902)

These layout tests demonstrate the expected behavior of window.name,
including the fact that one test can change the window.name value seen
by another test:

  • fast/dom/Window/window-property-shadowing-name-expected.txt: Added.
  • fast/dom/Window/window-property-shadowing-name.html: Added.
  • fast/dom/Window/window-property-clearing-expected.txt:
  • fast/dom/Window/window-property-clearing.html:


Modified these expected results not to depend on window.name. Since a
previous test may have explicitly set window.name, we can't rely on its
implicit value:

  • fast/encoding/mailto-always-utf-8-expected.txt:
  • fast/forms/mailto/advanced-get-expected.txt:
  • fast/forms/mailto/advanced-put-expected.txt:
  • fast/forms/mailto/get-multiple-items-expected.txt:
  • fast/forms/mailto/get-multiple-items-text-plain-expected.txt:
  • fast/forms/mailto/get-multiple-items-x-www-form-urlencoded-expected.txt:
  • fast/forms/mailto/get-non-ascii-always-utf-8-expected.txt:
  • fast/forms/mailto/get-non-ascii-expected.txt:
  • fast/forms/mailto/get-non-ascii-text-plain-expected.txt:
  • fast/forms/mailto/get-non-ascii-text-plain-latin-1-expected.txt:
  • fast/forms/mailto/get-overwrite-query-expected.txt:
  • fast/forms/mailto/post-append-query-expected.txt:
  • fast/forms/mailto/post-multiple-items-expected.txt:
  • fast/forms/mailto/post-multiple-items-multipart-form-data-expected.txt:
  • fast/forms/mailto/post-multiple-items-text-plain-expected.txt:
  • fast/forms/mailto/post-multiple-items-x-www-form-urlencoded-expected.txt:
  • fast/forms/mailto/post-text-plain-expected.txt:
  • fast/forms/mailto/post-text-plain-with-accept-charset-expected.txt:
  • http/tests/security/feed-urls-from-remote-expected.txt:
16:16 Changeset [29855] by mitz@apple.com
  • 2 edits in trunk/WebCore
  • Leopard build fix
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData):
16:13 Changeset [29854] by mrowe@apple.com
  • 8 edits
    4 adds in trunk

<rdar://problem/5600926> WebCore on Tiger must link to its own copy of SQLite 3.4 or newer (so HTML database behavior will be correct).

Reviewed by Tim Hatcher.

15:03 Changeset [29853] by mitz@apple.com
  • 4 edits in trunk

WebCore:

Reviewed by Adam Roben and Oliver Hunt.

  • fix <rdar://problem/5713131> REGRESSION (r29246): Many SVG font tests are failing
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData): Use double instead of float for consistent rounding behavior between Mac OS X and Windows.

LayoutTests:

Reviewed by Adam Roben and Oliver Hunt.

  • remove skipped tests after fixing <rdar://problem/5713131> REGRESSION (r29246): Many SVG font tests are failing
  • platform/win/Skipped:
15:03 Changeset [29852] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

Refactor ChangeLog path code

Reviewed by Adam.

  • Scripts/prepare-ChangeLog: Simplify code in one foreach() loop, and extract duplicate code in another foreach() loop.
14:33 Changeset [29851] by aroben@apple.com
  • 5 edits in trunk

Fix <rdar://5713302> Web Inspector on Windows is not using the localized strings

WebCore:

Fix <rdar://5713302> Web Inspector on Windows is not using the
localized strings

Reviewed by Steve.

  • WebCore.vcproj/WebCore.vcproj: Copy InspectorLocalizedStrings.js to $WebKitOutputDir.
  • page/InspectorController.cpp: Touch this to force the project to build.

WebKit/win:

Fix <rdar://5713302> Web Inspector on Windows is not using the
localized strings

Reviewed by Steve.

  • WebInspectorClient.cpp: (getWebKitBundle): Added. (WebInspectorClient::createPage): Call getWebKitBundle. (WebInspectorClient::localizedStringsURL): Implemented.
14:22 Changeset [29850] by eric@webkit.org
  • 4 edits
    2 adds in trunk

Reviewed by Nikolas.

Acid3 after double-attach
http://bugs.webkit.org/show_bug.cgi?id=17058

I believe SVGTextPathElement::buildPendingResource() was entirely
bogus, removing it did not cause any tests to fail.

Test: svg/custom/textPath-assert.svg

  • svg/SVGTextPathElement.cpp: (WebCore::SVGTextPathElement::insertedIntoDocument): remove buildPendingResource()
  • svg/SVGTextPathElement.h:
13:35 Changeset [29849] by aroben@apple.com
  • 2 edits in trunk/WebCore

Fix Bug 16234: Inspector should support searching for elements by CSS selectors

<http://bugs.webkit.org/show_bugs.cgi?id=16234>
<rdar://5712862>

Reviewed by Tim.

  • page/inspector/inspector.js: Use Document.querySelectorAll to search for elements by CSS selector. Also store a custom property on nodes being added to the search results to avoid showing the same node more than once.
12:20 Changeset [29848] by aroben@apple.com
  • 2 edits in trunk/LayoutTests
  • platform/win/Skipped: Added new failures.
11:55 Changeset [29847] by aroben@apple.com
  • 2 edits in trunk/WebCore

Fix <rdar://5711136> Full-screen Flash on 1up.com is unresponsive

Reviewed by Anders and Darin.

No test possible.

  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::wndProc): Set/release capture on mouse down/up, like Firefox does.
09:18 Changeset [29846] by ap@webkit.org
  • 6 edits in trunk

Reviewed by Darin.

<rdar://problem/5710692> All storage tests fail/crash

WebKit:

  • WebChromeClient.cpp: (WebChromeClient::exceededDatabaseQuota): Clients that do not implement database quota callbacks should have no database support - but not crash.

WebKitTools:

  • DumpRenderTree/mac/UIDelegate.mm: (-[UIDelegate webView:frame:exceededDatabaseQuotaForSecurityOrigin:database:]): Set an arbitrary quota of 5 megabytes.

(-[UIDelegate webView:frame:quotaForSecurityOrigin:toCreateDatabase:withEstimatedSize:]):
(-[UIDelegate webView:frame:quotaForSecurityOrigin:fromProposedQuota:database:]):
Removed unused (I think) code.


  • DumpRenderTree/win/UIDelegate.cpp: (UIDelegate::QueryInterface): IWebUIDelegatePrivate3 is now supported. (UIDelegate::webViewPainted): A stub implementation. (UIDelegate::exceededDatabaseQuota): Set an arbitrary quota of 5 megabytes.
  • DumpRenderTree/win/UIDelegate.h:
03:47 Changeset [29845] by ap@webkit.org
  • 2 edits in trunk/WebKitLibraries

Debug (external) build fix.

Removed _DEBUG preprocessor definition, which indicates that debug libraries are used (while they aren't).
This preprocessor definition is automatically set by Visual Studio as needed anyway.

  • win/tools/vsprops/debug.vsprops:
Note: See TracTimeline for information about the timeline view.