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

Timeline



Sep 5, 2008:

10:37 PM Changeset in webkit [36128] by timothy@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.exp

Build fix.

10:19 PM Changeset in webkit [36127] by timothy@apple.com
  • 1 edit in trunk/JavaScriptCore/JavaScriptCore.exp

Build fix.

10:19 PM Changeset in webkit [36126] by timothy@apple.com
  • 3 edits in trunk/WebKit/mac

Correct a typo in the setApplicationChromeModeEnabledEnabled:
method name, remove the extra "Enabled".

  • WebView/WebPreferences.mm:
  • WebView/WebPreferencesPrivate.h:
10:07 PM WebKit Team edited by timothy@apple.com
Remove Drosera. (diff)
10:06 PM WebKit Team edited by timothy@apple.com
Minor tweaks. (diff)
10:05 PM Web Inspector edited by timothy@apple.com
Fix a link to my name. (diff)
10:04 PM WebKit Team edited by oliver@apple.com
Minor updates i noticed (diff)
9:59 PM Changeset in webkit [36125] by Darin Adler
  • 3 edits in trunk/JavaScriptCore

2008-09-05 Darin Adler <Darin Adler>

Reviewed by Cameron Zwarich.

1.007x as fast on SunSpider overall
1.081x as fast on SunSpider math-cordic

  • VM/JSPropertyNameIterator.cpp: Moved functions out of here.
  • VM/JSPropertyNameIterator.h: (KJS::JSPropertyNameIterator::JSPropertyNameIterator): Moved this into the header and marked it inline. (KJS::JSPropertyNameIterator::create): Ditto. (KJS::JSPropertyNameIterator::next): Ditto.
9:58 PM Changeset in webkit [36124] by Darin Adler
  • 6 edits in trunk/JavaScriptCore

2008-09-05 Darin Adler <Darin Adler>

Reviewed by Geoff Garen.

1.007x as fast on SunSpider overall
1.167x as fast on SunSpider string/fasta

  • JavaScriptCore.exp: Updated.
  • kjs/SmallStrings.cpp: (KJS::SmallStrings::singleCharacterStringRep): Added.
  • kjs/SmallStrings.h: Added singleCharacterStringRep for clients that need just a UString, not a JSString.
  • kjs/identifier.cpp: (KJS::Identifier::add): Added special cases for single character strings so that the UString::Rep that ends up in the identifier table is the one from the single-character string optimization; otherwise we end up having to look it up in the identifier table over and over again. (KJS::Identifier::addSlowCase): Ditto. (KJS::Identifier::checkSameIdentifierTable): Made this function an empty inline in release builds so that callers don't have to put #ifndef NDEBUG at each call site.
  • kjs/identifier.h: (KJS::Identifier::add): Removed #ifndef NDEBUG around the calls to checkSameIdentifierTable. (KJS::Identifier::checkSameIdentifierTable): Added. Empty inline version for NDEBUG builds.
9:08 PM Changeset in webkit [36123] by alp@webkit.org
  • 8 edits in trunk

2008-09-05 Gustavo Noronha Silva <Gustavo Noronha Silva>

Reviewed by Alp Toker.

​https://bugs.webkit.org/show_bug.cgi?id=18346
[GTK] Remove build warnings

Applied some casts, and removed an unused typedef to make the
compiler happy, printing less warnings when building.

5:56 PM Changeset in webkit [36122] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix.

  • kjs/JSObject.h: Move the inline virtual destructor after a non-inline

virtual function so that the symbol for the vtable is not marked as a
weakly exported symbol.

5:21 PM Changeset in webkit [36121] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

Build fix for WebKitWin and Chromium

  • platform/FileSystem.h:
3:27 PM WebKit Team edited by alice.liu@apple.com
Added accessibility to Beth's and my sections (diff)
3:17 PM Changeset in webkit [36120] by eric@webkit.org
  • 9 edits in trunk/WebCore

Reviewed by Darin Adler.

Try to make Chromium compile with ToT:

  • Wrap a few places which depend on KJS:: in #if USE(JSC)
  • Include some windows forward declarations
  • dom/Node.h:
  • page/Console.h:
  • page/animation/CompositeAnimation.h:
  • platform/FileSystem.h:
  • platform/graphics/Image.h:
  • platform/text/AtomicString.h:
  • platform/text/String.cpp:
  • rendering/style/RenderStyle.h:
3:08 PM Changeset in webkit [36119] by dino@apple.com
  • 3 edits
    1 add in trunk/WebKitSite/specs/CSSVisualEffects

2008-09-04 Dean Jackson <​dino@apple.com>

Updates to draft specifications.

Animations

  • Keyframes must provide 0% and 100% to be valid <rdar://problem/6165732> [animations] Spec should say what happens if you omit the 0% or 100% keyframe
  • Mention that keyframes can be specified in any order <rdar://problem/6165728> [animations] Spec should say that keyframes can be in any order (It actually already said this, but I made it more clear)
  • Keyframe names must be IDENT not STRING <rdar://problem/6164640> [animations] should we allow quoted animation names

Transforms

  • Change transform operation values in CSSTransformValue - splitting 2d and 3d.
  • Removed definition of <angle> - now reference CSS Values and Units consequence is that rotations now support "turn" units (feel free to turn it up to 11) <rdar://problem/6177272> [transforms] Should support "turns" as a rotation unit
  • Added exception to CSSMatrix setMatrixValue() and inverse() methods <rdar://problem/6160283> [transforms] Specify error reporting for WebKitCSSMatrix
  • Added getCSSMatrix() method to CSSTransformValue
  • Added Table of Contents to Transforms
  • skewZ() transform function removed. It makes no sense.
  • Definition for skew() transform function was missing
  • Also added a new CSSTransformValue CSS_SKEW <rdar://problem/6137414> [transforms] skew() function definition is missing
  • Removed mention of transitions when discussion keyframes rule <rdar://problem/6103072> [animations] CSS Animation spec mentions transition keyframes
2:52 PM Changeset in webkit [36118] by hyatt@apple.com
  • 12 edits in trunk

Add support for runtime switchability of the RenderTheme.

2:51 PM Changeset in webkit [36117] by Darin Adler
  • 2 edits in trunk/JavaScriptCore

2008-09-05 Darin Adler <Darin Adler>

Reviewed by Sam Weinig.

1.011x as fast on SunSpider overall
1.028x as fast on SunSpider string tests

For small strings, use a loop rather than calling memcpy. The loop can
be faster because there's no function call overhead, and because it can
assume the pointers are aligned instead of checking that. Currently the
threshold is set at 20 characters, based on some testing on one particular
computer. Later we can tune this for various platforms by setting
USTRING_COPY_CHARS_INLINE_CUTOFF appropriately, but it does no great harm
if not perfectly tuned.

  • kjs/ustring.cpp: (KJS::overflowIndicator): Removed bogus const. (KJS::maxUChars): Ditto. (KJS::copyChars): Added. (KJS::UString::Rep::createCopying): Call copyChars instead of memcpy. Also eliminated need for const_cast. (KJS::UString::expandPreCapacity): Ditto. (KJS::concatenate): Ditto. (KJS::UString::spliceSubstringsWithSeparators): Ditto. (KJS::UString::append): Ditto.
2:09 PM Changeset in webkit [36116] by mrowe@apple.com
  • 1 copy in branches/squirrelfish-extreme

New branch.

11:52 AM Changeset in webkit [36115] by Antti Koivisto
  • 2 edits in trunk/WebCore

2008-09-05 Antti Koivisto <Antti Koivisto>

Qt build fix.

  • svg/SVGFEImageElement.cpp: (WebCore::SVGFEImageElement::build):
11:15 AM Changeset in webkit [36114] by Darin Adler
  • 15 edits in trunk

Updated some of the ChangeLog files to use the full 64-bit universal version of my name.

10:52 AM Changeset in webkit [36113] by kmccullough@apple.com
  • 3 edits in trunk/JavaScriptCore

2008-09-05 Kevin McCullough <​kmccullough@apple.com>

Reviewed by Sam and Alexey.

Make the profiler work with a null exec state. This will allow other
applications start the profiler to get DTrace probes going without
needing a WebView.

  • ChangeLog:
  • profiler/ProfileGenerator.cpp: (KJS::ProfileGenerator::ProfileGenerator): (KJS::ProfileGenerator::willExecute): (KJS::ProfileGenerator::didExecute):
  • profiler/Profiler.cpp: (KJS::Profiler::startProfiling): (KJS::Profiler::stopProfiling): (KJS::dispatchFunctionToProfiles):
