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

Timeline



May 26, 2007:

11:09 PM Changeset in webkit [21819] by hyatt
  • 5 edits
    6 adds in trunk

Fix for radar bug 5219562, PNG isn't displayed on SVG test suite.
Improve <object> error handling so that it recovers properly when
<td> is used inside it.

Reviewed by olliej

Added fast/invalid/td-inside-object.html

  • html/HTMLObjectElement.h: (WebCore::HTMLObjectElement::tagPriority):
  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleError):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate):
  • html/HTMLParserErrorCodes.h:
10:01 PM Changeset in webkit [21818] by hyatt
  • 2 edits in trunk/WebCore

Don't compare the "isset" variables in background layers, since they
lead us to turn off the Aqua look incorrectly, even when the background
matches the default.

Reviewed by olliej

  • rendering/RenderStyle.cpp: (WebCore::BackgroundLayer::operator==):
9:56 PM Changeset in webkit [21817] by hyatt
  • 3 edits in trunk/WebCore

Improve the </p> -> <p></p> error message to not spew two errors.
Fix a bug where errors reported inside document.write didn't get
reported as such because of a mistake in the notation of SegmentedString.

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::processCloseTag):
  • platform/SegmentedString.h: (WebCore::SegmentedSubstring::clear):
9:53 PM Changeset in webkit [21816] by hyatt
  • 4 adds in trunk/LayoutTests/fast/block/positioning

Add layout test for bug 13887.

9:52 PM Changeset in webkit [21815] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/WebKit/wx/WebKitSupport/EditorClientWx.cpp

Make sure the function returns a value.

9:51 PM Changeset in webkit [21814] by hyatt
  • 2 edits in trunk/WebCore

Fix for bug 13887, percentage height block not sized correctly inside
positioned percentage height containing block.

Reviewed by mitz

fast/block/positioning/complex-percentage-height.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcPercentageHeight):
8:45 PM Changeset in webkit [21813] by kevino
  • 16 adds in branches/wx-port-alpha/trunk/WebCore

More additions, hopefully the last batch.

8:34 PM Changeset in webkit [21812] by kevino
  • 3 adds in branches/wx-port-alpha/trunk/WebCore/rendering

More files the merge didn't mark as added.

8:31 PM Changeset in webkit [21811] by kevino
  • 1 add in branches/wx-port-alpha/trunk/WebCore/rendering/LayoutState.h

Another merge file that wasn't marked for addition to the branch

8:27 PM Changeset in webkit [21810] by kevino
  • 3 adds in branches/wx-port-alpha/trunk/WebCore/html

The merge didn't properly mark these files as added on the branch, so add them ourselves.

7:35 PM Changeset in webkit [21809] by kevino
  • 732 edits
    2 copies
    35 deletes in branches/wx-port-alpha/trunk

Sync wx port with TOT, and fix a bug with iconv detection in install-unix-extras.

4:09 PM Changeset in webkit [21808] by andersca
  • 3 edits in trunk/WebCore

Reviewed by Oliver Hunt.

Covered by fast/forms/box-shadow-override.html

I forgot to include adjustSliderThumbStyle in the previous patch and the
test didn't detect it because of a bug introduced in r21739.

  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::adjustSliderThumbStyle):
3:09 PM Changeset in webkit [21807] by weinig
  • 11 edits in trunk

LayoutTests:

Reviewed by Hyatt.

  • fast/repaint/box-shadow-h-expected.checksum:
  • fast/repaint/box-shadow-h-expected.png:
  • fast/repaint/box-shadow-h-expected.txt:
  • fast/repaint/box-shadow-h.html:
  • fast/repaint/box-shadow-v-expected.checksum:
  • fast/repaint/box-shadow-v-expected.png:
  • fast/repaint/box-shadow-v-expected.txt:
  • fast/repaint/box-shadow-v.html:

WebCore:

Reviewed by Hyatt.

Tests: fast/repaint/box-shadow-h.html

fast/repaint/box-shadow-v.html

  • rendering/RenderTable.cpp: (WebCore::RenderTable::layout):
2:57 PM Changeset in webkit [21806] by weinig
  • 5 edits in trunk

JavaScriptCore:

Reviewed by Darin.

Patch for ​http://bugs.webkit.org/show_bug.cgi?id=13854
Port of commit 667785 from kjs

  • special case calling String.localeCompare() with no parameters to return 0.
  • kjs/string_object.cpp: (KJS::StringProtoFunc::callAsFunction):

LayoutTests:

Reviewed by Darin.

Update tests ​http://bugs.webkit.org/show_bug.cgi?id=13854
Port of commit 667785 from kjs

  • update tests and results to match kde.
  • fast/js/kde/StringObject-expected.txt:
  • fast/js/kde/resources/StringObject.js:
10:38 AM Changeset in webkit [21805] by bdash
  • 4 edits in trunk

Versioning.

10:26 AM Changeset in webkit [21804] by bdash
  • 1 copy in tags/Safari-5522.8

