Timeline
Nov 8, 2009:
- 11:34 PM Changeset in webkit [50640] by
-
- 4 edits1 add in trunk
The XML tokenizer reports a parse error twice if it occurs before the document element is found.
https://bugs.webkit.org/show_bug.cgi?id=31144
XMLTokenizer::doEnd() uses an additional logic to report a parse failure in
documents that end prematurely but are not considered invalid by QXmlStream.
This is to stay compatible with the libxml2 implementation.
However, that code path would be also hit in situations when it should not,
i.e. the error would have already been caught and handled. As a result, the
same error would be reported twice.
No new tests, because the problem is already covered by
fast/parser/xml-declaration-missing-ending-mark.html.
Add the Qt-specific result for that test and unskip it.
- 10:39 PM Changeset in webkit [50639] by
-
- 5 edits in trunk/WebCore
2009-11-08 Keishi Hattori <casey.hattori@gmail.com>
Reviewed by Pavel Feldman.
Web Inspector: Inspector should support copy() in the command line
https://bugs.webkit.org/show_bug.cgi?id=31238
- inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::copyText): Added.
- inspector/InspectorBackend.h: Added copyText
- inspector/InspectorBackend.idl: Added copyText
- inspector/front-end/InjectedScript.js: (InjectedScript._copy): Added. (InjectedScript._ensureCommandLineAPIInstalled):
- 10:00 PM Changeset in webkit [50638] by
-
- 2 edits in trunk/WebCore
2009-11-08 Drew Wilson <atwilson@chromium.org>
Reviewed by Dimitri Glazkov.
V8 WorkerContextExecutionProxy does not handle SharedWorkers
https://bugs.webkit.org/show_bug.cgi?id=31226
Now checks to see what type of context is active and creates the
appropriate wrapper (DEDICATEDWORKERCONTEXT vs SHAREDWORKERCONTEXT).
Added support for converting to SharedWorkers and SharedWorkerContexts.
Test: Existing layout tests cover this case (start passing in Chrome).
- bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::initContextIfNeeded): Now generates the right type of DOMWrapper for SharedWorkerContexts. (WebCore::WorkerContextExecutionProxy::convertToV8Object): Added support for SHAREDWORKERCONTEXT. (WebCore::WorkerContextExecutionProxy::convertEventTargetToV8Object): Added support for SharedWorker and SharedWorkerContext.
- 9:52 PM Changeset in webkit [50637] by
-
- 3 edits4 adds in trunk
2009-11-08 Johnny Ding <johnnyding.webkit@gmail.com>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=31224
[V8] Return StyleSheet object instead of HTMLStyleElement w/document.styleSheets named property getter.
- fast/dom/StyleSheet/get-stylesheet-byname-expected.txt: Added.
- fast/dom/StyleSheet/get-stylesheet-byname.html: Added.
- fast/dom/StyleSheet/script-tests: Added.
- fast/dom/StyleSheet/script-tests/get-stylesheet-byname.js: Added.
2009-11-08 Johnny Ding <johnnyding.webkit@gmail.com>
Reviewed by Dimitri Glazkov.
https://bugs.webkit.org/show_bug.cgi?id=31224
[V8] Return StyleSheet object instead of HTMLStyleElement w/document.styleSheets named property getter.
Test: fast/dom/StyleSheet/get-stylesheet-byname.html
- bindings/v8/custom/V8StyleSheetListCustom.cpp: (WebCore::NAMED_PROPERTY_GETTER):
- 8:52 PM Changeset in webkit [50636] by
-
- 3 edits in trunk/WebKit/mac
Made the full-screen video HUD appear when playback stops, such as when
the end of the video is reached.
Reviewed by Eric Carlson.
- WebView/WebVideoFullscreenHUDWindowController.h: Cleaned up.
- WebView/WebVideoFullscreenHUDWindowController.mm:
(-[WebVideoFullscreenHUDWindowController scheduleTimeUpdate]): Updated
for the renaming of -updateRate to -updatePlayButton.
(-[WebVideoFullscreenHUDWindowController updatePlayButton]): Renamed
-updateRate to this.
(-[WebVideoFullscreenHUDWindowController updateRate]): This method now
responds to changes to the playback rate by updating the play button
and showing or hiding the HUD as necessary.
(-[WebVideoFullscreenHUDWindowController togglePlaying:]): Now only
toggles playing. UI updates are driven by -updateRate being called.
(-[WebVideoFullscreenHUDWindowController playing]): Cleaned up.
- 8:46 PM Changeset in webkit [50635] by
-
- 2 edits in trunk/WebCore
2009-11-08 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: "Search again" on scripts panel switches
to the script this search started with.
- inspector/front-end/Panel.js: (WebInspector.Panel.prototype.jumpToNextSearchResult):
- 6:43 PM Changeset in webkit [50634] by
-
- 3 edits in trunk/JavaScriptCore
- wtf/DateMath.cpp:
(WTF::parseDateFromNullTerminatedCharacters):
(JSC::gregorianDateTimeToMS):
(JSC::msToGregorianDateTime):
(JSC::parseDateFromNullTerminatedCharacters):
- wtf/DateMath.h:
(JSC::GregorianDateTime::GregorianDateTime):
Reviewed by NOBODY (speculative snow leopard and windows build fixes).
- 6:33 PM Changeset in webkit [50633] by
-
- 9 edits in trunk
JavaScriptCore: Hopefully, the last build fix.
Reviewed by NOBODY (chromium build fix).
Create better separation in DateMath about the JSC
and non-JSC portions. Also, only expose the non-JSC
version in the exports.
- JavaScriptCore.exp:
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
- wtf/DateMath.cpp:
(WTF::parseDateFromNullTerminatedCharacters):
(JSC::getUTCOffset):
(JSC::gregorianDateTimeToMS):
(JSC::msToGregorianDateTime):
(JSC::parseDateFromNullTerminatedCharacters):
- wtf/DateMath.h:
(JSC::gmtoffset):
JavaScriptGlue: Added the use jsc define for files that use
this config file and DateMath.h. This should
be able to go away when DateMath is properly
split into wtf and jsc portions which is this bug:
https://bugs.webkit.org/show_bug.cgi?id=31246
Reviewed by NOBODY (chromium build fix).
- config.h:
WebCore: * platform/network/HTTPParsers.cpp:
(WebCore::parseDate): Changed this to not
use a date parser that needs ExecState passed.
Reviewed by NOBODY (chromium build fix).
- 5:28 PM Changeset in webkit [50632] by
-
- 14 edits in trunk/WebCore
2009-11-08 Kent Tamura <tkent@chromium.org>
Reviewed by Darin Adler.
Simplify Icon interface.
https://bugs.webkit.org/show_bug.cgi?id=31154
- Remove Icon::createIconForFile(). createIconForFiles() covers createIconForFile()'s role.
- Remove FileChooser::chooseIcon()
- Change the parameter types of FileChooser constructor and the factory method, String -> const Vector<String>&, in order to support initialization with multiple files.
- Remove the icon loading code in IconChromiumWin.cpp, which doesn't work because of the sandbox.
No tests because it's just a refactoring.
- platform/FileChooser.cpp: (WebCore::FileChooser::FileChooser): (WebCore::FileChooser::create): (WebCore::FileChooser::chooseFile): (WebCore::FileChooser::chooseFiles):
- platform/FileChooser.h:
- platform/graphics/Icon.h:
- platform/graphics/chromium/IconChromiumLinux.cpp:
- platform/graphics/chromium/IconChromiumMac.cpp:
- platform/graphics/chromium/IconChromiumWin.cpp: (WebCore::Icon::createIconForFiles):
- platform/graphics/gtk/IconGtk.cpp: (WebCore::Icon::createIconForFiles):
- platform/graphics/haiku/IconHaiku.cpp:
- platform/graphics/mac/IconMac.mm: (WebCore::Icon::createIconForFiles):
- platform/graphics/qt/IconQt.cpp: (WebCore::Icon::createIconForFiles):
- platform/graphics/win/IconWin.cpp: (WebCore::Icon::createIconForFiles):
- platform/graphics/wx/IconWx.cpp:
- rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::RenderFileUploadControl):
- 5:18 PM Changeset in webkit [50631] by
-
- 4 edits3 adds in trunk
2009-11-08 Daniel Bates <dbates@webkit.org>
Reviewed by Adam Barth.
Allows same-origin plugin-based content to load.
Test: http/tests/security/xssAuditor/object-src-inject.html
- page/XSSAuditor.cpp: (WebCore::XSSAuditor::canLoadExternalScriptFromSrc): Modified to call XSSAuditor::isSameOriginResource. (WebCore::XSSAuditor::canLoadObject): Ditto. (WebCore::XSSAuditor::canSetBaseElementURL): Ditto. (WebCore::XSSAuditor::isSameOriginResource): Added.
- page/XSSAuditor.h:
2009-11-08 Daniel Bates <dbates@webkit.org>
Reviewed by Adam Barth.
Tests that the XSSAuditor prevents loading plugin-based content that is not
from the same-origin as the enclosing page.
- http/tests/security/xssAuditor/object-src-inject-expected.txt: Added.
- http/tests/security/xssAuditor/object-src-inject.html: Added.
- http/tests/security/xssAuditor/resources/echo-object-src.pl: Added.
- 3:54 PM Changeset in webkit [50630] by
-
- 2 edits in trunk/WebCore
Unreviewed build fix for chromium.
- platform/graphics/skia/GraphicsContextSkia.cpp:
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::fillRect):
(WebCore::GraphicsContext::strokePath):
(WebCore::GraphicsContext::strokeRect):
- 3:42 PM Changeset in webkit [50629] by
-
- 2 edits in trunk/WebCore
Unreviewed build fix for chromium.
Build fix for https://bugs.webkit.org/show_bug.cgi?id=31219
Clean up GraphicsContext's current concept of ColorSpace
- platform/graphics/skia/GraphicsContextSkia.cpp:
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::fillRect):
(WebCore::GraphicsContext::strokePath):
(WebCore::GraphicsContext::strokeRect):
- 1:02 PM Changeset in webkit [50628] by
-
- 3 edits in trunk/JavaScriptCore
For the change in DateMath.
Reviewed by NOBODY (chromium build fix).
- config.h:
- wtf/DateMath.cpp:
- 11:47 AM Changeset in webkit [50627] by
-
- 3 edits in trunk/WebCore
2009-11-08 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: [REGRESSION] committing style edit
clears elements panel selection.
- inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype.update):
- inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertyTreeElement.prototype.): (WebInspector.StylePropertyTreeElement.prototype):
- 11:09 AM Changeset in webkit [50626] by
-
- 3 edits in trunk/LayoutTests
2009-11-08 Shu Chang <Chang.Shu@nokia.com>
Reviewed by Holger Freyther.
Update texts in test description after test case change in r50579.
https://bugs.webkit.org/show_bug.cgi?id=31122
- editing/selection/shrink-selection-after-shift-pagedown-expected.txt:
- editing/selection/shrink-selection-after-shift-pagedown.html:
- 11:01 AM Changeset in webkit [50625] by
-
- 27 edits in trunk
2009-11-08 Janne Koskinen <janne.p.koskinen@digia.com>
Reviewed by Holger Freyther.
ResourceRequest to be class instead of struct
https://bugs.webkit.org/show_bug.cgi?id=30670
Started as a compilation fix for Symbian where the compiler makes a distinction between
class and struct in function argument signatures.
Changed all forward declarations of ResourceRequest to have class in the forward
declaration instead of struct and changed the definition of ResourceRequest to be class
and added access qualifiers where missing. Additionally two references of friend
struct ResourceRequestBase changed to class instead.
- history/HistoryItem.h:
- inspector/InspectorController.h:
- inspector/InspectorResource.h:
- loader/DocumentThreadableLoader.h:
- loader/FrameLoaderClient.h:
- loader/MainResourceLoader.h:
- loader/ResourceLoadNotifier.h:
- loader/SubresourceLoader.h:
- loader/SubresourceLoaderClient.h:
- loader/ThreadableLoader.h:
- loader/WorkerThreadableLoader.h:
- loader/appcache/ApplicationCache.h:
- loader/appcache/ApplicationCacheHost.h:
- platform/CrossThreadCopier.h:
- platform/network/ResourceHandle.h:
- platform/network/ResourceHandleClient.h:
- platform/network/ResourceRequestBase.h:
- platform/network/cf/ResourceRequest.h:
- platform/network/cf/ResourceRequestCFNet.h:
- platform/network/chromium/ResourceRequest.h:
- platform/network/curl/ResourceRequest.h:
- platform/network/qt/ResourceRequest.h:
- platform/network/soup/ResourceRequest.h:
- xml/XMLHttpRequest.h:
2009-11-08 Janne Koskinen <janne.p.koskinen@digia.com>
Reviewed by Holger Freyther.
ResourceRequest to be class instead of struct
https://bugs.webkit.org/show_bug.cgi?id=30670
Started as a compilation fix for Symbian where the compiler makes a distinction between
class and struct in function argument signatures.
Changed forward declaration of ResourceRequest to have class in the forward
declaration instead of struct.
- WebDownload.h:
- 10:51 AM Changeset in webkit [50624] by
-
- 2 edits in trunk/WebKitTools
2009-11-08 Shu Chang <Chang.Shu@nokia.com>
Reviewed by Holger Freyther.
[Qt] Added support for key code 8 (backspace) in EventSenderQt.
This helps to pass the test case below. Also replaced hardcoded
code numbers with defined constants.
https://bugs.webkit.org/show_bug.cgi?id=31185
Test: editing/undo/undo-deleteWord.html
- DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::keyDown):
- 9:32 AM Changeset in webkit [50623] by
-
- 6 edits4 adds in trunk
<rdar://problem/7363434> Crash inside RenderObject::localToAbsolute
below FrameView::layout
https://bugs.webkit.org/show_bug.cgi?id=31093
Reviewed by Simon Fraser.
WebCore:
Test: fast/block/positioning/relative-positioned-inline-container.html
In <http://trac.webkit.org/changeset/19148>, setStaticY() was changed
to mark the object for layout, doing so without marking its ancestors.
However, RenderBlock::skipLeadingWhitespace and
RenderBlock::skipTrailingWhitespace() call setStaticY() on a relative-
positioned inline container, causing it to be marked for layout without
ever going back to give it layout, and thus layout could end with a
dirty object still in the tree, leading to all sorts of badness.
The fix is to revert setStaticY() to not marking the object dirty, and
instead do it in the call sites that require it, which are in
RenderBlock and RenderFlexibleBox.
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::adjustPositionedBlock):
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::layoutHorizontalBox):
(WebCore::RenderFlexibleBox::layoutVerticalBox):
- rendering/RenderLayer.cpp:
- rendering/RenderLayer.h:
(WebCore::RenderLayer::setStaticY):
LayoutTests:
- fast/block/positioning/relative-positioned-inline-container-expected.checksum: Added.
- fast/block/positioning/relative-positioned-inline-container-expected.png: Added.
- fast/block/positioning/relative-positioned-inline-container-expected.txt: Added.
- fast/block/positioning/relative-positioned-inline-container.html: Added.
- 9:26 AM Changeset in webkit [50622] by
-
- 2 edits in trunk/WebKitTools
[Qt] [DRT] Fix wrong logic in LayoutTestController processWork
https://bugs.webkit.org/show_bug.cgi?id=31164
Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-11-05
Reviewed by Holger Freyther.
Fixed wrong logic to assume WorkQueue is done in QT's DRT.
- DumpRenderTree/qt/LayoutTestControllerQt.cpp:
(LayoutTestController::processWork):
- 9:25 AM Changeset in webkit [50621] by
-
- 4 edits in trunk/WebKitTools
[Qt][DRT] Replace queueScript by queueNonLoadingScript and queueLoadingScript method
https://bugs.webkit.org/show_bug.cgi?id=31158
Patch by Antonio Gomes <tonikitoo@webkit.org> on 2009-11-07
Reviewed by Holger Freyther.
By invoking a script queue'd by queueScript(), 'true' was beeing returned
always, which from WorkQueue prospective means that a load has been started
and the queue processing should stop and wait for the load to finish.
Spinning it off into a loading and a non-loading variants was the solution
adopted by Mac's DRT to work around this problem. The former keeps returning
'true' while the later executes the script synchronously and returns 'false'
making it possible to the WorkQueue to proceed right away.
- DumpRenderTree/qt/LayoutTestControllerQt.cpp:
(LayoutTestController::processWork):
(LayoutTestController::queueLoadingScript):
(LayoutTestController::queueNonLoadingScript):
- DumpRenderTree/qt/LayoutTestControllerQt.h:
- DumpRenderTree/qt/WorkQueueItem.h:
(LoadingScriptItem::LoadingScriptItem):
(LoadingScriptItem::invoke):
(NonLoadingScriptItem::NonLoadingScriptItem):
(NonLoadingScriptItem::invoke):
Nov 7, 2009:
- 7:59 PM Changeset in webkit [50620] by
-
- 2 edits in trunk/WebKitTools
Invoke prepare-ChangeLog via an absolute path rather than assuming it can be found in PATH.
Rubber-stamped by Cameron Zwarich.
- Scripts/commit-log-editor:
- 7:59 PM Changeset in webkit [50619] by
-
- 4 edits in trunk/WebKitTools
Fix <https://bugs.webkit.org/show_bug.cgi?id=28168>.
commit-log-editor does not support all the email address configurations that prepare-Changelog supports
Reviewed by Darin Adler.
Move logic for determining the name and email address to use in a ChangeLog entry from
prepare-ChangeLog to VCSUtils so that commit-log-editor can use it. It wants to check
whether the author of the patch matches committer, and therefore needs access to the
email address that would be used in a ChangeLog entry.
Based on a patch by Pierre d'Herbemont.
- Scripts/VCSUtils.pm:
- Scripts/commit-log-editor:
- Scripts/prepare-ChangeLog:
- Scripts/webkitdirs.pm:
- 6:31 PM Changeset in webkit [50618] by
-
- 2 edits1 add in trunk/WebCore
2009-11-07 Daniel Bates <dbates@webkit.org>
Reviewed by Darin Adler.
Fixes an issue where the onchange event handler is not fired when the
input field is autocompleted.
We cannot test this using DRT since DRT cannot emulate autocompletion.
So, a manual-test is included.
Tests: manual-tests/autocompletion-fire-onchange.html
- manual-tests/autocompletion-fire-onchange.html: Added.
- rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
- 2:40 PM Changeset in webkit [50617] by
-
- 1 edit4 adds in trunk/LayoutTests
2009-11-07 Dirk Pranke <dpranke@chromium.org>
Reviewed by Darin Adler.
Add in missing expectations.
- platform/mac/fast/forms/input-list-button-size-expected.checksum: Added.
- platform/mac/fast/forms/input-list-button-size-expected.png: Added.
- platform/mac/fast/forms/select-dirty-parent-pref-widths-expected.png: Added.
- platform/mac/fast/text/text-letter-spacing-expected.png: Added.
- 12:06 PM Changeset in webkit [50616] by
-
- 2 edits in trunk/WebCore
2009-11-07 Benjamin Otte <otte@gnome.org>
Reviewed by Holger Freyther.
Mark redrawn areas on image surfaces as dirty.
This is required to conform to the Cairo API, but is currently only
used by debugging tools like cairo-trace.
- platform/graphics/cairo/ImageBufferCairo.cpp: (WebCore::ImageBuffer::platformTransformColorSpace): (WebCore::putImageData):
- 1:03 AM Changeset in webkit [50615] by
-
- 2 edits in trunk/WebCore
2009-11-07 Joanmarie Diggs <joanmarie.diggs@gmail.com>
Reviewed by Xan Lopez.
https://bugs.webkit.org/show_bug.cgi?id=30878
[Gtk] atk_text_get_text() fails in entries when the end_offset is -1
If the end_offset is -1, use the String length as the end_offset.
- accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_text_get_text):
- 12:00 AM Changeset in webkit [50614] by
-
- 8 edits in trunk/WebCore
Fix for https://bugs.webkit.org/show_bug.cgi?id=31219 Clean up
GraphicsContext's current concept of ColorSpace
Reviewed by Adele Peterson.
ColorSpace is now called ColorType. The variables on the state we
appropriately re-named as well. I removed strokeColorSpace() and
fillColorSpace() from GraphicsContext since they were never called.
- platform/graphics/GraphicsContext.cpp:
(WebCore::GraphicsContext::setStrokeColor):
(WebCore::GraphicsContext::setFillColor):
(WebCore::GraphicsContext::setStrokePattern):
(WebCore::GraphicsContext::setFillPattern):
(WebCore::GraphicsContext::setStrokeGradient):
(WebCore::GraphicsContext::setFillGradient):
- platform/graphics/GraphicsContext.h:
(WebCore::):
- platform/graphics/GraphicsContextPrivate.h:
(WebCore::GraphicsContextState::GraphicsContextState):
- platform/graphics/cairo/GraphicsContextCairo.cpp:
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::strokePath):
- platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::calculateDrawingMode):
(WebCore::GraphicsContext::drawPath):
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::strokePath):
(WebCore::GraphicsContext::fillRect):
(WebCore::GraphicsContext::strokeRect):
- platform/graphics/qt/GraphicsContextQt.cpp:
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::strokePath):
(WebCore::GraphicsContext::fillRect):
- platform/graphics/wince/GraphicsContextWince.cpp:
(WebCore::GraphicsContext::fillPath):
(WebCore::GraphicsContext::fillRect):
Nov 6, 2009:
- 6:02 PM Changeset in webkit [50613] by
-
- 9 edits1 delete in trunk/WebCore
Fixes <http://webkit.org/b/31177>.
Web Inspector: Bind backspace to delete cookies and DOM Storage.
Reviewed by Tim Hatcher.
Refactor editing code from DOMStorageDataGrid to DataGrid, so other
places in the inspector can use editing in DataGrid (added a FIXME for
some stuff that needs to be generalized).
Also added deleting functionality to DataGrid, and implemented it for
Cookies and DOM Storage. The reason this patch is so big is because of
the refactoring of editing code, which won't be used yet in a cross-datagrid
way, but should be able to.
Additionally, moved the callbacks members from DOMStorageDataGrid to
DOMStorageItemsView, which allowed us to delete DOMStorageDataGrid, to
make the architecture of DOM Storage look a lot more like the Cookies view.
Lastly, added a preventDefault call in ElementsTreeOutline to prevent the
inspector from beeping at you when you delete an element.
- WebCore.gypi: Removed DOMStorageDataGrid.
- WebCore.vcproj/WebCore.vcproj: Removed DOMStorageDataGrid.
- inspector/front-end/CookieItemsView.js:
(WebInspector.CookieItemsView.prototype.dataGridForCookies):
(WebInspector.CookieItemsView.prototype._deleteButtonClicked):
(WebInspector.CookieItemsView.prototype._deleteCookieCallback):
- inspector/front-end/DOMStorageDataGrid.js: Removed.
- inspector/front-end/DOMStorageItemsView.js:
(WebInspector.DOMStorageItemsView.prototype._dataGridForDOMStorageEntries):
(WebInspector.DOMStorageItemsView.prototype._deleteButtonClicked):
(WebInspector.DOMStorageItemsView.prototype._refreshButtonClicked):
(WebInspector.DOMStorageItemsView.prototype._editingCallback):
(WebInspector.DOMStorageItemsView.prototype.deleteSelectedRow):
(WebInspector.DOMStorageItemsView.prototype._deleteCallback):
- inspector/front-end/DataGrid.js:
(WebInspector.DataGrid):
(WebInspector.DataGrid.prototype._ondblclick): Moved from DOMStorageDataGrid to DataGrid + Refactoring.
(WebInspector.DataGrid.prototype._startEditingColumnOfDataGridNode): Ditto.
(WebInspector.DataGrid.prototype._startEditing): Ditto.
(WebInspector.DataGrid.prototype._editingCommitted.moveToNextIfNeeded): Ditto.
(WebInspector.DataGrid.prototype._editingCommitted): Ditto.
(WebInspector.DataGrid.prototype._editingCancelled): Ditto.
(WebInspector.DataGrid.prototype.handleKeyEvent): Added case for delete/backspace.
- inspector/front-end/ElementsTreeOutline.js:
(WebInspector.ElementsTreeOutline.prototype.handleKeyEvent): Added preventDefault call.
- inspector/front-end/WebKit.qrc: Removed DOMStorageDataGrid.
- inspector/front-end/inspector.html: Removed DOMStorageDataGrid.
- 4:05 PM Changeset in webkit [50612] by
-
- 2 edits in trunk/JavaScriptCore
Windows build fix: export some symbols.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-06
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
- 3:59 PM Changeset in webkit [50611] by
-
- 2 edits in trunk/WebCore
Qt build fix: added an ExecState parameter.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-06
- bridge/qt/qt_runtime.cpp:
(JSC::Bindings::convertValueToQVariant):
(JSC::Bindings::convertQVariantToValue):
- 3:54 PM Changeset in webkit [50610] by
-
- 2 edits in trunk/JavaScriptCore
Build fix: updated export file.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-06
- JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
- 3:48 PM Changeset in webkit [50609] by
-
- 3 edits in trunk/JavaScriptCore
Build fix: added some #includes.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-06
- wtf/CurrentTime.h:
- wtf/DateMath.h:
- 3:33 PM Changeset in webkit [50608] by
-
- 27 edits1 add in trunk
Rolled back in r50590 with Windows build hopefully fixed.
- 3:00 PM Changeset in webkit [50607] by
-
- 9 edits4 adds in trunk
2009-11-06 Anantanarayanan G Iyengar <ananta@chromium.org>
Reviewed by Adam Barth.
Added layout tests to test document.open and window.open calls issued
by NPAPI plugins without a calling javascript context. The associated
webkit bug is https://bugs.webkit.org/show_bug.cgi?id=31067, which
affects Chromium.
- plugins/document-open-expected.txt: Added.
- plugins/document-open.html: Added.
- plugins/window-open-expected.txt: Added.
- plugins/window-open.html: Added.
2009-11-06 Anantanarayanan G Iyengar <ananta@chromium.org>
Reviewed by Adam Barth.
The associated webkit bug is https://bugs.webkit.org/show_bug.cgi?id=31067,
which affects Chromium only.
Changes to V8HTMLDocumentCustom.cpp are as below:-
- The HTMLDocumentOpen function would cause a crash in Chromium if there was no calling javascript context. We now check for this case and pass in NULL to the HTMLDocument::open function which can handle a NULL document parameter.
- The other functions like HTMLDocumentWrite, HTMLDocumentWriteln, etc had ASSERTS for a NULL caller frame, which was bogus as it would crash anyway. We now check for this case and return a failure.
Changes to V8DOMWindowCustom.cpp are as below:-
- Instead of failing the window.open call made by NPAPI for lack of a calling javascript context, we now use the entered context as the calling context.
Tests: plugins/document-open.html
plugins/window-open.html
- bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
- bindings/v8/custom/V8HTMLDocumentCustom.cpp: (WebCore::CALLBACK_FUNC_DECL):
2009-11-06 Anantanarayanan G Iyengar <ananta@chromium.org>
Reviewed by Adam Barth.
Added functionality to the layout test plugin to invoke document.open and
window.open with default arguments. The associated webkit bug is
https://bugs.webkit.org/show_bug.cgi?id=31067, which affects Chromium. Basically
window.open and document.open calls issued by NPAPI plugins via NPN_Invoke don't
work in Chromium (V8) if there is no calling javascript context. To achieve this
effect we invoke these functions in the layout test plugin in the NPP_SetWindow
for the window.open test case and in NPP_DestroyStream for the document.open test case.
- DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (testDocumentOpen): (testWindowOpen): (pluginAllocate):
- DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h:
- DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp: (NPP_New): (NPP_SetWindow): (NPP_DestroyStream):
- DumpRenderTree/win/TestNetscapePlugin/main.cpp: (NPP_New): (NPP_SetWindow): (NPP_NewStream): (NPP_DestroyStream):
- 2:58 PM Changeset in webkit [50606] by
-
- 2 edits in trunk/WebKitSite
2009-11-06 Dirk Schulze <krit@webkit.org>
Rubber stamped by Eric Seidel.
Update SVG status page. We support feMorphology now.
Set lightning filters to unimplemented.
- projects/svg/status.xml:
- 1:36 PM Changeset in webkit [50605] by
-
- 22 edits in trunk
2009-11-06 Steve Block <steveblock@google.com>
Reviewed by Eric Seidel.
Geolocation error code UNKNOWN_ERROR is deprecated.
https://bugs.webkit.org/show_bug.cgi?id=31184
Update test case of the value of the PositionError.UNKNOWN_ERROR constant in error.html.
Also remove redundant test cases of the value of all PositionError constants.
- fast/dom/Geolocation/resources/error.js: Modified. Update test of value of PositionError.UNKNOWN_ERROR constant.
- fast/dom/Geolocation/error-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/permission-denied-already-error.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/permission-denied-already-error-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/permission-denied-already-success.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/permission-denied-already-success-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/permission-denied.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/permission-denied-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/permission-denied-stops-watches.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/permission-denied-stops-watches-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/reentrant-error.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/reentrant-error-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/timeout-zero.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/timeout-zero-expected.txt: Modified. Expected result for above test.
- fast/dom/Geolocation/resources/watch.js: Modified. Remove tests of the value of PositionError constants.
- fast/dom/Geolocation/watch-expected.txt: Modified. Expected result for above test.
2009-11-06 Steve Block <steveblock@google.com>
Reviewed by Eric Seidel.
Geolocation error code UNKNOWN_ERROR is deprecated.
https://bugs.webkit.org/show_bug.cgi?id=31184
Remove this error code from PositionError, both for use from C++ code and from the JS object.
Updated fast/dom/Geolocation/error.html to test this.
- page/PositionError.h: Modified. (WebCore::PositionError::): Remove ErrorCode::UNKNOWN_ERROR.
- page/PositionError.idl: Modified. Remove UNKNOWN_ERROR constant.
- page/Geolocation.cpp: Modified. (WebCore::Geolocation::startRequest): Replace UNKNOWN_ERROR with POSITION_UNAVAILABLE.
- platform/gtk/GeolocationServiceGtk.cpp: Modified. (WebCore::GeolocationServiceGtk::startUpdating): Replace UNKNOWN_ERROR with POSITION_UNAVAILABLE.
- 1:08 PM Changeset in webkit [50604] by
-
- 2 edits in trunk/WebCore
2009-11-06 Dirk Schulze <krit@webkit.org>
Reviewed by Nikolas Zimmermann.
feMorphology filter is not implemented
https://bugs.webkit.org/show_bug.cgi?id=5863
The Implementation of feMorphology.
Test: We have allready a test for feMorphology
svg/W3C-SVG-1.1/filters-morph-01-f.svg
- svg/graphics/filters/SVGFEMorphology.cpp: (WebCore::FEMorphology::apply):
- 12:01 PM Changeset in webkit [50603] by
-
- 2 edits in trunk/WebCore
2009-11-06 Steve Block <steveblock@google.com>
Reviewed by Eric Seidel.
V8DOMWindowCustom.cpp is missing WEB_SOCKETS guard on include.
https://bugs.webkit.org/show_bug.cgi?id=31209
Build fix only. No new tests possible.
- bindings/v8/custom/V8DOMWindowCustom.cpp: Modified. Added WEB_SOCKETS guard on inclue of WebSockets.h.
- 11:53 AM Changeset in webkit [50602] by
-
- 2 edits in trunk/WebKitTools
2009-11-06 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
bugzilla-tool crashed with exception
https://bugs.webkit.org/show_bug.cgi?id=31092
- Scripts/modules/bugzilla.py: Change a ',' to a '%' to fix the error.
- 10:32 AM Changeset in webkit [50601] by
-
- 2 edits in trunk/WebKitSite
2009-11-06 Jeff Schiller <codedread@gmail.com>
Reviewed by Darin Adler.
SVG Status page update
https://bugs.webkit.org/show_bug.cgi?id=31179
- projects/svg/status.xml:
- 8:18 AM Changeset in webkit [50600] by
-
- 2 edits in trunk/WebCore
2009-11-06 Drew Wilson <atwilson@chromium.org>
Reviewed by David Levin.
V8 bindings do not support SharedWorkers as event targets
https://bugs.webkit.org/show_bug.cgi?id=31199
No new tests because existing layout tests suffice (they currently
crash on Chromium)
- bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventTargetToV8Object): Added clause to create a DOM wrapper for SharedWorkers.
- 7:19 AM Changeset in webkit [50599] by
-
- 3 edits in trunk/JavaScriptCore
Unreviewed trivial buildfix after r50595.
Patch by Gabor Loki <loki@inf.u-szeged.hu> on 2009-11-06
Rename the remaining rshiftPtr calls to rshift32
- jit/JITArithmetic.cpp:
(JSC::JIT::emit_op_rshift):
- jit/JITInlineMethods.h:
(JSC::JIT::emitFastArithImmToInt):
- 5:49 AM Changeset in webkit [50598] by
-
- 2 edits in trunk/WebCore
2009-11-06 Anton Muhin <antonm@chromium.org>
Reviewed by Adam Barth.
Do not unnecessarly synchronzie in weak reference callbacks.
https://bugs.webkit.org/show_bug.cgi?id=31191
- bindings/v8/DOMData.h: (WebCore::DOMData::handleWeakObject):
- 5:41 AM Changeset in webkit [50597] by
-
- 3 edits in trunk/WebCore
2009-11-06 Alexander Pavlov <apavlov@chromium.org>
Reviewed by Pavel Feldman.
Fix resource content search.
https://bugs.webkit.org/show_bug.cgi?id=31202
- inspector/front-end/AbstractTimelinePanel.js: (WebInspector.AbstractTimelinePanel.prototype.refresh): (WebInspector.AbstractTimelinePanel.prototype.removeItem):
- inspector/front-end/TimelinePanel.js: (WebInspector.TimelineGraph.prototype.refresh):
- 5:33 AM WebKit Team edited by
- (diff)
- 5:28 AM WebKit Team edited by
- (diff)
- 5:27 AM WebKit Team edited by
- (diff)
- 5:25 AM WebKit Team edited by
- (diff)
- 1:35 AM Changeset in webkit [50596] by
-
- 3 edits in trunk/LayoutTests
2009-11-04 Shinichiro Hamaji <hamaji@chromium.org>
Reviewed by Darin Adler.
css2.1/t040304-c64-uri-00-a-g.html is failing
https://bugs.webkit.org/show_bug.cgi?id=23271
This test isn't failing now because this test was fixed in
http://trac.webkit.org/changeset/47884
- platform/mac/css2.1/t040304-c64-uri-00-a-g-expected.checksum:
- platform/mac/css2.1/t040304-c64-uri-00-a-g-expected.png:
- 1:35 AM Changeset in webkit [50595] by
-
- 7 edits in trunk/JavaScriptCore
Tidy up the shift methods on the macro-assembler interface.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-06
Reviewed by Oliver Hunt.
Currently behaviour of shifts of a magnitude > 0x1f is undefined.
Instead defined that all shifts are masked to this range. This makes a lot of
practical sense, both since having undefined behaviour is not particularly
desirable, and because this behaviour is commonly required (particularly since
it is required bt ECMA-262 for shifts).
Update the ARM assemblers to provide this behaviour. Remove (now) redundant
masks from JITArithmetic, and remove rshiftPtr (this was used in case that
could be rewritten in a simpler form using rshift32, only optimized JSVALUE32
on x86-64, which uses JSVALUE64!)
- assembler/MacroAssembler.h:
- assembler/MacroAssemblerARM.h:
(JSC::MacroAssemblerARM::lshift32):
(JSC::MacroAssemblerARM::rshift32):
- assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::lshift32):
(JSC::MacroAssemblerARMv7::rshift32):
- assembler/MacroAssemblerX86_64.h:
- jit/JITArithmetic.cpp:
(JSC::JIT::emit_op_lshift):
(JSC::JIT::emit_op_rshift):
- 12:47 AM Changeset in webkit [50594] by
-
- 4 edits in trunk/JavaScriptCore
Rubber Stamped by Oliver Hunt.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-05
Remove a magic number (1) from the JIT, instead compute the value with OBJECT_OFFSET.
- jit/JITInlineMethods.h:
(JSC::JIT::emitPutJITStubArg):
(JSC::JIT::emitPutJITStubArgConstant):
(JSC::JIT::emitGetJITStubArg):
(JSC::JIT::emitPutJITStubArgFromVirtualRegister):
- jit/JITStubCall.h:
(JSC::JITStubCall::JITStubCall):
(JSC::JITStubCall::getArgument):
- jit/JITStubs.h:
Nov 5, 2009:
- 11:28 PM Changeset in webkit [50593] by
-
- 3 edits in trunk/JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=31159
Fix branchDouble behaviour on ARM THUMB2 JIT.
Patch by Zoltan Herczeg <zherczeg@inf.u-szeged.hu> on 2009-11-05
Reviewed by Gavin Barraclough.
The x86 branchDouble behaviour is reworked, and all JIT
ports should follow the x86 port. See bug 31104 and 31151
This patch contains a fix for the traditional ARM port
- assembler/ARMAssembler.h:
(JSC::ARMAssembler::):
(JSC::ARMAssembler::fmrs_r):
(JSC::ARMAssembler::ftosid_r):
- assembler/MacroAssemblerARM.h:
(JSC::MacroAssemblerARM::):
(JSC::MacroAssemblerARM::branchDouble):
(JSC::MacroAssemblerARM::branchConvertDoubleToInt32):
- 11:26 PM Changeset in webkit [50592] by
-
- 2 edits in trunk/WebCore
2009-11-05 Yuta Kitamura <yutak@chromium.org>
Reviewed by Eric Seidel.
Fix ASSERT(currentStyle = renderStyle()).
https://bugs.webkit.org/show_bug.cgi?id=31152
- dom/Element.cpp: (WebCore::Element::pseudoStyleCacheIsInvalid): We should have used "==" instead of "=".
- 10:52 PM Maintenance and architecture list edited by
- Changeset 50583 removed the "this is part of the KDE project" code … (diff)
- 10:52 PM Changeset in webkit [50591] by
-
- 19 edits in trunk
Rolled out r50590 because it doesn't build on Windows.
- 10:26 PM Changeset in webkit [50590] by
-
- 19 edits in trunk
JavaScriptCore: https://bugs.webkit.org/show_bug.cgi?id=31197
Implemented a timezone cache not based on Mac OS X's notify_check API.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-05
Reviewed by Oliver Hunt.
If the VM calculates the local timezone offset from UTC, it caches the
result until the end of the current VM invocation. (We don't want to cache
forever, because the user's timezone may change over time.)
This removes notify_* overhead on Mac, and, more significantly, removes
OS time and date call overhead on non-Mac platforms.
~8% speedup on Date microbenchmark on Mac. SunSpider reports maybe a tiny
speedup on Mac. (Speedup on non-Mac platforms should be even more noticeable.)
- JavaScriptCore.exp:
- interpreter/CachedCall.h:
(JSC::CachedCall::CachedCall):
- interpreter/Interpreter.cpp:
(JSC::Interpreter::execute):
- runtime/JSGlobalObject.h:
(JSC::DynamicGlobalObjectScope::DynamicGlobalObjectScope): Made the
DynamicGlobalObjectScope constructor responsible for checking whether a
dynamicGlobalObject has already been set. This eliminated some duplicate
client code, and allowed me to avoid adding even more duplicate client
code. Made DynamicGlobalObjectScope responsible for resetting the
local timezone cache upon first entry to the VM.
- runtime/DateConstructor.cpp:
(JSC::constructDate):
(JSC::callDate):
(JSC::dateParse):
(JSC::dateUTC):
- runtime/DateConversion.cpp:
(JSC::parseDate):
- runtime/DateConversion.h:
- runtime/DateInstance.cpp:
(JSC::DateInstance::gregorianDateTime):
- runtime/DateInstance.h:
- runtime/DateInstanceCache.h:
- runtime/DatePrototype.cpp:
(JSC::setNewValueFromTimeArgs):
(JSC::setNewValueFromDateArgs):
(JSC::dateProtoFuncSetYear):
- runtime/InitializeThreading.cpp:
(JSC::initializeThreadingOnce):
- runtime/JSGlobalData.cpp:
(JSC::JSGlobalData::JSGlobalData):
- runtime/JSGlobalData.h:
- wtf/DateMath.cpp:
(WTF::getCurrentUTCTime):
(WTF::getCurrentUTCTimeWithMicroseconds):
(WTF::getLocalTime):
(JSC::getUTCOffset): Use the new cache. Also, see below.
(JSC::gregorianDateTimeToMS):
(JSC::msToGregorianDateTime):
(JSC::initializeDates):
(JSC::parseDateFromNullTerminatedCharacters): Simplified the way this function
accounts for the local timezone offset, to accomodate our new caching API,
and a (possibly misguided) caller in WebCore. Also, see below.
- wtf/DateMath.h:
(JSC::GregorianDateTime::GregorianDateTime): Moved most of the code in
DateMath.* into the JSC namespace. The code needed to move so it could
naturally interact with ExecState and JSGlobalData to support caching.
Logically, it seemed right to move it, too, since this code is not really
as low-level as the WTF namespace might imply -- it implements a set of
date parsing and conversion quirks that are finely tuned to the JavaScript
language. Also removed the Mac OS X notify_* infrastructure.
WebCore: https://bugs.webkit.org/show_bug.cgi?id=31197
Implemented a timezone cache not based on Mac OS X's notify_check API.
Patch by Geoffrey Garen <ggaren@apple.com> on 2009-11-05
Updated for JavaScriptCore internal API change.
- platform/network/HTTPParsers.cpp:
(WebCore::parseDate): Pass 0 for ExecState, since we don't have one.
(This function probably shouldn't be using a JavaScript date parser
to begin with, but oh well.)
- 10:25 PM Changeset in webkit [50589] by
-
- 2 edits2 deletes in trunk/LayoutTests
2009-11-05 Daniel Bates <dbates@webkit.org>
Reviewed by Eric Seidel.
Removed obsolete test case which was included in the patch for bug #30107.
This test case is already covered by the test case fast/events/drag-and-drop.html,
which was included in the patch for bug #24731.
- fast/events/drag-and-drop-none-expected.txt: Removed.
- fast/events/drag-and-drop-none.html: Removed.
- platform/win/Skipped: Removed entry for drag-and-drop-none.html.
- 10:19 PM Changeset in webkit [50588] by
-
- 4 edits in trunk/WebCore
Revert 50562 because it broke Chromium. Not reviewed since this is a build fix and revert.
Patch by Alpha Lam <hclam@chromium.org> on 2009-11-05
- bindings/v8/V8GCController.cpp:
(WebCore::V8GCController::gcEpilogue):
- bindings/v8/V8GCController.h:
- bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::evaluate):
(WebCore::V8Proxy::runScript):
(WebCore::V8Proxy::callFunction):
- 10:05 PM Changeset in webkit [50587] by
-
- 3 edits4 adds in trunk
2009-11-05 Adam Barth <abarth@webkit.org>
Reviewed by Sam Weinig.
Cross-domain access to stylesheet text should not be allowed
https://bugs.webkit.org/show_bug.cgi?id=20527
Test that a script cannot read cross-origin cssRules.
- http/tests/security/cannot-read-cssrules-expected.txt: Added.
- http/tests/security/cannot-read-cssrules-redirect-expected.txt: Added.
- http/tests/security/cannot-read-cssrules-redirect.html: Added.
- http/tests/security/cannot-read-cssrules.html: Added.
2009-11-05 Adam Barth <abarth@webkit.org>
Reviewed by Sam Weinig.
Cross-domain access to stylesheet text should not be allowed
https://bugs.webkit.org/show_bug.cgi?id=20527
Check whether whether the current document can read the cssRules from
the style sheet. Firefox throws a security error here, but we return
null instead because that's what we usually do in these cases.
Test: http/tests/security/cannot-read-cssrules-redirect.html
http/tests/security/cannot-read-cssrules.html
- css/CSSStyleSheet.cpp: (WebCore::CSSStyleSheet::cssRules):
- 7:27 PM Changeset in webkit [50586] by
-
- 1 edit in trunk/WebCore/WebCore.gyp
Rubber stamped by Mark Rowe.
Adds some GYP generated files to svn:ignore
https://bugs.webkit.org/show_bug.cgi?id=31194
*.vcproj
*.rules
*.mk
*.scons
SConstruct
- 6:46 PM Changeset in webkit [50585] by
-
- 2 edits in trunk/WebCore
2009-11-05 Steve Block <steveblock@google.com>
Reviewed by Eric Seidel.
If the Geolocation service fails to start, invoke the error callback asynchronously.
https://bugs.webkit.org/show_bug.cgi?id=28276
All Geolocation callbacks must be invoked asynchronously.
See http://www.w3.org/TR/geolocation-API/#geolocation_interface
No new tests possible with current LayoutTestController.
- page/Geolocation.cpp: (WebCore::Geolocation::getCurrentPosition): Modified. Asserts that startRequest returned a notifier. (WebCore::Geolocation::watchPosition): Modified. Asserts that startRequest returned a notifier. (WebCore::Geolocation::startRequest): Modified. If the Geolocation service fails to start, set a fatal error on the notifier.
- 4:05 PM Changeset in webkit [50584] by
-
- 1 edit1 add in trunk/LayoutTests
Adding a resource that seems to be missing. Without it, several tests in
fast/dom/beforeload aren't really testing if the default event handling is
prevented, but instead are only testing if the beforeload event fires.
Rubber-stamped by Gavin Barraclough.
- fast/dom/beforeload/resources/fail.js: Added.
- 3:53 PM Changeset in webkit [50583] by
-
- 253 edits in trunk
2009-11-05 Chris Jerdonek <chris.jerdonek@gmail.com>
Reviewed by Eric Seidel.
Removed the "this is part of the KDE project" comments from
all *.h, *.cpp, *.idl, and *.pm files.
The maintenance and architecture page in the project wiki lists
this as a task.
This change includes no changes or additions to test cases
since the change affects only comments.
- wtf/wince/FastMallocWince.h:
2009-11-05 Chris Jerdonek <chris.jerdonek@gmail.com>
Reviewed by Eric Seidel.
Removed the "this is part of the KDE project" comments from
all *.h, *.cpp, *.idl, and *.pm files.
The maintenance and architecture page in the project wiki lists
this as a task.
This change includes no changes or additions to test cases
since the change affects only comments.
- bindings/scripts/CodeGeneratorV8.pm:
- bindings/scripts/IDLParser.pm:
- bindings/scripts/IDLStructure.pm:
- css/CSSInheritedValue.cpp:
- css/CSSInitialValue.cpp:
- css/CSSMediaRule.cpp:
- css/CSSNamespace.h:
- css/CSSProperty.cpp:
- css/CSSProperty.h:
- css/CSSRuleList.cpp:
- css/CSSRuleList.h:
- css/CSSSelector.h:
- css/CSSValueList.cpp:
- css/FontValue.cpp:
- css/MediaFeatureNames.cpp:
- css/MediaFeatureNames.h:
- css/Pair.h:
- css/SVGCSSStyleSelector.cpp:
- css/ShadowValue.cpp:
- css/StyleSheet.cpp:
- css/StyleSheetList.cpp:
- css/maketokenizer:
- dom/BeforeUnloadEvent.cpp:
- dom/BeforeUnloadEvent.h:
- dom/CSSMappedAttributeDeclaration.cpp:
- dom/EventNames.cpp:
- dom/EventTarget.cpp:
- dom/MappedAttributeEntry.h:
- dom/MouseRelatedEvent.h:
- dom/RangeException.h:
- dom/StyleElement.h:
- dom/Tokenizer.h:
- html/HTMLHeadElement.h:
- html/HTMLHeadingElement.cpp:
- html/HTMLHeadingElement.h:
- html/HTMLHtmlElement.h:
- html/HTMLImageLoader.h:
- html/HTMLMetaElement.h:
- html/HTMLModElement.cpp:
- html/HTMLModElement.h:
- html/HTMLOptionsCollection.cpp:
- html/HTMLPlugInElement.cpp:
- html/HTMLPreElement.cpp:
- html/HTMLPreElement.h:
- html/HTMLTableCellElement.cpp:
- html/HTMLTableCellElement.h:
- html/HTMLTableColElement.cpp:
- html/HTMLTableColElement.h:
- html/HTMLTablePartElement.cpp:
- html/HTMLTablePartElement.h:
- html/HTMLTitleElement.h:
- page/MouseEventWithHitTestResults.h:
- platform/StaticConstructors.h:
- platform/text/AtomicStringImpl.h:
- platform/text/qt/TextBreakIteratorQt.cpp:
- rendering/AutoTableLayout.h:
- rendering/CounterNode.cpp:
- rendering/EllipsisBox.cpp:
- rendering/EllipsisBox.h:
- rendering/FixedTableLayout.cpp:
- rendering/FixedTableLayout.h:
- rendering/HitTestRequest.h:
- rendering/HitTestResult.h:
- rendering/InlineRunBox.h:
- rendering/PointerEventsHitRules.cpp:
- rendering/PointerEventsHitRules.h:
- rendering/RenderBR.cpp:
- rendering/RenderBR.h:
- rendering/RenderButton.cpp:
- rendering/RenderButton.h:
- rendering/RenderFieldset.cpp:
- rendering/RenderFrameSet.cpp:
- rendering/RenderListItem.cpp:
- rendering/RenderTableRow.cpp:
- rendering/RenderView.h:
- rendering/RootInlineBox.h:
- rendering/SVGInlineTextBox.cpp:
- rendering/SVGInlineTextBox.h:
- rendering/TableLayout.h:
- rendering/break_lines.h:
- rendering/style/SVGRenderStyle.cpp:
- rendering/style/SVGRenderStyle.h:
- rendering/style/SVGRenderStyleDefs.cpp:
- rendering/style/SVGRenderStyleDefs.h:
- svg/GradientAttributes.h:
- svg/LinearGradientAttributes.h:
- svg/PatternAttributes.h:
- svg/RadialGradientAttributes.h:
- svg/SVGAElement.cpp:
- svg/SVGAngle.idl:
- svg/SVGAnimateColorElement.cpp:
- svg/SVGAnimateColorElement.h:
- svg/SVGAnimateElement.cpp:
- svg/SVGAnimateElement.h:
- svg/SVGAnimateTransformElement.h:
- svg/SVGAnimatedPathData.cpp:
- svg/SVGAnimatedPathData.h:
- svg/SVGAnimatedPoints.cpp:
- svg/SVGAnimatedPoints.h:
- svg/SVGAnimationElement.cpp:
- svg/SVGCircleElement.cpp:
- svg/SVGClipPathElement.cpp:
- svg/SVGColor.cpp:
- svg/SVGColor.idl:
- svg/SVGComponentTransferFunctionElement.cpp:
- svg/SVGComponentTransferFunctionElement.h:
- svg/SVGCursorElement.cpp:
- svg/SVGDefsElement.cpp:
- svg/SVGDescElement.cpp:
- svg/SVGDescElement.h:
- svg/SVGDocument.idl:
- svg/SVGElement.idl:
- svg/SVGElementInstanceList.cpp:
- svg/SVGElementInstanceList.h:
- svg/SVGEllipseElement.cpp:
- svg/SVGExternalResourcesRequired.cpp:
- svg/SVGFEBlendElement.cpp:
- svg/SVGFEBlendElement.h:
- svg/SVGFEColorMatrixElement.cpp:
- svg/SVGFEColorMatrixElement.h:
- svg/SVGFEComponentTransferElement.cpp:
- svg/SVGFEComponentTransferElement.h:
- svg/SVGFECompositeElement.cpp:
- svg/SVGFECompositeElement.h:
- svg/SVGFEFloodElement.cpp:
- svg/SVGFEFloodElement.h:
- svg/SVGFEFuncAElement.cpp:
- svg/SVGFEFuncAElement.h:
- svg/SVGFEFuncBElement.cpp:
- svg/SVGFEFuncBElement.h:
- svg/SVGFEFuncGElement.cpp:
- svg/SVGFEFuncGElement.h:
- svg/SVGFEFuncRElement.cpp:
- svg/SVGFEFuncRElement.h:
- svg/SVGFEGaussianBlurElement.cpp:
- svg/SVGFEGaussianBlurElement.h:
- svg/SVGFEImageElement.cpp:
- svg/SVGFEMergeElement.cpp:
- svg/SVGFEMergeElement.h:
- svg/SVGFEMergeNodeElement.cpp:
- svg/SVGFEOffsetElement.cpp:
- svg/SVGFEOffsetElement.h:
- svg/SVGFETileElement.cpp:
- svg/SVGFETileElement.h:
- svg/SVGFETurbulenceElement.cpp:
- svg/SVGFETurbulenceElement.h:
- svg/SVGFilterElement.cpp:
- svg/SVGGElement.cpp:
- svg/SVGGradientElement.cpp:
- svg/SVGHKernElement.idl:
- svg/SVGLangSpace.cpp:
- svg/SVGLangSpace.h:
- svg/SVGLength.cpp:
- svg/SVGLength.h:
- svg/SVGLength.idl:
- svg/SVGLengthList.cpp:
- svg/SVGLengthList.h:
- svg/SVGLineElement.cpp:
- svg/SVGLinearGradientElement.h:
- svg/SVGList.h:
- svg/SVGListTraits.h:
- svg/SVGLocatable.h:
- svg/SVGMaskElement.cpp:
- svg/SVGMatrix.idl:
- svg/SVGMetadataElement.cpp:
- svg/SVGMetadataElement.h:
- svg/SVGMetadataElement.idl:
- svg/SVGNumber.idl:
- svg/SVGNumberList.cpp:
- svg/SVGNumberList.h:
- svg/SVGPaint.cpp:
- svg/SVGPathElement.cpp:
- svg/SVGPathSeg.h:
- svg/SVGPathSegArc.cpp:
- svg/SVGPathSegArc.h:
- svg/SVGPathSegClosePath.cpp:
- svg/SVGPathSegClosePath.h:
- svg/SVGPathSegCurvetoCubic.cpp:
- svg/SVGPathSegCurvetoCubic.h:
- svg/SVGPathSegCurvetoCubicSmooth.cpp:
- svg/SVGPathSegCurvetoCubicSmooth.h:
- svg/SVGPathSegCurvetoQuadratic.cpp:
- svg/SVGPathSegCurvetoQuadratic.h:
- svg/SVGPathSegCurvetoQuadraticSmooth.cpp:
- svg/SVGPathSegCurvetoQuadraticSmooth.h:
- svg/SVGPathSegLineto.cpp:
- svg/SVGPathSegLineto.h:
- svg/SVGPathSegLinetoHorizontal.cpp:
- svg/SVGPathSegLinetoHorizontal.h:
- svg/SVGPathSegLinetoVertical.cpp:
- svg/SVGPathSegLinetoVertical.h:
- svg/SVGPathSegMoveto.cpp:
- svg/SVGPathSegMoveto.h:
- svg/SVGPatternElement.cpp:
- svg/SVGPoint.idl:
- svg/SVGPointList.cpp:
- svg/SVGPointList.h:
- svg/SVGPolyElement.cpp:
- svg/SVGPolygonElement.cpp:
- svg/SVGPolygonElement.h:
- svg/SVGPolylineElement.cpp:
- svg/SVGPolylineElement.h:
- svg/SVGPreserveAspectRatio.cpp:
- svg/SVGPreserveAspectRatio.h:
- svg/SVGRadialGradientElement.h:
- svg/SVGRect.idl:
- svg/SVGRectElement.cpp:
- svg/SVGRenderingIntent.h:
- svg/SVGSVGElement.idl:
- svg/SVGScriptElement.cpp:
- svg/SVGSetElement.cpp:
- svg/SVGSetElement.h:
- svg/SVGStopElement.cpp:
- svg/SVGStringList.cpp:
- svg/SVGStringList.h:
- svg/SVGStylable.cpp:
- svg/SVGStylable.h:
- svg/SVGStyleElement.cpp:
- svg/SVGStyleElement.h:
- svg/SVGStyledElement.h:
- svg/SVGStyledLocatableElement.cpp:
- svg/SVGStyledLocatableElement.h:
- svg/SVGStyledTransformableElement.cpp:
- svg/SVGStyledTransformableElement.h:
- svg/SVGSwitchElement.cpp:
- svg/SVGSymbolElement.cpp:
- svg/SVGTRefElement.cpp:
- svg/SVGTSpanElement.cpp:
- svg/SVGTSpanElement.h:
- svg/SVGTests.h:
- svg/SVGTextElement.cpp:
- svg/SVGTextElement.h:
- svg/SVGTextPathElement.cpp:
- svg/SVGTextPositioningElement.cpp:
- svg/SVGTextPositioningElement.h:
- svg/SVGTitleElement.cpp:
- svg/SVGTitleElement.h:
- svg/SVGTransform.cpp:
- svg/SVGTransform.h:
- svg/SVGTransform.idl:
- svg/SVGTransformList.cpp:
- svg/SVGTransformList.h:
- svg/SVGTransformable.h:
- svg/SVGURIReference.h:
- svg/SVGUnitTypes.h:
- svg/SVGUseElement.cpp:
- svg/SVGViewElement.cpp:
- svg/SVGZoomAndPan.cpp:
- svg/SVGZoomAndPan.h:
- svg/SVGZoomEvent.cpp:
- 3:28 PM Changeset in webkit [50582] by
-
- 17 edits3 moves5 adds2 deletes in trunk/LayoutTests
2009-11-05 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: beautify layout tests.
- inspector/console-dir-expected.txt:
- inspector/console-dir.html:
- inspector/console-dirxml-expected.txt:
- inspector/console-dirxml.html:
- inspector/console-format-collections-expected.txt:
- inspector/console-format-collections.html:
- inspector/console-format-expected.txt:
- inspector/console-format.html:
- inspector/console-tests-expected.txt:
- inspector/console-tests.html:
- inspector/console-tests.js: Added. (dumpConsoleMessages.callback): (dumpConsoleMessages): (dumpConsoleMessagesWithClasses.callback): (dumpConsoleMessagesWithClasses): (frontendScript_dumpConsoleMessages):
- inspector/css-syntax-highlight.html: Removed.
- inspector/elements-panel-structure-expected.txt:
- inspector/elements-panel-structure.html:
- inspector/elements-tests.js: Added. (frontendScript_expandDOMSubtree.processChildren): (frontendScript_expandDOMSubtree):
- inspector/evaluate-in-frontend.html:
- inspector/evaluate-in-frontend.js: Removed.
- inspector/inspector-test.js: Added. (): (evaluateInWebInspector): (notifyDone): (output): (window.didEvaluateForTestInFrontend):
- inspector/javascript-syntax-highlight.html: Removed.
- inspector/styles-iframe.html:
- inspector/syntax-highlight-css-expected.txt: Renamed from LayoutTests/inspector/css-syntax-highlight-expected.txt.
- inspector/syntax-highlight-css.html: Added.
- inspector/syntax-highlight-javascript-expected.txt: Renamed from LayoutTests/inspector/javascript-syntax-highlight-expected.txt.
- inspector/syntax-highlight-javascript.html: Added.
- inspector/syntax-highlight.js: Added. (doit.callback): (doit): (frontendScript_dumpSyntaxHighlight):
- inspector/timeline-trivial.html:
- platform/qt/Skipped:
- 3:25 PM Changeset in webkit [50581] by
-
- 5 edits in trunk/WebCore
2009-11-05 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Dimitri Glazkov.
REGRESSION Clean up security origin usage in DOM Storage.
https://bugs.webkit.org/show_bug.cgi?id=31188
Clean up security origin usage in DOM Storage. This fixes a bug in my
refactoring here: https://bugs.webkit.org/show_bug.cgi?id=31149
Instead of having StorageAreaSync's constructor (which is called in the
constructor for StorageAreaImpl) calling a method on StoargeAreaImpl to get the
database identifier, simply have StorageAreaImpl pass the identifier into
StorageAreaSync.
No test because there's no change in externally observable behavior.
- storage/StorageAreaImpl.cpp: (WebCore::StorageAreaImpl::StorageAreaImpl):
- storage/StorageAreaImpl.h:
- storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::create): (WebCore::StorageAreaSync::StorageAreaSync):
- storage/StorageAreaSync.h:
- 3:08 PM Changeset in webkit [50580] by
-
- 3 edits in trunk/WebCore
2009-11-05 Scott Violet <sky@google.com>
Reviewed by Dimitri Glazkov.
Need notification of scrolling frame
https://bugs.webkit.org/show_bug.cgi?id=31145
Adds FrameLoaderClient::didChangeScrollOffset that is called when
the frame scrolls. This will be used to know when history state
needs to be updated.
- loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::didChangeScrollOffset):
- page/FrameView.cpp: (WebCore::FrameView::valueChanged):
- 2:46 PM Changeset in webkit [50579] by
-
- 2 edits in trunk/LayoutTests
2009-11-05 Shu Chang <Chang.Shu@nokia.com>
Reviewed by Adele Peterson.
The number of selected lines in textarea after a shift-PageDown
may differ. The test case is modified so that it will not depend
on the exact number of lines selected. This should resolve regression
on platforms like Qt and GTK.
https://bugs.webkit.org/show_bug.cgi?id=31122
- editing/selection/shrink-selection-after-shift-pagedown.html:
- 2:17 PM Changeset in webkit [50578] by
-
- 4 edits in trunk/WebCore
2009-11-05 Kenneth Russell <kbr@google.com>
Reviewed by Dimitri Glazkov.
[V8] Update bindings to use new API for external arrays
https://bugs.webkit.org/show_bug.cgi?id=31181
No new tests; covered by existing WebGL tests.
- bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::setIndexedPropertiesToExternalArray): (WebCore::V8DOMWrapper::getTemplate): (WebCore::V8DOMWrapper::convertToV8Object):
- bindings/v8/V8DOMWrapper.h:
- bindings/v8/custom/V8CanvasArrayCustom.h: (WebCore::constructCanvasArray):
- 1:59 PM Changeset in webkit [50577] by
-
- 3 edits in trunk/WebCore
Not reviewed, Chromium build fix.
Patch by Alpha Lam <hclam@chromium.org> on 2009-11-05
50561 introduces a custom method for SVGMatrix, we need to implement
this in V8.
- bindings/v8/custom/V8CustomBinding.h:
Adding definition for V8SVGMatrixMultiply.
- bindings/v8/custom/V8SVGMatrixCustom.cpp:
(WebCore::CALLBACK_FUNC_DECL):
Implement V8SVGMatrixMultiply according to the same method in JSC.
- 1:09 PM Changeset in webkit [50576] by
-
- 2 edits in trunk/LayoutTests
2009-11-05 Pavel Feldman <pfeldman@chromium.org>
Not reviewed: empty line in expectations build fix.
- fast/events/event-listener-map-rehash-crash-expected.txt:
- 12:42 PM Changeset in webkit [50575] by
-
- 4 edits in trunk/WebCore
Revert 50569 since it broke QT. Build fix and just a revert, so no review.
Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-11-05
- page/DOMWindow.cpp:
(WebCore::DOMWindow::sessionStorage):
- page/Settings.cpp:
(WebCore::Settings::Settings):
(WebCore::Settings::setSessionStorageEnabled):
- page/Settings.h:
(WebCore::Settings::sessionStorageEnabled):
- 12:13 PM Changeset in webkit [50574] by
-
- 16 edits4 adds in trunk
2009-11-05 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
REGRESSION: Web Inspector doesn't show CSS rules properly for iframes
Test: inspector/styles-iframe.html
- inspector/front-end/DOMAgent.js: (WebInspector.DOMNode):
- inspector/front-end/InjectedScript.js: (InjectedScript.getStyles): (InjectedScript.getComputedStyle): (InjectedScript.addStyleSelector):
- inspector/front-end/TestController.js: (WebInspector.TestController.prototype.notifyDone): (WebInspector.TestController.prototype.runAfterPendingDispatches): (WebInspector.evaluateForTestInFrontend.invokeMethod): (WebInspector.evaluateForTestInFrontend):
- 12:09 PM Changeset in webkit [50573] by
-
- 6 edits2 adds in trunk
WebCore:
2009-11-05 Vitaly Repeshko <vitalyr@chromium.org>
Reviewed by Geoffrey Garen and Dimitri Glazkov.
Rehashing of EventListenerMap leads to loss of EvenListenerList.
https://bugs.webkit.org/show_bug.cgi?id=31027
Tested by new fast/events/event-listener-map-rehash-crash.html.
EventListenerMap modified to store pointers to listener vectors:
- dom/EventTarget.cpp: (WebCore::EventTargetData::~EventTargetData): (WebCore::EventTarget::addEventListener): (WebCore::EventTarget::removeEventListener): (WebCore::EventTarget::fireEventListeners): (WebCore::EventTarget::getEventListeners): (WebCore::EventTarget::removeAllEventListeners):
- dom/EventTarget.h:
Usages updated after interface changes:
- inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getEventListenersForNode):
- svg/SVGUseElement.cpp: (WebCore::SVGUseElement::transferEventListenersToShadowTree):
LayoutTests:
2009-11-05 Dimitri Glazkov <Dimitri Glazkov>
Reviewed by Geoffrey Garen.
Rehashing of EventListenerMap leads to loss of EvenListenerList.
https://bugs.webkit.org/show_bug.cgi?id=31027
- fast/events/event-listener-map-rehash-crash.html: Added.
- 12:05 PM Changeset in webkit [50572] by
-
- 2 edits in trunk/LayoutTests
Rubber-stamped by Adam Roben.
Add ruby tests and failing accessibility test to Windows Skipped list,
we should be able to re-add ruby tests, but not sure the correct course
of action yet.
- platform/win/Skipped:
- 11:57 AM Changeset in webkit [50571] by
-
- 2 edits in trunk/WebCore
2009-11-05 Dan Kegel <dank@chromium.org>
Reviewed by Dmitri Titov.
Add missing initialization for m_createdByParser.
https://bugs.webkit.org/show_bug.cgi?id=31089
Test: fast/dom/beforeload/pi-before-load.xhtml in Valgrind
- dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::ProcessingInstruction):
- 11:33 AM Changeset in webkit [50570] by
-
- 2 edits in trunk/WebKitTools
2009-11-04 Eric Seidel <eric@webkit.org>
No review, just add a line which got left out of the patch uploaded for commit.
commit-queue is crashing when trying to reject patches after unknown failures
https://bugs.webkit.org/show_bug.cgi?id=31091
- Scripts/bugzilla-tool: Set bug_log = None after closing to we don't re-close and crash.
- 11:27 AM Changeset in webkit [50569] by
-
- 4 edits in trunk/WebCore
2009-11-04 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Adam Barth.
DOM Storage runtime flag changes
https://bugs.webkit.org/show_bug.cgi?id=30602
Part 2/2.
Revert my changes to Settings and instead implement DOM Storage enabling via
the methods agreed upon in https://bugs.webkit.org/show_bug.cgi?id=30240
This stuff was (intentionally) never exposed to web pages or DRT, so there's no
LayoutTest visible changes and thus no tests.
- bindings/v8/custom/V8CustomBinding.h:
- bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::ACCESSOR_RUNTIME_ENABLER):
- page/DOMWindow.cpp: (WebCore::DOMWindow::sessionStorage):
- page/DOMWindow.idl:
- page/Settings.cpp: (WebCore::Settings::Settings):
- page/Settings.h:
- storage/Storage.cpp: (WebCore::Storage::setLocalStorageAvailable): (WebCore::Storage::localStorageAvailable): (WebCore::Storage::setSessionStorageAvailable): (WebCore::Storage::sessionStorageAvailable):
- storage/Storage.h:
- 10:33 AM Changeset in webkit [50568] by
-
- 3 edits in trunk/WebCore
Bug 31108 - [V8] REGRESSION: Pause on exception is broken
https://bugs.webkit.org/show_bug.cgi?id=31108
Reviewed by Pavel Feldman.
- bindings/v8/V8AbstractEventListener.cpp:
(WebCore::V8AbstractEventListener::invokeEventHandler):
- bindings/v8/V8Utilities.cpp:
(WebCore::reportException):
- 10:14 AM Changeset in webkit [50567] by
-
- 10 edits in trunk/WebKit/win
Make CFDictionaryPropertyBag::createInstance return a COMPtr
I also cleaned up CFDictionaryPropertyBag's class declaration a little
while I was at it.
Part of <http://webkit.org/b/25294> <rdar://problem/6803127> All
WebKit/win classes should return COMPtrs from their static constructor
members
Reviewed by Steve Falkenburg.
- CFDictionaryPropertyBag.cpp:
(CFDictionaryPropertyBag::CFDictionaryPropertyBag): Changed to
initialize m_refCount to 0. m_refCount gets increased to 1 by
createInstance.
(CFDictionaryPropertyBag::createInstance): Changed to return a COMPtr.
- CFDictionaryPropertyBag.h:
- Updated copyright years
- Fixed header guard to match current style
- Replaced #include of CoreFoundation.h with forward-declaration of CFMutableDictionaryRef
- Added #include of COMPtr.h
- Fixed opening brace placement in class declaration
- Made createInstance return a COMPtr
- Made constructor/destructor private
- Made QueryInterface and IPropertyBag functions private
- Removed unnecessary parameter names and MIDL comments
- WebCache.cpp:
(WebCache::statistics): Updated for change to
CFDictionaryPropertyBag::createInstance. Now uses releaseRef to place
the IPropertyBag pointers into the s array.
- WebCoreSupport/WebFrameLoaderClient.cpp:
(WebFrameLoaderClient::dispatchDidFailToStartPlugin):
- WebDatabaseManager.cpp:
(WebDatabaseManager::dispatchDidModifyDatabase):
- WebFrame.cpp:
(WebFrame::createJavaAppletWidget):
Updated for change to CFDictionaryPropertyBag::createInstance.
- WebHistory.cpp: Removed releaseUserInfo, which is no longer needed.
That function was also doing an unnecessary call to setDictionary(0).
(createUserInfoFromArray):
(createUserInfoFromHistoryItem):
Changed to return COMPtr.
(WebHistory::loadFromURL): Updated for change to
CFDictionaryPropertyBag::createInstance.
(WebHistory::removeAllItems): Updated for change to
CFDictionaryPropertyBag::createInstance, which fixed a leak! We had
forgotten to call releaseUserInfo here.
(WebHistory::removeItem):
(WebHistory::addItem):
(WebHistory::visitedURL):
- WebIconDatabase.cpp:
(postDidAddIconNotification):
Updated for change to CFDictionaryPropertyBag::createInstance.
- WebKitClassFactory.cpp:
(releaseRefFromCreateInstance): Added this overloaded function
template to abstract away the difference between createInstance
functions that return a ref'd pointer and createInstance functions
that return a COMPtr.
(WebKitClassFactory::CreateInstance): Changed to use
releaseRefFromCreateInstance.
- 9:59 AM Changeset in webkit [50566] by
-
- 4 edits2 adds in trunk
WebCore: We should not bubble up events if we drag something to an iframe that
has an invalid source.
https://bugs.webkit.org/show_bug.cgi?id=30469
Reviewed by Dmitri Titov.
Test: http/tests/misc/bubble-drag-events.html
- page/EventHandler.cpp:
(WebCore::EventHandler::handleDragAndDropForTarget):
(WebCore::EventHandler::updateDragAndDrop):
(WebCore::EventHandler::cancelDragAndDrop):
(WebCore::EventHandler::performDragAndDrop):
- page/EventHandler.h:
(WebCore::EventHandler::):
LayoutTests: Tests the drag event bubbling on different targets:
- iframe with invalid source
- iframe with valid source
- non-iframe droppable element
https://bugs.webkit.org/show_bug.cgi?id=30469
Reviewed by Dmitri Titov.
- http/tests/misc/bubble-drag-events-expected.txt: Added.
- http/tests/misc/bubble-drag-events.html: Added.
- 8:42 AM Changeset in webkit [50565] by
-
- 2 edits in trunk/WebKitTools
Remove bogus else clause in bugzilla-tool
Fixes <http://webkit.org/b/31125> REGRESSION (r47121): bugzilla-tool
create-bug raises exception after entering bug description
Reviewed by David Kilzer.
- Scripts/bugzilla-tool:
(CreateBug): else clauses are only hit if no exception is raised, so
it makes no sense to try to re-raise the exception in an else clause.
The exception will automatically be re-raised if it doesn't match any
of the except clauses, so we don't have to do anything special here at
all to get the desired behavior.
- 7:58 AM Changeset in webkit [50564] by
-
- 2 edits in trunk/WebCore
2009-11-05 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: speed up Timelines Clear by a factor of thousands.
- inspector/front-end/utilities.js: (Element.prototype.removeChildren):
- 7:48 AM Changeset in webkit [50563] by
-
- 2 edits in trunk/WebKit/qt
2009-11-05 Shu Chang <Chang.Shu@nokia.com>
Reviewed by Tor Arne Vestbø.
Add support for Shift-PageUp and Shift-PageDown key events.
https://bugs.webkit.org/show_bug.cgi?id=31166
Test: LayoutTests/editing/selection/shrink-selection-after-shift-pagedown.html
- WebCoreSupport/EditorClientQt.cpp: (WebCore::EditorClientQt::handleKeyboardEvent):
- 7:36 AM Changeset in webkit [50562] by
-
- 4 edits in trunk/WebCore
2009-11-05 Anton Muhin <antonm@chromium.org>
Reviewed by Adam Barth.
If high memory usage is detected, hint to V8 that it might be due
to external objects retained by V8 objects.
https://bugs.webkit.org/show_bug.cgi?id=31051
- bindings/v8/V8GCController.cpp: (WebCore::GetMemoryUsageInMB): (WebCore::V8GCController::gcEpilogue): (WebCore::V8GCController::checkMemoryUsage):
- bindings/v8/V8GCController.h:
- bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::evaluate): (WebCore::V8Proxy::runScript): (WebCore::V8Proxy::callFunction):
- 7:28 AM Changeset in webkit [50561] by
-
- 4 edits2 adds in trunk
2009-11-05 Jeff Schiller <codedread@gmail.com>
Reviewed by Simon Fraser.
Added test to ensure proper order of matrix multiplication with SVGMatrix.
https://bugs.webkit.org/show_bug.cgi?id=16062
- svg/dom/SVGMatrix-interface-expected.txt: Added.
- svg/dom/SVGMatrix-interface.xhtml: Added.
2009-11-05 Jeff Schiller <codedread@gmail.com>
Reviewed by Simon Fraser.
Correct order of matrix multiplication for SVGMatrix.
https://bugs.webkit.org/show_bug.cgi?id=16062
Test: svg/dom/SVGMatrix-interface.xhtml
- bindings/js/JSSVGMatrixCustom.cpp: (WebCore::JSSVGMatrix::multiply):
- svg/SVGMatrix.idl:
- 2:06 AM Changeset in webkit [50560] by
-
- 4 edits in trunk/WebCore
2009-11-04 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: Make resource-related records in timeline
actually take some time.
- English.lproj/localizedStrings.js:
- inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._formatRecord): (WebInspector.TimelinePanel.prototype._getRecordDetails): (WebInspector.TimelinePanel.prototype.reset):
- 1:51 AM Changeset in webkit [50559] by
-
- 2 edits in trunk
2009-11-05 Priit Laes <plaes@plaes.org>
Reviewed by Jan Alonzo.
[Gtk] Build failure with --enable-introspection
https://bugs.webkit.org/show_bug.cgi?id=31102
Add search and include paths for JSCore-1.0.gir required by
gobject-introspection tools.
- GNUmakefile.am:
- 1:31 AM Changeset in webkit [50558] by
-
- 6 edits in trunk/WebCore
2009-11-04 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Simplify LocalStorageThread
https://bugs.webkit.org/show_bug.cgi?id=30935
This is a re-submit of 50519. LocalStorageTask should have never been ref
counted. I've removed that and switched a PassRefPtr over to a PassOwnPtr.
On LocalStoragethread: Remove reference counting. Get rid of locking. Make some
of the method names a bit more clear. Assert proper thread usage. Join rather
than detaching the thread and doing an ad-hoc form of join. Avoid touching
variables on the background thread when simple to do so. Also create a generic
scheduleTask function rather than one for each task.
No behavior should have changed.
- storage/LocalStorageTask.h:
- storage/LocalStorageThread.cpp: (WebCore::LocalStorageThread::create): (WebCore::LocalStorageThread::LocalStorageThread): (WebCore::LocalStorageThread::~LocalStorageThread): (WebCore::LocalStorageThread::start): (WebCore::LocalStorageThread::threadEntryPointCallback): (WebCore::LocalStorageThread::threadEntryPoint): (WebCore::LocalStorageThread::scheduleTask): (WebCore::LocalStorageThread::terminate): (WebCore::LocalStorageThread::performTerminate):
- storage/LocalStorageThread.h:
- storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::StorageSyncManager): (WebCore::StorageSyncManager::~StorageSyncManager): (WebCore::StorageSyncManager::scheduleImport): (WebCore::StorageSyncManager::scheduleSync):
- storage/StorageSyncManager.h:
- 12:58 AM Changeset in webkit [50557] by
-
- 5 edits in trunk/WebCore
2009-11-04 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Calling databaseIdentifier on LocalStorage's background thread is not safe.
https://bugs.webkit.org/show_bug.cgi?id=31149
Calling SecurityOrigin::databaseIdentifier on LocalStorage's background thread
is not safe. databaseIdentifier does a bunch of string concatenation which
ref-counts StringImpls in some cases. This was caught by valgrind thread
sanitizer: http://code.google.com/p/chromium/issues/detail?id=25645
There's no way to test for such racyness, unfortunately.
- storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): (WebCore::StorageAreaSync::performImport):
- storage/StorageAreaSync.h:
- storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::fullDatabaseFilename):
- storage/StorageSyncManager.h:
- 12:40 AM Changeset in webkit [50556] by
-
- 2 edits in trunk/WebKit/qt
Last Qt 4.5 build fix (*sigh*)
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-05
- tests/qwebpage/tst_qwebpage.cpp:
(inputMethodHints): inputMethodHints() is only used for
Qt 4.6, so guard the whole function.
- 12:36 AM Changeset in webkit [50555] by
-
- 4 edits in trunk/WebCore
2009-11-03 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Clean up StorageAreaSync
https://bugs.webkit.org/show_bug.cgi?id=31100
Major fixes: Break the ref count cycle for StorageArea on the main
thread, not the background thread since the latter is not safe.
Length() needs to block on the import completing.
Small fixes: setItem needs to handle the copy on write case even if it
has an exception. setItem and removeItem should just bail from the
the function if the value hasn't changed rather than wrapping the end
in an if block. Clear should only send an event if it wasn't already
cleared. StorageAreaSync should assert that the final sync was
scheduled.
- storage/StorageAreaImpl.cpp: (WebCore::StorageAreaImpl::length): Forgot to block on the import. (WebCore::StorageAreaImpl::key): (WebCore::StorageAreaImpl::setItem): Handle the copy on write case even when there's an exception. (WebCore::StorageAreaImpl::removeItem): (WebCore::StorageAreaImpl::clear):
- storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::~StorageAreaSync): (WebCore::StorageAreaSync::scheduleFinalSync): (WebCore::StorageAreaSync::performImport): (WebCore::StorageAreaSync::markImported): (WebCore::StorageAreaSync::blockUntilImportComplete):
- storage/StorageAreaSync.h:
- 12:34 AM Web Inspector edited by
- Adding latest blog post (diff)
- 12:30 AM Changeset in webkit [50554] by
-
- 2 edits in trunk/WebKit/qt
Another prospective build fix against Qt 4.5 (build bot)
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-05
Don't compile & run the QGraphicsWebView portion of the
input methods auto test with Qt 4.5.
- tests/qwebpage/tst_qwebpage.cpp:
(tst_QWebPage::inputMethods_data):
(inputMethodHints):
(inputMethodEnabled):
(tst_QWebPage::inputMethods):
- 12:28 AM Changeset in webkit [50553] by
-
- 3 edits in trunk/JavaScriptCore
Use ARMv7 specific encoding for immediate constants on ARMv7 target
https://bugs.webkit.org/show_bug.cgi?id=31060
Patch by Gabor Loki <loki@inf.u-szeged.hu> on 2009-11-05
Reviewed by Gavin Barraclough.
- assembler/ARMAssembler.cpp:
(JSC::ARMAssembler::getOp2): Use INVALID_IMM
(JSC::ARMAssembler::getImm): Use encodeComplexImm for complex immediate
(JSC::ARMAssembler::moveImm): Ditto.
(JSC::ARMAssembler::encodeComplexImm): Encode a constant by one or two
instructions or a PC relative load.
- assembler/ARMAssembler.h: Use INVALID_IMM if a constant cannot be
encoded as an immediate constant.
(JSC::ARMAssembler::):
(JSC::ARMAssembler::movw_r): 16-bit immediate load
(JSC::ARMAssembler::movt_r): High halfword 16-bit immediate load
(JSC::ARMAssembler::getImm16Op2): Encode immediate constant for
movw_r and mowt_r
- 12:23 AM Changeset in webkit [50552] by
-
- 2 edits in trunk/WebKit/qt
Prospective build fix against Qt 4.5 (build bot)
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-05
- Api/qgraphicswebview.cpp:
(QGraphicsWebViewPrivate::setInputMethodEnabled): Guard the
use of Qt 4.6 specific API with #ifdefs.
- 12:12 AM Changeset in webkit [50551] by
-
- 3 edits in trunk/WebKit/qt
[Qt] Fix enabling of input method support on QGraphicsWebView.
https://bugs.webkit.org/show_bug.cgi?id=30605
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-01
Reviewed by Kenneth Christiansen.
Instead of setting the unsupported widget attribute on the
QGraphicsWidget in setInputMethodEnabled() set the
ItemAcceptsInputMethod GraphicsItem flag directly.
Changed the existing input method auto test to run once
on a QWebView and once on a QGraphicsWebView.
- Api/qgraphicswebview.cpp:
(QGraphicsWebViewPrivate::setInputMethodEnabled):
- tests/qwebpage/tst_qwebpage.cpp:
(tst_QWebPage::inputMethods_data):
(inputMethodHints):
(inputMethodEnabled):
(tst_QWebPage::inputMethods):
- 12:11 AM Changeset in webkit [50550] by
-
- 4 edits in trunk/WebKit/qt
Added QGraphicsWebView::modified property, for consistency
with QWebView.
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-04
Reviewed by Kenneth Christiansen.
- Api/qgraphicswebview.cpp:
(QGraphicsWebView::isModified):
- Api/qgraphicswebview.h:
- tests/qgraphicswebview/tst_qgraphicswebview.cpp:
(tst_QGraphicsWebView::qgraphicswebview):
- 12:11 AM Changeset in webkit [50549] by
-
- 4 edits in trunk/WebKit/qt
Removed status and progress properties of QGraphicsWebView.
Added loadProgress and statusBarMessage signals instead,
after API review.
Patch by Simon Hausmann <Simon Hausmann> on 2009-11-04
Reviewed by Kenneth Christiansen.
- Api/qgraphicswebview.cpp:
(QGraphicsWebViewPrivate::QGraphicsWebViewPrivate):
(QGraphicsWebView::setPage):
- Api/qgraphicswebview.h:
- tests/qgraphicswebview/tst_qgraphicswebview.cpp:
(tst_QGraphicsWebView::qgraphicswebview):
Nov 4, 2009:
- 11:53 PM Changeset in webkit [50548] by
-
- 2 edits in trunk/WebCore
Allow custom memory allocation control for WebCore's CachedResource
https://bugs.webkit.org/show_bug.cgi?id=31114
Reviewed by Eric Seidel.
Inherits CachedResource class from Noncopyable because its (its child
class) CachedCSSStyleSheet instantiated by 'new' in WebCore/loader/Cache.cpp:75
and it is no need to be copyable.
- loader/CachedResource.h:
- 11:07 PM Changeset in webkit [50547] by
-
- 3 edits in trunk/WebKitTools
2009-11-04 Eric Seidel <eric@webkit.org>
Reviewed by David Kilzer.
svn-apply's fixChangeLogPatch function seems broken
https://bugs.webkit.org/show_bug.cgi?id=30683
Update fixChangeLogPatch to be able to handle patches which
don't start at line 1.
Add unit tests for svn-apply to scm_unittest.py.
- Scripts/VCSUtils.pm:
- Scripts/modules/scm_unittest.py:
- 10:52 PM Changeset in webkit [50546] by
-
- 4 edits in trunk
2009-11-04 Mark Mentovai <mark@chromium.org>
Reviewed by Mark Rowe.
Provide TARGETING_TIGER and TARGETING_LEOPARD as analogues to
BUILDING_ON_TIGER and BUILDING_ON_LEOPARD. The TARGETING_ macros
consider the deployment target; the BUILDING_ON_ macros consider the
headers being built against.
- wtf/Platform.h:
2009-11-04 Mark Mentovai <mark@chromium.org>
Reviewed by Mark Rowe.
Separate the difference between HAVE(CGINTERPOLATION_MEDIUM), which
is true when building on 10.6 or later, and USE(CGINTERPOLATION_MEDIUM)
which is true when targeting 10.6 or later.
HAVE(CGINTERPOLATION_MEDIUM) indicates that kCGInterpolationMedium
is present in the CGInterpolationQuality enum, and must be handled
by a switch that has cases for each enumerated value.
USE(CGINTERPOLATION_MEDIUM) indicates that the product will only run
on 10.6 or later, and that CoreGraphics will understand when
InterpolationMedium is mapped to kCGInterpolationMedium at runtime.
- platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::GraphicsContext::setImageInterpolationQuality): (WebCore::GraphicsContext::imageInterpolationQuality):
- 10:39 PM Changeset in webkit [50545] by
-
- 2 edits in trunk/WebCore
2009-11-04 Dan Kegel <dank@chromium.org>
Reviewed by Alexey Proskuryakov.
Fix buffer overrun in WebCore::Page::userStyleSheetLocationChanged()
https://bugs.webkit.org/show_bug.cgi?id=31138
Test: LayoutTests/platform/mac/fast/loader/user-stylesheet-fast-path.html in Valgrind
- page/Page.cpp: (WebCore::Page::userStyleSheetLocationChanged):
- 10:27 PM Changeset in webkit [50544] by
-
- 3 edits in trunk/WebCore
Update the Web Inspector Timeline panel to better match the mock-up.
https://bugs.webkit.org/show_bug.cgi?id=31150
Reviewed by Pavel Feldman.
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel.prototype._getRecordDetails):
(WebInspector.TimelinePanel.prototype._dragWindow):
(WebInspector.TimelinePanel.prototype._resizeWindowLeft):
(WebInspector.TimelinePanel.prototype._resizeWindowRight):
(WebInspector.TimelineCategoryTreeElement.prototype.onattach):
(WebInspector.TimelineRecordTreeElement.prototype.onattach):
(WebInspector.TimelineRecordTreeElement.prototype.refresh):
(WebInspector.TimelineCategoryGraph):
- inspector/front-end/inspector.css:
- 9:33 PM Changeset in webkit [50543] by
-
- 1 edit in trunk/WebCore/ChangeLog
Fixed typos.
- 9:25 PM Changeset in webkit [50542] by
-
- 6 edits6 adds in trunk
WebCore:
2009-11-02 Justin Garcia <justin.garcia@apple.com>
Reviewed by Adele Peterson.
https://bugs.webkit.org/show_bug.cgi?id=25439
Deleting when in front of a block image removes character from previous paragraph
Added editing/deleting/25439-{1,2,3}.html
- editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::mergeParagraphs): Block images, tables and HRs cannot be made inline with other content. Instead of merging, just move the caret to just before the selection we deleted.
- editing/htmlediting.cpp: (WebCore::firstInSpecialElement): Added a FIXME. This function begins iterating up from pos.node(), which doesn't necessarily contain pos (suppose pos was [img, 0]). (WebCore::lastInSpecialElement): Ditto. (WebCore::isRenderedAsNonInlineTableImageOrHR): Moved from visible_units.cpp. Added a check for non-inline images.
- editing/htmlediting.h:
- editing/visible_units.cpp: (WebCore::startOfParagraph): Use moved/renamed function. Removed FIXME. The problem causing 5027702 is now squarely in first/lastInSpecialElement (WebCore::endOfParagraph): Ditto.
LayoutTests:
2009-11-03 Justin Garcia <justin.garcia@apple.com>
Reviewed by Adele Peterson.
https://bugs.webkit.org/show_bug.cgi?id=25439
Deleting when in front of a block image removes character from previous paragraph
- editing/deleting/delete-before-block-image-1-expected.txt: Added.
- editing/deleting/delete-before-block-image-1.html: Added.
- editing/deleting/delete-before-block-image-2-expected.txt: Added.
- editing/deleting/delete-before-block-image-2.html: Added.
- editing/deleting/delete-before-block-table-expected.txt: Added.
- editing/deleting/delete-before-block-table.html: Added.
- 9:02 PM Changeset in webkit [50541] by
-
- 2 edits in trunk/JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=31151
Fix branchDouble behaviour on ARM THUMB2 JIT.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-04
Reviewed by Oliver Hunt.
The ARMv7 JIT is currently using ARMv7Assembler::ConditionEQ to branch
for DoubleEqualOrUnordered, however this is incorrect – ConditionEQ won't
branch on unordered operands. Similarly, DoubleLessThanOrUnordered &
DoubleLessThanOrEqualOrUnordered use ARMv7Assembler::ConditionLO &
ARMv7Assembler::ConditionLS, whereas they should be using
ARMv7Assembler::ConditionLT & ARMv7Assembler::ConditionLE.
Fix these, and fill out the missing DoubleConditions.
- assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::):
(JSC::MacroAssemblerARMv7::branchDouble):
- 7:29 PM Changeset in webkit [50540] by
-
- 3 edits in trunk/JavaScriptCore
Rubber Stamped by Oliver Hunt.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-04
Enable native call optimizations on ARMv7. (Existing ARM_TRADITIONAL
implementation was generic, worked perfectly, just needed turning on).
- jit/JITOpcodes.cpp:
- wtf/Platform.h:
- 7:04 PM Changeset in webkit [50539] by
-
- 2 edits in trunk/JavaScriptCore
Rubber Stamped by Mark Rowe, Oliver Hunt, and Sam Weinig.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-04
Add a missing assert to the ARMv7 JIT.
- assembler/ARMv7Assembler.h:
(JSC::ARMThumbImmediate::ARMThumbImmediate):
- 6:22 PM Changeset in webkit [50538] by
-
- 2 edits in trunk/JavaScriptCore
Remove bogus op_ prefix on dumped version of three opcodes.
Rubber-stamped by Oliver Hunt.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::dump):
- 6:22 PM Changeset in webkit [50537] by
-
- 3 edits in trunk/JavaScriptCore
Fix dumping of constants in bytecode so that they aren't printed as large positive register numbers.
Reviewed by Sam Weinig.
We do this by having the registerName function return information about the constant if the register
number corresponds to a constant. This requires that registerName, and several functions that call it,
be converted to member functions of CodeBlock so that the constant value can be retrieved. The
ExecState also needs to be threaded down through these functions so that it can be passed on to
constantName when needed.
- bytecode/CodeBlock.cpp:
(JSC::constantName):
(JSC::CodeBlock::registerName):
(JSC::CodeBlock::printUnaryOp):
(JSC::CodeBlock::printBinaryOp):
(JSC::CodeBlock::printConditionalJump):
(JSC::CodeBlock::printGetByIdOp):
(JSC::CodeBlock::printPutByIdOp):
(JSC::CodeBlock::dump):
- bytecode/CodeBlock.h:
(JSC::CodeBlock::isConstantRegisterIndex):
- 5:42 PM Changeset in webkit [50536] by
-
- 4 edits2 adds in trunk
WebCore: Hang in Mail on attempting to change indent level.
<rdar://problem/7131805>
https://bugs.webkit.org/show_bug.cgi?id=31127
Reviewed by Adele Peterson.
The hang was caused by an infinite loop inside outdentRegion.
The code did not account for the fact that, when a list item
cointains multiple paragraphs, outdent moves all paragraphs at
once, invalidating some of the position we keep track of in the loop.
Some code refactoring has also been done to minimize duplicate code.
Test: editing/execCommand/outdent-multiparagraph-list.html
- editing/IndentOutdentCommand.cpp:
(WebCore::IndentOutdentCommand::indentRegion): Moved code in common with
outdentRegion to doApply.
(WebCore::IndentOutdentCommand::outdentRegion): Fixed endless loop.
(WebCore::IndentOutdentCommand::doApply): Some code refactoring.
- editing/IndentOutdentCommand.h: Added VisiblePosition parameters to
indentRegion and outdentRegion.
LayoutTests: Hang in Mail on attempting to change indent level
<rdar://problem/7131805>
https://bugs.webkit.org/show_bug.cgi?id=31127
Reviewed by Adele Peterson.
Added test to cover all the different code paths.
- editing/execCommand/outdent-multiparagraph-list-expected.txt: Added.
- editing/execCommand/outdent-multiparagraph-list.html: Added.
- 4:50 PM Changeset in webkit [50535] by
-
- 2 edits in trunk/WebCore
2009-11-04 Alpha Lam <hclam@chromium.org>
Reviewed by Eric Carlson.
Volume slider doesn't have a thumb
https://bugs.webkit.org/show_bug.cgi?id=31135
Fixed an incorrect if statement that prevents volume slider
being updated.
The statement checks if the slider value equals to the current
volume value of the media control to avoid updating the volume
slider. Updating the volume slider control shouldn't be within
this condition because we explicitly set them to be equals
during creation of the controls and also when mouse events are
received on the volume control.
No new tests because existing code breaks:
LayoutTests/media/video-volume-slider.html
It should now start passing on Chromium.
- rendering/MediaControlElements.cpp: (WebCore::MediaControlVolumeSliderElement::update): Update the volume slider regardless of the current volume value of the media control.
- 4:35 PM Changeset in webkit [50534] by
-
- 6 edits in trunk/WebCore
Need to properly disable applicationCache at runtime
https://bugs.webkit.org/show_bug.cgi?id=30417
Patch by Jenn Braithwaite <jennb@chromium.org> on 2009-11-04
Reviewed by David Levin.
Adding applicationCacheEnabled bit to V8 RuntimeEnabledFeatures.
No new exposed functionality, so no new tests.
- bindings/v8/RuntimeEnabledFeatures.cpp:
- bindings/v8/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setApplicationCacheEnabled):
(WebCore::RuntimeEnabledFeatures::applicationCacheEnabled):
- bindings/v8/custom/V8CustomBinding.h:
- bindings/v8/custom/V8DOMWindowCustom.cpp:
(WebCore::ACCESSOR_RUNTIME_ENABLER):
- page/DOMWindow.idl:
- 4:12 PM Changeset in webkit [50533] by
-
- 2 edits in trunk/JavaScriptCore
2009-11-04 Pavel Heimlich <tropikhajma@gmail.com>
Reviewed by Alexey Proskuryakov.
https://bugs.webkit.org/show_bug.cgi?id=30647
Solaris build failure due to strnstr.
- wtf/StringExtras.h: Enable strnstr on Solaris, too.
- 3:59 PM Changeset in webkit [50532] by
-
- 3 edits in trunk/LayoutTests
2009-11-04 Daniel Bates <dbates@webkit.org>
Reviewed by Eric Seidel.
Changed data format type from "text" to "text/plain" to pass Chrome tests.
- fast/events/drag-and-drop-dataTransfer-types-nocrash-expected.txt:
- fast/events/drag-and-drop-dataTransfer-types-nocrash.html:
- 3:59 PM Changeset in webkit [50531] by
-
- 6 edits in trunk/JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=31104
Refactor x86-specific behaviour out of the JIT.
Patch by Gavin Barraclough <barraclough@apple.com> on 2009-11-04
Reviewed by Oliver Hunt.
- Add explicit double branch conditions for ordered and unordered comparisons (presently the brehaviour is a mix).
- Refactor double to int conversion out into the MacroAssembler.
- Remove broken double to int conversion for !JSVALUE32_64 builds - this code was broken and slowing us down, fixing it showed it not to be an improvement.
- Remove exclusion of double to int conversion from (1 % X) cases in JSVALUE32_64 builds - if this was of benefit this is no longer the case; simplify.
- assembler/MacroAssemblerARM.h:
(JSC::MacroAssemblerARM::):
- assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::):
- assembler/MacroAssemblerX86Common.h:
(JSC::MacroAssemblerX86Common::):
(JSC::MacroAssemblerX86Common::convertInt32ToDouble):
(JSC::MacroAssemblerX86Common::branchDouble):
(JSC::MacroAssemblerX86Common::branchConvertDoubleToInt32):
- jit/JITArithmetic.cpp:
(JSC::JIT::emitBinaryDoubleOp):
(JSC::JIT::emit_op_div):
(JSC::JIT::emitSlow_op_jnless):
(JSC::JIT::emitSlow_op_jnlesseq):
- jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_jfalse):
- 3:48 PM Changeset in webkit [50530] by
-
- 2 edits in trunk/WebCore
Reviewed by Dan Bernstein.
https://bugs.webkit.org/show_bug.cgi?id=31143
Assertion failure in CredentialStorage::set() when proxy credentials are being set
No test, cannot test proxy behavior.
- platform/network/CredentialStorage.cpp: (WebCore::CredentialStorage::set): Account for the possibility of null url. Release mode changes are likely inconsequential - e.g. we used to add ":" to origin set, which is weird, but safe.
- 2:21 PM Changeset in webkit [50529] by
-
- 2 edits in trunk/WebCore
2009-11-04 Patrick Mueller <Patrick_Mueller@us.ibm.com>
Reviewed by Timothy Hatcher.
Web Inspector: Leftover Breakpoints in the Sidebar Pane
https://bugs.webkit.org/show_bug.cgi?id=30659
No new tests.
- inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.addScript):
- 1:44 PM Changeset in webkit [50528] by
-
- 7 edits in trunk/WebCore
2009-11-04 Jocelyn Turcotte <jocelyn.turcotte@nokia.com>
Reviewed by Timothy Hatcher.
WebInspector: Use a different method to identify the webkit port in
InspectorBackent::platform().
This corrects the inspector expected behavior with Qt on Windows.
https://bugs.webkit.org/show_bug.cgi?id=31116
- inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::platform): (WebCore::InspectorBackend::port):
- inspector/InspectorBackend.h:
- inspector/InspectorBackend.idl:
- inspector/front-end/InspectorControllerStub.js: (.WebInspector.InspectorControllerStub.prototype.port):
- inspector/front-end/inspector.css:
- inspector/front-end/inspector.js: (WebInspector.loaded): (WebInspector.toolbarDragStart):
- 1:00 PM Changeset in webkit [50527] by
-
- 7 edits in trunk
2009-11-04 Benjamin Otte <otte@gnome.org>
Reviewed by Gustavo Noronha.
Update Cairo requirement to 1.6.
- configure.ac:
2009-11-04 Benjamin Otte <otte@gnome.org>
Reviewed by Gustavo Noronha.
Update Cairo requirement to 1.6.
Also remove all conditional code and workarounds for older versions of
Cairo.
In particular, gain image quality by removing the use of
CAIRO_FILTER_NEAREST when rendering images and use the default
bilinear filter instead.
https://bugs.webkit.org/show_bug.cgi?id=19266
- platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::clipOut):
- platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::draw): (WebCore::BitmapImage::drawPattern):
- platform/graphics/cairo/PathCairo.cpp: (WebCore::Path::isEmpty): (WebCore::Path::boundingRect):
- platform/gtk/RenderThemeGtk.cpp: (WebCore::paintMozWidget):
- 12:46 PM Changeset in webkit [50526] by
-
- 2 edits in trunk/WebCore
wx build fix. Restore removed string conversion after cleanup.
- 12:22 PM Changeset in webkit [50525] by
-
- 15 edits2 adds in trunk
2009-11-04 Chris Fleizach <Chris Fleizach>
Reviewed by Beth Dakin.
Need to implement ARIA role="combobox"
https://bugs.webkit.org/show_bug.cgi?id=31096
- accessibility/aria-combobox.html: Added.
- platform/gtk/Skipped:
- platform/mac/accessibility/aria-combobox-expected.txt: Added.
- platform/win/Skipped:
2009-11-04 Chris Fleizach <Chris Fleizach>
Reviewed by Beth Dakin.
Need to implement ARIA role="combobox"
https://bugs.webkit.org/show_bug.cgi?id=31096
Test: accessibility/aria-combobox.html
- accessibility/AccessibilityObject.h: (WebCore::AccessibilityObject::isComboBox): (WebCore::AccessibilityObject::isExpanded): (WebCore::AccessibilityObject::expandObject): (WebCore::AccessibilityObject::increment): (WebCore::AccessibilityObject::decrement):
- accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::expandObject): (WebCore::AccessibilityRenderObject::isExpanded): (WebCore::createARIARoleMap):
- accessibility/AccessibilityRenderObject.h:
- accessibility/mac/AccessibilityObjectWrapper.mm: (-[AccessibilityObjectWrapper accessibilityAttributeNames]): (-[AccessibilityObjectWrapper accessibilityAttributeValue:]): (-[AccessibilityObjectWrapper accessibilityPerformShowMenuAction]):
- html/HTMLAttributeNames.in:
2009-11-04 Chris Fleizach <Chris Fleizach>
Reviewed by Beth Dakin.
Need to implement ARIA role="combobox"
https://bugs.webkit.org/show_bug.cgi?id=31096
- DumpRenderTree/AccessibilityUIElement.cpp: (showMenuCallback): (getIsExpandedCallback): (AccessibilityUIElement::getJSClass):
- DumpRenderTree/AccessibilityUIElement.h:
- DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::isExpanded): (AccessibilityUIElement::showMenu):
- DumpRenderTree/mac/AccessibilityUIElementMac.mm: (AccessibilityUIElement::isExpanded): (AccessibilityUIElement::showMenu):
- DumpRenderTree/win/AccessibilityUIElementWin.cpp: (AccessibilityUIElement::isSelected): (AccessibilityUIElement::isExpanded): (AccessibilityUIElement::showMenu):
- 12:12 PM Changeset in webkit [50524] by
-
- 4 edits in trunk/WebCore
2009-11-04 Kelly Norton <knorton@google.com>
Reviewed by Pavel Feldman.
Fixes naming inconsistencies in TimelineRecordFactory.
https://bugs.webkit.org/show_bug.cgi?id=31132
- inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::willPaint): (WebCore::InspectorTimelineAgent::willChangeXHRReadyState): (WebCore::InspectorTimelineAgent::willLoadXHR): (WebCore::InspectorTimelineAgent::willEvaluateScript): (WebCore::InspectorTimelineAgent::willSendResourceRequest): (WebCore::InspectorTimelineAgent::didReceiveResourceResponse): (WebCore::InspectorTimelineAgent::didFinishLoadingResource):
- inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createXHRReadyStateChangeRecord): (WebCore::TimelineRecordFactory::createXHRLoadRecord): (WebCore::TimelineRecordFactory::createEvaluateScriptRecord): (WebCore::TimelineRecordFactory::createMarkTimelineRecord): (WebCore::TimelineRecordFactory::createResourceSendRequestRecord): (WebCore::TimelineRecordFactory::createResourceReceiveResponseRecord): (WebCore::TimelineRecordFactory::createResourceFinishRecord): (WebCore::TimelineRecordFactory::createPaintRecord):
- inspector/TimelineRecordFactory.h:
- 12:04 PM Changeset in webkit [50523] by
-
- 9 edits in trunk/WebCore
2009-11-04 Eric Z. Ayers <zundel@google.com>
Reviewed by Pavel Feldman.
Followon to bug 31080, which protects Timeline
instrumentation in the case where InspectorTimelineAgent
is enabled or disabled during an event dispatch.
- bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::evaluate):
- dom/Document.cpp: (WebCore::Document::recalcStyle):
- dom/Node.cpp: (WebCore::Node::dispatchGenericEvent):
- html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::write):
- page/DOMWindow.cpp: (WebCore::DOMWindow::inspectorTimelineAgent): (WebCore::DOMWindow::dispatchEvent):
- page/DOMWindow.h:
- page/FrameView.cpp: (WebCore::FrameView::layout): (WebCore::FrameView::paintContents):
- xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::callReadyStateChangeListener):
- 11:55 AM Changeset in webkit [50522] by
-
- 2 edits in trunk/WebKit/wx
Reviewed by Kevin Ollivier.
Fix an assert on shutdown when wxWebView has captured the mouse,
and make sure wxWebView's right click handling is only active when
the WebCore popup menu is used.
https://bugs.webkit.org/show_bug.cgi?id=31131
- 11:35 AM Changeset in webkit [50521] by
-
- 5 edits in trunk/WebCore
Revert 50519 while I work out what went wrong.
Patch by Jeremy Orlow <jorlow@chromium.org> on 2009-11-04
- storage/LocalStorageThread.cpp:
(WebCore::LocalStorageThread::create):
(WebCore::LocalStorageThread::LocalStorageThread):
(WebCore::LocalStorageThread::start):
(WebCore::LocalStorageThread::localStorageThreadStart):
(WebCore::LocalStorageThread::localStorageThread):
(WebCore::LocalStorageThread::scheduleImport):
(WebCore::LocalStorageThread::scheduleSync):
(WebCore::LocalStorageThread::terminate):
(WebCore::LocalStorageThread::performTerminate):
- storage/LocalStorageThread.h:
- storage/StorageSyncManager.cpp:
(WebCore::StorageSyncManager::StorageSyncManager):
(WebCore::StorageSyncManager::~StorageSyncManager):
(WebCore::StorageSyncManager::scheduleImport):
(WebCore::StorageSyncManager::scheduleSync):
- storage/StorageSyncManager.h:
- 11:32 AM Changeset in webkit [50520] by
-
- 4 edits in trunk/WebCore
2009-11-04 Pavel Feldman <pfeldman@chromium.org>
Reviewed by Timothy Hatcher.
Web Inspector: Add basic support for resource events and marks.
Couple of drive-by fixes. Enabling the panel!
- English.lproj/localizedStrings.js:
- inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype._formatRecord): (WebInspector.TimelinePanel.prototype._getRecordDetails): (WebInspector.TimelinePanel.prototype.reset): (WebInspector.TimelineCategoryTreeElement.prototype._onCheckboxClicked): (WebInspector.TimelineRecordTreeElement.prototype.onattach):
- inspector/front-end/inspector.js: (WebInspector._createPanels):
- 11:11 AM Changeset in webkit [50519] by
-
- 5 edits in trunk/WebCore
2009-11-03 Jeremy Orlow <jorlow@chromium.org>
Reviewed by Darin Fisher.
Simplify LocalStorageThread
https://bugs.webkit.org/show_bug.cgi?id=30935
On LocalStoragethread: Remove reference counting. Get rid of locking. Make some
of the method names a bit more clear. Assert proper thread usage. Join rather
than detaching the thread and doing an ad-hoc form of join. Avoid touching
variables on the background thread when simple to do so. Also create a generic
scheduleTask function rather than one for each task.
No behavior should have changed.
- storage/LocalStorageThread.cpp: (WebCore::LocalStorageThread::create): (WebCore::LocalStorageThread::LocalStorageThread): (WebCore::LocalStorageThread::~LocalStorageThread): (WebCore::LocalStorageThread::start): (WebCore::LocalStorageThread::threadEntryPointCallback): (WebCore::LocalStorageThread::threadEntryPoint): (WebCore::LocalStorageThread::scheduleTask): (WebCore::LocalStorageThread::terminate): (WebCore::LocalStorageThread::performTerminate):
- storage/LocalStorageThread.h:
- storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::StorageSyncManager): (WebCore::StorageSyncManager::~StorageSyncManager): (WebCore::StorageSyncManager::scheduleImport): (WebCore::StorageSyncManager::scheduleSync):
- storage/StorageSyncManager.h:
- 10:28 AM Changeset in webkit [50518] by
-
- 2 edits in trunk/WebKitTools
2009-11-04 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
commit-queue is crashing when trying to reject patches after unknown failures
https://bugs.webkit.org/show_bug.cgi?id=31091
- Scripts/bugzilla-tool:
- patchid? was a copy/paste mistake. This code has no 'patch' variable so we have to find out what the current patch is by asking bugzilla again.
- Discovered that this code was also leaking file descriptors, so fixed that.
- 10:21 AM Changeset in webkit [50517] by
-
- 2 edits in trunk/JavaScriptCore
2009-11-04 Mark Mentovai <mark@chromium.org>
Reviewed by Eric Seidel.
Remove BUILDING_ON_LEOPARD from JavaScriptCore.gyp. This is supposed
to be set as needed only in wtf/Platform.h.
- JavaScriptCore.gyp/JavaScriptCore.gyp:
- 10:13 AM Changeset in webkit [50516] by
-
- 4 edits in trunk
2009-11-04 Vadim Zeitlin <vadim@wxwidgets.org>
Reviewed by Eric Seidel.
[wx] Small cleanup: avoid unnecessary wxString::mb_str() calls.
- platform/graphics/wx/FontPlatformDataWx.cpp: (WebCore::FontPlatformData::computeHash):
2009-11-04 Vadim Zeitlin <vadim@wxwidgets.org>
Reviewed by Eric Seidel.
[wx] Small cleanup: avoid unnecessary wxString::mb_str() calls.
- WebFrame.cpp: (wxWebFrame::SetPageSource): (wxWebFrame::LoadURL):
- 9:17 AM Changeset in webkit [50515] by
-
- 7 edits in trunk/WebCore
2009-11-04 Kelly Norton <knorton@google.com>
Reviewed by Pavel Feldman.
Adds lightweight network resources to InspectorTimelineAgent.
https://bugs.webkit.org/show_bug.cgi?id=31065
- inspector/InspectorController.cpp: (WebCore::InspectorController::willSendRequest): (WebCore::InspectorController::didReceiveResponse): (WebCore::InspectorController::didFinishLoading): (WebCore::InspectorController::didFailLoading):
- inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::willSendResourceRequest): (WebCore::InspectorTimelineAgent::didReceiveResourceResponse): (WebCore::InspectorTimelineAgent::didFinishLoadingResource):
- inspector/InspectorTimelineAgent.h: (WebCore::):
- inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createResourceSendRequestTimelineRecord): (WebCore::TimelineRecordFactory::createResourceReceiveResponseTimelineRecord): (WebCore::TimelineRecordFactory::createResourceFinishTimelineRecord):
- inspector/TimelineRecordFactory.h:
- inspector/front-end/TimelineAgent.js:
- 9:08 AM Changeset in webkit [50514] by
-
- 2 edits in trunk/WebCore
2009-11-04 Yael Aharon <yael.aharon@nokia.com>
Reviewed by Tor Arne Vestbø.
[Qt] ASSERT failure when receiving 401 HTTP Authentication response.
https://bugs.webkit.org/show_bug.cgi?id=31077
Allow sending the response body under the same conditions that we
allow it to finish without reporting an error.
- platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::sendResponseIfNeeded):
- 8:58 AM Changeset in webkit [50513] by
-
- 2 edits in trunk/WebKitTools
Make run-webkit-tests work for the Debug_Internal Windows configuration
In Debug_Internal, DumpRenderTree.exe and ImageDiff.exe have no _debug
suffix.
Fixes <http://webkit.org/b/31123>.
Reviewed by Sam Weinig.
- Scripts/run-webkit-tests: Don't add the _debug suffix in
Debug_Internal, either.
- 8:13 AM Changeset in webkit [50512] by
-
- 2 edits in trunk/WebCore
Sort WebCore.base.exp
Rubber-stamped by Dan Bernstein.
- WebCore.base.exp: Sorted.
- 4:12 AM Changeset in webkit [50511] by
-
- 2 edits in trunk/LayoutTests
Put test into skiplist added in r50489 because
layoutTestController.showWebInspector() is unimplemented now.
Patch by Csaba Osztrogonác <Csaba Osztrogonác> on 2009-11-04
- platform/qt/Skipped: inspector/timeline-trivial.html added.
- 3:27 AM Changeset in webkit [50510] by
-
- 6 edits1 add3 deletes in trunk
Unreviewed rollout.
Patch by Csaba Osztrogonác <Csaba Osztrogonác> on 2009-11-04
Revert r50496 because it broke all layout tests on QtBuildBot.
WebCore:
- WebCore.pro:
- platform/graphics/qt/FontCacheQt.cpp:
(WebCore::fontCache):
(WebCore::FontCache::FontCache):
(WebCore::FontCache::getTraitsInFamily):
(WebCore::FontPlatformDataCacheKey::FontPlatformDataCacheKey):
(WebCore::FontPlatformDataCacheKey::isHashTableDeletedValue):
(WebCore::FontPlatformDataCacheKey::):
(WebCore::FontPlatformDataCacheKey::operator==):
(WebCore::FontPlatformDataCacheKey::hash):
(WebCore::FontPlatformDataCacheKey::computeHash):
(WebCore::FontPlatformDataCacheKey::hashTableDeletedSize):
(WebCore::FontPlatformDataCacheKeyHash::hash):
(WebCore::FontPlatformDataCacheKeyHash::equal):
(WebCore::FontPlatformDataCacheKeyTraits::emptyValue):
(WebCore::FontPlatformDataCacheKeyTraits::constructDeletedValue):
(WebCore::FontPlatformDataCacheKeyTraits::isDeletedValue):
(WebCore::FontCache::getCachedFontPlatformData):
(WebCore::FontCache::getCachedFontData):
(WebCore::FontCache::getLastResortFallbackFont):
(WebCore::FontCache::releaseFontData):
(WebCore::FontCache::purgeInactiveFontData):
(WebCore::FontCache::addClient):
(WebCore::FontCache::removeClient):
(WebCore::FontCache::invalidate):
- platform/graphics/qt/FontFallbackListQt.cpp: Added.
(WebCore::FontFallbackList::FontFallbackList):
(WebCore::FontFallbackList::invalidate):
(WebCore::FontFallbackList::releaseFontData):
(WebCore::FontFallbackList::determinePitch):
(WebCore::FontFallbackList::fontDataAt):
(WebCore::FontFallbackList::fontDataForCharacters):
(WebCore::FontFallbackList::setPlatformFont):
- platform/graphics/qt/FontPlatformData.h:
(WebCore::FontPlatformData::pixelSize):
- platform/graphics/qt/FontPlatformDataQt.cpp:
(WebCore::FontPlatformData::FontPlatformData):
LayoutTests:
- svg/text/resources: Removed.
- svg/text/resources/text-font-invalid.svg: Removed.
- svg/text/text-font-invalid-expected.txt: Removed.
- svg/text/text-font-invalid.html: Removed.
- 2:35 AM Changeset in webkit [50509] by
-
- 2 edits in trunk/WebCore
2009-11-04 Joanmarie Diggs <joanmarie.diggs@gmail.com>
Reviewed by Jan Alonzo.
https://bugs.webkit.org/show_bug.cgi?id=31044
[Gtk] assertion when webkit_accessible_get_index_in_parent attempts to get parent of the web view
- accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (atkParentOfWebView): (webkit_accessible_get_index_in_parent): (webkit_accessible_get_parent):
- 2:28 AM Changeset in webkit [50508] by
-
- 5 edits2 adds in trunk/WebCore
2009-11-04 Dominik Röttsches <dominik.roettsches@access-company.com>
Reviewed by Eric Seidel.
https://bugs.webkit.org/show_bug.cgi?id=15914
[GTK] Implement Unicode functionality using GLib
Initial version of this patch by Jürg Billeter and Naiem Shaik.
Patch 2/4 - Moving TextCodecs to GLib
Added probing for a hard-coded lists of text encodings.
The basis of this list is taken from the encodings supported by iconv,
then extended by e.g. tis-620, windows-1251, euc-kr, windows-1253 and
a number of Chinese ones.
Probing is necessary with the current design of text codecs
as iconv/GLib do not support enumerating available encodings.
- GNUmakefile.am:
- platform/ThreadGlobalData.cpp: (WebCore::ThreadGlobalData::ThreadGlobalData): (WebCore::ThreadGlobalData::~ThreadGlobalData):
- platform/text/TextEncoding.cpp: (WebCore::TextEncoding::encode):
- platform/text/TextEncodingRegistry.cpp: (WebCore::buildBaseTextCodecMaps): (WebCore::extendTextCodecMaps):
- platform/text/gtk/TextCodecGtk.cpp: Added. (WebCore::): (WebCore::newTextCodecGtk): (WebCore::TextCodecGtk::isEncodingAvailable): (WebCore::TextCodecGtk::registerEncodingNames): (WebCore::TextCodecGtk::registerCodecs): (WebCore::TextCodecGtk::registerBaseEncodingNames): (WebCore::TextCodecGtk::registerBaseCodecs): (WebCore::TextCodecGtk::registerExtendedEncodingNames): (WebCore::TextCodecGtk::registerExtendedCodecs): (WebCore::TextCodecGtk::TextCodecGtk): (WebCore::TextCodecGtk::~TextCodecGtk): (WebCore::TextCodecGtk::releaseIConv): (WebCore::TextCodecGtk::createIConvDecoder): (WebCore::TextCodecGtk::createIConvEncoder): (WebCore::TextCodecGtk::decode): (WebCore::TextCodecGtk::encode):
- platform/text/gtk/TextCodecGtk.h: Added.
- 2:19 AM Changeset in webkit [50507] by
-
- 9 edits in trunk
2009-11-04 Martin Robinson <martin.james.robinson@gmail.com>
Reviewed by Jan Alonzo.
[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623
Unify redudant methods which will always return the same value.
- platform/gtk/PasteboardGtk.cpp: (WebCore::Pasteboard::writeSelection):
- platform/gtk/PasteboardHelper.h:
2009-11-04 Martin Robinson <martin.james.robinson@gmail.com>
Reviewed by Jan Alonzo.
[GTK] Enable DOM clipboard and drag-and-drop access
https://bugs.webkit.org/show_bug.cgi?id=30623
Move handling of target list to PasteboardHelperGtk.
- WebCoreSupport/PasteboardHelperGtk.cpp: (WebKit::PasteboardHelperGtk::PasteboardHelperGtk): (WebKit::PasteboardHelperGtk::~PasteboardHelperGtk): (WebKit::PasteboardHelperGtk::fullTargetList):
- WebCoreSupport/PasteboardHelperGtk.h:
- webkit/webkitprivate.cpp: (WebKit::pasteboardHelperInstance): (webkit_init):
- webkit/webkitprivate.h:
- webkit/webkitwebview.cpp: (webkit_web_view_dispose): (webkit_web_view_init): (webkit_web_view_get_copy_target_list): (webkit_web_view_get_paste_target_list):
- 1:31 AM Changeset in webkit [50506] by
-
- 2 edits in trunk/WebCore
2009-11-04 Joanmarie Diggs <joanmarie.diggs@gmail.com>
Reviewed by Jan Alonzo.
https://bugs.webkit.org/show_bug.cgi?id=31035
[GTK] some accessibility tests hitting assertion in debug builds
Removes the assertions in webkit_accessible_ref_child; adds sanity checks.
Any app or AT can attempt to ref a child at a bogus index.
- accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_ref_child):
- 1:15 AM Changeset in webkit [50505] by
-
- 4 edits in trunk/WebKitTools
2009-11-04 Eric Seidel <eric@webkit.org>
Reviewed by Adam Barth.
committers.py needs a way to store non-bugzilla email addresses
https://bugs.webkit.org/show_bug.cgi?id=31037
Make Committer and Reviewer constructors take a single email or a list of emails.
Change committer_by_bugzilla_email functions to committer_by_email to support lookup by any email.
Expose reviewers(), used by validate-committer-lists on bug 30970.
- Scripts/modules/committers.py:
- Scripts/modules/committers_unittest.py: Added tests for the new code.
- 12:58 AM Changeset in webkit [50504] by
-
- 2 edits in trunk/WebCore
2009-11-04 Benjamin Otte <otte@gnome.org>
Reviewed by Jan Alonzo.
[gtk] Use gst_element_class_set_details_simple()
Cosmetic change, just code simplification
- platform/graphics/gtk/VideoSinkGStreamer.cpp:
- 12:50 AM Changeset in webkit [50503] by
-
- 3 edits in trunk/WebKit/qt
2009-11-04 Yael Aharon <yael.aharon@nokia.com>
Reviewed by Simon Hausmann.
[Qt] REGRESSION: Allow applications to use their own QWidget bypassing QWebView.
https://bugs.webkit.org/show_bug.cgi?id=30979
Decouple QWebViewPrivate from QWebPageClient, and automatically create
QWebPageWidgetClient whenever the view is QWidget based.
- Api/qwebpage.cpp: (QWebPageWidgetClient::QWebPageWidgetClient): (QWebPageWidgetClient::scroll): (QWebPageWidgetClient::update): (QWebPageWidgetClient::setInputMethodEnabled): (QWebPageWidgetClient::setInputMethodHint): (QWebPageWidgetClient::cursor): (QWebPageWidgetClient::updateCursor): (QWebPageWidgetClient::palette): (QWebPageWidgetClient::screenNumber): (QWebPageWidgetClient::ownerWidget): (QWebPageWidgetClient::pluginParent): (QWebPage::setView):
- Api/qwebview.cpp: (QWebView::~QWebView): (QWebView::setPage): (QWebView::event):
- 12:42 AM Changeset in webkit [50502] by
-
- 6 edits in trunk/WebCore
2009-11-04 Kelly Norton <knorton@google.com>
Reviewed by Timothy Hatcher.
Adds paint rectangle information to TimelineAgent's didPaint callback.
https://bugs.webkit.org/show_bug.cgi?id=31087
- inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::willPaint):
- inspector/InspectorTimelineAgent.h:
- inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createPaintTimelineRecord):
- inspector/TimelineRecordFactory.h:
- page/FrameView.cpp: (WebCore::FrameView::paintContents):
- 12:33 AM Changeset in webkit [50501] by
-
- 12 edits in trunk
2009-11-04 Jaime Yap <jaimeyap@google.com>
Reviewed by Timothy Hatcher.
This patch adds API to the console object for annotating the
inspector timeline. This allows developers to mark logical
checkpoints in their apps and have them overlaid in the event
record tree.
tests updated: LayoutTests/fast/dom/Window/window-properties.html
https://bugs.webkit.org/show_bug.cgi?id=31082
- inspector/InspectorController.cpp: (WebCore::InspectorController::markTimeline):
- inspector/InspectorController.h:
- inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::didMarkTimeline):
- inspector/InspectorTimelineAgent.h: (WebCore::):
- inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createMarkTimelineRecord):
- inspector/TimelineRecordFactory.h:
- inspector/front-end/TimelineAgent.js:
- page/Console.cpp: (WebCore::Console::markTimeline):
- page/Console.h:
- page/Console.idl:
