Timeline


and

06/24/09:

23:56 Changeset [45145] by barraclough@apple.com
  • 8 edits in branches/nitro-extreme/JavaScriptCore

Merged r44520:44523 into nitro-extreme branch.

23:16 Changeset [45144] by darin@chromium.org
  • 14 edits
    4 deletes in trunk/WebCore

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
  • 3 edits
    2 adds in trunk

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
  • 34 edits
    11 copies
    6 deletes in branches/nitro-extreme

Merged r44514:44520 into nitro-extreme branch.

20:34 Changeset [45141] by barraclough@apple.com
  • 8 edits
    2 copies in branches/nitro-extreme/JavaScriptCore

Merged r44513:44514 into nitro-extreme branch.

20:30 Changeset [45140] by barraclough@apple.com
  • 53 edits
    8 copies
    3 deletes in branches/nitro-extreme

Merged r44492:44513 into nitro-extreme branch.

20:18 Changeset [45139] by barraclough@apple.com
  • 4 edits in branches/nitro-extreme/JavaScriptCore

Merged r44491:44492 into nitro-extreme branch.

20:07 Changeset [45138] by weinig@apple.com
  • 3 edits in trunk/JavaScriptCore

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
  • 80 edits
    4 copies
    1 delete in branches/nitro-extreme

Merged r44460:44491 into nitro-extreme branch.

20:00 Changeset [45136] by barraclough@apple.com
  • 12 edits in branches/nitro-extreme

Merged r44455:44460 into nitro-extreme branch.

19:56 Changeset [45135] by barraclough@apple.com
  • 13 edits in branches/nitro-extreme/JavaScriptCore

Merged r44454:44455 into nitro-extreme branch.

19:38 Changeset [45134] by abarth@webkit.org
  • 5 edits
    2 adds in trunk/WebCore

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
  • 4 edits in branches/WWDC-2009-branch

Versioning.

19:04 Changeset [45132] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.19

New tag.

18:52 Changeset [45131] by eric@webkit.org
  • 2 edits in trunk/WebCore

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
  • 5 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/WebCore

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
  • 5 edits
    1 add in trunk

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
  • 4 edits in branches/WWDC-2009-branch

Versioning.

17:25 Changeset [45126] by barraclough@apple.com
  • 64 edits
    11 copies
    1 delete in branches/nitro-extreme

Merged r44437:44454 into nitro-extreme branch.

17:20 Changeset [45125] by barraclough@apple.com
  • 6 edits in branches/nitro-extreme/JavaScriptCore

Merged r44436:44437 into nitro-extreme branch.

16:56 Changeset [45124] by darin@chromium.org
  • 16 edits
    2 copies in trunk/WebCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 49 edits
    11 copies in branches/nitro-extreme

Merged r44412:44436 into nitro-extreme branch.

16:33 Changeset [45118] by barraclough@apple.com
  • 10 edits in branches/nitro-extreme/JavaScriptCore

Merged r44411:44412 into nitro-extreme branch.

16:32 Changeset [45117] by treat@webkit.org
  • 2 edits in trunk/WebCore

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

Fix Qt build.

  • WebCore.pro:
16:23 Changeset [45116] by mrowe@apple.com
  • 5 edits in branches/WWDC-2009-branch/WebCore

Merge r44877.

16:23 Changeset [45115] by mrowe@apple.com
  • 3 edits in branches/WWDC-2009-branch/JavaScriptCore

Merge r44845.

16:23 Changeset [45114] by mrowe@apple.com
  • 11 edits in branches/WWDC-2009-branch/JavaScriptCore

Merge r44844.

16:23 Changeset [45113] by mrowe@apple.com
  • 2 edits in branches/WWDC-2009-branch/LayoutTests

Merge r44262.

16:00 Changeset [45112] by cfleizach@apple.com
  • 2 edits in trunk/WebKitTools

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
  • 3 edits in trunk/WebCore

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
  • 18 edits in trunk/WebCore

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
  • 11 edits
    2 adds in trunk

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
  • 132 edits
    31 copies
    6 deletes in branches/nitro-extreme

Merged r44350:44411 into nitro-extreme branch.

