Timeline



Apr 7, 2009:

10:33 PM Changeset in webkit [42313] by mrowe@apple.com
  • 1 edit in trunk/WebKitTools/BuildSlaveSupport/build.webkit.org-config

Ignore passwords.json to prevent it from accidentally being checked in.

10:29 PM Changeset in webkit [42312] by mrowe@apple.com
  • 4 edits
    1 move
    5 adds
    1 delete in trunk/WebKitTools

Land the current build.webkit.org configuration for safekeeping.

  • BuildSlaveSupport/build.webkit.org-config/README: Added.
  • BuildSlaveSupport/build.webkit.org-config/Makefile:
  • BuildSlaveSupport/build.webkit.org-config/buildbot.tac:
  • BuildSlaveSupport/build.webkit.org-config/config.json: Added.
  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
  • BuildSlaveSupport/build.webkit.org-config/public_html/buildbot.css: Renamed from BuildSlaveSupport/build.webkit.org-config/buildbot.css.
  • BuildSlaveSupport/build.webkit.org-config/public_html/index.html: Added.
  • BuildSlaveSupport/build.webkit.org-config/public_html/robots.txt: Added.
  • BuildSlaveSupport/build.webkit.org-config/webkit/init.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/auth.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/basesteps.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/builders.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/factories.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/schedulers.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/status.py: Removed.
  • BuildSlaveSupport/build.webkit.org-config/webkit/steps.py: Removed.
10:08 PM Changeset in webkit [42311] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Ignore the mysterious exception that Cygwin's Python is throwing when attempting
to remove the layout-test-results directory.

  • BuildSlaveSupport/test-result-archive:
9:54 PM Changeset in webkit [42310] by kevino@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Kevin Ollivier.

Remove no longer needed debug warning and Windows-specific codepaths
(as we now use the Safari/Win port's timer there)

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

9:34 PM Changeset in webkit [42309] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Change the working directory before removing the test result directory
in the hopes of making the Windows build slaves happier.

  • BuildSlaveSupport/test-result-archive:
9:30 PM Changeset in webkit [42308] by mrowe@apple.com
  • 3 edits in trunk/WebKitTools

Make the links to tests in the uploaded results.html files on build.webkit.org point to the test files.

Reviewed by Adam Roben.

  • Scripts/VCSUtils.pm: Add a function that will return the location of a file relative to the root of the working copy.
  • Scripts/run-webkit-tests: Convert the path in to a remote URL if the path is below the layout tests directory.
8:36 PM Changeset in webkit [42307] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Reviewed by Adam Roben.

  • make this test insensitive to platform differences in Thai word breaking dictionaries
  • editing/selection/thai-word-at-document-end.html:
8:21 PM Changeset in webkit [42306] by Adam Roben
  • 2 edits in trunk/WebCore

Fix many <video> regression tests on Windows

Reviewed by Dave Hyatt.

  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWin::getNaturalSize): Call GetMovieNaturalBoundsRect, which returns the movie's natural bounds (duh), rather than GetMovieBox, which returns the movie's current displayed size/position.
8:05 PM Changeset in webkit [42305] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Remove the layout-test-results directory after we have archived it so that
log files and results are reset between runs of the tests.

  • BuildSlaveSupport/test-result-archive:
7:09 PM Changeset in webkit [42304] by Adam Roben
  • 1 edit
    6 adds in trunk/LayoutTests

Check in files that were forgotten in r42293

  • platform/win/http/tests/xmlhttprequest/web-apps/012-expected.txt:
  • platform/win/http/tests/xmlhttprequest/web-apps/013-expected.txt:
6:00 PM Changeset in webkit [42303] by beidson@apple.com
  • 3 edits in trunk/LayoutTests

2009-04-07 Brady Eidson <beidson@apple.com>

Update layout test results for new exceptions.

  • fast/dom/Window/window-properties-expected.txt:
  • platform/mac-leopard/fast/dom/Window/window-properties-expected.txt:
5:26 PM Changeset in webkit [42302] by beidson@apple.com
  • 6 edits
    4 adds in trunk

WebCore:

2009-04-07 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler and Sam Weinig

<rdar://problem/5968249> "Private Browsing" doesn't affect HTML 5 LocalStorage

The expected behavior for persistent storage when private browsing mode is undefined by the spec.
For now we're going with a "read-only when in private browsing" policy until we can get feedback
and/or get the behavior specified in the spec.

Note that I purposefully made the change to restrict SessionStorage to read-only, also, with the
understanding that the spec allows for SessionStorage to persist across relaunches, even though
our implementation currently doesn't do this.

  • dom/DOMCoreException.idl: Add some new ExceptionCodes defined in HTML5, one of which is needed for LocalStorage changes (QUOTA_EXCEEDED_ERR)
  • dom/ExceptionCode.cpp:
  • dom/ExceptionCode.h:
  • storage/StorageArea.cpp: (WebCore::StorageArea::internalSetItem): If private browsing is enabled, throw the QUOTA_EXCEEDED_ERR exception as the spec allows. (WebCore::StorageArea::internalRemoveItem): If private browsing is enabled, silently fail to remove the item. (WebCore::StorageArea::internalClear): If private browsing is enabled, silently fail to clear the area.

LayoutTests:

2009-04-07 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

Tests for <rdar://problem/5968249> "Private Browsing" doesn't affect HTML 5 LocalStorage

  • storage/domstorage/localstorage/private-browsing-affects-storage-expected.txt: Added.
  • storage/domstorage/localstorage/private-browsing-affects-storage.html: Added.
  • storage/domstorage/sessionstorage/private-browsing-affects-storage-expected.txt: Added.
  • storage/domstorage/sessionstorage/private-browsing-affects-storage.html: Added.
5:18 PM Changeset in webkit [42301] by dino@apple.com
  • 8 edits in trunk/WebCore

2009-04-07 Dean Jackson <dino@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6148255>

Standalone media documents can not always handle everything
using a <video> element. In these cases switch to an <embed>
element that uses the QuickTime plugin, in the hope that it
will have more luck. Typical media types that trigger this
are streamed media and QTVR.

Equivalent Windows fix to come once reviewed. Note also that
this area of code needs a cleanup:
https://bugs.webkit.org/show_bug.cgi?id=25085

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::mediaPlayerSawUnsupportedTracks):
  • html/HTMLMediaElement.h:
  • loader/MediaDocument.cpp: (WebCore::MediaDocument::mediaElementSawUnsupportedTracks): (WebCore::MediaDocument::replaceVideoWithEmbed):
  • loader/MediaDocument.h:
    • removes the <video> element and puts in an <embed>
  • platform/graphics/MediaPlayer.h: (WebCore::MediaPlayerClient::mediaPlayerSawUnsupportedTracks):
    • new internal method for flagging a problem
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::updateStates):
    • looks for the types of content we don't handle

(WebCore::MediaPlayerPrivate::rateChanged):
(WebCore::MediaPlayerPrivate::sizeChanged):
(WebCore::MediaPlayerPrivate::timeChanged):
(WebCore::MediaPlayerPrivate::didEnd):
(WebCore::MediaPlayerPrivate::disableUnsupportedTracks):
(WebCore::MediaPlayerPrivate::sawUnsupportedTracks):

  • inform the MediaClient that we have a potential problem
5:00 PM Changeset in webkit [42300] by andersca@apple.com
  • 4 edits in trunk/WebKit/mac

2009-04-07 Anders Carlsson <andersca@apple.com>

Fix Tiger build for real this time.

  • Plugins/WebNetscapePluginView.h:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView getVariable:forURL:value:length:]): (-[WebNetscapePluginView setVariable:forURL:value:length:]): (-[WebNetscapePluginView getAuthenticationInfoWithProtocol:host:port:scheme:realm:username:usernameLength:password:passwordLength:]):
  • Plugins/npapi.mm: (NPN_GetValueForURL): (NPN_SetValueForURL):
4:59 PM Changeset in webkit [42299] by mrowe@apple.com
  • 1 edit
    1 copy in trunk/WebKitTools

Helper script for uploading layout test results for display on build.webkit.org.

  • BuildSlaveSupport/test-result-archive: Copied from WebKitTools/BuildSlaveSupport/built-product-archive.
4:38 PM Changeset in webkit [42298] by mitz@apple.com
  • 3 edits in trunk/WebCore

Reviewed by Darin Adler.

  • fix <rdar://problem/6767428> Resurrect +[WebFontCache fontWithFamily:traits:size:] for clients that call it directly

WebFontCache is a class in the WebCore framework and is not WebKit API,
but apparently (perhaps mistakenly) some clients call this private
interface.

This change resurrects the interface and gives it reasonable behavior
(however, note that WebCore will not call the old interface, so hacks that
override it will have no effect on WebKit clients).

  • platform/mac/WebFontCache.h:
  • platform/mac/WebFontCache.mm: (+[WebFontCache fontWithFamily:traits:size:]): Added. Call through to +fontWithFamily:traits:weight:size:.
4:09 PM Changeset in webkit [42297] by hyatt@apple.com
  • 20 edits in trunk

WebCore:

2009-04-07 David Hyatt <hyatt@apple.com>

Reviewed by Adam Roben

Fix ScrollView's algorithm that determines whether or not to show scrollbars. There was a bug
with the method mainly because RenderView will size itself to the viewport's size, making it
impossible to really know the document's "true" size.

