Timeline


and

06/24/09:

23:56 Changeset [45145] by barraclough@apple.com

Merged r44520:44523 into nitro-extreme branch.

23:16 Changeset [45144] by darin@chromium.org

2009-06-24 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

Combined LocalStorageArea and SessionStorageArea into StorageArea since
(after my other refactorings) there are no longer substantial
differences between the two.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorage.cpp: (WebCore::LocalStorage::storageArea):
  • storage/LocalStorage.h:
  • storage/LocalStorageArea.cpp: Removed.
  • storage/LocalStorageArea.h: Removed.
  • storage/SessionStorage.cpp: (WebCore::SessionStorage::copy): (WebCore::SessionStorage::storageArea):
  • storage/SessionStorage.h:
  • storage/SessionStorageArea.cpp: Removed.
  • storage/SessionStorageArea.h: Removed.
  • storage/StorageArea.cpp: (WebCore::StorageArea::createLocalStorage): (WebCore::StorageArea::StorageArea): (WebCore::StorageArea::createSessionStorage): (WebCore::StorageArea::copy): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::scheduleFinalSync): (WebCore::StorageArea::blockUntilImportComplete): (WebCore::StorageArea::dispatchStorageEvent):
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): (WebCore::StorageAreaSync::scheduleFinalSync): (WebCore::StorageAreaSync::syncTimerFired): (WebCore::StorageAreaSync::performImport):
  • storage/StorageAreaSync.h:
  • storage/StorageSyncManager.h:
22:01 Changeset [45143] by mitz@apple.com

WebCore:

Reviewed by Simon Fraser.

  • fix <rdar://problem/7001817> REGRESSION (r41902): Base position track at UCSC Genome Browser doesn't work because image map prevents img from hit-testing

Test: fast/replaced/image-map-2.html

  • rendering/RenderImage.cpp: (WebCore::RenderImage::nodeAtPoint): Do not reset 'inside' to false if the image map failed the hit test.

LayoutTests:

Reviewed by Simon Fraser.

  • test for <rdar://problem/7001817> REGRESSION (r41902): Base position track at UCSC Genome Browser doesn't work because image map prevents img from hit-testing
  • fast/replaced/image-map-2-expected.txt: Added.
  • fast/replaced/image-map-2.html: Added.
20:41 Changeset [45142] by barraclough@apple.com

Merged r44514:44520 into nitro-extreme branch.

20:34 Changeset [45141] by barraclough@apple.com

Merged r44513:44514 into nitro-extreme branch.

20:30 Changeset [45140] by barraclough@apple.com

Merged r44492:44513 into nitro-extreme branch.

20:18 Changeset [45139] by barraclough@apple.com

Merged r44491:44492 into nitro-extreme branch.

20:07 Changeset [45138] by weinig@apple.com

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

Reviewed by Gavin Barraclough.

Make the opcode sampler work once again.

  • jit/JIT.h: (JSC::JIT::compileGetByIdProto): (JSC::JIT::compileGetByIdSelfList): (JSC::JIT::compileGetByIdProtoList): (JSC::JIT::compileGetByIdChainList): (JSC::JIT::compileGetByIdChain): (JSC::JIT::compilePutByIdTransition): (JSC::JIT::compileCTIMachineTrampolines): (JSC::JIT::compilePatchGetArrayLength):
  • jit/JITStubCall.h: (JSC::JITStubCall::call):
20:06 Changeset [45137] by barraclough@apple.com

Merged r44460:44491 into nitro-extreme branch.

20:00 Changeset [45136] by barraclough@apple.com

Merged r44455:44460 into nitro-extreme branch.

19:56 Changeset [45135] by barraclough@apple.com

Merged r44454:44455 into nitro-extreme branch.

19:38 Changeset [45134] by abarth@webkit.org

2009-06-22 Adam Barth <abarth@webkit.org>

Reviewed by Dimitri Glazkov.

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

Refactor V8DOMMap to support isolated worlds.

  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::evaluateInNewWorld):
  • bindings/v8/ScriptController.h:
  • bindings/v8/V8DOMMap.cpp: (WebCore::DOMDataStore::InternalDOMWrapperMap::InternalDOMWrapperMap): (WebCore::DOMDataStore::allStores): (WebCore::DOMDataStore::allStoresMutex): (WebCore::DOMDataStore::domData): (WebCore::ScopedDOMDataStore::ScopedDOMDataStore): (WebCore::ScopedDOMDataStore::~ScopedDOMDataStore): (WebCore::StaticDOMDataStore::StaticDOMDataStore): (WebCore::): (WebCore::MainThreadDOMData::MainThreadDOMData): (WebCore::MainThreadDOMData::getStore): (WebCore::ChildThreadDOMData::ChildThreadDOMData): (WebCore::ChildThreadDOMData::getStore): (WebCore::DOMDataStore::DOMDataStore): (WebCore::DOMDataStore::~DOMDataStore): (WebCore::DOMDataStoreHandle::DOMDataStoreHandle): (WebCore::DOMDataStoreHandle::~DOMDataStoreHandle): (WebCore::::forget): (WebCore::getDOMNodeMap): (WebCore::getDOMObjectMap): (WebCore::getActiveDOMObjectMap): (WebCore::getDOMSVGElementInstanceMap): (WebCore::getDOMSVGObjectWithContextMap): (WebCore::DOMData::getCurrent): (WebCore::DOMData::handleWeakObject): (WebCore::DOMData::ensureDeref): (WebCore::weakDOMObjectCallback): (WebCore::weakActiveDOMObjectCallback): (WebCore::weakNodeCallback): (WebCore::weakSVGElementInstanceCallback): (WebCore::weakSVGObjectWithContextCallback): (WebCore::DOMData::derefObject): (WebCore::DOMData::derefDelayedObjects): (WebCore::DOMData::derefDelayedObjectsInCurrentThread): (WebCore::DOMData::removeObjectsFromWrapperMap): (WebCore::removeAllDOMObjectsInCurrentThreadHelper): (WebCore::visitDOMNodesInCurrentThread): (WebCore::visitDOMObjectsInCurrentThread): (WebCore::visitActiveDOMObjectsInCurrentThread): (WebCore::visitDOMSVGElementInstancesInCurrentThread): (WebCore::visitSVGObjectsInCurrentThread):
  • bindings/v8/V8DOMMap.h: (WebCore::DOMDataStoreHandle::getStore):
  • bindings/v8/V8IsolatedWorld.cpp: Added. (WebCore::getIsolatedWorldKey): (WebCore::contextWeakReferenceCallback): (WebCore::V8IsolatedWorld::evaluate): (WebCore::V8IsolatedWorld::V8IsolatedWorld): (WebCore::V8IsolatedWorld::~V8IsolatedWorld): (WebCore::V8IsolatedWorld::getEntered):
  • bindings/v8/V8IsolatedWorld.h: Added. (WebCore::V8IsolatedWorld::getDOMDataStore):
19:06 Changeset [45133] by mrowe@apple.com

Versioning.

19:04 Changeset [45132] by mrowe@apple.com

New tag.

18:52 Changeset [45131] by eric@webkit.org

2009-06-24 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Bug 26604: Search doesn't work in Web Inspector Profiler
https://bugs.webkit.org/show_bug.cgi?id=26604

Seems like search was damaged in revision 42808.

  • inspector/front-end/ProfileView.js: (WebInspector.ProfileView.prototype.refresh): Here and in other functions: nodes we're searching in are profile data grid nodes, so there is no more need for '_dataGridNode' references. (WebInspector.ProfileView.prototype.searchCanceled): (WebInspector.ProfileView.prototype.performSearch.matchesQuery): Fixed accidental semicolon that caused 'matchesQuery' always return true. (WebInspector.ProfileView.prototype.performSearch): To perform search correctly in the case of bottom up tree, we need to populate the tree, because there's no 1-to-1 correspondence between profile nodes and data grid nodes in this case. (WebInspector.ProfileView.prototype._jumpToSearchResult):
18:32 Changeset [45130] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

Extend FastAllocBase.h with 'using WTF::FastAllocBase' to avoid

unnecessary WTF
usings.
Remove existing unnecessary WTF
usings.
  • interpreter/Interpreter.h:
  • profiler/CallIdentifier.h:
  • runtime/ScopeChain.h:
  • wtf/FastAllocBase.h:
18:07 Changeset [45129] by simon.fraser@apple.com

2009-06-24 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6450239&6574516>

Fix flashing issues caused by compositing layers rendering content before
a deferred layout has happened. Because the -viewWillDraw machinery doesn't
work for composited layers, we need to use scheduleViewUpdate() to queue
up a layout via the run loop observer in WebKit, whenever we know we
are going to be painting soon.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::setContentsNeedDisplay): (WebCore::RenderLayerBacking::setContentsNeedDisplayInRect):
17:45 Changeset [45128] by levin@chromium.org

JavaScriptCore:

2009-06-24 David Levin <levin@chromium.org>

Fix all builds.

  • bytecode/CodeBlock.h:
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Register.h:

WebCore:

2009-06-24 David Levin <levin@chromium.org>

Fix all builds.

  • ForwardingHeaders/wtf/FastAllocBase.h: Added.
17:42 Changeset [45127] by mrowe@apple.com

Versioning.

17:25 Changeset [45126] by barraclough@apple.com

Merged r44437:44454 into nitro-extreme branch.

17:20 Changeset [45125] by barraclough@apple.com

Merged r44436:44437 into nitro-extreme branch.

16:56 Changeset [45124] by darin@chromium.org

2009-06-24 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

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

Split the syncing portions of LocalStorageArea into StorageAreaSync.
This name will make more sense in the next patch (in this set) when
LocalStorageArea and SessionStorageArea are merged to become simply
StorageArea. (Thus the synching portion of StorageArea is in
StorageAreaSync.)

This looks like a big patch, but really all it's doing is splitting
code and patching split-related things up.

  • GNUmakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::create): (WebCore::LocalStorageArea::LocalStorageArea): (WebCore::LocalStorageArea::scheduleFinalSync): (WebCore::LocalStorageArea::itemChanged): (WebCore::LocalStorageArea::itemRemoved): (WebCore::LocalStorageArea::areaCleared): (WebCore::LocalStorageArea::blockUntilImportComplete):
  • storage/LocalStorageArea.h:
  • storage/LocalStorageTask.cpp: (WebCore::LocalStorageTask::LocalStorageTask):
  • storage/LocalStorageTask.h: (WebCore::LocalStorageTask::createImport): (WebCore::LocalStorageTask::createSync):
  • storage/LocalStorageThread.cpp: (WebCore::LocalStorageThread::scheduleImport): (WebCore::LocalStorageThread::scheduleSync):
  • storage/LocalStorageThread.h:
  • storage/SessionStorageArea.cpp: (WebCore::SessionStorageArea::blockUntilImportComplete):
  • storage/SessionStorageArea.h:
  • storage/StorageArea.h:
  • storage/StorageAreaSync.cpp: Copied from WebCore/storage/LocalStorageArea.cpp.
  • storage/StorageAreaSync.h: Copied from WebCore/storage/LocalStorageArea.h.
  • storage/StorageSyncManager.cpp: (WebCore::StorageSyncManager::scheduleImport): (WebCore::StorageSyncManager::scheduleSync):
  • storage/StorageSyncManager.h:
16:55 Changeset [45123] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits CodeBlock class from FastAllocBase because it
has been instantiated by 'new' in JavaScriptCore/bytecode/CodeBlock.h:217.

  • bytecode/CodeBlock.h:
16:55 Changeset [45122] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits BytecodeGenerator class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/parser/Nodes.cpp:1892.

  • bytecompiler/BytecodeGenerator.h:
16:54 Changeset [45121] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits Register class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/JSVariableObject.h:149.

  • interpreter/Register.h:
16:53 Changeset [45120] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits HashMap class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/Structure.cpp:458.

  • wtf/HashMap.h:
16:52 Changeset [45119] by barraclough@apple.com

Merged r44412:44436 into nitro-extreme branch.

16:33 Changeset [45118] by barraclough@apple.com

Merged r44411:44412 into nitro-extreme branch.

16:32 Changeset [45117] by treat@webkit.org

2009-06-24 Adam Treat <adam.treat@torchmobile.com>

Fix Qt build.

  • WebCore.pro:
16:23 Changeset [45116] by mrowe@apple.com

Merge r44877.

16:23 Changeset [45115] by mrowe@apple.com

Merge r44845.

16:23 Changeset [45114] by mrowe@apple.com

Merge r44844.

16:23 Changeset [45113] by mrowe@apple.com

Merge r44262.

16:00 Changeset [45112] by cfleizach@apple.com

Windows build fix for fallout from

Bug 26668: AX: need a way to retrieve the language for an element

15:47 Changeset [45111] by levin@chromium.org

2009-06-24 David Levin <levin@chromium.org>

Reviewed by David Hyatt.

Bug 26696: Member functions in DataGridColumnList should return pointers instead of PassRefPtr.
https://bugs.webkit.org/show_bug.cgi?id=26696

  • html/DataGridColumnList.cpp: (WebCore::DataGridColumnList::itemWithName): (WebCore::DataGridColumnList::add):
  • html/DataGridColumnList.h: (WebCore::DataGridColumnList::item): (WebCore::DataGridColumnList::primaryColumn): (WebCore::DataGridColumnList::sortColumn):
15:19 Changeset [45110] by weinig@apple.com

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

Reviewed by Dave "Messy" Hyatt.

Little bit of style cleanup.

  • html/DataGridColumn.cpp:
  • html/DataGridColumn.h:
  • html/DataGridColumn.idl:
  • html/DataGridColumnList.cpp:
  • html/DataGridColumnList.h:
  • html/DataGridColumnList.idl:
  • html/HTMLDataGridCellElement.cpp:
  • html/HTMLDataGridCellElement.h:
  • html/HTMLDataGridCellElement.idl:
  • html/HTMLDataGridColElement.cpp:
  • html/HTMLDataGridColElement.h:
  • html/HTMLDataGridColElement.idl:
  • html/HTMLDataGridElement.h:
  • html/HTMLDataGridElement.idl:
  • html/HTMLDataGridRowElement.cpp:
  • html/HTMLDataGridRowElement.h:
  • html/HTMLDataGridRowElement.idl:
15:14 Changeset [45109] by cfleizach@apple.com

Bug 26668: AX: need a way to retrieve the language for an element
https://bugs.webkit.org/show_bug.cgi?id=26668

15:12 Changeset [45108] by barraclough@apple.com

Merged r44350:44411 into nitro-extreme branch.

15:04 Changeset [45107] by beidson@apple.com

2009-06-24 Brady Eidson <beidson@apple.com>

Reviewed by Dan Bernstein.


<rdar://problem/6893811> Instead of downloading files linked from Google Earth, file contents displayed in browser window as text.

  • platform/network/mac/WebCoreURLResponse.mm: (createBinaryExtensionsSet): Add '.kmz' to the list of known-to-be-binary extensions.
15:01 Changeset [45106] by oliver@apple.com

<rdar://problem/6940519> REGRESSION (Safari 4 Public Beta - TOT): google.com/adplanner shows blank page instead of site details in "basic research'

Reviewed by Darin Adler.

The problem was caused by the page returned with a function using a
var declaration list containing around ~3000 variables. The solution
to this is to flatten the comma expression representation and make
codegen comma expressions and initializer lists iterative rather than
recursive.

14:53 Changeset [45105] by barraclough@apple.com

Merged r44340:44350 into nitro-extreme branch.

14:12 Changeset [45104] by mitz@apple.com

Rephrased change log

14:10 Changeset [45103] by eric@webkit.org

2009-06-24 Nicolas Weber <thakis@chromium.org>

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=26685
Accomodate for backwards-incompatible skia api changes.

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::clearRect): (WebCore::GraphicsContext::setCompositeOperation):
  • platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::ImageBuffer::ImageBuffer):
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::paintSkBitmap): (WebCore::Image::drawPattern):
  • platform/graphics/skia/PlatformContextSkia.cpp: (PlatformContextSkia::State::State): (PlatformContextSkia::setupPaintCommon): (PlatformContextSkia::setXfermodeMode): (PlatformContextSkia::applyClipFromImage):
  • platform/graphics/skia/PlatformContextSkia.h:
  • platform/graphics/skia/SkiaUtils.cpp: (WebCore::): (WebCore::WebCoreCompositeToSkiaComposite):
  • platform/graphics/skia/SkiaUtils.h:
14:05 Changeset [45102] by mitz@apple.com

Reviewed by Anders Carlsson.

