Timeline



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)

May 24, 2007:

11:58 PM Changeset in webkit [21746] by hyatt
  • 8 edits in trunk/WebCore

Rework line numbering (again). This time I finally think I have
it right for all the document.write insanity that can occur.

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::reportErrorToConsole):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserDocumentWriteMessage):
  • html/HTMLParserErrorCodes.h:
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::write):
  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::lineNumberPtr): (WebCore::HTMLTokenizer::processingContentWrittenByScript):
  • platform/SegementedSring.cpp (WebCore::SegmentedString::setExcludeLineNumbers):
  • platform/SegmentedString.h: (WebCore::SegmentedSubstring::SegmentedSubstring): (WebCore::SegmentedSubstring::clear): (WebCore::SegmentedSubstring::excludeLineNumbers): (WebCore::SegmentedSubstring::setExcludeLineNumbers): (WebCore::SegmentedString::excludeLineNumbers): (WebCore::SegmentedString::advance):
11:56 PM Web Inspector edited by smith@sideshowbarker.net
boldface Inspect Element (diff)
11:48 PM Web Inspector edited by smith@sideshowbarker.net
added WI UI image (diff)
11:13 PM Changeset in webkit [21745] by ggaren
  • 2 edits in trunk/WebCore

Reviewed by Adam Roben.

Added a debug-only CachedPage counter. This helped me track down an SVG
timer bug in my recent page cache work.


  • history/CachedPage.cpp:
11:05 PM Changeset in webkit [21744] by sfalken
  • 1 copy in tags/Safari-522.8.3b

New tag.

9:42 PM Changeset in webkit [21743] by hyatt
  • 4 edits in trunk/WebCore

Improve <form> error reporting to catch <form>s inside table parts.
Tweak the wording of some of the error messages to be a bit better.

Reviewed by olliej

  • html/HTMLParser.cpp: (WebCore::HTMLParser::insertNode): (WebCore::HTMLParser::processCloseTag): (WebCore::HTMLParser::popBlock):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate):
  • html/HTMLParserErrorCodes.h:
6:28 PM Changeset in webkit [21742] by weinig
  • 6 edits
    8 adds in trunk

LayoutTests:

Reviewed by Hyatt.

Test cases for:
http://bugs.webkit.org/show_bug.cgi?id=4104
first-letter not updating style with hover
http://bugs.webkit.org/show_bug.cgi?id=13169
css-formatted first letter won't hide dynamically

  • fast/css/first-letter-hover-expected.checksum: Added.
  • fast/css/first-letter-hover-expected.png: Added.
  • fast/css/first-letter-hover-expected.txt: Added.
  • fast/css/first-letter-hover.html: Added.
  • fast/css/first-letter-visibility-expected.checksum: Added.
  • fast/css/first-letter-visibility-expected.png: Added.
  • fast/css/first-letter-visibility-expected.txt: Added.
  • fast/css/first-letter-visibility.html: Added.

WebCore:

Reviewed by Hyatt.

http://bugs.webkit.org/show_bug.cgi?id=4104
first-letter not updating style with hover
http://bugs.webkit.org/show_bug.cgi?id=13169
css-formatted first letter won't hide dynamically

Update first letter when RenderBlock::setStyle is called.

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::checkOneSelector):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::createElement):
  • dom/Document.h: (WebCore::Document::usesFirstLetterRules): (WebCore::Document::setUsesFirstLetterRules):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::setStyle): (WebCore::RenderBlock::updateFirstLetter):
6:15 PM Changeset in webkit [21741] by kevino
  • 5 edits in branches/wx-port-alpha/trunk/WebCore/platform

Fix crash caused by Widget::setCursor not checking for a null implementation. This can happen because we don't implement image cursors yet. Also add a wxBitmap accessor to BitmapImage to make it easier to add such support in the future.

5:09 PM Creating and Submitting Layout Tests and Patches created by KWhiteRight@gmail.com
4:57 PM Changeset in webkit [21740] by weinig
  • 5 edits in trunk/LayoutTests

Reviewed by Darin.

Updated the test and the results.

  • fast/forms/select-item-background-clip-expected.checksum:
  • fast/forms/select-item-background-clip-expected.png:
  • fast/forms/select-item-background-clip-expected.txt:
  • fast/forms/select-item-background-clip.html:
4:51 PM Changeset in webkit [21739] by weinig
  • 3 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

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

WebCore:

Reviewed by Darin.

Test: fast/forms/slider-thumb-shared-style.html

RenderSlider mutates the thumb's style, therefore it must not be shared
with another slider's thumb, a situation that could occur if the sliders
themselves shared style.

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::createThumbStyle): Always use a private copy for the thumb style.
4:48 PM Changeset in webkit [21738] by adele
  • 6 edits
    3 adds in trunk

LayoutTests:

Reviewed by Darin.

Test for <rdar://problem/5211921> Form values sometimes get transposed (iGoogle)

  • fast/history/redirect-load-no-form-restoration-expected.txt: Added.
  • fast/history/redirect-load-no-form-restoration.html: Added.
  • fast/history/resources/redirect-load.html: Added.

WebCore:

Reviewed by Darin.

Fix for <rdar://problem/5211921> Form values sometimes get transposed (iGoogle)

Test: fast/history/redirect-load-no-form-restoration.html

  • history/HistoryItem.cpp: (WebCore::HistoryItem::setURL): Clear the document state when the url changes for a HistoryItem. (WebCore::HistoryItem::isCurrentDocument): Added. Compares the HistoryItem url to the document's url. This might not be the best way to do this, but it works for now.
  • history/HistoryItem.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::saveDocumentState): Only set the document (and form) state if the document is current for the history item.
4:36 PM WikiStart edited by KWhiteRight@gmail.com
(diff)
4:35 PM Changeset in webkit [21737] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/Bakefiles/presets.bkl

Remove tabs that accidently got into the file.

4:28 PM Changeset in webkit [21736] by weinig
  • 5 edits in trunk/JavaScriptCore

Reviewed by Darin

Patch for http://bugs.webkit.org/show_bug.cgi?id=13855
Port patch 666176 to JavaScriptCore

  • Renamed JSValue::downcast() to JSValue::asCell() which makes the function meaning cleaner. It's modeled after Harri Porten change in KDE trunk.
  • kjs/collector.cpp: (KJS::Collector::protect): (KJS::Collector::unprotect): (KJS::Collector::collectOnMainThreadOnly):
  • kjs/object.h: (KJS::JSValue::isObject):
  • kjs/string_object.cpp: (KJS::StringProtoFunc::callAsFunction):
  • kjs/value.h: (KJS::JSValue::asCell): (KJS::JSValue::isNumber): (KJS::JSValue::isString): (KJS::JSValue::isObject): (KJS::JSValue::getNumber): (KJS::JSValue::getString): (KJS::JSValue::getObject): (KJS::JSValue::getUInt32): (KJS::JSValue::mark): (KJS::JSValue::marked): (KJS::JSValue::type): (KJS::JSValue::toPrimitive): (KJS::JSValue::toBoolean): (KJS::JSValue::toNumber): (KJS::JSValue::toString): (KJS::JSValue::toObject):
4:20 PM Changeset in webkit [21735] by hyatt
  • 59 edits in trunk/LayoutTests/tables

Update results for better whitespace handling in framesets and tables.

4:05 PM Changeset in webkit [21734] by weinig
  • 3 edits
    2 adds in trunk

LayoutTests:

Reviewed by Adele.

  • fast/forms/select-double-onchange-expected.txt: Added.
  • fast/forms/select-double-onchange.html: Added.

WebCore:

Reviewed by Adele.

Test: fast/forms/select-double-onchange.html

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::menuListOnChange): Changed to update m_lastOnChangeIndex before dispatching the change event to avoid reentrancy.
3:58 PM Changeset in webkit [21733] by hyatt
  • 3 edits in trunk/LayoutTests/webarchive

Update results for better whitespace handling in framesets and tables.

3:56 PM Changeset in webkit [21732] by darin
  • 13 edits in trunk/WebCore

Reviewed by Hyatt.

  • fix <rdar://problem/5226451> REGRESSION (21618): Mail block quotes are missing the lines on the side

This patch fixes three problems:

1) makes the "skip canLoad check" boolean also skip the check

that prevents any loads while the document is in provisional
state; this is the proximate cause of the bug

  • loader/SubresourceLoader.cpp: (WebCore::SubresourceLoader::create): Don't check the frame's state if skipCanLoadCheck is true.

2) moves the "skip canLoad check" boolean to the Request object;

the old implementation would cause that flag to affect the
new request we served, which might not be the resource with
that flag set

3) fixes error-handling code path that would leak requests

  • html/HTMLImageLoader.cpp: (WebCore::HTMLImageLoader::updateFromElement): Pass false to the CachedImage constructor to indicate we are not making this object for the cache.
  • loader/Cache.cpp: (WebCore::createResource): Pass true to the CachedImage constructor to indicate we are making this object for the cache. (WebCore::Cache::requestResource): Add new code that assumes the object will already have the inCache bit set, and that will delete the object and return 0 if the cache is disabled and the load failed.
  • loader/CachedImage.h:
  • loader/CachedImage.cpp: (WebCore::CachedImage::CachedImage): Added a forCache boolean parameter. Always false for the constructor that's only used outside the cache code, and passed in as a boolean for the constructor that's used both in cache and outside cache.
  • loader/CachedResource.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::CachedResource): Added a forCache boolean parameter that determines the initial state of the m_inCache flag. This is needed to prevent a resource from being destroyed if an error occurs during the initial load.
  • loader/DocLoader.cpp: (WebCore::DocLoader::requestCSSStyleSheet): Added FIXME. (WebCore::DocLoader::setLoadInProgress): Added null check.
  • loader/Request.h:
  • loader/Request.cpp: (WebCore::Request::Request): Added a shouldSkipCanLoadCheck boolean here, since we need to track this for each request.
  • loader/loader.h:
  • loader/loader.cpp: (WebCore::Loader::load): Pass the skipCanLoadCheck boolean to the Request constructor rather than to the servePendingRequests function. (WebCore::Loader::servePendingRequests): Add a loop so we can handle cases where the request fails immediately without leaking the request and thinking that we're loading forever.