In order to solve this problem ScrollView now has a minimumContentsSize method. This method
returns the document size excluding the RenderView from the calculation. This width/height
is now cached on RenderView in m_docWidth/m_docHeight member variables.

  • WebCore.base.exp:
  • page/FrameView.cpp: (WebCore::FrameView::adjustPageHeight): (WebCore::FrameView::minimumContentsSize):
  • page/FrameView.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::updateScrollbars):
  • platform/ScrollView.h: (WebCore::ScrollView::minimumContentsSize):
  • rendering/RenderView.cpp: (WebCore::RenderView::layout): (WebCore::RenderView::calcDocHeight): (WebCore::RenderView::calcDocWidth):
  • rendering/RenderView.h: (WebCore::RenderView::docWidth): (WebCore::RenderView::docHeight):

WebKit/mac:

2009-04-07 David Hyatt <hyatt@apple.com>

Reviewed by Adam Roben

Mac portion of fix to make DumpRenderTree always produce accurate scrollbar results. Change
updateScrollers to call minimumContentsSize when a WebHTMLView is inside the WebDynamicScrollbarsView.

  • WebView/WebDynamicScrollBarsView.m: (-[WebDynamicScrollBarsView updateScrollers]):

LayoutTests:

2009-04-07 David Hyatt <hyatt@apple.com>

Reviewed by Adam Roben

Land updated layout test results now that DRT is doing the right thing
with scrollbar results.

  • platform/mac/css2.1/t0803-c5502-mrgn-r-02-c-expected.checksum:
  • platform/mac/css2.1/t0803-c5502-mrgn-r-02-c-expected.png:
  • platform/mac/css2.1/t0803-c5502-mrgn-r-02-c-expected.txt:
  • platform/mac/css2.1/t0803-c5505-mrgn-02-c-expected.checksum:
  • platform/mac/css2.1/t0803-c5505-mrgn-02-c-expected.png:
  • platform/mac/css2.1/t0803-c5505-mrgn-02-c-expected.txt:
  • platform/mac/fast/block/float/013-expected.checksum:
  • platform/mac/fast/block/float/013-expected.png:
  • platform/mac/fast/block/float/013-expected.txt:
3:56 PM Changeset in webkit [42296] by Dimitri Glazkov
  • 2 edits in trunk/WebCore

2009-04-07 Dimitri Glazkov <Dimitri Glazkov>

Build fix, not reviewed.

  • WebCore.xcodeproj/project.pbxproj: Made ScriptObject.h private.
3:37 PM Changeset in webkit [42295] by Dimitri Glazkov
  • 15 edits
    1 copy in trunk/WebCore

2009-04-07 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=25063
Refactor InspectorController to use ScriptObject/State.

  • GNUmakefile.am: Added ScriptState.cpp.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • WebCoreSources.bkl: Ditto.
  • bindings/js/JSInspectorControllerCustom.cpp: (WebCore::JSInspectorController::addSourceToFrame): Fixed argument index typo.
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call): Added reportExceptions parameter. (WebCore::ScriptFunctionCall::construct): Ditto.
  • bindings/js/ScriptFunctionCall.h: Ditto.
  • bindings/js/ScriptObject.cpp: (WebCore::handleException): Added exception-reporting heloper. (WebCore::ScriptObject::set): Changed to use handleException. (WebCore::ScriptGlobalObject::set): Ditto,
  • bindings/js/ScriptObject.cpp: Added ScriptGlobalObject. (WebCore::ScriptGlobalObject::set): Added. (WebCore::ScriptGlobalObject::getObject): Added.
  • bindings/js/ScriptObject.h: Added ScriptGlobalObject decls. (WebCore::ScriptGlobalObject::ScriptGlobalObject): Added.
  • bindings/js/ScriptState.cpp: Added.
  • bindings/js/ScriptState.h: Added scriptStateFromPage decl.
  • inspector/ConsoleMessage.cpp: (WebCore::ConsoleMessage::addToConsole): Changed to not report exceptions

to avoid re-entrancy.

  • inspector/InspectorController.cpp: Refactored to use ScriptState/Object.
  • inspector/InspectorController.h: Ditto.
3:30 PM Changeset in webkit [42294] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-07 Adam Langley <agl@google.com>

Reviewed by Darin Fisher.

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

Skia was previously calculating the leading value incorrectly, leading
us to add fHeight to get the height of a line of text. Now that Skia
is calculating fLeading correctly, we can remove fHeight.

This doesn't affect any layout tests.

  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformInit):
3:27 PM Changeset in webkit [42293] by Adam Roben
  • 4 edits in trunk

WebCore:

2009-04-07 Brian Weinstein <bweinstein@gmail.com>

Reviewed by Darin Adler.

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

This fix removes the status code from the status message in a ResourceResponse, which allows a few more tests in http to pass.

  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::platformLazyInit):

LayoutTests:

2009-04-07 Brian Weinstein <bweinstein@gmail.com>

Reviewed by Darin Adler.

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

Tests in http/tests/xmlhttprequest/web-apps were taking results from
platform/mac instead of LayoutTests, so added the values from LayoutTests
into platform/win, and removed a few tests from the skip list that are fixed
by the fix to platform/network/cf/ResourceResponseCFNet.cpp

  • platform/win/Skipped:
  • platform/win/http: Added.
  • platform/win/http/tests: Added.
  • platform/win/http/tests/xmlhttprequest: Added.
  • platform/win/http/tests/xmlhttprequest/web-apps: Added.
  • platform/win/http/tests/xmlhttprequest/web-apps/012-expected.txt: Copied from LayoutTests/http/tests/xmlhttprequest/web-apps/012-expected.txt.
  • platform/win/http/tests/xmlhttprequest/web-apps/013-expected.txt: Copied from LayoutTests/http/tests/xmlhttprequest/web-apps/013-expected.txt.
3:23 PM Changeset in webkit [42292] by Adam Roben
  • 2 edits in trunk/WebKitTools

2009-04-11 Brian Weinstein <bweinstein@gmail.com>

Reviewed by Darin Adler.

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

Allows timeout to be set in run-webkit-tests as a command line argument.

  • Scripts/run-webkit-tests:
3:06 PM Changeset in webkit [42291] by Simon Fraser
  • 5 edits in trunk/WebCore

2009-04-07 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein

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

Clean up repaint logic when RenderLayers become and stop being composited.

  • rendering/RenderLayer.h:
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::rendererContentChanged): (WebCore::RenderLayer::repaintIncludingNonCompositingDescendants): New compositing-only method that repaints a layer and all its non-composited descendants. Takes a repaintContainer for performance; all the repaints necessarily share the same repaintContainer.
  • rendering/RenderLayerCompositor.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateLayerCompositingState): Rather than use calculateCompositedBounds() to compute the repaint rect (which gave the wrong answer when the composited layer tree was in flux), use the new repaintOnCompositingChange() method.

(WebCore::RenderLayerCompositor::repaintOnCompositingChange):
Call repaintIncludingNonCompositingDescendants(), and if necessary make sure
that the view/layer drawing synchronization happens.

(WebCore::RenderLayerCompositor::computeCompositingRequirements):
Do a repaintOnCompositingChange() when a layer is going to be composited. This is
a good place because we haven't started to change the compositing tree.

(WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):
After we've finished updating all the descendant layers, we can do a repaint for
layers that ceased compositing.

3:04 PM Changeset in webkit [42290] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-07 Adam Langley <agl@google.com>

Reviewed by Darin Fisher.

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

Skia is changing from reporting the XRange to reporting the XMin and
XMax. Obviously these are equivalent, but other users of the metrics
need the two numbers to be separate.

This doesn't affect any layout tests.

  • platform/graphics/chromium/SimpleFontDataLinux.cpp: (WebCore::SimpleFontData::platformInit):
2:53 PM Changeset in webkit [42289] by Adam Roben
  • 2 edits in trunk/WebCore

Make the look of <select>s match the system look on Vista

Reviewed by Dave Hyatt.

  • rendering/RenderThemeWin.cpp: Added new constants. (WebCore::RenderThemeWin::getThemeData): On Vista, use the CP_DROPDOWNBUTTONRIGHT part for drop down buttons. (WebCore::RenderThemeWin::paintMenuList): On Vista, use the CP_BORDER part to draw the border of <select>s. (WebCore::RenderThemeWin::paintMenuListButton): On Vista, outset the drop down button to make its border coincide with the <select>'s border.
2:53 PM Changeset in webkit [42288] by Adam Roben
  • 3 edits
    2 adds in trunk/WebCore

Move isRunningOnVistaOrLater to a shared location

Rubber-stamped in advance by Dave Hyatt.

  • WebCore.vcproj/WebCore.vcproj: Added SystemInfo.{cpp,h}, and let VS have its way with the rest of the file.
  • platform/win/ScrollbarThemeWin.cpp: Moved isRunningOnVistaOrLater from here...
  • platform/win/SystemInfo.cpp: Added. (WebCore::isRunningOnVistaOrLater): ...to here.
  • platform/win/SystemInfo.h: Added.
2:42 PM Changeset in webkit [42287] by Dimitri Glazkov
  • 4 edits in trunk/WebCore

2009-04-07 Paul Godavari <paul@chromium.org>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=24692
Enable PopupMenuChromium to handle HTML select popups using native
cocoa controls on the Mac, all other platforms are unchanged.

We also split out the storage container for the popup menu items
so that ChromeClientChromium can access them for forwarding to
the embedding host (Chromium.app or test_shell).

  • page/chromium/ChromeClientChromium.h:
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::items): (WebCore::PopupContainer::PopupContainer): (WebCore::PopupContainer::~PopupContainer): (WebCore::PopupContainer::showPopup): (WebCore::PopupContainer::showExternal): (WebCore::PopupContainer::menuItemHeight): (WebCore::popupData): (WebCore::PopupListBox::pointToRowIndex): (WebCore::PopupListBox::getRowBounds): (WebCore::PopupListBox::isSelectableItem): (WebCore::PopupListBox::updateFromElement): (WebCore::PopupListBox::layout): (WebCore::PopupListBox::clear): (WebCore::PopupMenu::show):
  • platform/chromium/PopupMenuChromium.h: (WebCore::PopupItem::): (WebCore::PopupItem::PopupItem):