8:33 AM Changeset in webkit [36112] by Adam Roben
  • 2 edits in trunk/WebCore

2008-09-05 Dirk Schulze <​vbs85@gmx.de>

Gtk build fix

  • GNUmakefile.am:
3:36 AM Changeset in webkit [36111] by Antti Koivisto
  • 2 edits in trunk/WebCore

2008-09-05 Antti Koivisto <Antti Koivisto>

Another build fix.

  • svg/SVGFEImageElement.cpp: (WebCore::SVGFEImageElement::notifyFinished):
3:03 AM Changeset in webkit [36110] by Antti Koivisto
  • 3 edits in trunk/WebCore

2008-09-05 Antti Koivisto <Antti Koivisto>

Build fixes.


  • WebCore.xcodeproj/project.pbxproj:
  • svg/graphics/filters/SVGFEImage.cpp: (WebCore::FEImage::cachedImage):
2:28 AM Changeset in webkit [36109] by Antti Koivisto
  • 31 edits
    2 adds in trunk/WebCore

2008-09-05 Antti Koivisto <Antti Koivisto>

Reviewed by Darin.

Most of the implementation for ​https://bugs.webkit.org/show_bug.cgi?id=17998
When a resource is cached locally, WebKit should follow RFC 2616 "Specific end-to-end revalidation" instead of "Unspecified end-to-end revalidation"

Implement HTTP 1.1 "Specific end-to-end revalidation" for WebCore memory cache. This patch does
not yet enable it for the biggest use case, reloading. However it is good for general browsing as
well. Doing this in WebCore level as opposed to relying on disk cache has big benefit that
we avoid re-decoding resources, especially images.


To be exact the enabled case is not actually the "Specific end-to-end revalidation" since it does not include
CacheControl: max-age=0 header. That would be added in reload case.


The approach for revalidation is to kick the original resource out from the memory cache
and create a new CachedResource that represents the revalidation request. In case
we get 304 back for the request we put the original resource back to the cache, update
its expiration date and switch the clients registered to revalidation resource to be
clients of the original resource.


All heap allocated CachedImage pointers now use CachedResourceHandle<CachedImage> (and so on) instead.
This allows updating the handles to point to the original resource when the revalidation succeeds. It
also acts as refcounting smart pointer.

  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSFontFaceSource.h:
  • css/CSSImportRule.h:
  • dom/Clipboard.h: (WebCore::Clipboard::dragImage):
  • dom/ProcessingInstruction.h:
  • dom/ScriptElement.h:
  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::isWaitingForScripts):
  • dom/XMLTokenizer.h:
  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::setImage): (WebCore::HTMLImageLoader::updateFromElement): (WebCore::HTMLImageLoader::notifyFinished):
  • html/HTMLImageLoader.h: (WebCore::HTMLImageLoader::image):
  • html/HTMLLinkElement.h:
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::reset): (WebCore::HTMLTokenizer::scriptHandler): (WebCore::HTMLTokenizer::notifyFinished):
  • html/HTMLTokenizer.h:
  • loader/Cache.cpp: (WebCore::Cache::revalidateResource): (WebCore::Cache::revalidationSucceeded): (WebCore::Cache::revalidationFailed):
  • loader/Cache.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::CachedResource): (WebCore::CachedResource::~CachedResource): (WebCore::CachedResource::isExpired): (WebCore::CachedResource::setResponse): (WebCore::CachedResource::deleteIfPossible): (WebCore::CachedResource::setResourceToRevalidate): (WebCore::CachedResource::clearResourceToRevalidate): (WebCore::CachedResource::switchClientsToRevalidatedResource): (WebCore::CachedResource::canUseCacheValidator): (WebCore::CachedResource::mustRevalidate):
  • loader/CachedResource.h: (WebCore::CachedResource::canDelete): (WebCore::CachedResource::registerHandle): (WebCore::CachedResource::unregisterHandle): (WebCore::CachedResource::isCacheValidator): (WebCore::CachedResource::resourceToRevalidate): (WebCore::CachedResource::setExpirationDate):
  • loader/CachedResourceHandle.cpp: Added. (WebCore::CachedResourceHandleBase::setResource):
  • loader/CachedResourceHandle.h: Added. (WebCore::CachedResourceHandleBase::~CachedResourceHandleBase): (WebCore::CachedResourceHandleBase::get): (WebCore::CachedResourceHandleBase::operator!): (WebCore::CachedResourceHandleBase::operator UnspecifiedBoolType): (WebCore::CachedResourceHandleBase::CachedResourceHandleBase): (WebCore::CachedResourceHandleBase::operator=): (WebCore::CachedResourceHandle::CachedResourceHandle): (WebCore::CachedResourceHandle::get): (WebCore::CachedResourceHandle::operator->): (WebCore::CachedResourceHandle::operator=): (WebCore::CachedResourceHandle::operator==): (WebCore::CachedResourceHandle::operator!=): (WebCore::operator==): (WebCore::operator!=):
  • loader/DocLoader.cpp: (WebCore::DocLoader::checkForReload):
  • loader/UserStyleSheetLoader.h:
  • loader/loader.cpp: (WebCore::Loader::Host::servePendingRequests): (WebCore::Loader::Host::didFinishLoading): (WebCore::Loader::Host::didFail): (WebCore::Loader::Host::didReceiveResponse): (WebCore::Loader::Host::didReceiveData):
  • page/EventHandler.cpp: (WebCore::EventHandler::selectCursor):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::setCachedImage): (WebCore::RenderImage::imageChanged):
  • rendering/RenderImage.h: (WebCore::RenderImage::cachedImage): (WebCore::RenderImage::imagePtr):
  • rendering/style/RenderStyle.h:
  • rendering/style/StyleCachedImage.h: (WebCore::StyleCachedImage::data): (WebCore::StyleCachedImage::cachedImage):
  • svg/SVGFEImageElement.h:
  • svg/graphics/filters/SVGFEImage.h:
  • xml/XSLImportRule.h:

Sep 4, 2008:

9:58 PM Changeset in webkit [36108] by beidson@apple.com
  • 3 edits
    2 adds in trunk/WebCore

2008-09-04 Brady Eidson <​beidson@apple.com>

Reviewed by Mitz

<rdar://problem/6180236> - Safari times out connections after 1 or 2 minutes

A 60-second default timeout was added in ​http://trac.webkit.org/changeset/17144 in an attempt
to model default NSURLRequest behavior in a cross-platform manner.

Sadly by always enforcing this 60 second timeout, WebCore was stomping over the wishes of any Webkit
client that wished to enforce a much larger default timeout using NSURLRequest API.

Additionally, upon reviewing what all other browsers do, it seems apparent that "no limit" is desirable
behavior on the web and this restores previous Safari/WebKit behavior.

It would be easy to write a layout test for this, but to be effective it would have
to run for at least 61 seconds, which seems insane until will can parallelize run-webkit-tests

  • manual-tests/timeout-test.html: Added.
  • manual-tests/timeout-test.php: Added.
  • platform/network/ResourceRequestBase.h: (WebCore::ResourceRequestBase::ResourceRequestBase): Rename the constant to "unspecifiedTimeoutInterval" and make it UINT_MAX so platforms that do set it have an effective "no timeout." (Windows, for example)
  • platform/network/mac/ResourceRequestMac.mm: (WebCore::ResourceRequest::doUpdatePlatformRequest): If the timeout for this request is "unspecifiedTimeoutInterval", then don't bother setting the timeout using NSURLRequest API, allowing WebKit applications to enforce their own default timeout.
9:26 PM Changeset in webkit [36107] by mitz@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

Reviewed by Beth Dakin.

  • fix <rdar://problem/6198514> Changing a button's opacity triggers relayout

Test: fast/repaint/button-spurious-layout-hint.html

  • rendering/RenderButton.cpp: (WebCore::RenderButton::setStyle): Reset the inner block's style box flex to 0 to avoid getting a spurious layout hint.

LayoutTests:

Reviewed by Beth Dakin.

  • pixel test for <rdar://problem/6198514> Changing a button's opacity triggers relayout
  • fast/repaint/button-spurious-layout-hint.html: Added.
  • platform/mac/fast/repaint/button-spurious-layout-hint-expected.checksum: Added.
  • platform/mac/fast/repaint/button-spurious-layout-hint-expected.png: Added.
  • platform/mac/fast/repaint/button-spurious-layout-hint-expected.txt: Added.