3:29 PM Changeset in webkit [21731] by hyatt
  • 4 edits in trunk/WebCore

Add error reporting for residual style.

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleResidualStyleCloseTagAcrossBlocks): (WebCore::HTMLParser::reopenResidualStyleTags):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate):
  • html/HTMLParserErrorCodes.h:
3:13 PM Changeset in webkit [21730] by darin
  • 2 edits in trunk/WebKit
  • English.lproj/StringsNotToBeLocalized.txt: Update for recent changes.
2:49 PM Changeset in webkit [21729] by hyatt
  • 24 edits in trunk/LayoutTests/dom/xhtml/level2/html

Land updated tests.

1:57 PM Changeset in webkit [21728] by oliver
  • 2 edits in trunk/WebKit

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

Reviewed by Adam, Darin, and Geoff.

Fix for <rdar://problem/5000470> REGRESSION: The IM reconvert
function returns incorrect symbol due to inconsistent range
domains in TSM

Text Services Management uses ranges provided by the
NSTextInput API to index into the string return by
-[WebHTMLView string]. As a result some input methods
incorrectly get their candidate text from the beginning
of the document instead of from the input element.

TSM prefers to query -textStorage over -string so as a
workaround we provide an implementation of -textStorage that
returns the content of the current text input. TSM only ever
queries the result of textStorage as an NSAttributedString so
we do not need to implement a fake NSTextStorage class

This should not cause harm to anything else as textStorage is
actually a method on NSTextView, which we clearly are not. TSM
only queries the method because it uses respondsToSelector to
control behaviour.

  • WebView/WebHTMLView.mm: (-[WebHTMLView textStorage]):
1:45 PM Changeset in webkit [21727] by hyatt
  • 4 edits in trunk/WebCore

Report <script/> as a warning instead of an error for now, since we
actually do close it. (Sigh.)

Reviewed by beth

  • html/HTMLParser.cpp: (WebCore::HTMLParser::parseToken): (WebCore::HTMLParser::reportErrorToConsole):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate): (WebCore::isWarning):
  • html/HTMLParserErrorCodes.h:
1:26 PM Changeset in webkit [21726] by hyatt
  • 5 edits in trunk/WebCore

Add error reporting for the </p> -> <p></p> quirk.

Add error reporting when a stray close tag is encountered with no
corresponding open tag.

Reviewed by adele

  • html/HTMLParser.cpp: (WebCore::HTMLParser::HTMLParser): (WebCore::HTMLParser::processCloseTag): (WebCore::HTMLParser::popBlock):
  • html/HTMLParser.h: (WebCore::HTMLParser::popBlock):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate):
  • html/HTMLParserErrorCodes.h:
1:16 PM Changeset in webkit [21725] by kmccullo
  • 2 edits in trunk/LayoutTests

Reviewed by.

  • Do you ever feel like Charlie Brown? *sigh*
  • fast/dom/HTMLHeadElement/resources/head-check.js:
1:06 PM Changeset in webkit [21724] by kmccullo
  • 5 edits
    9 deletes in trunk

LayoutTests:

Reviewed by.

  • Rolling back change to fix the build until I have a good fix for data: urls.
  • ChangeLog:
  • fast/dom/HTMLHeadElement/resources/head-check.js:
  • fast/dom/frame-loading-via-document-write-expected.txt: Removed.
  • fast/dom/frame-loading-via-document-write.html: Removed.
  • fast/dom/resources/frame-loading-via-document-write.js: Removed.
  • fast/dom/resources/green250x50.png: Removed.
  • fast/dom/resources/success.html: Removed.
  • http/tests/security/frame-loading-via-document-write-expected.txt: Removed.
  • http/tests/security/frame-loading-via-document-write.html: Removed.
  • http/tests/security/resources/frame-loading-via-document-write.js: Removed.
  • http/tests/security/resources/success.html: Removed.

WebCore:

Reviewed by.

  • Rolling back change to fix the build until I have a good fix for data: urls.
  • ChangeLog:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::load): (WebCore::FrameLoader::open):
  • platform/String.cpp:
12:54 PM Changeset in webkit [21723] by hyatt
  • 6 edits in trunk/WebCore

Add two more error messages to the HTMLParser.

Report when the </br> -> <br> quirk is triggered.

Also report whenever the broken XML self-closing tag syntax is used on
tags that can actually have an end tag (either optional or required).

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::parseToken):
  • html/HTMLParserErrorCodes.cpp: (WebCore::htmlParserErrorMessageTemplate):
  • html/HTMLParserErrorCodes.h: (WebCore::):
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag):
  • html/HTMLTokenizer.h: (WebCore::Token::reset):
12:47 PM Changeset in webkit [21722] by spadma
  • 2 edits in S60/trunk/WebKit

deepa, reviewed by Sachin

DESC: [S60] Scrollbar covers some parts of web page in RTL mode (MNAN-734F5H)
http://bugs.webkit.org/show_bug.cgi?id=13831

  • BrowserControl/src/BrCtl.cpp:
11:47 AM Changeset in webkit [23406] by weinig
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by NOBODY (OOPS!).

Build fix.

  • WebCore.vcproj/WebCore.vcproj:
11:34 AM Changeset in webkit [23405] by andersca
  • 3 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

<rdar://5211882>
Need to relaunch Boomer to recognize newly installed QT plug-in


<rdar://5061867>
After installing Flash, Safari would still not display pages that needed it


  • plugins/win/PluginDatabaseWin.cpp: (WebCore::PluginDatabaseWin::refresh): Return whether the plugin database changed or not.


(WebCore::PluginDatabaseWin::findPlugin):
New function that tries to find a plug-in given its url and MIME type.


(WebCore::PluginDatabaseWin::createPluginView):
If no plugin is found, refresh the database and search for the plugin again if the database changed.

11:30 AM Changeset in webkit [21721] by weinig
  • 2 edits
    1 move in trunk/WebCore

Reviewed by Anders.

Rename JSHTMLFrameSetCustom.cpp to JSHTMLFrameSetElementCustom.cpp
for consistency.

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLFrameSetCustom.cpp: Removed.
  • bindings/js/JSHTMLFrameSetElementCustom.cpp: Added.
11:26 AM Changeset in webkit [21720] by hyatt
  • 6 edits in trunk/WebCore

Fix for <rdar://problem/5208440> (13753)

REGRESSION: Raw text needs to be pulled outside of tables

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::handleError):
  • html/HTMLTableColElement.cpp: (WebCore::HTMLTableColElement::checkDTD):
  • html/HTMLTableElement.cpp: (WebCore::HTMLTableElement::checkDTD):
  • html/HTMLTableRowElement.cpp: (WebCore::HTMLTableRowElement::checkDTD):
  • html/HTMLTableSectionElement.cpp: (WebCore::HTMLTableSectionElement::checkDTD):
10:55 AM Changeset in webkit [23404] by adele
  • 2 edits in branches/WindowsMerge/WebCore
10:45 AM Changeset in webkit [21719] by adele
  • 4 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

  • fast/forms/menulist-separator-painting-expected.checksum: Added.
  • fast/forms/menulist-separator-painting-expected.png: Added.
  • fast/forms/menulist-separator-painting-expected.txt: Added.
  • fast/forms/menulist-separator-painting.html: Added.

WebCore:

Reviewed by Darin.

Test: fast/forms/menulist-separator-painting.html

  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::drawConvexPolygon): Changed this function to actually do both filling and stroking when needed, even though all existing callers set the stroke to none.
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintMenuListButton): Added calls to set the stroke style and thickness when painting the arrows and the separator.
10:40 AM Changeset in webkit [21718] by thatcher
  • 3 edits in trunk/WebCore

Reviewed by Sam Weinig.

Remove the OBJC_NEW_PROPERTIES #ifdef and only export the new @property syntax.

  • bindings/objc/DOMObject.h:
  • bindings/scripts/CodeGeneratorObjC.pm:
10:36 AM Changeset in webkit [21717] by spadma
  • 1 edit in S60/trunk/S60Internals/S60Headers.zip

Update S60Headers.zip for font access

10:20 AM Changeset in webkit [21716] by weinig
  • 5 edits in trunk

LayoutTests:

Reviewed by Anders.

  • update results to account for the new JSHTMLFrameSetElement that is being autogenerated and can now produce a constructor.
  • fast/dom/Window/element-constructors-on-window-expected.txt:
  • fast/dom/Window/window-properties-expected.txt:

WebCore:

Reviewed by Anders.

Uncomment HTMLFramesSetElement constructor now that it exists.

  • page/DOMWindow.idl:
10:04 AM Changeset in webkit [21715] by andersca
  • 2 edits in trunk/WebCore

