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

Timeline



Feb 25, 2010:

11:40 PM Changeset in webkit [55270] by mrowe@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7688455> REGRESSION(r55205): Crash inside dispatchEventsOnWindowAndFocusedNode when clicking link from bookmarks view

Reviewed by Brady Eidson.

Null-check the document's page to avoid dereferencing a null page.

No regression test is added as this appears to be dependent on an interaction with Safari's bookmarks view that is
not testable from within DumpRenderTree.

  • page/FocusController.cpp:

(WebCore::dispatchEventsOnWindowAndFocusedNode):

10:49 PM Changeset in webkit [55269] by darin@chromium.org
  • 4 edits in trunk/WebKit/chromium

2010-02-25 Darin Fisher <​darin@chromium.org>

Reviewed by David Levin.

Remove deprecated cookie methods.
​https://bugs.webkit.org/show_bug.cgi?id=35420

  • public/WebFrameClient.h:
  • public/WebKitClient.h:
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::setCookies): (WebCore::ChromiumBridge::cookies): (WebCore::ChromiumBridge::cookieRequestHeaderFieldValue): (WebCore::ChromiumBridge::rawCookies): (WebCore::ChromiumBridge::deleteCookie): (WebCore::ChromiumBridge::cookiesEnabled):
8:56 PM QtWebKitMediaElementSupport edited by nicholas.young@nokia.com
Added information about QtMultimedia Integration (diff)
5:32 PM Changeset in webkit [55268] by dpranke@chromium.org
  • 4 edits
    1 add in trunk/WebKitTools

2010-02-25 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Eric Seidel.

Add a 'passing' port implementation to new-run-webkit-tests that
acts as a wrapper around an existing implementation but stubs out
the actual test invocations (instead, the expected results are echoed
back to the harness). This is useful for coverage and perf testing
of the harness (especially perf testing as it essentially provides
a lower bound on how fast the harness can run).

Also added a --nostart-helper flag to new-run-webkit-tests so that
you can skip starting the layout_test_helper and actually run the
harness even if you don't have a build of that port.

Also fix a bug in the 'test' port implementation to actually
create the results directory under /tmp instead of /.

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

  • Scripts/webkitpy/layout_tests/port/factory.py: Modified.
  • Scripts/webkitpy/layout_tests/port/passing.py: Added.
  • Scripts/webkitpy/layout_tests/port/test.py: Added.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py: Modified.
5:14 PM Changeset in webkit [55267] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-02-25 Eric Seidel <​eric@webkit.org>

Fix typo in my last change. No review.

Rename run-chromium-webkit-tests to new-run-webkit-tests to stop confusion
​https://bugs.webkit.org/show_bug.cgi?id=35408

  • Scripts/new-run-webkit-tests:
5:06 PM Changeset in webkit [55266] by oliver@apple.com
  • 3 edits in trunk/WebCore

2010-02-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Simon Fraser.

Multiple repaints on apple.com
​https://bugs.webkit.org/show_bug.cgi?id=35409

apple.com was triggering the fast scaling path for background images due to
repeated repaints as more content came in. This occured due to a two problems
in the logic to detect scaling. The first is that the main context is flipped
on mac so fails the identity or translation check. We work around this by adding
an function that allows the scaling for a flipped CTM. The other problem was that
we were looking at the destination rect size instead of the destination tile size
when deciding if the size we were drawn at would cause scaling.

  • platform/graphics/transforms/AffineTransform.h: (WebCore::AffineTransform::isIdentityOrTranslationOrFlipped):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelScaleObserver::shouldPaintBackgroundAtLowQuality): (WebCore::RenderBoxModelObject::paintFillLayerExtended):
4:55 PM Changeset in webkit [55265] by dpranke@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-02-25 Dirk Pranke <​dpranke@chromium.org>

Reviewed by Eric Seidel.

Test whether or not we preserve the hash fragments for "about:blank"
URLs. (Note that currently WebKit passes this but Chromium doesn't).

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

  • fast/loader/about-blank-hash-kept-expected.txt: Added.
  • fast/loader/about-blank-hash-kept.html: Added.
4:49 PM Changeset in webkit [55264] by eric@webkit.org
  • 1 edit
    2 moves in trunk/WebKitTools

2010-02-25 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Rename run-chromium-webkit-tests to new-run-webkit-tests to stop confusion
​https://bugs.webkit.org/show_bug.cgi?id=35408

  • Scripts/new-run-webkit-tests: Added.
  • Scripts/run-chromium-webkit-tests: Removed.
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py: Renamed from WebKitTools/Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py.
4:43 PM Changeset in webkit [55263] by enrica@apple.com
  • 20 edits
    2 adds
    6 deletes in trunk

Extra layout on keypress after a space (problem with rebalanceWhitespaceAt in InsertTextCommand).
​https://bugs.webkit.org/show_bug.cgi?id=30946
<rdar://problem/7639184>

Reviewed by Adele Peterson.

WebCore:

Do text insertion and whitespace rebalancing around the insertion in one step so that there's only one layout.
This patch was originally made by Justin Garcia.

Test: editing/inserting/rebalance-whitespace-1.html

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::rebalanceWhitespaceAt): No behavior changes here,
just pushed the code that determined the extent of whitespace surrounding a position
to its own helper function.

  • editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::insertTextIntoNodeAndRebalanceWhitespace): Added. Find
whitespace surrounding the insertion position, add the text to insert, rebalance that entire
string, then insert it into the document.
(WebCore::InsertTextCommand::input):

  • editing/InsertTextCommand.h:
  • editing/htmlediting.cpp:

(WebCore::isWhitespace): Moved from CompositeEditCommand.cpp.
(WebCore::extentOfWhitespaceForRebalancingAt): Moved code from rebalanceWhitespaceAt into this helper
function. Obtains the offset of the start and end of whitespace around a particular position.

  • editing/htmlediting.h:

LayoutTests:

There are now regular spaces where nbsps were used unnecessarily before. Also,
multi-character insertions now have their whitespace rebalanced, and this is
reflected in a few tests.

  • editing/execCommand/5142012-3-expected.txt:
  • editing/inserting/rebalance-whitespace-1-expected.txt: Added.
  • editing/inserting/rebalance-whitespace-1.html: Added.
  • editing/pasteboard/5521237-expected.txt:
  • platform/mac/editing/execCommand/5482023-expected.checksum:
  • platform/mac/editing/execCommand/5482023-expected.png: Removed.
  • platform/mac/editing/execCommand/5482023-expected.txt:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-2-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-formatting-expected.checksum:
  • platform/mac/editing/execCommand/remove-formatting-expected.png: Removed.
  • platform/mac/editing/execCommand/remove-formatting-expected.txt:
  • platform/mac/editing/inserting/editable-html-element-expected.checksum:
  • platform/mac/editing/inserting/editable-html-element-expected.png: Removed.
  • platform/mac/editing/inserting/editable-html-element-expected.txt:
  • platform/mac/editing/pasteboard/4989774-expected.checksum:
  • platform/mac/editing/pasteboard/4989774-expected.png: Removed.
  • platform/mac/editing/pasteboard/4989774-expected.txt:
  • platform/mac/editing/selection/4983858-expected.checksum:
  • platform/mac/editing/selection/4983858-expected.png: Removed.
  • platform/mac/editing/selection/4983858-expected.txt:
4:22 PM Changeset in webkit [55262] by ap@apple.com
  • 10 edits
    2 adds in trunk

Reviewed by Anders Carlsson.

​https://bugs.webkit.org/show_bug.cgi?id=35406
<rdar://problem/6945502> Make generic array methods work with JavaArray

Test: java/array-sort.html

Made RuntimeArray inherit from JSArray, keeping the promise given in ClassInfo.

  • bridge/runtime_array.cpp: (JSC::RuntimeArray::RuntimeArray): (JSC::RuntimeArray::~RuntimeArray):
  • bridge/runtime_array.h: (JSC::RuntimeArray::classInfo): (JSC::RuntimeArray::getConcreteArray):
3:21 PM Changeset in webkit [55261] by mitz@apple.com
  • 2 edits in trunk/WebCore

Fixed a regression introduced in r44243, which made the assertion in checkListItems()
ineffective.

Reviewed by Alexey Proskuryakov.

  • dom/SelectElement.cpp:

(WebCore::SelectElementData::checkListItems): Copy the items vector before recomputing it,
then assert that the newly-computed vector is equal to the copy.

3:04 PM Changeset in webkit [55260] by krit@webkit.org
  • 8 edits
    4 adds in trunk

2010-02-25 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG's tspan is no member of SVGRenderBase, this can cause crashes on filters
​https://bugs.webkit.org/show_bug.cgi?id=35354

This makes the base class RenderSVGInline of RenderSVGTSpan and RenderSVGInlineText
dependent on SVGBaseRenderer.
The SVG spec want as to use the object bounding box of the text root as the bounding box
for text-childs. So we search for the text root and use it's bounding box, stroke rect and
repaint rect for further calculations.