New tag.

8:36 AM Changeset in webkit [21803] by ap
  • 3 edits
    2 adds in trunk

Reviewed by Darin.

​http://bugs.webkit.org/show_bug.cgi?id=13605
REGRESSION: Browser crashes when switching Kotoeri input modes

The problem could no longer be reproduced with Kotoeri thanks to
a fix in <​http://trac.webkit.org/projects/webkit/changeset/21728>,
but we still need to protect from invalid ranges passed from
input methods.

Test: editing/input/replace-invalid-range.html

  • page/mac/WebCoreFrameBridge.mm: (-[WebCoreFrameBridge selectNSRange:]): Added a null check.
8:26 AM Changeset in webkit [21802] by darin
  • 1 edit in trunk/WebKit/ChangeLog

"on this makefile itself"

8:24 AM Changeset in webkit [21801] by darin
  • 2 edits in trunk/WebKit

Reviewed by Tim Hatcher.

  • MigrateHeaders.make: Added dependency on this header file itself, which is useful when you change the sed command or other aspect of this file. My build failed until I made this fix.
7:40 AM Changeset in webkit [21800] by darin
  • 2 edits in trunk/WebCore
  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::doUpdateResourceResponse): Alexey pointed out that I was wrong about object lifetime, so rolling this change out.
7:32 AM Changeset in webkit [21799] by darin
  • 2 edits in trunk/WebCore

Reviewed by Anders.

  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::doUpdateResourceResponse): Fixed code where a temporary String is used after destroyed.
6:36 AM Changeset in webkit [21798] by bdash
  • 2 edits in trunk/WebCore

2007-05-26 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Not testable.

  • platform/graphics/Color.cpp: (WebCore::Color::light): (WebCore::Color::dark):
6:32 AM Changeset in webkit [21797] by bdash
  • 7 edits in trunk

2007-05-26 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Covered by fast/frames/inline-object-inside-frameset.html

  • rendering/RenderFrame.h: (WebCore::RenderFrame::isFrame): Added.
  • rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::paint): Removed check for inline objects that is no longer needed. (WebCore::RenderFrameSet::isChildAllowed): Added. Allows the child only if it is a frame or another frameset.
  • rendering/RenderFrameSet.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::isFrame): Added.

2007-05-26 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • fast/frames/inline-object-inside-frameset-expected.txt:
6:29 AM Changeset in webkit [21796] by bdash
  • 5 edits
    4 adds in trunk

2007-05-26 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

Test: fast/forms/box-shadow-override.html

  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::adjustCheckboxStyle): (WebCore::RenderTheme::adjustRadioStyle):
  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::adjustButtonStyle): (WebCore::RenderThemeMac::adjustMenuListStyle): (WebCore::RenderThemeMac::adjustSliderTrackStyle): (WebCore::RenderThemeMac::adjustSearchFieldStyle): (WebCore::RenderThemeMac::adjustSearchFieldCancelButtonStyle): (WebCore::RenderThemeMac::adjustSearchFieldDecorationStyle): (WebCore::RenderThemeMac::adjustSearchFieldResultsDecorationStyle): (WebCore::RenderThemeMac::adjustSearchFieldResultsButtonStyle):

2007-05-26 Mitz Pettel <​mitz@webkit.org>

Reviewed by Darin.

  • fast/forms/box-shadow-override-expected.checksum: Added.
  • fast/forms/box-shadow-override-expected.png: Added.
  • fast/forms/box-shadow-override-expected.txt: Added.
  • fast/forms/box-shadow-override.html: Added.
6:14 AM Changeset in webkit [21795] by bdash
  • 2 edits in trunk/WebCore

2007-05-26 Holger Freyther <​zecke@selfish.org>

Reviewed by Anders.

  • platform/gdk/FrameGdk.cpp: Catch up to make it compile (WebCore::FrameGdk::FrameGdk): The Settings is now owned by the Page
12:05 AM Changeset in webkit [23416] by oliver
  • 2 edits in branches/WindowsMerge/WebKitWin

2007-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Hyatt.


Fix rdar://problem/5212399 <object> with PNG src rendered as image document (causes Acid2 to fail)

  • WebFrame.cpp: (WebFrame::objectContentType):

May 25, 2007:

8:06 PM Changeset in webkit [21794] by weinig
  • 2 edits in trunk/WebCore

Reviewed by Oliver of the Hunt.

Fixed <rdar://problem/5230247> at least 43 xhtml tests fail in
Leopard because they are being interpreted as html

Adds a work around for <rdar://problem/5230154> which explicitly
sets local .xhtml files to have the MIME type application/xhtml+xml.

  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::doUpdateResourceResponse):
7:49 PM Changeset in webkit [23415] by aroben
  • 2 edits in branches/WindowsMerge/LayoutTests
  • win/Skipped: Corrected a filename and commented out a disabled test.