Try fixing the buildbot build.


  • loader/ImageDocument.cpp: (WebCore::ImageDocument::resizeImageToFit): (WebCore::ImageDocument::imageClicked):
9:57 AM Changeset in webkit [23403] by weinig
  • 2 edits in branches/WindowsMerge/WebCore

Build fix.

  • WebCore.vcproj/WebCore.vcproj:
9:39 AM Changeset in webkit [21714] by weinig
  • 8 edits
    1 add in trunk/WebCore

Reviewed by Darin.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13853
Autogenerate the JS bindings for the HTMLFrameSetElement

  • DerivedSources.make:
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLElementWrapperFactory.cpp:
  • bindings/js/JSHTMLFrameSetCustom.cpp: Added. (WebCore::JSHTMLFrameSetElement::canGetItemsForName): (WebCore::JSHTMLFrameSetElement::nameGetter):
  • bindings/js/kjs_html.cpp: (KJS::): (KJS::JSHTMLElement::classInfo): (KJS::JSHTMLElement::accessors): (KJS::JSHTMLElement::getOwnPropertySlot):
  • bindings/js/kjs_html.h: (KJS::JSHTMLElement::):
  • html/HTMLFrameSetElement.idl:
9:25 AM Changeset in webkit [21713] by ddkilzer
  • 1 edit in trunk/LayoutTests/ChangeLog

Remove conflict markers.

9:06 AM Changeset in webkit [21712] by spadma
  • 14 edits in S60/trunk

yongjzha, reviewed by sachin

DESC: enable font access api.
http://bugs.webkit.org/show_bug.cgi?id=13832

WARNING: NO TEST CASES ADDED OR CHANGED

  • bridge/WebCoreTextRendererFactory.h:
  • khtml/rendering/font.cpp: (Font::update):
  • kwq/KWQFont.cpp: (QFont::QFont): (QFont::operator= ): (QFont::Renderer): (QFont::setGenericFamily):
  • kwq/KWQFont.h:
8:41 AM Changeset in webkit [21711] by zack
  • 2 edits in trunk/WebCore

Fixing check for incorrent font size - it should
check fixed not normal.

r=andersca

8:11 AM Changeset in webkit [21710] by harrison
  • 2 edits in trunk/WebKit

Reviewed by Tim Hatcher.

<rdar://problem/5225343> REGRESSION: With View Source window opened, navigating to a different URL in the browser window results in a crash at WebCore::FrameLoader::frameHasLoaded()


_private->loader->frameLoader() was not being nil checked.

  • WebView/WebDataSource.mm: (-[WebDataSource request]): Add nil check for _private->loader->frameLoader()
7:10 AM Changeset in webkit [21709] by hausmann
  • 2 edits in trunk/WebKitQt

Fix multipart HTTP POSTs in the Qt port

5:37 AM Changeset in webkit [21708] by bdash
  • 2 edits in trunk/WebCore

Speculative buildbot build fix.

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty):
2:53 AM Changeset in webkit [21707] by aroben
  • 1 edit in trunk/WebCore/page/Chrome.cpp

Didn't mean to commit this

2:42 AM Changeset in webkit [21706] by aroben
  • 11 edits in trunk/WebCore

Made Chrome::addMessageToConsole more flexible

addMessageToConsole now takes two additional parameters to specify the
message source (HTML, JS, CSS, XML) and level (error, warning, log). To
maintain our old behavior, all current messages were given
JSMessageSource and ErrorMessageLevel parameters. Messages matching
these characteristics are passed up to the ChromeClient.

Reviewed by Hyatt.

2:42 AM Changeset in webkit [21705] by zimmermann
  • 630 edits in branches/feature-branch

Reviewed by Oliver.

Add new code path dumping SVG text engine specific layout test results.
Finally the dumped results are actually accurate and respond to the
rendered glyphs. This has never been correct so far.

1:56 AM Changeset in webkit [21704] by hyatt
  • 8 edits in trunk/WebCore

Rework how HTML handles line numbering so that it is not fooled by
\r without a following \n and so that document.write doesn't lead
to line numbering confusion.

Reviewed by aroben

  • html/HTMLParser.cpp: (WebCore::HTMLParser::reportErrorToConsole):
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::processListing): (WebCore::HTMLTokenizer::parseSpecial): (WebCore::HTMLTokenizer::parseComment): (WebCore::HTMLTokenizer::parseServer): (WebCore::HTMLTokenizer::parseProcessingInstruction): (WebCore::HTMLTokenizer::parseText): (WebCore::HTMLTokenizer::parseEntity): (WebCore::HTMLTokenizer::parseTag): (WebCore::HTMLTokenizer::write): (WebCore::HTMLTokenizer::processToken): (WebCore::HTMLTokenizer::setSrc):
  • html/HTMLTokenizer.h: (WebCore::HTMLTokenizer::lineNumberPtr):
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::addLine):
  • loader/TextDocument.cpp: (WebCore::TextTokenizer::write):
  • platform/SegmentedString.cpp: (WebCore::SegmentedString::SegmentedString): (WebCore::SegmentedString::operator=): (WebCore::SegmentedString::clear):
  • platform/SegmentedString.h: (WebCore::SegmentedString::SegmentedString): (WebCore::SegmentedString::advance): (WebCore::SegmentedString::escaped):
1:47 AM Changeset in webkit [21703] by hyatt
  • 2 edits in trunk/LayoutTests/fast/css

Update some tests that darin missed.

1:46 AM Changeset in webkit [21702] by hyatt
  • 1 edit in trunk/LayoutTests/fast/dom/background-shorthand-csstext-expected.txt

Land corrected background shorthand test now that I yanked the bogus alert.

12:47 AM Changeset in webkit [21701] by hausmann
  • 4 edits in trunk/WebKitQt

Share WebCoreHttp and therefore HTTP connections among multiple
QWebNetworkInterface instances by moving the code into
QWebNetworkManager.

12:01 AM Changeset in webkit [23402] by sfalken
  • 4 edits in branches/WindowsMerge/WebKitWin

Added IWebFramePrivate::isDescendantOfFrame.

  • Interfaces/IWebFramePrivate.idl:
  • WebFrame.cpp: (WebFrame::isDescendantOfFrame):
  • WebFrame.h:

May 23, 2007:

11:01 PM Changeset in webkit [21700] by hyatt
  • 1 edit in trunk/LayoutTests/fast/dom/background-shorthand-csstext.html

Remove a stray alert in a testcase that I wrote a while back.

10:58 PM Changeset in webkit [21699] by hyatt
  • 2 adds in trunk/LayoutTests/fast/table

Add a table test for comment nodes.

10:48 PM Changeset in webkit [21698] by hyatt
  • 3 edits in trunk/WebCore

Make the delete/InsertRows/Cells methods as well as the rowIndex method
able to work properly even when whitespace or comment nodes are present.

Reviewed by aroben

fast/table/rowindex-comment-nodes.html

  • html/HTMLTableRowElement.cpp: (WebCore::HTMLTableRowElement::rowIndex): (WebCore::HTMLTableRowElement::insertCell): (WebCore::HTMLTableRowElement::deleteCell):
  • html/HTMLTableSectionElement.cpp: (WebCore::HTMLTableSectionElement::insertRow): (WebCore::HTMLTableSectionElement::deleteRow):
10:39 PM Changeset in webkit [21697] by hyatt
  • 4 adds in trunk/LayoutTests/fast/forms

Land optgroup DTD test.

10:36 PM Changeset in webkit [21696] by hyatt
  • 3 edits in trunk/WebCore

<optgroup> should allow <script> and text nodes inside it, just like
<select> does.

Reviewed by aroben

fast/forms/stuff-on-my-optgroup.html

  • html/HTMLOptGroupElement.cpp: (WebCore::HTMLOptGroupElement::checkDTD):
  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::checkDTD):
10:33 PM Changeset in webkit [21695] by hyatt
  • 4 adds in trunk/LayoutTests/fast/layers

Add gigantic z-index test.

10:23 PM Changeset in webkit [21694] by hyatt
  • 2 edits in trunk/WebCore

Framesets should allow whitespace text nodes as children.

Reviewed by aroben

  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::checkDTD):
9:06 PM Changeset in webkit [23401] by aliceli1
  • 2 edits in branches/WindowsMerge/LayoutTests

Reviewed by.

  • win/Skipped: remove fixed tests
8:28 PM Changeset in webkit [21693] by bdash
  • 2 edits in trunk/WebCore

Fix the Qt build after r21685.

  • platform/qt/CursorQt.cpp: (WebCore::notAllowedCursor): (WebCore::zoomInCursor): (WebCore::zoomOutCursor):
8:23 PM Changeset in webkit [21692] by bdash
  • 2 edits in trunk/WebCore

Reviewed by Adam.

Fix the gdk build following changes in r21685.

  • platform/gdk/CursorGdk.cpp: (WebCore::zoomInCursor): (WebCore::zoomOutCursor):
8:07 PM Changeset in webkit [21691] by hyatt
  • 2 edits in trunk/WebCore

Fix for <rdar://problem/5222611> REGRESSION: Paint ordering problems at
http://buymeasoda.com/dev/issues/image_onload/image_onload.html

Clamp z-index so that values that exceed maxint (or that are below minint)
are clamped to those values.

Reviewed by darin

  • css/cssstyleselector.cpp: (WebCore::CSSStyleSelector::applyProperty):