Test: svg/filters/filter-on-tspan.svg

  • rendering/RenderSVGInline.cpp: (WebCore::RenderSVGInline::objectBoundingBox): Find the text root and give back it's bounding box. (WebCore::RenderSVGInline::strokeBoundingBox): same for stroke rect (WebCore::RenderSVGInline::repaintRectInLocalCoordinates): same for repaint rect
  • rendering/RenderSVGInline.h: (WebCore::RenderSVGInline::toSVGRenderBase): RenderSVGInline is part of SVGRenderBase now.
  • rendering/RenderSVGTSpan.h: (WebCore::RenderSVGTSpan::renderName): Removed bounding box code. Was just a hack for filters and maskers.
  • rendering/SVGRenderSupport.cpp: (WebCore::findTextRootObject): Search for the text root.
  • rendering/SVGRenderSupport.h:
  • svg/graphics/SVGPaintServerGradient.cpp: moved findTextRootObject to SVGRenderSupport

2010-02-25 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG's tspan is no member of SVGRenderBase, this can cause crashes on filters
​https://bugs.webkit.org/show_bug.cgi?id=35354

Check if filters work for tspan according to the SVG Spec. The test shouldn't
throw an assert now and a green rect should be visible.

  • platform/mac/svg/filters/filter-on-tspan-expected.checksum: Added.
  • platform/mac/svg/filters/filter-on-tspan-expected.png: Added.
  • platform/mac/svg/filters/filter-on-tspan-expected.txt: Added.
  • svg/filters/filter-on-tspan.svg: Added.
3:02 PM Changeset in webkit [55259] by krit@webkit.org
  • 4 edits
    4 adds in trunk

2010-02-25 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Use-element doesn't transform clipPath
​https://bugs.webkit.org/show_bug.cgi?id=35375

If the use element is a child of clipPath, any settings on x, y or transform does not
transform the clipPath it references.
The use-element just misses this transformation in toClipPath.

Test: svg/custom/use-on-clip-path-with-transformation.svg

  • platform/graphics/cairo/PathCairo.cpp: Gtk port translated the path to the wrong direction. (WebCore::Path::translate):
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::toClipPath):

2010-02-25 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

Use-element doesn't transform clipPath
​https://bugs.webkit.org/show_bug.cgi?id=35375

Check if the clipPath gets transformed on setting a transform or translation for use.

  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.checksum: Added.
  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.png: Added.
  • platform/mac/svg/custom/use-on-clip-path-with-transformation-expected.txt: Added.
  • svg/custom/use-on-clip-path-with-transformation.svg: Added.
2:52 PM Changeset in webkit [55258] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

2010-02-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoff Garen.

JSC crashes like crazy in the JSPropertyNameIterator destructor

Add back null check of m_cachedStructure. Curse last minute changes.

  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::~JSPropertyNameIterator):
2:30 PM Changeset in webkit [55257] by jianli@chromium.org
  • 7 edits in trunk

File.type support.
​https://bugs.webkit.org/show_bug.cgi?id=35361

Reviewed by Dmitry Titov.

WebCore:

Test: LayoutTests/editing/pasteboard/file-input-files-access.html

  • html/File.cpp:

(WebCore::File::File):

  • html/File.h:

(WebCore::File::type):

  • html/File.idl:

LayoutTests:

Update the layout test to test File.type.

  • editing/pasteboard/file-input-files-access-expected.txt:
  • editing/pasteboard/script-tests/file-input-files-access.js:

(fileListShouldBe):
(runTest):

2:15 PM Changeset in webkit [55256] by oliver@apple.com
  • 7 edits in trunk

2010-02-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Race condition in JSPropertyNameIterator and Structure destruction
​https://bugs.webkit.org/show_bug.cgi?id=35398

JSPropertyNameIterator and Structure have a cyclic dependency that they
manage by clearing the appropriate reference in each other during their
destruction. However if the Structure is destroyed while the
JSPropertyNameIterator is dead but not yet finalized the Structures
WeakGCPtr will return null, and so prevent Structure from clearing
the m_cachedStructure pointer of the iterator. When the iterator is
then finalised the m_cachedStructure is invalid, and the attempt to
clear the structures back reference fails.

To fix this we simply make JSPropertyNameIterator keep the Structure
alive, using the weak pointer to break the ref cycle.

  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::~JSPropertyNameIterator): The iterator now keeps m_cachedStructure alive itself, so no longer needs to check for it being cleared
  • runtime/JSPropertyNameIterator.h: (JSC::JSPropertyNameIterator::setCachedStructure): Add an assertion to ensure correct usage (JSC::JSPropertyNameIterator::cachedStructure): Add .get()
  • runtime/Structure.cpp: (JSC::Structure::~Structure): Add an assertion that our iterator isn't already dead, and remove the now unnecessary attempt to clear the ref in the iterator
  • runtime/WeakGCPtr.h: (JSC::WeakGCPtr::hasDeadObject): An assert-only function to allow us to assert correct behaviour in the Structure destructor

2010-02-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Race condition in JSPropertyNameIterator and Structure destruction
​https://bugs.webkit.org/show_bug.cgi?id=35398

Add test to ensure that this race condition doesn't occur.

  • fast/js/script-tests/for-in-cached.js: (cacheClearing):
2:14 PM Changeset in webkit [55255] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-25 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Javascript console should not attempt to scroll to end of console if console is not visible

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._scheduleScrollIntoView.scrollIntoView): (WebInspector.ConsoleView.prototype._scheduleScrollIntoView): (WebInspector.ConsoleView.prototype.addMessage):
1:56 PM Changeset in webkit [55254] by eric@webkit.org
  • 2 edits
    2 adds in trunk/WebKit/chromium

2010-02-25 James Hawkins <​jhawkins@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Implement WebLabelElement.

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

  • WebKit.gyp:
  • public/WebLabelElement.h: Added.
  • src/WebLabelElement.cpp: Added.
1:04 PM Changeset in webkit [55253] by ap@apple.com
  • 3 edits in trunk/WebCore

Qt and Gtk build fixes.

  • GNUmakefile.am:
  • WebCore.pro: Added CRuntimeObject.{c,cpp}.
1:00 PM Changeset in webkit [55252] by ap@apple.com
  • 2 edits in trunk/WebCore

Windows build fix.

  • WebCore.vcproj/WebCore.vcproj: Added CRuntimeObject.{c,cpp}.
12:56 PM Changeset in webkit [55251] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Tiger build fix.

  • java/Makefile: I can't get $(or ...) work on Tiger, so changed to use a single condition.
12:23 PM Changeset in webkit [55250] by ap@apple.com
  • 33 edits
    10 adds in trunk

Reviewed by Geoffrey Garen.

​https://bugs.webkit.org/show_bug.cgi?id=35394
<rdar://problem/7685262> Make passing objects between Java and plug-ins work

  • WebCore.PluginHostProcess.exp: WebKit now subclasses RuntimeObject, so it needed more exports.
  • bridge/runtime_object.cpp: (JSC::Bindings::callRuntimeObject): (JSC::Bindings::callRuntimeConstructor): Assert that a runtime object is passed as function.
  • bridge/runtime_object.h: Moved RuntimeObject into Bindings namespace, matching other related classes.
  • bridge/jni/jni_jsobject.mm: (JavaJSObject::toString): Pass rootObject to convertValueToJValue(). It's not needed when constructing a string, but this function now takes it for the sake of Object. (JavaJSObject::convertValueToJObject): Check that object class is JavaRuntimeObject, not just RuntimeObject.
  • bridge/jni/jsc/JNIBridgeJSC.cpp: (JavaField::setValueToInstance): Pass rootObject to convertValueToJValue(). (JavaArray::setValueAt): Ditto.
  • bridge/jni/jsc/JNIUtilityPrivate.h: convertValueToJValue() now takes a RootObject argument, because one is needed to gcProtect an object ghtat is wrapped into JSObject.
  • bridge/jni/jsc/JNIUtilityPrivate.cpp: (JSC::Bindings::convertValueToJValue): Convert JavaScript objects to Java JSObject ones. This was already happening in other code paths, which we should change to use common code.
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::newRuntimeObject): Create an appropriate RuntimeObject subclass, which is JavaRuntimeObject for Java. (JavaInstance::invokeMethod): Unwrap returned JavaObjects that contain JS objects.
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/objc/WebScriptObject.mm: (+[WebScriptObject _convertValueToObjcValue:originRootObject:rootObject:]):
  • bridge/c/CRuntimeObject.cpp: Added. (JSC::Bindings::CRuntimeObject::CRuntimeObject): (JSC::Bindings::CRuntimeObject::~CRuntimeObject): (JSC::Bindings::CRuntimeObject::getInternalCInstance):
  • bridge/c/CRuntimeObject.h: Added. (JSC::Bindings::CRuntimeObject::classInfo):
  • bridge/c/c_instance.cpp: (JSC::Bindings::CInstance::newRuntimeObject):
  • bridge/c/c_instance.h:
  • bridge/c/c_utility.cpp: (JSC::Bindings::convertValueToNPVariant):
  • bridge/jni/jsc/JavaInstanceJSC.h:
  • bridge/jni/jsc/JavaRuntimeObject.cpp: Added. (JSC::Bindings::): (JSC::Bindings::JavaRuntimeObject::JavaRuntimeObject): (JSC::Bindings::JavaRuntimeObject::~JavaRuntimeObject): (JSC::Bindings::JavaRuntimeObject::getInternalJavaInstance):
  • bridge/jni/jsc/JavaRuntimeObject.h: Added. (JSC::Bindings::JavaRuntimeObject::classInfo):
  • bridge/jsc/BridgeJSC.h:
  • bridge/objc/ObjCRuntimeObject.h: Added. (JSC::Bindings::ObjCRuntimeObject::classInfo):
  • bridge/objc/ObjCRuntimeObject.mm: Added. (JSC::Bindings::): (JSC::Bindings::ObjCRuntimeObject::ObjCRuntimeObject): (JSC::Bindings::ObjCRuntimeObject::~ObjCRuntimeObject): (JSC::Bindings::ObjCRuntimeObject::getInternalObjCInstance):
  • bridge/objc/objc_instance.h:
  • bridge/objc/objc_instance.mm: (ObjcInstance::newRuntimeObject):
  • bridge/objc/objc_runtime.mm: (JSC::Bindings::callObjCFallbackObject):
  • bridge/runtime_root.h: Added RuntimeObject subclasses for each instance type, and use them for type casting.