7:31 PM Changeset in webkit [23414] by aroben
  • 2 edits in branches/WindowsMerge/LayoutTests
  • win/Skipped: Removed tests that fail on Mac as well to keep the list Windows-specific.
7:20 PM Changeset in webkit [23413] by ggaren
  • 3 edits in branches/WindowsMerge/WebKitWin

Build fix -- forgot to commit a little bit in my last patch.

5:58 PM Changeset in webkit [23412] by aroben
  • 2 edits in branches/WindowsMerge/LayoutTests
  • win/Skipped: Finish adding all currently failing tests.
5:40 PM Changeset in webkit [23411] by ggaren
  • 11 edits in branches/WindowsMerge/WebKitWin

WebKitWin:

Reviewed by Darin Adler, Adam Roben.

Fixed <rdar://problem/5055182> The page cache has no global cap.


The main WebKit changes are:

  1. Gutted IWebBackForwardList::setPageCacheSize and IWebBackForwardList:: pageCacheSize because they're deprecated on Mac now. I didn't remove them, though, because doing so can cause COM badness.


  1. Added IWebPreferences::setUsesPageCache IWebPreferences:: usesPageCache because that's the new Mac API. I added them to the end of IWebPreferences to avoid COM badness.


  1. Centralized calculation of object cache and page cache sizes inside WebPreferences.


Changed our old behavior of reading a preference and applying a fudge
factor with a new behavior of just using the preference directly. The
old behavior was confusing and often inappropriate. (For example, if
you set a page cache size of 100, a 256MB machine would somewhat
arbitrarily reduce that number to 98.)

5:28 PM Changeset in webkit [21793] by ggaren
  • 31 edits in trunk

WebCore:

Reviewed by Darin Adler.

Fixed <rdar://problem/5055182> The page cache has no global cap.


Beefed up the PageCache class to track which HistoryItems are
cached, and decide how to evict old HistoryItems as new ones are
added.

  • history/BackForwardList.cpp: Removed support for managing the page cache through the BackForwardList since this is deprecated in Mac's WebBackForwardList, and doesn't exist on other platforms.


Added a Page* back pointer so that Mac WebBackForwardList can still support
its deprecated API, using the new hotness. This isn't a big burden, since
Page is responsible for calling BackForwardList::close upon destruction,
anyway.

  • history/HistoryItem.cpp: Replaced CachedPage* with boolean that tells you whether you're in the page cache. This prevents us from having more than one persistent repository for cached pages.
  • history/PageCache.cpp: If you're playing along at home, this code is the reason for the patch.
  • loader/FrameLoader.cpp: If you're playing along at home, the minuses in this file are the reason for the patch.


Replaced call to FrameLoaderClient::loadedFromCachedPage with its one-line
implementation because WebCore calling itself through WebKit is uncivilized.

  • page/Settings.cpp: Added support for removing a Page's HistoryItems from the page cache.

WebKit:

Reviewed by Darin Adler.

Fixed <rdar://problem/5055182> The page cache has no global cap.


The main WebKit changes are:

  1. Changed -[WebBackForwardList setPageCacheSize] and -[WebBackForwardList pageCacheSize] to accomodate the new global page cache model, updating their documentation.


  1. Added -[WebPreferences setShouldUsePageCache] and -[WebPreferences shouldUsePageCache] as pending public API.


  1. Centralized calculation of object cache and page cache sizes inside WebPreferences.


Cchanged our old behavior of reading a preference and applying a fudge
factor with a new behavior of just using the preference directly. The
old behavior was confusing and often inappropriate. (For example, if
you set a page cache size of 100, a 256MB machine would somewhat
arbitrarily reduce that number to 98. ???)

  • WebView/WebView.mm: Added support for two flags to determine whether to use the page cache. If either -[WebBackForwardList setPageCacheSize:0] or -[WebPreferences setShouldUsePageCache:NO] is called, we don't use the page cache.
5:23 PM Changeset in webkit [23410] by aroben
  • 2 edits in branches/WindowsMerge/LayoutTests
  • win/Skipped: Added a bunch more failing tests.
4:56 PM Changeset in webkit [21792] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/WebCore/platform/wx/SharedTimerWx.cpp

Never fire the timer immediately, even with a 0 second delay, in order to give the calling function time to complete.

4:53 PM Changeset in webkit [21791] by andersca
  • 1 edit in trunk/WebCore/ChangeLog

Reviewed by Darin.

  • fix another regression from r21739: ::-webkit-slider-thumb style not applied.

Test: fast/forms/slider-thumb-stylability.html

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::setStyle): (WebCore::RenderSlider::createThumbStyle):
4:50 PM Changeset in webkit [21790] by darin
  • 3 edits
    3 adds in trunk

JavaScriptCore:

Reviewed by Darin.

  • Add a explicit check for arguments. Previously check was done with getDirect, but since the arguments is created on-demand in ActivationImp, it doesn't show up in the test. 'arguments' should always be in the VarDeclNode's evaluation scope.
  • kjs/nodes.cpp: (VarDeclNode::evaluate): Additional check if the var decl identifier is 'arguments'