2:28 PM Changeset in webkit [42286] by ggaren@apple.com
  • 13 edits in trunk/WebCore

2009-04-07 Geoffrey Garen <ggaren@apple.com>

Reviewed by Oliver Hunt.


Added some double-checking to event handler marking. Now, when the
wrapper that's responsible for marking an event handler's JavaScript
function gets collected, it clears the event handler's JavaScript function
so we won't dereference it after it's collected.


In theory, we would never dereference a JavaScript event handler after
its node's wrapper was collected anyway, but it's nice to be safe.

  • bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::mark):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::jsFunction): (WebCore::JSEventListener::invalidateJSFunction): (WebCore::JSEventListener::markJSFunction):
  • bindings/js/JSEventListener.h:
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::mark):
  • bindings/js/JSNodeCustom.cpp: (WebCore::markEventListeners):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::mark):
  • bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::mark):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::mark):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::mark):
  • bindings/scripts/CodeGeneratorJS.pm:
  • dom/EventListener.h: (WebCore::EventListener::invalidateJSFunction): (WebCore::EventListener::markJSFunction): (WebCore::markIfNotNull):
2:27 PM Changeset in webkit [42285] by kov@webkit.org
  • 3 edits in trunk/WebKit/mac

2009-04-07 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Anders Carlsson.

Trying to fix Tiger build.

  • Plugins/WebNetscapePluginView.h:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView getAuthenticationInfoWithProtocol:host:port:scheme:realm:username:usernameLength:password:passwordLength:]):
2:19 PM Changeset in webkit [42284] by andersca@apple.com
  • 6 edits in trunk/WebKit

WebKit:

2009-04-07 Anders Carlsson <andersca@apple.com>

Try to fix the Leopard build once more.

  • WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

2009-04-07 Anders Carlsson <andersca@apple.com>

Try to fix the Leopard build once more.


  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView getVariable:forURL:value:length:]):
2:15 PM Changeset in webkit [42283] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-07 David Levin <levin@chromium.org>

Build fix.

r44280 undid the change in r42268 that made CookieJar.h private.
This redoes the change.

  • WebCore.xcodeproj/project.pbxproj:
2:15 PM Changeset in webkit [42282] by levin@chromium.org
  • 4 edits in trunk/JavaScriptCore

2009-04-07 David Levin <levin@chromium.org>

Reviewed by Sam Weinig and Geoff Garen.

https://bugs.webkit.org/show_bug.cgi?id=25039
UString refactoring to support UChar* sharing.

No change in sunspider perf.

  • runtime/SmallStrings.cpp: (JSC::SmallStringsStorage::SmallStringsStorage):
  • runtime/UString.cpp: (JSC::initializeStaticBaseString): (JSC::initializeUString): (JSC::UString::BaseString::isShared): Encapsulate the meaning behind the refcount == 1 checks because this needs to do slightly more when sharing is added. (JSC::concatenate): (JSC::UString::append): (JSC::UString::operator=):
  • runtime/UString.h: Make m_baseString part of a union to get rid of casts, but make it protected because it is tricky to use it correctly since it is only valid when the Rep is not a BaseString. The void* will be filled in when sharing is added.

Add constructors due to the making members protected and it make ensuring proper
initialization work better (like in SmallStringsStorage).
(JSC::UString::Rep::create):
(JSC::UString::Rep::Rep):
(JSC::UString::Rep::):
(JSC::UString::BaseString::BaseString):
(JSC::UString::Rep::setBaseString):
(JSC::UString::Rep::baseString):

2:15 PM Changeset in webkit [42281] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-07 Jian Li <jianli@chromium.org>

Reviewed by Darin Fisher.

Upstream changes to WorkerContextExecutionProxy for enabling V8 preemption.
https://bugs.webkit.org/show_bug.cgi?id=25034

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::evaluate):
1:03 PM Changeset in webkit [42280] by mitz@apple.com
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Darin Adler.

  • fix <rdar://problem/6764359> Thai word selection misbehaves for the last sentence in the document

Test: editing/selection/thai-word-at-document-end.html

  • editing/visible_units.cpp:
    • Added a BoundarySearchContextAvailability enum used to indicate whether the caller to a boundary search function may be able to provide additional text in the search direction and call again.
    • Added a named type for the boundary search function signature. Changed the signature to take a context availability parameter and a boolean out parameter indicating whether more context is needed to perform the search.

(WebCore::previousBoundary): If the beginning of the range was reached

but the last search wanted more context, perform the search again, this
time indicating that there is no earlier text.

(WebCore::nextBoundary): Similarly, in the other direction.
(WebCore::startWordBoundary): Check whether more context may be available

and ask for more context if needed.

(WebCore::endWordBoundary): Ditto.
(WebCore::previousWordPositionBoundary): Ditto.
(WebCore::nextWordPositionBoundary): Ditto.
(WebCore::startSentenceBoundary): Updated signature.
(WebCore::endSentenceBoundary): Ditto.
(WebCore::previousSentencePositionBoundary): Ditto.
(WebCore::nextSentencePositionBoundary): Ditto.

LayoutTests:

Reviewed by Darin Adler.

  • test for <rdar://problem/6764359> Thai word selection misbehaves for the last sentence in the document
  • editing/selection/thai-word-at-document-end-expected.txt: Added.
  • editing/selection/thai-word-at-document-end.html: Added.
12:51 PM Changeset in webkit [42279] by ggaren@apple.com
  • 7 edits in trunk/WebCore

2009-04-07 Geoffrey Garen <ggaren@apple.com>

Rubber stamped by Sam Weinig.


A little renaming:


function => jsFunction
listener => jsFunction

  • bindings/js/JSEventListener.cpp: (WebCore::JSAbstractEventListener::handleEvent): (WebCore::JSEventListener::JSEventListener): (WebCore::JSEventListener::~JSEventListener): (WebCore::JSEventListener::jsFunction): (WebCore::JSEventListener::mark): (WebCore::JSProtectedEventListener::JSProtectedEventListener): (WebCore::JSProtectedEventListener::~JSProtectedEventListener): (WebCore::JSProtectedEventListener::jsFunction):
  • bindings/js/JSEventListener.h: (WebCore::JSProtectedEventListener::create):
  • bindings/js/JSLazyEventListener.cpp: (WebCore::JSLazyEventListener::jsFunction): (WebCore::JSLazyEventListener::parseCode):
  • dom/EventListener.h: (WebCore::EventListener::jsFunction):
12:49 PM Changeset in webkit [42278] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-07 Anders Carlsson <andersca@apple.com>

...and try to fix the Leopard build.


  • Plugins/npapi.mm: (NPN_GetAuthenticationInfo):
12:47 PM Changeset in webkit [42277] by beidson@apple.com
  • 3 edits in trunk/WebCore

2009-04-07 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler

While working on <rdar://problem/5968249>, noticed some glaring problems with LocalStorage.

  • page/DOMWindow.cpp: (WebCore::DOMWindow::localStorage): Return the cached m_localStorage object if it exists to avoid creating multiple representations for the same underlying StorageArea.
  • page/DOMWindow.h: (WebCore::DOMWindow::optionalLocalStorage): Return m_localStorage, not m_sessionStorage.
12:38 PM Changeset in webkit [42276] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-07 Darin Adler <Darin Adler>

Roll out incorrect build fix.

  • WebCore.NPAPI.exp: Reverted.
12:33 PM Changeset in webkit [42275] by Darin Adler
  • 2 edits in trunk/WebCore

2009-04-07 Darin Adler <Darin Adler>

Another build fix.

  • WebCore.NPAPI.exp: Added new functions.
12:14 PM Changeset in webkit [42274] by andersca@apple.com
  • 2 edits in trunk/WebCore

2009-04-07 Anders Carlsson <andersca@apple.com>

Another build fix.


  • plugins/npfunctions.h:
11:28 AM Changeset in webkit [42273] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-04-07 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Unreviewed build fix. Fix non-Mac builds.

  • bridge/npapi.h:
10:57 AM Changeset in webkit [42272] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-07 Anders Carlsson <andersca@apple.com>

Try to fix the Tiger build.

  • Plugins/WebBaseNetscapePluginView.mm:
10:45 AM Changeset in webkit [42271] by Simon Fraser
  • 8 edits in trunk/WebCore

2009-04-07 Simon Fraser <Simon Fraser>

Reviewed by Dave Hyatt

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

Clean up the accelerated compositing code that determines the format of
the GraphicsLayer hierarchy, being more explicit about which parts have
to happen before and after descendant layers have been updated. Also remove
some unhelpful caching of state.

  • dom/Document.cpp: (WebCore::Document::recalcStyle): Only call updateCompositingLayers() if we know we're not going to be doing a layout soon.
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::setDrawsContent): Toggling drawsContent from false to true should do a setNeedsDisplay().
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::styleChanged): updateLayerCompositingState() no longer updates the layer geometry, so do that here if we're not going to rebuild layers anyway. Also, updateLayerCompositingState() no longer does a setContentsNeedDisplay, so do that if the style change indicates that a repaint is required.
  • rendering/RenderLayerBacking.h:
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::RenderLayerBacking): (WebCore::RenderLayerBacking::updateAfterLayout): The 'drawing optimizations' logic has been rolled into updateGraphicsLayerConfiguration() and updateGraphicsLayerGeometry().