10:57 AM Changeset in webkit [55249] by kov@webkit.org
  • 20 edits in trunk

2010-02-25 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

Adopt the same string as the one that is expected by
fast/parser/fragment-parser.html for the description used by the
<isindex> element.

  • platform/gtk/LocalizedStringsGtk.cpp: (WebCore::searchableIndexIntroduction):
10:46 AM Changeset in webkit [55248] by pfeldman@chromium.org
  • 12 edits in trunk

2010-02-25 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: evaluate-on-hover does not work on HTML files.

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

  • inspector/front-end/SourceCSSTokenizer.re2js:
  • inspector/front-end/SourceHTMLTokenizer.js: (WebInspector.SourceHTMLTokenizer): (WebInspector.SourceHTMLTokenizer.prototype.set line): (WebInspector.SourceHTMLTokenizer.prototype._isExpectingAttribute): (WebInspector.SourceHTMLTokenizer.prototype._isExpectingAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._setExpectingAttribute): (WebInspector.SourceHTMLTokenizer.prototype._setExpectingAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._attrValueTokenType): (WebInspector.SourceHTMLTokenizer.prototype.nextToken):
  • inspector/front-end/SourceHTMLTokenizer.re2js:
  • inspector/front-end/SourceJavaScriptTokenizer.js: (WebInspector.SourceJavaScriptTokenizer):
  • inspector/front-end/SourceJavaScriptTokenizer.re2js:
  • inspector/front-end/SourceTokenizer.js: (WebInspector.SourceTokenizer.prototype.set condition): (WebInspector.SourceTokenizer.prototype.get condition): (WebInspector.SourceTokenizer.prototype.get subTokenizer): (WebInspector.SourceTokenizer.prototype.getLexCondition): (WebInspector.SourceTokenizer.prototype.setLexCondition):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter): (WebInspector.TextEditorHighlighter.prototype.set mimeType): (WebInspector.TextEditorHighlighter.prototype.highlight): (WebInspector.TextEditorHighlighter.prototype._highlightInChunks): (WebInspector.TextEditorHighlighter.prototype._highlightLines): (WebInspector.TextEditorHighlighter.prototype._highlightLine):
  • inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype._paintLine):
10:27 AM Changeset in webkit [55247] by jorlow@chromium.org
  • 5 edits in trunk/JavaScriptCore

Make the context that was passed to the ThreadFunction accessible.
​https://bugs.webkit.org/show_bug.cgi?id=35379

Patch by Jochen Eisinger <​jochen@chromium.org> on 2010-02-25
Reviewed by Jeremy Orlow.

When a database is opened, right now you
don't have any context from where it is opened. The problem is that
the actual calls that open a database go through the sqlite3 vfs
layer, so there's no easy way to pass this function down to to
platform/sql/chromium/SQLFileSystemChromium*.cpp

This patch will allow you to get from anywhere within webkit a pointer
to the Thread object that actually created the thread you're currently
on (in case of the database, this can be either a thread forked of
from the main thread or from a worker thread), and query the object
for context information.

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp:

(WTF::threadContext):

  • wtf/ThreadingPthreads.cpp:

(WTF::):
(WTF::identifierByPthreadHandle):
(WTF::establishIdentifierForPthreadHandle):
(WTF::pthreadHandleForIdentifier):
(WTF::contextForIdentifier):
(WTF::createThreadInternal):
(WTF::currentThread):
(WTF::threadContext):

  • wtf/ThreadingWin.cpp:

(WTF::):
(WTF::threadMap):
(WTF::storeThreadHandleByIdentifier):
(WTF::threadHandleForIdentifier):
(WTF::contextForIdentifier):
(WTF::createThreadInternal):
(WTF::threadContext):

10:20 AM Changeset in webkit [55246] by jorlow@chromium.org
  • 5 edits in trunk/JavaScriptCore

Reverting to re-submit with better change log.

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp:

(WTF::isMainThread):

  • wtf/ThreadingPthreads.cpp:

(WTF::identifierByPthreadHandle):
(WTF::establishIdentifierForPthreadHandle):
(WTF::pthreadHandleForIdentifier):
(WTF::createThreadInternal):
(WTF::currentThread):

  • wtf/ThreadingWin.cpp:

(WTF::threadMap):
(WTF::storeThreadHandleByIdentifier):
(WTF::threadHandleForIdentifier):
(WTF::createThreadInternal):

10:18 AM Changeset in webkit [55245] by kov@webkit.org
  • 3 edits in trunk/WebKit/gtk

2010-02-25 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[Gtk] crashed when destroying
​https://bugs.webkit.org/show_bug.cgi?id=31271

NULL-check the page before relaying the focus out event, since
this might happen when destroying the widget without destroying
its parent, and we currently crash.

  • tests/testwebview.c: (delayed_destroy): (test_webkit_web_view_destroy): (main):
  • webkit/webkitwebview.cpp: (webkit_web_view_focus_in_event):
10:16 AM Changeset in webkit [55244] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-02-25 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Simple style cleanups.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::plugRemovedCallback): (WebCore::plugAddedCallback): (WebCore::PluginView::platformStart):
10:16 AM Changeset in webkit [55243] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2010-02-25 Adam Barth <​abarth@webkit.org>

Reviewed by David Levin.

EWS leaks memory slowly
​https://bugs.webkit.org/show_bug.cgi?id=35395

The EWS bots leak memory very slowly. If you run them for about a
month, each one will take up around 1 GB of virutal memory. If you run
several of them on one machine, you'll eventually exhaust all available
memory and grind the bots to a halt.

This patch introduces a --exit-after-iteration option to the queues so
that we run them for a finite amount of time. Once they exit and
restart, they'll reclaim the leaked memory. I'm not sure how many
iterations I'll end up running them for. I'll need to sort that out
operationally, but my initial guess is around 1000.

  • Scripts/webkitpy/commands/queues.py:
  • Scripts/webkitpy/commands/queues_unittest.py:
10:13 AM Changeset in webkit [55242] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-25 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Darin Adler.

Optimize decoding of Latin-1 text by exploiting the fact that most of it will
be ASCII-only data.

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

  • platform/text/TextCodecLatin1.cpp: (WebCore::TextCodecLatin1::decode):
9:55 AM Changeset in webkit [55241] by pfeldman@chromium.org
  • 6 edits in trunk

2010-02-25 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Dimitri Glazkov.

Web Inspector: make script lines count calculation lazy.

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

  • inspector/front-end/Script.js: (WebInspector.Script): (WebInspector.Script.prototype.get linesCount):
  • inspector/front-end/ScriptsPanel.js:
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):
9:20 AM Changeset in webkit [55240] by levin@chromium.org
  • 5 edits in trunk/WebCore

Change hardcoded gcc paths to be Solaris friendly
​https://bugs.webkit.org/show_bug.cgi?id=35213

Patch by James Choi <​jchoi42@pha.jhu.edu> on 2010-02-25
Reviewed by David Levin.

  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/IDLParser.pm:
  • css/make-css-file-arrays.pl:
  • dom/make_names.pl:
9:13 AM Changeset in webkit [55239] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-02-24 Darin Fisher <​darin@chromium.org>

Reviewed by David Levin.

Add a missing WebString.h include.
​https://bugs.webkit.org/show_bug.cgi?id=35360

This is required since some of the methods have implementations that
return a WebString.

  • public/WebCookieJar.h:
9:04 AM Changeset in webkit [55238] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-02-25 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Workaround Java plugins not drawing themselves properly on first
expose by doing a late size-allocate after 'plug-added' has been
emitted on the embedding GtkSocket. It's unclear to me if this is
a bug in our side or theirs, but this should be pretty safe and
fixes the annoyance while we investigate it further.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::plugAddedCallback): (WebCore::PluginView::platformStart):
8:05 AM Changeset in webkit [55237] by jocelyn.turcotte@nokia.com
  • 3 edits in trunk/WebCore

Add missing include guards

Reviewed by Tor Arne Vestbø.

  • loader/CrossOriginPreflightResultCache.h:
  • loader/MainResourceLoader.h:
8:04 AM Changeset in webkit [55236] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/WebCore

[Qt] Restrict the optimization flags for Symbian to release builds.