Fix the most recently seen kind of crash seen in
<rdar://problem/5983224>

  • WebView/WebHTMLView.mm: (-[WebHTMLView adjustPageHeightNew:top:bottom:limit:]): Null-check the Frame.
14:01 Changeset [45101] by barraclough@apple.com

Merged r44170:44340 into nitro-extreme branch.

13:51 Changeset [45100] by jmalonzo@webkit.org

2009-06-24 Jan Michael Alonzo <jmalonzo@webkit.org>

Gtk build fix. Add files that were added in r45093 and r45096

  • GNUmakefile.am:
13:36 Changeset [45099] by barraclough@apple.com

Merged r44076:44170 into nitro-extreme branch.

13:33 Changeset [45098] by beidson@apple.com

2009-06-24 Brady Eidson <beidson@apple.com>

Fix 64-bit SnowLeopard build.

  • html/DataGridColumnList.cpp: (WebCore::DataGridColumnList::remove): (WebCore::DataGridColumnList::move):
13:15 Changeset [45097] by barraclough@apple.com

2009-06-24 Gavin Barraclough <barraclough@apple.com>

Reviewed by Geoff Garen.

Enable JIT_OPTIMIZE_METHOD_CALLS on the branch, implementation matches current implemenatation in ToT.

  • jit/JIT.h:
  • jit/JITPropertyAccess.cpp: (JSC::JIT::emit_op_method_check): (JSC::JIT::emitSlow_op_method_check): (JSC::JIT::emit_op_get_by_id): (JSC::JIT::compileGetByIdHotPath): (JSC::JIT::emitSlow_op_get_by_id): (JSC::JIT::compileGetByIdSlowCase):
13:06 Changeset [45096] by hyatt@apple.com

Fix Mac build.

12:44 Changeset [45095] by rwlbuis@webkit.org

Reviewed by Niko.

https://bugs.webkit.org/show_bug.cgi?id=26392
Bug 26392: In html, modification of xlink:href of an newly inserted svg image does not work.
https://bugs.webkit.org/show_bug.cgi?id=26328
Bug 26328: changing href attribute of svg images dows not work when changing display attribute as well

React to href updates even when there is no renderer, i.e. display=none.

Tests: svg/custom/js-update-image-and-display.svg

svg/custom/js-update-image-and-display2.svg
svg/custom/js-update-image-and-display3.svg

12:41 Changeset [45094] by mitz@apple.com

Add Windows test results for a couple of search field tests

12:22 Changeset [45093] by hyatt@apple.com

WebCore:

2009-06-24 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

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

Add basic back-end column support to datagrid.

Added fast/dom/HTMLDataGridElement/ column tests.

  • DerivedSources.cpp:
  • DerivedSources.make:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/DataGridColumn.cpp: Added. (WebCore::DataGridColumn::setPrimary):
  • html/DataGridColumn.h: Added. (WebCore::DataGridColumn::create): (WebCore::DataGridColumn::id): (WebCore::DataGridColumn::setId): (WebCore::DataGridColumn::label): (WebCore::DataGridColumn::setLabel): (WebCore::DataGridColumn::type): (WebCore::DataGridColumn::setType): (WebCore::DataGridColumn::sortable): (WebCore::DataGridColumn::setSortable): (WebCore::DataGridColumn::sortDirection): (WebCore::DataGridColumn::setSortDirection): (WebCore::DataGridColumn::primary): (WebCore::DataGridColumn::detachFromColumnList): (WebCore::DataGridColumn::DataGridColumn):
  • html/DataGridColumn.idl: Added.
  • html/DataGridColumnList.cpp: Added. (WebCore::DataGridColumnList::~DataGridColumnList): (WebCore::DataGridColumnList::itemWithName): (WebCore::DataGridColumnList::add): (WebCore::DataGridColumnList::remove): (WebCore::DataGridColumnList::move): (WebCore::DataGridColumnList::clear): (WebCore::DataGridColumnList::primaryColumnChanged):
  • html/DataGridColumnList.h: Added. (WebCore::DataGridColumnList::create): (WebCore::DataGridColumnList::length): (WebCore::DataGridColumnList::item): (WebCore::DataGridColumnList::primaryColumn): (WebCore::DataGridColumnList::sortColumn):
  • html/DataGridColumnList.idl: Added.
  • html/HTMLDataGridColElement.cpp: (WebCore::HTMLDataGridColElement::sortable): (WebCore::HTMLDataGridColElement::setSortable): (WebCore::HTMLDataGridColElement::sortDirection): (WebCore::HTMLDataGridColElement::setSortDirection):
  • html/HTMLDataGridColElement.h:
  • html/HTMLDataGridColElement.idl:
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::HTMLDataGridElement):
  • html/HTMLDataGridElement.h: (WebCore::HTMLDataGridElement::columns):
  • html/HTMLDataGridElement.idl:
  • rendering/RenderDataGrid.cpp: (WebCore::RenderDataGrid::paintObject): (WebCore::RenderDataGrid::paintColumnHeaders): (WebCore::RenderDataGrid::rebuildColumns):
  • rendering/RenderDataGrid.h: (WebCore::RenderDataGrid::gridElement):

LayoutTests:

2009-06-24 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

Add test for https://bugs.webkit.org/show_bug.cgi?id=26687.

  • fast/dom/HTMLDataGridElement/DataGridColumns-basic-expected.txt: Added.
  • fast/dom/HTMLDataGridElement/DataGridColumns-basic.html: Added.
12:14 Changeset [45092] by mrowe@apple.com

Merge r44951.

12:13 Changeset [45091] by mrowe@apple.com

Merge r44941.

12:13 Changeset [45090] by mrowe@apple.com

Merge r44940.

12:13 Changeset [45089] by mrowe@apple.com

Merge r44908.

12:13 Changeset [45088] by mrowe@apple.com

Merge r44906.

10:29 Changeset [45087] by jberlin@apple.com

2009-06-24 Jessie Berlin <jberlin@apple.com>

Reviewed by Adam Roben.

Partially fixes: https://bugs.webkit.org/show_bug.cgi?id=24735
(<rdar://problem/5015942>)
Where on windows it was not possible to set an element as the drag
image using setDragImage on the dataTransfer object.


Does not "fix" the case of dragging a link where the default link image
is still used, even when the -webkit-user-drag is set to "element". This
is the same behavior as is found on OS X.


Added a manual test because it is not possible to check that what is
contained in the image snapshot is indeed the requested element.

  • dom/Clipboard.h: (WebCore::Clipboard::dragImageElement): Made getting the raw pointer from the RefPtr a const operation.
  • manual-tests/drag-with-div-or-image-as-data-image.html: Added.
  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::createDragImage): Get an image of the rendered element and its subtree.
09:48 Changeset [45086] by darin@chromium.org

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

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=26683
Fix Chromium build bustage: Add custom binding for HTMLDataGridElement.dataSource

This change just adds a stub implementation for now to help fix the build.

  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8HTMLDataGridElementCustom.cpp: Added.
08:53 Changeset [45085] by ddkilzer@apple.com

Build fixes for ENABLE(PLUGIN_PROXY_FOR_VIDEO)

Reviewed by Adam Roben.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::deliverNotification): Removed ExceptionCode parameter from togglePlayState(). (WebCore::HTMLMediaElement::initialURL): Don't convert a KURL object to a String when assigning to a KURL variable.
08:44 Moving to Git edited by treat@kde.org
(diff)
08:00 Changeset [45084] by zimmermann@webkit.org

Reviewed by Adam Treat.

Save memory by not storing attribute values in member variables, if not absolutely needed.
Also fixes bugs where we're substituting variables too early (noticeable with the upcoming <select> element).

07:35 Changeset [45083] by zimmermann@webkit.org

Reviewed by Adam Roben.
Forgot to initialize m_task member variable. Results in crashes sometimes.

07:34 Changeset [45082] by zimmermann@webkit.org

Not reviewed. Forgot to include within last patch.

07:30 Changeset [45081] by zimmermann@webkit.org

Reviewed by Nikolas Zimmermann.

Fixes: https://bugs.webkit.org/show_bug.cgi?id=26474
Adding WML <card> title attribute handling.

07:02 Changeset [45080] by zecke@webkit.org

2009-06-24 Jiahua Huang <jhuangjiahua@gmail.com>

Reviewed by Holger Freyther.

[Gtk] Add Undo/Redo support to WebKitGtk
https://bugs.webkit.org/show_bug.cgi?id=26573

Implement EditorClient::undo and other interested funcs.

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::registerCommandForUndo): (WebKit::EditorClient::registerCommandForRedo): (WebKit::EditorClient::clearUndoRedoOperations): (WebKit::EditorClient::canUndo): (WebKit::EditorClient::canRedo): (WebKit::EditorClient::undo): (WebKit::EditorClient::redo): (WebKit::EditorClient::EditorClient):
  • WebCoreSupport/EditorClientGtk.h:
03:00 Changeset [45079] by hausmann@webkit.org

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

Reviewed by Tor Arne Vestbø.

Fix the build without ENABLE_VIDEO.

  • html/CanvasRenderingContext2D.cpp: Add #if ENABLE guards.
  • html/CanvasRenderingContext2D.h: Ditto.
02:26 Changeset [45078] by oliver@apple.com

Temporarily disable failing test

01:56 Changeset [45077] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

Add new drag tests to the Skipped lists for non-mac platforms.

Skipped tests:
fast/events/drag-to-navigate.html
fast/events/prevent-drag-to-navigate.html

  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
01:15 Changeset [45076] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Maciej Stachowiak.

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

Inherits ScopeChainNode class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/ScopeChain.h:95.

  • wtf/RefPtr.h:
01:12 Changeset [45075] by eric@webkit.org

2009-06-24 Siddhartha Chattopadhyay <sidchat@google.com>

Reviewed by Eric Seidel.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26200 which I introduced earlier. I had
the smartReplace set TRUE initially, which was wrong since the purpose of this call is
to just replace the selection, and place the caret at the end of the selection it replaced.

  • editing/Editor.cpp: (WebCore::Editor::markMisspellingsAfterTypingToPosition):
  • manual-tests/textarea-caret-position-after-auto-spell-correct.html: Added.
01:00 Changeset [45074] by eric@webkit.org

2009-06-24 Jeff Johnson <opendarwin@lapcatsoftware.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26638 WebKitErrors.m: _initWithPluginErrorCode: does not set localizedDescription

Add localized descriptions for plugin errors.
The localized strings already existed but were unused; now NSLocalizedDescriptionKey is added to the NSError userInfo.

  • Misc/WebKitErrors.m: (-[NSError _initWithPluginErrorCode:contentURL:pluginPageURL:pluginName:MIMEType:]):
00:54 Changeset [45073] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits Deque class from FastAllocBase because it has been
instantiated by 'new' with DEFINE_STATIC_LOCAL macro in
JavaScriptCore/wtf/MainThread.cpp:62.

  • wtf/Deque.h:
00:53 Changeset [45072] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

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

Inherits RefPtr class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/StructureChain.cpp:41.

  • wtf/RefPtr.h:
00:52 Changeset [45071] by oliver@apple.com

Attempting to make test more reliable.

00:47 Changeset [45070] by eric@webkit.org

2009-06-24 Kevin Watters <kevinwatters@gmail.com>

Reviewed by Kevin Ollivier.

Don't use an unsigned int for intervalInMS because we sometimes initialize
it with a negative value.

  • platform/wx/SharedTimerWx.cpp: (WebCore::setSharedTimerFireTime):
00:45 Changeset [45069] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Inherits HashSet class from FastAllocBase, because it has been
instantiated by 'new' in JavaScriptCore/runtime/Collector.h:116.

  • wtf/HashSet.h:
00:40 Changeset [45068] by eric@webkit.org

2009-06-24 Zoltan Horvath <hzoltan@inf.u-szeged.hu>

Reviewed by Darin Adler.

Inherits Vector class from FastAllocBase because it has been
instantiated by 'new' in JavaScriptCore/runtime/Structure.cpp:633.

  • wtf/Vector.h:
00:34 Changeset [45067] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

Reviewed by Maciej Stachowiak.

Spell Maciej's name right and add Brady Eidson.

  • Scripts/modules/bugzilla.py:
00:26 Changeset [45066] by eric@webkit.org

2009-06-24 Norbert Leser <norbert.leser@nokia.com>

Reviewed by Maciej Stachoviak.

The BytecodeGenerator objects were instantiated on stack, which takes up ~38kB per instance
(each instance includes copy of JSC::CodeBlock with large SymbolTable, etc.).
Specifically, since there is nested invocation (e.g., GlobalCode --> FunctionCode),
the stack overflows immediately on Symbian hardware (max. 80 kB).
Proposed change allocates generator objects on heap.
Performance impact (if any) should be negligible and change is proposed as general fix,
rather than ifdef'd for SYMBIAN.

  • parser/Nodes.cpp: (JSC::ProgramNode::generateBytecode): (JSC::EvalNode::generateBytecode): (JSC::EvalNode::bytecodeForExceptionInfoReparse): (JSC::FunctionBodyNode::generateBytecode): (JSC::FunctionBodyNode::bytecodeForExceptionInfoReparse):
00:20 Changeset [45065] by eric@webkit.org

2009-06-24 Eric Seidel <eric@webkit.org>

No review, build fix only.

The previous checkin had an improper merge. This fixes the build.

  • page/DragController.cpp: (WebCore::DragController::tryDocumentDrag):
00:07 Changeset [45064] by eric@webkit.org

2009-06-05 Eric Seidel <eric@webkit.org>

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=25922
Fix dropEffect = "none" to work as expected.

  • fast/events/drag-to-navigate-expected.txt: Copied from LayoutTests/editing/selection/doubleclick-whitespace-img-crash-expected.txt.
  • fast/events/drag-to-navigate.html: Added.
  • fast/events/prevent-drag-to-navigate-expected.txt: Copied from LayoutTests/editing/selection/doubleclick-whitespace-img-crash-expected.txt.
  • fast/events/prevent-drag-to-navigate.html: Added.
  • fast/events/resources/file-for-drag-to-navigate.html: Added.
  • fast/events/resources/file-for-prevent-drag-to-navigate.html: Added.

2009-06-05 Eric Seidel <eric@webkit.org>

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=25922
JS setting dropEffect = "none" causes tryDHTMLDrag
to return DragOperationNone. Which is also the value
tryDHTMLDrag previously used to indicate JS did not
want to handle the drag.

Make it possible for the DragController::try* methods
to return a bool to indicate if javascript accepted
or rejected the drag event, separate from the DragOperation.

Tests:

  • fast/events/drag-to-navigate.html
  • fast/events/prevent-drag-to-navigate.html
  • page/DragController.cpp: (WebCore::DragController::dragEnteredOrUpdated): (WebCore::DragController::tryDocumentDrag): (WebCore::defaultOperationForDrag): (WebCore::DragController::tryDHTMLDrag):
  • page/DragController.h:

06/23/09:

23:40 Changeset [45063] by oliver@apple.com

Add expected results for test i landed earlier

23:26 Changeset [45062] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

Fix missing comment message during land-patches and SVN.commit_with_message respecting --dry-run
https://bugs.webkit.org/show_bug.cgi?id=26669

  • Scripts/bugzilla-tool:
  • Scripts/modules/scm.py:
23:25 Changeset [45061] by barraclough@apple.com

Merged r44075:44076 into nitro-extreme branch. (Adds infrastructure for op_method_check, but does not implement optimization.)

23:19 Changeset [45060] by oliver@apple.com

<rdar://problem/6164797> Add Canvas API to allow drawing of <video> frames
<https://bugs.webkit.org/show_bug.cgi?id=25920>

Reviewed by Sam Weinig and Dave Hyatt.

Add support for drawing the contents of the video element to the canvas
in accordance with the current HTML5 draft.

Test: media/video-canvas.html

22:55 Changeset [45059] by ggaren@apple.com

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

Reviewed by Sam Weinig.

Bit off a tiny bit more of standardizing opcode behavior to help with result
caching.


SunSpider reports no change, v8 maybe a tiny speedup.

  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_to_jsnumber): (JSC::JIT::emitSlow_op_to_jsnumber): (JSC::JIT::emit_op_convert_this): (JSC::JIT::emitSlow_op_convert_this):
22:37 Changeset [45058] by barraclough@apple.com

Merged r44030:44075 into nitro-extreme branch.

22:31 Changeset [45057] by ggaren@apple.com

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

Reviewed by Sam Weinig.

Bit off a tiny bit more of standardizing opcode behavior to help with result
caching -- including removing my old enemy, op_resolve_function, because
it was non-standard, and removing it felt better than helping it limp along.