7:13 PM Changeset in webkit [23400] by weinig
  • 2 edits in branches/WindowsMerge/WebCore

Build fix.

  • WebCore.vcproj/WebCore.vcproj:
6:46 PM Changeset in webkit [21690] by ddkilzer
  • 4 edits
    4 adds in trunk

LayoutTests:

Reviewed by David Hyatt.

  • fast/css/stale-style-selector-crash-1-expected.txt: Added.
  • fast/css/stale-style-selector-crash-1.html: Added.
  • fast/css/stale-style-selector-crash-2-expected.txt: Added.
  • fast/css/stale-style-selector-crash-2.html: Added.

WebCore:

Reviewed by David Hyatt.

Tests: fast/css/stale-style-selector-crash-1.html

fast/css/stale-style-selector-crash-2.html

  • dom/Document.cpp: (WebCore::Document::Document): Initialize the m_didCalculateStyleSelector flag to 'false'. (WebCore::Document::updateStyleSelector): Do not bail out if we have ever calculated a non-trivial style selector, even if at the moment we are pending style sheets. We could have calculated a non-trivial selector already either because at some point there were no pending style sheets (so the currently pending ones were added dynamically) or because we were forced to do a layout ignoring pending style sheets. Either way, once there is a style selector, we need to keep it up to date, otherwise we can crash under recalcStyle() during updateRendering() or simply not recalculate style correctly for dynamic changes. (WebCore::Document::recalcStyleSelector): Set m_didCalculateStyleSelector to 'true' when creating a new style selector.
  • dom/Document.h:
6:43 PM Changeset in webkit [21689] by andersca
  • 3 edits in trunk/WebCore

Reviewed by Adam.

<rdar://problem/5224749> Show a zoom/unzoom cursor for resizable image documents


  • WebCore.xcodeproj/project.pbxproj: Install the zoom cursors.


  • loader/ImageDocument.cpp: (WebCore::ImageDocument::resizeImageToFit): Set the cursor to the zoom in cursor.


(WebCore::ImageDocument::restoreImageSize):
If the image fits, reset the cursor. Otherwise, set it to the zoom in cursor.

6:36 PM Changeset in webkit [21688] by andersca
  • 5 edits in trunk/WebKit

Reviewed by Geoff.

<rdar://problem/3663808> Resize large images to fit in the browser window


Add new WebPreferences SPI.


  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebPreferences.m: (+[WebPreferences initialize]): (-[WebPreferences enableAutomaticImageResizing]): (-[WebPreferences setEnableAutomaticImageResizing:]):
  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm: (-[WebView _updateWebCoreSettingsFromPreferences:]):
6:11 PM Changeset in webkit [21687] by darin
  • 1234 edits in trunk

LayoutTests:

Reviewed by Hyatt.

  • updated the many tests with an extra newline or two at the end because of the bug where plain-text serialization will consider nodes that are not rendered

WebCore:

Reviewed by Hyatt.

  • fix problems where unrendered nodes were affecting on plain text serialization
  • editing/TextIterator.cpp: (WebCore::TextIterator::advance): Don't call handleXXX functions or exitNode for nodes without renderers. (WebCore::shouldEmitNewlineAfterNode): Only consider rendered nodes when deciding if this is the last node in the document.
6:04 PM Changeset in webkit [23399] by andersca
  • 2 edits in branches/WindowsMerge/WebCore

Fix build.


  • platform/win/CursorWin.cpp: (WebCore::zoomInCursor): (WebCore::zoomOutCursor):
6:01 PM Changeset in webkit [21686] by andersca
  • 6 edits in trunk/WebCore

Reviewed by Geoff.

<rdar://problem/3663808> Resize large images to fit in the browser window


  • WebCore.exp: Add new settings method.


  • loader/ImageDocument.cpp: (WebCore::ImageEventListener::ImageEventListener): New class that handles resize events and click events and forwards them to the image document.


(WebCore::ImageTokenizer::ImageTokenizer):
(WebCore::ImageTokenizer::writeRawData):
(WebCore::ImageTokenizer::finish):
(WebCore::ImageDocument::ImageDocument):
(WebCore::ImageDocument::createDocumentStructure):
(WebCore::ImageDocument::cachedImage):
Let the ImageDocument class keep track of its document structure.


(WebCore::ImageDocument::scale):
Returns the scale that should be used when resizing the image.


(WebCore::ImageDocument::resizeImage):
Update the image size.


(WebCore::ImageDocument::imageClicked):
Toggle between resized and not resized. When restoring the size, scroll the image so that the area under
the mouse stays the same.


(WebCore::ImageDocument::imageChanged):
When the image size is known, resize the image if needed.


(WebCore::ImageDocument::restoreImageSize):
Restore the image size.


(WebCore::ImageDocument::imageNeedsResizing):
Return whether the image is too big for its window or not.


(WebCore::ImageDocument::windowSizeChanged):
Called when the window's size changes. Determine if the window fits or not and resize it if it doesn't fit.


(WebCore::ImageEventListener::handleEvent):
Forward events to the image document.


  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setEnableAutomaticImageResizing):
  • page/Settings.h: (WebCore::Settings::enableAutomaticImageResizing): Add new setting.
5:50 PM Changeset in webkit [21685] by andersca
  • 7 edits
    2 adds in trunk/WebCore

Reviewed by Darin.

Add -webkit-zoom-in and -webkit-zoom-out cursors.


  • Resources/zoomInCursor.png: Added.
  • Resources/zoomOutCursor.png: Added.
  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
  • css/CSSValueKeywords.in:
  • page/EventHandler.cpp: (WebCore::selectCursor):
  • platform/Cursor.h:
  • platform/mac/CursorMac.mm: (WebCore::zoomInCursor): (WebCore::zoomOutCursor):
  • rendering/RenderStyle.h: (WebCore::):
5:01 PM Changeset in webkit [21684] by kevino
  • 1 edit in branches/wx-port-alpha/trunk/WebKit/wx/WebView.cpp

Define rint ourselves for MSVS 2005 too, as it doesn't seem to be defined there either.

4:32 PM Changeset in webkit [21683] by weinig
  • 6 edits in trunk/WebCore

Reviewed by Darin.

Part one of patch for http://bugs.webkit.org/show_bug.cgi?id=13830
Auto-generate JS DOM bindings for HTMLDocument and most of the rest
of HTMLElement

  • Just the HTMLElement part.
  • bindings/js/kjs_html.cpp: (KJS::JSHTMLElement::getValueProperty): (KJS::JSHTMLElement::putValueProperty):
  • bindings/js/kjs_html.h: (KJS::JSHTMLElement::):
  • html/HTMLElement.idl:
4:19 PM Changeset in webkit [21682] by kmccullo
  • 4 edits
    9 adds in trunk

LayoutTests:

Reviewed by Darin.

Update tests for http://bugs.webkit.org/show_bug.cgi?id=13830
Auto-generate JS DOM bindings for HTMLDocument and most of
the rest of HTMLElement

  • Corrects prototype chain for JSHTMLDocument and adds test for HTMLFormElement.
  • fast/dom/prototype-chain-expected.txt:
  • fast/dom/prototype-chain.html:

WebCore:

Reviewed by Darin.

  • rdar://problem/5183523
  • We now check the document's URL to see if it can load a resource in the case of a URL click, instead of checking the referrer, which is blank for local files.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::load): (WebCore::FrameLoader::open):
  • platform/PlatformString.h: (WebCore::debugPrintString):
4:00 PM Changeset in webkit [21681] by weinig
  • 20 edits
    1 add in trunk

LayoutTests:

Reviewed by Darin.

Update tests for http://bugs.webkit.org/show_bug.cgi?id=13830
Auto-generate JS DOM bindings for HTMLDocument and most of
the rest of HTMLElement

  • Corrects prototype chain for JSHTMLDocument and adds test for HTMLFormElement.
  • fast/dom/prototype-chain-expected.txt:
  • fast/dom/prototype-chain.html:

WebCore:

Reviewed by Darin.

Part two of patch for http://bugs.webkit.org/show_bug.cgi?id=13830
Auto-generate JS DOM bindings for HTMLDocument and most of the rest
of HTMLElement

  • Just the HTMLDocument part.
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLDocumentCustom.cpp: Added. (WebCore::JSHTMLDocument::canGetItemsForName): (WebCore::JSHTMLDocument::nameGetter): (WebCore::JSHTMLDocument::all): (WebCore::JSHTMLDocument::setAll): (WebCore::JSHTMLDocument::location): (WebCore::JSHTMLDocument::setLocation): (WebCore::JSHTMLDocument::open): (WebCore::writeHelper): (WebCore::JSHTMLDocument::write): (WebCore::JSHTMLDocument::writeln): (WebCore::JSHTMLDocument::clear):
  • bindings/js/kjs_dom.cpp: (KJS::toJS):
  • bindings/js/kjs_dom.h:
  • bindings/js/kjs_html.cpp:
  • bindings/js/kjs_html.h:
  • bindings/objc/DOMPrivate.h:
  • bindings/scripts/CodeGeneratorJS.pm: Add [Deletable] property for Document.all and fix HasOverridingNameGetter to allow correct prototype chain
  • dom/Document.cpp: (WebCore::Document::plugins): adds alias to embeds() for bindings
  • dom/Document.h:
  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::width): (WebCore::HTMLDocument::height): (WebCore::HTMLDocument::dir): (WebCore::HTMLDocument::setDir): (WebCore::HTMLDocument::designMode): (WebCore::HTMLDocument::setDesignMode): (WebCore::HTMLDocument::bgColor): (WebCore::HTMLDocument::setBgColor): (WebCore::HTMLDocument::fgColor): (WebCore::HTMLDocument::setFgColor): (WebCore::HTMLDocument::alinkColor): (WebCore::HTMLDocument::setAlinkColor): (WebCore::HTMLDocument::linkColor): (WebCore::HTMLDocument::setLinkColor): (WebCore::HTMLDocument::vlinkColor): (WebCore::HTMLDocument::setVlinkColor): (WebCore::HTMLDocument::captureEvents): (WebCore::HTMLDocument::releaseEvents):
  • html/HTMLDocument.h:
  • html/HTMLDocument.idl:

WebKit:

Reviewed by Darin.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13830
Auto-generate JS DOM bindings for HTMLDocument and most of the rest
of HTMLElement

  • MigrateHeaders.make: add DOMHTMLDocumentPrivate.h
  • WebKit.xcodeproj/project.pbxproj:
3:58 PM Changeset in webkit [21680] by hyatt
  • 1 edit in trunk/WebCore/html/HTMLTokenizer.cpp
3:55 PM Changeset in webkit [21679] by hyatt
  • 2 edits in trunk/WebCore

Fix for <rdar://problem/5221838> REGRESSION (r21581): no content in idisk
window at mac.com.

Back out the change to make self-closed <script/> into a Dashboard quirk
only.

Reviewed by aroben

  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::parseTag):
1:53 PM Changeset in webkit [21678] by oliver
  • 2 edits in trunk/WebKit

Reviewed by Geoff.

Fix for <rdar://problem/5223782> REGRESSION: Can't drag and drop a standalone image

The main resource for a standalone image webarchive has the same mimetype as the
underlying image.

  • Misc/WebNSPasteboardExtras.mm: (-[NSPasteboard _web_writePromisedRTFDFromArchive:containsImage:]):
1:28 PM Changeset in webkit [21677] by thatcher
  • 2 edits in tags/WebCore-418.22/WebCore

Versioning.

1:27 PM Changeset in webkit [21676] by thatcher
  • 2 edits in tags/WebCore-418.22/WebCore

Merge fix r21674.

1:26 PM Changeset in webkit [21675] by thatcher
  • 1 copy in tags/WebCore-418.22

New tag.

1:24 PM Changeset in webkit [21674] by thatcher
  • 2 edits in branches/Safari-2-0-branch/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-10-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Eric.

<rdar://problem/4150976> Safari crashes in HTMLImageLoader::dispatchLoadEvent() on particular page (archive attached)

I couldn't figure out how to make an automated test for this; even once I got DumpRenderTree
to mouse in and out it did not crash.


  • khtml/html/html_imageimpl.cpp: (HTMLImageLoader::dispatchLoadEvent): Check if CachedImage is null before using it.
1:22 PM Changeset in webkit [21673] by thatcher
  • 2 edits in tags/WebCore-418.21/WebCore

Rolling out a commit on the wrong tag.

1:17 PM Changeset in webkit [21672] by thatcher
  • 2 edits in tags/WebCore-418.21/WebCore

Merged fix from TOT to Safari-2-0-branch

2005-10-05 Maciej Stachowiak <mjs@apple.com>

Reviewed by Eric.

<rdar://problem/4150976> Safari crashes in HTMLImageLoader::dispatchLoadEvent() on particular page (archive attached)

I couldn't figure out how to make an automated test for this; even once I got DumpRenderTree
to mouse in and out it did not crash.


  • khtml/html/html_imageimpl.cpp: (HTMLImageLoader::dispatchLoadEvent): Check if CachedImage is null before using it.
10:19 AM Changeset in webkit [21671] by kmccullo
  • 1 edit
    2 adds in trunk/LayoutTests

Reviewed by.

  • Fixed build breakage.
  • fast/dom/open-and-close-by-DOM-expected.txt: Added.
  • fast/dom/open-and-close-by-DOM.html: Added.
10:11 AM Changeset in webkit [21670] by pyeh
  • 2 edits in trunk/WebCore

<rdar://problem/5078453> AXNext/PrevLineEndTextMarkerForTextMarker returns kAXErrorNoValue for attachment range

Floating objects, such as aligned images, are not included as part of any line. So when the given
text marker is on the floating object, the returned line edge cannot be found and asserts are fired.
In addition, this paints an inconsistent text layout (in the accessibility sense) of where the floating
object resides. Character/word methods report floating object at the beginning of the text line(s). Yet
line methods never present floating objects anywhere.
To fix this, the ax line methods will now do more checking to ensure floating object nodes are included
when determining the line range/position. That way, assistive apps won't get stuck using the line methods
to read line-by-line

  • bridge/mac/WebCoreAXObject.mm: (updateAXLineStartForVisiblePosition): Added a helper method to adjust line start position to account for floating objects (-[WebCoreAXObject doAXLeftLineTextMarkerRangeForTextMarker:]): (-[WebCoreAXObject doAXRightLineTextMarkerRangeForTextMarker:]): (-[WebCoreAXObject doAXNextLineEndTextMarkerForTextMarker:]): (-[WebCoreAXObject doAXPreviousLineStartTextMarkerForTextMarker:]): Fixed up line routines to include floating object nodes in line position calculation. startOfLine/endOLine may return null for position next to a floating position. So now there's extra checks to prevent returning null line position/ranges back to AX.
10:02 AM Changeset in webkit [21669] by kmccullo
  • 1 edit
    2 deletes in trunk/LayoutTests

Reviewed by.

  • Removing failing tests until I've fixed them.
  • fast/dom/open-and-close-by-DOM-expected.txt: Removed.
  • fast/dom/open-and-close-by-DOM.html: Removed.
9:22 AM Changeset in webkit [21668] by hausmann
  • 9 edits in trunk

Added support for per-QWebPage network interface handlers, clean up the
http and file loading and fixed a bug in the HTTP redirect handling.

8:45 AM Changeset in webkit [23398] by sfalken
  • 2 edits in branches/WindowsMerge/WebKitWin

Build fix. Fix continuous rebuilding of Interfaces by turning off Generate Type Library for every IDL except WebKit.idl.

  • WebKit.vcproj/Interfaces.vcproj:
7:00 AM Changeset in webkit [21667] by rwlbuis
  • 10 edits
    4 adds in trunk

Reviewed by Dave Hyatt.

http://bugs.webkit.org/show_bug.cgi?id=13174
line-height in font shorthand does not override a previously stated line-height property

Make sure line-height is calculated against definite font-size and
uses the last set line-height, ie. directly or through font shorthand.

5:14 AM Changeset in webkit [21666] by lars
  • 4 edits in trunk/WebKitQt

Reviewed by Zack

Don't do HTTP downloads in a second thread. Simplifies
the code significantly and fixes crashes on some
Web pages.

4:07 AM Changeset in webkit [21665] by lars
  • 2 edits in trunk/WebCore

Reviewed by Zack and Valgrind

Initialize variable.

4:04 AM Changeset in webkit [21664] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by Darin.

http://bugs.webkit.org/show_bug.cgi?id=13784
REGRESSION (r13744-13750): Crash with empty gradient when drawing to canvas

Fix crash by testing for zero stops, and returning transparent black
in this case.

3:48 AM Changeset in webkit [21663] by rwlbuis
  • 3 edits
    4 adds in trunk

Reviewed by Darin.

http://bugs.webkit.org/show_bug.cgi?id=13801
Crash when loading nonexisting symbol

Check for referenced element and when not found just return.

3:01 AM Changeset in webkit [21662] by lars
  • 3 edits in trunk/WebKitQt

Fix Host: line in HTTP headers and resolving of relative URLs
when redirecting and clean up qHash forward declaration a bit.

1:48 AM Changeset in webkit [21661] by hausmann
  • 2 edits in trunk/WebKitQt

Fix signal/slot connection for load progress.

May 22, 2007:

10:37 PM Changeset in webkit [23397] by aliceli1
  • 5 edits in branches/WindowsMerge/WebKitWin

Reviewed by Oliver.


Enabling smartInsertDelete to resolve
<rdar://problem/5082891> editing/deleting/smart-delete-003.html (and 004) fail


  • Interfaces/IWebView.idl:
  • WebEditorClient.cpp: (WebEditorClient::smartInsertDeleteEnabled):
  • WebView.cpp: (WebView::WebView): (WebView::initWithFrame): (WebView::toggleSmartInsertDelete): (WebView::setSmartInsertDeleteEnabled): (WebView::smartInsertDeleteEnabled):
  • WebView.h:
10:02 PM Changeset in webkit [21660] by bdash
  • 1 edit in tags/Safari-5522.7.1/WebKit/Configurations/Version.xcconfig

Versioning.

9:52 PM Changeset in webkit [21659] by bdash
  • 3 edits in tags/Safari-5522.7.1/WebKit

Merge in r21576.

9:46 PM Changeset in webkit [21658] by bdash
  • 3 edits in tags/Safari-5522.7.1/WebKit

Merge in r21569.

9:05 PM Changeset in webkit [21657] by bdash
  • 1 copy in tags/Safari-5522.7.1

New tag.

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

Rubber stamped by Adam.

Fix license.

  • bindings/js/JSHTMLAnchorElementCustom.cpp:
6:26 PM Changeset in webkit [21655] by weinig
  • 13 edits in trunk

LayoutTests:

Reviewed by Adam.