Reviewed by Tor Arne Vestbø.

  • WebCore.pro:
7:06 AM Changeset in webkit [55235] by eric@webkit.org
  • 9 edits
    1 add in trunk

2010-02-25 Jarkko Sakkinen <​jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds GraphicsContext3D QtWebKit implementation.
​https://bugs.webkit.org/show_bug.cgi?id=35153

  • WebCore.pri:
  • WebCore.pro:
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/qt/GraphicsContext3DQt.cpp: Added. (WebCore::GraphicsContext3DInternal::isValid): (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::~GraphicsContext3DInternal): (WebCore::GraphicsContext3DInternal::getProcAddress): (WebCore::GraphicsContext3D::create): (WebCore::GraphicsContext3D::GraphicsContext3D): (WebCore::GraphicsContext3D::~GraphicsContext3D): (WebCore::GraphicsContext3D::platformGraphicsContext3D): (WebCore::GraphicsContext3D::platformTexture): (WebCore::GraphicsContext3D::makeContextCurrent): (WebCore::GraphicsContext3D::beginPaint): (WebCore::GraphicsContext3D::endPaint): (WebCore::GraphicsContext3D::reshape): (WebCore::GraphicsContext3D::activeTexture): (WebCore::GraphicsContext3D::attachShader): (WebCore::GraphicsContext3D::bindAttribLocation): (WebCore::GraphicsContext3D::bindBuffer): (WebCore::GraphicsContext3D::bindFramebuffer): (WebCore::GraphicsContext3D::bindRenderbuffer): (WebCore::GraphicsContext3D::bindTexture): (WebCore::GraphicsContext3D::blendColor): (WebCore::GraphicsContext3D::blendEquation): (WebCore::GraphicsContext3D::blendEquationSeparate): (WebCore::GraphicsContext3D::blendFunc): (WebCore::GraphicsContext3D::blendFuncSeparate): (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData): (WebCore::GraphicsContext3D::checkFramebufferStatus): (WebCore::GraphicsContext3D::clearColor): (WebCore::GraphicsContext3D::clear): (WebCore::GraphicsContext3D::clearDepth): (WebCore::GraphicsContext3D::clearStencil): (WebCore::GraphicsContext3D::colorMask): (WebCore::GraphicsContext3D::compileShader): (WebCore::GraphicsContext3D::copyTexImage2D): (WebCore::GraphicsContext3D::copyTexSubImage2D): (WebCore::GraphicsContext3D::cullFace): (WebCore::GraphicsContext3D::depthFunc): (WebCore::GraphicsContext3D::depthMask): (WebCore::GraphicsContext3D::depthRange): (WebCore::GraphicsContext3D::detachShader): (WebCore::GraphicsContext3D::disable): (WebCore::GraphicsContext3D::disableVertexAttribArray): (WebCore::GraphicsContext3D::drawArrays): (WebCore::GraphicsContext3D::drawElements): (WebCore::GraphicsContext3D::enable): (WebCore::GraphicsContext3D::enableVertexAttribArray): (WebCore::GraphicsContext3D::finish): (WebCore::GraphicsContext3D::flush): (WebCore::GraphicsContext3D::framebufferRenderbuffer): (WebCore::GraphicsContext3D::framebufferTexture2D): (WebCore::GraphicsContext3D::frontFace): (WebCore::GraphicsContext3D::generateMipmap): (WebCore::GraphicsContext3D::getActiveAttrib): (WebCore::GraphicsContext3D::getActiveUniform): (WebCore::GraphicsContext3D::getAttribLocation): (WebCore::GraphicsContext3D::getContextAttributes): (WebCore::GraphicsContext3D::getError): (WebCore::GraphicsContext3D::getString): (WebCore::GraphicsContext3D::hint): (WebCore::GraphicsContext3D::isBuffer): (WebCore::GraphicsContext3D::isEnabled): (WebCore::GraphicsContext3D::isFramebuffer): (WebCore::GraphicsContext3D::isProgram): (WebCore::GraphicsContext3D::isRenderbuffer): (WebCore::GraphicsContext3D::isShader): (WebCore::GraphicsContext3D::isTexture): (WebCore::GraphicsContext3D::lineWidth): (WebCore::GraphicsContext3D::linkProgram): (WebCore::GraphicsContext3D::pixelStorei): (WebCore::GraphicsContext3D::polygonOffset): (WebCore::GraphicsContext3D::readPixels): (WebCore::GraphicsContext3D::releaseShaderCompiler): (WebCore::GraphicsContext3D::renderbufferStorage): (WebCore::GraphicsContext3D::sampleCoverage): (WebCore::GraphicsContext3D::scissor): (WebCore::GraphicsContext3D::shaderSource): (WebCore::GraphicsContext3D::stencilFunc): (WebCore::GraphicsContext3D::stencilFuncSeparate): (WebCore::GraphicsContext3D::stencilMask): (WebCore::GraphicsContext3D::stencilMaskSeparate): (WebCore::GraphicsContext3D::stencilOp): (WebCore::GraphicsContext3D::stencilOpSeparate): (WebCore::GraphicsContext3D::texParameterf): (WebCore::GraphicsContext3D::texParameteri): (WebCore::GraphicsContext3D::uniform1f): (WebCore::GraphicsContext3D::uniform1fv): (WebCore::GraphicsContext3D::uniform2f): (WebCore::GraphicsContext3D::uniform2fv): (WebCore::GraphicsContext3D::uniform3f): (WebCore::GraphicsContext3D::uniform3fv): (WebCore::GraphicsContext3D::uniform4f): (WebCore::GraphicsContext3D::uniform4fv): (WebCore::GraphicsContext3D::uniform1i): (WebCore::GraphicsContext3D::uniform1iv): (WebCore::GraphicsContext3D::uniform2i): (WebCore::GraphicsContext3D::uniform2iv): (WebCore::GraphicsContext3D::uniform3i): (WebCore::GraphicsContext3D::uniform3iv): (WebCore::GraphicsContext3D::uniform4i): (WebCore::GraphicsContext3D::uniform4iv): (WebCore::GraphicsContext3D::uniformMatrix2fv): (WebCore::GraphicsContext3D::uniformMatrix3fv): (WebCore::GraphicsContext3D::uniformMatrix4fv): (WebCore::GraphicsContext3D::useProgram): (WebCore::GraphicsContext3D::validateProgram): (WebCore::GraphicsContext3D::vertexAttrib1f): (WebCore::GraphicsContext3D::vertexAttrib1fv): (WebCore::GraphicsContext3D::vertexAttrib2f): (WebCore::GraphicsContext3D::vertexAttrib2fv): (WebCore::GraphicsContext3D::vertexAttrib3f): (WebCore::GraphicsContext3D::vertexAttrib3fv): (WebCore::GraphicsContext3D::vertexAttrib4f): (WebCore::GraphicsContext3D::vertexAttrib4fv): (WebCore::GraphicsContext3D::vertexAttribPointer): (WebCore::GraphicsContext3D::viewport): (WebCore::GraphicsContext3D::getBooleanv): (WebCore::GraphicsContext3D::getBufferParameteriv): (WebCore::GraphicsContext3D::getFloatv): (WebCore::GraphicsContext3D::getFramebufferAttachmentParameteriv): (WebCore::GraphicsContext3D::getIntegerv): (WebCore::GraphicsContext3D::getProgramiv): (WebCore::GraphicsContext3D::getProgramInfoLog): (WebCore::GraphicsContext3D::getRenderbufferParameteriv): (WebCore::GraphicsContext3D::getShaderiv): (WebCore::GraphicsContext3D::getShaderInfoLog): (WebCore::GraphicsContext3D::getShaderSource): (WebCore::GraphicsContext3D::getTexParameterfv): (WebCore::GraphicsContext3D::getTexParameteriv): (WebCore::GraphicsContext3D::getUniformfv): (WebCore::GraphicsContext3D::getUniformiv): (WebCore::GraphicsContext3D::getUniformLocation): (WebCore::GraphicsContext3D::getVertexAttribfv): (WebCore::GraphicsContext3D::getVertexAttribiv): (WebCore::GraphicsContext3D::getVertexAttribOffset): (WebCore::GraphicsContext3D::texImage2D): (WebCore::GraphicsContext3D::texSubImage2D): (WebCore::GraphicsContext3D::createBuffer): (WebCore::GraphicsContext3D::createFramebuffer): (WebCore::GraphicsContext3D::createProgram): (WebCore::GraphicsContext3D::createRenderbuffer): (WebCore::GraphicsContext3D::createShader): (WebCore::GraphicsContext3D::createTexture): (WebCore::GraphicsContext3D::deleteBuffer): (WebCore::GraphicsContext3D::deleteFramebuffer): (WebCore::GraphicsContext3D::deleteProgram): (WebCore::GraphicsContext3D::deleteRenderbuffer): (WebCore::GraphicsContext3D::deleteShader): (WebCore::GraphicsContext3D::deleteTexture): (WebCore::GraphicsContext3D::sizeInBytes): (WebCore::GraphicsContext3D::synthesizeGLError): (WebCore::GraphicsContext3D::getImageData):

