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

Timeline



Dec 26, 2008:

1:31 PM Changeset in webkit [39485] by ap@webkit.org
  • 2 edits in trunk/LayoutTests

2008-12-26 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

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

Remove ensureTrailingSlash from appcache test.

  • http/tests/appcache/resources/fail-on-update.php:
1:19 PM Changeset in webkit [39484] by Antti Koivisto
  • 2 edits in trunk/WebCore

2008-12-26 Zalan Bujtas <zbujtas@gmail.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=22999
Check if database thread exists.

  • storage/Database.cpp: (WebCore::Database::openAndVerifyVersion): (WebCore::Database::markAsDeletedAndClose): (WebCore::Database::tableNames):
1:01 PM Changeset in webkit [39483] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

2008-12-26 Sam Weinig <sam@webkit.org>

Reviewed by Alexey Proskuryakov.

Remove unused method.

  • runtime/Structure.h: Remove mutableTypeInfo.
12:00 PM Changeset in webkit [39482] by ap@webkit.org
  • 6 edits in trunk/LayoutTests

2008-12-26 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

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

Attempt to fix the failiing XHR tests on tiger buildbot.
Remove the no longer needed ensureTrailingSlash function.

  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-invalidation.php:
  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache-timeout.php:
  • http/tests/xmlhttprequest/resources/access-control-basic-preflight-cache.php:
  • http/tests/xmlhttprequest/resources/portabilityLayer.php: Make sys_get_temp_dir() return "/tmp" when it can't find the temp directory through other means.
  • http/tests/xmlhttprequest/resources/reset-temp-file.php:
11:54 AM Changeset in webkit [39481] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Sam Weinig.

  • fix <rdar://problem/6467608> lastVisitWasFailure flag persists in global history after a successful visit
  • History/WebHistory.mm: (-[WebHistory _visitedURL:withTitle:wasFailure:]): Changed to always update the wasFailure flag on the HistoryItem.
11:05 AM Changeset in webkit [39480] by ap@webkit.org
  • 2 edits in trunk/LayoutTests

Reviewed by Anders Carlsson.

Fix a layout test that was broken by the previous appcache commit
(it only passed every other time as a result).

  • http/tests/appcache/fail-on-update.html:
10:59 AM Changeset in webkit [39479] by Simon Fraser
  • 5 edits in trunk/LayoutTests

2008-12-26 Simon Fraser <Simon Fraser>

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

Update more svg-as-image test results affected by the fix
for SVGImage drawing.

  • platform/mac/fast/borders/svg-as-border-image-2-expected.checksum:
  • platform/mac/fast/borders/svg-as-border-image-2-expected.png:
  • platform/mac/fast/borders/svg-as-border-image-expected.checksum:
  • platform/mac/fast/borders/svg-as-border-image-expected.png:
10:52 AM Changeset in webkit [39478] by ap@webkit.org
  • 3 edits
    3 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=23001
A call to applicationCache.update() from a cached event listener should be ignored

Test: http/tests/appcache/idempotent-update.html

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::checkIfLoadIsComplete): Only reset m_status after dispatching events.
9:40 AM Changeset in webkit [39477] by ap@webkit.org
  • 4 edits
    16 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=22997
ASSERTION FAILED: !m_resources.contains(url) in ApplicationCache::addResource()

Tests: http/tests/appcache/top-frame-1.html

http/tests/appcache/top-frame-2.html
http/tests/appcache/top-frame-3.html
http/tests/appcache/top-frame-4.html

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::cacheForMainRequest): Fixed one more instance of top level frame being used for caching purposes.
3:03 AM Changeset in webkit [39476] by ap@webkit.org
  • 3 edits in trunk/LayoutTests

2008-12-26 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=22950
Added more logging to reset-temp-file to help expose potential issues that could cause test failures (on the tiger buildbot).

If there are issues in reset-temp-file, they are printed from the test access-control-basic-allow-preflight-cache.html

  • http/tests/xmlhttprequest/access-control-basic-allow-preflight-cache.html:
  • http/tests/xmlhttprequest/resources/reset-temp-file.php:
12:46 AM Changeset in webkit [39475] by ap@webkit.org
  • 2 edits in trunk/WebCore

2008-12-26 Dmitry Titov <dimich@chromium.org>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=22987
Fix for broken test editing/execCommand/5763082.html

  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::mergeIdenticalElements): Used local RefPtr to hold onto function parameter across multiple calls.

Dec 25, 2008:

1:12 PM Changeset in webkit [39474] by ap@webkit.org
  • 7 edits
    5 adds in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=22993
Application cache shouldn't be inherited by subframes

<rdar://problem/6284708> AppCache crashes in ApplicationCacheResource::addType()
This crash happened because main resource for subframe was looked up in top frame's
appcache. If not for the spec change, it could have been fixed by preventing the load
altogether.

Test: http/tests/appcache/foreign-iframe-main.html
I intend to write additional tests for the behavior change.

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::shouldLoadResourceFromApplicationCache):
  • loader/DocumentLoader.h: (WebCore::DocumentLoader::applicationCache):
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::load):
  • loader/appcache/DOMApplicationCache.cpp: (WebCore::DOMApplicationCache::associatedCache): Removed DocumentLoader::topLevelApplicationCache(), and changed callers accordingly.
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::selectCache): (WebCore::ApplicationCacheGroup::selectCacheWithoutManifestURL): Removed checks for the frame being top-level one, now that subframes are cached independently.
6:18 AM Changeset in webkit [39473] by Antti Koivisto
  • 6 edits in trunk/WebCore

2008-12-25 Antti Koivisto <Antti Koivisto>

Reviewed by Oliver Hunt.

<rdar://problem/6465669> Frequent !isPurgeable() assertion in WebCore::CachedResource::addClient


Disallow turning resources that are being revalidated to purgable state.


No test, the condition is difficult to produce in DRT.

  • loader/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::allClientsRemoved):
  • loader/CachedImage.cpp: (WebCore::CachedImage::destroyDecodedData):
  • loader/CachedResource.cpp: (WebCore::CachedResource::isSafeToMakePurgeable): (WebCore::CachedResource::makePurgeable):
  • loader/CachedResource.h:
  • loader/CachedScript.cpp: (WebCore::CachedScript::destroyDecodedData):
2:55 AM Changeset in webkit [39472] by ap@webkit.org
  • 2 edits in trunk/WebCore

Qt build fix.

  • WebCore.pro: Added ThreadGlobalData.cpp.
Note: See TracTimeline for information about the timeline view.