8:11 PM Changeset in webkit [36106] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

2008-09-04 Gavin Barraclough <​barraclough@apple.com>

Reviewed by Geoffrey Garen.


Fixed an off-by-one error that would cause the StructureIDChain to
be one object too short.


Can't construct a test case because other factors make this not crash
(yet!).

  • kjs/StructureID.cpp: (KJS::StructureIDChain::StructureIDChain):
6:26 PM Changeset in webkit [36105] by cwzwarich@webkit.org
  • 2 edits in trunk/SunSpider

2008-09-04 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Reviewed by Maciej Stachowiak.

Add a '--args' option to SunSpider, so that we can pass '-j' to TraceMonkey
and '--expose-gc' to V8. Also, rename the '--v8' option to '--v8-suite' and
correct the punctuation in its documentation.

  • sunspider:
4:53 PM Changeset in webkit [36104] by kevino@webkit.org
  • 10 edits
    2 adds in trunk

wx build fixes.

4:27 PM Changeset in webkit [36103] by mitz@apple.com
  • 3 edits
    2 deletes in trunk

WebCore:

Reviewed by Mark Rowe.

  • css/CSSGrammar.y:

LayoutTests:

  • css1/color_and_background/invalid_color-expected.txt: Removed.
  • css1/color_and_background/invalid_color.html: Removed.
2:44 PM Changeset in webkit [36102] by mrowe@apple.com
  • 10 edits in trunk

Fix ​https://bugs.webkit.org/show_bug.cgi?id=20639.
Bug 20639: ENABLE_DASHBOARD_SUPPORT does not need to be a FEATURE_DEFINE

Reviewed by Eric Seidel.

2:25 PM Changeset in webkit [36101] by adele@apple.com
  • 4 edits in trunk/JavaScriptCore

2008-09-04 Adele Peterson <​adele@apple.com>

Build fix.

2:00 PM Changeset in webkit [36100] by mrowe@apple.com
  • 4 edits in trunk

Mac build fix.

1:44 PM Changeset in webkit [36099] by eric@webkit.org
  • 2 edits in trunk/WebCore

Build fix only, no review.

  • dom/XMLTokenizer.cpp: Fix the Chromium merge build by adding a missing header (the Mac files must include it somewhere).
1:26 PM Changeset in webkit [36098] by jmalonzo@webkit.org
  • 6 edits in trunk

2008-09-04 Marco Barisione <​marco.barisione@collabora.co.uk>

Reviewed by Eric Seidel.

​http://bugs.webkit.org/show_bug.cgi?id=20380
[GTK][AUTOTOOLS] Include autotoolsconfig.h from config.h

  • kjs/config.h: Include the configuration header generated by autotools if available.

2008-09-04 Marco Barisione <​marco.barisione@collabora.co.uk>

Reviewed by Eric Seidel.

​http://bugs.webkit.org/show_bug.cgi?id=20380
[GTK][AUTOTOOLS] Include autotoolsconfig.h from config.h

  • configure.ac: Rename the configuration header from aconfig.h to autotoolsconfig.h.

2008-09-04 Marco Barisione <​marco.barisione@collabora.co.uk>

Reviewed by Eric Seidel.

​http://bugs.webkit.org/show_bug.cgi?id=20380
[GTK][AUTOTOOLS] Include autotoolsconfig.h from config.h

  • config.h: Include the configuration header generated by autotools if available.
1:16 PM Changeset in webkit [36097] by mrowe@apple.com
  • 6 edits in trunk

Presto change-o!

11:15 AM Changeset in webkit [36096] by mitz@apple.com
  • 7 edits in trunk/WebCore

Rubber-stamped by Dave Hyatt.

  • rename CachedResource::allReferencesRemoved() to allClientsRemoved()
  • loader/CachedFont.cpp: (WebCore::CachedFont::allClientsRemoved):
  • loader/CachedFont.h:
  • loader/CachedImage.cpp: (WebCore::CachedImage::allClientsRemoved):
  • loader/CachedImage.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::removeClient):
  • loader/CachedResource.h: (WebCore::CachedResource::allClientsRemoved):
11:08 AM Changeset in webkit [36095] by cwzwarich@webkit.org
  • 4 edits
    6 adds in trunk