SunSpider reports no change, v8 maybe a tiny speedup.


  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump):
  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp:
  • bytecompiler/BytecodeGenerator.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
  • jit/JIT.h:
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_get_scoped_var): (JSC::JIT::emit_op_put_scoped_var): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emitSlow_op_to_primitive):
  • jit/JITStubs.cpp:
  • jit/JITStubs.h:
  • parser/Nodes.cpp: (JSC::FunctionCallResolveNode::emitBytecode):
22:16 Changeset [45056] by barraclough@apple.com

Merged r44029:44030 into nitro-extreme branch.

21:56 Changeset [45055] by barraclough@apple.com

Merged r44000:44029 into nitro-extreme branch.

21:41 Changeset [45054] by barraclough@apple.com

Merged r43853:44000 into nitro-extreme branch.

21:33 Changeset [45053] by barraclough@apple.com

Merged r43848:43853 into nitro-extreme branch.

21:26 Changeset [45052] by barraclough@apple.com

Merged r43781:43848 into nitro-extreme branch.

21:19 Changeset [45051] by ggaren@apple.com

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

Reviewed by Sam Weinig.


Bit off a tiny bit of standardizing opcode behavior to help with result
caching.


0.6% SunSpider speedup. 0.3% v8 speedup.

  • jit/JITInlineMethods.h: (JSC::JIT::emitLoad): Accomodate a base register that overlaps with payload by loading tag before payload, to avoid stomping base/payload.
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_mov): Abide by the standard "tag in regT1, payload in regT0" semantics.

(JSC::JIT::emit_op_get_global_var):
(JSC::JIT::emit_op_put_global_var): Ditto. Also, removed some irrelevent
loads while I was at it. The global object's "d" pointer never changes
after construction.

21:16 Changeset [45050] by barraclough@apple.com

Merged r43780:43781 into nitro-extreme branch.

20:13 Changeset [45049] by barraclough@apple.com

Merged r43750:43780 into nitro-extreme branch.

20:11 Changeset [45048] by mrowe@apple.com

Merge r44799.

20:11 Changeset [45047] by mrowe@apple.com

Merge r44749.

20:11 Changeset [45046] by mrowe@apple.com

Merge r44519.

20:11 Changeset [45045] by mrowe@apple.com

Merge r44266.

20:11 Changeset [45044] by mrowe@apple.com

Merge r44134.

20:11 Changeset [45043] by mrowe@apple.com

Merge r44133.

20:11 Changeset [45042] by mrowe@apple.com

Merge r44125.

20:11 Changeset [45041] by mrowe@apple.com

Merge r44124.

19:59 Changeset [45040] by barraclough@apple.com

Merged r43666:43750 into nitro-extreme branch.

19:47 Changeset [45039] by oliver@apple.com

<rdar://problem/6992806> REGRESSION: Enumeration can skip new properties in cases of prototypes that have more than 64 (26593)
<https://bugs.webkit.org/show_bug.cgi?id=26593>

Reviewed by Gavin Barraclough.

Add tests to ensure we correctly invalidate caching that depends on structure chains
that include dictionaries.

19:45 WikiStart edited by whzeng@126.com
(diff)
19:41 Changeset [45038] by agl@chromium.org

2009-06-23 Adam Langley <agl@google.com>

Reviewed by Eric Seidel.

Chromium: Fix crash with inherited font-size in <option>

https://bugs.webkit.org/show_bug.cgi?id=26656
http://code.google.com/p/chromium/issues/detail?id=14853

In r42597 (https://bugs.webkit.org/show_bug.cgi?id=25244), I changed
the <select> handing for Chromium to fix a rendering bug. However,
although the font-size is correctly ignored, getRowHeight wasn't
updated and so was calculating the height of the rows in an
inconsistent manner. This can lead to a crash.

  • manual-tests/optgroup-empty-and-nested.html: adding test case for crash
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::getRowHeight):
19:38 WikiStart edited by whzeng@126.com
Add maxthon's port. (diff)
19:37 Changeset [45037] by barraclough@apple.com

Merged r43660:43666 into nitro-extreme branch.

19:18 Changeset [45036] by barraclough@apple.com

Merged r43642:43660 into nitro-extreme branch.

19:13 Changeset [45035] by barraclough@apple.com

Merged r43641:43642 into nitro-extreme branch.

19:08 Changeset [45034] by barraclough@apple.com

Merged r43619:43641 into nitro-extreme branch.

19:02 Changeset [45033] by barraclough@apple.com

Merged r43618:43619 into nitro-extreme branch.

18:50 Changeset [45032] by eric@webkit.org

2009-06-23 Takeshi Yoshino <tyoshino@google.com>

Reviewed by Mark Rowe.

Bug 26537: Builds from command-line fail if custom build product directory is set and ~/Library/Preferences/xcodebuild.plist exists
https://bugs.webkit.org/show_bug.cgi?id=26537

Let determineBaseProductDir subroutine remove
~/Library/Preferences/xcodebuild.plist. It can prevent xcodebuild from
respecting global settings such as a custom build products directory
(<rdar://problem/5585899>).

  • Scripts/webkitdirs.pm:
18:49 Changeset [45031] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Patch by Antti Koivisto.
Reviewed by Brady Eidson.

<rdar://problem/6988010> Failed resources not reloaded when built against Tiger SDK

A latent bug that cache revalidation uncovered. When a CachedResource failed to load,
it should always be revalidated upon reload.

  • loader/CachedResource.cpp: (WebCore::CachedResource::canUseCacheValidator): (WebCore::CachedResource::mustRevalidate):
18:45 Changeset [45030] by barraclough@apple.com

Merged r43598:43618 into nitro-extreme branch.

18:40 Changeset [45029] by barraclough@apple.com

Merged r43583:43598 into nitro-extreme branch.

18:36 Changeset [45028] by barraclough@apple.com

Merged r43582:43583 into nitro-extreme branch.

18:33 Changeset [45027] by ojan@chromium.org

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Mark Rowe.

Resets the results for a test that was missed in
http://trac.webkit.org/changeset/45016.

  • editing/execCommand/5142012-3-expected.txt:
18:31 Changeset [45026] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

Make SCM.run_command smarter, and make all previous
os.system and subprocess.popen use SCM.run_command instead.
https://bugs.webkit.org/show_bug.cgi?id=26666

This makes it easier to handle errors in a standard way throughout all the code.
Since this new code raises by default when the exit_code != 0,
we should prevent future problems of bugzilla-tool continuing after
a git or svn command failed.

  • Scripts/modules/scm.py:
18:29 Changeset [45025] by barraclough@apple.com

Merged r43560:43582 into nitro-extreme branch.

18:14 Changeset [45024] by barraclough@apple.com

Merged r43559:43560 into nitro-extreme branch.

18:00 Changeset [45023] by mrowe@apple.com

Versioning.

17:57 Changeset [45022] by mrowe@apple.com

Versioning.

17:44 Changeset [45021] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6950660> REGRESSION: iSale 5.5.3 crashes after I click a template to load

Due to some subtle WebKit changes - presumably some delegate callback behavior - a latent bug in
iSale was uncovered where they aren't properly retaining their FrameLoadDelegate, and we ended up
calling back to a dealloc'ed object.

  • WebView/WebView.mm: (-[WebView _needsAdobeFrameReloadingQuirk]): Use more intelligent C++-style initialization. (-[WebView _needsKeyboardEventDisambiguationQuirks]): Ditto. (-[WebView _needsFrameLoadDelegateRetainQuirk]): YES for iSale versions under 5.6 (-[WebView dealloc]): Release the delegate if the quirk is in effect. (-[WebView setFrameLoadDelegate:]): Retain the new delegate if the quirk is in effect.
17:41 Changeset [45020] by barraclough@apple.com

Merged r43558:43559 into nitro-extreme branch.

17:40 Changeset [45019] by mitz@apple.com

Reviewed by Darin Adler.

  • fix <rdar://problem/6634228> Crash in [NSScroller mouseDown:] beneath EventHandler::passMouseDownEventToWidget()
  • page/mac/EventHandlerMac.mm: (WebCore::EventHandler::passMouseDownEventToWidget): Surround the -mouseDown: call with calls to Widget::beforeMouseDown() and Widget::afterMouseDown(), which prevent the widget's view hierarchy from being removed from its parent view while the scroller is tracking the mouse.
17:39 Changeset [45018] by weinig@apple.com

WebCore:

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

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26516
Add initial implementation of DataGridDataSource

  • Initial implementation only supports the initialize callback function.

Test: fast/dom/HTMLDataGridElement/DataGridDataSource-basic.html

  • WebCore.xcodeproj/project.pbxproj:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bindings/js/JSDataGridDataSource.cpp: Added. (WebCore::JSDataGridDataSource::JSDataGridDataSource): (WebCore::JSDataGridDataSource::~JSDataGridDataSource): (WebCore::JSDataGridDataSource::initialize):
  • bindings/js/JSDataGridDataSource.h: Added. (WebCore::JSDataGridDataSource::create): (WebCore::JSDataGridDataSource::isJSDataGridDataSource): (WebCore::JSDataGridDataSource::jsDataSource): (WebCore::asJSDataGridDataSource):
  • bindings/js/JSHTMLDataGridElementCustom.cpp: Added. (WebCore::JSHTMLDataGridElement::listener): (WebCore::JSHTMLDataGridElement::setListener):
  • html/DataGridDataSource.h: Added. (WebCore::DataGridDataSource::~DataGridDataSource): (WebCore::DataGridDataSource::isJSDataGridDataSource):
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::HTMLDataGridElement): (WebCore::HTMLDataGridElement::setDataSource): (WebCore::HTMLDataGridElement::initializationTimerFired):
  • html/HTMLDataGridElement.h: (WebCore::HTMLDataGridElement::dataSource):
  • html/HTMLDataGridElement.idl:

LayoutTests:

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

Reviewed by Dave Hyatt.

Test for https://bugs.webkit.org/show_bug.cgi?id=26516
Add initial implementation of DataGridDataSource

  • fast/dom/HTMLDataGridElement: Added.
  • fast/dom/HTMLDataGridElement/DataGridDataSource-basic-expected.txt: Added.
  • fast/dom/HTMLDataGridElement/DataGridDataSource-basic.html: Added.
17:09 Changeset [45017] by barraclough@apple.com

2009-06-23 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam Weinig.

Remove 'arguments' field from Register union (again).
This time do so without breaking tests (radical, I know).

  • interpreter/CallFrame.h: (JSC::ExecState::optionalCalleeArguments): (JSC::ExecState::setArgumentCount): (JSC::ExecState::init):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::dumpRegisters): (JSC::Interpreter::unwindCallFrame): (JSC::Interpreter::privateExecute): (JSC::Interpreter::retrieveArguments):
  • interpreter/Register.h: (JSC::Register::withInt): (JSC::Register::): (JSC::Register::Register): (JSC::Register::i):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_tear_off_arguments):
  • runtime/Arguments.h: (JSC::JSActivation::copyRegisters): (JSC::Register::arguments):
  • runtime/JSActivation.cpp: (JSC::JSActivation::argumentsGetter):
  • runtime/JSActivation.h:
16:56 Changeset [45016] by jparent@chromium.org

WebCore:

2009-06-23 Ryosuke Niwa <rniwa@google.com>

Reviewed by Justin Garcia.

Undo the changeset 21212 to fix the regression bug 14062.
The changeset 21212 attempted to fix rdar://problem/5002441.
The changeset 21212 modified InsertTextCommand::input so as to convert
all spaces in text to non-breaking spaces for rdar://problem/5002441.
However, the bug was originally caused by rebalanceWhitespaceAt
and this workaround introduced a regression bug 14062.
Because rebalanceWhitespaceAt appears to behave correctly now,
the workaround introduced in 21212 is no longer needed.

Test: editing/inserting/space-after-removeformat.html

  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::input):

LayoutTests:

2009-06-23 Ryosuke Niwa <rniwa@google.com>

Reviewed by Justin Garcia.

Undo the changeset 21212 to fix the regression bug 14062.
The changeset 21212 attempted to fix rdar://problem/5002441.
The changeset 21212 modified InsertTextCommand::input so as to convert
all spaces in text to non-breaking spaces for rdar://problem/5002441.
However, the bug was originally caused by rebalanceWhitespaceAt
and this workaround introduced a regression bug 14062.
Because rebalanceWhitespaceAt appears to behave correctly now,
the workaround introduced in 21212 is no longer needed.

The following test cases are affected by this change.
This patch removes unappropriate non-breaking spaces (&nbsp;) and
inserts normal spaces instead.

  • editing/deleting/2610675-1-expected.txt: Replaced.
  • editing/deleting/2610675-2-expected.txt: Replaced.
  • editing/deleting/2610675-3-expected.txt: Replaced.
  • editing/inserting/insert-before-link-1-expected.txt: Replaced.
  • editing/inserting/space-after-removeformat-expected.txt: Added.
  • editing/inserting/space-after-removeformat.html: Added.
  • platform/mac/editing/execCommand/5482023-expected.checksum: Replaced.
  • platform/mac/editing/execCommand/5482023-expected.png: Replaced.
  • platform/mac/editing/execCommand/5482023-expected.txt: Replaced.
  • platform/mac/editing/input/text-input-controller-expected.txt: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.checksum: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.png: Replaced.
  • platform/mac/editing/inserting/editable-html-element-expected.txt: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.checksum: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.png: Replaced.
  • platform/mac/editing/pasteboard/4989774-expected.txt: Replaced.
  • platform/mac/editing/selection/4983858-expected.checksum: Replaced.
  • platform/mac/editing/selection/4983858-expected.png: Replaced.
  • platform/mac/editing/selection/4983858-expected.txt: Replaced.
16:42 Changeset [45015] by bdakin@apple.com

WebCore:

2009-06-23 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26523 <col>
elements width can't be changed with javascript

This patch both fixes setting the width attribute directly in
JavaScript, and setting the width on the style attribute.

  • html/HTMLTableColElement.cpp: (WebCore::HTMLTableColElement::parseMappedAttribute): If we have a new width, call setNeedsLayoutAndPrefWidthsRecalc()

Call calcPrefWidths() on the table cols.

  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::recalcColumn):
  • rendering/FixedTableLayout.cpp: (WebCore::FixedTableLayout::calcWidthArray):


Call layoutIfNeeded on the table cols.

  • rendering/RenderTable.cpp: (WebCore::RenderTable::layout):
  • rendering/RenderTableCol.cpp: (WebCore::RenderTableCol::clippedOverflowRectForRepaint): Use new table() convenience function. (WebCore::RenderTableCol::calcPrefWidths): Just call setPrefWidthsDirty(false) on ourself and our children. (WebCore::RenderTableCol::table):
  • rendering/RenderTableCol.h:

LayoutTests:

2009-06-23 Beth Dakin <bdakin@apple.com>

Reviewed by Dave Hyatt.

Test for https://bugs.webkit.org/show_bug.cgi?id=26523 <col>
elements width can't be changed with javascript

  • fast/dom/HTMLTableColElement: Added.
  • fast/dom/HTMLTableColElement/resize-table-using-col-width.html: Added.
  • platform/mac/fast/dom/HTMLTableColElement: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.png: Added.
  • platform/mac/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.txt: Added.
16:35 Changeset [45014] by ojan@chromium.org

WebCore:

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Dan Bernstein.

Add logic to CSSStyleDeclaration::diff to deal with font-sizes that are
keyword values. When diff is called on a CSSStyleDeclaration, we check
the keywordSize to see if font-size matches a keyword value.

This ensures that when we diff a CSSMutableStyleDeclaration returned from
copyInheritableProperties on a CSSComputedStyleDeclaration that we
correctly identify matching font-sizes.

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

Test: editing/inserting/font-size-clears-from-typing-style.html

  • css/CSSComputedStyleDeclaration.cpp: (WebCore::CSSComputedStyleDeclaration::cssPropertyMatches):
  • css/CSSComputedStyleDeclaration.h:
  • css/CSSStyleDeclaration.cpp: (WebCore::CSSStyleDeclaration::cssPropertyMatches): (WebCore::CSSStyleDeclaration::diff):
  • css/CSSStyleDeclaration.h:

LayoutTests:

2009-06-23 Ojan Vafai <ojan@chromium.org>

Reviewed by Dan Bernstein.

This test hits an edge case where typingStyle would never get cleared.
In addition to making every text insertion go into its own text node,
this caused large performance problems.

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

  • editing/execCommand/5770834-1-expected.txt:

These results match how they were before r43243. It's not clear to me
why changes to font-size are affecting text-align styling though.

  • editing/inserting/font-size-clears-from-typing-style-expected.txt: Added.
  • editing/inserting/font-size-clears-from-typing-style.html: Added.
  • editing/inserting/resources/TEMPLATE.html: Copied from LayoutTests/editing/execCommand/resources/TEMPLATE.html.
  • editing/inserting/resources/font-size-clears-from-typing-style.js: Added.
  • platform/mac/editing/execCommand/remove-formatting-2-expected.txt:
16:18 Changeset [45013] by ggaren@apple.com

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

Reviewed by Sam Weinig.


Removed some result register tracking cruft in preparation for a new
result tracking mechanism.


SunSpider reports no change.

  • assembler/AbstractMacroAssembler.h:
  • assembler/X86Assembler.h: (JSC::X86Assembler::JmpDst::JmpDst): No need to track jump targets in machine code; we already do this in bytecode.
  • jit/JIT.cpp: (JSC::JIT::JIT): (JSC::JIT::emitTimeoutCheck): Make sure to save and restore the result registers, so an opcode with a timeout check can still benefit from result register caching.

(JSC::JIT::privateCompileMainPass):
(JSC::JIT::privateCompileSlowCases): Removed calls to killLastResultRegister()
in preparation for something new.

  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_jnless): (JSC::JIT::emit_op_jnlesseq):
  • jit/JITInlineMethods.h: (JSC::JIT::emitGetFromCallFrameHeaderPtr): (JSC::JIT::emitGetFromCallFrameHeader32):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_jmp): (JSC::JIT::emit_op_jfalse): (JSC::JIT::emit_op_jtrue): (JSC::JIT::emit_op_jeq_null): (JSC::JIT::emit_op_jneq_null): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_jsr): (JSC::JIT::emit_op_sret): (JSC::JIT::emit_op_jmp_scopes): ditto
  • jit/JITStubCall.h: (JSC::JITStubCall::JITStubCall): (JSC::JITStubCall::getArgument): added a mechanism for reloading an argument you passed to a JIT stub, for use in emitTimeoutCheck.
16:10 Changeset [45012] by treat@webkit.org

2009-06-23 Joe Mason <joe.mason@torchmobile.com>

Reviewed by Adam Treat.

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

  • Scripts/prepare-ChangeLog: Added --git-index mode to list only the changes which are already staged in the index. Useful to create an entry for what you're about to commit while ignoring unstaged changes.
16:06 Changeset [45011] by treat@webkit.org

2009-06-23 Yong Li <yong.li@torchmobile.com>

Reviewed by George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=26654
Add the proper export define for the JavaScriptCore API when building for WINCE.

15:38 Changeset [45010] by andersca@apple.com

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

  • Fix <rdar://problem/6965672>


Defer calls to WKPCInvalidateRect, so we don't try to invalidate while waiting for a reply, since
that is too early.


  • Plugins/Hosted/NetscapePluginHostProxy.h: (WebKit::NetscapePluginHostProxy::isProcessingRequests):
  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy): (WebKit::NetscapePluginHostProxy::processRequests): (WKPCInvalidateRect):
15:25 Changeset [45009] by kmccullough@apple.com

WebCore:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6994790> CrashTracer: [USER] 8 crashes in Mail trying to
add a blank line to pasted Facebook content (under
CompositeEditCommand::positionAvoidingSpecialElementBoundary)

  • Just added null checks and I will create a new bug to prevent the bad behavior.
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::positionAvoidingSpecialElementBoundary):

LayoutTests:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6994790> CrashTracer: [USER] 8 crashes in Mail trying to
add a blank line to pasted Facebook content (under
CompositeEditCommand::positionAvoidingSpecialElementBoundary)

  • editing/inserting/return-with-object-element-expected.txt: Added.
  • editing/inserting/return-with-object-element.html: Added.
14:51 Changeset [45008] by weinig@apple.com

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

Reviewed by Geoffrey Garen.

Remove now-useless inplace variants of binary ops.

  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_bitand): (JSC::JIT::emit_op_bitor): (JSC::JIT::emit_op_bitxor): (JSC::JIT::emit_op_add): (JSC::JIT::emit_op_sub): (JSC::JIT::emit_op_mul):
14:48 Changeset [45007] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Build fix.

Wrap shouldDoSoftwarePaint() in #if USE(ACCELERATED_COMPOSITING) to avoid warning when
ACCELERATED_COMPOSITING is not defined.

  • rendering/RenderLayer.cpp:
14:43 Changeset [45006] by weinig@apple.com

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

Reviewed by Geoffrey Garen.

Move off memory operands to aid in re-enabling result caching.

  • No regression measured.
  • jit/JIT.h:
  • jit/JITArithmetic.cpp: (JSC::JIT::emit_op_negate): (JSC::JIT::emit_op_jnless): (JSC::JIT::emit_op_jnlesseq): (JSC::JIT::emit_op_lshift): (JSC::JIT::emit_op_rshift): (JSC::JIT::emit_op_bitand): (JSC::JIT::emitBitAnd32Constant): (JSC::JIT::emitBitAnd32InPlace): (JSC::JIT::emit_op_bitor): (JSC::JIT::emitBitOr32Constant): (JSC::JIT::emitBitOr32InPlace): (JSC::JIT::emit_op_bitxor): (JSC::JIT::emitBitXor32Constant): (JSC::JIT::emitBitXor32InPlace): (JSC::JIT::emit_op_bitnot): (JSC::JIT::emit_op_post_inc): (JSC::JIT::emit_op_post_dec): (JSC::JIT::emit_op_pre_inc): (JSC::JIT::emitSlow_op_pre_inc): (JSC::JIT::emit_op_pre_dec): (JSC::JIT::emitSlow_op_pre_dec): (JSC::JIT::emit_op_add): (JSC::JIT::emitAdd32Constant): (JSC::JIT::emitAdd32InPlace): (JSC::JIT::emitSlow_op_add): (JSC::JIT::emitSlowAdd32Constant): (JSC::JIT::emit_op_sub): (JSC::JIT::emitSlow_op_sub): (JSC::JIT::emitSub32ConstantLeft): (JSC::JIT::emitSub32ConstantRight): (JSC::JIT::emitSub32InPlaceLeft): (JSC::JIT::emitSub32InPlaceRight): (JSC::JIT::emitBinaryDoubleOp): (JSC::JIT::emit_op_mul): (JSC::JIT::emitMul32InPlace): (JSC::JIT::emit_op_div): (JSC::JIT::emit_op_mod):
  • jit/JITCall.cpp: (JSC::JIT::compileOpCallVarargs):
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_loop_if_less): (JSC::JIT::emit_op_loop_if_lesseq): (JSC::JIT::emit_op_instanceof): (JSC::JIT::emit_op_to_primitive): (JSC::JIT::emit_op_not): (JSC::JIT::emit_op_jneq_ptr): (JSC::JIT::emit_op_eq): (JSC::JIT::emit_op_neq): (JSC::JIT::emit_op_to_jsnumber):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::emit_op_get_by_val): (JSC::JIT::emit_op_put_by_val):
14:25 Changeset [45005] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt.

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

Fix reflection painting on elements with compositing layers.

Test: compositing/reflections/reflection-on-composited.html

  • rendering/RenderLayer.h: Add a new PaintLayerPaintingReflection flag which is set while painting inside reflections. (WebCore::RenderLayer::setPaintingInsideReflection): Add a setter so that RenderLayerBacking can call it.
  • rendering/RenderLayer.cpp: (WebCore::shouldDoSoftwarePaint): Utility to clarify the logic of when a composited layer should do a software paint.

(WebCore::RenderLayer::paintLayer):
Use shouldDoSoftwarePaint() to determine when to software-paint a reflected layer.
Remove transform-related tests from the reflection-painting if test because they appear
to be unnecessary.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer): Add the code to paint reflections to the composited layer painting method.
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint): Set the PaintLayerPaintingReflection flag.
13:26 Changeset [45004] by ggaren@apple.com

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

Reviewed by Sam Weinig.


Fixed some missing and/or misplaced labels in bytecode generation, so
we don't have to work around them in JIT code generation.

  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitJumpSubroutine):
  • parser/Nodes.cpp: (JSC::TryNode::emitBytecode):
13:01 Changeset [45003] by antti@apple.com

WebCore:

2009-06-23 Antti Koivisto <antti@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=26643
Memory cache should respect Cache-Control: no-store


Make resources with Cache-Control: no-store uncacheable in the memory cache.
They get evicted immediately when all clients are gone and are always
reloaded from the server.

Conforms to RFC2616 14.9.2 and seems also match Firefox behavior.

  • loader/CachedResource.cpp: (WebCore::CachedResource::removeClient): (WebCore::CachedResource::canUseCacheValidator): (WebCore::CachedResource::mustRevalidate):
  • platform/network/ResourceResponseBase.cpp: (WebCore::ResourceResponseBase::ResourceResponseBase): (WebCore::ResourceResponseBase::parseCacheControlDirectives): (WebCore::ResourceResponseBase::cacheControlContainsNoStore):
  • platform/network/ResourceResponseBase.h:

LayoutTests:

2009-06-23 Antti Koivisto <antti@apple.com>

Reviewed by Darin Adler.


https://bugs.webkit.org/show_bug.cgi?id=26643
Memory cache should respect Cache-Control: no-store


Update the test to cover no-store.

  • http/tests/cache/subresource-expiration-expected.txt:
  • http/tests/cache/subresource-expiration.html:
12:48 Changeset [45002] by beidson@apple.com

Windows build fix

12:46 Changeset [45001] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

First part of https://bugs.webkit.org/show_bug.cgi?id=26652

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paint): (WebCore::RenderLayer::paintLayer):
  • rendering/RenderLayer.h: (WebCore::RenderLayer::):
  • rendering/RenderReplica.cpp: (WebCore::RenderReplica::paint): Replace the 3 boolean params to RenderLayer::paintLayer() with a bit mask for readability.
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer): (WebCore::RenderLayerBacking::paintContents):
  • rendering/RenderLayerBacking.h: Remove the unused 'haveTransparency' param to paintIntoLayer().
12:12 Changeset [45000] by kmccullough@apple.com

WebCore:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6902203> REGRESSION: 'Return' insertion point incorrect
after manually quoting blank line then hitting return

If the insertion point is both the begining and ending visible position
then we need to insert at the end instead of the begining, which is what
would normally happen if the insertion point was as the begining.

  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):

LayoutTests:

2009-06-23 Kevin McCullough <kmccullough@apple.com>

Reviewed by Adele Peterson.

<rdar://problem/6902203> REGRESSION: 'Return' insertion point incorrect
after manually quoting blank line then hitting return

  • editing/inserting/insert-br-quoted-007-expected.txt: Added.
  • editing/inserting/insert-br-quoted-007.html: Added.
12:05 Changeset [44999] by beidson@apple.com

2009-06-23 Brady Eidson <beidson@apple.com>

Reviewed by Sam Weinig, and looked over by Eric Carlson.

<rdar://problem/6980466> Media elements should provide option for changing pitch correction

Introduce a new boolean property to HTMLMediaElement called webkitPreservesPitch for script to control this.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Currently the platforms that have a flexible pitch preservation property (Quicktime on Mac and Win) have had it on by default, so continue that default for now. (WebCore::HTMLMediaElement::loadResource): (WebCore::HTMLMediaElement::webkitPreservesPitch): (WebCore::HTMLMediaElement::setWebkitPreservesPitch):
  • html/HTMLMediaElement.h:
  • html/HTMLMediaElement.idl:
  • platform/graphics/MediaPlayer.cpp: (WebCore::NullMediaPlayerPrivate::setPreservesPitch): (WebCore::MediaPlayer::MediaPlayer): (WebCore::MediaPlayer::preservesPitch): (WebCore::MediaPlayer::setPreservesPitch):
  • platform/graphics/MediaPlayer.h:
  • platform/graphics/MediaPlayerPrivate.h: (WebCore::MediaPlayerPrivateInterface::setPreservesPitch):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::createQTMovie): Rework to allow recreation of the movie with almost the exact same attributes as before, with pitch preservation being the difference. (WebCore::MediaPlayerPrivate::setPreservesPitch): Recreate the QTMovie if the flag actually changes. (WebCore::MediaPlayerPrivate::updateStates): Restore current time and rate if this is a recreated movie. (WebCore::MediaPlayerPrivate::timeChanged): Clear the time-to-be-restored, as this callback means it has been effectively overwritten.
  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.cpp: (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::setPreservesPitch):
  • platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h:
  • platform/graphics/win/QTMovieWin.cpp: (QTMovieWinPrivate::QTMovieWinPrivate): (QTMovieWinPrivate::~QTMovieWinPrivate): (QTMovieWinPrivate::task): Restore current time and rate if this is a re-created movie. (QTMovieWin::play): Clear the time-to-be-restored, as this effectively overwrites it. (QTMovieWin::pause): Ditto. (QTMovieWin::setRate): Ditto. (QTMovieWin::setCurrentTime): Ditto. (QTMovieWin::setPreservesPitch): Recreate the Movie if the flag actually changes. (QTMovieWin::load): Rework to allow recreation of the Movie with similar state as the movie before it, but changing the pitch preservation flag.
  • platform/graphics/win/QTMovieWin.h:
12:04 Changeset [44998] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

<rdar://problem/6998286> Crash after leaving page with <video>


Don't try to post events after going into the page cache.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::documentWillBecomeInactive): Cancel pending events and stop event timers so we won't try to post events after going into the cache.
11:52 Changeset [44997] by darin@chromium.org

2009-06-23 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Brady Eidson.

Simplify the interaction between LocalStorageArea/SessionStorageArea
and StorageArea by creating a "blockUntilImportComplete()" function
rather than bouncing back and forth between the child and parent
classes in a somewhat unintuitive manner.

This patch also paves the way for LocalStorageArea and
SessionStorageArea being merged into one. It's part of several in a
set which are separating syncing (to disk) code from the rest of the
implementation so that local storage and session storage's code can be
unified.

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

  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::blockUntilImportComplete):
  • storage/LocalStorageArea.h:
  • storage/StorageArea.cpp: (WebCore::StorageArea::length): (WebCore::StorageArea::key): (WebCore::StorageArea::getItem): (WebCore::StorageArea::setItem): (WebCore::StorageArea::removeItem): (WebCore::StorageArea::clear): (WebCore::StorageArea::contains):
  • storage/StorageArea.h: (WebCore::StorageArea::blockUntilImportComplete):
11:36 Changeset [44996] by simon.fraser@apple.com

2009-06-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6997929> Assertion at WebCore::RenderLayer::updateClipRects with hardware acceleration

The updatingControlTints() painting pass goes through compositing layer boundaries,
so we need to ensure we don't cache clip rects computed with the wrong root by
setting temporaryClipRects to true.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::contentOffsetInCompostingLayer): Remove a printf that I left in by mistake.
11:03 Changeset [44995] by andersca@apple.com

WebCore:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

WebCore side of <rdar://problem/6946094>.


  • platform/win/ClipboardWin.cpp: (WebCore::ClipboardWin::setExternalDataObject):
  • platform/win/ClipboardWin.h:

WebKit/win:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

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

  • Interfaces/IWebUIDelegate.idl: Add a newPasteboard out parameter to willPerformDragSourceAction.


  • WebCoreSupport/WebDragClient.cpp: (WebDragClient::willPerformDragSourceAction): Implement this. If the client returns a new pasteboard, use it for the drag.


  • WebCoreSupport/WebInspectorDelegate.h: (WebInspectorDelegate::willPerformDragSourceAction): Add the new parameter.

WebKitTools:

2009-06-23 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.


Update for WebKit changes.


  • DumpRenderTree/win/UIDelegate.h: (UIDelegate::willPerformDragSourceAction):
10:29 Changeset [44994] by darin@chromium.org

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

Reviewed by Darin Adler.

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

Add setFormContentType and setFormData since it is awkward for Chromium
to use setFormInfoFromRequest to set the underlying members. Also, add
a clearChildren method that Chromium needs. These methods are used as
part of the Chromium code that serializes a HistoryItem for storing in
the browser's session history.

  • history/HistoryItem.cpp:
  • history/HistoryItem.h:
08:45 Changeset [44993] by treat@webkit.org

2009-06-23 Joe Mason <joe.mason@torchmobile.com>

Reviewed by Adam Treat.

Authors: Yong Li <yong.li@torchmobile.com>, Joe Mason <joe.mason@torchmobile.com>

https://bugs.webkit.org/show_bug.cgi?id=26611
Implement currentThreadStackBase on WINCE by adding a global,
g_stackBase, which must be set to the address of a local variable
by the caller before calling any WebKit function that invokes JSC.