(WebCore::RenderLayerBacking::updateGraphicsLayerConfiguration):
Group that code that decides what GraphicsLayers are needed into this method,
which deals with clipping layers and 'direct compositing': images that can be
rendered via the compositor.

(WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
Call setDrawsContent() if updateGraphicsLayerConfiguration() didn't already
figure it out for image layers.

(WebCore::RenderLayerBacking::isSimpleContainerCompositingLayer):
Remove the m_simpleCompositingLayerStatusDirty cache indicator, since we just
computed it every time anyway.

  • rendering/RenderLayerCompositor.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateLayerCompositingState): updateGraphicsLayers() was renamed to updateGraphicsLayerConfiguration(), which calls clippedByAncestor() etc. itself rather than having everything passed in. Repaints are taken care of by styleChanged() and setDrawsContent().

(WebCore::RenderLayerCompositor::calculateCompositedBounds):
!layer->isComposited() && layer->transform() is a synonym for paintsWithTransform().

(WebCore::RenderLayerCompositor::setCompositingParent):
No need to call updateGraphicsLayerGeometry() here, because we're going to
rebuild layers anyway.

(WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):
Add comments, and test layerBacking instead of layer->isComposited().

10:41 AM Changeset in webkit [42270] by andersca@apple.com
  • 2 edits in trunk/WebKit

2009-04-07 Anders Carlsson <andersca@apple.com>

Another attempt at fixing the build.


  • WebKit.xcodeproj/project.pbxproj:
10:33 AM Changeset in webkit [42269] by andersca@apple.com
  • 2 edits in trunk/WebCore

2009-04-07 Anders Carlsson <andersca@apple.com>

Fix build.


  • WebCore.base.exp:
10:22 AM Changeset in webkit [42268] by andersca@apple.com
  • 16 edits in trunk

WebCore:

2009-04-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6667001> NPAPI: need NPN_Get/SetValueForURL() and NPN_GetAuthenticationInfo()


  • WebCore.base.exp: Add symbols needed by WebKit.


  • WebCore.xcodeproj/project.pbxproj: Make CookieJar a private header.


  • bridge/npapi.h: Add function definitions.


  • plugins/npfunctions.h: Fix the NPN_SetValueForURLProcPtr signature.

WebKit/mac:

2009-04-07 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6667001>
NPAPI: need NPN_Get/SetValueForURL() and NPN_GetAuthenticationInfo()


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCGetCookies): (WKPCGetProxy): (WKPCSetCookies): (WKPCGetAuthenticationInfo): New MIG functions to be used by the plug-in host.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::getCookies): (WebKit::NetscapePluginInstanceProxy::setCookies): (WebKit::NetscapePluginInstanceProxy::getProxy): (WebKit::NetscapePluginInstanceProxy::getAuthenticationInfo): Implement these.


  • Plugins/Hosted/WebKitPluginClient.defs: Add callbacks.


  • Plugins/WebBaseNetscapePluginView.h:
  • Plugins/WebBaseNetscapePluginView.mm: (-[WebBaseNetscapePluginView URLWithCString:]): Factor this out of URLWithCString.


(-[WebBaseNetscapePluginView requestWithURLCString:]):
Call URLWithCString.


(WebKit::proxiesForURL):
Return a string representation of proxies for a given URL.


(WebKit::getAuthenticationInfo):
Get the authentication info for a given host/protocol/scheme/realm/port combination.


  • Plugins/WebNetscapePluginPackage.mm: (-[WebNetscapePluginPackage _tryLoad]): Initialize the new vtable functions.


  • Plugins/WebNetscapePluginView.h:
  • Plugins/WebNetscapePluginView.mm: (-[WebNetscapePluginView getVariable:forURL:value:length:]): (-[WebNetscapePluginView setVariable:forURL:value:length:]): (-[WebNetscapePluginView getAuthenticationInfoWithProtocol:host:port:scheme:realm:username:usernameLength:password:passwordLength:]): Implement these.


  • Plugins/npapi.mm: (NPN_GetValueForURL): (NPN_SetValueForURL): (NPN_GetAuthenticationInfo): Call the plug-in view functions.
10:13 AM Changeset in webkit [42267] by Adam Roben
  • 2 edits in trunk/WebCore

Fix <rdar://6520933> REGRESSION (3.2.1-ToT): Scroll bar disappears in WebView and Bookmarks view after switching theme or turning on High Contrast mode

Reviewed by Dave Hyatt.

  • platform/win/ScrollbarThemeWin.cpp: (WebCore::ScrollbarThemeWin::themeChanged): Null out scrollbarTheme after closing it so that we'll re-open the theme next time we paint.
10:12 AM Changeset in webkit [42266] by Adam Roben
  • 2 edits in trunk/WebCore

Small cleanup/correctness fix in ScrollbarThemeWin

Reviewed by Dave Hyatt.

  • platform/win/ScrollbarThemeWin.cpp: Removed the now-unused haveTheme boolean. (WebCore::checkAndInitScrollbarTheme): Check IsThemeActive() before calling OpenThemeData. This seems to be what MSDN recommends. (WebCore::ScrollbarThemeWin::themeChanged): Null-check scrollbarTheme before closing it, rather than indirectly null-checking it via checking haveTheme.
10:08 AM Changeset in webkit [42265] by mitz@apple.com
  • 6 edits
    2 adds in trunk

WebCore:

Reviewed by John Sullivan.

  • when -webkit-line-break: after-white-space is specified but the text does not auto-wrap, do not shrink the trailing space to fit in the available width

Test: fast/text/whitespace/nowrap-line-break-after-white-space.html

  • when shrinking such trailing space, do not shrink below zero

Covered by existing tests

  • rendering/bidi.cpp: (WebCore::RenderBlock::computeHorizontalPositionsForLine): (WebCore::RenderBlock::layoutInlineChildren):

LayoutTests:

Reviewed by John Sullivan.

  • test that when -webkit-line-break: after-white-space is combined with white-space: nowrap, the trailing space is not shrunk to fit the in the available width
  • updates results for that and for not shrinking trailing space to less than zero
  • fast/text/whitespace/nowrap-line-break-after-white-space-expected.txt: Added.
  • fast/text/whitespace/nowrap-line-break-after-white-space.html: Added.
  • platform/mac/fast/block/float/editable-text-overlapping-float-expected.txt:
  • platform/mac/fast/inline/long-wrapped-line-expected.txt:
  • platform/mac/fast/text/whitespace/tab-character-basics-expected.txt:
7:25 AM Changeset in webkit [42264] by eric@webkit.org
  • 5 edits in trunk/WebCore

Reviewed by Darin Adler.

Move RangeBoundaryPoint off of Position, per Darin's suggestion
https://bugs.webkit.org/show_bug.cgi?id=24966

Rename setToChild to setToBeforeChild (since that's what the method does)

  • dom/Position.h: (WebCore::Position::offsetInContainerNode):
  • dom/Range.cpp: (WebCore::Range::insertNode): (WebCore::Range::selectNodeContents): (WebCore::boundaryNodeWillBeRemoved):
  • dom/Range.h: (WebCore::Range::startPosition): (WebCore::Range::endPosition):
  • dom/RangeBoundaryPoint.h: (WebCore::RangeBoundaryPoint::RangeBoundaryPoint): (WebCore::RangeBoundaryPoint::container): (WebCore::RangeBoundaryPoint::childBefore): (WebCore::RangeBoundaryPoint::position): (WebCore::RangeBoundaryPoint::offset): (WebCore::RangeBoundaryPoint::clear): (WebCore::RangeBoundaryPoint::set): (WebCore::RangeBoundaryPoint::setOffset): (WebCore::RangeBoundaryPoint::setToBeforeChild): (WebCore::RangeBoundaryPoint::setToStartOfNode): (WebCore::RangeBoundaryPoint::setToEndOfNode): (WebCore::RangeBoundaryPoint::childBeforeWillBeRemoved): (WebCore::RangeBoundaryPoint::invalidateOffset):
7:25 AM Changeset in webkit [42263] by eric@webkit.org
  • 3 edits in trunk/WebCore

Reviewed by Darin Adler.

Add containerNode(), computeOffsetInContainerNode(),
computeNodeBeforePosition() and computeNodeAfterPosition().
https://bugs.webkit.org/show_bug.cgi?id=24763

These functions will replace uses of rangeCompliantPosition()
in future patches.

No changes in behavior, thus no tests.

  • dom/Position.cpp: (WebCore::Position::containerNode): (WebCore::Position::computeOffsetInContainerNode): (WebCore::Position::computeNodeBeforePosition): (WebCore::Position::computeNodeAfterPosition): (WebCore::Position::anchorType):
  • dom/Position.h: (WebCore::Position::):
4:48 AM Changeset in webkit [42262] by xan@webkit.org
  • 2 edits in trunk/JavaScriptCore

2009-04-04 Xan Lopez <xlopez@igalia.com>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=25033
dtoa.cpp segfaults with g++ 4.4.0

g++ 4.4.0 seems to be more strict about aliasing rules, so it
produces incorrect code if dtoa.cpp is compiled with
-fstrict-aliasing (it also emits a ton of warnings, so fair enough
I guess). The problem was that we were only casting variables to
union types in order to do type punning, but GCC and the C
standard require that we actually use a union to store the value.

This patch does just that, the code is mostly copied from the dtoa
version in GCC:
http://gcc.gnu.org/viewcvs/trunk/libjava/classpath/native/fdlibm/dtoa.c?view=markup.

  • wtf/dtoa.cpp: (WTF::ulp): (WTF::b2d): (WTF::ratio): (WTF::hexnan): (WTF::strtod): (WTF::dtoa):

Apr 6, 2009:

4:43 PM Changeset in webkit [42261] by weinig@apple.com
  • 5 edits in trunk/WebCore

2009-04-06 Sam Weinig <sam@webkit.org>

Reviewed by Mark Rowe.

Bottleneck dispatching mutation events in a new dispatchMutationEvent
function.

  • dom/CharacterData.cpp: (WebCore::CharacterData::dispatchModifiedEvent):
  • dom/ContainerNode.cpp: (WebCore::dispatchChildInsertionEvents): (WebCore::dispatchChildRemovalEvents):
  • dom/Node.cpp: (WebCore::Node::dispatchSubtreeModifiedEvent): (WebCore::Node::dispatchMutationEvent):
  • dom/Node.h:
4:11 PM Changeset in webkit [42260] by Dimitri Glazkov
  • 9 edits in trunk/WebCore

2009-04-06 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=25059
Remove references to dominantScript, getGenericFontForScript.

  • platform/graphics/chromium/FontCacheChromiumWin.cpp: Removed references

to getGenericFontForScript.

(WebCore::FontCache::getLastResortFallbackFont): Removed calls to dominantScript.

  • platform/graphics/chromium/FontCacheLinux.cpp: Ditto.
  • rendering/RenderThemeChromiumLinux.cpp: (WebCore::defaultGUIFont): Removed Document* arg. (WebCore::RenderThemeChromiumLinux::systemFont): Ditto.
  • rendering/RenderThemeChromiumLinux.h: Ditto.
  • rendering/RenderThemeChromiumMac.h: Ditto.
  • rendering/RenderThemeChromiumMac.mm: Ditto. (WebCore::RenderThemeChromiumMac::systemFont): Ditto.
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::defaultGUIFont): Ditto. (WebCore::RenderThemeChromiumWin::systemFont): Ditto.
  • rendering/RenderThemeChromiumWin.h: Ditto.