2008-09-04 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Bug 20616: Incorporate V8 benchmarks in testing
<​https://bugs.webkit.org/show_bug.cgi?id=20616>

This is a first attempt at running the V8 benchmarks within SunSpider,
with the --v8 option. The number of iterations is fixed for each
benchmark, and was chosen to make each of the benchmarks run between
500 ms and 600 ms in V8 on my machine, a 2.16 GHz MacBook Pro.

SunSpider:

  • sunspider:
  • tests/LIST-V8: Added.
  • tests/v8-crypto.js: Added.
  • tests/v8-deltablue.js: Added.
  • tests/v8-earley-boyer.js: Added.
  • tests/v8-raytrace.js: Added.
  • tests/v8-richards.js: Added.

WebKitTools:

  • Scripts/run-sunspider:
10:51 AM Changeset in webkit [36094] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

wx buildbot fix. More robust handling of clean when makefiles are not built or are not completely made.

9:10 AM Changeset in webkit [36093] by Adam Roben
  • 2 edits in trunk/WebKit/win

Make JavaScriptCoreGenerated build first instead of WTF

JavaScriptCoreGenerated does some setup work that we want to happen
before WTF builds.

  • WebKit.vcproj/WebKit.sln: Reversed the dependencies of WTF and JavaScriptCoreGenerated.
9:10 AM Changeset in webkit [36092] by Adam Roben
  • 2 edits in trunk/WebKitLibraries

Ignore warning LNK4221 on Windows

This warning is emitted when an object file with no public symbols is
passed to the linker/librarian. This often occurs in WebCore for files
that have been disabled via ENABLE()/USE() macros.

Rubberstamped by Anders Carlsson.

  • win/tools/vsprops/common.vsprops: Ignore warning LNK4221.
9:09 AM Changeset in webkit [36091] by Adam Roben
  • 4 edits in trunk/WebCore

Windows build fix after r36071

We were getting these errors:

error C2356: initialization segment must not change during translation
unit

This was happening because multiple files #included by
DerivedSources.cpp were themselves #including StaticConstructors.h. I
fixed the error by adding header guards to StaticConstructors.h so its
contents will only be included once.

But it's also not a good idea for StaticConstructors.h to end up in
DerivedSources.cpp, since it ends up "polluting" all the source files
we have in there. So I removed all the files that include
StaticConstructors.h and added some preprocessor directives to
DerivedSources.cpp to catch this error in the future.

  • DerivedSources.cpp: Removed the *Names.cpp files, which include StaticConstructors.h, and added some preprocessor directives to make sure we don't end up accidentally including StaticConstructors.h in the future.
  • WebCore.vcproj/WebCore.vcproj: Added the *Names.cpp files.
  • platform/StaticConstructors.h: Added header guards.
9:09 AM Changeset in webkit [36090] by Adam Roben
  • 3 edits in trunk/WebCore

Windows build fix

  • platform/graphics/win/FontPlatformData.h: Added a missing #include of PassRefPtr.h, and corrected the capitalization of RefCounted.h.
  • platform/text/PlatformString.h: Added a missing #include of PassRefPtr.h.
7:24 AM Changeset in webkit [36089] by vestbo@webkit.org
  • 9 edits in trunk

2008-09-04 Tor Arne Vestbø <​tavestbo@trolltech.com>

Reviewed by Simon

Fix the QtWebKit build to match changes in r36016

4:44 AM Changeset in webkit [36088] by vestbo@webkit.org
  • 7 edits
    2 moves in trunk

2008-09-04 Tor Arne Vestbø <​tavestbo@trolltech.com>

Reviewed by Simon.

Re-enable support for user stylesheets in QtWebKit

QtWebKit now follows the FRAME_LOADS_USER_STYLESHEET
code path, which allows us to keep API support for
loading user style sheets from remote URLs.

As part of the change UserStyleSheetLoader.cpp/h was
moved from WebCore/loader/mac to WebCore/loader.

2:29 AM Changeset in webkit [36087] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Fix the 64-bit build.

  • VM/CodeBlock.cpp:

(KJS::CodeBlock::printStructureID): Store the instruction offset into an unsigned local
to avoid a warning related to format specifiers.
(KJS::CodeBlock::printStructureIDs): Ditto.