LayoutTests:

Reviewed by Darin.

  • fast/js/resources/vardecl-preserve-arguments.js: Added.
  • fast/js/vardecl-preserve-arguments-expected.txt: Added.
  • fast/js/vardecl-preserve-arguments.html: Added.
4:45 PM Changeset in webkit [21789] by darin
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

Added a test that ::-webkit-slider-thumb style is actually applied.

  • fast/forms/slider-thumb-stylability-expected.checksum: Added.
  • fast/forms/slider-thumb-stylability-expected.png: Added.
  • fast/forms/slider-thumb-stylability-expected.txt: Added.
  • fast/forms/slider-thumb-stylability.html: Added.

WebCore:

Reviewed by Darin.

  • fix another regression from r21739: ::-webkit-slider-thumb style not applied.

Test: fast/forms/slider-thumb-stylability.html

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::setStyle): (WebCore::RenderSlider::createThumbStyle):
4:33 PM Changeset in webkit [21788] by mjs
  • 1 edit
    3 moves in trunk/LayoutTests

Reviewed by Geoff.


  • fix layout test that is failing due to hardcoded file paths in the results by moving it to the http tests
  • fast/history/redirect-load-no-form-restoration-expected.txt: Removed.
  • fast/history/redirect-load-no-form-restoration.html: Removed.
  • fast/history/resources/redirect-load.html: Removed.
  • http/tests/navigation/redirect-load-no-form-restoration-expected.txt: Added.
  • http/tests/redirect-load-no-form-restoration.html: Removed.
  • http/tests/resources/redirect-load.html: Removed.
4:27 PM Changeset in webkit [21787] by kmccullo
  • 2 edits in trunk

LayoutTests:

Reviewed by .

  • updating previous comment.
  • ChangeLog:

WebCore:

Reviewed by.

  • Updating previous comment.
  • ChangeLog:
4:21 PM Changeset in webkit [21786] by justing
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Harrison.


<rdar://problem/5228141>
REGRESSION: Can't apply style to selection that ends after an image

  • editing/style/5228141-expected.checksum: Added.
  • editing/style/5228141-expected.png: Added.
  • editing/style/5228141-expected.txt: Added.
  • editing/style/5228141.html: Added.

WebCore:

Reviewed by Harrison.

<rdar://problem/5228141>
REGRESSION: Can't apply style to selection that ends after an image

  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyInlineStyle): Use range compliant equivalents when creating the Range used to compute pastEndNode(), otherwise Range's constructor ASSERTs and pastEndNode() returns null. Added a testcase. If the incoming Range is collapsed at [br, 0], callers want the style to be applied to the br, so expand the Range to include it. We got away without expanding before because pastEndNode of a Range from [br, 0] to [br, 0] is the node after the br. Using range compliant equivalents changed that. There's already test coverage for this change. Changed while loop termination for finding the end of a run so that we don't process pastEnd. end.node() may now be non-atomic and we'd blow past it. There's already test coverage for this change.
4:02 PM Changeset in webkit [21785] by bdakin
  • 2 edits in trunk/WebCore

Rubber stamped by Hyatt

Adding an ASSERT for good measure.

  • page/FrameView.cpp: (WebCore::FrameView::layout):
3:59 PM Changeset in webkit [21784] by kmccullo
  • 4 edits
    8 copies
    2 adds in trunk

LayoutTests:

Reviewed by Geoff.

  • rdar://problem/5183523
  • Change load to check the URL as well as the referrer since it is possilbe to have a referrer which limits a load even thought the URL would allow it. There is a bug about how we need to re-evaluate the whole load architecture: <rdar://problem/5227992> Rethink canLoad() to have a consistent and comprehensive architecture.
  • These tests do not dumpAsText since DRT doesn't seem to be able to handle text in frames. If it is in the future then these tests already have sufficient text to change to that output and give correct results.
  • fast/dom/frame-loading-via-document-write-expected.txt: Added.
  • fast/dom/frame-loading-via-document-write.html: Added.
  • fast/dom/resources/abe.png: Added.
  • http/tests/security/frame-loading-via-document-write-expected.txt: Added.
  • http/tests/security/frame-loading-via-document-write.html: Added.
  • http/tests/security/resources/abe.png: Added.

WebCore:

Reviewed by Geoff.

  • rdar://problem/5183523
  • Change load to check the URL as well as the referrer since it is possilbe to have a referrer which limits a load even thought the URL would allow it. There is a bug about how we need to re-evaluate the whole load architecture: <rdar://problem/5227992> Rethink canLoad() to have a consistent and comprehensive architecture.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadSubframe): (WebCore::FrameLoader::load): (WebCore::FrameLoader::canLoad):
  • loader/FrameLoader.h:
3:45 PM Changeset in webkit [21783] by staikos
  • 2 edits in trunk/JavaScriptCore