4:04 PM Changeset in webkit [42259] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

4:02 PM Changeset in webkit [42258] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.5

New tag.

3:42 PM Changeset in webkit [42257] by christian@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2009-04-06 Christian Dywan <christian@twotoasts.de>

Rubber-stamped by Oliver Hunt.

  • de.po: Added German translation for WebKitGTK+.
2:56 PM Changeset in webkit [42256] by ggaren@apple.com
  • 4 edits in trunk/WebCore

2009-04-06 Geoffrey Garen <ggaren@apple.com>

Reviewed by Darin Adler.


Fixed https://bugs.webkit.org/show_bug.cgi?id=21260
Unbounded memory growth when churning elements with anonymous event handler functions

The problem was that a protected event listener's scope chain would end
up with a reference to the Node protecting it, causing a cycle. The
solution is to stop protecting event listeners and rely on marking instead.

This patch converts most Node event listeners to use marking instead of
GC protection.

  • bindings/js/JSDOMBinding.cpp: (WebCore::isObservableThroughDOM): (WebCore::markDOMNodesForDocument): Refactored the test for whether a Node needs marking, and added a case for marking the node in order to mark its event listeners.
  • bindings/js/JSNodeCustom.cpp: (WebCore::markEventListeners): (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener): (WebCore::JSNode::mark): Added code to mark a Node's event listeners when marking the Node. Changed code using protected event listeners to use event listeners.
  • dom/Node.idl: Changed code using protected event listeners to use event listeners.
1:58 PM Changeset in webkit [42255] by weinig@apple.com
  • 3 edits in trunk/WebCore

2009-04-06 Sam Weinig <sam@webkit.org>

Reviewed by Darin Adler.

Remove unchecked bool return values for some dispatch event functions.

  • dom/Node.cpp: (WebCore::Node::dispatchSubtreeModifiedEvent): (WebCore::Node::dispatchUIEvent): (WebCore::Node::dispatchWebKitAnimationEvent): (WebCore::Node::dispatchWebKitTransitionEvent): (WebCore::Node::dispatchProgressEvent):
  • dom/Node.h:
1:48 PM Changeset in webkit [42254] by darin@chromium.org
  • 3 edits in trunk/WebCore

2009-04-06 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25058
Fix chromium build bustage caused by r42216

  • bindings/scripts/CodeGeneratorJS.pm:
  • page/DOMWindow.idl:
12:48 PM Changeset in webkit [42253] by Dimitri Glazkov
  • 2 edits in trunk/LayoutTests

2009-04-06 Dimitri Glazkov <Dimitri Glazkov>

Not reviewed because it's stupidly obvious.

Correct the expected result by adding an extra line break.

  • fast/dom/HTMLSelectElement/length-not-overridden-expected.txt: Added

an extra line.

12:39 PM Changeset in webkit [42252] by kov@webkit.org
  • 1 edit
    1 add in trunk/WebKit/gtk/po

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Sam Weinig.

  • pt_BR.po: Added Brazilian Portuguese translation for WebKitGTK+.
12:39 PM Changeset in webkit [42251] by kov@webkit.org
  • 12 edits
    6 adds in trunk

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Sam Weinig.

Added rules to maintain the localization support. We cannot simply
use whatever gettextize gives us because our build system is
non-recursive.

  • GNUmakefile.am:
  • configure.ac:

WebCore/

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=20575
No localization in WebKitGtk

  • platform/gtk/FileChooserGtk.cpp: Remove usage of glib/gi18n.h, since there are not translatable strings.
  • platform/gtk/LocalizedStringsGtk.cpp: Use the glib i18n header that is specific for libraries.

WebKit/gtk/

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=20575
No localization in WebKitGtk

Implemented localization support for WebKitGTK+.

  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ContextMenuClientGtk.cpp:
  • po/POTFILES: Added.
  • po/README: Added.
  • po/webkit.pot: Added.
  • webkit/webkitdownload.cpp: (_WebKitDownloadPrivate::webkit_download_class_init):
  • webkit/webkitprivate.cpp: (webkit_init):
  • webkit/webkitwebframe.cpp:

WebKit/gtk/po/

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Sam Weinig.

  • GNUmakefile.am, POTFILES, webkit.pot: Initial setup of the localization infra-structure.
11:41 AM Changeset in webkit [42250] by darin@chromium.org
  • 2 edits in trunk/WebCore

Revert r42249 to fix the build.

11:23 AM Changeset in webkit [42249] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-06 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25058
Fix chromium build bustage caused by r42216

  • page/DOMWindow.idl:
11:08 AM Changeset in webkit [42248] by Dimitri Glazkov
  • 3 edits in trunk/WebCore

2009-04-06 Mike Belshe <mike@belshe.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24978
Use custom implementations of a few Node methods so that
we don't over-aggressively create JS wrappers here.

  • bindings/v8/custom/V8NodeCustom.cpp: Added custom implementations.
  • dom/Node.idl: Removed JSC-specifier, making impl custom for V8 as well.
11:01 AM Changeset in webkit [42247] by Dimitri Glazkov
  • 3 edits
    2 adds in trunk

WebCore:

2009-04-06 Dave Moore <davemoore@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25031
Make the V8 element collections check for named properties *before*
checking if there are any elements with the same name. Otherwise
it is both incorrect and slow.

Test: fast/dom/HTMLSelectElement/length-not-overridden.html

Both of these interceptors were attempting to find an element in the
collection that had a name or id of the property name before checking
for a JS property with that name.

  • bindings/v8/V8Collection.h: (WebCore::collectionNamedPropertyGetter): (WebCore::nodeCollectionNamedPropertyGetter):

LayoutTests:

2009-04-06 Dave Moore <davemoore@google.com>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=25031
Test for ensuring that named elements don't override properties on collections

  • fast/dom/HTMLSelectElement/length-not-overridden-expected.txt: Added.
  • fast/dom/HTMLSelectElement/length-not-overridden.html: Added.
10:51 AM Changeset in webkit [42246] by Simon Fraser
  • 5 edits
    4 adds in trunk

2009-04-06 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler

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

Pull the call to updateGraphicsLayerGeometry() out of updateGraphicsLayers(),
because we need to call it at the end of rebuildCompositingLayerTree() once
we've determined which descendant layers are composited, otherwise
calculateCompositedBounds() can give the wrong answer.

Now that updateLayerCompositingState() doesn't end up calling updateGraphicsLayerGeometry(),
call that explicitly from styleChanged(), if we know a layer update is not pending,
and similarly from updateAfterLayout().

Test: compositing/overflow/overflow-positioning.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::styleChanged):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateAfterLayout): (WebCore::RenderLayerBacking::updateGraphicsLayers): (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree):
9:01 AM Changeset in webkit [42245] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-06 Mike Belshe <mike@belshe.com>

Reviewed by Alexey Proskuryakov.

Add accessors to AtomicString which take String/const char* arguments
so that we don't accidentally thrash the AtomicString table.

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

  • platform/text/AtomicString.h:
8:47 AM Changeset in webkit [42244] by vestbo@webkit.org
  • 7 edits in trunk/WebKit/qt

2009-04-06 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Rename DOM API select function to findFirst, findAll, etc.

8:37 AM Changeset in webkit [42243] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2009-04-06 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Tor Arne Vestbø.

Add /Programs to the productDir for GTK+, instead of adding
/JavaScriptCore, when running jsc.

  • Scripts/run-sunspider:
7:17 AM Changeset in webkit [42242] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[GTK+] Correct the column and row cursor