2:14 AM Changeset in webkit [36086] by alp@webkit.org
  • 2 edits in trunk/WebCore

2008-09-04 Alp Toker <​alp@nuanti.com>

Reviewed by Eric.

Remove left-over QT and CAIRO platform checks.

  • html/CanvasRenderingContext2D.cpp:
1:58 AM Changeset in webkit [36085] by mitz@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Fix typo

1:39 AM Changeset in webkit [36084] by eric@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Mark Rowe.

Fix leak of TextMetrix due to over-ref as see on buildbot.

  • html/TextMetrics.h: use adoptRef since RefCounted starts @ refcount 1 instead of 0 now.
1:00 AM Changeset in webkit [36083] by cwzwarich@webkit.org
  • 3 edits in trunk/JavaScriptCore

2008-09-04 Cameron Zwarich <​cwzwarich@uwaterloo.ca>

Rubber-stamped by Oliver Hunt.

Correct the spelling of 'entryIndices'.

  • kjs/PropertyMap.cpp: (KJS::PropertyMap::get): (KJS::PropertyMap::getLocation): (KJS::PropertyMap::put): (KJS::PropertyMap::insert): (KJS::PropertyMap::remove): (KJS::PropertyMap::checkConsistency):
  • kjs/PropertyMap.h: (KJS::PropertyMapHashTable::entries): (KJS::PropertyMap::getOffset): (KJS::PropertyMap::putOffset): (KJS::PropertyMap::offsetForTableLocation):
12:26 AM Changeset in webkit [36082] by mitz@apple.com
  • 5 edits
    4 adds in trunk

WebCore:

Reviewed by Dave Hyatt.

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutOnlyPositionedObjects): In the positioned movement only case, call tryLayoutDoingPositionedMovementOnly() and fall back on doing a full layout if that fails. (WebCore::RenderBlock::layoutPositionedObjects): Ditto.
  • rendering/RenderBox.h: (WebCore::RenderBox::tryLayoutDoingPositionedMovementOnly): Renamed layoutDoingPositionedMovementOnly to this, and made this function check if the width changed. If it did, return, leaving the object dirty. The caller can then call layout(). The width can change even in the "positioned movement only" case if the object is shrink-to-fit and the available width constraint is met. (This was the case in the bug).
  • rendering/RenderObject.h: (WebCore::RenderObject::tryLayoutDoingPositionedMovementOnly): Renamed layoutDoingPositionedMovementOnly() to this.

LayoutTests:

Reviewed by Dave Hyatt.

  • fast/block/positioning/move-with-auto-width.html: Added.
  • platform/mac/fast/block/positioning/move-with-auto-width-expected.checksum: Added.
  • platform/mac/fast/block/positioning/move-with-auto-width-expected.png: Added.
  • platform/mac/fast/block/positioning/move-with-auto-width-expected.txt: Added.
12:21 AM Changeset in webkit [36081] by ggaren@apple.com
  • 4 edits
    2 adds in trunk

JavaScriptCore:

2008-09-03 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Cameron Zwarich.


Fixed <rdar://problem/6193925> REGRESSION: Crash occurs at
KJS::Machine::privateExecute() when attempting to load my Mobile Gallery
(​http://www.me.com/gallery/#home)


also


​https://bugs.webkit.org/show_bug.cgi?id=20633 Crash in privateExecute
@ cs.byu.edu


The underlying problem was that we would cache prototype properties
even if the prototype was a dictionary.


The fix is to transition a prototype back from dictionary to normal
status when an opcode caches access to it. (This is better than just
refusing to cache, since a heavily accessed prototype is almost
certainly not a true dictionary.)

  • VM/Machine.cpp: (KJS::Machine::tryCacheGetByID):
  • kjs/JSObject.h:

LayoutTests:

2008-09-04 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Cameron Zwarich.


Test for <rdar://problem/6193925> REGRESSION: Crash occurs at
KJS::Machine::privateExecute() when attempting to load my Mobile Gallery
(​http://www.me.com/gallery/#home)


also


​https://bugs.webkit.org/show_bug.cgi?id=20633 Crash in privateExecute
@ cs.byu.edu

  • fast/js/pic/dictionary-prototype-expected.txt: Added.
  • fast/js/pic/dictionary-prototype.html: Added.
Note: See TracTimeline for information about the timeline view.