2007-05-25 George Staikos <​staikos@kde.org>

Reviewed by Maciej.

  • Use COMPILER(GCC), not PLATFORM(GCC) - as Platform.h defines
  • wtf/FastMalloc.h:
3:43 PM Changeset in webkit [21782] by andersca
  • 5 edits in trunk/WebCore

Reviewed by Brady.

<rdar://problem/5209536>
​http://bugs.webkit.org/show_bug.cgi?id=13757
REGRESSION: Leaks seen on buildbot after r21497.


Fix multipart data loading so it works like Tiger Safari. When one part of a multipart connection has been loaded,
move the subresource loader from the subresource loaders set and into a special multipart subresource loader set. This set
is not considered when asking a document loader if it's currently loading.


  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::stopLoading): Always stop the multipart subresource loaders here.


(WebCore::DocumentLoader::subresourceLoaderFinishedLoadingOnePart):
New function that moves the loader from m_subresourceLoaders to m_multipartSubresourceLoaders and updates the document loader's state.


  • loader/ProgressTracker.cpp: (WebCore::ProgressTracker::incrementProgress): If a progress item already exists, reuse it. This can happen in some rare circumstances when dealing with multipart data.


  • loader/SubresourceLoader.cpp: (WebCore::SubresourceLoader::didReceiveResponse): Call subresourceLoaderFinishedLoadingOnePart if the response is multipart.
3:20 PM Changeset in webkit [21781] by thatcher
  • 2 edits in trunk/WebKit

Reviewed by Kevin Decker.

<rdar://problem/5219089> Changes for migration of DictionaryServices

  • WebView/WebHTMLView.mm: (-[WebHTMLView _lookUpInDictionaryFromMenu:]): look for the HIDictionaryWindowShow symbol in HIToolbox
2:52 PM Changeset in webkit [21780] by pyeh
  • 2 edits in trunk/WebCore

<rdar://problem/4918397> Exception raise inspecting note contents in the background

  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject accessibilityDescription]): Only allow asking the attachmentView for description if attachmentView supports this description attribute
2:51 PM Changeset in webkit [21779] by ggaren
  • 3 edits in trunk/WebKit

Reviewed by Oliver Hunt.


Fixed global initializer (like you fix a dog).


I'm not sure how our script missed this. I tested, and it generally doesn't
seem to work very well.

  • WebView/WebHTMLView.mm: Allocate lazily to avoid the performance hit of a global initializer. (promisedDataClient): (-[WebHTMLViewPrivate dealloc]): (-[WebHTMLViewPrivate clear]): (-[WebHTMLView setPromisedDragTIFFDataSource:WebCore::]):
2:44 PM Changeset in webkit [21778] by bdakin
  • 2 edits in trunk/WebCore

Reviewed by Oliver and Hyatt.

Fix for <rdar://problem/5228363> REGRESSION: In Mail, attempting to
create a To Do results in a crash

This regressed with r21635. Now that -boundingBox updates layout if
it is needed, we infinitely recurse when Mail draws a custom
highlight. The fix is to put a guard around laying out the render
tree.

  • page/FrameView.cpp: (WebCore::FrameViewPrivate::reset): (WebCore::FrameView::layout):
2:06 PM Changeset in webkit [23409] by kmccullo
  • 2 edits in branches/WindowsMerge/LayoutTests
  • win/Skipped: Updated the underlying bug for a test.
2:04 PM Changeset in webkit [21777] by pyeh
  • 2 edits in trunk/WebCore

<rdar://problem/5221986> selection in web areas doesn't seem to be working for text navigation

  • bridge/mac/WebCoreAXObject.mm: (-[WebCoreAXObject accessibilityIsAttributeSettable:]): Fixed up incorrect string comparison for AXSelectedTextMarkerRange ax attribute so this attribute can actually announce its settability.
1:37 PM Changeset in webkit [21776] by beidson
  • 2 edits in trunk/WebKit

Reviewed by Darin

<rdar://problem/5228371> - REGRESSION - Certain mail message bodies display as empty

This is due to ​http://trac.webkit.org/projects/webkit/changeset/21480 which unintentionally
made applewebdata urls result in check.call(false) instead of check.call(true)

Best place for a fix is to have the FrameLoaderClient::canHandleRequest() call return true, which
really is rooted in WebView

<rdar://problem/5229587> tracks adding a layout test

  • WebView/WebView.mm: (+[WebView _canHandleRequest:]): Return true for applewebdata URLs
1:24 PM Changeset in webkit [21775] by darin
  • 3 edits in trunk/LayoutTests
  • updated tests results that were affected by Hyatt's change to turn the odious <script/> quirk back on
  • fast/parser/script-tag-with-trailing-slash-expected.txt: Updated test result to reflect now-expected failure. It would be good some day to change the test to be more clear that this is working as designed.
  • fast/tokenizer/script_extra_close-expected.txt: Updated.