Make col and row work in WebCore/manual-tests/cursor.html. The issue
was spotted in the WebKit inspector.

5:59 AM Changeset in webkit [42241] by vestbo@webkit.org
  • 2 edits in trunk/WebCore

2009-04-06 Tor Arne Vestbø <tor.arne.vestbo@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Don't show and hide the platformPluginWidget, as it's our QWebView

5:50 AM QtWebKitContrib edited by Simon Hausmann
(diff)
5:50 AM QtWebKitContrib edited by Simon Hausmann
(diff)
5:49 AM QtWebKitContrib edited by Simon Hausmann
(diff)
4:58 AM Changeset in webkit [42240] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-05 Xan Lopez <xlopez@igalia.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25042
destroying webview widget directly causes crash.

Do not connect the WebView to its own screen-changed signal, set a
default handler instead.

  • webkit/webkitwebview.cpp:
4:57 AM Changeset in webkit [42239] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-05 Xan Lopez <xlopez@igalia.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=25042
destroying webview widget directly causes crash.

GtkWidget::screen-changed can be emitted when a widget is being
destroyed, protect against it.

  • webkit/webkitwebview.cpp:
4:25 AM Changeset in webkit [42238] by Simon Hausmann
  • 9 edits
    5 adds in trunk

WebCore:

2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Add new files to the Qt build.
Export helper function from the Qt JSC binding, needed in WebKit/qt.

WebKit/qt:

2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>

Ariya Hidayat <ariya.hidayat@nokia.com>
Tor Arne Vestbø <tor.arne.vestbo@nokia.com>
Genevieve Mak <gen@staikos.net>

Reviewed by Tor Arne Vestbø, Simon Hausmann

First revision of new API in the Qt port to access the DOM.

4:13 AM Changeset in webkit [42237] by Simon Hausmann
  • 3 edits in trunk/WebCore

2009-04-06 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Tor Arne Vestbø.

Changed ScriptFunctionCall's last argument from a const char* as the function name to a String.

Internally the function name is stored as a WebCore::String and the Qt port wants to use this
constructor programmatically where the function name is also a WebCore::String.

2:51 AM Changeset in webkit [42236] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-04-06 Mike Belshe <mike@belshe.com>

Reviewed by Eric Seidel.

HTMLCanvasElement crash when ImageBuffer creation fails.
https://bugs.webkit.org/show_bug.cgi?id=23212

Check for NULL before using the ImageBuffer as we might
be low on memory and creation may have failed.

Test case creation blocked by:
https://bugs.webkit.org/show_bug.cgi?id=25055

  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createImageBuffer):

Apr 5, 2009:

3:48 PM Changeset in webkit [42235] by ggaren@apple.com
  • 13 edits in trunk/WebCore

2009-04-05 Geoffrey Garen <ggaren@apple.com>

[Originally] Reviewed by Sam Weinig.

Rolled back in r42214 with crashes fixed.

[Removed irrelevant ExecState* parameter from event handler registration code.]


[No change in behavior.]

  • bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
  • bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener):
  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
  • bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
  • bindings/scripts/CodeGeneratorJS.pm:
10:26 AM Changeset in webkit [42234] by Simon Hausmann
  • 2 edits in trunk/WebCore

2009-04-05 Erik L. Bunce <elbunce@xendom.com>

Reviewed by Simon Hausmann.

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

Fix an assert failure when dropping an 'empty' text/uri-list on a QWebView.

7:56 AM Changeset in webkit [42233] by Simon Hausmann
  • 2 edits in trunk/WebCore

Fix the Qt build

1:21 AM Changeset in webkit [42232] by jmalonzo@webkit.org
  • 2 edits in trunk

2009-04-05 Mike Hommey <glandium@debian.org>

Reviewed by Holger Freyther.

Filter out all C++ symbols
https://bugs.webkit.org/show_bug.cgi?id=24960

Considering the public API is all C, we can just filter out all
C++ mangled symbols, which will avoid exporting symbols in some
corner cases such as gcc bugs on specific architectures, etc.

  • autotools/symbols.filter:
1:16 AM Changeset in webkit [42231] by jmalonzo@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-04-05 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] Implement LayoutTestControllerGtk::setPrivateBrowsingEnabled
https://bugs.webkit.org/show_bug.cgi?id=24487

Updated documentation and put more information about the
private-browsing setting for WebKitGtk.

  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::webkit_web_settings_class_init):

Apr 4, 2009:

8:56 PM Changeset in webkit [42230] by kevino@webkit.org
  • 9 edits in trunk

Build fixes for wxMac/Tiger.

5:06 PM Changeset in webkit [42229] by kevino@webkit.org
  • 8 edits in trunk

wx build fixes for the Windows port.

1:25 PM Changeset in webkit [42228] by Simon Fraser
  • 2 edits in trunk/WebCore

2009-04-04 Simon Fraser <Simon Fraser>

Reviewed by Cameron Zwarich

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

Ensure that mapPoint(const IntPoint&) calls the FloatPoint
version to avoid infinite recursion.

No test because mapPoint(const IntPoint&) isn't ever called
on Mac, so I can't reproduce.

  • platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::mapPoint):
12:33 AM Changeset in webkit [42227] by mitz@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Mark Rowe.

  • fix <rdar://problem/6757862> REGRESSION: Cannot use the mouse to place the insertion point inside a Thai character

Test: fast/text/international/thai-offsetForPosition-inside-character.html

  • platform/graphics/mac/CoreTextController.cpp: (WebCore::CoreTextController::offsetForPosition): Changed to use cursorMovementIterator instead of characterBreakIterator, allowing this function to return a valid cursor position even if it is not at a character boundary.

LayoutTests:

Reviewed by Mark Rowe.

  • test for <rdar://problem/6757862> REGRESSION: Cannot use the mouse to place the insertion point inside a Thai character
  • fast/text/international/thai-offsetForPosition-inside-character-expected.txt: Added.
  • fast/text/international/thai-offsetForPosition-inside-character.html: Added.
12:27 AM Changeset in webkit [42226] by mitz@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Mark Rowe.

  • fix an assertion failure when clicking at the beginning of a glyph cluster at the beginning of a text run

Test: fast/text/offsetForPosition-cluster-at-zero.html

  • platform/graphics/mac/CoreTextController.cpp: (WebCore::CoreTextController::offsetForPosition): Corrected the bound check on firstGlyphBeforeCluster, letting it be 0 or -1, but not less than -1.

LayoutTests:

Reviewed by Mark Rowe.

  • test for a bug where computing the text offset for a position at the beginning of a glyph cluster at the beginning of a Core Text run resulted in an attempt to read past the end of a vector
  • fast/text/offsetForPosition-cluster-at-zero-expected.txt: Added.
  • fast/text/offsetForPosition-cluster-at-zero.html: Added.
12:20 AM Changeset in webkit [42225] by weinig@apple.com
  • 2 edits in trunk/LayoutTests

2009-04-04 Sam Weinig <sam@webkit.org>

Update test result.

  • dom/xhtml/level2/html/HTMLFormElement10-expected.txt:

Apr 3, 2009:

6:06 PM Changeset in webkit [42224] by timothy@apple.com
  • 2 edits in trunk/WebCore

Allows 16 HTML/CSS standard colors to be shown as swatch backgrounds
when the names are given.

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

Reviewed by Timothy Hatcher.

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.updateTitle):
5:56 PM Changeset in webkit [42223] by weinig@apple.com
  • 8 edits
    4 adds in trunk

WebCore:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Darin Adler.

<rdar://problem/6330929>
https://bugs.webkit.org/show_bug.cgi?id=21456

Don't update the document pointer for all inactive windows on navigations.
This change causes us to differ slightly from Firefox when accessing the
document from within a closure tied to a navigated context, but as all
browsers differ on this edge case, I don't foresee compatibility issues.

Test: http/tests/security/cross-frame-access-document-direct.html

  • bindings/js/JSDOMWindowBase.cpp: (WebCore::JSDOMWindowBase::~JSDOMWindowBase):
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::clearWindowShell): (WebCore::ScriptController::initScript): (WebCore::ScriptController::updateDocument):
  • bindings/js/ScriptController.h:

LayoutTests:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Darin Adler.

Update tests for <rdar://problem/6330929>
https://bugs.webkit.org/show_bug.cgi?id=21456

  • dom/xhtml/level2/html/HTMLFormElement10-expected.txt:
  • fast/dom/Window/dom-access-from-closure-iframe-expected.txt:
  • fast/dom/Window/dom-access-from-closure-window-expected.txt:
  • http/tests/security/cross-frame-access-document-direct-expected.txt: Added.
  • http/tests/security/cross-frame-access-document-direct.html: Added.
  • http/tests/security/resources/cross-frame-iframe-for-document-direct-test-victim.html: Added.
  • http/tests/security/resources/cross-frame-iframe-for-document-direct-test.html: Added.
5:49 PM Changeset in webkit [42222] by cmarrin@apple.com
  • 4 edits
    2 adds in trunk

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

We were starting transitions with 0 duration and delay, which is
useless. With accelerated compositing on, this went through a code
path that tried to use m_toStyle, which never got set because the
transitions was short circuited before it had a chance to. So I
both protect against using that null pointer and avoid starting
the transition in the first place.

5:41 PM Changeset in webkit [42221] by timothy@apple.com
  • 2 edits in trunk/WebCore

Fixes a bug where you could not type any more in the Web Inspector's
search field after the first match is selected in the Resources panel.

https://bugs.webkit.org/show_bug.cgi?id=23982
rdar://problem/6592367

Reviewed by Darin Adler.

  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._jumpToSearchResult): Use the window object of the frame to set the selection, not the main window.