Update tests for http://bugs.webkit.org/show_bug.cgi?id=13833
Add ObjC DOM binding for HTMLMarqeeElement

  • fast/dom/wrapper-classes-expected.txt:
  • fast/dom/wrapper-classes.html: HTMLMarqueeElement now is a DOMHTMLMarqueeElement

WebCore:

Reviewed by Adam.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13833
Add ObjC DOM binding for HTMLMarqeeElement

  • Also adds missing DOMHTMLFramePrivate.
  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/DOM.mm: (WebCore::createElementClassMap):
  • bindings/objc/DOMHTML.h:
  • bindings/objc/DOMInternal.h:
  • bindings/objc/DOMPrivate.h:
  • bindings/objc/PublicDOMInterfaces.h:

WebKit:

Reviewed by Adam.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13833
Add ObjC DOM binding for HTMLMarqeeElement

  • Also adds missing DOMHTMLFramePrivate.
  • MigrateHeaders.make:
6:19 PM Changeset in webkit [21654] by justing
  • 3 edits
    7 adds in trunk

LayoutTests:

Reviewed by darin


http://bugs.webkit.org/show_bug.cgi?id=13804
REGRESSION(r21611): Several new layout test pixel failures (misspelling markers)

  • editing/selection/13804-expected.checksum: Added.
  • editing/selection/13804-expected.png: Added.
  • editing/selection/13804-expected.txt: Added.
  • editing/selection/13804.html: Added. These were missing:
  • editing/selection/5076323-1-expected.png: Added.
  • editing/selection/5076323-2-expected.png: Added.
  • editing/selection/5076323-3-expected.png: Added.

WebCore:

Reviewed by darin, who found the fix independently.

http://bugs.webkit.org/show_bug.cgi?id=13804
REGRESSION(r21611): Several new layout test pixel failures (misspelling markers)


r21611 exposed a bug in the way removeMarkers computes
the end of the range to remove markers from. It also
introduced a bug: removeMarkers expects a startOffset
and a length, not a startOffset and an endOffset (added
a layout test).

  • dom/Document.cpp: (WebCore::Document::removeMarkers): The other removeMarkers expects a startOffset and a length, not a startOffset and an endOffset. (WebCore::Document::removeMarkers): Fixed the endOffset from length and startOffset calculation.
5:55 PM Changeset in webkit [21653] by hyatt
  • 2 adds in trunk/LayoutTests/fast/dom

Add layout test for better background dumping.

5:43 PM Changeset in webkit [21652] by hyatt
  • 6 edits in trunk/WebCore

Fix for bug 13802, background colors can't be reset to default
color. Fix the value cssText method to do the right thing for background
and background-position values.

Reviewed by beth

fast/dom/background-shorthand-csstext.html

  • css/CSSInitialValue.h: (WebCore::CSSInitialValue:::m_implicit): (WebCore::CSSInitialValue::isImplicitInitialValue):
  • css/CSSMutableStyleDeclaration.cpp: (WebCore::CSSMutableStyleDeclaration::getPropertyValue): (WebCore::CSSMutableStyleDeclaration::getLayeredShorthandValue):
  • css/CSSMutableStyleDeclaration.h:
  • css/CSSValue.h: (WebCore::CSSValue::isImplicitInitialValue):
  • css/cssparser.cpp: (WebCore::CSSParser::parseValue): (WebCore::CSSParser::parseBackgroundShorthand): (WebCore::CSSParser::parseShorthand):
4:47 PM Changeset in webkit [21651] by darin
  • 10 edits in trunk

LayoutTests:

Reviewed by Geoff.

  • test for <rdar://problem/5215830> REGRESSION: ebay.com layout is wrong
  • fast/dom/iframe-document.html: Updated test to expect no document property.
  • fast/dom/iframe-document-expected.txt: Updated.
  • fast/forms/focus2.html: Changed test to use ownerDocument instead of document.

WebCore:

Reviewed by Geoff.

  • fix <rdar://problem/5215830> REGRESSION: ebay.com layout is wrong

The code on ebay was looking for the "document" property on a iframe.
We changed that to be the ownerDocument rather than the contentDocument,
which was not what ebay was expecting. The best fix seems to be removing
the document property altogether.

Test: fast/dom/iframe-document.html

  • bindings/js/kjs_html.h: Remove ElementDocument.
  • bindings/js/kjs_html.cpp: (KJS::JSHTMLElement::getValueProperty): Ditto.

WebKit:

Reviewed by Geoff.

  • WebInspector/webInspector/treeoutline.js: Use ownerDocument instead of non-standard document property.
4:43 PM Changeset in webkit [21650] by adele
  • 3 edits in trunk/WebCore

Reviewed by Darin.

A little cleanup from my last checkin.

  • page/Chrome.h: Formatting change.
  • page/mac/ChromeMac.mm: Fixed copyright.
4:21 PM Changeset in webkit [21649] by adele
  • 6 edits
    4 adds in trunk

LayoutTests:

Reviewed by Darin.

Test for http://bugs.webkit.org/show_bug.cgi?id=13796
Crash in [NSInputContext currentInputContext] when you click on this map

  • fast/frames/frame-display-none-focus-expected.txt: Added.
  • fast/frames/frame-display-none-focus.html: Added.
  • fast/frames/resources/iframe-focus.html: Added.

WebCore:

Reviewed by Darin.

Fix for http://bugs.webkit.org/show_bug.cgi?id=13796
Crash in [NSInputContext currentInputContext] when you click on this map

This is another case of <rdar://problem/5171145> Safari crashed closing tab in NSInputContext updateInputContexts

Test: fast/frames/frame-display-none-focus.html

  • WebCore.xcodeproj/project.pbxproj: Added ChromeMac.mm
  • page/Chrome.h:
  • page/mac/ChromeMac.mm: Added. (WebCore::Chrome::focusNSView): Moved code from Widget::setFocus so it can be used whenever we need to make a view the first responder.
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::focusDocumentView): Calls focusNSView.
  • platform/mac/WidgetMac.mm: (WebCore::Widget::setFocus): Calls focusNSView.
4:18 PM Changeset in webkit [21648] by justing
  • 10 edits
    4 adds
    4 deletes in trunk

LayoutTests:

Reviewed by darin


<rdar://problem/4875189> List is deleted when replacing contents with Chinese/Japanese characters

Moved from unsupported-content because we've supported editing lists
for a while now:

  • editing/deleting/4875189-expected.checksum: Added.
  • editing/deleting/4875189-expected.png: Added.
  • editing/deleting/4875189-expected.txt: Added.
  • editing/deleting/4875189.html: Added.
  • editing/unsupported-content/list-delete-002-expected.checksum: Removed.
  • editing/unsupported-content/list-delete-002-expected.png: Removed.
  • editing/unsupported-content/list-delete-002-expected.txt: Removed.
  • editing/unsupported-content/list-delete-002.html: Removed.


Reflects new behavior:

  • editing/deleting/delete-select-all-003-expected.checksum:
  • editing/deleting/delete-select-all-003-expected.png:
  • editing/deleting/delete-select-all-003-expected.txt:


Ditto and also removed a FIXME:

  • editing/deleting/list-item-1-expected.checksum:
  • editing/deleting/list-item-1-expected.png:
  • editing/deleting/list-item-1-expected.txt:
  • editing/deleting/list-item-1.html:

WebCore:

Reviewed by darin


<rdar://problem/4875189>
List is deleted when replacing contents with Chinese/Japanese characters

  • editing/htmlediting.cpp: (WebCore::isSpecialElement): Lists are no longer special. They can be removed by removing their content and then then hitting delete inside of the empty list item that's left over. Even when that list item is the first thing in the editable region.
4:02 PM Changeset in webkit [21647] by adele
  • 2 edits in trunk/WebKit

Reviewed by Darin.

Adding some asserts to help detect other cases of <rdar://problem/5171145>

  • WebCoreSupport/WebFrameBridge.mm: (-[WebFrameBridge makeFirstResponder:]):
3:59 PM Changeset in webkit [21646] by darin
  • 3 edits in trunk/WebCore

Reviewed by Tim Hatcher.

  • fix <rdar://problem/5206089> REGRESSION: hovering over amazon.com "product categories" tab makes page content disappear

The contentWindow property was doing a cross-frame security check, but it shouldn't.
Window objects are responsible for protecting themselves from cross-frame activity.
Because contentWindow returned undefined, we then ended up setting visibility on the
body of the main frame to "hidden".

No layout test yet. When we make cross-domain tests, we should include this one.

  • html/HTMLFrameElement.idl: Removed CheckFrameSecurity attribute from contentWindow.
  • html/HTMLIFrameElement.idl: Ditto.
3:53 PM Changeset in webkit [23396] by adachan
  • 2 edits in branches/WindowsMerge/WebCore

2007-05-22 Ada Chan <adachan@apple.com>

Fix build.

  • plugins/win/PluginViewWin.cpp: (WebCore::PluginViewWin::handleMouseEvent):
3:52 PM Changeset in webkit [21645] by kmccullo
  • 1 edit
    4 adds in trunk/LayoutTests

Reviewed by.

  • Forgot to checkin for fix for rdar://problem/4981886
  • fast/dom/open-and-close-by-DOM-expected.txt: Added.
  • fast/dom/open-and-close-by-DOM.html: Added.
  • fast/dom/resources/close.html: Added.
  • fast/dom/resources/middle.html: Added.
  • http/tests/security/local-image-from-remote.html:
2:57 PM Changeset in webkit [23395] by aliceli1
  • 2 edits in branches/WindowsMerge/LayoutTests

Reviewed by.

  • win/Skipped: removed fixed tests
2:38 PM Changeset in webkit [23394] by justing
  • 4 edits in branches/WindowsMerge

WebCoreWin:

Reviewed by oliver

Build fix.

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

WebKitWin:

Reviewed by oliver


Build fix.

  • WebView.cpp: (WebView::clearFocusNode):
2:17 PM Changeset in webkit [23393] by sfalken
  • 4 edits in branches/WindowsMerge/WebKitWin

Reviewed by Ada.


Added 3 view-related methods to IWebFramePrivate.

  • Interfaces/IWebFramePrivate.idl: Added 3 new methods
  • WebFrame.cpp: (WebFrame::hasScrollBars): Added. (WebFrame::contentBounds): Added. (WebFrame::frameBounds): Added.
  • WebFrame.h:
1:49 PM Changeset in webkit [21644] by yongjzha
  • 4 edits in S60/trunk/JavaScriptCore

2007-05-22 bujtas <zbujtas@gmail.com>

Reviewed by Yongjun
DESC: fix out of boundary crah in prce
http://bugs.webkit.org/show_bug.cgi?id=13826

  • pcre/internal.h:
  • pcre/pcre.c: (check_escape): (is_counted_repeat): (read_repeat_counts): (check_posix_syntax): (compile_branch): (pcre_compile): (match):
  • pcre/study.c: (set_bit): (set_start_bits):
1:43 PM Changeset in webkit [21643] by yongjzha
  • 4 edits in S60/branches/3.1m/JavaScriptCore

2007-05-22 bujtas <zbujtas@gmail.com>

Reviewed by Yongjun
DESC: fix out of boundary crah in prce
http://bugs.webkit.org/show_bug.cgi?id=13826

  • pcre/internal.h:
  • pcre/pcre.c: (check_escape): (is_counted_repeat): (read_repeat_counts): (check_posix_syntax): (compile_branch): (pcre_compile): (match):
  • pcre/study.c: (set_bit): (set_start_bits):
12:02 PM Changeset in webkit [21642] by justing
  • 11 edits
    4 adds in trunk

LayoutTests:

Reviewed by oliver

<rdar://problem/5213963>
REGRESSION(r21467): Active selection remains in text field after clicking on a button

Demonstrates bug:

  • editing/selection/5213963-expected.checksum: Added.
  • editing/selection/5213963-expected.png: Added.
  • editing/selection/5213963-expected.txt: Added.
  • editing/selection/5213963.html: Added. We now clear the selection on mouse down (but not before the mousedown event is fired):
  • fast/forms/focus-selection-input-expected.txt:
  • fast/forms/focus-selection-textarea-expected.txt:

WebCore:

Reviewed by oliver

<rdar://problem/5213963>
REGRESSION(r21467): Active selection remains in text field after clicking on a button

  • page/EventHandler.cpp: (WebCore::EventHandler::dispatchMouseEvent): Call FocusController::setFocusedNode, because Document::setFocusedNode no longer clears the selection. Begin passing the new focused frame to FocusController::setFocusedNode, because when the new focus node is null, the selection should only be cleared if the focused frame isn't changing. (WebCore::Element::focus): Begin passing the new focused frame. (WebCore::Element::blur): Ditto.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::focus): Ditto.
  • html/HTMLTextAreaElement.cpp: (WebCore::HTMLTextAreaElement::focus): Ditto.
  • page/Frame.cpp: (WebCore::Frame::setFocusedNodeIfNeeded): Ditto.
  • page/FocusController.cpp: (WebCore::clearSelectionIfNeeded): Take in the new focused frame. If it's not changing, don't clear the selection. (WebCore::FocusController::setFocusedNode): Take in the new focused frame.
  • page/FocusController.h:
12:02 PM Changeset in webkit [21641] by beidson
  • 2 edits in trunk/WebKit

Reviewed by Kevin (Sarge)

<rdar://problem/5219885> - Crash after closing a inline popup ad at http://news.yahoo.com/

This regressed in http://trac.webkit.org/projects/webkit/changeset/21618

  • WebView/WebFrame.mm: (-[WebFrame dataSource]): Null check the frameloader
8:11 AM Changeset in webkit [21640] by staikos
  • 2 edits in trunk/WebKitQt

back this out for zack

1:53 AM Changeset in webkit [21639] by zack
  • 2 edits in trunk/WebCore

Path implementing more Cliboard from Marius Bugge Monsen.

Adding implementations of some of the missing methods.
r=me

12:31 AM Changeset in webkit [21638] by hausmann
  • 2 edits in trunk/WebKitQt

Fix http headers for POST.

May 21, 2007:

11:53 PM Changeset in webkit [21637] by justing
  • 2 edits in trunk/LayoutTests/editing/execCommand

Updating expected results for r21074.

8:50 PM Changeset in webkit [23392] by sfalken
  • 4 edits in branches/WindowsMerge/WebKitWin

Reviewed by Adam, Ada.


Added IWebFramePrivate::size().

  • Interfaces/IWebFramePrivate.idl: Added size.
  • WebFrame.cpp: (WebFrame::size): Added.
  • WebFrame.h: Added size.
8:10 PM Changeset in webkit [21636] by oliver
  • 4 edits
    4 adds in trunk

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

LayoutTests:

Reviewed by Sam.


Layout test for http://bugs.webkit.org/show_bug.cgi?id=13789

  • svg/custom/getscreenctm-in-mixed-content-expected.checksum: Added.
  • svg/custom/getscreenctm-in-mixed-content-expected.png: Added.
  • svg/custom/getscreenctm-in-mixed-content-expected.txt: Added.
  • svg/custom/getscreenctm-in-mixed-content.xhtml: Added.

WebCore:

Reviewed by Sam.


Fix for http://bugs.webkit.org/show_bug.cgi?id=13789
-- SVGLocatable::getScreenCTM() faulty


Needed to update SVGLength to handle the case where the SVGElement is
not the root document element -- we do this by falling back on the
renderer for the context.


For <svg> elements embedded as mixed content in xhtml we consider the
absolutePosition of the parent to be the origin for the <svg> element.

  • ksvg2/svg/SVGLength.cpp: (WebCore::SVGLength::PercentageOfViewport):
  • ksvg2/svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::getScreenCTM):
7:14 PM Changeset in webkit [23391] by weinig
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Oliver.

Build fix.

  • WebCore.vcproj/WebCore.vcproj:
7:14 PM Changeset in webkit [21635] by thatcher
  • 2 edits in trunk/WebCore

Reviewed by Adam.

<rdar://problem/5084400> -[DOMRange lineBoxRects] returns an empty array when the DOM nodes have just been created

Do a updateLayoutIgnorePendingStylesheets before asking the Node or Range for it's box rects.

  • bindings/objc/DOM.mm: (-[DOMNode boundingBox]): (-[DOMNode lineBoxRects]): (-[DOMRange boundingBox]): (-[DOMRange lineBoxRects]):
6:42 PM Changeset in webkit [21634] by weinig
  • 2 edits in trunk/WebCore

Reviewed by George Staikos.

Fix Qt build.

  • WebCore.pro: add missing IDL files.
6:29 PM Changeset in webkit [21633] by weinig
  • 14 edits
    1 add in trunk

LayoutTests:

Reviewed by Darin.

  • fast/dom/Window/element-constructors-on-window-expected.txt:
  • fast/dom/Window/element-constructors-on-window.html:
  • fast/dom/Window/window-properties-expected.txt:

WebCore:

Reviewed by Darin.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13803
Autogenerate the JS bindings for the HTMLMarqueeElement.

  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLElementWrapperFactory.cpp:
  • bindings/js/kjs_css.cpp: (KJS::DOMCSSRule::getOwnPropertySlot): Fix possible null pointer issue. (KJS::DOMCSSRule::put): Fix possible null pointer issue.
  • bindings/js/kjs_html.cpp: (KJS::): (KJS::JSHTMLElement::classInfo): (KJS::JSHTMLElement::accessors): (KJS::JSHTMLElement::getOwnPropertySlot): Fix null pointer issue. (KJS::HTMLElementFunction::callAsFunction): (KJS::JSHTMLElement::put):
  • bindings/js/kjs_html.h: (KJS::JSHTMLElement::):
  • html/HTMLMarqueeElement.cpp: (WebCore::HTMLMarqueeElement::start): (WebCore::HTMLMarqueeElement::stop):
  • html/HTMLMarqueeElement.h: Add start() and stop() functions.
  • html/HTMLMarqueeElement.idl: Added.
  • page/DOMWindow.idl: Add marquee constructor.
5:08 PM Changeset in webkit [21632] by adele
  • 4 edits in trunk

WebCore:

Fix by Darin, reviewed by me.

Fix for <rdar://problem/5171145> Safari crashed closing tab in NSInputContext updateInputContexts

  • platform/mac/WidgetMac.mm: (WebCore::safeRemoveFromSuperview): Added. If the view is the window's first responder, then set the window's first responder to nil so we don't leave the window pointing to a view that's no longer in it. (WebCore::Widget::setFocus): Don't attempt to focus the view if it has no window. (WebCore::Widget::removeFromSuperview): Call safeRemoveFromSuperview. (WebCore::Widget::afterMouseDown): ditto.

WebKit:

Fix by Darin, reviewed by me.

