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

Timeline



Nov 7, 2009:

7:59 PM Changeset in webkit [50620] by mrowe@apple.com
  • 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 mrowe@apple.com
  • 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 dbates@webkit.org
  • 2 edits
    1 add in trunk/WebCore

2009-11-07 Daniel Bates <​dbates@webkit.org>

Reviewed by Darin Adler.

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


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 eric@webkit.org
  • 1 edit
    4 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 eric@webkit.org
  • 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 eric@webkit.org
  • 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 Beth Dakin
  • 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 bweinstein@apple.com
  • 9 edits
    1 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 ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix: export some symbols.

Patch by Geoffrey Garen <​ggaren@apple.com> on 2009-11-06

3:59 PM Changeset in webkit [50611] by ggaren@apple.com
  • 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 ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix: updated export file.

Patch by Geoffrey Garen <​ggaren@apple.com> on 2009-11-06

3:48 PM Changeset in webkit [50609] by ggaren@apple.com
  • 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 ggaren@apple.com
  • 27 edits
    1 add in trunk

Rolled back in r50590 with Windows build hopefully fixed.

3:00 PM Changeset in webkit [50607] by eric@webkit.org
  • 9 edits
    4 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:-

  1. 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.
  2. 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:-

  1. 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 krit@webkit.org
  • 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 eric@webkit.org
  • 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 krit@webkit.org
  • 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 eric@webkit.org
  • 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 eric@webkit.org
  • 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 eric@webkit.org
  • 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 eric@webkit.org
  • 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 zoltan@webkit.org
  • 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 eric@webkit.org
  • 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 eric@webkit.org
  • 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 xan@webkit.org
(diff)
5:28 AM WebKit Team edited by xan@webkit.org
(diff)
5:27 AM WebKit Team edited by xan@webkit.org
(diff)
5:25 AM WebKit Team edited by xan@webkit.org
(diff)
1:35 AM Changeset in webkit [50596] by hamaji@chromium.org
  • 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 barraclough@apple.com
  • 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 barraclough@apple.com
  • 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:
Note: See TracTimeline for information about the timeline view.