5:37 PM Changeset in webkit [42220] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-03 Kevin Decker <kdecker@apple.com>

Reviewed by Anders Carlsson.

<rdar://problem/6756512>


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost): Tweaked the visibleName property.
5:34 PM Changeset in webkit [42219] by Stephanie Lewis
  • 2 edits in trunk/WebKitTools

2009-04-03 Stephanie Lewis <Stephanie Lewis>

Reviewed by Mark Rowe.

Fix running DumpRenderTree with a root.

  • Scripts/run-webkit-tests:
4:49 PM Changeset in webkit [42218] by weinig@apple.com
  • 7 edits
    40 adds in trunk

WebCore:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Fix for https://bugs.webkit.org/show_bug.cgi?id=22081
<rdar://problem/6346030>
Don't allow overriding the return value of window.location.

Tests: fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter.html

fast/dom/Window/Location/location-override-toString-on-proto-using-with.html
fast/dom/Window/Location/location-override-toString-on-proto.html
fast/dom/Window/Location/location-override-toString-using-defineGetter.html
fast/dom/Window/Location/location-override-toString-using-with.html
fast/dom/Window/Location/location-override-toString.html
fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter.html
fast/dom/Window/Location/location-override-valueOf-on-proto-using-with.html
fast/dom/Window/Location/location-override-valueOf-on-proto.html
fast/dom/Window/Location/location-override-valueOf-using-defineGetter.html
fast/dom/Window/Location/location-override-valueOf-using-with.html
fast/dom/Window/Location/location-override-valueOf.html
fast/dom/Window/Location/window-override-location-using-defineGetter.html
fast/dom/Window/Location/window-override-window-using-defineGetter.html
fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString.html
fast/dom/Window/Location/window-shadow-location-using-string.html
fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field.html

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::defineGetter):
  • bindings/js/JSLocationCustom.cpp: (WebCore::JSLocation::customPut): (WebCore::JSLocation::defineGetter): (WebCore::JSLocationPrototype::customPut): (WebCore::JSLocationPrototype::defineGetter):
  • bindings/scripts/CodeGeneratorJS.pm:
  • page/Location.idl:

LayoutTests:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Anders Carlsson.

Tests for https://bugs.webkit.org/show_bug.cgi?id=22081
<rdar://problem/6346030>
Don't allow overriding the return value of window.location.

  • fast/dom/Window/Location: Added.
  • fast/dom/Window/Location/location-override-toString-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto-using-defineGetter.html: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto-using-with-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto-using-with.html: Added.
  • fast/dom/Window/Location/location-override-toString-on-proto.html: Added.
  • fast/dom/Window/Location/location-override-toString-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-using-defineGetter.html: Added.
  • fast/dom/Window/Location/location-override-toString-using-with-expected.txt: Added.
  • fast/dom/Window/Location/location-override-toString-using-with.html: Added.
  • fast/dom/Window/Location/location-override-toString.html: Added.
  • fast/dom/Window/Location/location-override-valueOf-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto-using-defineGetter.html: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto-using-with-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto-using-with.html: Added.
  • fast/dom/Window/Location/location-override-valueOf-on-proto.html: Added.
  • fast/dom/Window/Location/location-override-valueOf-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-using-defineGetter.html: Added.
  • fast/dom/Window/Location/location-override-valueOf-using-with-expected.txt: Added.
  • fast/dom/Window/Location/location-override-valueOf-using-with.html: Added.
  • fast/dom/Window/Location/location-override-valueOf.html: Added.
  • fast/dom/Window/Location/resources: Added.
  • fast/dom/Window/Location/resources/location-tests-functions.js: Added. (normalizeURL):
  • fast/dom/Window/Location/resources/pass.html: Added.
  • fast/dom/Window/Location/resources/window-shadow-location-using-js-object-with-toString-iframe.html: Added.
  • fast/dom/Window/Location/resources/window-shadow-location-using-string-iframe.html: Added.
  • fast/dom/Window/Location/window-override-location-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/window-override-location-using-defineGetter.html: Added.
  • fast/dom/Window/Location/window-override-window-using-defineGetter-expected.txt: Added.
  • fast/dom/Window/Location/window-override-window-using-defineGetter.html: Added.
  • fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString-expected.txt: Added.
  • fast/dom/Window/Location/window-shadow-location-using-js-object-with-toString.html: Added.
  • fast/dom/Window/Location/window-shadow-location-using-string-expected.txt: Added.
  • fast/dom/Window/Location/window-shadow-location-using-string.html: Added.
  • fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field-expected.txt: Added.
  • fast/dom/Window/Location/window-shadow-window-using-js-object-with-location-field.html: Added.
  • http/tests/security/cross-frame-access-location-put-expected.txt:
4:19 PM Changeset in webkit [42217] by ggaren@apple.com
  • 13 edits in trunk/WebCore

2009-04-03 Geoffrey Garen <ggaren@apple.com>

Rolled out r42214 since it seems to have caused crashes on the buildbot.

  • bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
  • bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener):
  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
  • bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
  • bindings/scripts/CodeGeneratorJS.pm:
4:13 PM Changeset in webkit [42216] by weinig@apple.com
  • 23 edits
    6 adds in trunk

WebCore:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Oliver Hunt.

Fix for <rdar://problem/6476356>
https://bugs.webkit.org/show_bug.cgi?id=23148

  • Use the window object the Location and History objects are directly associated with instead of the lexical global object to pick the object prototype to serve as the base of the their respective prototype chains.
  • Re-factor as necessary to allow passing the correct global object to the createPrototype functions.

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

http/tests/security/cross-frame-access-location-prototype.html

  • bindings/js/JSAudioConstructor.cpp: (WebCore::JSAudioConstructor::JSAudioConstructor):
  • bindings/js/JSDOMBinding.cpp: (WebCore::getCachedDOMStructure): (WebCore::cacheDOMStructure):
  • bindings/js/JSDOMBinding.h: (WebCore::getDOMStructure): (WebCore::getDOMPrototype):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::history): (WebCore::JSDOMWindow::location):
  • bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::location):
  • bindings/js/JSImageConstructor.cpp: (WebCore::JSImageConstructor::JSImageConstructor):
  • bindings/js/JSMessageChannelConstructor.cpp: (WebCore::JSMessageChannelConstructor::JSMessageChannelConstructor):
  • bindings/js/JSNamedNodesCollection.h: (WebCore::JSNamedNodesCollection::createPrototype):
  • bindings/js/JSOptionConstructor.cpp: (WebCore::JSOptionConstructor::JSOptionConstructor):
  • bindings/js/JSRGBColor.h: (WebCore::JSRGBColor::createPrototype):
  • bindings/js/JSWebKitCSSMatrixConstructor.cpp: (WebCore::JSWebKitCSSMatrixConstructor::JSWebKitCSSMatrixConstructor):
  • bindings/js/JSWebKitPointConstructor.cpp: (WebCore::JSWebKitPointConstructor::JSWebKitPointConstructor):
  • bindings/js/JSWorkerConstructor.cpp: (WebCore::JSWorkerConstructor::JSWorkerConstructor):
  • bindings/js/JSXMLHttpRequestConstructor.cpp: (WebCore::JSXMLHttpRequestConstructor::JSXMLHttpRequestConstructor):
  • bindings/js/JSXSLTProcessorConstructor.cpp: (WebCore::JSXSLTProcessorConstructor::JSXSLTProcessorConstructor):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/objc/objc_runtime.h: (JSC::Bindings::ObjcFallbackObjectImp::createPrototype):
  • bridge/runtime_array.h: (JSC::RuntimeArray::createPrototype):
  • bridge/runtime_method.h: (JSC::RuntimeMethod::createPrototype):
  • bridge/runtime_object.h: (JSC::RuntimeObjectImp::createPrototype):
  • page/DOMWindow.idl:

LayoutTests:

2009-04-03 Sam Weinig <sam@webkit.org>

Reviewed by Oliver Hunt.

Tests for <rdar://problem/6476356>
https://bugs.webkit.org/show_bug.cgi?id=23148

Test for using the correct global object for location and history object
prototype chain creation.

  • http/tests/security/cross-frame-access-history-prototype-expected.txt: Added.
  • http/tests/security/cross-frame-access-history-prototype.html: Added.
  • http/tests/security/cross-frame-access-location-prototype-expected.txt: Added.
  • http/tests/security/cross-frame-access-location-prototype.html: Added.
  • http/tests/security/resources/cross-frame-history-prototype-iframe.html: Added.
  • http/tests/security/resources/cross-frame-location-prototype-iframe.html: Added.
3:24 PM Changeset in webkit [42215] by dimich@chromium.org
  • 3 edits in trunk/WebCore

2009-04-03 Jian Li <jianli@chromium.org>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=25009
Upstream changes to WorkerContextExecutionProxy for V8 bindings in order to use V8EventListenerList as container.

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy): (WebCore::WorkerContextExecutionProxy::dispose): (WebCore::WorkerContextExecutionProxy::FindOrCreateEventListener): (WebCore::WorkerContextExecutionProxy::RemoveEventListener):
  • bindings/v8/WorkerContextExecutionProxy.h:
3:09 PM Changeset in webkit [42214] by ggaren@apple.com
  • 13 edits in trunk/WebCore

2009-04-03 Geoffrey Garen <ggaren@apple.com>

Reviewed by Sam Weinig.

Removed irrelevant ExecState* parameter from event handler registration code.