2010-02-25 Jarkko Sakkinen <​jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds enabling and disabling of WebGL support to QWebSettings.
​https://bugs.webkit.org/show_bug.cgi?id=35153

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): (QWebSettings::QWebSettings):
  • Api/qwebsettings.h:

2010-02-25 Jarkko Sakkinen <​jarkko.sakkinen@tieto.com>

Reviewed by Kenneth Rohde Christiansen.

Qt WebGL support

Adds toggling of WebGL support to QtLauncher.
​https://bugs.webkit.org/show_bug.cgi?id=35153

  • QtLauncher/main.cpp: (LauncherWindow::toggleWebGL): (LauncherWindow::setupUI):
6:52 AM BuildingQtOnSymbian edited by jocelyn.turcotte@nokia.com
(diff)
6:50 AM BuildingQtOnSymbian edited by jocelyn.turcotte@nokia.com
(diff)
6:49 AM Changeset in webkit [55234] by eric@webkit.org
  • 5 edits in trunk/JavaScriptCore

2010-02-25 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Jeremy Orlow.

Make the context that was passed to the ThreadFunction accessible.
​https://bugs.webkit.org/show_bug.cgi?id=35379

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp: (WTF::threadContext):
  • wtf/ThreadingPthreads.cpp: (WTF::): (WTF::identifierByPthreadHandle): (WTF::establishIdentifierForPthreadHandle): (WTF::pthreadHandleForIdentifier): (WTF::contextForIdentifier): (WTF::createThreadInternal): (WTF::currentThread): (WTF::threadContext):
  • wtf/ThreadingWin.cpp: (WTF::): (WTF::threadMap): (WTF::storeThreadHandleByIdentifier): (WTF::threadHandleForIdentifier): (WTF::contextForIdentifier): (WTF::createThreadInternal): (WTF::threadContext):
6:34 AM Changeset in webkit [55233] by pfeldman@chromium.org
  • 4 edits
    2 moves in trunk/WebCore

2010-02-25 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Swap images for activate and deactivate breakpoints.

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

  • WebCore.gypi:
  • inspector/front-end/Images/breakpointsActivateButtonGlyph.png: Renamed from WebCore/inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png.
  • inspector/front-end/Images/breakpointsDeactivateButtonGlyph.png: Renamed from WebCore/inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png.
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
6:19 AM Changeset in webkit [55232] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-02-25 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

Go back on symlink resolution if we are loading the flash plugin,
and the path contains /netscape/, following what Chromium does.

Thanks to Evan Martin for the help on this!

  • plugins/gtk/PluginPackageGtk.cpp: (WebCore::PluginPackage::load):
5:45 AM Changeset in webkit [55231] by pfeldman@chromium.org
  • 8 edits in trunk/WebCore

2010-02-23 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Scripts panel shows blank source when stopping on a breakpoint on refresh.

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

  • inspector/front-end/Panel.js: (WebInspector.Panel.prototype.canShowSourceLine): (WebInspector.Panel.prototype.showSourceLine):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.canShowSourceLine): (WebInspector.ResourcesPanel.prototype.showSourceLine):
  • inspector/front-end/Script.js: (WebInspector.Script):
  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.show): (WebInspector.ScriptsPanel.prototype.get searchableViews): (WebInspector.ScriptsPanel.prototype.addScript): (WebInspector.ScriptsPanel.prototype._resourceLoadingFinished): (WebInspector.ScriptsPanel.prototype.addBreakpoint): (WebInspector.ScriptsPanel.prototype.removeBreakpoint): (WebInspector.ScriptsPanel.prototype.reset): (WebInspector.ScriptsPanel.prototype.canShowSourceLine): (WebInspector.ScriptsPanel.prototype.showSourceLine): (WebInspector.ScriptsPanel.prototype._scriptOrResourceForURLAndLine): (WebInspector.ScriptsPanel.prototype.showView): (WebInspector.ScriptsPanel.prototype._sourceFrameForScriptOrResource): (WebInspector.ScriptsPanel.prototype._showScriptOrResource): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu.optionCompare): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):
  • inspector/front-end/inspector.js: (WebInspector.documentClick.followLink): (WebInspector.documentClick): (WebInspector._choosePanelToShowSourceLine): (WebInspector.canShowSourceLine): (WebInspector.showSourceLine):
5:24 AM Changeset in webkit [55230] by eric@webkit.org
  • 10 edits
    3 adds in trunk

2010-02-25 Ben Murdoch <​benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
​https://bugs.webkit.org/show_bug.cgi?id=34585

  • fast/events/touch/basic-single-touch-events-expected.txt: Update expected target element.
  • fast/events/touch/script-tests/basic-single-touch-events.js: ditto.
  • fast/events/touch/script-tests/touch-target.js: Added.
  • fast/events/touch/touch-target-expected.txt: Added.
  • fast/events/touch/touch-target.html: Added.

2010-02-25 Ben Murdoch <​benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
​https://bugs.webkit.org/show_bug.cgi?id=34585

Currently the target of a touch is set to the resulting node of the hit test where the touch currently
is. This does not match the behavior of iPhone or Android. This patch uses a hashmap on the EventHandler
to keep track of the target element when a touch is first started. This target is then used as the target
for subsequent touches with the same id. This matches observed behavior on iPhone and Android.

Tests:
fast/events/touch/touch-target.html: Added.
fast/events/touch/basic-single-touch-events.html: Updated.

  • page/EventHandler.cpp: (WebCore::EventHandler::handleTouchEvent): Store the originating target element of a touch in a hashmap

so that we can reuse that target for future events caused by that touch. This matches observed behavior
on iPhone and Android.

  • page/EventHandler.h: Add hashmap as a member.
  • platform/PlatformTouchPoint.h: (WebCore::PlatformTouchPoint::id): Store the touch point id as unsigned.
  • platform/qt/PlatformTouchPointQt.cpp: (WebCore::PlatformTouchPoint::PlatformTouchPoint): Cast platform touch id from signed to unsigned. Qt API

docs state that it will always be >= 0.

2010-02-25 Ben Murdoch <​benm@google.com>

Reviewed by Kenneth Rohde Christiansen.

The target element of a Touch should be the target where that touch originated, not where it is now.
​https://bugs.webkit.org/show_bug.cgi?id=34585

  • DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::addTouchPoint): Fix a bug where touch points were not being given unique ids.
4:49 AM Changeset in webkit [55229] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-02-25 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Unskip a bunch of (AFAICT) non-flakey HTTP tests that we currently
pass.

  • platform/gtk/Skipped:
3:48 AM Changeset in webkit [55228] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

Add a convenient center() method to IntRect
​https://bugs.webkit.org/show_bug.cgi?id=35346

Reviewed by Simon Fraser.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

As it is a convenience method, and implemention is
based on FloatRect's one, patch does not provide a
layout test.

  • platform/graphics/IntRect.h:

(WebCore::IntRect::center):

3:24 AM Changeset in webkit [55227] by eric@webkit.org
  • 11 edits
    1 add in trunk/WebCore

2010-02-25 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Fake workers implementation that simulates workers using iframe and timer,
needed to support workers debugging. Also, a facility to inject scripts into
inspected page upon load.
​https://bugs.webkit.org/show_bug.cgi?id=35148

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::addScriptToEvaluateOnLoad): (WebCore::InspectorBackend::removeAllScriptsToEvaluateOnLoad):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::addScriptToEvaluateOnLoad): (WebCore::InspectorController::removeAllScriptsToEvaluateOnLoad):
  • inspector/InspectorController.h:
  • inspector/front-end/InjectedFakeWorker.js: Added. (InjectedFakeWorker.Worker): (InjectedFakeWorker.FakeWorker): (InjectedFakeWorker.FakeWorker.prototype.postMessage): (InjectedFakeWorker.FakeWorker.prototype.terminate): (InjectedFakeWorker.FakeWorker.prototype._onmessageWrapper): (InjectedFakeWorker.FakeWorker.prototype._dispatchMessage): (InjectedFakeWorker.FakeWorker.prototype._handleException): (InjectedFakeWorker.FakeWorker.prototype._buildWorker): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext.handler): (InjectedFakeWorker.FakeWorker.prototype._setupWorkerContext): (InjectedFakeWorker.FakeWorker.prototype._importScripts): (InjectedFakeWorker.FakeWorker.prototype._loadScript): (InjectedFakeWorker.URL): (InjectedFakeWorker.URL.prototype.urlRegEx.split): (InjectedFakeWorker.URL.prototype.mockLocation): (InjectedFakeWorker.URL.prototype.completeWith): (InjectedFakeWorker.URL.prototype.sameOrigin): (InjectedFakeWorker.DOMCoreException.formatError): (InjectedFakeWorker.DOMCoreException): (InjectedFakeWorker.noop):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.setInjectedScriptSource): (.WebInspector.InspectorBackendStub.prototype.addScriptToEvaluateOnLoad): (.WebInspector.InspectorBackendStub.prototype.removeAllScriptsToEvaluateOnLoad):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
2:04 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)

Feb 24, 2010:

11:30 PM Changeset in webkit [55226] by eric@webkit.org
  • 5 edits in trunk

2010-02-24 Sam Kerner <​skerner@chromium.org>

Reviewed by Darin Fisher.

Expose WebFrame::setCanHaveScrollbars(). This allows a view
which is being resized to not need scroll bars to ensure that
they are not drawn.

Existing function setAllowsScrolling() was renamed
setCanHaveScrollbars(), to be consistant with change 37159:
​http://trac.webkit.org/changeset/37159

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

  • WebKit/chromium/public/WebFrame.h:
  • WebKit/chromium/src/ChromeClientImpl.cpp:
  • WebKit/chromium/src/WebFrameImpl.cpp:
  • WebKit/chromium/src/WebFrameImpl.h:
11:14 PM Changeset in webkit [55225] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-02-24 Nicholas Young <​nicholas.young@nokia.com>

Reviewed by Eric Carlson.

Add mediaPlayerOwningDocument() to MediaPlayerClient.
​https://bugs.webkit.org/show_bug.cgi?id=35374

No new tests. These are interface changes only.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::mediaPlayerOwningDocument): Implementation for media element.
  • html/HTMLMediaElement.h:
  • platform/graphics/MediaPlayer.h: (WebCore::MediaPlayerClient::mediaPlayerOwningDocument): New virtual method.
10:56 PM Changeset in webkit [55224] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Optimized Font::normalizeSpaces() by caching the String::characters()
instead of repeatedly calling operator[]

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

  • platform/graphics/Font.cpp: (WebCore::Font::normalizeSpaces):
10:32 PM Changeset in webkit [55223] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Jungshik Shin <​jshin@chromium.org>

Reviewed by David Levin.

[Chromium] Traditional Chinese Chrome on Windows should use PMingLiu instead of Simsun for Han characters
​https://bugs.webkit.org/show_bug.cgi?id=35319

No visible change in the layout test.

  • platform/graphics/chromium/FontUtilsChromiumWin.cpp: (WebCore::):
10:28 PM Changeset in webkit [55222] by Martin Robinson
  • 4 edits in trunk/WebKit/gtk

2010-02-24 Krzysztof Kotlenga <​pocek@users.sf.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] Creation of a WebkitWebView widget is very slow
​https://bugs.webkit.org/show_bug.cgi?id=30032

  • WebCoreSupport/EditorClientGtk.cpp: (WebKit::EditorClient::ignoreWordInSpellDocument): Change 'langs' to 'dicts' to reflect the fact that the list just holds Enchant dictionaries now. (WebKit::EditorClient::learnWord): Ditto. (WebKit::EditorClient::checkSpellingOfString): Ditto. (WebKit::EditorClient::getGuessesForWord): Ditto.
  • webkit/webkitprivate.h: Remove the now unused SpellLanguage struct.
  • webkit/webkitwebsettings.cpp: Change function call to reflect new webkit_web_settings_get_enchant_dicts name. (get_enchant_broker): Add this method which returns the enchant broker singleton. (free_spell_checking_language): The list contents have changed, so change the way each element is freed. (webkit_web_settings_finalize): Change to reflect 'spell_checking_languages_list' to 'enchant_dicts' member name change. (webkit_web_settings_set_property): Use the broker singleton here instead of making a new one for each language. The Enchant dictionary is now the list payload. (webkit_web_settings_copy): More name-change updates. (webkit_web_settings_get_enchant_dicts): Ditto.
10:15 PM Changeset in webkit [55221] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Anthony Ricaud <​rik@webkit.org>

Reviewed by Pavel Feldman.

Web Inspector: Reduce the timer to show the eval popover
​https://bugs.webkit.org/show_bug.cgi?id=35344

  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._mouseMove):
9:40 PM Changeset in webkit [55220] by eric@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-02-24 Chris Evans <​cevans@chromium.org>

Reviewed by Darin Adler.

Add test for SVG that used to cause crash.
It has been fixed on trunk for a while, thanks to the <use> rewrite,
but we want to make sure we do not regress.

  • svg/dom/use-transform.svg: Added.
  • svg/dom/use-transform-expected.txt: Added.
9:24 PM Changeset in webkit [55219] by eric@webkit.org
  • 3 edits
    1 add in trunk

2010-02-24 Jay Campan <​jcampan@google.com>

Reviewed by David Levin.

Don't show the autofill popup when the input text is disabled or read only.

Test: manual-tests/chromium/no-autofill-on-readonly.html

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

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill):
  • manual-tests/chromium/no-autofill-on-readonly.html: Added.

2010-02-24 Jay Campan <​jcampan@google.com>

Reviewed by David Levin.

Don't show the autofill popup when the input text is disabled or read only.

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

  • src/EditorClientImpl.cpp: (WebKit::EditorClientImpl::autofill):
9:07 PM Changeset in webkit [55218] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-02-24 Jesus Sanchez-Palencia <​jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] QtLauncher has a border when running on QGraphicsView mode
​https://bugs.webkit.org/show_bug.cgi?id=35352

Fix 2-pixels frame on each border of QtLauncher when running on QGraphicsView mode.

  • QtLauncher/webview.cpp: (WebViewGraphicsBased::WebViewGraphicsBased):
8:43 PM Changeset in webkit [55217] by eric@webkit.org
  • 8 edits in trunk/WebCore

2010-02-24 Dominic Mazzoni <​dmazzoni@chromium.org>

Reviewed by Jeremy Orlow.

Wraps includes of SVG headers so that it's possible to
compile the V8 bindings with SVG disabled, e.g. 'enable_svg=0'.
​https://bugs.webkit.org/show_bug.cgi?id=35345

No new tests.

  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8Proxy.cpp:
  • bindings/v8/custom/V8CSSValueCustom.cpp:
  • bindings/v8/custom/V8DocumentCustom.cpp:
  • bindings/v8/custom/V8ElementCustom.cpp:
  • bindings/v8/custom/V8EventCustom.cpp:
  • bindings/v8/custom/V8SVGElementCustom.cpp:
8:15 PM Changeset in webkit [55216] by mjs@apple.com
  • 3 edits in trunk/WebCore

Unreviewed build fix.

Fix gtk build.

Include JSC headers as runtime/ instead of JavaScriptCore/

  • bindings/js/JSDOMWrapper.h:
  • bindings/js/ScriptWrappable.h:
8:07 PM Changeset in webkit [55215] by mjs@apple.com
  • 12 edits
    2 adds in trunk/WebCore

2010-02-24 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Oliver Hunt.

Cache JavaScript wrappers inline in DOM nodes
​https://bugs.webkit.org/show_bug.cgi?id=35226

<rdar://problem/7664202>


7.4% speedup on Dromaeo DOM Core tests.
2.3% speedup on Hixie DOM Core tests.

This fixes the following things from the last attempt:

  • Now builds in both debug and release and passes all tests
  • Properly use a WeakGCPtr, not just a raw pointer, in ScriptWrappable
  • Properly look in the appropriate per-document or per-world map
  • Added an assert that would have caught any of the problems I found
  • Handle clearing the inline cache properly in all cases


  • bindings/js/JSDOMBinding.cpp: (WebCore::cacheDOMObjectWrapper): Adjust for name changes. (WebCore::forgetDOMNode): Clear wrapper pointer. (WebCore::cacheDOMNodeWrapper): Cache inline too if caching for normal world.
  • bindings/js/JSDOMBinding.h: (WebCore::Document::getWrapperCache): Inlined. (WebCore::domObjectWrapperMapFor): Renamed to start with lowercase. Moved to header to inline.
  • bindings/js/JSDOMWrapper.h: Added. (WebCore::DOMObject::DOMObject): Factored this out of JSDOMBinding.h to avoid include cycle. I think the class should be renamed, I picked a forward-looking header name because we already have a DOMObject.h
  • bindings/js/JSNodeCustom.cpp: (WebCore::createWrapperInline): Renamed version of original createWrapper. (WebCore::createWrapper): Call createWrapperInline. Out-of-line version. (WebCore::toJSNewlyCreated): Call createWrapperInline instead of createWrapper.
  • bindings/js/JSNodeCustom.h: Added. (WebCore::getCachedDOMNodeWrapper): Moved from JSDOMBinding.cpp and moved here, so it could inline everywhere without creating an include cycle. Consider inline cache. (WebCore::toJS): Moved to header to inline.
  • bindings/js/ScriptWrappable.h: (WebCore::ScriptWrappable::ScriptWrappable): Implement this in the obvious way for JavaScriptCore. (Using a WeakGCPtr). (WebCore::ScriptWrappable::wrapper): (WebCore::ScriptWrappable::setWrapper): (WebCore::ScriptWrappable::clearWrapper):
  • bindings/scripts/CodeGeneratorJS.pm: Include CustomHeader heaaders in the header, not just the impl file, so they can add inlining.
  • dom/Node.idl: Add CustomHeader directive.


Add new files to build.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
7:51 PM Changeset in webkit [55214] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-02-24 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Jeremy Orlow.

Expose Database object of currently active task on the database thread
​https://bugs.webkit.org/show_bug.cgi?id=35341

  • storage/DatabaseThread.cpp: (WebCore::DatabaseThread::DatabaseThread): (WebCore::DatabaseThread::databaseThread):
  • storage/DatabaseThread.h: (WebCore::DatabaseThread::getDatabaseOfCurrentTask):
7:34 PM Changeset in webkit [55213] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-02-24 Nicholas Young <​nicholas.young@nokia.com>

Reviewed by Eric Carlson.