08:44 Changeset [44992] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by NOBODY, layout tests fix.


'timeupdate' events are, by definition, timing dependent so it isn't
a good idea to log them in layout test results.

  • media/event-attributes-expected.txt:
  • media/event-attributes.html:
07:49 Changeset [44991] by eric.carlson@apple.com

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6978938> Need HTML 5 event attributes for media elements

Test: media/event-attributes.html

  • html/HTMLAttributeNames.in: Add media event attribute names.
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::parseMappedAttribute): Handle media event attributes.

2009-06-23 Eric Carlson <eric.carlson@apple.com>

Reviewed by Darin Adler.


<rdar://problem/6978938> Need HTML 5 event attributes for media elements

  • media/event-attributes-expected.txt: Added.
  • media/event-attributes.html: Added.
  • media/progress-event-total.html: Cleanup, use event handler attributes.
  • media/progress-event.html: Ditto.
  • media/video-size-intrinsic-scale.html: Remove duplicate 'waitForEvent("ratechange")'
  • media/video-source-add-src.html: Ditto.
  • media/video-source-error.html: Ditto.
  • media/video-timeupdate-during-playback.html: Ditto.
  • platform/gtk/Skipped: Skip new test.
06:03 Changeset [44990] by hausmann@webkit.org

2009-06-23 Thiago Macieira <thiago.macieira@nokia.com>

Reviewed by Simon Hausmann.

Fix Qt build with Phonon.

The #include <phonon> header no longer exists. And the <Phonon> or
<Phonon/Phonon> headers have never existed (neither for us nor for the
Phonon sources). You have to select each and every header that you do
want now.

05:43 Changeset [44989] by vestbo@webkit.org

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

Reviewed by Simon Hausmann.

Remove duplicated code in Qt & GTK logging initialization

We now share the getChannelFromName() function in Logging.cpp

  • platform/gtk/LoggingGtk.cpp:
  • platform/qt/LoggingQt.cpp:
05:43 Changeset [44988] by vestbo@webkit.org

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

Reviewed by Simon Hausmann.

Rename the plugin logging channel to plural form 'Plugins'

This matches both the other logging channel names, as well as the
WebKit logging channel used by Safari/Mac for plugins.

  • platform/Logging.cpp:
  • platform/Logging.h:
  • platform/gtk/LoggingGtk.cpp:
  • platform/mac/LoggingMac.mm:
  • platform/qt/LoggingQt.cpp:
  • platform/win/LoggingWin.cpp:
  • plugins/PluginDebug.h:
  • plugins/gtk/PluginPackageGtk.cpp:
  • plugins/mac/PluginPackageMac.cpp:
  • plugins/mac/PluginViewMac.cpp:
  • plugins/qt/PluginPackageQt.cpp:
03:17 Changeset [44987] by vestbo@webkit.org

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

Reviewed by Simon Hausmann.

Fix NPAPI mouse translation issues on Mac

The WindowRef we pass to the plugin refers to the the top level window,
so the x and y positions have to be relative to this position, but we
have to manually compensate for title bar decorations and other parents
of the QWebView since contentsToWindow() only traverses to the QWebView.

Previously we did this compensation when passing on mouse coordinates to
the plugin, which caused various issues with translations such as not
being able to close the Flash settings dialog, or the hand cursor not
appearing over links.

We now do the compensation as part of the call to NPP_SetWindow, and
then pass mouse coordinates in global context without any compensation,
similar to how both Safari and Mozilla does it.

03:11 Changeset [44986] by vestbo@webkit.org

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

Rubber-stamped by Simon Hausmann.

[Qt] Resolve absolute path to library install_name manually

Since our target.path is set to the Qt directory (to be able
to do 'make install') the 'absolute_library_soname' option
to QMake did not do the right thing.

We now compute the absolute path manually using DESTDIR.

  • WebCore.pro:
02:17 Changeset [44985] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor. (I converted the server side to Perl
after Sam reviewed the patch because the PHP scripts didn't agree with
the Tiger and Windows buildbots.)

  • http/tests/security/xssAuditor/img-onerror-tricky-expected.txt: Added.
  • http/tests/security/xssAuditor/img-onerror-tricky.html: Added.
  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php: Removed.
  • http/tests/security/xssAuditor/resources/echo-intertag.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-property.pl: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
  • http/tests/security/xssAuditor/script-tag.html:
02:14 Changeset [44984] by oliver@apple.com

Fix stupid performance problem in the LiteralParser

Reviewed by Alexey Proskuryakov.

The LiteralParser was making a new UString in order to use
toDouble, however UString's toDouble allows a much wider range
of numberic strings than the LiteralParser accepts, and requires
an additional heap allocation or two for the construciton of the
UString. To rectify this we just call WTF::dtoa directly using
a stack allocated buffer to hold the validated numeric literal.

02:11 Changeset [44983] by abarth@webkit.org

Revert 44981. The buildbots hate me.

02:01 Changeset [44982] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

  • Scripts/modules/scm.py: Fix commit_with_message to return the commit output.
01:51 Changeset [44981] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor. (I converted the server side to Perl
after Sam reviewed the patch because the PHP scripts didn't agree with
the Tiger and Windows buildbots.)

  • http/tests/security/xssAuditor/img-onerror-tricky-expected.txt: Added.
  • http/tests/security/xssAuditor/img-onerror-tricky.html: Added.
  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php: Removed.
  • http/tests/security/xssAuditor/resources/echo-intertag.pl: Added.
  • http/tests/security/xssAuditor/resources/echo-property.pl: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
  • http/tests/security/xssAuditor/script-tag.html:
01:46 Changeset [44980] by eric@webkit.org

2009-06-23 Eric Seidel <eric@webkit.org>

Reviewed by Mark Rowe.

  • Scripts/modules/scm.py: Fix error seen when commiting r44979.
01:40 Changeset [44979] by eric@webkit.org

2009-06-18 Eric Seidel <eric@webkit.org>

Reviewed by Dave Levin.

WebKit needs a script to interact with bugzilla and automate
parts of the patch posting and commit processes.
https://bugs.webkit.org/show_bug.cgi?id=26283

This is really a first-draft tool.
It's to the point where it's useful to more people than just me now though.
Git support works. SVN support is written, but mostly untested.

This tool requires BeautifulSoup and mechanize python modules to run:
sudo easy_install BeautifulSoup
sudo easy_install mechanize

More important than the tool itself are the Bugzilla, Git and SVN class abstractions
which I hope will allow easy writing of future tools.

The tool currently implements 10 commands, described below.

Helpers for scripting dealing with the commit queue:
bugs-to-commit Bugs in the commit queue
patches-to-commit Patches attached to bugs in the commit queue

Dealing with bugzilla:
reviewed-patches BUGID r+'d patches on a bug
apply-patches BUGID Applies all patches on a bug to the local working directory without committing.
land-and-update BUGID Lands the current working directory diff and updates the bug.
land-patches [options] BUGID Lands all patches on a bug optionally testing them first
obsolete-attachments BUGID Marks all attachments on a bug as obsolete.
commit-message Prints a commit message suitable for the uncommitted changes.

These effectively replace git-send-bugzilla:
post-diff BUGID Attaches the current working directory diff to a bug as a patch file.
post-commits BUGID COMMITISH Attaches a range of local commits to a bug as patch files.

post-diff works for SVN and Git, post-commits only works for SCMs with local-commit support (like Git)

land-* commands in a Git environment only work with simple patches due to svn-apply bugs:
https://bugs.webkit.org/show_bug.cgi?id=26299
https://bugs.webkit.org/show_bug.cgi?id=26300

This script follows python style (similar to how for Obj-C we follow AppKit style)
http://www.python.org/doc/essays/styleguide.html
The Python community has a strong style culture and the WebKit style guide is silent re: Python.

I've filed a bug to update the WebKit style guide to mention python:
https://bugs.webkit.org/show_bug.cgi?id=26524

  • Scripts/bugzilla-tool: Added.
00:48 Changeset [44978] by abarth@webkit.org

Revert 44977 because Tiger and Windows don't like the new tests.

00:11 Changeset [44977] by abarth@webkit.org

2009-06-23 Adam Barth <abarth@webkit.org>

Reviewed by Sam Weinig.

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

More tests for the XSSAuditor.

  • http/tests/security/xssAuditor/link-onclick-expected.txt: Added.
  • http/tests/security/xssAuditor/link-onclick.html: Added.
  • http/tests/security/xssAuditor/property-escape-expected.txt: Added.
  • http/tests/security/xssAuditor/property-escape.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post-and-notify.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-post.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-utf-7.php: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag.php:
  • http/tests/security/xssAuditor/resources/echo-property.php: Added.
  • http/tests/security/xssAuditor/resources/redir.php: Added.
  • http/tests/security/xssAuditor/resources/xss.js: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-convoluted.html: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-open-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-post-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-post.html: Added.
  • http/tests/security/xssAuditor/script-tag-redirect-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-redirect.html: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-utf-7.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-source-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-source.html: Added.
00:00 Changeset [44976] by ggaren@apple.com

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

Reviewed by Sam Weinig.


For member function calls, emit "this" directly into the "this" slot
for the function call, instead of moving it there later. This reduces
time spent in op_mov during certain calls, like "a.b.c()".


1%-2% speedup on v8, mostly richards and delta-blue.

  • parser/Nodes.cpp: (JSC::FunctionCallDotNode::emitBytecode):

06/22/09:

22:53 Changeset [44975] by ap@webkit.org

2009-06-22 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Alexey Proskuryakov.

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

Skip layout tests for DST if the test runs not in PST/PDT.

  • fast/js/date-DST-time-cusps-expected.txt:
  • fast/js/date-big-setdate-expected.txt:
  • fast/js/resources/date-DST-time-cusps.js:
  • fast/js/resources/date-big-setdate.js:
22:44 Changeset [44974] by oliver@apple.com

Bug 26640: JSON.stringify needs to special case Boolean objects
<https://bugs.webkit.org/show_bug.cgi?id=26640>

Reviewed by Alexey Proskuryakov.

Add special case handling of the Boolean object so we match current
ES5 errata.

21:53 Changeset [44973] by simon.fraser@apple.com

2009-06-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Oliver Hunt.

<rdar://problem/6991146> Scrollbars in hardware layers don't change to the
inactive look in background windows

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::paintLayer): Don't short-circuit the "painting" pass that is used to udpate control tints.
21:36 Changeset [44972] by dglazkov@chromium.org

2009-06-22 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=26567
Upstream V8 bindings code generator. With this change, also added interface ancestor
traversal to IDL parser, which is necessary for V8 bindings. The traversal is used
to determine whether an interface is a Node to facilitate proper casting and storage
in V8DOMMap.

  • bindings/scripts/CodeGenerator.pm: Added parent traversal sub, needed

by V8 bindings.

  • bindings/scripts/CodeGeneratorV8.pm: Added.
  • bindings/scripts/IDLParser.pm: Modified to allow limited parsing for

parent traversal cases.

20:37 Changeset [44971] by barraclough@apple.com

2009-06-22 Gavin Barraclough <barraclough@apple.com>

Reviewed by Sam Weinig.

Remove 'arguments' field from Register union. Having JSCell derived types in the union is
dangerous since it opens the possibility for the field to be written as a raw pointer but
then read as a JSValue. This will lead to statle data being read for the tag, which may
be dangerous. Having removed Arguments* types form Register, all arguments objects must
always explicitly be stored in the register file as JSValues.

  • interpreter/CallFrame.h: (JSC::ExecState::optionalCalleeArguments):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::unwindCallFrame): (JSC::Interpreter::privateExecute): (JSC::Interpreter::retrieveArguments):
  • interpreter/Register.h: (JSC::Register::):
  • jit/JITStubs.cpp: (JSC::JITStubs::cti_op_tear_off_arguments):
  • runtime/Arguments.h: (JSC::JSActivation::copyRegisters):
  • runtime/JSActivation.cpp: (JSC::JSActivation::argumentsGetter):
  • runtime/JSActivation.h:
20:07 Changeset [44970] by mrowe@apple.com

Roll out r44967 as that build slave hates freedom.

20:05 Changeset [44969] by weinig@apple.com

WebCore:

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

Reviewed by Gavin Barraclough.

Add event handler attribute getter/setters to HTMLFrameSetElement
as specified in HTML 5. These match the ones on HTMLBodyElement.

  • dom/Node.h:
  • html/HTMLFrameSetElement.cpp: (WebCore::HTMLFrameSetElement::parseMappedAttribute): (WebCore::HTMLFrameSetElement::onblur): (WebCore::HTMLFrameSetElement::setOnblur): (WebCore::HTMLFrameSetElement::onerror): (WebCore::HTMLFrameSetElement::setOnerror): (WebCore::HTMLFrameSetElement::onfocus): (WebCore::HTMLFrameSetElement::setOnfocus): (WebCore::HTMLFrameSetElement::onload): (WebCore::HTMLFrameSetElement::setOnload): (WebCore::HTMLFrameSetElement::onbeforeunload): (WebCore::HTMLFrameSetElement::setOnbeforeunload): (WebCore::HTMLFrameSetElement::onmessage): (WebCore::HTMLFrameSetElement::setOnmessage): (WebCore::HTMLFrameSetElement::onoffline): (WebCore::HTMLFrameSetElement::setOnoffline): (WebCore::HTMLFrameSetElement::ononline): (WebCore::HTMLFrameSetElement::setOnonline): (WebCore::HTMLFrameSetElement::onresize): (WebCore::HTMLFrameSetElement::setOnresize): (WebCore::HTMLFrameSetElement::onstorage): (WebCore::HTMLFrameSetElement::setOnstorage): (WebCore::HTMLFrameSetElement::onunload): (WebCore::HTMLFrameSetElement::setOnunload):
  • html/HTMLFrameSetElement.h:
  • html/HTMLFrameSetElement.idl:

LayoutTests:

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

Reviewed by Gavin Barraclough.

Add event handler attribute getter/setters to HTMLFrameSetElement
as specified in HTML 5. These match the ones on HTMLBodyElement.

  • fast/dom/event-attribute-availability-expected.txt:
  • fast/dom/resources/event-attribute-availability.js:
18:27 Changeset [44968] by oliver@apple.com

Bug 26591: Support revivers in JSON.parse
<https://bugs.webkit.org/show_bug.cgi?id=26591>

Reviewed by Darin Adler.

Add reviver support to JSON.parse. This completes the JSON object.

18:27 Changeset [44967] by mrowe@apple.com

Update build.webkit.org config for new machine.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
18:04 Changeset [44966] by darin@apple.com

2009-06-22 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

Bug 25425: DOM attribute getter/setter functions should use const AtomicString& type
https://bugs.webkit.org/show_bug.cgi?id=25425

  • bindings/scripts/CodeGeneratorJS.pm: Added handling for ReflectURL.
  • bindings/scripts/CodeGeneratorObjC.pm: Ditto.
  • dom/Element.cpp: (WebCore::Element::getURLAttribute): Added. For use implementing getters for ReflectURL.
  • dom/Element.h: Added getURLAttribute.
  • html/HTMLImageElement.idl: Use Reflect and ReflectURL.
17:41 Changeset [44965] by jianli@chromium.org

2009-06-22 Jian Li <jianli@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 26626: Make WorkerContextExecutionProxy::toV8Object handle exception
objects.
https://bugs.webkit.org/show_bug.cgi?id=26626

  • bindings/v8/WorkerContextExecutionProxy.cpp: (WebCore::WorkerContextExecutionProxy::ToV8Object):
16:24 Changeset [44964] by hyatt@apple.com

2009-06-22 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

Add RenderDataGrid to the build. Give it some basic default styling (that more or less matches a listbox). It has a default size
of 300x150.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • css/html4.css:
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::createRenderer):
  • html/HTMLDataGridElement.h:
  • html/HTMLDataGridElement.idl:
  • rendering/RenderDataGrid.cpp: Added. (WebCore::RenderDataGrid::RenderDataGrid): (WebCore::RenderDataGrid::~RenderDataGrid): (WebCore::RenderDataGrid::calcPrefWidths): (WebCore::RenderDataGrid::paintObject): (WebCore::RenderDataGrid::valueChanged): (WebCore::RenderDataGrid::invalidateScrollbarRect): (WebCore::RenderDataGrid::isActive):
  • rendering/RenderDataGrid.h: Added. (WebCore::RenderDataGrid::renderName): (WebCore::RenderDataGrid::canHaveChildren): (WebCore::RenderDataGrid::scrollbarCornerPresent):
16:01 Changeset [44963] by sfalken@apple.com

Remove errant line of code mistakenly checked in.

  • DumpRenderTree/win/DumpRenderTree.cpp: (main):
16:00 Changeset [44962] by sfalken@apple.com

Pass correct value to setShouldPaintNativeControls.


Rubber stamped by Mark Rowe.

  • DumpRenderTree/win/DumpRenderTree.cpp: (main):
15:45 Changeset [44961] by simon.fraser@apple.com

2009-06-22 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=26430
<rdar://problem/6944442> Elements jump around when they become composited (WWDC checkers demo)
<rdar://problem/6989006> Lots of time spent in RenderLayerCompositor::calculateCompositedBounds()

during window resize when using hardware layers

Fix an ordering dependency which caused the compositing position of a layer
to be affected by some later sibling, which affected the compositing ancestor's
composited bounds.

The fix is to only update the compositing positions of layers in some subtree
when updateAfterLayout() has been called on all the layers in that subtree,
from the ancestor down.

Also cache the composited bounds of a RenderLayerBacking, now that we can
guarantee that those bounds are accurate.

Test: compositing/sibling-positioning.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPositions): (WebCore::RenderLayer::hitTestLayer):
  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::RenderLayerBacking): (WebCore::RenderLayerBacking::updateAfterLayout): (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry): (WebCore::RenderLayerBacking::contentOffsetInCompostingLayer): (WebCore::RenderLayerBacking::paintContents): (WebCore::RenderLayerBacking::compositedBounds): (WebCore::RenderLayerBacking::setCompositedBounds):
  • rendering/RenderLayerBacking.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateBacking): (WebCore::RenderLayerCompositor::updateLayerCompositingState): (WebCore::RenderLayerCompositor::calculateCompositedBounds): (WebCore::RenderLayerCompositor::layerWillBeRemoved): (WebCore::RenderLayerCompositor::computeCompositingRequirements): (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree): (WebCore::RenderLayerCompositor::updateCompositingChildrenGeometry):
  • rendering/RenderLayerCompositor.h:
15:41 Moving to Git edited by joe.mason@torchmobile.com
Add link to evmar's doc (diff)
15:31 Moving to Git edited by joe.mason@torchmobile.com
Discussion of patch review branches (diff)
14:59 Changeset [44960] by barraclough@apple.com

Merged r43552:43558 into nitro-extreme branch.

14:58 Changeset [44959] by darin@chromium.org

2009-06-22 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

First step in https://bugs.webkit.org/show_bug.cgi?id=25376

Add StorageSyncManager.cpp/h

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:

In preparation for combining SessionStorage and LocalStorage into a
single StorageManager, move the syncing bits out of LocalStorage and
into its own class.

The next step is to combine LocalStorageArea and SessionStorageArea
into one. Then SessionStorage and LocalStorage can be combined.

Note that combining them will cut down on code paths that must be
proxied, eliminate virtual dispatch, and allow Chromium to (eventually)
write historical sessionStorage to disk when memory is getting tight.

Also remove a couple bits of cruft including code for quotas which is
unnecessary (since a meta-data db is unnecessary since you can just
count bytes as you read the local storage databases into memory).

  • storage/LocalStorage.cpp: (WebCore::LocalStorage::LocalStorage): (WebCore::LocalStorage::storageArea): (WebCore::LocalStorage::close):
  • storage/LocalStorage.h:
  • storage/LocalStorageArea.cpp: (WebCore::LocalStorageArea::LocalStorageArea): (WebCore::LocalStorageArea::scheduleFinalSync): (WebCore::LocalStorageArea::syncTimerFired): (WebCore::LocalStorageArea::performImport):
  • storage/LocalStorageArea.h: (WebCore::LocalStorageArea::create):
  • storage/LocalStorageTask.cpp: (WebCore::LocalStorageTask::performTask):
  • storage/LocalStorageTask.h: (WebCore::LocalStorageTask::): (WebCore::LocalStorageTask::createImport):
  • storage/LocalStorageThread.cpp:
  • storage/LocalStorageThread.h:
  • storage/SessionStorage.h:
  • storage/StorageSyncManager.cpp: Copied from WebCore/storage/LocalStorage.cpp. (WebCore::StorageSyncManager::create): (WebCore::StorageSyncManager::StorageSyncManager): (WebCore::StorageSyncManager::fullDatabaseFilename): (WebCore::StorageSyncManager::close): (WebCore::StorageSyncManager::scheduleImport): (WebCore::StorageSyncManager::scheduleSync):
  • storage/StorageSyncManager.h: Copied from WebCore/storage/LocalStorage.h. (WebCore::):
14:47 Changeset [44958] by barraclough@apple.com

Merged r43550:43552 into nitro-extreme branch - fix error in merging changelogs.

14:43 Changeset [44957] by barraclough@apple.com

Merged r43550:43552 into nitro-extreme branch.

14:38 Changeset [44956] by darin@chromium.org

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

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=26622
Add ChromiumDataObject::copy()

  • platform/chromium/ChromiumDataObject.cpp:
  • platform/chromium/ChromiumDataObject.h:
14:31 Changeset [44955] by timothy@apple.com

Add Mail on Tiger and Leopard to the list of applications that need the WebView init thread workaround.

<rdar://problem/6929524> Console shows WebKit Threading Violations from Mail

Reviewed by Anders Carlsson.

  • WebView/WebView.mm: (clientNeedsWebViewInitThreadWorkaround): Return true for com.apple.Mail.
14:30 Moving to Git edited by treat@kde.org
(diff)
14:25 Changeset [44954] by timothy@apple.com

Add a silhouette image for the enable view of the Resource panel.

Reviewed by Kevin McCullough.

  • inspector/front-end/Images/resourcesSilhouette.png: Added.
  • inspector/front-end/inspector.css:
14:02 Changeset [44953] by levin@chromium.org

2009-06-22 Kyle Prete <kylep@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=26552
Add V8 JS bindings for Audio Constructor.

  • bindings/v8/custom/V8CustomBinding.h: Added a callback declaration for audio ctor to macro.
  • bindings/v8/custom/V8HTMLAudioElementConstructor.cpp: Added the whole file.
13:14 Changeset [44952] by ap@webkit.org

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=26600
StringImpl::copy() no longer copies empty string correctly

  • platform/text/StringImpl.cpp: (WebCore::StringImpl::ustring): Changed method calling syntax to look less like calling a static method. (WebCore::StringImpl::copy): Reverted part of r41917 that caused the problem.
12:51 Changeset [44951] by mitz@apple.com

Reviewed by Anders Carlsson.

  • speculative fix for <rdar://problem/6889082> Crash at -[WebHTMLView(WebPrivate) _updateMouseoverWithEvent:]

The crash seems to happen because lastHitView points to a deleted
object. Since -close calls -_clearLastHitViewIfSelf, I speculate that
lastHitView has been set to an already-closed view.

  • WebView/WebHTMLView.mm: (-[WebHTMLView hitTest:]): Return nil if the view is closed.
12:34 Changeset [44950] by levin@chromium.org

2009-06-22 David Levin <levin@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix layout tests to account for removal of DOMStringList, which was done
in http://trac.webkit.org/changeset/44943.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/js/global-constructors-expected.txt:
12:20 Changeset [44949] by barraclough@apple.com

Merged r43544:43550 into nitro-extreme branch.

11:54 Changeset [44948] by jshin@chromium.org

2009-06-22 Xiaomei Ji <xji@chromium.org>

Reviewed by Eric Seidel

Not auto-testable since it involves sending a keyboard event to
the popup, which is not possible (eventSender sends the key
events through webview, we want to go through the webwidget).

This patch is one part of the fix for issue "keyboard selection in
Hebrew select element does not work in Windows". The other part of the
fix is in chromium's webkit/glue layer.
https://bugs.webkit.org/show_bug.cgi?id=25899

  • manual-tests/keyboard_select_non_english.html: Added.
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::isCharacterTypeEvent): Added. Check whether the event is a character type event. "Char" in Windows or "KeyDown" in Mac is character type event. (WebCore::PopupListBox::typeAheadFind): Since m_lastCharTime is used to indicate whether user types multiple characters continuely as a search prefix or not, it should be only assigned when the event is character type event.
11:51 Changeset [44947] by kmccullough@apple.com

2009-06-22 Kevin McCullough <kmccullough@apple.com>

Reviewed by Darin Adler.

<rdar://problem/6115819> Notify of profile start in console

Put a message in the console that announces that a profile has started,
similar to how a console message announces that a profile has finished.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::addProfile): Use renamed function. (WebCore::InspectorController::addProfileFinishedMessageToConsole): Renamed. (WebCore::InspectorController::addStartProfilingMessageToConsole): New function that logs the message about the profile starting. (WebCore::InspectorController::startUserInitiatedProfiling): Ditto
  • inspector/InspectorController.h:
  • inspector/front-end/ProfilesPanel.js: Print the "run" of the profile's title if there are multiple profiles, and take into account the fact that titles are displayed twice as much now. (WebInspector.ProfilesPanel.prototype.displayTitleForProfileLink):
  • page/Console.cpp: (WebCore::Console::profile): Call new function.
11:47 Changeset [44946] by hyatt@apple.com

2009-06-22 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

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

Add IDL interface for datagrid cell elements. Add a few more properties to the datagrid itself that
correspond to ones you find on <select> elements.

Add an expanded property to row elements.

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/HTMLAttributeNames.in:
  • html/HTMLDataGridCellElement.cpp: Added. (WebCore::HTMLDataGridCellElement::HTMLDataGridCellElement): (WebCore::HTMLDataGridCellElement::label): (WebCore::HTMLDataGridCellElement::setLabel): (WebCore::HTMLDataGridCellElement::focused): (WebCore::HTMLDataGridCellElement::setFocused): (WebCore::HTMLDataGridCellElement::checked): (WebCore::HTMLDataGridCellElement::setChecked): (WebCore::HTMLDataGridCellElement::indeterminate): (WebCore::HTMLDataGridCellElement::setIndeterminate): (WebCore::HTMLDataGridCellElement::progress): (WebCore::HTMLDataGridCellElement::setProgress):
  • html/HTMLDataGridCellElement.h: Added. (WebCore::HTMLDataGridCellElement::endTagRequirement): (WebCore::HTMLDataGridCellElement::tagPriority):
  • html/HTMLDataGridCellElement.idl: Added.
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::autofocus): (WebCore::HTMLDataGridElement::setAutofocus): (WebCore::HTMLDataGridElement::disabled): (WebCore::HTMLDataGridElement::setDisabled): (WebCore::HTMLDataGridElement::size): (WebCore::HTMLDataGridElement::setSize):
  • html/HTMLDataGridElement.h:
  • html/HTMLDataGridElement.idl:
  • html/HTMLDataGridRowElement.cpp: (WebCore::HTMLDataGridRowElement::checkDTD): (WebCore::HTMLDataGridRowElement::expanded): (WebCore::HTMLDataGridRowElement::setExpanded):
  • html/HTMLDataGridRowElement.h:
  • html/HTMLDataGridRowElement.idl:
  • html/HTMLElementsAllInOne.cpp:
  • html/HTMLTagNames.in:
11:32 Changeset [44945] by sfalken@apple.com

Fix last DumpRenderTree change to correctly set preferences flag without crashing.


Reviewed by Mark Rowe.

  • DumpRenderTree/win/DumpRenderTree.cpp: (main):
11:32 Changeset [44944] by weinig@apple.com

WebCore:

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

Reviewed by Mark Rowe.

Add event handler attribute getter/setters to HTMLBodyElement
as specified in HTML 5.

  • Removes erroneous inclusion of the onresize and onunload event handler attributes in Element and Document.
  • dom/Document.cpp: (WebCore::Document::getWindowAttributeEventListener):
  • dom/Document.h:
  • dom/Document.idl:
  • dom/Element.idl:
  • dom/Node.cpp:
  • dom/Node.h: onblur, onerror, onfocus, and onload all need to be declared virtual, since their behavior for HTMLBodyElement differs from the base implementation.
  • html/HTMLBodyElement.cpp: (WebCore::HTMLBodyElement::onblur): (WebCore::HTMLBodyElement::setOnblur): (WebCore::HTMLBodyElement::onerror): (WebCore::HTMLBodyElement::setOnerror): (WebCore::HTMLBodyElement::onfocus): (WebCore::HTMLBodyElement::setOnfocus): (WebCore::HTMLBodyElement::onload): (WebCore::HTMLBodyElement::setOnload): (WebCore::HTMLBodyElement::onbeforeunload): (WebCore::HTMLBodyElement::setOnbeforeunload): (WebCore::HTMLBodyElement::onmessage): (WebCore::HTMLBodyElement::setOnmessage): (WebCore::HTMLBodyElement::onoffline): (WebCore::HTMLBodyElement::setOnoffline): (WebCore::HTMLBodyElement::ononline): (WebCore::HTMLBodyElement::setOnonline): (WebCore::HTMLBodyElement::onresize): (WebCore::HTMLBodyElement::setOnresize): (WebCore::HTMLBodyElement::onstorage): (WebCore::HTMLBodyElement::setOnstorage): (WebCore::HTMLBodyElement::onunload): (WebCore::HTMLBodyElement::setOnunload):
  • html/HTMLBodyElement.h:
  • html/HTMLBodyElement.idl:

LayoutTests:

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

Reviewed by Mark Rowe.

Add event handler attribute getter/setters to HTMLBodyElement
as specified in HTML 5.

  • fast/dom/event-attribute-availability-expected.txt:
  • fast/dom/resources/event-attribute-availability.js:
11:22 Changeset [44943] by ap@webkit.org

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26601
Remove disabled code for appcache dynamic entries

This also removes code for DOM 3 Core DOMStringList, which we don't use for anything else yet.

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • bindings/js/JSDOMStringListCustom.cpp: Removed.
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8DOMStringListCustom.cpp: Removed.
  • dom/DOMStringList.cpp: Removed.
  • dom/DOMStringList.h: Removed.
  • dom/DOMStringList.idl: Removed.
  • dom/StaticStringList.cpp: Removed.
  • dom/StaticStringList.h: Removed.
  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::shouldLoadResourceFromApplicationCache):
  • loader/appcache/ApplicationCache.cpp: (WebCore::ApplicationCache::addResource):
  • loader/appcache/ApplicationCache.h:
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::didFinishLoadingManifest): (WebCore::ApplicationCacheGroup::checkIfLoadIsComplete):
  • loader/appcache/ApplicationCacheResource.cpp: (WebCore::ApplicationCacheResource::dumpType):
  • loader/appcache/ApplicationCacheResource.h: (WebCore::ApplicationCacheResource::):
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::storeUpdatedType):
  • loader/appcache/DOMApplicationCache.cpp:
  • loader/appcache/DOMApplicationCache.h:
  • loader/appcache/DOMApplicationCache.idl:
  • page/DOMWindow.idl:
10:56 Changeset [44942] by sfalken@apple.com

Windows build fix.

  • DerivedSources.cpp:
  • html/HTMLElementsAllInOne.cpp:
10:33 Changeset [44941] by ap@webkit.org

Reviewed by John Sullivan.

<rdar://problem/6956606> REGRESSION (S4Beta -> Final): After the password is input,
Japanese can't be input.

WebKit uses a per-frame input context for most editable fields, and application's global
context for password fields, manually controlling secure input state and the list of available
input methods.

We were setting TSMDocument property to disable non-Roman input methods after focus changed
to a password field, but before selection was updated. This resulted in the property being
applied to a wrong TSMDocument in some cases, because TSMGetActiveDocument() uses input
context cached by +[NSApplication updateWindows], we change context based on selection,
not on focus.

  • page/mac/FrameMac.mm: (WebCore::Frame::setUseSecureKeyboardEntry): Since there is only one context that we use for password fields - the global one - there is no need to get the active one.
10:20 Changeset [44940] by levin@chromium.org

WebCore:

2009-06-22 David Levin <levin@chromium.org>

Reviewed by David Hyatt and Eric Seidel.

REGRESSION: When the main page (ScrollView) has a custom scrollbar, it crashes on destruction.
https://bugs.webkit.org/show_bug.cgi?id=26326

Test: scrollbars/scrollbar-crash-on-refresh.html

  • dom/Document.cpp: (WebCore::Document::detach): Gives the FrameView a change to do any necessary cleanup on Document::detach() which is where the renderArena gets detroyed.
  • page/FrameView.cpp: (WebCore::FrameView::detachCustomScrollbars): Gets rid of any custom scrollbars (if the docment supplied them).
  • page/FrameView.h:
  • platform/Scrollbar.h: (WebCore::Scrollbar::isCustomScrollbar):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy): Removed the check for document()->frame(). If frame() is 0 in this code, then the call to animation() is also incorrect (since it does document()->frame()->animation()).
  • rendering/RenderScrollbar.h: (WebCore::RenderScrollbar::isCustomScrollbar):