1:19 PM Changeset in webkit [21774] by darin
  • 1 edit
    4 deletes in trunk/LayoutTests
  • Hixie (who originally wrote this test) and Hyatt say that this test is invalid, and should be removed -- and it's failing too so gotta remove it
  • fast/block/basic/017-expected.checksum: Removed.
  • fast/block/basic/017-expected.png: Removed.
  • fast/block/basic/017-expected.txt: Removed.
  • fast/block/basic/017.html: Removed.
1:00 PM Changeset in webkit [21773] by darin
  • 2 edits in trunk/WebCore

Reviewed by Alice.

  • fix <rdar://problem/5228048> REGRESSION: In a new Mail message, the caret isn't immediately inserted when tabbing into or clicking in the message body

This latent problem was uncovered by <​http://trac.webkit.org/projects/webkit/changeset/21476>,
which removed some code that did extra work whenever setCaretVisible made the caret visible.

  • page/Frame.cpp: (WebCore::Frame::selectionLayoutChanged): Call invalidateCaretRect when turning the caret on. All other call sites that touched m_caretPaint were doing this, but this was not.
12:34 PM Changeset in webkit [21772] by yongjzha
  • 3 edits in S60/trunk/JavaScriptCore

2007-05-23 malaiya,

reviewed by Yongjun
DESC: disabled 'SUPPORT_UTF8'flag support in prce
​http://bugs.webkit.org/show_bug.cgi?id=13847
TSW ID : ELWI-6Z2AAU


  • pcre/pcre-config.h:
  • pcre/pcre.c: (pcre_compile):
12:13 PM Changeset in webkit [21771] by darin
  • 2 edits in trunk/WebCore

Reviewed by Harrison.

  • DerivedSources.make: Tweaked spelling.
12:11 PM Changeset in webkit [21770] by thatcher
  • 1 edit in trunk/WebCore/css/CSSGrammar.y

Touching this to force DerivedSources.make to rebuild.

12:07 PM Changeset in webkit [21769] by darin
  • 5 edits in trunk/LayoutTests
  • updated some test results that were affected by Anders's change to make image documents use quirks mode
  • dom/xhtml/level2/html/frame-expected.txt: Updated.
  • dom/xhtml/level2/html/iframe-expected.txt: Updated.
  • editing/pasteboard/copy-standalone-image-expected.txt: Updated.
  • fast/events/standalone-image-drag-to-editable-expected.txt: Updated.
11:59 AM Changeset in webkit [21768] by ggaren
  • 5 edits in trunk/WebKit

Reviewed by Darin Adler, Tim Hatcher.


"unsigned" => "WebNSUInteger" in public API.

  • History/WebBackForwardList.h:
  • WebView/WebScriptDebugDelegate.h:
  • WebView/WebUIDelegate.h:
  • WebView/WebViewPrivate.h:
11:51 AM Changeset in webkit [21767] by weinig
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/forms/search-cancel-button-style-sharing-expected.checksum: Added.
  • fast/forms/search-cancel-button-style-sharing-expected.png: Added.
  • fast/forms/search-cancel-button-style-sharing-expected.txt: Added.
  • fast/forms/search-cancel-button-style-sharing.html: Added.

WebCore:

Reviewed by Darin.

Test: fast/forms/search-cancel-button-style-sharing.html

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::createCancelButtonStyle): Prevented sharing of the cancel button style between search fields that share style, because each search field mutates the style to show and hide the cancel button.
11:39 AM Changeset in webkit [21766] by andersca
  • 3 edits in trunk/WebKitTools

Fix build.

  • DumpRenderTree/DumpRenderTree.m:
  • DumpRenderTree/FrameLoadDelegate.m:
11:04 AM Changeset in webkit [21765] by harrison
  • 3 edits in trunk/WebCore

Reviewed by Kevin and Geoff.

  • DerivedSources.make: Add an inclusion guard for CSSGrammar.h and XPathGrammar.h, because older versions of bison do not


  • css/maketokenizer: Unconditionally define the flex basic types that are not part of newer versions of flex. Was only needed on Linux. Now needed on Mac OS 10.5, and is OK to do on 10.4.9.
10:50 AM Changeset in webkit [21764] by ggaren
  • 11 edits
    1 delete in trunk

WebCore:

Reviewed by Darin Adler.


Moved JavaScriptCore-related pending public API to public API.

Moved -JSObject

  • bindings/objc/WebScriptObjectPendingPublic.h: from here
  • bindings/objc/WebScriptObject.h: to here
  • bindings/objc/WebScriptObject.mm: and out of its temporary category

WebKit:

Reviewed by Darin Adler.


Moved JavaScriptCore-related pending public API to public API.

  • MigrateHeaders.make: Hack to prevent <JavaScriptCore/JSBase.h> from automatically converting to <WebKit/JSBase.h>


Moved -windowObject and -globalContext

  • WebView/WebFramePrivate.h: from here
  • WebView/WebFrame.h: to here
  • WebView/WebFrame.mm: and out of its temporary category