supportsMuting() should be an internal interface between MediaPlayer and MediaPlayerPrivate.
​https://bugs.webkit.org/show_bug.cgi?id=35327

No new tests. Refactoring Only.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::setMuted): Do not check supportsMuting() (WebCore::HTMLMediaElement::updateVolume): Set volume and muted
  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::setVolume): Check supportsMuting() (WebCore::MediaPlayer::setMuted): Check supportsMuting()
  • platform/graphics/MediaPlayer.h: Remove supportsMuting()
6:46 PM Changeset in webkit [55212] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Anders Carlsson.

Keep this test’s behavior the same after r55203.

  • fast/dom/script-tests/prototype-inheritance-2.js: Explicitly set the inner document’s body

display property to “none”. Before r55203, this happened as a side effect of parsing the
<frameset> tag.

5:09 PM Changeset in webkit [55211] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

5:07 PM Changeset in webkit [55210] by dpranke@chromium.org
  • 5 edits
    1 copy in trunk/WebKitTools

2010-02-23 Dirk Pranke <​dpranke@chromium.org>

Reviewed by David Levin.

When the run-chromium-webkit-tests code was landed and the code was
refactored into the 'port' package, I accidentally broke using
http_server.py or websocket_server.py as command-line scripts
(the constructors needed a port object they weren't getting). This
change fixes them so that --server start|stop actually works.

As a part of this, the two files need to be able to call port.get(),
but doing that is awkward from a file inside the package, so I moved
get() into factory.py and imported that into init.py so that
http_server.py and websocket_server.py can just import factory.

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

  • Scripts/webkitpy/layout_tests/port/init.py:
  • Scripts/webkitpy/layout_tests/port/factory.py:
  • Scripts/webkitpy/layout_tests/port/http_server.py:
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
5:06 PM Changeset in webkit [55209] by mrowe@apple.com
  • 1 copy in tags/Safari-533.1

New tag.

4:58 PM Changeset in webkit [55208] by dpranke@chromium.org
  • 4 edits in trunk/WebKitTools

2010-02-24 Dirk Pranke <​dpranke@chromium.org>

Reviewed by David Levin.

Fix the function signature for check_sys_deps on the mac port, and
fix the ordering of port_obj.check_sys_deps() and
port_obj.start_helper() (helper needs to be started before we check
the system configuration).

​http://bugs.webkit.org/show_bug.cgi?id=35367

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py:
4:24 PM Changeset in webkit [55207] by pkasting@chromium.org
  • 31 edits in trunk

[Chromium API] Disambiguate allowJavaScript from didNotAllowScript
​https://bugs.webkit.org/show_bug.cgi?id=35205

Patch by Adam Barth <​abarth@webkit.org> on 2010-02-24
Reviewed by Darin Fisher.

WebCore:

For clients that want to show a user interface element when JavaScript
was blocked on a page, we need to disambiguate between querying the
client for whether JavaScript is enabled from actually failing to
execute some script.

This patch adds a new FrameLoaderClient callback for when WebCore would
like to execute JavaScript but fails to because JavaScript is disabled.

This patch also touches every client of canExecuteScripts so they can
indicate whether we should make this callback. I was hoping there was
a better choke point, but my first two attempts were wrong in subtle
ways. pkasting points out that this will be easy to screw up in the
future, so it's better to make all the clients be explicit.

  • WebCore.PluginHostProcess.exp:
  • bindings/ScriptControllerBase.cpp:

(WebCore::ScriptController::canExecuteScripts):
(WebCore::ScriptController::executeScript):

  • bindings/js/JSEventListener.cpp:

(WebCore::JSEventListener::handleEvent):

  • bindings/js/JSLazyEventListener.cpp:

(WebCore::JSLazyEventListener::initializeJSFunction):

  • bindings/js/ScheduledAction.cpp:

(WebCore::ScheduledAction::execute):

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::bindingRootObject):
(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::jsObjectForPluginElement):
(WebCore::ScriptController::executeScriptInWorld):

  • bindings/js/ScriptController.h:

(WebCore::):

  • bindings/js/ScriptControllerMac.mm:

(WebCore::ScriptController::windowScriptObject):

  • bindings/js/ScriptDebugServer.cpp:

(WebCore::ScriptDebugServer::setJavaScriptPaused):

  • bindings/js/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/js/ScriptState.cpp:

(WebCore::scriptStateFromNode):

  • bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::windowScriptNPObject):
(WebCore::ScriptController::createScriptObjectForPluginElement):

  • bindings/v8/ScriptController.h:

(WebCore::):

  • bindings/v8/ScriptEventListener.cpp:

(WebCore::createAttributeEventListener):

  • bindings/v8/V8Proxy.cpp:

(WebCore::V8Proxy::retrieve):

  • dom/ScriptElement.cpp:

(WebCore::ScriptElementData::evaluateScript):

  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::startElementNs):

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::controls):

  • html/HTMLTokenizer.cpp:

(WebCore::HTMLTokenizer::parseTag):
(WebCore::HTMLTokenizer::processToken):

  • inspector/InspectorController.cpp:

(WebCore::canPassNodeToJavaScript):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::isProcessingUserGesture):
(WebCore::FrameLoader::open):
(WebCore::FrameLoader::dispatchDidClearWindowObjectsInAllWorlds):
(WebCore::FrameLoader::dispatchDidClearWindowObjectInWorld):

  • loader/FrameLoaderClient.h:

(WebCore::FrameLoaderClient::didNotAllowScript):

WebKit/chromium:

Plumb didNotAllowScript through Chromium's WebKit API.

  • public/WebFrameClient.h:

(WebKit::WebFrameClient::didNotAllowScript):

  • src/DebuggerAgentImpl.cpp:

(WebKit::DebuggerAgentImpl::createUtilityContext):

  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::didNotAllowScript):

  • src/FrameLoaderClientImpl.h:
  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::bindToWindowObject):

WebKit/mac:

Make these two callsites explicit about not running script immediately.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::getWindowNPObject):
(WebKit::NetscapePluginInstanceProxy::demarshalValueFromArray):

4:17 PM Changeset in webkit [55206] by pkasting@chromium.org
  • 6 edits in trunk/WebCore

Add call to FrameLoaderClient::allowPlugins everywhere arePluginsEnabled is called
​https://bugs.webkit.org/show_bug.cgi?id=34997

Patch by Adam Barth <​abarth@webkit.org> on 2010-02-24
Reviewed by Darin Fisher.

If we want to let the FrameLoaderClient override arePluginsEnabled,
then we need to call out to the FrameLoaderClient every time we read
the setting.

We don't have testing infrustructure for these changes, which is lame.
I'm supposed to fix that in Bug 33991.

  • dom/DOMImplementation.cpp:

(WebCore::DOMImplementation::createDocument):

  • loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::substituteMIMETypeFromPluginDatabase):

  • loader/PluginDocument.cpp:

(WebCore::PluginTokenizer::writeRawData):

  • page/Page.cpp:

(WebCore::Page::pluginData):

  • plugins/MimeType.cpp:

(WebCore::MimeType::enabledPlugin):

3:19 PM CommitQueue edited by rik@webkit.org
commiters.py is now in webkitpy (diff)
3:17 PM Changeset in webkit [55205] by jamesr@google.com
  • 2 edits
    2 adds in trunk/WebCore

2010-02-24 James Robinson <​jamesr@chromium.org>

Reviewed by Dmitry Titov.

Do not fire focus events while a modal dialog is up.
​https://bugs.webkit.org/show_bug.cgi?id=33962

Modifies the FocusController to check the frame's page's defersLoading() flag before firing blur/focus events.
This flag is set while a modal dialog (like a window.alert or window.confirm) is up. Firing the events causes
assertion failures, since when the dialog is dismissed the PageGroupLoadDeferrer assumes that no script has run.

Manual tests only as DumpRenderTree does not support modal dialogs

  • manual-tests/modal-dialog-blur-selfclose.html: Added.
  • manual-tests/modal-dialog-blur.html: Added.
  • page/FocusController.cpp: (WebCore::dispatchEventsOnWindowAndFocusedNode):
2:58 PM Changeset in webkit [55204] by jamesr@google.com
  • 2 edits in trunk/WebKitTools

2010-02-24 James Robinson <​jamesr@chromium.org>

Reviewed by Eric Seidel.

Provide default username parameter to SVN.commit_with_message().

  • Scripts/webkitpy/scm.py:
2:48 PM Changeset in webkit [55203] by mitz@apple.com
  • 3 edits
    3 adds in trunk

<rdar://problem/7018611> innerHTML applies meta/link/title tags from a detached html element

Reviewed by Anders Carlsson.

WebCore:

Test: fast/parser/fragment-parser.html

Ensure that fragment parsing has no side effects on the fragment’s owner
document.

  • html/HTMLParser.cpp:

(WebCore::HTMLParser::insertNode): Don’t call
dispatchDocumentElementAvailable() for fragments.
(WebCore::HTMLParser::handleError): Don’t copy attributes to the owner
document’s <html> and <body> elements when a redundant <html> or <body>
is encountered while parsing a fragment.
(WebCore::HTMLParser::framesetCreateErrorCheck): Don’t change the owner
document’s <body> element’s style when parsing a fragment.
(WebCore::HTMLParser::createHead): Don’t attach the new <head> to the
ownder document of a fragment.