LayoutTests:

2009-06-22 David Levin <levin@chromium.org>

Reviewed by David Hyatt and Eric Seidel.

REGRESSION: When the main page (ScrollView) has a custom scrollbar, it crashes on destruction.
https://bugs.webkit.org/show_bug.cgi?id=26326

Test that refreshing an iframe with custom scrollbars works without crashing.

  • scrollbars/resources/page-with-custom-scrollbars.html: Added.
  • scrollbars/scrollbar-crash-on-refresh-expected.txt: Added.
  • scrollbars/scrollbar-crash-on-refresh.html: Added.
09:50 Changeset [44939] by mitz@apple.com

Reviewed by Dave Hyatt.

  • fix <rdar://problem/6990938> REGRESSION (r42787): After showing and hiding the Find banner, the WebHTMLView's height is not restored
  • WebView/WebFrameView.mm: (-[WebFrameView setFrameSize:]): Mark the FrameView for layout when the WebFrameView's size changes.
  • WebView/WebView.mm: (-[WebView setFrameSize:]): Left the resize logic here, but only for the single view model.
09:42 Changeset [44938] by mitz@apple.com

WebCore:

Reviewed by Simon Fraser.

Test: fast/repaint/background-generated.html

  • rendering/RenderObject.cpp: (WebCore::mustRepaintFillLayers): Return true if the layer does not specify a size and the image uses the container size.

LayoutTests:

Reviewed by Simon Fraser.

  • fast/repaint/background-generated.html: Added.
  • platform/mac/fast/repaint/background-generated-expected.checksum: Added.
  • platform/mac/fast/repaint/background-generated-expected.png: Added.
  • platform/mac/fast/repaint/background-generated-expected.txt: Added.
09:34 Changeset [44937] by hyatt@apple.com

2009-06-22 David Hyatt <hyatt@apple.com>

Reviewed by Sam Weinig.

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

Add IDL for HTMLDataGridRowElement.

  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • html/HTMLAttributeNames.in:
  • html/HTMLDataGridColElement.cpp: (WebCore::HTMLDataGridColElement::primary): (WebCore::HTMLDataGridColElement::setPrimary):
  • html/HTMLDataGridColElement.h:
  • html/HTMLDataGridColElement.idl:
  • html/HTMLDataGridElement.cpp: (WebCore::HTMLDataGridElement::checkDTD): (WebCore::HTMLDataGridElement::multiple): (WebCore::HTMLDataGridElement::setMultiple):
  • html/HTMLDataGridElement.h:
  • html/HTMLDataGridElement.idl:
  • html/HTMLDataGridRowElement.cpp: Added. (WebCore::HTMLDataGridRowElement::HTMLDataGridRowElement): (WebCore::HTMLDataGridRowElement::selected): (WebCore::HTMLDataGridRowElement::setSelected): (WebCore::HTMLDataGridRowElement::focused): (WebCore::HTMLDataGridRowElement::setFocused):
  • html/HTMLDataGridRowElement.h: Added. (WebCore::HTMLDataGridRowElement::tagPriority):
  • html/HTMLDataGridRowElement.idl: Added.
  • html/HTMLTagNames.in:
09:32 Changeset [44936] by darin@apple.com

2009-06-22 Darin Adler <darin@apple.com>

  • platform/gtk/Skipped: Skip two new tests.
08:44 Changeset [44935] by darin@apple.com

2009-06-22 Darin Adler <darin@apple.com>

  • fast/js/JSON-stringify-expected.txt: Update results.
06:22 Changeset [44934] by yael.aharon@nokia.com

2009-06-19 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Holger Freyther.

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

Fix a crash in case that the QNetworkReply::readReady signal is
"stuck" in deferred mode due to JavaScript alert, and in the meantime,
QNetworkreply::finished signal is received and processed before
QNetworkReply::readReady is processed.

  • manual-tests/qt/unload-alert.html: Added.
  • platform/network/qt/QNetworkReplyHandler.cpp:
  • platform/network/qt/QNetworkReplyHandler.h:
04:55 Changeset [44933] by vestbo@webkit.org

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

Fix the Qt build.

  • WebCore.pro:
00:33 Changeset [44932] by sfalken@apple.com

Set up global native controls flag before creating the first WebView.


Reviewed by Darin Adler.

  • DumpRenderTree/win/DumpRenderTree.cpp: (main):

06/21/09:

23:35 Changeset [44931] by oliver@apple.com

Bug 26592: Support standard toJSON functions
<https://bugs.webkit.org/show_bug.cgi?id=26592>

Reviewed by Darin Adler

Add support for the standard Date.toJSON function.

23:17 Changeset [44930] by darin@apple.com

2009-06-21 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

Bug 25425: DOM attribute getter/setter functions should use const AtomicString& type
https://bugs.webkit.org/show_bug.cgi?id=25425

  • bindings/scripts/CodeGeneratorJS.pm: Tweaked mechanism for includes to be a bit more consistent and to make SVGElement.h be included in the header rather than in every implementation file that includes the header. Added code to use getAttribute and setAttribute directly when the [Reflect] extended attribute is used.
  • bindings/scripts/CodeGeneratorObjC.pm: Ditto.
  • html/HTMLElement.idl: Used [Reflect] for all the attributes in this class that reflect content attributes. Restricting this to one class for now to keep the patch small and start out slowly.
22:08 Changeset [44929] by oliver@apple.com

Bug 26594: JSC needs to support Date.toISOString
<https://bugs.webkit.org/show_bug.cgi?id=26594>

Reviewed by Sam Weinig

Add support for Date.toISOString.

21:59 Changeset [44928] by weinig@apple.com

WebCore:

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

Reviewed by Oliver Hunt.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26596
Only expose event handler attributes to elements and documents.

Test: fast/dom/event-attribute-availability.html

  • dom/Document.idl:
  • dom/Element.idl:
  • dom/Node.cpp:
  • dom/Node.h:
  • dom/Node.idl:

LayoutTests:

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

Reviewed by Oliver Hunt.

Test for https://bugs.webkit.org/show_bug.cgi?id=26596
Only expose event handler attributes to elements and documents.

  • fast/dom/event-attribute-availability-expected.txt: Added.
  • fast/dom/event-attribute-availability.html: Added.
  • fast/dom/resources/event-attribute-availability.js: Added.
18:39 Changeset [44927] by abarth@webkit.org

WebCore:

2009-06-21 Daniel Bates <dbates@intudata.com>

Reviewed by Adam Barth.

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


Fix to enable XSSAuditor on child windows.


Test: http/tests/security/xssAuditor/link-opens-new-window.html


  • page/XSSAuditor.cpp: (WebCore::XSSAuditor::XSSAuditor): (WebCore::XSSAuditor::isEnabled): (WebCore::XSSAuditor::canEvaluate): (WebCore::XSSAuditor::canCreateInlineEventListener): (WebCore::XSSAuditor::canLoadExternalScriptFromSrc): (WebCore::XSSAuditor::canLoadObject):
  • page/XSSAuditor.h: Removed method setXSSAuditorEnabled, and field m_isEnabled. Moved implementation of isEnabled to XSSAuditor.cpp and changed implementation to query Settings.

LayoutTests:

2009-06-21 Daniel Bates <dbates@intudata.com>

Reviewed by Adam Barth.


Test for https://bugs.webkit.org/show_bug.cgi?id=26580


Tests that XSSAuditor (if enabled) prevents script execution in child window.

  • http/tests/security/xssAuditor/link-opens-new-window-expected.txt: Added.
  • http/tests/security/xssAuditor/link-opens-new-window.html: Added.
  • http/tests/security/xssAuditor/resources/echo-intertag-and-notify.php: Added.
17:23 Changeset [44926] by mitz@apple.com

Rubber-stamped by Mark Rowe.

  • make repaint tests more reliable when run in Safari
  • fast/repaint/resources/repaint.js: (runRepaintTest): Increased the timeout from 0 to 100, to ensure that painting occurs once before repaintTest() is called.
17:03 Changeset [44925] by mrowe@apple.com

Set a sensible user agent string for the HTTP requests that Sparkle makes (checking for and downloading updates).

Reviewed by Sam Weinig.

  • WebKitLauncher/WebKitNightlyEnablerSparkle.m:

(userAgentStringForSparkle):
(initializeSparkle):

16:32 Changeset [44924] by oliver@apple.com

Fix build by removing dead code.

16:02 Changeset [44923] by oliver@apple.com

Bug 26587: Support JSON.parse
<https://bugs.webkit.org/show_bug.cgi?id=26587>

Reviewed by Darin Adler and Cameron Zwarich.

Extend the LiteralParser to support the full strict JSON
grammar, fix a few places where the grammar was incorrectly
lenient. Doesn't yet support the JSON.parse reviver function
but that does not block the JSON.parse functionality itself.

15:46 Changeset [44922] by levin@chromium.org

2009-06-21 David Levin <levin@chromium.org>

Reviewed by NOBODY (speculative build fix for windows).

Simply removed some whitespace form this file to make windows build wtf and
hopefully copy the new MessageQueque.h so that WebCore picks it up.

  • wtf/Assertions.cpp:
15:16 Changeset [44921] by levin@chromium.org

2009-06-21 David Levin <levin@chromium.org>

Reviewed by NOBODY.

Speculative windows build fix (idea by Mark Rowe).

  • WebCore.vcproj/WebCore.vcproj:
15:03 Changeset [44920] by mrowe@apple.com

Add a hook to the WebKit launcher application to allow a link on the nightly build start page to
trigger an update via the built-in software update mechanism.

Reviewed by Sam Weinig.

  • WebKitLauncher/WebKitLauncher.xcodeproj/project.pbxproj:
  • WebKitLauncher/WebKitLauncherURLProtocol.h: Added.
  • WebKitLauncher/WebKitLauncherURLProtocol.m: Added.

(+[WebKitLauncherURLProtocol load]):
(+[WebKitLauncherURLProtocol canInitWithRequest:]): Only allow use of the x-webkit-launcher scheme from .webkit.org subdomains.
(+[WebKitLauncherURLProtocol canonicalRequestForRequest:]):
(-[WebKitLauncherURLProtocol startLoading]):
(-[WebKitLauncherURLProtocol stopLoading]):
(-[WebKitLauncherURLProtocol handleIsWebKitLauncherAvailableJS]): Return a brief JavaScript snippet that can be used to programatically
determine whether the x-webkit-launcher is available and working.
(-[WebKitLauncherURLProtocol handleCheckForUpdates]): Trigger a software update on the main thread.
(-[WebKitLauncherURLProtocol resourceNotFound]): Fail with a generic "File does not exist" error.

15:02 Changeset [44919] by levin@chromium.org

2009-06-21 David Levin <levin@chromium.org>

Reviewed by NOBODY (Speculative gtk build fix).

  • GNUmakefile.am:
14:56 Changeset [44918] by levin@chromium.org

2009-06-21 David Levin <levin@chromium.org>

Reviewed by NOBODY.

Speculative tiger build fix and a windows build fix.
More fixes may still be in order.

  • WebCore.vcproj/WebCoreCommon.vsprops:
  • dom/default/PlatformMessagePortChannel.h: (WebCore::PlatformMessagePortChannel::MessagePortQueue::tryGetMessage):
14:46 Changeset [44917] by levin@chromium.org

2009-06-21 David Levin <levin@chromium.org>

Reviewed by NOBODY (Speculative tiger build fix).

  • dom/default/PlatformMessagePortChannel.h: (WebCore::PlatformMessagePortChannel::MessagePortQueue::~MessagePortQueue):
14:32 Changeset [44916] by levin@chromium.org

WebCore:

2009-06-21 Drew Wilson <atwilson@google.com>

Reviewed by David Levin.

<https://bugs.webkit.org/show_bug.cgi?id=26448>

Added optimized GC for MessagePorts when the entangled port is run by the same thread.
Fixed bug in isProxyFor() that was not properly throwing an exception when trying to clone the entangled port.

  • bindings/js/JSDOMBinding.cpp: (WebCore::markActiveObjectsForContext): Now marks remotely entangled ports as in-use, in addition to those with pending activity.
  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::mark): Now checks if the entangled port is local (run by same thread) and if so mark()s it.
  • dom/MessagePort.cpp: (WebCore::MessagePort::postMessage): (WebCore::MessagePort::disentangle): Removes cloned ports from the ScriptExecutionContext - this allows cloned ports to be GC'd as otherwise they look like remotely entangled ports. (WebCore::MessagePort::start): (WebCore::MessagePort::locallyEntangledPort): Added API for fetching the entangled port if it is run by the same thread
  • dom/MessagePort.h:
  • dom/MessagePortProxyWrapper.h:
  • dom/default/MessagePortProxy.cpp: (WebCore::MessagePortProxyWrapper::locallyEntangledPort): Added API for fetching the entangled port if it is run by the same thread (WebCore::MessagePortProxy::hasPendingActivity): Changed definition of hasPendingActivity() to be stricter - only returns true if there are pending messages. (WebCore::MessagePortProxy::locallyEntangledPort):
  • dom/default/MessagePortProxy.h:

LayoutTests:

2009-06-21 Drew Wilson <atwilson@google.com>

Reviewed by David Levin.

<https://bugs.webkit.org/show_bug.cgi?id=26448>

New tests for MessagePort GC cases that weren't previously covered.

  • fast/events/message-channel-gc-4-expected.txt: Added. Test for case where both MessagePorts are in-transit (cloned) when a GC occurs.
  • fast/events/message-channel-gc-4.html-disabled: Added.
14:32 Changeset [44915] by levin@chromium.org

JavaScriptCore:

2009-06-21 Drew Wilson <atwilson@google.com>

Reviewed by David Levin.

<https://bugs.webkit.org/show_bug.cgi?id=25043>
Added support for multi-threaded MessagePorts.

  • wtf/MessageQueue.h: (WTF::::appendAndCheckEmpty):

Added API to test whether the queue was empty before adding an element.

WebCore:

2009-06-21 Drew Wilson <atwilson@google.com>

Reviewed by David Levin.

<https://bugs.webkit.org/show_bug.cgi?id=25043>
Removed obsolete MessagePort.startConversation(), active and onclose APIs.

Refactored MessagePortProxy into MessagePortChannel and a platform-dependent PlatformMessagePortChannel
implementation. Modified APIs to simplify cross-process implementations by moving the messaging code
entirely into the platform-dependent proxy.

Created a thread-safe default PlatformMessagePortChannel implementation.

Changed DOMWindow messaging to create the MessageEvent in the target ScriptExecutionContext to match how
cross-thread MessagePorts work.

  • GNUMakefile.am:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Added MessagePortChannel/PlatformMessagePortChannel files.

  • bindings/js/JSMessagePortCustom.cpp: (WebCore::JSMessagePort::mark):

Changed ports to not mark their entangled pair as reachable, per the spec.

  • bindings/v8/custom/V8MessagePortCustom.cpp:
  • dom/MessageChannel.cpp: (WebCore::MessageChannel::MessageChannel):

Updated to use PlatformMessagePortChannel::createChannel() to entangle the ports.

  • dom/MessagePort.cpp: (WebCore::MessagePort::MessagePort): (WebCore::MessagePort::~MessagePort): (WebCore::MessagePort::postMessage): (WebCore::MessagePort::disentangle): (WebCore::MessagePort::messageAvailable): (WebCore::MessagePort::start): (WebCore::MessagePort::close): (WebCore::MessagePort::entangle): (WebCore::MessagePort::contextDestroyed): (WebCore::MessagePort::dispatchMessages): (WebCore::MessagePort::setOnmessage): (WebCore::MessagePort::hasPendingActivity):

Changed these APIs to delegate to new PlatformMessagePortChannel APIs.

  • dom/MessagePort.h:

Renamed isQueueOpen() to started().

  • dom/MessagePort.idl:

Removed startConversation and onclose.

  • dom/MessagePortProxy.h: Removed.
  • dom/MessagePortChannel.cpp: Added. (WebCore::MessagePortChannel::EventData::create): (WebCore::MessagePortChannel::EventData::EventData): (WebCore::MessagePortChannel::~MessagePortChannel):
  • dom/MessagePortChannel.h: Added. (WebCore::MessagePortChannel::EventData::message): (WebCore::MessagePortChannel::EventData::channel):