Moved -didClearWindowObject:forFrame:

  • WebView/WebViewPrivate.h: from here
  • WebView/WebFrameLoadDelegate.h: to here
10:43 AM Changeset in webkit [21763] by darin
  • 2 edits in trunk/LayoutTests
  • fast/table/rowindex-comment-nodes-expected.txt: Update to reflect success. A recent check-in fixed this test, but nobody checked in updated results.
10:27 AM Changeset in webkit [21762] by darin
  • 8 edits in trunk/LayoutTests
  • updated some test results that were affected by Hyatt's fix for <rdar://problem/5208440> REGRESSION: Raw text needs to be pulled outside of tables (13753)

The whitespace in the DOM has changed the child indices of various elements,
which shows up in the caret and selection logging.

  • editing/deleting/5026848-2-expected.txt: Updated.
  • editing/deleting/5026848-3-expected.txt: Updated.
  • editing/deleting/5126166-expected.txt: Updated.
  • editing/deleting/delete-block-table-expected.txt: Updated.
  • fast/forms/input-double-click-selection-gap-bug-expected.txt: Updated.
  • fast/replaced/selection-rect-in-table-cell-expected.txt: Updated.
  • fast/text/textIteratorNilRenderer-expected.txt: Updated.
10:26 AM Changeset in webkit [21761] by weinig
  • 3 edits
    3 adds in trunk

JavaScriptCore:

Reviewed by Darin.

  • ​http://bugs.webkit.org/show_bug.cgi?id=13623 (Decompilation of function doesn't compile with "++(x,y)")
  • Create the error node based on the actual node, not the node inside parenthesis
  • Fix applies to postfix, prefix and typeof operators
  • Produces run-time ReferenceError like other non-lvalue assignments etc.
  • kjs/grammar.y: Create {Prefix,Postfix}ErrorNode based on the actual node, not the based on the node returned by "nodeInsideAllParens()". Same for TypeOfValueNode.

LayoutTests:

Reviewed by Darin.

  • fast/js/resources/toString-prefix-postfix-preserve-parens.js: Added.
  • fast/js/toString-prefix-postfix-preserve-parens-expected.txt: Added.
  • fast/js/toString-prefix-postfix-preserve-parens.html: Added.
10:14 AM Changeset in webkit [21760] by weinig
  • 6 edits
    5 adds in trunk

LayoutTests:

Reviewed by Darin.

  • http/tests/multipart/invalid-image-data-standalone-expected.checksum: Added.
  • http/tests/multipart/invalid-image-data-standalone-expected.png: Added.
  • http/tests/multipart/invalid-image-data-standalone-expected.txt: Added.
  • http/tests/multipart/invalid-image-data-standalone.html: Added.
  • http/tests/multipart/resources/green-100x100.png: Added.

WebCore:

Reviewed by Darin.

Test: http/tests/multipart/invalid-image-data-standalone.html

  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::isLoadingMultipartContent): Added.
  • loader/DocumentLoader.h:
  • loader/ImageDocument.cpp: (WebCore::ImageTokenizer::finish): If the image is multipart, make a copy of the current part instead of using the resource data, which is going to be overwritten by the next part.
  • loader/MainResourceLoader.h: (WebCore::MainResourceLoader::isLoadingMultipartContent): Added.
9:52 AM Changeset in webkit [21759] by andersca
  • 2 edits in trunk/WebCore

Fix no-svg build.


  • bindings/js/kjs_html.cpp: (KJS::HTMLElementFunction::callAsFunction):
9:15 AM Changeset in webkit [21758] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Mitz.

<rdar://problem/5226788> After resizing the window larger than the image, hovering over the image still shows magnify glass


One part of the bug was already fixed by my previous commit. This fixes the case where the window is sized so the image fit
after it has been zoomed in.


  • loader/ImageDocument.cpp: (WebCore::ImageDocument::windowSizeChanged):
9:06 AM Changeset in webkit [21757] by sullivan
  • 2 edits in trunk/WebKit

Reviewed by Anders and Tim


  • fixed <rdar://problem/5226000> REGRESSION: In Gmail and Mail, a hang occurs when attempting to grammar/spellcheck a word in a reply
  • WebCoreSupport/WebEditorClient.mm: (WebEditorClient::checkGrammarOfString): Fixed recently-introduced false assumption that NSNotFound == -1
9:06 AM Changeset in webkit [21756] by sullivan
  • 2 edits in trunk/WebCore

Reviewed by Anders and Tim

  • WebCore change for: <rdar://problem/5226000> REGRESSION: In Gmail and Mail, a hang occurs when attempting to grammar/spellcheck a word in a reply
  • editing/Editor.cpp: (WebCore::findFirstBadGrammarInRange): Use the more obvious way to check whether a range is empty. In this case the more obvious way would also have caused no problem to occur. Also added an assertion. Though this change is enough to fix the symptom of the bug, the real fix is in WebKit.