LayoutTests:

  • fast/parser/fragment-parser-expected.txt: Added.
  • fast/parser/fragment-parser.html: Added.
  • fast/parser/script-tests/fragment-parser.js: Added.
2:31 PM Changeset in webkit [55202] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

squash for checkin

2:09 PM Changeset in webkit [55201] by levin@chromium.org
  • 7 edits
    2 adds in trunk/WebCore

Need to create a CanvasSurface base class for HTMLCanvasElement.
​https://bugs.webkit.org/show_bug.cgi?id=35322

Reviewed by Darin Adler.

This is an initial step in making the OffscreenCanvas object.

No new functionality so no new tests.

  • GNUmakefile.am: Added CanvasSurface to the build.
  • WebCore.gypi: ditto
  • WebCore.pro: ditto
  • WebCore.vcproj/WebCore.vcproj: ditto
  • WebCore.xcodeproj/project.pbxproj: ditto
  • dom/CanvasSurface.cpp: Added.
  • dom/CanvasSurface.h: Added.
  • html/HTMLCanvasElement.h: Made HTMLCanvasElement inherit from CanvasSurface.
1:58 PM Changeset in webkit [55200] by ap@apple.com
  • 2 edits in trunk/WebKitTools

Reviewed by Dan Bernstein.

​https://bugs.webkit.org/show_bug.cgi?id=35357
Two editing tests fail after DumpRenderTree run loop changes

AppKit decided that it wanted to group all editing commands for some reason (and thus undo
reverted them all at once).

  • DumpRenderTree/mac/DumpRenderTree.mm: (dump): Reverted the change that made DumpRenderTree use -[NSApplication run]. (runTest): Ditto. (-[DumpRenderTreeApplication isRunning]): Override isRunning with a function that always returns YES. This is another way to make the Java plug-in work.
1:52 PM Changeset in webkit [55199] by pkasting@chromium.org
  • 2 edits in trunk/WebCore

Fix regression in calculating an animated image's start time.
​https://bugs.webkit.org/show_bug.cgi?id=35115

Reviewed by Adam Barth.

  • platform/graphics/BitmapImage.cpp:

(WebCore::BitmapImage::startAnimation):

1:20 PM Changeset in webkit [55198] by oliver@apple.com
  • 6 edits in trunk

2010-02-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

[REGRESSION in r55185] EXC_BAD_ACCESS on opening inspector.
​https://bugs.webkit.org/show_bug.cgi?id=35335

compileGetDirectOffset modifies the contents of the object register
when the object is not using the inline storage array. As the object
register contains our 'this' pointer we can't allow it to be clobbered.
The fix is simply to copy the register into a separate scratch register
when we're loading off an object that doesn't use inline storage.

  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdSelfList):
  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::privateCompileGetByIdSelfList):

2010-02-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

[REGRESSION in r55185] EXC_BAD_ACCESS on opening inspector.
​https://bugs.webkit.org/show_bug.cgi?id=35335

Add tests for caching getter lookup on large objects (eg. those not
using the inline storage array).

  • fast/js/pic/cached-getter-setter-expected.txt:
  • fast/js/pic/cached-getter-setter.html:
1:15 PM Changeset in webkit [55197] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-02-24 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Unskip 3 passing cookies tests.

  • platform/gtk/Skipped:
12:51 PM Changeset in webkit [55196] by mitz@apple.com
  • 3 edits
    4 adds in trunk

<rdar://problem/7682827> Text with :first-letter lingers after being removed

Reviewed by Simon Fraser.

WebCore:

Test: fast/css/first-letter-set-text.html

  • rendering/RenderTextFragment.cpp:

(WebCore::RenderTextFragment::setTextInternal): Set this back as the
text node’s renderer, as removing the first letter has resets the node’s
renderer.

LayoutTests:

  • fast/css/first-letter-set-text-expected.checksum: Added.
  • fast/css/first-letter-set-text-expected.png: Added.
  • fast/css/first-letter-set-text-expected.txt: Added.
  • fast/css/first-letter-set-text.html: Added.
11:31 AM Changeset in webkit [55195] by Philippe Normand
  • 2 edits in trunk/LayoutTests

2010-02-24 Philippe Normand <​pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[GTK] DRT implement execCommand()
​https://bugs.webkit.org/show_bug.cgi?id=35351

Skip test that fails in GTK+.

  • platform/gtk/Skipped:
10:15 AM Changeset in webkit [55194] by Philippe Normand
  • 1 edit
    1 add in trunk/LayoutTests

2010-02-24 Philippe Normand <​pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

Adding a GTK layout test result for a new test
added in r55159.

  • platform/gtk/fast/repaint/repaint-during-scroll-expected.txt: Added.
8:20 AM Changeset in webkit [55193] by ariya@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Ariya Hidayat <​ariya.hidayat@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Faster cut-off for rectangle fill without shadow.
​https://bugs.webkit.org/show_bug.cgi?id=35337

  • platform/graphics/qt/GraphicsContextQt.cpp:
7:39 AM QtWebKitTableOfFeatures46 edited by Henry Haverinen
(diff)
7:05 AM Changeset in webkit [55192] by vestbo@webkit.org
  • 3 edits
    1 delete in trunk

[Qt] Remove QGVLauncher

Patch by Jesus Sanchez-Palencia <​jesus.palencia@openbossa.org> on 2010-02-19
Reviewed by Kenneth Rohde Christiansen.

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

6:04 AM Changeset in webkit [55191] by xan@webkit.org
  • 2 edits in trunk

2010-02-24 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Enable SharedWorkers by default, since that's been the default for
a long time in our build-webkit configuration.

  • configure.ac:
6:01 AM Changeset in webkit [55190] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/WebCore

[Qt] Corrects build break of QtLauncher on Windows.
The VERSION variable was not set anymore while building in trunk.

Reviewed by nobody, build fix.

  • WebCore.pro:
5:46 AM Changeset in webkit [55189] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Get rid of an extra call to
setNPWindowIfNeeded. updatePluginWidget already calls this for us
if needed.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::platformStart):
5:43 AM Changeset in webkit [55188] by Adam Roben
  • 2 edits in trunk/WebKitTools

2010-02-23 Adam Roben <Adam Roben>

Make commit-log-editor find just-added ChangeLog files

Fixes <​http://webkit.org/b/35294> commit-log-editor doesn't find
just-added ChangeLog files

Reviewed by Dave Levin.

  • Scripts/commit-log-editor: (top level): Modified the regular expression that's used to find modified ChangeLog files to also look for just-added ChangeLog files.
5:21 AM Changeset in webkit [55187] by apavlov@chromium.org
  • 6 edits in trunk/WebCore

2010-02-24 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Activate/Deactivate breaks does not look consistent with rest of the toolbar.

Fix style and add the new images to the related file sets.
​https://bugs.webkit.org/show_bug.cgi?id=35307

  • WebCore.gypi:
  • inspector/front-end/Images/deactivateBreakpointsButtonGlyph.png:
  • inspector/front-end/Images/deactivateBreakpointsDisabledButtonGlyph.png:
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
4:44 AM Changeset in webkit [55186] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-02-24 Xan Lopez <​xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Try to sanitize a bit the geometry management of plugins.

Stop doing contradictory things in updatePluginWidget (we were at
the same time pretending the geometry setting was delayed until
::paint and setting the geometry not once but twice) and just set
it any time the windowRect or clipRect for the plugin has changed.

This is closer to what the Mac port does, and fixes instances of
the plugins not being drawn until the window is resized or
scrolled. Other than that all manual and layout tests seems to
still work.

  • plugins/gtk/PluginViewGtk.cpp: (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::setNPWindowIfNeeded):
4:20 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)
4:17 AM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com
(diff)
1:02 AM Changeset in webkit [55185] by oliver@apple.com
  • 7 edits in trunk/JavaScriptCore

2010-02-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Speed up getter performance in the jit
​https://bugs.webkit.org/show_bug.cgi?id=35332

Implement getter lookup caching in the interpreter.
The getter stubs are generated through basically the
same code paths as the normal get_by_id caching.
Instead of simply loading a property and returning,
we load the getter slot, and pass the getter, base value
and return address to a shared stub used for getter
dispatch.

  • jit/JIT.h: (JSC::JIT::compileGetByIdProto): (JSC::JIT::compileGetByIdSelfList): (JSC::JIT::compileGetByIdProtoList): (JSC::JIT::compileGetByIdChainList): (JSC::JIT::compileGetByIdChain):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITStubs.cpp: (JSC::JITThunks::tryCacheGetByID): (JSC::DEFINE_STUB_FUNCTION):
  • jit/JITStubs.h: (JSC::):
  • runtime/GetterSetter.h:
12:51 AM Changeset in webkit [55184] by hamaji@chromium.org
  • 3 edits in trunk/WebKitTools

2010-02-24 Shinichiro Hamaji <​hamaji@chromium.org>

Reviewed by Darin Adler.

check-webkit-style false positive for WebCore forwarding header
​https://bugs.webkit.org/show_bug.cgi?id=34604

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:
Note: See TracTimeline for information about the timeline view.