15:04 Changeset [45107] by beidson@apple.com
  • 2 edits in trunk/WebCore

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
  • 6 edits
    3 adds in trunk

<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
  • 29 edits
    3 copies in branches/nitro-extreme

Merged r44340:44350 into nitro-extreme branch.

14:12 Changeset [45104] by mitz@apple.com
  • 1 edit in trunk/WebKit/mac/ChangeLog

Rephrased change log

14:10 Changeset [45103] by eric@webkit.org
  • 8 edits in trunk/WebCore

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
  • 2 edits in trunk/WebKit/mac

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
  • 1286 edits
    98 copies
    17 deletes in branches/nitro-extreme

Merged r44170:44340 into nitro-extreme branch.

13:51 Changeset [45100] by jmalonzo@webkit.org
  • 2 edits in trunk/WebCore

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
  • 242 edits
    60 copies
    9 deletes in branches/nitro-extreme

Merged r44076:44170 into nitro-extreme branch.

13:33 Changeset [45098] by beidson@apple.com
  • 2 edits in trunk/WebCore

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
  • 3 edits in branches/nitro-extreme/JavaScriptCore

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
  • 1 add in trunk/WebCore/bindings/js/JSDataGridColumnListCustom.cpp

Fix Mac build.

12:44 Changeset [45095] by rwlbuis@webkit.org
  • 3 edits
    12 adds in trunk

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
  • 2 edits
    5 adds in trunk/LayoutTests

Add Windows test results for a couple of search field tests

12:22 Changeset [45093] by hyatt@apple.com
  • 16 edits
    8 adds in trunk

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
  • 2 edits in branches/WWDC-2009-branch/WebKit/mac

Merge r44951.

12:13 Changeset [45091] by mrowe@apple.com
  • 4 edits in branches/WWDC-2009-branch

Merge r44941.

12:13 Changeset [45090] by mrowe@apple.com
  • 8 edits
    3 adds in branches/WWDC-2009-branch

Merge r44940.

12:13 Changeset [45089] by mrowe@apple.com
  • 3 edits
    3 adds in branches/WWDC-2009-branch

Merge r44908.

12:13 Changeset [45088] by mrowe@apple.com
  • 6 edits in branches/WWDC-2009-branch

Merge r44906.

10:29 Changeset [45087] by jberlin@apple.com
  • 3 edits
    1 add in trunk/WebCore

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
  • 2 edits
    1 add in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 13 edits in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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

07:34 Changeset [45082] by zimmermann@webkit.org
  • 1 edit
    1 add in trunk/WebCore

Not reviewed. Forgot to include within last patch.

07:30 Changeset [45081] by zimmermann@webkit.org
  • 5 edits in trunk/WebCore

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
  • 3 edits in trunk/WebKit/gtk

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
  • 3 edits in trunk/WebCore

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
  • 1 edit
    1 move in trunk/LayoutTests

Temporarily disable failing test

01:56 Changeset [45077] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits
    1 add in trunk/WebCore

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
  • 2 edits in trunk/WebKit/mac

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/LayoutTests

Attempting to make test more reliable.

00:47 Changeset [45070] by eric@webkit.org
  • 2 edits in trunk/WebCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/WebKitTools

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/WebCore

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
  • 4 edits
    6 adds in trunk

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
  • 1 edit
    1 add in trunk/LayoutTests

Add expected results for test i landed earlier

23:26 Changeset [45062] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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
  • 29 edits in branches/nitro-extreme/JavaScriptCore

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
  • 15 edits
    2 adds in trunk

<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
  • 2 edits in branches/nitro-extreme/JavaScriptCore

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
  • 90 edits
    9 copies
    2 deletes in branches/nitro-extreme

Merged r44030:44075 into nitro-extreme branch.

22:31 Changeset [45057] by ggaren@apple.com
  • 12 edits in branches/nitro-extreme/JavaScriptCore

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
  • 17 edits
    1 copy in branches/nitro-extreme/JavaScriptCore

Merged r44029:44030 into nitro-extreme branch.

21:56 Changeset [45055] by barraclough@apple.com
  • 18 edits
    8 copies in branches/nitro-extreme

Merged r44000:44029 into nitro-extreme branch.

21:41 Changeset [45054] by barraclough@apple.com
  • 192 edits
    51 copies
    6 deletes in branches/nitro-extreme

Merged r43853:44000 into nitro-extreme branch.

21:33 Changeset [45053] by barraclough@apple.com
  • 9 edits
    18 copies in branches/nitro-extreme

Merged r43848:43853 into nitro-extreme branch.

21:26 Changeset [45052] by barraclough@apple.com
  • 203 edits
    84 copies
    47 deletes in branches/nitro-extreme

Merged r43781:43848 into nitro-extreme branch.

21:19 Changeset [45051] by ggaren@apple.com
  • 3 edits in branches/nitro-extreme/JavaScriptCore

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
  • 10 edits in branches/nitro-extreme/JavaScriptCore

Merged r43780:43781 into nitro-extreme branch.

20:13 Changeset [45049] by barraclough@apple.com
  • 72 edits
    8 copies in branches/nitro-extreme

Merged r43750:43780 into nitro-extreme branch.

20:11 Changeset [45048] by mrowe@apple.com
  • 3 edits
    2 adds in branches/WWDC-2009-branch

Merge r44799.

20:11 Changeset [45047] by mrowe@apple.com
  • 14 edits in branches/WWDC-2009-branch/WebCore

Merge r44749.

20:11 Changeset [45046] by mrowe@apple.com
  • 15 edits
    4 adds in branches/WWDC-2009-branch

Merge r44519.

20:11 Changeset [45045] by mrowe@apple.com
  • 13 edits in branches/WWDC-2009-branch/WebCore

Merge r44266.

20:11 Changeset [45044] by mrowe@apple.com
  • 2 edits in branches/WWDC-2009-branch/LayoutTests/http/tests/security/postMessage

Merge r44134.

20:11 Changeset [45043] by mrowe@apple.com
  • 4 edits
    1 add in branches/WWDC-2009-branch/LayoutTests

Merge r44133.

20:11 Changeset [45042] by mrowe@apple.com
  • 1 edit in branches/WWDC-2009-branch/WebCore/ChangeLog

Merge r44125.

20:11 Changeset [45041] by mrowe@apple.com
  • 4 edits
    3 adds in branches/WWDC-2009-branch

Merge r44124.

19:59 Changeset [45040] by barraclough@apple.com
  • 167 edits
    73 copies
    97 deletes in branches/nitro-extreme

Merged r43666:43750 into nitro-extreme branch.

19:47 Changeset [45039] by oliver@apple.com
  • 7 edits
    2 adds in trunk

<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
  • 3 edits in trunk/WebCore

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
  • 85 edits
    13 copies
    1 delete in branches/nitro-extreme

Merged r43660:43666 into nitro-extreme branch.

19:18 Changeset [45036] by barraclough@apple.com
  • 27 edits
    15 copies in branches/nitro-extreme

Merged r43642:43660 into nitro-extreme branch.

19:13 Changeset [45035] by barraclough@apple.com
  • 43 edits in branches/nitro-extreme

Merged r43641:43642 into nitro-extreme branch.

19:08 Changeset [45034] by barraclough@apple.com
  • 47 edits
    6 copies in branches/nitro-extreme

Merged r43619:43641 into nitro-extreme branch.

19:02 Changeset [45033] by barraclough@apple.com
  • 14 edits in branches/nitro-extreme/JavaScriptCore

Merged r43618:43619 into nitro-extreme branch.

18:50 Changeset [45032] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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
  • 2 edits in trunk/WebCore

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
  • 25 edits
    13 copies in branches/nitro-extreme

Merged r43598:43618 into nitro-extreme branch.

18:40 Changeset [45029] by barraclough@apple.com
  • 32 edits
    21 copies
    2 deletes in branches/nitro-extreme

Merged r43583:43598 into nitro-extreme branch.

18:36 Changeset [45028] by barraclough@apple.com
  • 2 edits in branches/nitro-extreme/JavaScriptCore

Merged r43582:43583 into nitro-extreme branch.

18:33 Changeset [45027] by ojan@chromium.org
  • 2 edits in trunk/LayoutTests

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
  • 3 edits in trunk/WebKitTools

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
  • 21 edits
    4 copies in branches/nitro-extreme

Merged r43560:43582 into nitro-extreme branch.

18:14 Changeset [45024] by barraclough@apple.com
  • 8 edits in branches/nitro-extreme

Merged r43559:43560 into nitro-extreme branch.

18:00 Changeset [45023] by mrowe@apple.com
  • 4 edits in branches/WWDC-2009-branch

Versioning.

17:57 Changeset [45022] by mrowe@apple.com
  • 1 edit in branches/WWDC-2009-branch/WebKitLibraries/win/tools/scripts/PRODUCTVERSION

Versioning.

17:44 Changeset [45021] by beidson@apple.com
  • 2 edits in trunk/WebKit/mac

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
  • 11 edits in branches/nitro-extreme/JavaScriptCore

Merged r43558:43559 into nitro-extreme branch.

17:40 Changeset [45019] by mitz@apple.com
  • 2 edits in trunk/WebCore

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
  • 10 edits
    7 adds in trunk

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
  • 8 edits in branches/nitro-extreme/JavaScriptCore

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
  • 20 edits
    2 adds in trunk

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
  • 8 edits
    6 adds in trunk

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
  • 8 edits
    5 adds in trunk

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
  • 9 edits in branches/nitro-extreme/JavaScriptCore

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
  • 2 edits in trunk/WebKitTools

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
  • 2 edits in trunk/JavaScriptCore

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
  • 3 edits in trunk/WebKit/mac

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
  • 3 edits
    2 adds in trunk

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
  • 3 edits in branches/nitro-extreme/JavaScriptCore

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
  • 2 edits in trunk/WebCore

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
  • 6 edits in branches/nitro-extreme/JavaScriptCore

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
  • 6 edits
    4 adds in trunk

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
  • 3 edits in branches/nitro-extreme/JavaScriptCore

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
  • 7 edits in trunk

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
  • 2 edits in trunk/WebCore

Windows build fix

12:46 Changeset [45001] by simon.fraser@apple.com
  • 6 edits in trunk/WebCore

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
  • 4 edits
    2 adds in trunk

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
  • 13 edits in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 5 edits in trunk/WebCore

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
  • 3 edits in trunk/WebCore

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
  • 10 edits in trunk

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
  • 3 edits in trunk/WebCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 3 edits in trunk/LayoutTests

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
  • 11 edits
    2 adds in trunk

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
  • 2 edits in trunk/WebCore

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
  • 3 edits in trunk/WebCore

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
  • 12 edits in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 2 edits
    25 copies
    1 delete in trunk/LayoutTests

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
  • 3 edits in trunk/JavaScriptCore

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
  • 2 edits
    1 copy
    25 deletes in trunk/LayoutTests

Revert 44981. The buildbots hate me.

02:01 Changeset [44982] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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
  • 2 edits
    25 adds
    1 delete in trunk/LayoutTests

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
  • 2 edits in trunk/WebKitTools

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
  • 1 edit
    5 adds in trunk/WebKitTools

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
  • 2 edits
    22 deletes in trunk/LayoutTests

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

00:11 Changeset [44977] by abarth@webkit.org
  • 2 edits
    22 adds in trunk/LayoutTests

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
  • 2 edits in branches/nitro-extreme/JavaScriptCore

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
  • 5 edits in trunk/LayoutTests

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
  • 6 edits in trunk

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
  • 2 edits in trunk/WebCore

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
  • 3 edits
    1 add in trunk/WebCore

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
  • 8 edits in branches/nitro-extreme/JavaScriptCore

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
  • 2 edits in trunk/WebKitTools

Roll out r44967 as that build slave hates freedom.

20:05 Changeset [44969] by weinig@apple.com
  • 8 edits in trunk

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
  • 5 edits in trunk

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
  • 2 edits in trunk/WebKitTools

Update build.webkit.org config for new machine.

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

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
  • 2 edits in trunk/WebCore

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
  • 10 edits
    2 adds in trunk/WebCore

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
  • 2 edits in trunk/WebKitTools

Remove errant line of code mistakenly checked in.

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

Pass correct value to setShouldPaintNativeControls.


Rubber stamped by Mark Rowe.

  • DumpRenderTree/win/DumpRenderTree.cpp: (main):