Changed EventData to hold a reference to a MessagePortChannel object instead of a MessagePort to enable cross-thread messaging.

(WebCore::MessagePortChannel::MessagePortChannel):
(WebCore::MessagePortChannel::create):

  • dom/ScriptExecutionContext.cpp: (WebCore:ScriptExecutionContext::dispatchMessagePortEvents):

Renamed isQueueOpen() to started().

  • dom/default/PlatformMessagePortChannel.cpp: Added. (WebCore::MessagePortChannel::entangleIfOpen): (WebCore::MessagePortChannel::disentangle): (WebCore::MessagePortChannel::postMessageToRemote): (WebCore::MessagePortChannel::tryGetMessageFromRemote): (WebCore::MessagePortChannel::close): (WebCore::MessagePortChannel::isConnectedTo): (WebCore::MessagePortChannel::hasPendingActivity): (WebCore::PlatformMessagePortChannel::PlatformMessagePortChannel): (WebCore::PlatformMessagePortChannel::~PlatformMessagePortChannel): (WebCore::PlatformMessagePortChannel::createChannel): (WebCore::PlatformMessagePortChannel::entangleIfOpen): (WebCore::PlatformMessagePortChannel::disentangle): (WebCore::PlatformMessagePortChannel::setRemotePort): (WebCore::PlatformMessagePortChannel::remotePort): (WebCore::PlatformMessagePortChannel::entangledChannel): (WebCore::PlatformMessagePortChannel::setEntangledChannel): (WebCore::PlatformMessagePortChannel::postMessageToRemote): (WebCore::PlatformMessagePortChannel::tryGetMessageFromRemote): (WebCore::PlatformMessagePortChannel::isConnectedTo): (WebCore::PlatformMessagePortChannel::close): (WebCore::PlatformMessagePortChannel::closeInternal): (WebCore::PlatformMessagePortChannel::hasPendingActivity):
  • dom/default/PlatformMessagePortChannel.h: Added. (WebCore::PlatformMessagePortChannel::MessagePortQueue::create): (WebCore::PlatformMessagePortChannel::MessagePortQueue::tryGetMessage): (WebCore::PlatformMessagePortChannel::MessagePortQueue::appendAndCheckEmpty): (WebCore::PlatformMessagePortChannel::MessagePortQueue::isEmpty): (WebCore::PlatformMessagePortChannel::MessagePortQueue::MessagePortQueue):
  • page/DOMWindow.cpp: (WebCore::PostMessageTimer::PostMessageTimer):

Changed DOMWindow messaging to create the MessageEvent in the target ScriptExecutionContext to match how cross-thread MessagePorts work.

(WebCore::PostMessageTimer::event):
(WebCore::DOMWindow::postMessage):
(WebCore::DOMWindow::postMessageTimerFired):

LayoutTests:

2009-06-21 Drew Wilson <atwilson@google.com>

Reviewed by David Levin.

<https://bugs.webkit.org/show_bug.cgi?id=25043>
Updated MessageChannel/MessagePorts tests to reflect latest spec (close event has been removed).
Added more tests of port cloning.

  • fast/events/message-channel-gc-2-expected.txt:
  • fast/events/message-channel-gc-2.html-disabled:
  • fast/events/message-channel-gc-3-expected.txt:
  • fast/events/message-channel-gc-3.html-disabled:
  • fast/events/message-channel-gc-expected.txt:
  • fast/events/message-channel-gc.html-disabled:
  • fast/events/message-port-clone-expected.txt: Added.
  • fast/events/message-port-clone.html-disabled: Added.
  • fast/events/message-port-expected.txt:
  • fast/events/message-port-no-wrapper-expected.txt: Added.
  • fast/events/message-port-onclose-expected.txt: Removed.
  • fast/events/message-port-onclose.html-disabled: Removed.
  • fast/events/message-port.html-disabled:
13:56 Changeset [44914] by mitz@apple.com

WebCore:

Reviewed by Sam Weinig.

Test: fast/css/resize-single-axis.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::resize): Ignore movement along the non-resizing axis.

LayoutTests:

Reviewed by Sam Weinig.

  • fast/css/resize-single-axis-expected.txt: Added.
  • fast/css/resize-single-axis.html: Added.
13:47 Changeset [44913] by weinig@apple.com

WebCore:

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

Reviewed by Eric Seidel

Fix for https://bugs.webkit.org/show_bug.cgi?id=26578
event.currentTarget for listener registered on window should point to the window object like in Firefox

Test: fast/events/event-trace.html

  • dom/Node.cpp: (WebCore::Node::dispatchGenericEvent): Make the DOMWindow the currentTarget when events are dispatched to it. We previously used the document because DOMWindow was not yet an EventTarget.

LayoutTests:

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

Reviewed by Eric Seidel.

Test for https://bugs.webkit.org/show_bug.cgi?id=26578
event.currentTarget for listener registered on window should point to the window object like in Firefox

  • fast/events/event-trace-expected.txt: Added.
  • fast/events/event-trace.html: Added.
13:16 Changeset [44912] by mitz@apple.com

WebCore:

Reviewed by Sam Weinig.

Test: fast/css/resize-value-compared.html

  • rendering/style/StyleRareInheritedData.cpp: (WebCore::StyleRareInheritedData::operator==): Compare the resize member.

LayoutTests:

Reviewed by Sam Weinig.

  • fast/css/resize-value-compared-expected.txt: Added.
  • fast/css/resize-value-compared.html: Added.
00:19 Changeset [44911] by darin@apple.com

2009-06-21 Darin Adler <darin@apple.com>

  • platform/gtk/Skipped: Skip one more test.

06/20/09:

23:19 Changeset [44910] by darin@apple.com

2009-06-20 Darin Adler <darin@apple.com>

  • platform/gtk/Skipped: Skip some new tests.
23:15 Changeset [44909] by darin@apple.com

2009-06-20 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6964221> Need more processing of pluginspage.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::createPlugin): Check protocolInHTTPFamily.
22:42 Changeset [44908] by alice.liu@apple.com

WebCore:

2009-06-19 Alice Liu <alice.liu@apple.com>

Fix https://bugs.webkit.org/show_bug.cgi?id=26568
Repro crash animating GIF if previously used in a closed window's back/forward list
Also filed as <rdar://problem/6978362>

Reviewed by Maciej Stachowiak.

Can't test this bug with an automated layout test since it requires b/f caching

  • manual-tests/animated-gif-bfcache-crash.html: Added.
  • manual-tests/resources/containsAnimatedGif.html: Added.

Adding checks for hostWindow() since there is no guarantee that the Page is alive

  • platform/ScrollView.cpp: (WebCore::ScrollView::scrollRectIntoViewRecursively): (WebCore::ScrollView::contentsToScreen): (WebCore::ScrollView::screenToContents): (WebCore::ScrollView::wheelEvent):

LayoutTests:

2009-06-19 Alice Liu <alice.liu@apple.com>

Fix https://bugs.webkit.org/show_bug.cgi?id=26568
Repro crash animating GIF if previously used in a closed window's back/forward list
Also filed as <rdar://problem/6978362>

Reviewed by Maciej Stachowiak.

Adding resource used in WebCore/manual-tests/animated-gif-bfcache-crash.html

  • http/tests/misc/resources/animated-gif-with-offsets.gif: Added.
19:43 Changeset [44907] by mrowe@apple.com

Roll out the Windows portion of r44905 as it broke the build.

19:05 Changeset [44906] by weinig@apple.com

WebCore:

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

Reviewed by Adam Barth.

Fix for https://bugs.webkit.org/show_bug.cgi?id=26554
Shadowing of top and parent

  • page/DOMWindow.idl:

LayoutTests:

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

Reviewed by Adam Barth.

Test for https://bugs.webkit.org/show_bug.cgi?id=26554

Test writing to parent and top.

  • http/tests/security/cross-frame-access-put-expected.txt:
  • http/tests/security/cross-frame-access-put.html:
  • http/tests/security/resources/cross-frame-iframe-for-put-test.html:
16:07 Changeset [44905] by darin@apple.com

WebKit/mac:

2009-06-20 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6964221> Need more processing of pluginspage.

  • WebCoreSupport/WebFrameLoaderClient.mm: (WebFrameLoaderClient::createPlugin): Check protocolInHTTPFamily.

WebKit/win:

2009-06-20 Darin Adler <darin@apple.com>

Reviewed by Sam Weinig.

<rdar://problem/6964221> Need more processing of pluginspage.

  • WebCoreSupport/WebFrameLoaderClient.cpp: (WebFrameLoaderClient::createPlugin): Check protocolInHTTPFamily.
14:57 Changeset [44904] by jmalonzo@webkit.org

Reviewed by Gustavo Noronha and Xan Lopez.

[Gtk] Implement DRT XSS auditor support
https://bugs.webkit.org/show_bug.cgi?id=26571

  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init): (webkit_web_settings_set_property): (webkit_web_settings_get_property): (webkit_web_settings_copy):
  • webkit/webkitwebview.cpp: (webkit_web_view_update_settings): (webkit_web_view_settings_notify):
  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetWebViewToConsistentStateBeforeTesting):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setXSSAuditorEnabled):
14:56 Changeset [44903] by jmalonzo@webkit.org

2009-06-20 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Gustavo Noronha and Xan Lopez.

webkit_web_back_forward_list_add_item() should add a ref for the history item
https://bugs.webkit.org/show_bug.cgi?id=26517

Add a ref to the WebHistoryItem when it's added to the back
forward list through webkit_web_back_forward_list_add_item.
This keeps the history item alive and prevents pywebkitgtk from
crashing

  • tests/testwebbackforwardlist.c: (test_webkit_web_history_item_lifetime): (test_webkit_web_back_forward_list_order): (test_webkit_web_back_forward_list_add_item):
  • webkit/webkitwebbackforwardlist.cpp: (webkit_web_back_forward_list_add_item):
14:54 Changeset [44902] by mrowe@apple.com

Fix <https://bugs.webkit.org/show_bug.cgi?id=15106>.
Bug 15106: REGRESSION: Can't drag text as element when user-select is set to none

Reviewed by Dan Bernstein.

We allow selections to begin within an element that has -webkit-user-select: none set,
unless the element is draggable as dragging should take priority over starting a selection.

Test: editing/selection/user-drag-element-and-user-select-none.html

  • dom/Node.cpp:

(WebCore::Node::canStartSelection):

Test for <https://bugs.webkit.org/show_bug.cgi?id=15106>.
Bug 15106: REGRESSION: Can't drag text as element when user-select is set to none

  • editing/selection/user-drag-element-and-user-select-none-expected.txt: Added.
  • editing/selection/user-drag-element-and-user-select-none.html: Added.
13:34 Changeset [44901] by darin@apple.com

WebCore:

2009-06-20 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 26528: REGRESSION (r44674): Assertion in TextIterator::pushFullyClippedState while counting matches for "a" on apple.com
https://bugs.webkit.org/show_bug.cgi?id=26528
rdar://problem/6985329

  • editing/TextIterator.cpp: (WebCore::TextIterator::handleReplacedElement): When entering a text control, start at the top of the shadow tree (by calling shadowTreeRootNode). Also remove assumption that innerTextElement will never be 0 since RenderTextControl doesn't really guarantee this.

LayoutTests:

2009-06-20 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 26528: REGRESSION (r44674): Assertion in TextIterator::pushFullyClippedState while counting matches for "a" on apple.com
https://bugs.webkit.org/show_bug.cgi?id=26528
rdar://problem/6985329

  • editing/selection/find-in-text-control-expected.txt: Updated to expect finding text in <input type="text"> and <input type="search">, but not in <input type="password">.
  • editing/selection/find-in-text-control.html: Added tests for <input> elements as well as <textarea> elements. Also tweaked result output format to accomodate multiple text nodes.
11:06 Changeset [44900] by ddkilzer@apple.com

Fix namespace comment in SegmentedVector.h

  • wtf/SegmentedVector.h: Updated namespace comment to reflect new namespace after r44897.
06:27 Changeset [44899] by ddkilzer@apple.com

2009-06-20 Zoltan Herczeg <zherczeg@inf.u-szeged.hu>

Bug 24986: ARM JIT port
<https://bugs.webkit.org/show_bug.cgi?id=24986>

Reviewed by Oliver Hunt.

An Iterator added for SegmentedVector. Currently
only the pre ++ operator is supported.

  • wtf/SegmentedVector.h: (WTF::SegmentedVectorIterator::~SegmentedVectorIterator): (WTF::SegmentedVectorIterator::operator*): (WTF::SegmentedVectorIterator::operator->): (WTF::SegmentedVectorIterator::operator++): (WTF::SegmentedVectorIterator::operator==): (WTF::SegmentedVectorIterator::operator!=): (WTF::SegmentedVectorIterator::operator=): (WTF::SegmentedVectorIterator::SegmentedVectorIterator): (WTF::SegmentedVector::alloc): (WTF::SegmentedVector::begin): (WTF::SegmentedVector::end):
06:20 Changeset [44898] by kov@webkit.org

2009-06-20 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Jan Alonzo.

Initial test case for loading statuses.

  • tests/testloading.c: Added. (load_finished_cb): (status_changed_cb): (test_loading_status): (main):
06:14 Changeset [44897] by ddkilzer@apple.com

2009-06-20 Zoltan Herczeg <zherczeg@inf.u-szeged.hu>

Bug 24986: ARM JIT port
<https://bugs.webkit.org/show_bug.cgi?id=24986>

Reviewed by Oliver Hunt.

Move SegmentedVector to /wtf subdirectory
and change "namespace JSC" to "namespace WTF"

Additional build file updates by David Kilzer.

  • GNUmakefile.am: Updated path to SegmentedVector.h.
  • JavaScriptCore.order: Updated SegmentedVector namespace from JSC to WTF in mangled C++ method name.
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: Removed reference to bytecompiler\SegmentedVector.h.
  • JavaScriptCore.vcproj/WTF/WTF.vcproj: Added reference to wtf\SegmentedVector.h.
  • JavaScriptCore.xcodeproj/project.pbxproj: Moved SegmentedVector.h definition from bytecompiler subdirectory to wtf subdirectory.
  • bytecompiler/BytecodeGenerator.h: Updated #include path to
SegmentedVector.h and prepended WTF
namespace to its use.
  • parser/Lexer.h: Ditto.
  • wtf/SegmentedVector.h: Renamed from JavaScriptCore/bytecompiler/SegmentedVector.h. (WTF::SegmentedVector::SegmentedVector): (WTF::SegmentedVector::~SegmentedVector): (WTF::SegmentedVector::size): (WTF::SegmentedVector::at): (WTF::SegmentedVector::operator[]): (WTF::SegmentedVector::last): (WTF::SegmentedVector::append): (WTF::SegmentedVector::removeLast): (WTF::SegmentedVector::grow): (WTF::SegmentedVector::clear): (WTF::SegmentedVector::deleteAllSegments): (WTF::SegmentedVector::segmentExistsFor): (WTF::SegmentedVector::segmentFor): (WTF::SegmentedVector::subscriptFor): (WTF::SegmentedVector::ensureSegmentsFor): (WTF::SegmentedVector::ensureSegment):
05:34 Changeset [44896] by kov@webkit.org

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

Rubber-stamped by Jan Alonzo.

Fix the ordering in which the new load status goes into the enum
to avoid ABI breakage.

  • webkit/webkitwebframe.h:
02:48 Changeset [44895] by jmalonzo@webkit.org

2009-06-20 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by NOBODY.

Speculative Qt build fix - add HTMLDataGridColElement to the build script.

  • WebCore.pro:
02:20 Changeset [44894] by abarth@webkit.org

2009-06-19 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

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

Remove a memcpy by retrieving the already existing copy of a string
instead of making a new one.

  • bindings/js/ScriptSourceCode.h: (WebCore::ScriptSourceCode::ScriptSourceCode): (WebCore::ScriptSourceCode::source):
  • bindings/js/StringSourceProvider.h: (WebCore::StringSourceProvider::source):
01:11 Changeset [44893] by xan@webkit.org

2009-06-20 Xan Lopez <xlopez@igalia.com>

Skip new accessibility test, we still lack and accessibilityController.

  • platform/gtk/Skipped:
01:00 Changeset [44892] by xan@webkit.org

2009-06-20 Xan Lopez <xlopez@igalia.com>

We don't seem to support these new HTTP tests either, skip them.

  • platform/gtk/Skipped:
00:53 Changeset [44891] by xan@webkit.org

2009-06-20 Xan Lopez <xlopez@igalia.com>

Skip test that requires eventSender.

  • platform/gtk/Skipped:
Note: See TracTimeline for information about the timeline view.