Fix for <rdar://problem/5171145> Safari crashed closing tab in NSInputContext updateInputContexts

  • WebView/WebFrameView.mm: (-[WebFrameView _setDocumentView:]): If the old view is the first responder, then set the window's first responder to nil so we don't leave the window pointing to a view that's no longer in it.
4:13 PM Changeset in webkit [21631] by weinig
  • 4 edits in trunk/LayoutTests
3:03 PM Changeset in webkit [21630] by hyatt
  • 3 edits in trunk/WebCore

Make a line gutter backdrop so that tiny view source docs don't look
bad.

Reviewed by aroben

  • css/view-source.css:
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::createContainingTable):
2:45 PM Changeset in webkit [23390] by adachan
  • 4 edits in branches/WindowsMerge/WebKitWin

2007-05-21 Ada Chan <adachan@apple.com>

Reviewed by Steve.

Added IWebViewPrivate::loadBackForwardListFromOtherView(). It replicates the backforward
list from another webview and loads that in the current webview.

  • Interfaces/IWebViewPrivate.idl:
  • WebView.cpp: (WebView::loadBackForwardListFromOtherView):
  • WebView.h:
2:32 PM Changeset in webkit [21629] by hyatt
  • 4 edits in trunk/WebCore

Make sure to distinguish between links to internal resources and
real links (anchors) in view source mode.

Reviewed by aroben

  • css/view-source.css:
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::addViewSourceToken): (WebCore::HTMLViewSourceDocument::addLink):
  • html/HTMLViewSourceDocument.h:
1:00 PM Changeset in webkit [23389] by weinig
  • 2 edits in branches/WindowsMerge/WebCore

Reviewed by Adam.

Build fix.

  • WebCore.vcproj/WebCore.vcproj:
12:37 PM S60CheckoutAndBuild edited by bradley.morrison@nokia.com
(diff)
12:35 PM S60IDE created by bradley.morrison@nokia.com
initial revision
12:07 PM Changeset in webkit [21628] by beidson
  • 2 edits in trunk/WebKit

Making the importance of my last change more clear

  • WebView/WebViewPrivate.h: "Leave for Dashboard, people!"
11:55 AM S60ConfigureComputer edited by bradley.morrison@nokia.com
link to s60IDE page (diff)
11:45 AM Changeset in webkit [21627] by beidson
  • 3 edits in trunk/WebKit

Reviewed by Kevin (Sarge)

<rdar://problem/5217124> - Re-add mistakenly removed SPI

  • WebView/WebView.mm: (-[WebView handleAuthenticationForResource:challenge:fromDataSource:]):
  • WebView/WebViewPrivate.h:
11:32 AM Changeset in webkit [21626] by spadma
  • 2 edits in S60/branches/3.1m/WebKit

spadma

DESC: Update loc file layout


  • data/Webkit.loc:
11:32 AM Changeset in webkit [21625] by spadma
  • 2 edits in S60/trunk/WebKit

spadma

DESC: Update loc file layout


  • data/Webkit.loc:
11:19 AM Changeset in webkit [21624] by weinig
  • 17 edits in trunk

LayoutTests:

Reviewed by Adam and Darin.

  • fast/dom/Window/element-constructors-on-window-expected.txt:
  • fast/dom/Window/window-properties-expected.txt:

WebCore:

Reviewed by Adam and Darin.

Patch for http://bugs.webkit.org/show_bug.cgi?id=13791
Autogenerate the JS bindings for the HTMLIFrameElement and HTMLFrameElement.

  • DerivedSources.make:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSHTMLAnchorElementCustom.cpp:
  • bindings/js/JSHTMLElementWrapperFactory.cpp:
  • bindings/js/kjs_html.cpp: (KJS::): (KJS::JSHTMLElement::classInfo): (KJS::JSHTMLElement::accessors): (KJS::HTMLElementFunction::callAsFunction):
  • bindings/js/kjs_html.h: (KJS::JSHTMLElement::):
  • bindings/scripts/CodeGeneratorJS.pm:
  • html/HTMLFrameElement.idl:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::location):
  • html/HTMLFrameElementBase.h:
  • html/HTMLFrameOwnerElement.h: (WebCore::HTMLFrameOwnerElement::contentWindow):
  • html/HTMLIFrameElement.idl:
  • page/DOMWindow.idl:
10:57 AM Changeset in webkit [21623] by andersca
  • 2 edits in trunk/WebKit

Reviewed by Ada.

<rdar://problem/5200816> REGRESSION: With Shiira 1.2.2 , I can't open embedded link in flash object by clicking (http:/www.adobe.com )


Null check the request.


  • WebView/WebView.mm: (+[WebView _canHandleRequest:]):
6:57 AM Changeset in webkit [21622] by lars
  • 8 edits
    6 adds in trunk/LayoutTestResults

Updating and adding test cases.

6:36 AM Changeset in webkit [21621] by lars
  • 4 edits in trunk/WebKitQt

Reviewed by Simon

Remove the userHandle methods from QWebnetworkJob again.
They don't really give us anything and just clutter the API.

6:09 AM Changeset in webkit [21620] by hausmann
  • 2 edits in trunk/WebKitQt

Export the new qt network API.

5:33 AM Changeset in webkit [21619] by lars
  • 5 edits
    3 adds
    4 deletes in trunk

Reviewed by Zack.

Add an API layer for network downloads. Basically QWebnetworkInterface
is an interface class for downloading resources. QWebnetworkJob describes
the actual object to download.

QWebNetworkInterface has a default implementation that replaces the
old ResourceHandleManager class in the Qt port.

Remove the ResourceHandleManager class, it is now part of
QWebNetworkInterface. Adapt ResourceHandle to the new way
of things.

4:04 AM Changeset in webkit [21618] by mjs
  • 15 edits in trunk/WebKit

Reviewed by Geoff.


<rdar://problem/5205358> REGRESSION (r21367): All messages appear entirely blank when running Mail off of tip of tree WebKit


The fix is to return nil from [WebFrame dataSource] when it has not loaded anything but the fake
empty initial document. However, WebKit still needs the real data source internally, so I also
added a [WebFrame _dataSource] method that skips this check, and made WebKit use it throughout.

  • Misc/WebNSAttributedStringExtras.mm: (fileWrapperForElement):
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView dataSource]):
  • Plugins/WebNullPluginView.mm: (-[WebNullPluginView viewDidMoveToWindow]):
  • Plugins/WebPluginController.mm: (-[WebPluginController URLPolicyCheckReferrer]):
  • WebCoreSupport/WebFrameBridge.mm: (-[WebFrameBridge dataSource]): (-[WebFrameBridge redirectDataToPlugin:]):
  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::makeDocumentView): (WebFrameLoaderClient::forceLayoutForNonHTML): (WebFrameLoaderClient::prepareForDataSourceReplacement): (WebFrameLoaderClient::canCachePage):
  • WebCoreSupport/WebViewFactory.mm: (-[WebViewFactory bridgeForView:]):
  • WebKit.xcodeproj/project.pbxproj:
  • WebView/WebArchiver.mm: (+[WebArchiver archiveFrame:]): (+[WebArchiver archiveMainResourceForFrame:]): (+[WebArchiver _archiveWithMarkupString:fromFrame:nodes:]): (+[WebArchiver archiveSelectionInFrame:]):
  • WebView/WebFrame.mm: (-[WebFrame _loadURL:referrer:intoChild:]): (-[WebFrame _addChild:]): (-[WebFrame _dataSource]): (-[WebFrame DOMDocument]): (-[WebFrame dataSource]):
  • WebView/WebFrameInternal.h:
  • WebView/WebHTMLView.mm: (-[WebHTMLView _setPrinting:minimumPageWidth:maximumPageWidth:adjustViewSize:]):
  • WebView/WebRenderNode.mm: (-[WebRenderNode initWithWebFrameView:]):
  • WebView/WebView.mm: (-[WebView _mainFrameOverrideEncoding]): (-[WebView mainFrameURL]): (-[WebView mainFrameTitle]): (-[WebView mainFrameIcon]): (-[WebView validateUserInterfaceItemWithoutDelegate:]): (-[WebView replaceSelectionWithArchive:]): (-[WebView _isLoading]): (-[WebView _performTextSizingSelector:withObject:onTrackingDocs:selForNonTrackingDocs:newScaleFactor:]): (-[WebView _notifyTextSizeMultiplierChanged]):
12:40 AM Changeset in webkit [21617] by hyatt
  • 28 edits in trunk/LayoutTests/tables

Update test results for improved colgroup span handling.

12:31 AM Changeset in webkit [21616] by hyatt
  • 4 adds in trunk/LayoutTests/fast/table

Add new layout test for spanning colgroup rules=groups bug.

12:30 AM Changeset in webkit [21615] by hyatt
  • 5 edits in trunk/WebCore

Second half of fix for 13793, make sure rules=groups works properly with
spanning colgroups.

Also make sure colgroups get an actual span of 1 (instead of incorrectly
giving them a span of 0).

Reviewed by mitz

fast/table/colgroup-spanning-groups-rules.html

  • html/HTMLTableColElement.cpp: (WebCore::HTMLTableColElement::HTMLTableColElement):
  • rendering/RenderTable.cpp: (WebCore::RenderTable::colElement):
  • rendering/RenderTable.h:
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::collapsedLeftBorder): (WebCore::RenderTableCell::collapsedRightBorder):
Note: See TracTimeline for information about the timeline view.