8:51 AM Changeset in webkit [21755] by andersca
  • 2 edits in trunk/WebCore

Reviewed by Darin (Sorta).

  • loader/ImageDocument.cpp: (WebCore::ImageDocument::restoreImageSize): Use removeProperty here.
8:37 AM Changeset in webkit [21754] by staikos
  • 2 edits in trunk/WebKitQt

The http loader should only ask for http cookies. (not, for example, data: cookies)

7:47 AM Changeset in webkit [21753] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/WebKit/wx/wxwebkit.bkl

Relink wxWebKit if webcore or jscore dependencies change.

6:37 AM Changeset in webkit [21752] by hausmann
  • 4 edits in trunk/WebKitTools

For the keyDown binding use QStringList instead of QList<QString>. The former is a default
registered metatype. Also print out critical messages in dumprendertree's qt message handler.

6:37 AM Changeset in webkit [21751] by hausmann
  • 3 edits in trunk/JavaScriptCore

Fix crash in Qt JavaScript bindings when the arguments used on the Qt side are not
registered with QMetaType.

2:46 AM Changeset in webkit [21750] by ggaren
  • 4 edits in trunk/WebCore

Reviewed by Darin Adler.

Some cleanup to Cache while Fixing <rdar://problem/5055182> The page cache
has no global cap.


  • loader/Cache.cpp: Removed destructor to help identify this class as a singleton.
  • loader/Cache.h: Made constructor and destructor private to match the singleton model of PageCache.


Put structs related to Cache inside its declaration to fix some namespace
conflicts I had while writing my patch. (They don't exist anymore, but
I figured this might help future development.)

2:35 AM Changeset in webkit [23408] by ggaren
  • 1 edit in branches/WindowsMerge/WebKitWin/ChangeLog

removing conflict markers from ChangeLog

2:27 AM Changeset in webkit [23407] by ggaren
  • 4 edits in branches/WindowsMerge

WebCoreWin:

Reviewed by Darin Adler.

Fixed a bug with Settings where a number of different objects held bare
Settings* data members, even though the Page owns the Settings object and
deletes it upon destruction.

Added NULL checks since the Page can be deleted before the frame/document.

  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::load):

WebKitWin:

Reviewed by Darin Adler.

Fixed a bug with Settings where a number of different objects held bare
Settings* data members, even though the Page owns the Settings object and
deletes it upon destruction.


Added NULL checks since the Page can be deleted before the frame/document.

2:13 AM Changeset in webkit [21749] by ggaren
  • 38 edits in trunk/WebCore

Reviewed by Darin Adler.

Fixed a bug with Settings where a number of different objects held bare
Settings* data members, even though the Page owns the Settings object and
deletes it upon destruction.

Added NULL checks since the Page can be deleted before the frame/document.

  • page/Settings.cpp: Removed reliance on setNeedsReapplyStylesForSettingsChange.
  • css/cssstyleselector.cpp: CSSStyleSelector was the worst offender. It held a number of back pointers that were not guaranteed. I removed the Settings*, Frame*, and FrameView* back pointers, but others could probably go, too. I replaced them with a single Document* pointer, which is guaranteed because the Document owns the CSSStyleSelector.
  • page/Page.cpp: Removed setNeedsReapplyStylesForSettingsChange -- the Settings object is now a proper subcontroller of Page, so it doesn't need to use a global table to find its page anymore. Also, this function confusingly implied that the Frames in a page can have heterogeneous settings, which is not true.
12:42 AM Changeset in webkit [21748] by bdakin
  • 4 edits in trunk/WebKit

Reviewed by Geoff.

Fix for <rdar://problem/5023545> QuickBooks Pro 2007:hang/crash
after closing QuickBooks Tutorial Center with Leopard9A377

We can hit a race condition where drawRect will be called after the
WebView has closed. Quickbooks does not properly close the WebView
and set the UIDelegate to nil, so the UIDelegate is stale and we
crash. This is a regression because the code that uses the
UIDelegate in the drawRect code path was only added recently. The
method that the UIDelegate calls into is new -- it does not exist
on Tiger -- so there is no harm in not running this code for
applications linked against older WebKits. Other applications may
run into this same bug so I am not doing a bundle
check...particularly because, as I mentioned, the new UIDelegate
call would not be implemented by older clients anyway.

  • Misc/WebKitVersionChecks.h:
  • WebView/WebHTMLView.mm: (-[WebHTMLView drawSingleRect:]):
12:10 AM Web Inspector edited by smith@sideshowbarker.net
HUD = Heads-Up Display (diff)
12:07 AM Changeset in webkit [21747] by hyatt
  • 2 edits in trunk/WebCore

Make sure -webkit-border-fit accounts for padding as well as border.

Reviewed by aroben

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::borderFitAdjust):
12:02 AM Web Inspector edited by smith@sideshowbarker.net
added link to Surfin' Safari blog entry (diff)
Note: See TracTimeline for information about the timeline view.