15:45 Changeset [44961] by simon.fraser@apple.com
  • 7 edits
    2 adds in trunk

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
  • 5 edits in branches/nitro-extreme

Merged r43552:43558 into nitro-extreme branch.

14:58 Changeset [44959] by darin@chromium.org
  • 15 edits
    2 copies in trunk/WebCore

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
  • 2 edits in branches/nitro-extreme

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

14:43 Changeset [44957] by barraclough@apple.com
  • 5 edits
    7 copies in branches/nitro-extreme

Merged r43550:43552 into nitro-extreme branch.

14:38 Changeset [44956] by darin@chromium.org
  • 3 edits in trunk/WebCore

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
  • 2 edits in trunk/WebKit/mac

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
  • 2 edits
    1 add in trunk/WebCore

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
  • 2 edits
    1 add in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 2 edits in trunk/WebKit/mac

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
  • 3 edits in trunk/LayoutTests

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
  • 3 edits in branches/nitro-extreme/WebCore

Merged r43544:43550 into nitro-extreme branch.

11:54 Changeset [44948] by jshin@chromium.org
  • 2 edits
    1 add in trunk/WebCore

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
  • 5 edits in trunk/WebCore

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
  • 17 edits
    3 adds in trunk/WebCore

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
  • 2 edits in trunk/WebKitTools

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
  • 13 edits in trunk

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
  • 24 edits
    7 deletes in trunk/WebCore

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
  • 3 edits in trunk/WebCore

Windows build fix.

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

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
  • 8 edits
    3 adds in trunk

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
  • 3 edits in trunk/WebKit/mac

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
  • 3 edits
    4 adds in trunk

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
  • 15 edits
    3 adds in trunk/WebCore

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
  • 2 edits in trunk/LayoutTests

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

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

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
  • 3 edits
    1 add in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 2 edits in trunk/WebKitTools

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
  • 6 edits in trunk

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
  • 4 edits in trunk/WebCore

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
  • 3 edits
    3 adds in trunk

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
  • 7 edits
    3 adds in trunk

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
  • 4 edits
    3 adds in trunk

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
  • 2 edits in trunk/LayoutTests

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
  • 2 edits in trunk/WebKitTools

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
  • 2 edits in trunk/JavaScriptCore

Fix build by removing dead code.

16:02 Changeset [44923] by oliver@apple.com
  • 7 edits
    3 adds in trunk

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/WebCore

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
  • 2 edits
    2 adds in trunk/WebKitTools

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
  • 2 edits in trunk/WebCore

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
  • 3 edits in trunk/WebCore

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
  • 2 edits in trunk/WebCore

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
  • 9 edits
    2 adds in trunk

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
  • 23 edits
    1 move
    7 adds
    2 deletes in trunk

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
  • 3 edits
    2 adds in trunk

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
  • 3 edits
    2 adds in trunk

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
  • 3 edits
    2 adds in trunk

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
  • 2 edits in trunk/LayoutTests

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
  • 2 edits in trunk/LayoutTests

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

  • platform/gtk/Skipped: Skip some new tests.
23:15 Changeset [44909] by darin@apple.com
  • 2 edits in trunk/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.
22:42 Changeset [44908] by alice.liu@apple.com
  • 3 edits
    3 adds in trunk

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
  • 2 edits in trunk/WebKit/win

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

19:05 Changeset [44906] by weinig@apple.com
  • 6 edits in trunk

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
  • 4 edits in trunk/WebKit

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
  • 6 edits in trunk

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
  • 3 edits in trunk/WebKit/gtk

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
  • 3 edits
    2 adds in trunk

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
  • 5 edits in trunk

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
  • 2 edits in trunk/JavaScriptCore

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
  • 2 edits in trunk/JavaScriptCore

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
  • 3 edits
    1 add in trunk

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
  • 8 edits
    1 move in trunk/JavaScriptCore

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
  • 2 edits in trunk/WebKit/gtk

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
  • 2 edits in trunk/WebCore

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
  • 3 edits in trunk/WebCore

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
  • 2 edits in trunk/LayoutTests

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
  • 2 edits in trunk/LayoutTests

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
  • 2 edits in trunk/LayoutTests

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.