No change in behavior.

  • bindings/js/JSDOMApplicationCacheCustom.cpp: (WebCore::JSDOMApplicationCache::addEventListener): (WebCore::JSDOMApplicationCache::removeEventListener):
  • bindings/js/JSDOMGlobalObject.cpp: (WebCore::JSDOMGlobalObject::findOrCreateJSProtectedEventListener): (WebCore::JSDOMGlobalObject::findJSEventListener): (WebCore::JSDOMGlobalObject::findOrCreateJSEventListener):
  • bindings/js/JSDOMGlobalObject.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::addEventListener):
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::addEventListener): (WebCore::JSMessagePort::removeEventListener):
  • bindings/js/JSNodeCustom.cpp: (WebCore::JSNode::addEventListener): (WebCore::JSNode::removeEventListener):
  • bindings/js/JSSVGElementInstanceCustom.cpp: (WebCore::JSSVGElementInstance::addEventListener):
  • bindings/js/JSWorkerContextCustom.cpp: (WebCore::JSWorkerContext::addEventListener): (WebCore::JSWorkerContext::removeEventListener):
  • bindings/js/JSWorkerCustom.cpp: (WebCore::JSWorker::addEventListener): (WebCore::JSWorker::removeEventListener):
  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::addEventListener): (WebCore::JSXMLHttpRequest::removeEventListener):
  • bindings/js/JSXMLHttpRequestUploadCustom.cpp: (WebCore::JSXMLHttpRequestUpload::addEventListener): (WebCore::JSXMLHttpRequestUpload::removeEventListener):
  • bindings/scripts/CodeGeneratorJS.pm:
3:00 PM Changeset in webkit [42213] by kdecker@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Anders Carlsson.

<rdar://problem/6756512>


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::spawnPluginHost): Tweaked the visibleName property.
2:21 PM Changeset in webkit [42212] by Adam Roben
  • 4 edits in trunk/LayoutTests

Skip tests that rely on layoutTestController.waitForPolicyDelegate on platforms that don't implement that functionality

Implementing is covered by the following bugs:
<https://bugs.webkit.org/show_bug.cgi?id=25036>
<https://bugs.webkit.org/show_bug.cgi?id=25037>
<https://bugs.webkit.org/show_bug.cgi?id=25038>

Rubber-stamped by Mark Rowe.

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped: Skip tests that use layoutTestController.waitForPolicyDelegate.
2:09 PM Changeset in webkit [42211] by mitz@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

Reviewed by Simon Fraser.

  • fix <rdar://problem/6634771> Under full-page zoom, cannot scroll MobileMe contact list

Test: fast/css/zoom-in-length-round-trip.html

  • dom/Element.cpp: (WebCore::adjustForLocalZoom): For zoom factors greater than 1, ensure that the returned value is equal to the original unscaled value. (WebCore::adjustForAbsoluteZoom): Ditto.

LayoutTests:

Reviewed by Simon Fraser.

  • test for <rdar://problem/6634771> Under full-page zoom, cannot scroll MobileMe contact list
  • fast/css/zoom-in-length-round-trip-expected.txt: Added.
  • fast/css/zoom-in-length-round-trip.html: Added.
1:44 PM Changeset in webkit [42210] by sullivan@apple.com
  • 2 edits in trunk/WebKit/mac

2009-04-03 John Sullivan <sullivan@apple.com>

fixed <rdar://problem/6355573> [WebView _setCacheModel:] leaks the result of _CFURLCacheCopyCacheDirectory

Reviewed by Adam Roben

  • WebView/WebView.mm: (+[WebView _setCacheModel:]): use WebCFAutorelease rather than autorelease on result of method that returns CFStringRef
1:35 PM Changeset in webkit [42209] by Simon Fraser
  • 2 edits in trunk/LayoutTests

2009-04-03 Simon Fraser <Simon Fraser>

Update expected result due to rounding difference after r42207.

  • platform/mac/svg/hixie/perf/007-expected.txt:
12:29 PM Changeset in webkit [42208] by cmarrin@apple.com
  • 9 edits in trunk

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

This fix essentially does a -viewWillDraw call for layout. It adds
a CFRunLoopObserver which performs layout just before drawing on the
Mac platform. This makes sure layout is complete before rendering and
avoids a flash.

11:57 AM Changeset in webkit [42207] by Simon Fraser
  • 3 edits in trunk/WebCore

2009-04-02 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein

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

Optimize mapPoint(), mapRect() and mapQuad() for identity matrix and translations.

  • platform/graphics/transforms/TransformationMatrix.cpp: (WebCore::TransformationMatrix::mapPoint): (WebCore::TransformationMatrix::mapRect): (WebCore::TransformationMatrix::mapQuad):
  • platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::mapPoint):
11:39 AM Changeset in webkit [42206] by Simon Fraser
  • 3 edits in trunk/WebCore

2009-04-02 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein

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

Add containsPoint() and containsQuad() methods to FloatQuad.

  • platform/graphics/FloatQuad.cpp: (WebCore::dot): (WebCore::isPointInTriangle): (WebCore::FloatQuad::containsPoint): (WebCore::FloatQuad::containsQuad):
  • platform/graphics/FloatQuad.h:
11:28 AM Changeset in webkit [42205] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

2009-04-03 Anders Carlsson <andersca@apple.com>

Reviewed by Kevin Decker.

WebKit side of <rdar://problem/6752953>.


Pass the clip rect to the plug-in host.


  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::resize):
  • Plugins/Hosted/WebKitPluginHost.defs:
11:16 AM Changeset in webkit [42204] by kdecker@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Darin.

  • loader/ProgressTracker.cpp: Included DocumentLoader.h (WebCore::ProgressTracker::incrementProgress): Limit the 50%-clamp rule to documents that use WebCore's layout system.
11:06 AM Changeset in webkit [42203] by eric.carlson@apple.com
  • 5 edits
    3 adds in trunk

2009-04-03 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6336092> REGRESSION: movie does not resize correctly

Test: media/video-size-intrinsic-scale.html

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::naturalSize): Use QTMovieCurrentSizeAttribute instead of QTMovieNaturalSizeAttribute so we return the correct size for movies saved with a non-default matrix. (WebCore::MediaPlayerPrivate::setSize): Don't resize the QTMovieView because [QTMovieView setFrame] also resizes the movie itself, and as the renderer calls setSize immediately when a movie reports a size change (QTMovieSizeDidChangeNotification) we can get into a feedback loop observing the size change and resetting the size, and this can cause QuickTime to miss correctly setting a movie's size when the media size changes after opening (as happens with an rtsp movie once the server sends the track sizes). Instead, remember the size passed to paint() and resize the view when it changes. (WebCore::MediaPlayerPrivate::paint): Resize the view when passed a new size.
  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWin::getNaturalSize): Use GetMovieBox instead of GetMovieNaturalBoundsRect so we return the correct size for movies saved with non-default matrix.

2009-04-03 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6336092> REGRESSION: movie does not resize correctly

  • media/content/scaled-matrix.mov: Added. QuickTime movie saved with 2x scale.
  • media/video-size-intrinsic-scale-expected.txt: Added.
  • media/video-size-intrinsic-scale.html: Added, test that element with a scaled movie is sized correctly.
  • media/video-test.js: Modify so it can be included in <head> element. (logConsole): New, wraps creating console <div>, doesn't assume document.body exists. (consoleWrite): Call logConsole().
10:52 AM Changeset in webkit [42202] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-03 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

Restore FrameLoader::policyDocumentLoader to fix the Chromium build.
https://bugs.webkit.org/show_bug.cgi?id=25028

  • loader/FrameLoader.h: (WebCore::FrameLoader::policyDocumentLoader):
9:58 AM Changeset in webkit [42201] by Darin Adler
  • 2 edits in trunk/WebCore

Fixed a backwards comment in ChangeLog.

9:28 AM Changeset in webkit [42200] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-04-02 Michael Nordman <Michael Nordman>

Reviewed by Alexey Proskuryakov.

Use long long, not int64, when working in webcore (sorry).
https://bugs.webkit.org/show_bug.cgi?id=25011

No change in functionality, so no tests.

  • platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::getAppCacheID): (WebCore::ResourceResponse::setAppCacheID):
8:33 AM QtWebKitTodo edited by Simon Hausmann
(diff)
6:00 AM Changeset in webkit [42199] by kov@webkit.org
  • 14 edits in trunk

2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

Require GTK+ >= 2.10; 2.8 is already very old, and some very
useful APIs are only available since 2.10.

  • configure.ac:

WebCore/

2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

Remove pre-processor checks for GTK+ >= 2.10, since we now require
at least that version.

  • platform/graphics/gtk/FontPlatformDataGtk.cpp: (WebCore::FontPlatformData::FontPlatformData):
  • platform/gtk/ContextMenuItemGtk.cpp: (WebCore::gtkStockIDFromContextMenuAction):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
  • platform/gtk/LocalizedStringsGtk.cpp: (WebCore::contextMenuItemTagSelectAll):
  • platform/gtk/MouseEventGtk.cpp: (WebCore::PlatformMouseEvent::PlatformMouseEvent):
  • platform/gtk/PasteboardGtk.cpp: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::documentFragment):
  • platform/gtk/WheelEventGtk.cpp: (WebCore::PlatformWheelEvent::PlatformWheelEvent):

WebKit/gtk

2009-04-03 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Holger Freyther.

Remove pre-processor checks for GTK+ >= 2.10, since we now require
at least that version.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::clipboard_clear_contents_cb): (WebKit::EditorClient::respondToChangedSelection):
  • webkit/webkitwebframe.cpp:
  • webkit/webkitwebview.cpp:
Note: See TracTimeline for information about the timeline view.