Timeline


and

08/23/10:

23:54 Changeset [65866] by loki@webkit.org
  • 2 edits in trunk/JavaScriptCore

[WINCE] Buildfix for GeneratedJITStubs after r64818
https://bugs.webkit.org/show_bug.cgi?id=44469

Patch by Patrick Gansterer <paroga@paroga.com> on 2010-08-23
Rubber-stamped by Gabor Loki.

Use " THUNK_RETURN_ADDRESS_OFFSET" instead of "#offset#".

  • jit/JITStubs.cpp:
22:14 Changeset [65865] by commit-queue@webkit.org
  • 2 edits
    2 deletes in trunk/LayoutTests

2010-08-23 Hironori Bono <hbono@chromium.org>

Reviewed by Kent Tamura.

Removes the rebaselined results for special-key-events-in-input-text.html
because my test_shell change fixes it and produces the expected output.
https://bugs.webkit.org/show_bug.cgi?id=44156

  • platform/chromium-mac/fast/events/special-key-events-in-input-text-expected.txt: Removed.
  • platform/chromium-win/fast/events/special-key-events-in-input-text-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
21:20 Changeset [65864] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

HTMLScriptRunner::create
https://bugs.webkit.org/show_bug.cgi?id=44474

  • html/HTMLDocumentParser.cpp: (WebCore::HTMLDocumentParser::HTMLDocumentParser):
  • html/HTMLScriptRunner.h: (WebCore::HTMLScriptRunner::create):
21:01 Changeset [65863] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

HTMLTreeBuilder should use adoptPtr
https://bugs.webkit.org/show_bug.cgi?id=44473

We should really use this pattern everywhere.

  • html/HTMLDocumentParser.cpp: (WebCore::HTMLDocumentParser::HTMLDocumentParser):
  • html/HTMLTreeBuilder.h: (WebCore::HTMLTreeBuilder::create):
20:51 Changeset [65862] by mrobinson@webkit.org
  • 2 edits
    1 delete in trunk/LayoutTests

2010-08-23 Martin Robinson <mrobinson@igalia.com>

Skip another test which shows one pixel differences between the 32-bit and 64-bit bot.

  • platform/gtk/Skipped: Skip newly failing test.
  • platform/gtk/svg/custom/recursive-filter-expected.txt: Removed.
20:49 Changeset [65861] by jhoneycutt@apple.com
  • 3 edits in trunk/LayoutTests

Add tests to the Skipped list.

Rubber-stamped by Brady Eidson.

  • platform/mac-wk2/Skipped:
  • platform/win/Skipped:
20:43 Changeset [65860] by jcivelli@chromium.org
  • 4 edits
    1 add in trunk/WebKit/chromium

2010-08-23 Jay Civelli <jcivelli@chromium.org>

Reviewed by Darin Fisher.

Made WebFrame not report the text from hidden frames.
(some pages contain hidden frames with garbage text that
should not be indexed or used to detect the page's language).
https://bugs.webkit.org/show_bug.cgi?id=39456

  • WebKit.gyp:
  • public/WebCString.h: (WebKit::operator<):
  • public/WebURL.h: (WebKit::operator<):
  • src/WebCString.cpp: (WebKit::WebCString::compare):
  • src/WebFrameImpl.cpp: (WebKit::frameContentAsPlainText):
  • tests/RunAllTests.cpp: (main):
  • tests/WebFrameTest.cpp: Added.
20:25 Changeset [65859] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

Set the ResourceResponse HTTP headers in RessourceHandleWin
https://bugs.webkit.org/show_bug.cgi?id=44444

  • platform/network/win/ResourceHandleWin.cpp: (WebCore::queryHTTPHeader): (WebCore::ResourceHandle::onRequestComplete):
20:24 Changeset [65858] by dumi@chromium.org
  • 1 edit
    1 delete in trunk/LayoutTests

Unreviewed, removing a Chromium-specific expectation.

  • platform/chromium-win/editing/execCommand/insert-list-xml-expected.txt: Removed.
20:04 Changeset [65857] by commit-queue@webkit.org
  • 4 edits in trunk/WebCore

2010-08-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

Support all available biBitCount values in BitmapInfo
https://bugs.webkit.org/show_bug.cgi?id=43724

Add an enum with all possible values for biBitCount.
Also remove explicit initialization of bmiHeader members,
because that is already done in the constructor.

  • platform/win/BitmapInfo.cpp: Added property svn:eol-style. (WebCore::bitmapInfoForSize): (WebCore::BitmapInfo::create): (WebCore::BitmapInfo::createBottomUp):
  • platform/win/BitmapInfo.h: Added property svn:eol-style. (WebCore::BitmapInfo::): (WebCore::BitmapInfo::bytesPerLine): (WebCore::BitmapInfo::paddedBytesPerLine): (WebCore::BitmapInfo::paddedWidth):
  • platform/win/PopupMenuWin.cpp: (WebCore::PopupMenuWin::paint):
19:30 Changeset [65856] by tkent@chromium.org
  • 6 edits in trunk

2010-08-23 Kent Tamura <tkent@chromium.org>

Unreviewed, build fix for r65852.

  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded):
  • rendering/TextControlInnerElements.cpp: (WebCore::InputFieldSpeechButtonElement::InputFieldSpeechButtonElement): (WebCore::InputFieldSpeechButtonElement::create):
  • rendering/TextControlInnerElements.h:

2010-08-23 Kent Tamura <tkent@chromium.org>

Unreviewed, build fix for r65852.

  • src/WebNode.cpp: (WebKit::WebNode::parentNode):
19:26 Changeset [65855] by mrobinson@webkit.org
  • 2 edits
    36 adds in trunk/LayoutTests

2010-08-23 Martin Robinson <mrobinson@igalia.com>

Rebaseline new tests and skip tests which produce incorrect output.

  • platform/gtk/Skipped:
  • platform/gtk/fast/canvas/setWidthResetAfterForcedRender-expected.txt: Added.
  • platform/gtk/fast/repaint/multicol-repaint-expected.txt: Added.
  • platform/gtk/fast/repaint/view-background-from-body-1-expected.txt: Copied from LayoutTests/platform/mac/fast/repaint/view-background-from-body-1-expected.txt.
  • platform/gtk/fast/repaint/view-background-from-body-2-expected.txt: Copied from LayoutTests/platform/mac/fast/repaint/view-background-from-body-2-expected.txt.
  • platform/gtk/http/tests/misc/slow-loading-image-in-pattern-expected.txt: Added.
  • platform/gtk/mathml/presentation/attributes-expected.txt: Added.
  • platform/gtk/mathml/presentation/fenced-expected.txt: Added.
  • platform/gtk/mathml/presentation/fractions-expected.txt: Added.
  • platform/gtk/mathml/presentation/fractions-vertical-alignment-expected.txt: Added.
  • platform/gtk/mathml/presentation/mo-expected.txt: Added.
  • platform/gtk/mathml/presentation/over-expected.txt: Added.
  • platform/gtk/mathml/presentation/roots-expected.txt: Added.
  • platform/gtk/mathml/presentation/row-alignment-expected.txt: Added.
  • platform/gtk/mathml/presentation/row-expected.txt: Added.
  • platform/gtk/mathml/presentation/style-expected.txt: Added.
  • platform/gtk/mathml/presentation/subsup-expected.txt: Added.
  • platform/gtk/mathml/presentation/tokenElements-expected.txt: Added.
  • platform/gtk/mathml/presentation/under-expected.txt: Added.
  • platform/gtk/mathml/presentation/underover-expected.txt: Added.
  • platform/gtk/svg/batik/paints/patternRegions-positioned-objects-expected.txt: Added.
  • platform/gtk/svg/custom/convolution-crash-expected.txt: Copied from LayoutTests/platform/mac/svg/custom/convolution-crash-expected.txt.
  • platform/gtk/svg/custom/marker-empty-path-expected.txt: Copied from LayoutTests/platform/mac/svg/custom/marker-empty-path-expected.txt.
  • platform/gtk/svg/custom/path-zero-strokewidth-expected.txt: Added.
  • platform/gtk/svg/custom/pattern-incorrect-tiling-expected.txt: Added.
  • platform/gtk/svg/custom/pattern-no-pixelation-expected.txt: Copied from LayoutTests/platform/mac/svg/custom/pattern-no-pixelation-expected.txt.
  • platform/gtk/svg/custom/recursive-filter-expected.txt: Added.
  • platform/gtk/svg/custom/recursive-gradient-expected.txt: Added.
  • platform/gtk/svg/custom/recursive-mask-expected.txt: Added.
  • platform/gtk/svg/custom/recursive-pattern-expected.txt: Added.
  • platform/gtk/svg/custom/relative-sized-content-with-resources-expected.txt: Added.
  • platform/gtk/svg/custom/svg-fonts-without-missing-glyph-expected.txt: Added.
  • platform/gtk/svg/filters/filter-empty-g-expected.txt: Copied from LayoutTests/platform/mac/svg/filters/filter-empty-g-expected.txt.
  • platform/gtk/svg/in-html/circle-expected.txt: Added.
  • platform/gtk/svg/zoom/page/zoom-mask-with-percentages-expected.txt: Added.
19:14 Changeset [65854] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Eric Seidel.

MarkupAccumulator::appendStartMarkup should be broken down into pieces
https://bugs.webkit.org/show_bug.cgi?id=44288

Extracted appendText, appendComment, appendProcessingInstruction, appendElement and appendCDATASection.
Also simplified the conditionals in appendText.

No new tests are added since this is a cleanup.

  • editing/markup.cpp: (WebCore::MarkupAccumulator::appendText): (WebCore::MarkupAccumulator::appendComment): (WebCore::MarkupAccumulator::appendProcessingInstruction): (WebCore::MarkupAccumulator::appendElement): (WebCore::MarkupAccumulator::appendCDATASection): (WebCore::MarkupAccumulator::appendStartMarkup):
18:49 Changeset [65853] by simon.fraser@apple.com
  • 5 edits in trunk

2010-08-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Alexey Proskuryakov.

TreeWalker is not calling acceptNode function in filter object
https://bugs.webkit.org/show_bug.cgi?id=35296

Fix incorrect shadowing of the 'function' variable, which could result
in badness when asking for arguments.callee.

  • bindings/js/JSNodeFilterCondition.cpp: (WebCore::JSNodeFilterCondition::acceptNode):
18:39 Changeset [65852] by darin@apple.com
  • 23 edits in trunk

WebCore: Changed type of Node::parentNode to ContainerNode so we don't have to cast,
since parent are all containers.

Reviewed by Geoff Garen.

  • css/CSSStyleSelector.h: Changed m_parentNode to ContainerNode*.
  • dom/Document.h: Fixed the type of TreeShared in the Node constructor.

Yes, it's here.

  • dom/Node.cpp:

(WebCore::Node::eventParentNode): Removed now-unneeded typecast dance.
(WebCore::eventTargetAsSVGElementInstance):Changed type to ContainerNode*.

  • dom/Node.h: Made Node inherit from TreeShared<ContainerNode> instead of

from TreeShared<Node>. Removed bogus comment. Changed return type of parentNode
and shadowParentNode functions to ContainerNode*. Fixed using to use
TreeShared<ContainerNode>.

  • dom/Position.h: Include ContainerNode.h instead of Node.h.
  • html/HTMLConstructionSite.cpp:

(WebCore::HTMLConstructionSite::findFosterSite): Removed now-unneeded typecast.

  • rendering/MediaControlElements.h:
  • rendering/SVGShadowTreeElements.h:
  • rendering/TextControlInnerElements.h:

Changed return type of shadowParentNode functions to ContainerNode.

  • rendering/RenderMeter.cpp:

(WebCore::RenderMeter::updatePartsState): Cast the node to HTMLElement* before
passing it to the function that creates the shadow element. We know the node is
a HTMLMeterElement, but the node function doesn't know that.

  • rendering/RenderProgress.cpp:

(WebCore::RenderProgress::updatePartsState): Ditto.

  • rendering/RenderSlider.cpp:

(WebCore::SliderThumbElement::SliderThumbElement): More of the same.
(WebCore::SliderThumbElement::create): Ditto.
(WebCore::RenderSlider::updateFromElement): Ditto.

  • rendering/RenderTextControl.cpp:

(WebCore::RenderTextControl::createSubtreeIfNeeded): Ditto.

  • rendering/RenderTextControlSingleLine.cpp:

(WebCore::RenderTextControlSingleLine::createSubtreeIfNeeded): Ditto.

  • rendering/SVGShadowTreeElements.cpp:

(WebCore::SVGShadowTreeRootElement::SVGShadowTreeRootElement): Ditto.
(WebCore::SVGShadowTreeRootElement::shadowParentNode): Ditto.

  • rendering/ShadowElement.cpp:

(WebCore::ShadowBlockElement::create): Ditto.
(WebCore::ShadowBlockElement::ShadowBlockElement): Ditto.
(WebCore::ShadowBlockElement::createForPart): Ditto.
(WebCore::ShadowInputElement::create): Ditto.
(WebCore::ShadowInputElement::ShadowInputElement): Ditto.

  • rendering/ShadowElement.h:

(WebCore::ShadowElement::ShadowElement): Ditto.
(WebCore::ShadowElement::shadowParent): Ditto.
(WebCore::ShadowElement::shadowParentNode): Ditto.

  • rendering/TextControlInnerElements.cpp:

(WebCore::TextControlInnerElement::TextControlInnerElement): Ditto.
(WebCore::TextControlInnerElement::create): Ditto.
(WebCore::TextControlInnerTextElement::TextControlInnerTextElement): Ditto.
(WebCore::TextControlInnerTextElement::create): Ditto.
(WebCore::SpinButtonElement::SpinButtonElement): Ditto.
(WebCore::SpinButtonElement::create): Ditto.

  • svg/SVGElement.cpp:

(WebCore::SVGElement::eventParentNode): Removed now-unneeded typecast.

  • svg/SVGStyledElement.cpp:

(WebCore::SVGStyledElement::title): Changed type to ContainerNode*.

WebKit2: * WebProcess/InjectedBundle/API/c/WKBundleNode.cpp:
(WKBundleNodeGetParent): Update since parent now returns a
ContainerNode*. The toRef function is a bit too picky to convert
it without a bit of help.

Reviewed by Geoff Garen.

18:25 Changeset [65851] by mrobinson@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] The Mozilla theme drawing code incorrectly renders scrollbar backgrounds
https://bugs.webkit.org/show_bug.cgi?id=44388

No new tests as this functionality is currently unused. When the
the new scrollbar theme code lands for GTK+, this will be tested
by scrollbar pixel tests.

  • platform/gtk/gtk2drawing.c: Expose a method to paint scrolled window backgrounds and disable incorrect rendering of scrollbar trough backgrounds. (moz_gtk_scrolled_window_paint): Added. (moz_gtk_scrollbar_trough_paint): Disable incorrect background render.
  • platform/gtk/gtkdrawing.h: Add scrolled window widget type in the enum.
18:23 Changeset [65850] by mrobinson@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] The Mozilla theme drawing API should expose extra information about scrollbar geometry
https://bugs.webkit.org/show_bug.cgi?id=44385

Expose the trough_under_steppers property in MozGtkScrollbarMetrics.

No new tests as this does not change functionality.

  • platform/gtk/gtk2drawing.c: Set the trough_under_steppers member when accessing theme data.
  • platform/gtk/gtkdrawing.h: Add the trough_under_steppers member.
17:44 Changeset [65849] by darin@apple.com
  • 6 edits in trunk/WebCore

Parser DOM tree manipulation functions do not need to be virtual.

Reviewed by Geoff Garen.

  • dom/ContainerNode.h: Made parserAddChild, parserRemoveChild, and

parserInsertBefore non-virtual.

  • dom/Node.cpp: Removed the parserAddChild, parserRemoveChild, and

parserInsertBefore functions, which should never have been in the Node class.

  • dom/Node.h: Ditto.
  • html/HTMLConstructionSite.cpp:

(WebCore::HTMLConstructionSite::attach): Changed argument type to ContainerNode*
because we never need to attach a child to a non-container node.
(WebCore::HTMLConstructionSite::findFosterSite): Cast the result of the parent
function to ContainerNode*. A parent is always guaranteed to be a container, but
the parent function returns a Node* so that callers can use it without including
ContainerNode's header. That seems like something we can easily fix later.

  • html/HTMLConstructionSite.h: Changed AttachmentSite::parent and the

attach function to ContainerNode* instead of Node*.

17:29 Changeset [65848] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-23 Adam Barth <abarth@webkit.org>

Update the expected results for this test. I think the new expected
results are better than the old ones, but we'll check with some editing
folks to be sure.

  • editing/execCommand/insert-list-xml-expected.txt:
17:04 Changeset [65847] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

2010-08-23 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

[REGRESSION] Interpreter incorrectly excludes prototype chain when validating put_by_id_transition
https://bugs.webkit.org/show_bug.cgi?id=44240
<rdar://problem/8328995>

Fix an error I introduced when cleaning up the interpreter side of the logic
to prevent setters being called in object initialisers.

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
16:55 Changeset [65846] by dumi@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed, updating Chromium expectations.

  • platform/chromium/test_expectations.txt:
16:52 Changeset [65845] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

editing/pasteboard/bad-placeholder.html fails with --html5-treebuilder
https://bugs.webkit.org/show_bug.cgi?id=44463

The problem here is that using the documentElement as the context puts
the tree builder into the BeforeHead insertion mode, which strips
leading spaces. This code is confused about what it wants, but it
certainly doesn't want to be in that insertion mode. Looking through
the callers, they'd much rather be in the InBody insertion mode. We
can get them there by creating a fake body element as the context
element.

In the long term, all this code needs to be changed to move away from
deprecatedCreateContextualFragment, which does a bunch of nasty stuff
like removing certain kinds of elements. However, that's a battle for
another day.

  • editing/markup.cpp: (WebCore::createFragmentFromMarkup):
16:49 Changeset [65844] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

fast/xsl/default-html.html fails with HTML5 fragment parsing
https://bugs.webkit.org/show_bug.cgi?id=44450

This patch is a step down a trail of tears. As far as I can tell,
there's no spec for XSLTProcessor.transformToFragment. This patch
attempts to infer the proper behavior from test cases and the Mozilla
wiki.

  • xml/XSLTProcessor.cpp: (WebCore::createFragmentFromSource):
16:48 Changeset [65843] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

editing/pasteboard/paste-visible-script.html is broken with --html5-treebuilder
https://bugs.webkit.org/show_bug.cgi?id=44457

Turns out there are two more checks we need for fragment scripting
permission. Not the most beautiful design, but it seems work.

  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::insertScriptElement):
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTag):
16:44 Changeset [65842] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-08-23 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

fast/dom/script-innerHTML-x.xhtml fails when run with the HTML5 TreeBuilder in fragment mode
https://bugs.webkit.org/show_bug.cgi?id=44447

This special handling for script/style used to exist in
setInnerHTML. HTML5 moves this logic into the HTML and XML
parsers instead of in setInnerHTML.

In order to share this logic between WebKit's two XML parsers
I had to clean up a bit of the libxml2 parser and add a
new appendFragmentSource method.

Covered by fast/dom/script-innerHTML-x.xhtml.

  • dom/XMLDocumentParser.cpp: (WebCore::XMLDocumentParser::end):
    • Now that libxml2 is calling finish() for fragments (Qt already was) I went through and removed this unneeded style update after fragment parsing.

(WebCore::XMLDocumentParser::parseDocumentFragment):

  • Yay for shared code!
  • This is where I added the style/script hack moved from setInnerHTML.
  • dom/XMLDocumentParser.h:
    • Fix indent.
  • dom/XMLDocumentParserLibxml2.cpp:
    • Removed parseDocumentFragment and moved necessary libxml-specific logic into appendFragmentSource.

(WebCore::XMLDocumentParser::appendFragmentSource):

  • dom/XMLDocumentParserQt.cpp: (WebCore::XMLDocumentParser::appendFragmentSource):
16:32 Changeset [65841] by mrobinson@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-23 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] The 64-bit Release bot is showing crashes on tests that use mouse buttons
https://bugs.webkit.org/show_bug.cgi?id=44465

Initialize the axes field of mouse button and motion events to null.
This is the value that should be used for events originating from the
mouse device.

  • DumpRenderTree/gtk/EventSender.cpp: (prepareMouseButtonEvent): Initalize button event axes field to null. (mouseMoveToCallback): Initialize motion event axes field to null.
16:12 Changeset [65840] by commit-queue@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-08-23 Michael Saboff <msaboff@apple.com>

Reviewed by Oliver Hunt.

Fixed case where a single character search string in a string.replace()
did not properly handle back reference replacement. The fix is to
check for a '$' as part of the check to see if we can execute the
single character replace optimization.
https://bugs.webkit.org/show_bug.cgi?id=44067

  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncReplace):
16:07 Changeset [65839] by kbr@google.com
  • 11 edits in trunk

2010-08-23 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

Remove references to ArrayBuffer and ArrayBufferView from GraphicsContext3D
https://bugs.webkit.org/show_bug.cgi?id=44455

Updated Safari, Qt and Chromium WebGL ports to avoid referencing
ArrayBuffer and ArrayBufferView types from GraphicsContext3D.

Ran all WebGL layout tests; no new regressions. Built and tested
WebKit on Mac OS X; built Chromium on Mac OS X and Linux.

  • src/GraphicsContext3D.cpp: (WebCore::GraphicsContext3DInternal::bufferData): (WebCore::GraphicsContext3DInternal::bufferSubData):

2010-08-23 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

Remove references to ArrayBuffer and ArrayBufferView from GraphicsContext3D
https://bugs.webkit.org/show_bug.cgi?id=44455

Updated Safari, Qt and Chromium WebGL ports to avoid referencing
ArrayBuffer and ArrayBufferView types from GraphicsContext3D.

Ran all WebGL layout tests; no new regressions. Built and tested
WebKit on Mac OS X; built Chromium on Mac OS X and Linux.

  • html/canvas/WebGLBuffer.cpp:
  • html/canvas/WebGLBuffer.h:
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::bufferData): (WebCore::WebGLRenderingContext::bufferSubData): (WebCore::WebGLRenderingContext::texImage2D): (WebCore::WebGLRenderingContext::texSubImage2D): (WebCore::WebGLRenderingContext::simulateVertexAttrib0):
  • platform/graphics/GraphicsContext3D.cpp: (WebCore::GraphicsContext3D::extractTextureData):
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/chromium/GLES2Canvas.cpp: (WebCore::GLES2Canvas::getQuadVertices):
  • platform/graphics/mac/GraphicsContext3DMac.mm: (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData):
  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData):
15:59 Changeset [65838] by eric@webkit.org
  • 4 edits in trunk

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65703.
http://trac.webkit.org/changeset/65703
https://bugs.webkit.org/show_bug.cgi?id=44460

"Causing lots of crashes on Snow Leopard when running the
layout tests" (Requested by eseidel2 on #webkit).

  • WebProcess/Plugins/Netscape/NetscapePluginModule.cpp: (WebKit::NetscapePluginModule::tryLoad):

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65703.
http://trac.webkit.org/changeset/65703
https://bugs.webkit.org/show_bug.cgi?id=44460

"Causing lots of crashes on Snow Leopard when running the
layout tests" (Requested by eseidel2 on #webkit).

  • DumpRenderTree/TestNetscapePlugIn/main.cpp: (NP_Initialize): (NP_GetEntryPoints):
15:30 Changeset [65837] by dumi@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed, updating some Chromium expectations.

  • platform/chromium/test_expectations.txt:
15:23 Changeset [65836] by senorblanco@chromium.org
  • 4 edits in trunk/WebCore

2010-08-20 Stephen White <senorblanco@chromium.org>

Reviewed by Kenneth Russell.

[CHROMIUM] Fix some failing layout tests w/ACCELERATED_2D_CANVAS
https://bugs.webkit.org/show_bug.cgi?id=44346

LayoutTests/fast/canvas/canvas-incremental-repaint.html (top middle
pane).
Failing because we were not applying the CTM in clearRect(). Now using
the fast path when the CTM is identity, and a fillRect() for the rest.
LayoutTests/fast/canvas/canvas-strokeRect.html
LayoutTests/fast/canvas/shadow-offset-[1-7].html
Failing because we weren't switching to the software path when a
shadow is present.
Also refactor the two versions of fillRect(), and use TRIANGLE_STRIP
strip instead of TRIANGLES, which lets us get rid of the element array
and use drawArrays() instead of drawElements().

Covered by the above layout tests.

  • platform/graphics/chromium/GLES2Canvas.cpp: (WebCore::GLES2Canvas::GLES2Canvas): (WebCore::GLES2Canvas::~GLES2Canvas): Remove m_quadIndices (now unused). (WebCore::GLES2Canvas::clearRect): Use a glClear() fast path for the identity-CTM clear, and fillRect() for the rest. (WebCore::GLES2Canvas::fillRect): Refactor the two versions of fillRect(). (WebCore::GLES2Canvas::drawTexturedRect): (WebCore::GLES2Canvas::drawTexturedRectTile): Get rid of the ELEMENT_ARRAY_BUFFER bind. Use drawArrays() instead of drawElements(). (WebCore::GLES2Canvas::getQuadVertices): Re-order the vertices so they form a triangle strip.
  • platform/graphics/chromium/GLES2Canvas.h: Remove m_quadIndices (now unused).
  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::fillRect): Check for a draw looper (shadow), and drop to the software path.
15:12 Changeset [65835] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Patrick Gansterer <paroga@paroga.com>

Reviewed by Adam Roben.

Move filehandling into fileLoadTimer callback
https://bugs.webkit.org/show_bug.cgi?id=43714

Also add mimetype detection for local files.

  • platform/network/ResourceHandleInternal.h: (WebCore::ResourceHandleInternal::ResourceHandleInternal):
  • platform/network/win/ResourceHandleWin.cpp: (WebCore::ResourceHandle::start): (WebCore::ResourceHandle::fileLoadTimer):
14:59 Changeset [65834] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

2010-08-23 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

JSON.stringify is much slower than Firefox on particular pathological input
https://bugs.webkit.org/show_bug.cgi?id=44456

Make StringBuilder::reserveCapacity reserve additional space so we don't end up
repeatedly copying the entire result string.

  • runtime/StringBuilder.h: (JSC::StringBuilder::append): (JSC::StringBuilder::reserveCapacity):
14:52 Changeset [65833] by commit-queue@webkit.org
  • 6 edits in trunk/WebCore

2010-08-23 Iain Merrick <husky@google.com>

Reviewed by Steve Block.

Update JNI bridge for V8 after renaming of JavaString::utf8().
https://bugs.webkit.org/show_bug.cgi?id=44419

This fixes the build on Android. No new functionality, so no new tests.

  • bridge/jni/v8/JNIBridgeV8.cpp: (JavaField::JavaField):
  • bridge/jni/v8/JNIBridgeV8.h: (JSC::Bindings::JavaField::type):
  • bridge/jni/v8/JavaClassV8.cpp: (JavaClass::JavaClass):
  • bridge/jni/v8/JavaNPObjectV8.cpp: (JSC::Bindings::JavaNPObjectGetProperty):
  • bridge/jni/v8/JavaStringV8.h: (JSC::Bindings::JavaStringImpl::utf8):
14:31 Changeset [65832] by commit-queue@webkit.org
  • 4 edits
    3 adds in trunk

2010-08-23 Mihai Parparita <mihaip@chromium.org>

Reviewed by David Hyatt.

Assertion failure in FrameView::layout when modifying the DOM during
pagehide with PageCache enabled
https://bugs.webkit.org/show_bug.cgi?id=43152

Always unschedule any pending relayouts when changing the view, not just
when we're not using the page cache. Otherwise touching the DOM during
the pagehide handler can lead to layouts that will be done by the time
the view has changed, which triggers an assert.

  • fast/loader/resources/unschedule-relayout-after-unload-target.html: Added.
  • fast/loader/unschedule-relayout-after-unload-expected.txt: Added.
  • fast/loader/unschedule-relayout-after-unload.html: Added.

2010-08-23 Mihai Parparita <mihaip@chromium.org>

Reviewed by David Hyatt.

Assertion failure in FrameView::layout when modifying the DOM during
pagehide with PageCache enabled
https://bugs.webkit.org/show_bug.cgi?id=43152

Always unschedule any pending relayouts when changing the view, not just
when we're not using the page cache. Otherwise touching the DOM during
the pagehide handler can lead to layouts that will be done by the time
the view has changed, which triggers an assert.

Test: fast/loader/unschedule-relayout-after-unload.html

  • page/Frame.cpp: (WebCore::Frame::setView): move unscheduleRelayout outside page cache check.
  • page/FrameView.cpp: (WebCore::FrameView::layout): remove early return now that assert should be correct (and the referenced rdar:// got fixed).
14:29 Changeset [65831] by jhoneycutt@apple.com
  • 4 edits in trunk

Layout tests do not run when using WebKitTestRunner in a Debug build.
https://bugs.webkit.org/show_bug.cgi?id=44409

Reviewed by Dan Bernstein.

WebKit2:

  • UIProcess/Launcher/win/ProcessLauncherWin.cpp:

(WebKit::ProcessLauncher::launchProcess):
Use WebKit2WebProcess_debug.exe when building Debug.

WebKitTools:

  • WebKitTestRunner/win/TestControllerWin.cpp:

(WTR::TestController::initializeInjectedBundlePath):
Use InjectedBundle_debug.dll when building Debug.

14:27 Changeset [65830] by jhoneycutt@apple.com
  • 3 edits in trunk/LayoutTests

Add more tests to the WebKit2 skipped lists.

Rubber-stamped by Mark Rowe.

  • platform/mac-wk2/Skipped:
  • platform/win-wk2/Skipped:
13:43 Changeset [65829] by jamesr@google.com
  • 1 edit
    2 adds in trunk/LayoutTests

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65812.
http://trac.webkit.org/changeset/65812
https://bugs.webkit.org/show_bug.cgi?id=44451

Broke fast/events/special-key-events-in-input-text.html for
chromium (Requested by jamesr on #webkit).

  • platform/chromium-mac/fast/events/special-key-events-in-input-text-expected.txt: Added.
  • platform/chromium-win/fast/events/special-key-events-in-input-text-expected.txt: Added.
13:20 Changeset [65828] by rniwa@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

2010-08-19 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

There is no test for unicode-bidi / direction support in ApplyStyleCommand::applyInlineStyle
https://bugs.webkit.org/show_bug.cgi?id=43989

Added a test for MakeTextWritingDirectionNatural, MakeTextWritingDirectionLeftToRight,
and MakeTextWritingDirectionRightToLeft commands.

  • editing/style/make-text-writing-direction-inline.html: Added.
  • editing/style/script-tests/make-text-writing-direction-inline.js: Added. (modifyWritingDirection): (selectFirstWord): (selectSecondAndThirdWords):
13:09 Changeset [65827] by jianli@chromium.org
  • 19 edits
    2 adds in trunk

Handle blob resource.
https://bugs.webkit.org/show_bug.cgi?id=43941

Reviewed by Darin Fisher.

JavaScriptCore:

  • JavaScriptCore.exp: Add an export that is neede by BlobResourceHandle.

WebCore:

To provide lower level blob reosurce handling for all possible ports,
BlobResourceHandle derived from ResourceHandle is implemented. It provides
both synchronous and asynchronous resource loading for blob URL.

BlobResourceHandle needs to create a FileStreamProxy instance in order to
asynchronous file stream operation. To achive this, a hook createAsyncFileStream
is added to ResourceHandleClient interface. When ResourceLoader implements
ths hook, it creates and returns FileStreamProxy.

BlobResourceHandle.* is not added to chromium port since it will implement
its own blob resource handling.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::createAsyncFileStream): Create and return FileStreamProxy.

  • loader/ResourceLoader.h:
  • page/SecurityOrigin.cpp: Add the support to get and validate the origin of blob URL.

(WebCore::SecurityOrigin::create):
(WebCore::SecurityOrigin::canLoad):

  • platform/network/BlobRegistryImpl.cpp:

(WebCore::BlobRegistryImpl::appendStorageItems): Fix a bug that the length is not subtracted.

  • platform/network/BlobResourceHandle.cpp: Added.
  • platform/network/BlobResourceHandle.h: Added.
  • platform/network/HTTPParsers.cpp:

(WebCore::parseRange):

  • platform/network/HTTPParsers.h:
  • platform/network/ResourceHandle.cpp: Hook up with asynchronous blob resource handling.

(WebCore::ResourceHandle::create):

  • platform/network/ResourceHandle.h:
  • platform/network/ResourceHandleClient.h:

(WebCore::ResourceHandleClient::createAsyncFileStream): Add a hook.

  • platform/network/mac/ResourceHandleMac.mm:

(WebCore::ResourceHandle::loadResourceSynchronously): Hook up with synchronous blob resource handling.

13:03 Changeset [65826] by inferno@chromium.org
  • 3 edits
    2 adds in trunk

2010-08-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dimitri Glazkov.

Fix security origin calculation in createPattern. Need to use
cachedImage->response().url() instead of cachedImage->url().
https://bugs.webkit.org/show_bug.cgi?id=44399.

Test: http/tests/security/canvas-remote-read-remote-image-redirect.html

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createPattern):

2010-08-23 Abhishek Arya <inferno@chromium.org>

Reviewed by Dimitri Glazkov.

Tests that calling getImageData(), toDataURL() on a canvas tainted by
a createPattern of a different origin image using redirects from same origin
is not allowed.
https://bugs.webkit.org/show_bug.cgi?id=44399

  • http/tests/security/canvas-remote-read-remote-image-redirect-expected.txt: Added.
  • http/tests/security/canvas-remote-read-remote-image-redirect.html: Added.
12:27 Changeset [65825] by simon.fraser@apple.com
  • 1 edit
    1 add in trunk/LayoutTests

2010-08-23 Simon Fraser <simon.fraser@apple.com>

Add missing test result.

  • platform/mac/fast/table/simple_paint-expected.txt: Copied from LayoutTests/platform/chromium-linux/fast/table/simple_paint-expected.txt.
12:22 Changeset [65824] by simon.fraser@apple.com
  • 3 edits
    3 adds in trunk

2010-08-23 Simon Fraser <simon.fraser@apple.com>

Reviewed by Alexey Proskuryakov.

TreeWalker is not calling acceptNode function in filter object
https://bugs.webkit.org/show_bug.cgi?id=35296

We only accepted raw functions as the NodeFilter on TreeWalker. Fix this to
look for an 'acceptNode' function on the filter object, and use that if present.
Also throw an exception if the filter object does not have an acceptNode function.

Test: fast/dom/TreeWalker/acceptNode-filter.html

  • bindings/js/JSNodeFilterCondition.cpp: (WebCore::JSNodeFilterCondition::acceptNode):
12:18 Changeset [65823] by commit-queue@webkit.org
  • 3 edits
    3 deletes in trunk

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65814.
http://trac.webkit.org/changeset/65814
https://bugs.webkit.org/show_bug.cgi?id=44443

broke svg/custom/missing-xlink.svg (Requested by kling on
#webkit).

  • fast/dom/script-tests/setAttributeNS-prefix-and-null-namespace.js: Removed.
  • fast/dom/setAttributeNS-prefix-and-null-namespace-expected.txt: Removed.
  • fast/dom/setAttributeNS-prefix-and-null-namespace.html: Removed.

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65814.
http://trac.webkit.org/changeset/65814
https://bugs.webkit.org/show_bug.cgi?id=44443

broke svg/custom/missing-xlink.svg (Requested by kling on
#webkit).

  • dom/Element.cpp: (WebCore::Element::setAttributeNS):
12:10 Changeset [65822] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitSite

Clean up some HTML typos in crashlogs.html

Rubber-stamped by Maciej Stachowiak.

  • quality/crashlogs.html: Fixed "</code>>" typos added in

r56300. Changed "<tt>" tag with style attribute to "<code>"
that was missed in r56300.

11:59 Changeset [65821] by jianli@chromium.org
  • 4 edits in trunk

Remove unneeded BlobRegistryImpl.* and WebBlobRegistryImpl.* from
chromium project files.
https://bugs.webkit.org/show_bug.cgi?id=44442

Reviewed by David Levin.

WebCore:

  • WebCore.gypi:

WebKit/chromium:

  • WebKit.gyp:
11:28 Changeset [65820] by mthole@apple.com
  • 2 edits in trunk/WebKit/mac

WebPDFView should remove itself as the delegate of its PDFView before being dealloced
https://bugs.webkit.org/show_bug.cgi?id=44441

Reviewed by Dan Bernstein.

  • WebView/WebPDFView.mm:

(-[WebPDFView dealloc]): Set the PDFView's delegate to nil.

11:12 Changeset [65819] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Many LayoutTests crash when run with --html5-treebuilder
https://bugs.webkit.org/show_bug.cgi?id=44440

Our list of special tags is out of sync with the HTML5 spec. This
patch adds HTML to the list, which fixs a ton of crashers when parsing
fragments. We neet to sync up the list at some point, but we want to
make sure we have test coverage for all those changes, so I've left
that for a future patch.

  • html/HTMLTreeBuilder.cpp:
10:58 Changeset [65818] by leandro@webkit.org
  • 2 edits in trunk/WebKit/efl

2010-08-23 Leandro Pereira <leandro@profusion.mobi>

[EFL] Build fix for platforms that do not require linking with "dl".
CMake includes the variable CMAKE_DL_LIBS that evaluates to the
platform-specific libraries.

  • CMakeListsEfl.txt: Use ${CMAKE_DL_LIBS} instead of dl.
09:56 Changeset [65817] by dbates@webkit.org
  • 2 edits in trunk/WebKit2

2010-08-23 Daniel Bates <dbates@rim.com>

Rubber-stamped by Adam Roben.

Substitute "sizeof("javascript:") - 1" for the hardcoded constant 11 when
extracting the JavaScript code portion of a JavaScript URL so as to make
it clearer that the 11 represents the length of the scheme portion of
a JavaScript URL.

  • WebProcess/Plugins/PluginView.cpp: (WebKit::PluginView::performJavaScriptURLRequest):
09:54 Changeset [65816] by loislo@chromium.org
  • 2 edits in trunk/WebCore

2010-08-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

WebInspector: Context menu in Scripts panel was broken.
https://bugs.webkit.org/show_bug.cgi?id=44431

  • inspector/InspectorFrontendHost.cpp: (WebCore::FrontendMenuProvider::contextMenuItemSelected): (WebCore::FrontendMenuProvider::contextMenuCleared):
09:37 Changeset [65815] by commit-queue@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-08-23 Satish Sampath <satish@chromium.org>

Reviewed by Jeremy Orlow.

Remove obsolete public/API methods in chromium port
https://bugs.webkit.org/show_bug.cgi?id=44421

  • public/WebSpeechInputController.h: (WebKit::WebSpeechInputController::startRecognition): (WebKit::WebSpeechInputController::cancelRecognition): (WebKit::WebSpeechInputController::stopRecording):
  • public/WebSpeechInputListener.h:
09:34 Changeset [65814] by andreas.kling@nokia.com
  • 3 edits
    3 adds in trunk

2010-08-23 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Tor Arne Vestbø.

setAttributeNS() should throw NAMESPACE_ERR for prefixed qualifiedName with null namespace
https://bugs.webkit.org/show_bug.cgi?id=44432

Test: fast/dom/setAttributeNS-prefix-and-null-namespace.html

  • dom/Element.cpp: (WebCore::Element::setAttributeNS): Throw NAMESPACE_ERR if namespace is null and the qualifiedName has a prefix.

2010-08-23 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Tor Arne Vestbø.

setAttributeNS() should throw NAMESPACE_ERR for prefixed qualifiedName with null namespace
https://bugs.webkit.org/show_bug.cgi?id=44432

  • fast/dom/script-tests/setAttributeNS-prefix-and-null-namespace.js: Added.
  • fast/dom/setAttributeNS-prefix-and-null-namespace-expected.txt: Added.
  • fast/dom/setAttributeNS-prefix-and-null-namespace.html: Added.
09:18 Changeset [65813] by commit-queue@webkit.org
  • 3 edits in trunk/WebCore

2010-08-23 Ryuan Choi <ryuan.choi@samsung.com>

Reviewed by Antonio Gomes.

[EFL] rendering was broken when missing plugin.
https://bugs.webkit.org/show_bug.cgi?id=43395

Implement RenderThemeEfl::systemFont to render "Missing plugin" when we
don't have proper plugin.
In this case, RenderEmbeddedObject::paintReplaced call systemFont and
pass returned font to GraphicsContext.

  • platform/efl/RenderThemeEfl.cpp: (WebCore::RenderThemeEfl::setDefaultFontSize): (WebCore::RenderThemeEfl::systemFont):
  • platform/efl/RenderThemeEfl.h:
08:30 Changeset [65812] by commit-queue@webkit.org
  • 1 edit
    2 deletes in trunk/LayoutTests

2010-08-23 Hironori Bono <hbono@chromium.org>

Reviewed by Kent Tamura.

Removes the rebaselined results for special-key-events-in-input-text.html
because my test_shell change fixes it and produces the expected output.
https://bugs.webkit.org/show_bug.cgi?id=44156

  • platform/chromium-mac/fast/events/special-key-events-in-input-text-expected.txt: Removed.
  • platform/chromium-win/fast/events/special-key-events-in-input-text-expected.txt: Removed.
08:20 Changeset [65811] by xan@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-23 Xan Lopez <xlopez@igalia.com>

Reviewed by Martin Robinson.

[GTK] run-launcher fails if WebKitGTK+ compiled with GTK+-3.0 support
https://bugs.webkit.org/show_bug.cgi?id=42026

If a GTK+ 3.x build is present return that as the library to use,
otherwise fallback to the GTK+ 2.x build.

  • Scripts/webkitdirs.pm:
07:40 Changeset [65810] by commit-queue@webkit.org
  • 5 edits in trunk/WebCore

2010-08-23 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: remove javascript breakpoint specific code from BreakpointSidebarPane
https://bugs.webkit.org/show_bug.cgi?id=44327

  • inspector/front-end/BreakpointManager.js: (WebInspector.BreakpointManager.prototype.setBreakpoint): (WebInspector.BreakpointManager.prototype.restoredBreakpoint): (WebInspector.BreakpointManager.prototype._setBreakpoint): (WebInspector.BreakpointManager.prototype._removeBreakpoint): (WebInspector.BreakpointManager.prototype._setBreakpointOnBackend): (WebInspector.Breakpoint):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane): (WebInspector.BreakpointsSidebarPane.prototype.addBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype._breakpointRemoved): (WebInspector.JSBreakpointItem): (WebInspector.JSBreakpointItem.prototype.compareTo): (WebInspector.JSBreakpointItem.prototype.element): (WebInspector.JSBreakpointItem.prototype._breakpointClicked): (WebInspector.JSBreakpointItem.prototype._checkboxClicked): (WebInspector.JSBreakpointItem.prototype._enableChanged): (WebInspector.JSBreakpointItem.prototype._textChanged): (WebInspector.JSBreakpointItem.prototype._removed):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype._breakpointAdded): (WebInspector.ScriptsPanel.prototype._breakpointRemoved):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype.addBreakpoint): (WebInspector.SourceFrame.prototype._addBreakpointToSource):
07:07 Changeset [65809] by loislo@chromium.org
  • 9 edits in trunk/WebCore

2010-08-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: inspector protocol should be switched from array based
message format to object based message format.
Almost all the protocol related things is generated by CodeGeneratorInspector.pm
It was changed a bit. As result InspectorBackendStub.js wraps the
calls into Request objects. InspectorBackendDispatcher.cpp unwraps
these objects and calls corresponding agents. These two files and
RemoteInspectorFrontend are generated by CodeGeneratorInspector.pm
Dispatching part of WebInspector also was adjusted for handling Event
objects produced by RemoteInspectorFrontend.cpp and Response objects
produced by InspectorBackendDispatcher.cpp
https://bugs.webkit.org/show_bug.cgi?id=44338

  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setAttachedWindow):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontendClientLocal.cpp: (WebCore::InspectorFrontendClientLocal::setAttachedWindow):
  • inspector/InspectorValues.h: (WebCore::InspectorObject::find):
  • inspector/front-end/Callback.js: (WebInspector.Callback.prototype.processResponse):
  • inspector/front-end/inspector.js: (WebInspector.dispatch): (WebInspector_syncDispatch): (WebInspector.dispatchMessageFromBackend): (WebInspector.reportProtocolError):
06:58 Changeset [65808] by abecsi@webkit.org
  • 5 edits
    1 add in trunk

2010-08-23 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Find needed forwarding headers for WebKit2 automatically
https://bugs.webkit.org/show_bug.cgi?id=44336

  • MiniBrowser/qt/BrowserWindow.cpp: Use normal (not framework style) includes here. The new script does not satisfy the needs of this file however using normal includes is generally better here.
  • MiniBrowser/qt/MiniBrowser.pro: Added include paths.
  • Scripts/enumerate-included-framework-headers: Enumerates headers included from the given frameworks by files in the current directory (and subdirectories). In the future this needs to be rewritten in perl for being able to use on Windows.

2010-08-23 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Ariya Hidayat.

[Qt] Find needed forwarding headers for WebKit2 automatically
https://bugs.webkit.org/show_bug.cgi?id=44336

  • DerivedSources.pro: Use the new enumerate-included-framework-headers script for generating the list of forwarding headers we want to create.
06:23 Changeset [65807] by jorlow@chromium.org
  • 8 edits
    3 deletes in trunk

2010-08-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Remove IDBDatabase.description per spec changes
https://bugs.webkit.org/show_bug.cgi?id=44264

Update layout tests to match changes. One layout test is now
obsolete.

  • storage/indexeddb/database-basics-expected.txt:
  • storage/indexeddb/database-description-expected.txt: Removed.
  • storage/indexeddb/database-description.html: Removed.
  • storage/indexeddb/script-tests/database-basics.js: (openSuccess):
  • storage/indexeddb/script-tests/database-description.js: Removed.

2010-08-19 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Remove IDBDatabase.description per spec changes
https://bugs.webkit.org/show_bug.cgi?id=44264

There's now no longer any way to access the description from JS.
The main premise of the manual test is now obsolete. Over time we'll
be able to add to it again. While I'm at it, I cleaned up the format
of the manual test to be easier to follow (code and usage wise).

  • manual-tests/indexed-database.html:
  • storage/IDBDatabase.cpp: (WebCore::IDBDatabase::IDBDatabase):
  • storage/IDBDatabase.h:
  • storage/IDBDatabase.idl:
06:06 WinCE edited by paroga@paroga.com
(diff)
05:49 Changeset [65806] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

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

Reviewed by Yury Semikhatsky.

Web Inspector: element's css: pseudo-class locations
https://bugs.webkit.org/show_bug.cgi?id=44344

Also contains a drive-by fix for 44301 (gray out read-only).

  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.update.computedStyleCallback): (WebInspector.StylesSidebarPane.prototype.update): (WebInspector.StylesSidebarPane.prototype._rebuildUpdate): (WebInspector.StylesSidebarPane.prototype._rebuildSectionsForStyleRules): (WebInspector.StylePropertiesSection):
05:34 Changeset [65805] by zoltan@webkit.org
  • 3 edits
    2 adds in trunk/WebKit2

[Qt] Build fix.

  • DerivedSources.pro:
  • WebKit2.pro:
  • WebProcess/Plugins/Netscape/qt: Added.
  • WebProcess/Plugins/Netscape/qt/NetscapePluginQt.cpp: Added.

(WebKit::NetscapePlugin::platformPostInitialize):
(WebKit::NetscapePlugin::platformPaint):
(WebKit::toNP):
(WebKit::NetscapePlugin::platformHandleMouseEvent):
(WebKit::NetscapePlugin::platformHandleWheelEvent):
(WebKit::NetscapePlugin::platformSetFocus):
(WebKit::NetscapePlugin::platformHandleMouseEnterEvent):
(WebKit::NetscapePlugin::platformHandleMouseLeaveEvent):

05:28 Changeset [65804] by loislo@chromium.org
  • 9 edits in trunk/WebCore

2010-08-23 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65803.
http://trac.webkit.org/changeset/65803
https://bugs.webkit.org/show_bug.cgi?id=44416

windows build failed (Requested by loislo on #webkit).

  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp:
  • inspector/InspectorController.h:
  • inspector/InspectorFrontendClientLocal.cpp: (WebCore::InspectorFrontendClientLocal::setAttachedWindow):
  • inspector/InspectorValues.h:
  • inspector/front-end/Callback.js: (WebInspector.Callback.prototype.processResponse):
  • inspector/front-end/inspector.js: (WebInspector.dispatch.delayDispatch): (WebInspector.dispatch): (WebInspector_syncDispatch): (WebInspector.dispatchMessageFromBackend): (WebInspector.reportProtocolError):
05:14 Changeset [65803] by loislo@chromium.org
  • 9 edits in trunk/WebCore

2010-08-23 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: inspector protocol should be switched from array based
message format to object based message format.
Almost all the protocol related things is generated by CodeGeneratorInspector.pm
It was changed a bit. As result InspectorBackendStub.js wraps the
calls into Request objects. InspectorBackendDispatcher.cpp unwraps
these objects and calls corresponding agents. These two files and
RemoteInspectorFrontend are generated by CodeGeneratorInspector.pm
Dispatching part of WebInspector also was adjusted for handling Event
objects produced by RemoteInspectorFrontend.cpp and Response objects
produced by InspectorBackendDispatcher.cpp
https://bugs.webkit.org/show_bug.cgi?id=44338

  • inspector/CodeGeneratorInspector.pm:
  • inspector/Inspector.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setAttachedWindow):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontendClientLocal.cpp: (WebCore::InspectorFrontendClientLocal::setAttachedWindow):
  • inspector/InspectorValues.h: (WebCore::InspectorObject::find):
  • inspector/front-end/Callback.js: (WebInspector.Callback.prototype.processResponse):
  • inspector/front-end/inspector.js: (WebInspector.dispatch): (WebInspector_syncDispatch): (WebInspector.dispatchMessageFromBackend): (WebInspector.reportProtocolError):
05:09 Changeset [65802] by alex@webkit.org
  • 2 edits in trunk/WebCore

2010-08-23 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

Make the copy of the inspector files silent.

  • GNUmakefile.am:
04:44 Changeset [65801] by andreas.kling@nokia.com
  • 3 edits in trunk/WebCore

2010-08-23 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] -webkit-text-stroke is broken due to Qt::TextBypassShaping
https://bugs.webkit.org/show_bug.cgi?id=44403

Don't bypass the Harfbuzz shaping if we're drawing text with a stroke.
For canvas, always use complex text shaping since stroke and fill are decoupled.

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::measureText): (WebCore::CanvasRenderingContext2D::drawTextInternal):
  • platform/graphics/qt/FontQt.cpp: (WebCore::drawTextCommon):
04:35 Changeset [65800] by rolandsteiner@chromium.org
  • 2 edits in trunk/LayoutTests

2010-08-23 Roland Steiner <rolandsteiner@chromium.org>

Unreviewed change to drt_expectations.txt.

Add new entry to platform/chromium/drt_expectations.txt

  • platform/chromium/drt_expectations.txt:
02:49 Changeset [65799] by commit-queue@webkit.org
  • 7 edits
    3 adds in trunk/LayoutTests

2010-08-23 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: add DOM breakpoints test
https://bugs.webkit.org/show_bug.cgi?id=44251

  • http/tests/inspector/debugger-test2.js: Added. (initialize_DebuggerTest.InspectorTest.startDebuggerTest.startTest): (initialize_DebuggerTest.InspectorTest.startDebuggerTest): (initialize_DebuggerTest.InspectorTest.completeDebuggerTest.completeTest): (initialize_DebuggerTest.InspectorTest.completeDebuggerTest): (initialize_DebuggerTest.InspectorTest.waitUntilPaused): (initialize_DebuggerTest.InspectorTest.resumeExecution): (initialize_DebuggerTest.InspectorTest.showScriptSource): (initialize_DebuggerTest.InspectorTest._scriptsAreParsed): (initialize_DebuggerTest.InspectorTest._showScriptSource): (initialize_DebuggerTest):
  • http/tests/inspector/inspector-test2.js: (initialize_InspectorTest): (initialize_InspectorTest.): (runTest.runTestInFrontend): (runTest):
  • inspector/debugger-breakpoints-not-activated-on-reload.html:
  • inspector/dom-breakpoints-expected.txt: Added.
  • inspector/dom-breakpoints.html: Added.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
02:38 Changeset [65798] by philn@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-23 Philippe Normand <pnormand@igalia.com>

Unreviewed, skipping more media tests

[GStreamer] multiple <video> elements + PulseAudio are crash-prone
https://bugs.webkit.org/show_bug.cgi?id=44235

  • platform/gtk/Skipped: Skip media/video-zoom-controls.html and media/adopt-node-crash.html
02:35 Changeset [65797] by mnaganov@chromium.org
  • 16 edits
    2 adds in trunk/WebCore

2010-08-23 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Extract profiler-related code and data from InspectorController into
InspectorProfilerAgent.

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

01:10 Changeset [65796] by girish@forwardbias.in
  • 2 edits in trunk/WebKit/qt

[Qt] Allow wmode=transparent in QWebView on Maemo5 after r65775.

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

Reviewed by Ariya Hidayat.

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::createPlugin):

00:59 Changeset [65795] by ariya@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Crash when purging the scratch buffer for the shadow
https://bugs.webkit.org/show_bug.cgi?id=44384

Patch by Ariya Hidayat <ariya@sencha.com> on 2010-08-23
Reviewed by Kenneth Rohde Christiansen.

WebCore::Timer can't be used in a static object bcause it relies on
thread global data, which is invalid once the application instance is
destroyed. To overcome the problem, use QObject's timer support for
the ShadowBuffer class.

  • platform/graphics/qt/ContextShadow.cpp:

(WebCore::):
(WebCore::ShadowBuffer::ShadowBuffer):
(WebCore::ShadowBuffer::schedulePurge):
(WebCore::ShadowBuffer::timerEvent):

08/22/10:

23:27 Changeset [65794] by tkent@chromium.org
  • 2 edits in trunk/LayoutTests

2010-08-22 Kent Tamura <tkent@chromium.org>

Unreviewed, test expectation update.

  • platform/chromium/drt_expectations.txt:
23:11 Changeset [65793] by philn@webkit.org
  • 4 edits in trunk/WebCore

2010-08-23 Philippe Normand <pnormand@igalia.com>

Reviewed by Xan Lopez.

[GStreamer] don't expose the private player in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=44332

Only a pointer to our playbin2 element instance is required
instead of the whole MediaPlayerPrivateGStreamer instance.

  • platform/graphics/gstreamer/GStreamerGWorld.cpp: (WebCore::GStreamerGWorld::createGWorld): (WebCore::GStreamerGWorld::GStreamerGWorld): (WebCore::GStreamerGWorld::~GStreamerGWorld): (WebCore::GStreamerGWorld::enterFullscreen): (WebCore::GStreamerGWorld::exitFullscreen):
  • platform/graphics/gstreamer/GStreamerGWorld.h: (WebCore::GStreamerGWorld::pipeline):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::createGSTPlayBin):
20:50 Changeset [65792] by andreas.kling@nokia.com
  • 3 edits in trunk/WebKitTools

2010-08-22 No'am Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Qt test browser: enable a QGLWidget viewport with a command line argument
https://bugs.webkit.org/show_bug.cgi?id=43010

  • QtTestBrowser/launcherwindow.cpp: (LauncherWindow::applyPrefs):
  • QtTestBrowser/main.cpp: (LauncherApplication::handleUserOptions):
17:52 Changeset [65791] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-08-22 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsContext: Simplify getting the clip bounding rect
https://bugs.webkit.org/show_bug.cgi?id=44396

Move the Qt 4.8 version check for QPainter::clipBoundingRect()
into a GraphicsContextPlatformPrivate method.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContextPlatformPrivate::clipBoundingRect): (WebCore::GraphicsContext::beginTransparencyLayer): (WebCore::GraphicsContext::clipOut): (WebCore::GraphicsContext::clipOutEllipseInRect):
17:30 Changeset [65790] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/efl

2010-08-22 Rafael Antognolli <antognolli@profusion.mobi>

Reviewed by Antonio Gomes.

[EFL] Remove duplicated include entry from default theme.
https://bugs.webkit.org/show_bug.cgi?id=44345

This used to create the group webkit/widget/entry twice in
the theme. It's leading to a bug with the newer versions of
EFL.

  • DefaultTheme/default.edc:
17:14 Changeset [65789] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-08-22 Marc Mutz <marc.mutz@kdab.com>

Reviewed by Antonio Gomes.

[Qt] Actually emit the QWebView::selectionChanged() signal

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

  • Api/qwebview.cpp: (QWebView::setPage): Connect QWebPage::selectionChanged() to QWebView::selectionChanged()
15:42 Changeset [65788] by commit-queue@webkit.org
  • 18 edits in trunk

2010-08-22 Juha Savolainen <juha.savolainen@weego.fi>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add QTouchEvents support for WebKit2
https://bugs.webkit.org/show_bug.cgi?id=44330

This patch adds support for QTouchEvents in WebKit2. A new WebEvent-class
is created for touchevents, WebTouchEvent. Also touchpoints needed a new
class, WebPlatformTouchPoint. This is similar solution like in other
events(like MouseEvent) on WebKit2. These classes are introduced in WebEvent.h
Also there was a need to create an empty constructor to PlatformTouchPoint-class.

  • platform/PlatformTouchPoint.h: (WebCore::PlatformTouchPoint::PlatformTouchPoint):

2010-08-22 Juha Savolainen <juha.savolainen@weego.fi>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add QTouchEvents support for WebKit2
https://bugs.webkit.org/show_bug.cgi?id=44330

This patch adds support for QTouchEvents in WebKit2. A new WebEvent-class
is created for touchevents, WebTouchEvent. Also touchpoints needed a new
class, WebPlatformTouchPoint. This is similar solution like in other
events(like MouseEvent) on WebKit2. These classes are introduced in WebEvent.h
Also there was a need to create an empty constructor to PlatformTouchPoint-class.

  • DerivedSources.pro:
  • Shared/CoreIPCSupport/WebPageMessageKinds.h: (WebPageMessage::):
  • Shared/WebEvent.h: (WebKit::WebEvent::): (WebKit::WebTouchPoint::): (WebKit::WebTouchPoint::WebTouchPoint): (WebKit::WebTouchPoint::id): (WebKit::WebTouchPoint::state): (WebKit::WebTouchPoint::screenPosX): (WebKit::WebTouchPoint::screenPosY): (WebKit::WebTouchPoint::posX): (WebKit::WebTouchPoint::posY): (WebKit::WebTouchPoint::setState): (WebKit::WebTouchPoint::encode): (WebKit::WebTouchPoint::decode): (WebKit::WebTouchEvent::WebTouchEvent): (WebKit::WebTouchEvent::type): (WebKit::WebTouchEvent::touchPoints): (WebKit::WebTouchEvent::encode): (WebKit::WebTouchEvent::decode): (WebKit::WebTouchEvent::isTouchEventType):
  • Shared/WebEventConversion.cpp: (WebKit::WebKit2PlatformTouchPoint::WebKit2PlatformTouchPoint): (WebKit::WebKit2PlatformTouchEvent::WebKit2PlatformTouchEvent): (WebKit::platform):
  • Shared/WebEventConversion.h:
  • Shared/qt/WebEventFactoryQt.cpp: (WebKit::webEventTypeForEvent): (WebKit::WebEventFactory::createWebTouchEvent):
  • Shared/qt/WebEventFactoryQt.h:
  • UIProcess/API/qt/qgraphicswkview.cpp: (QGraphicsWKView::event): (QGraphicsWKView::touchEvent):
  • UIProcess/API/qt/qgraphicswkview.h:
  • UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::touchEvent):
  • UIProcess/API/qt/qwkpage_p.h:
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::touchEvent):
  • UIProcess/WebPageProxy.h:
  • WebProcess/WebPage/WebPage.cpp: (WebKit::WebPage::touchEvent): (WebKit::WebPage::didReceiveMessage):
  • WebProcess/WebPage/WebPage.h:
14:41 Changeset [65787] by dbates@webkit.org
  • 29 edits
    2 adds in trunk

2010-08-22 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

Encapsulate document marker management into DocumentMarkerController
https://bugs.webkit.org/show_bug.cgi?id=44383

Modify call sites in the Chromium port to use DocumentMarkerController.

No functionality was changed, so no new tests.

  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::stopFinding): (WebKit::WebFrameImpl::addMarker): (WebKit::WebFrameImpl::setMarkerActive):

2010-08-22 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

Encapsulate document marker management into DocumentMarkerController
https://bugs.webkit.org/show_bug.cgi?id=44383

Modify call sites in the Apple Windows port to use DocumentMarkerController.

No functionality was changed, so no new tests.

  • WebFrame.cpp: (WebFrame::unmarkAllMisspellings): (WebFrame::unmarkAllBadGrammar):
  • WebView.cpp: (WebView::rectsForTextMatches):

2010-08-22 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

Encapsulate document marker management into DocumentMarkerController
https://bugs.webkit.org/show_bug.cgi?id=44383

Modify call sites in the Apple Mac port to use DocumentMarkerController.

No functionality was changed, so no new tests.

  • WebView/WebFrame.mm: (-[WebFrame _unmarkAllBadGrammar]): (-[WebFrame _unmarkAllMisspellings]):
  • WebView/WebHTMLView.mm: (-[WebHTMLView unmarkAllTextMatches]): (-[WebHTMLView rectsForTextMatches]):

2010-08-22 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

Encapsulate document marker management into DocumentMarkerController
https://bugs.webkit.org/show_bug.cgi?id=44383

Modify call sites in the EFL port to use DocumentMarkerController.

No functionality was changed, so no new tests.

  • ewk/ewk_frame.cpp: (ewk_frame_text_matches_unmark_all):

2010-08-22 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

Encapsulate document marker management into DocumentMarkerController
https://bugs.webkit.org/show_bug.cgi?id=44383

Moves the document marker management code in Document.cpp into its own
class called DocumentMarkerController.

No functionality was changed, so no new tests.

  • Android.mk: Added DocumentMarkerController.cpp
  • CMakeLists.txt: Ditto.
  • GNUmakefile.am: Added DocumentMarkerController.cpp and DocumentMarkerController.h.
  • WebCore.exp.in: Substituted symbols ZN7WebCore24DocumentMarkerController13removeMarkersENS_14DocumentMarker10MarkerTypeE and ZN7WebCore24DocumentMarkerController23renderedRectsForMarkersENS_14DocumentMarker10MarkerTypeE for ZN7WebCore8Document13removeMarkersENS_14DocumentMarker10MarkerTypeE and ZN7WebCore8Document23renderedRectsForMarkersENS_14DocumentMarker10MarkerTypeE, respectively.
  • WebCore.gypi: Added DocumentMarkerController.cpp and DocumentMarkerController.h.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • dom/Document.cpp: (WebCore::Document::removedLastRef): Modified to call DocumentMarkerController::detach(). (WebCore::Document::~Document): Removed "deleteAllValues(m_markers)" as DocumentMarkerController is stored in OwnPtr; so it will be destroyed automatically on Document destruction. (WebCore::Document::textInserted): Modified to use marker controller. (WebCore::Document::textRemoved): Ditto.
  • dom/Document.h: (WebCore::Document::markers): Added.
  • dom/DocumentMarkerController.cpp: Added. (WebCore::placeholderRectForMarker): (WebCore::DocumentMarkerController::detach): (WebCore::DocumentMarkerController::addMarker): (WebCore::DocumentMarkerController::removeMarkers): (WebCore::DocumentMarkerController::copyMarkers): (WebCore::DocumentMarkerController::markerContainingPoint): (WebCore::DocumentMarkerController::markersForNode): (WebCore::DocumentMarkerController::renderedRectsForMarkers): (WebCore::DocumentMarkerController::repaintMarkers): (WebCore::DocumentMarkerController::setRenderedRectForMarker): (WebCore::DocumentMarkerController::invalidateRenderedRectsForMarkersInRect): (WebCore::DocumentMarkerController::shiftMarkers): (WebCore::DocumentMarkerController::setMarkersActive):
  • dom/DocumentMarkerController.h: Added. (WebCore::DocumentMarkerController::~DocumentMarkerController):
  • editing/Editor.cpp: (WebCore::Editor::ignoreSpelling): Modified to use marker controller. (WebCore::findFirstMisspellingInRange): Ditto. (WebCore::findFirstGrammarDetailInRange): Ditto. (WebCore::Editor::advanceToNextMisspelling): Ditto. (WebCore::Editor::markAllMisspellingsAndBadGrammarInRanges): Ditto. (WebCore::Editor::changeBackToReplacedString): Ditto.
  • editing/SplitTextNodeCommand.cpp: (WebCore::SplitTextNodeCommand::doApply): Ditto. (WebCore::SplitTextNodeCommand::doUnapply): Ditto.
  • page/Frame.cpp: (WebCore::Frame::markAllMatchesForText): Ditto. (WebCore::Frame::setMarkedTextMatchesAreHighlighted): Ditto. (WebCore::Frame::respondToChangedSelection): Ditto.
  • page/FrameView.cpp: (WebCore::FrameView::getTickmarks): Ditto. (WebCore::FrameView::paintContents): Ditto.
  • page/Page.cpp: (WebCore::Page::unmarkAllTextMatches): Ditto.
  • rendering/HitTestResult.cpp: (WebCore::HitTestResult::spellingToolTip): Ditto. (WebCore::HitTestResult::replacedString): Ditto.
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): Ditto. (WebCore::InlineTextBox::paintTextMatchMarker): Ditto. (WebCore::InlineTextBox::computeRectForReplacementMarker): Ditto. (WebCore::InlineTextBox::paintDocumentMarkers): Ditto.
  • rendering/SVGInlineTextBox.cpp: (WebCore::SVGInlineTextBox::computeTextMatchMarkerRect): Ditto.
14:32 Changeset [65786] by jianli@chromium.org
  • 6 edits in trunk

Add the blob URL member to FormData.
https://bugs.webkit.org/show_bug.cgi?id=44387

Reviewed by Darin Fisher.

WebCore:

This is in preparation to switch blob implementation to using BlobData
model. When a blob is added to a FormData, it is represented as a blob
URL in the list.

  • platform/network/FormData.cpp:

(WebCore::FormData::appendBlob):

  • platform/network/FormData.h:

(WebCore::FormDataElement::FormDataElement):
(WebCore::FormDataElement::):
(WebCore::operator==):

WebKit/chromium:

As the result of adding the blob URL member to the FormData, we need
to update the corresponding WebKit API for chromium.

  • public/WebHTTPBody.h:

(WebKit::WebHTTPBody::Element::):

  • src/WebHTTPBody.cpp:

(WebKit::WebHTTPBody::elementAt):
(WebKit::WebHTTPBody::appendBlob):

11:38 Changeset [65785] by eric@webkit.org
  • 4 edits in trunk

2010-08-22 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

HTML5 TreeBuilder builds wrong DOM for <a><svg><tr><input></a>
https://bugs.webkit.org/show_bug.cgi?id=44390

  • html5lib/resources/adoption01.dat:

2010-08-22 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

HTML5 TreeBuilder builds wrong DOM for <a><svg><tr><input></a>
https://bugs.webkit.org/show_bug.cgi?id=44390

The HTML5 spec has changed since Adam and I original wrote
the HTMLTreeBuilder. Most important for this change was resolution of:
http://www.w3.org/Bugs/Public/show_bug.cgi?id=9580

I also removed our "phrasing" tag support since that was also removed
from the spec as part of other bug fixes.

This is tested by tonyg's <a><svg><tr><input></a> test in adoption01.dat.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processCloseWhenNestedTag): (WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody): (WebCore::HTMLTreeBuilder::furthestBlockForFormattingElement): (WebCore::HTMLTreeBuilder::processEndTag):
05:12 Changeset [65784] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-08-22 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] TransparencyLayer: Always use anti-aliasing and smooth pixmap transform
https://bugs.webkit.org/show_bug.cgi?id=44394

  • platform/graphics/qt/TransparencyLayer.h: (WebCore::TransparencyLayer::TransparencyLayer): Set the Antialiasing and SmoothPixmapTransforms render hints for all TransparencyLayers.
01:38 Changeset [65783] by andreas.kling@nokia.com
  • 4 edits in trunk

2010-08-22 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Gradient: Support inner radius larger than outer radius
https://bugs.webkit.org/show_bug.cgi?id=44392

  • platform/graphics/qt/GradientQt.cpp: (WebCore::Gradient::platformGradient): Reverse the (radial) gradient if r0 > r1.

2010-08-22 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Gradient: Support inner radius larger than outer radius
https://bugs.webkit.org/show_bug.cgi?id=44392

Unskip two tests that now pass:

  • canvas/philip/tests/2d.gradient.radial.outside2.html
  • canvas/philip/tests/2d.gradient.radial.outside3.html
  • platform/qt/Skipped:
00:30 Changeset [65782] by ariya@webkit.org
  • 5 edits in trunk/WebCore

[Qt] Layer approach to support generic shadow handling
https://bugs.webkit.org/show_bug.cgi?id=44380

Patch by Ariya Hidayat <ariya@sencha.com> on 2010-08-22
Reviewed by Dirk Schulze.

The pair beginShadowLayer and endShadowLayer creates a temporary image
where the caller can draw onto, using the returned QPainter. When
endShadowLayer is called, the temporary image will be filtered, using
the specified shadow color and blur radius, and drawn to the graphics
context.

  • platform/graphics/GraphicsContext.h:
  • platform/graphics/qt/ContextShadow.cpp:

(WebCore::ContextShadow::beginShadowLayer):
(WebCore::ContextShadow::endShadowLayer):

  • platform/graphics/qt/ContextShadow.h:
  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::fillRect):
(WebCore::GraphicsContext::contextShadow):

08/21/10:

20:01 Changeset [65781] by weinig@apple.com
  • 6 edits in trunk/WebKitTools

Check in missing files.

18:58 Changeset [65780] by weinig@apple.com
  • 44 edits in trunk

Remove specialized retain/release functions in favor of a single one for WKTypeRefs
https://bugs.webkit.org/show_bug.cgi?id=44386

Reviewed by Dan Bernstein.

WebKit2:

  • UIProcess/API/C/WKArray.cpp:

(WKArrayGetSize):

  • UIProcess/API/C/WKArray.h:
  • UIProcess/API/C/WKBackForwardList.cpp:

(WKBackForwardListCopyForwardListWithLimit):

  • UIProcess/API/C/WKBackForwardList.h:
  • UIProcess/API/C/WKBackForwardListItem.cpp:

(WKBackForwardListItemCopyTitle):

  • UIProcess/API/C/WKBackForwardListItem.h:
  • UIProcess/API/C/WKBase.h:
  • UIProcess/API/C/WKContext.cpp:
  • UIProcess/API/C/WKContext.h:
  • UIProcess/API/C/WKData.cpp:

(WKDataGetSize):

  • UIProcess/API/C/WKData.h:
  • UIProcess/API/C/WKDictionary.cpp:

(WKDictionaryCopyKeys):

  • UIProcess/API/C/WKDictionary.h:
  • UIProcess/API/C/WKError.h:
  • UIProcess/API/C/WKFormSubmissionListener.cpp:

(WKFormSubmissionListenerContinue):

  • UIProcess/API/C/WKFormSubmissionListener.h:
  • UIProcess/API/C/WKFrame.cpp:

(WKFrameGetPage):

  • UIProcess/API/C/WKFrame.h:
  • UIProcess/API/C/WKFramePolicyListener.cpp:

(WKFramePolicyListenerIgnore):

  • UIProcess/API/C/WKFramePolicyListener.h:
  • UIProcess/API/C/WKNavigationData.cpp:

(WKNavigationDataCopyURL):

  • UIProcess/API/C/WKNavigationData.h:
  • UIProcess/API/C/WKPage.cpp:
  • UIProcess/API/C/WKPage.h:
  • UIProcess/API/C/WKPageNamespace.cpp:

(WKPageNamespaceGetContext):

  • UIProcess/API/C/WKPageNamespace.h:
  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesCreate):
(WKPreferencesGetLocalStorageEnabled):

  • UIProcess/API/C/WKPreferences.h:
  • UIProcess/API/C/WKString.cpp:
  • UIProcess/API/C/WKString.h:
  • UIProcess/API/C/WKType.cpp:

(WKRetain):
(WKRelease):

  • UIProcess/API/C/WKType.h:
  • UIProcess/API/C/WKURL.cpp:

(WKURLGetTypeID):

  • UIProcess/API/C/WKURL.h:
  • UIProcess/API/cpp/WKRetainPtr.h:
  • UIProcess/API/win/WKView.cpp:
  • UIProcess/API/win/WKView.h:
  • UIProcess/WebPreferences.cpp:

(WebKit::WebPreferences::shared):

  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandle.cpp:

(WKBundleNodeHandleGetTypeID):

  • WebProcess/InjectedBundle/API/c/WKBundleNodeHandle.h:
  • WebProcess/InjectedBundle/API/c/WKBundleScriptWorld.cpp:

(WKBundleScriptWorldNormalWorld):

  • WebProcess/InjectedBundle/API/c/WKBundleScriptWorld.h:

WebKitTools:

  • MiniBrowser/mac/AppDelegate.m:

(didRecieveMessageFromInjectedBundle):
(didNavigateWithNavigationData):
(-[BrowserAppDelegate init]):
(-[BrowserAppDelegate applicationWillTerminate:]):

  • MiniBrowser/mac/BrowserStatisticsWindowController.m:

(-[BrowserStatisticsWindowController initWithThreadedWKContextRef:processWKContextRef:]):
(-[BrowserStatisticsWindowController dealloc]):

  • MiniBrowser/mac/BrowserWindowController.m:

(-[BrowserWindowController initWithPageNamespace:]):
(-[BrowserWindowController fetch:]):
(-[BrowserWindowController windowWillClose:]):
(-[BrowserWindowController applicationTerminating]):
(closePage):
(runJavaScriptAlert):
(runJavaScriptConfirm):
(runJavaScriptPrompt):
(-[BrowserWindowController updateProvisionalURLForFrame:]):

  • MiniBrowser/mac/WebBundle/WebBundleMain.m:

(didClearWindowObjectForFrame):

  • MiniBrowser/win/BrowserView.cpp:

(BrowserView::goToURL):

  • WebKitTestRunner/win/PlatformWebViewWin.cpp:

(WTR::PlatformWebView::~PlatformWebView):

11:40 Changeset [65779] by eric.carlson@apple.com
  • 6 edits in trunk

2010-08-21 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media engine should not be asked to open all urls
https://bugs.webkit.org/show_bug.cgi?id=44370

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::load): Do nothing with urls with a type attribute that would be rejected by canPlayType().

2010-08-21 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media engine should not be asked to open all urls
https://bugs.webkit.org/show_bug.cgi?id=44370

  • media/video-source-none-supported-expected.txt:
  • media/video-source-none-supported.html:
  • media/video-source-type.html:
03:11 Changeset [65778] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-21 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] http/tests/incremental should be enabled
https://bugs.webkit.org/show_bug.cgi?id=44284

  • platform/qt/Skipped: Enable http/tests/incremental, disabling only one failing test (#44282)
02:23 Changeset [65777] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-21 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] All sputnik/Conformance/10_Execution_Contexts/10.2_Entering_An_Execution_Context/10.2.2_Eval_Code/ tests can be re-enabled
https://bugs.webkit.org/show_bug.cgi?id=44192

Tested on several scenarios, including qt-4.7 and qt-4.6

  • platform/qt/Skipped:
01:33 Changeset [65776] by ariya@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Faster bounding rect for the shadow clip region
https://bugs.webkit.org/show_bug.cgi?id=44369

Patch by Ariya Hidayat <ariya@sencha.com> on 2010-08-21
Reviewed by Kenneth Rohde Christiansen.

Like in r65650, we should use the faster QPainter::clipBoundingRect
when it is available.

  • platform/graphics/qt/ContextShadow.cpp:

(WebCore::ContextShadow::drawShadowRect):

08/20/10:

22:14 Changeset [65775] by girish@forwardbias.in
  • 2 edits in trunk/WebCore

[Qt] When using the raster graphics system on Maemo5, allow
Flash to render directly into the raster window surface.
wmode=transparent is now supported as a result of this change.

Reviewed by Ariya Hidayat.

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

  • plugins/qt/PluginViewQt.cpp:

(WebCore::PluginView::paintUsingImageSurfaceExtension):

22:06 Changeset [65774] by ap@apple.com
  • 3 edits
    3 adds in trunk

2010-08-20 Alexey Proskuryakov <ap@apple.com>

Reviewed by Brady Eidson.

https://bugs.webkit.org/show_bug.cgi?id=38428
HTTP code 500 (and other non-4xx codes) wrongfully treated as success for subresources

  • http/tests/misc/resources/script-500.php: Added.
  • http/tests/misc/script-500-expected.txt: Added.
  • http/tests/misc/script-500.html: Added.

2010-08-20 Alexey Proskuryakov <ap@apple.com>

Reviewed by Brady Eidson.

https://bugs.webkit.org/show_bug.cgi?id=38428
HTTP code 500 (and other non-4xx codes) wrongfully treated as success for subresources

Test: http/tests/misc/script-500.html

  • loader/loader.cpp: (WebCore::Loader::Host::didReceiveData): Treat all HTTP codes >= 400 as error ones, since they are.
20:39 Changeset [65773] by andreas.kling@nokia.com
  • 2 edits in trunk/WebKitTools

2010-08-20 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

WebKit does not build with Python 2.3
https://bugs.webkit.org/show_bug.cgi?id=44373

Replaced the function sorted() by a call to sort()
on the entry list.
The function sorted() has been added to Python 2.4.
The function method sort() of List works with older
versions of Python.

  • Scripts/create-html-entity-table:
18:18 Changeset [65772] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-20 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] Enable some http/ tests which are passing for some time
https://bugs.webkit.org/show_bug.cgi?id=44366

These are reported as "passing, require confirmation on bot". Some of them
fail on my machines, so I'm enabling only the ones which pass.

  • platform/qt/Skipped: enable several http tests
18:01 Changeset [65771] by commit-queue@webkit.org
  • 7 edits in trunk/WebKit/qt

2010-08-20 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>

Reviewed by Ariya Hidayat.

[Qt] constify core() and kit() parameters
https://bugs.webkit.org/show_bug.cgi?id=44347

  • Api/qwebframe.cpp: (QWebFrame::evaluateJavaScript):
  • Api/qwebframe_p.h:
  • Api/qwebhistory.cpp: (QWebHistoryItemPrivate::core):
  • Api/qwebhistory_p.h:
  • Api/qwebpage.cpp: (QWebPagePrivate::core):
  • Api/qwebpage_p.h:
17:35 Changeset [65770] by kinuko@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-08-20 Kinuko Yasuda <kinuko@chromium.org>

Unreviewed; build fix for chromium (and remove duplicated ChangeLog entry).

17:25 Changeset [65769] by tonyg@chromium.org
  • 10 edits
    4 adds in trunk

2010-08-20 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Adam Barth.

Crash in WebCore::Node::createRendererIfNeeded()
https://bugs.webkit.org/show_bug.cgi?id=44129

  • html5lib/resources/adoption02.dat: A crazy DOM from https://bugs.webkig.org/show_bug.cgi?id=44170 which previously crashed. The expectation doesn't appear to be correct, but it behaves the same in Firefox 4, so I'm going to file a spec bug.
  • fast/parser/remove-misnested-iframe-in-beforeload-expected.txt:
  • fast/parser/remove-misnested-iframe-in-beforeload.html: Misnest a node in a table and remove it during its beforeload handler. The node should not be in the tree.
  • fast/parser/remove-misnested-iframe-parent-in-beforeload-expected.txt:
  • fast/parser/remove-misnested-iframe-parent-in-beforeload.html: Misnest a node in a table and remove its parent during its beforeload handler. The adoption agency should have already changed its parent to the one before the table and it and its parent should be removed.

2010-08-20 Tony Gentilcore <tonyg@chromium.org>

Reviewed by Adam Barth.

Crash in WebCore::Node::createRendererIfNeeded()
https://bugs.webkit.org/show_bug.cgi?id=44129

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::insertBefore): Factor out core bit to insertBetween. (WebCore::ContainerNode::insertBetween): Factored out of insertBefore. (WebCore::ContainerNode::parserInsertBefore): Similar to insertBefore, but doesn't handle reparenting or dispatch DOM mutation events. (WebCore::ContainerNode::removeChild): Factor out core bit to removeBetween. (WebCore::ContainerNode::removeBetween): Facotred out of removeChild. (WebCore::ContainerNode::parserRemoveChild): Similar to removeChild, but doesn't handle reparenting or dispatch DOM mutation events. (WebCore::ContainerNode::addChildCommon): (WebCore::ContainerNode::parserAddChild): (WebCore::ContainerNode::legacyParserAddChild):
  • dom/ContainerNode.h:
  • dom/Node.cpp: (WebCore::Node::parserRemoveChild): (WebCore::Node::parserInsertBefore):
  • dom/Node.h:
  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::attach): (WebCore::HTMLConstructionSite::attachAtSite): Use new parserInsertBefore.
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::passTokenToLegacyParser): (WebCore::HTMLTreeBuilder::reparentChildren): (WebCore::HTMLTreeBuilder::callTheAdoptionAgency): Was missing a key bit from the spec about removing the old parent if it exists.
17:18 Changeset [65768] by kinuko@chromium.org
  • 2 edits in trunk/WebCore

2010-08-20 Kinuko Yasuda <kinuko@chromium.org>

Unreviewed; build fix. Had included wrong version of build file.

17:13 Changeset [65767] by mrobinson@webkit.org
  • 4 edits in trunk

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Joseph Pecoraro.

[GTK] Inspector extensions tests fail on GTK+ bots because onSelectionChanged is missing
https://bugs.webkit.org/show_bug.cgi?id=44342

  • platform/gtk/Skipped: Unskip tests which are now passing.

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Joseph Pecoraro.

[GTK] Inspector extensions tests fail on GTK+ bots because onSelectionChanged is missing
https://bugs.webkit.org/show_bug.cgi?id=44342

  • GNUmakefile.am: Fix dependency tracking for copied web inspector files.
17:04 Changeset [65766] by jamesr@google.com
  • 2 edits in trunk/WebKit/chromium

2010-08-20 James Robinson <jamesr@chromium.org>

Synchronize default of use_accelerated_compositing gyp variable with downstream. Unreviewed.

  • features.gypi:
17:02 Changeset [65765] by kinuko@chromium.org
  • 2 edits
    2 adds in trunk/WebKit/chromium

2010-08-20 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Darin Fisher.

[chromium] Add chromium-side callback implementation for FileSystem API
https://bugs.webkit.org/show_bug.cgi?id=44350

Add WebFileSystemCallbacks that calls back the WebCore's implementation.

  • WebKit.gyp:
  • src/WebFileSystemCallbacksImpl.cpp: Added.
  • src/WebFileSystemCallbacksImpl.h: Added.
16:59 Changeset [65764] by crogers@google.com
  • 4 edits
    6 adds in branches/audio/WebCore

Split out HRTFPanner file into: HRTFKernel, HRTFElevation, HRTFDatabase, HRTFPanner files

16:49 Changeset [65763] by kinuko@chromium.org
  • 9 edits
    2 adds in trunk/WebCore

2010-08-20 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Darin Fisher.

Add Callback implementation for FileSystem API
https://bugs.webkit.org/show_bug.cgi?id=44349

No new tests; tests will be added later.

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • storage/FileSystemCallbacks.cpp: Added.
  • storage/FileSystemCallbacks.h: Added.
  • storage/EntryCallback.h: Nits fix
  • storage/FileSystemCallback.h: Nits fix
16:48 Changeset [65762] by rniwa@webkit.org
  • 1 edit
    3 deletes in trunk/LayoutTests

2010-08-20 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65757.
http://trac.webkit.org/changeset/65757
https://bugs.webkit.org/show_bug.cgi?id=44371

"fails on chromium windows and there is also a bug in the test
script." (Requested by rniwa on #webkit).

  • editing/style/make-text-writing-direction-inline-expected.txt: Removed.
  • editing/style/make-text-writing-direction-inline.html: Removed.
  • editing/style/script-tests/make-text-writing-direction-inline.js: Removed.
16:23 Changeset [65761] by andreas.kling@nokia.com
  • 3 edits in trunk/WebCore

2010-08-20 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] WebKit does not compile with --3d-canvas
https://bugs.webkit.org/show_bug.cgi?id=44335

  • PlatformGLObject => Platform3DObject
  • Implemented getAttachedShaders()
  • Implemented bufferData() and bufferSubData() for ArrayBuffer*
  • Removed reference to nonexistent variable in getImageData()
  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal): (WebCore::GraphicsContext3D::attachShader): (WebCore::GraphicsContext3D::getAttachedShaders): (WebCore::GraphicsContext3D::bindAttribLocation): (WebCore::GraphicsContext3D::bindBuffer): (WebCore::GraphicsContext3D::bindFramebuffer): (WebCore::GraphicsContext3D::bindRenderbuffer): (WebCore::GraphicsContext3D::bindTexture): (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData): (WebCore::GraphicsContext3D::compileShader): (WebCore::GraphicsContext3D::detachShader): (WebCore::GraphicsContext3D::framebufferRenderbuffer): (WebCore::GraphicsContext3D::framebufferTexture2D): (WebCore::GraphicsContext3D::getActiveAttrib): (WebCore::GraphicsContext3D::getActiveUniform): (WebCore::GraphicsContext3D::getAttribLocation): (WebCore::GraphicsContext3D::isBuffer): (WebCore::GraphicsContext3D::isFramebuffer): (WebCore::GraphicsContext3D::isProgram): (WebCore::GraphicsContext3D::isRenderbuffer): (WebCore::GraphicsContext3D::isShader): (WebCore::GraphicsContext3D::isTexture): (WebCore::GraphicsContext3D::linkProgram): (WebCore::GraphicsContext3D::shaderSource): (WebCore::GraphicsContext3D::useProgram): (WebCore::GraphicsContext3D::validateProgram): (WebCore::GraphicsContext3D::getProgramiv): (WebCore::GraphicsContext3D::getProgramInfoLog): (WebCore::GraphicsContext3D::getShaderiv): (WebCore::GraphicsContext3D::getShaderInfoLog): (WebCore::GraphicsContext3D::getShaderSource): (WebCore::GraphicsContext3D::getUniformfv): (WebCore::GraphicsContext3D::getUniformiv): (WebCore::GraphicsContext3D::getUniformLocation): (WebCore::GraphicsContext3D::getImageData):
  • platform/graphics/qt/GraphicsLayerQt.h:
16:03 Changeset [65760] by mitz@apple.com
  • 2 edits in trunk/WebCore

#ifdef-out some Leopard-and-earlier code.

Reviewed by Dave Kilzer.

  • platform/graphics/mac/FontCustomPlatformData.cpp:

(WebCore::FontCustomPlatformData::~FontCustomPlatformData):

15:43 Changeset [65759] by kbr@google.com
  • 3 edits in trunk/WebCore

2010-08-20 Kenneth Russell <kbr@google.com>

Unreviewed, speculative Chromium build fix. Forward declare
IntSize in WebGLRenderingContext.h and include IntSize.h in .cpp.

  • html/canvas/WebGLRenderingContext.cpp:
  • html/canvas/WebGLRenderingContext.h:
15:19 Changeset [65758] by eric.carlson@apple.com
  • 3 edits
    2 adds in trunk

2010-08-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media element canPlayType("application/octet-stream") not handled correctly
https://bugs.webkit.org/show_bug.cgi?id=44343

Test: media/media-can-play-octet-stream.html

  • platform/graphics/MediaPlayer.cpp: (WebCore::applicationOctetStream): New, accessor for static string used more than once. (WebCore::textPlain): Ditto. (WebCore::codecs): Ditto. (WebCore::chooseBestEngineForTypeAndCodecs): Special case "application/octet-stream". (WebCore::MediaPlayer::load): Use static static string methods. (WebCore::MediaPlayer::supportsType): Special case "application/octet-stream".

2010-08-20 Eric Carlson <eric.carlson@apple.com>

Reviewed by Dan Bernstein.

Media element canPlayType("application/octet-stream") not handled correctly
https://bugs.webkit.org/show_bug.cgi?id=44343

  • media/media-can-play-octet-stream-expected.txt: Added.
  • media/media-can-play-octet-stream.html: Added.
15:14 Changeset [65757] by rniwa@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

2010-08-20 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

There is no test for unicode-bidi / direction support in ApplyStyleCommand::applyInlineStyle
https://bugs.webkit.org/show_bug.cgi?id=43989

Added a test for MakeTextWritingDirectionNatural, MakeTextWritingDirectionLeftToRight,
and MakeTextWritingDirectionRightToLeft commands.

  • editing/style/make-text-writing-direction-inline.html: Added.
  • editing/style/script-tests/make-text-writing-direction-inline.js: Added. (modifyWritingDirection): (selectFirstWord): (selectSecondAndThirdWords):
15:12 Changeset [65756] by kbr@google.com
  • 4 edits
    4 adds in trunk

2010-08-20 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

Implement HTMLVideoElement support for texImage2D and texSubImage2D
https://bugs.webkit.org/show_bug.cgi?id=33852

Test: fast/canvas/webgl/tex-image-and-sub-image-2d-with-video.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::WebGLRenderingContext): (WebCore::WebGLRenderingContext::videoFrameToImage): (WebCore::WebGLRenderingContext::texImage2D): (WebCore::WebGLRenderingContext::texSubImage2D): (WebCore::WebGLRenderingContext::LRUImageBufferCache::LRUImageBufferCache): (WebCore::WebGLRenderingContext::LRUImageBufferCache::imageBuffer): (WebCore::WebGLRenderingContext::LRUImageBufferCache::bubbleToFront):
  • html/canvas/WebGLRenderingContext.h:

2010-08-20 Adrienne Walker <enne@google.com>

Reviewed by Kenneth Russell.

Implement HTMLVideoElement support for texImage2D and texSubImage2D
https://bugs.webkit.org/show_bug.cgi?id=33852

  • fast/canvas/webgl/resources/quadrants.mp4: Added.
  • fast/canvas/webgl/resources/quadrants.ogv: Added.
  • fast/canvas/webgl/tex-image-and-sub-image-2d-with-video-expected.txt: Added.
  • fast/canvas/webgl/tex-image-and-sub-image-2d-with-video.html: Added.
14:54 Changeset [65755] by dumi@chromium.org
  • 3 edits
    2 adds in trunk

WebCore: Removing an incorrect ASSERT.
https://bugs.webkit.org/show_bug.cgi?id=44151

Reviewed by Adam Barth.

A failure in the preflight step of a transaction wrapper does not
guarantee that the transaction is rolled back (and it shouldn't).

Test: storage/change-version-no-crash-on-preflight-failure.html

  • storage/SQLTransaction.cpp:

(WebCore::SQLTransaction::openTransactionAndPreflight):

LayoutTests: Check that a failure in the preflight step of a transaction wrapper does not trigger any assertion.
https://bugs.webkit.org/show_bug.cgi?id=44151

Reviewed by Adam Barth.

  • storage/change-version-no-crash-on-preflight-failure-expected.txt: Added.
  • storage/change-version-no-crash-on-preflight-failure.html: Added.
14:44 Changeset [65754] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-08-20 David Leong <david.leong@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Update Symbian capabilities to remove AllFiles and DRM

Add CONFIG(production) variation for set of DLL capabilities.
DRM and AllFiles capabilities need to be audited for the sis
package to be signed.

No new tests.

  • WebCore.pro: Updated Symbian Capabilities.
14:14 Changeset [65753] by joepeck@webkit.org
  • 2 edits in trunk/WebCore

2010-08-20 Joseph Pecoraro <joepeck@webkit.org>

Reviewed by David Kilzer.

Do Not Copy Subversion Related Files in Inspector Build Phase
https://bugs.webkit.org/show_bug.cgi?id=44353

  • WebCore.xcodeproj/project.pbxproj: remove .svn directories in the "Copy Inspector Resources" build phase.
14:03 Changeset [65752] by simon.fraser@apple.com
  • 3 edits
    4 adds in trunk

2010-08-20 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

<rdar://problem/8245719> backface-visibility and reflections don't play nicely together.

On elements with a reflection and backface-visibility: hidden, set the doubleSided property on the
reflection flattening layer so that backface-visibility works in all configurations.

Test: compositing/reflections/backface-hidden-reflection.html

  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::updateBackfaceVisibility): Call setDoubleSided: on the structural layer for reflections. (WebCore::GraphicsLayerCA::ensureStructuralLayer): Fix a comment typo, and call updateBackfaceVisibility() when we gain a structural layer.
13:37 Changeset [65751] by commit-queue@webkit.org
  • 6 edits in trunk/WebKit2

2010-08-20 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Antonio Gomes.

[Qt] Implement cursor change in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=44263

Implemented PageClient::setCursor.

  • UIProcess/API/qt/qgraphicswkview.cpp: Added updateCursor slot to be connected to the QWKPage::setCursor signal. (QGraphicsWKView::QGraphicsWKView): (QGraphicsWKView::updateCursor): Sets the cursor of the widget.
  • UIProcess/API/qt/qgraphicswkview.h:
  • UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::setCursor): Added implementation for PageClient::setCursor. Emitting singal through the QWKPage.
  • UIProcess/API/qt/qwkpage.h: Added setCursor signal.
  • UIProcess/API/qt/qwkpage_p.h:
13:19 Changeset [65750] by johnnyg@google.com
  • 1 edit
    3 adds in trunk/LayoutTests

2010-08-20 John Gregg <johnnyg@google.com>

Unreviewed, chromium rebaseline.

New test was added in r65746, it requires a new chromium-linux baseline.

  • platform/chromium-linux/fast/repaint/multicol-repaint-expected.checksum: Added.
  • platform/chromium-linux/fast/repaint/multicol-repaint-expected.png: Added.
  • platform/chromium-linux/fast/repaint/multicol-repaint-expected.txt: Added.
12:44 Changeset [65749] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

2010-08-19 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Adam Barth.

style correction in markup.cpp
https://bugs.webkit.org/show_bug.cgi?id=44318

Make markup.cpp pass check-webkit-style.

No new tests are added since this is a cleanup.

  • editing/markup.cpp: (WebCore::MarkupAccumulator::appendAttributeValue): (WebCore::appendEscapedContent): (WebCore::MarkupAccumulator::appendStartMarkup): (WebCore::completeURLs): (WebCore::isElementPresentational): (WebCore::isSpecialAncestorBlock): (WebCore::shouldIncludeWrapperForFullySelectedRoot): (WebCore::createMarkup): (WebCore::fillContainerFromString): (WebCore::createFragmentFromText): (WebCore::createFragmentFromNodes):
12:23 Changeset [65748] by tony@chromium.org
  • 3 edits
    2 adds in trunk

2010-08-20 Tony Chang <tony@chromium.org>

Reviewed by Adam Barth.

crash when trying to access a stale Node pointer in FocusController::setFocusedNode
https://bugs.webkit.org/show_bug.cgi?id=44226

  • fast/events/focus-change-crash2-expected.txt: Added.
  • fast/events/focus-change-crash2.html: Added.

2010-08-20 Tony Chang <tony@chromium.org>

Reviewed by Adam Barth.

crash when trying to access a stale Node pointer in FocusController::setFocusedNode
https://bugs.webkit.org/show_bug.cgi?id=44226

Test: fast/events/focus-change-crash2.html

  • page/FocusController.cpp: (WebCore::FocusController::setFocusedNode): add a ref to prevent the focused node from being deleted
12:02 Changeset [65747] by tony@chromium.org
  • 3 edits
    2 adds in trunk

2010-08-20 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] fix a null pointer crash when dispatching JS keyboard events
https://bugs.webkit.org/show_bug.cgi?id=44313

  • fast/events/initkeyboardevent-crash-expected.txt: Added.
  • fast/events/initkeyboardevent-crash.html: Added.

2010-08-20 Tony Chang <tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] fix a null pointer crash when dispatching JS keyboard events
https://bugs.webkit.org/show_bug.cgi?id=44313

  • src/WebInputEventConversion.cpp: (WebKit::WebKeyboardEventBuilder::WebKeyboardEventBuilder):
12:02 BuildingQtOnOSX edited by ariya@webkit.org
mention QtXmlPatterns (diff)
11:49 Changeset [65746] by simon.fraser@apple.com
  • 3 edits
    4 adds in trunk

2010-08-20 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Slider labels do not update as you move the sliders on this page
https://bugs.webkit.org/show_bug.cgi?id=41020

When repainting after layout in a multicol element, we need to adjust the
repaint rect for the columns.

Test: fast/repaint/multicol-repaint.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::layoutBlock):
11:33 Changeset [65745] by jianli@chromium.org
  • 8 edits
    1 copy
    1 move in trunk/WebCore

Move FileStreamClient to platform and add AsyncFileStream interface.
https://bugs.webkit.org/show_bug.cgi?id=44224

Reviewed by David Levin.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/FileStreamProxy.cpp: Account for the change.

(WebCore::FileStreamProxy::FileStreamProxy):
(WebCore::FileStreamProxy::stop):

  • html/FileStreamProxy.h: Make it derive from AsyncFileStream.h.
  • platform/AsyncFileStream.h: Added.
  • platform/FileStreamClient.h: Renamed from WebCore/html/FileStreamClient.h.
11:25 BuildingQtOnOSX edited by ariya@webkit.org
Fix for potential include problem (diff)
11:24 Changeset [65744] by leandro@webkit.org
  • 4 edits
    1 add in trunk

2010-08-20 Leandro Pereira <leandro@profusion.mobi>

[EFL] Unreviewed. Fix build when using GNU gold.
When linking with GNU gold, some symbols are not found: explicitly
link with the required libraries.

  • CMakeListsEfl.txt: Depend on Fontconfig, PNG, JPEG and dl libraries.

2010-08-20 Leandro Pereira <leandro@profusion.mobi>

[EFL] Unreviewed. Fix build when using GNU gold.
When linking with GNU gold, some symbols are not found: explicitly
link with the required libraries.

  • cmake/FindFontconfig.cmake: Added.
  • cmake/OptionsEfl.cmake: Find Fontconfig, JPEG, and PNG libraries.
11:12 Changeset [65743] by jcivelli@chromium.org
  • 6 edits
    7 deletes in trunk/WebKit/chromium

2010-08-20 Jay Civelli <jcivelli@chromium.org>

Reviewed by Darin Fisher.

Removing the deprecated class WebEvent and friends.
https://bugs.webkit.org/show_bug.cgi?id=44296

  • WebKit.gyp:
  • public/WebEvent.h: Removed.
  • public/WebEventListener.h: Removed.
  • public/WebMutationEvent.h: Removed.
  • public/WebNode.h:
  • src/EventListenerWrapper.cpp: (WebKit::EventListenerWrapper::webDOMEventListenerDeleted):
  • src/EventListenerWrapper.h:
  • src/WebEvent.cpp: Removed.
  • src/WebEventListener.cpp: Removed.
  • src/WebEventListenerPrivate.cpp: Removed.
  • src/WebEventListenerPrivate.h: Removed.
  • src/WebNode.cpp: (WebKit::WebNode::removeEventListener):
11:00 Changeset [65742] by inferno@chromium.org
  • 5 edits
    2 adds in trunk

2010-08-20 Abhishek Arya <inferno@chromium.org>

Reviewed by Darin Fisher.

Prevent use of stale notification presenter pointer in notifications by instead using
a notification center pointer and deriving the presenter from it. Notification presenter
gets properly destroyed using disconnectFrame function inside notification center. Add
null checks for notification presenter.
https://bugs.webkit.org/show_bug.cgi?id=43645

Test: fast/notifications/notifications-window-close-crash.html

  • notifications/Notification.cpp: (WebCore::Notification::Notification): (WebCore::Notification::create): (WebCore::Notification::show): (WebCore::Notification::cancel): (WebCore::Notification::contextDestroyed): (WebCore::Notification::finishLoading):
  • notifications/Notification.h: (WebCore::Notification::detachPresenter):
  • notifications/NotificationCenter.h: (WebCore::NotificationCenter::createHTMLNotification): (WebCore::NotificationCenter::createNotification):

2010-08-20 Abhishek Arya <inferno@chromium.org>

Reviewed by Darin Fisher.

Manual test to check that canceling the notification on a closed window does
not result in crash.
https://bugs.webkit.org/show_bug.cgi?id=43645

  • fast/notifications/notifications-window-close-crash-expected.txt: Added.
  • fast/notifications/notifications-window-close-crash.html: Added.
10:50 Changeset [65741] by mrobinson@webkit.org
  • 3 edits in trunk/WebCore

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Xan Lopez.

r64526 broke the GTK+-3 build
https://bugs.webkit.org/show_bug.cgi?id=44254

No new tests, as this is covered by manual-tests/cursor.html.

  • platform/gtk/CursorGtk.cpp: (WebCore::createPixmapFromBits): Added this method which turns the inline data structures into GdkPixmaps using Cairo and GDK-Cairo. (WebCore::createNamedCursor): Modified this method to use the new helper.
  • platform/gtk/CursorGtk.h: Changed all inline data structures to be unsigned char arrays (which Cairo requires). The progress cursor is known in X11 icon themes as "left_ptr_watch." This change to the name preserves the old behavior of taking this icon from the theme when available. It seems that this worked in the past due to a fluke or failure of the previous code.
10:24 Changeset [65740] by zmo@google.com
  • 14 edits in trunk

2010-08-19 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Fix failing WebGL tests in Chromium in-process-webgl port
https://bugs.webkit.org/show_bug.cgi?id=44305

  • src/WebGraphicsContext3DDefaultImpl.cpp: (WebKit::WebGraphicsContext3DDefaultImpl::getVertexAttribOffset): Add implementation.

2010-08-19 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Fix failing WebGL tests in Chromium in-process-webgl port
https://bugs.webkit.org/show_bug.cgi?id=44305

  • fast/canvas/webgl/context-attributes-alpha-depth-stencil-antialias.html: Add "#ifdef GLES" around precision keywords.
  • fast/canvas/webgl/gl-bind-attrib-location-test-expected.txt: Re-enable the failing tests.
  • fast/canvas/webgl/gl-bind-attrib-location-test.html: Add "#ifdef GLES" around precision keywords.
  • fast/canvas/webgl/read-pixels-pack-alignment-expected.txt: Ditto.
  • fast/canvas/webgl/read-pixels-pack-alignment.html: Ditto.
  • fast/canvas/webgl/resources/fragmentShader.frag: Ditto.
  • fast/canvas/webgl/texImage2DImageDataTest.html: Ditto.
  • fast/canvas/webgl/texture-active-bind.html: Ditto.
  • fast/canvas/webgl/texture-complete.html: Ditto.
  • fast/canvas/webgl/texture-transparent-pixels-initialized.html: Ditto.
  • platform/chromium/test_expectations.txt: Ditto.
10:13 Changeset [65739] by mrobinson@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-20 Martin Robinson <mrobinson@igalia.com>

[GTK] Inspector extensions tests fail on GTK+ bots because onSelectionChanged is missing
https://bugs.webkit.org/show_bug.cgi?id=44342

  • platform/gtk/Skipped: Skip failing tests.
10:07 Changeset [65738] by johnnyg@google.com
  • 2 edits in trunk/LayoutTests

2010-08-20 John Gregg <johnnyg@google.com>

Unreviewed, test expectations.

Add fail expectations for some SVG tests after r65729. See bug 44341.

  • platform/chromium/test_expectations.txt:
09:32 Changeset [65737] by mrobinson@webkit.org
  • 7 edits in trunk

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Pavel Feldman.

Web Inspector: Inspector tests were disabled for GTK.
https://bugs.webkit.org/show_bug.cgi?id=43977

  • platform/gtk/Skipped: Unskip tests which should now be passing.

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Pavel Feldman.

Web Inspector: Inspector tests were disabled for GTK.
https://bugs.webkit.org/show_bug.cgi?id=43977

Allow setting the inspector resources path via an environment variables
and add a method, inspectorFilesPath, to the WebKit InspectorClient that
encapsulates this logic.

  • WebCoreSupport/InspectorClientGtk.cpp: (WebKit::InspectorClient::openInspectorFrontend): Use the new inspectorFilesPath method to get the path to the resource. (WebKit::InspectorClient::inspectorFilesPath): Added. (WebKit::InspectorFrontendClient::localizedStringsURL): Use inspectorFilesPath method.
  • WebCoreSupport/InspectorClientGtk.h: Add method and member variables for caching the result.

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Pavel Feldman.

Web Inspector: Inspector tests were disabled for GTK.
https://bugs.webkit.org/show_bug.cgi?id=43977

  • Scripts/old-run-webkit-tests: Set the WEBKIT_INSPECTOR_PATH to the appropriate path before running tests.
09:01 Changeset [65736] by mrobinson@webkit.org
  • 2 edits in trunk/WebCore

2010-08-20 Martin Robinson <mrobinson@igalia.com>

Reviewed by Dirk Schulze.

[Cairo] Eliminate full-surface copy from canvas.drawImage(...)
https://bugs.webkit.org/show_bug.cgi?id=44190

Covered by canvas layout tests.

  • platform/graphics/cairo/ImageBufferCairo.cpp: (WebCore::ImageBuffer::drawsUsingCopy): Return false now. (WebCore::ImageBuffer::clip): Add a link to the bug which tracks this feature. (WebCore::ImageBuffer::draw): Construct the bitmap image without the full-surface copy. (WebCore::ImageBuffer::drawPattern): Ditto.
08:13 Changeset [65735] by alex@webkit.org
  • 2 edits in trunk/WebCore

2010-08-20 Alejandro G. Castro <alex@igalia.com>

Reviewed by Xan Lopez.

[REGRESSION] r65608 broke gtk distcheck
https://bugs.webkit.org/show_bug.cgi?id=44333

Define the inspector.html as a noinst_DATA intead of using the dir
suffix, which is used for installation.

  • GNUmakefile.am:
08:07 Changeset [65734] by dbates@webkit.org
  • 4 edits in trunk/WebKitTools

2010-08-20 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

svn-apply doesn't detect empty line with Windows line endings
after property value
https://bugs.webkit.org/show_bug.cgi?id=43981

Part 2 of 2

Add unit tests that were not landed in the first commit.

  • Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
    • Added the following test cases: "SVN: binary file (isBinary true) using Windows line endings" "SVN: property diff, followed by file change diff using Windows line endings" "SVN: two consecutive property diffs using Windows line endings" "SVN: binary file with executable bit change usng Windows line endings" "SVN: binary file followed by property change on different file using Windows line endings" "SVN: binary file followed by file change on different file using Windows line endings" "SVN: file change diff with property change, followed by property change diff using Windows line endings" "SVN: file change diff with property change, followed by file change diff using Windows line endings"
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnProperty.pl:
    • Added the following test cases: "add svn:executable, followed by empty line and start of next diff using Windows line endings" "add svn:executable, followed by empty line and start of next property diff using Windows line endings" "multi-line '+' change, followed by empty line and start of next diff using Windows line endings" "multi-line '+' change, followed by empty line and start of next property diff using Windows line endings" "add svn:executable, followed by empty line and start of binary patch using Windows line endings" "multi-line '+' change, followed by empty line and start of binary patch using Windows line endings" "multi-line '-' change, followed by multi-line '+' change, empty line, and start of binary patch using Windows line endings" "single-line '+' with trailing new line using Windows line endings" "single-line '+' with trailing new line, followed by empty line and start of binary patch using Windows line endings" "single-line '-' change with trailing new line, and single-line '+' change using Windows line endings" "multi-line '-' change with trailing new line, and multi-line '+' change using Windows line endings"
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl:
    • Added the following test cases: "single-line '-' change followed by empty line with Windows line endings" "multi-line '+' change and start of binary patch with Windows line endings"
07:48 Changeset [65733] by demarchi@webkit.org
  • 2 edits in trunk/WebKit

2010-08-20 Lucas De Marchi <lucas.demarchi@profusion.mobi>

Unreviewed build fix.

As opposed to WebCore, JavascriptCore and wtf, webkit is always a
shared library. After r65366, libewebkit.so was not being installed
anymore when building with SHARED_CORE=0 (which is the default). This
partially reverts that revision in order to fix this issue.

  • CMakeLists.txt: always install webkit library.
07:45 Changeset [65732] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-08-20 Daniel Bates <dbates@rim.com>

Reviewed by Eric Seidel.

svn-apply doesn't detect empty line with Windows line endings
after property value
https://bugs.webkit.org/show_bug.cgi?id=43981

Fixes an issue where an empty line following the property value
in an SVN property change diff would not be detected if it had
Windows line endings (i.e. ended in CRLF). Hence it was consumed
as if it was part of a multi-line property value.

  • Scripts/VCSUtils.pm:
    • Modified parseSvnPropertyValue() to detect empty line with Windows line endings.
    • Added toWindowsLineEndings().
  • Scripts/webkitperl/VCSUtils_unittest/parseDiff.pl:
    • Added the following test cases: "SVN: binary file (isBinary true) using Windows line endings" "SVN: property diff, followed by file change diff using Windows line endings" "SVN: two consecutive property diffs using Windows line endings" "SVN: binary file with executable bit change usng Windows line endings" "SVN: binary file followed by property change on different file using Windows line endings" "SVN: binary file followed by file change on different file using Windows line endings" "SVN: file change diff with property change, followed by property change diff using Windows line endings" "SVN: file change diff with property change, followed by file change diff using Windows line endings"
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnProperty.pl:
    • Added the following test cases: "add svn:executable, followed by empty line and start of next diff using Windows line endings" "add svn:executable, followed by empty line and start of next property diff using Windows line endings" "multi-line '+' change, followed by empty line and start of next diff using Windows line endings" "multi-line '+' change, followed by empty line and start of next property diff using Windows line endings" "add svn:executable, followed by empty line and start of binary patch using Windows line endings" "multi-line '+' change, followed by empty line and start of binary patch using Windows line endings" "multi-line '-' change, followed by multi-line '+' change, empty line, and start of binary patch using Windows line endings" "single-line '+' with trailing new line using Windows line endings" "single-line '+' with trailing new line, followed by empty line and start of binary patch using Windows line endings" "single-line '-' change with trailing new line, and single-line '+' change using Windows line endings" "multi-line '-' change with trailing new line, and multi-line '+' change using Windows line endings"
  • Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl:
    • Added the following test cases: "single-line '-' change followed by empty line with Windows line endings" "multi-line '+' change and start of binary patch with Windows line endings"
06:38 Changeset [65731] by yurys@chromium.org
  • 12 edits in trunk

2010-08-19 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: implement breaking on DOM node subtree mutations.
https://bugs.webkit.org/show_bug.cgi?id=42886

Add two entries to Web Inspector DOM element context menu:

  1. "Stop on subtree modifications": will break on adding/removing of any element which is a child of the DOM element
  2. "Remove breakpoints": will remove all breakpoints associated with the DOM element
  • English.lproj/localizedStrings.js:
  • bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::breakProgram):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::breakProgram): (WebCore::ScriptDebugServer::breakProgramCallback): (WebCore::ScriptDebugServer::handleV8DebugEvent):
  • bindings/v8/ScriptDebugServer.h:
  • inspector/Inspector.idl:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::~InspectorDOMAgent): (WebCore::InspectorDOMAgent::discardBindings): (WebCore::InspectorDOMAgent::setDOMBreakpoint): (WebCore::InspectorDOMAgent::removeDOMBreakpoint): (WebCore::InspectorDOMAgent::didInsertDOMNode): (WebCore::InspectorDOMAgent::didRemoveDOMNode): (WebCore::InspectorDOMAgent::hasBreakpoint): (WebCore::InspectorDOMAgent::pauseOnBreakpoint): (WebCore::InspectorDOMAgent::updateSubtreeBreakpoints):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._populateTagContextMenu):
  • inspector/front-end/Settings.js:

2010-08-19 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: enable DOM breakpoints for chromium
https://bugs.webkit.org/show_bug.cgi?id=42886

04:34 Changeset [65730] by yurys@chromium.org
  • 7 edits in trunk/WebCore

2010-08-20 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: discard InjectedScript reference on ScriptState when clearing injected scripts
https://bugs.webkit.org/show_bug.cgi?id=44328

Otherwise if the reference is not cleared we may remove InjectedScript from the
map on InjectedScriptHost but keep it on ScriptState and try to reuse it later.

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::discardInjectedScript):
  • bindings/v8/V8HiddenPropertyName.h:
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::discardInjectedScript): (WebCore::InjectedScriptHost::injectedScriptFor):
  • inspector/InjectedScript.h: (WebCore::InjectedScript::scriptState):
  • inspector/InjectedScriptHost.cpp: (WebCore::InjectedScriptHost::discardInjectedScripts):
  • inspector/InjectedScriptHost.h:
02:13 Changeset [65729] by zimmermann@webkit.org
  • 4 edits
    4 adds in trunk

2010-08-20 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Clippers are rasterized, when applied to scaled target object
https://bugs.webkit.org/show_bug.cgi?id=44325

Use SVGImageBufferTools logic for RenderSVGResourceClipper, just like RenderSVGResourceMasker/Gradient,
to avoid pixelation, when the clipper is applied to a scaled target object, or when the document is scaled.

Test: svg/clip-path/clip-path-pixelation.svg

  • rendering/RenderSVGResourceClipper.cpp: (WebCore::RenderSVGResourceClipper::applyClippingToContext): (WebCore::RenderSVGResourceClipper::createClipData):
  • rendering/RenderSVGResourceClipper.h:

2010-08-20 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Clippers are rasterized, when applied to scaled target object
https://bugs.webkit.org/show_bug.cgi?id=44325

  • platform/mac/svg/clip-path/clip-path-pixelation-expected.checksum: Added.
  • platform/mac/svg/clip-path/clip-path-pixelation-expected.png: Added.
  • platform/mac/svg/clip-path/clip-path-pixelation-expected.txt: Added.
  • svg/clip-path/clip-path-pixelation.svg: Added.
01:12 Changeset [65728] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-08-20 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Initialize GDK before loading plugins
https://bugs.webkit.org/show_bug.cgi?id=44324

Attempt to call gdk_init_check() before loading any plugins.
This prevents various crashes and freezes in Adobe's Flash plugin.

  • plugins/qt/PluginPackageQt.cpp: (WebCore::initializeGdkIfPossible): (WebCore::PluginPackage::load):
00:58 Changeset [65727] by mitz@apple.com
  • 2 edits in trunk/WebCore

Follow-up to r41020.

Reviewed by Mark Rowe.

No change in behavior, thus no new tests.

  • platform/graphics/mac/FontPlatformDataMac.mm:

(WebCore::FontPlatformData::FontPlatformData): Asserted that the NSFont parameter is not nil,
and removed nil check.
(WebCore::FontPlatformData::setFont): Asserted that the NSFont parameter is not nil and that
this is not the deleted value, and removed code to handle those cases.

08/19/10:

21:51 Changeset [65726] by commit-queue@webkit.org
  • 4 edits in trunk

2010-08-19 Vincent Scheib <scheib@chromium.org>

Reviewed by David Levin.

[chromium] TilingData::tilePositionY has typo of X where Y should be used
https://bugs.webkit.org/show_bug.cgi?id=44195

Corrected error, and rewrote functions to use for loop instead of recursion.

Unit Tests added to WebKit/chromium/tests/TilingDataTest.cpp

  • platform/graphics/chromium/TilingData.cpp: (WebCore::TilingData::tilePositionX): (WebCore::TilingData::tilePositionY):

2010-08-19 Vincent Scheib <scheib@chromium.org>

Reviewed by David Levin.

[chromium] TilingData::tilePositionY has typo of X where Y should be used
https://bugs.webkit.org/show_bug.cgi?id=44195

Unit tests added for tileSize and tilePosition.

  • tests/TilingDataTest.cpp: (WebCore::TEST):
21:28 Changeset [65725] by tkent@chromium.org
  • 2 edits
    1 delete in trunk/LayoutTests

2010-08-19 Kent Tamura <tkent@chromium.org>

Unreviewed, test expectation update.

Update for r65723.

  • platform/chromium-win/fast/forms/listbox-selection-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
20:57 Changeset [65724] by commit-queue@webkit.org
  • 5 edits
    1 copy
    3 moves
    3 adds in trunk

2010-08-19 Mihai Parparita <mihaip@chromium.org>

Reviewed by Brady Eidson.

Assertion failure when going back inside frame during onload
https://bugs.webkit.org/show_bug.cgi?id=44217

Adds a test that does a history.back() within the main frame during
onload of a child frame. If HISTORY_ALWAYS_ASYNC was set to 0, this
would crash the DRT with an assertion failure, since we hadn't called
updateForFrameLoadCompleted in the main frame yet.

(renames history-back-within-subframe to history-back-within-subframe-hash
to differentiate between the two otherwise similar tests)

  • fast/history/history-back-within-subframe-hash-expected.txt: Renamed from LayoutTests/fast/history/history-back-within-subframe-expected.txt.
  • fast/history/history-back-within-subframe-hash.html: Renamed from LayoutTests/fast/history/history-back-within-subframe.html.
  • fast/history/history-back-within-subframe-url-expected.txt: Added
  • fast/history/history-back-within-subframe-url.html: Added.
  • fast/history/resources/history-back-within-subframe-hash-2.html: Renamed from LayoutTests/fast/history/resources/history-back-within-subframe-2.html.
  • fast/history/resources/history-back-within-subframe-url-2.html: Added.
  • fast/history/resources/history-back-within-subframe-url-3.html: Added.

2010-08-19 Mihai Parparita <mihaip@chromium.org>

Reviewed by Brady Eidson.

Assertion failure when going back inside frame during onload
https://bugs.webkit.org/show_bug.cgi?id=44217

Fix assertion failure when doing a history.back() within the main frame
during onload of a child frame. This would happen when
HISTORY_ALWAYS_ASYNC was set to 0, because we would only compare the
current frame document sequence numbers when determining if we're in the
same document, instead of also recursing over child frames.

Test: fast/history/history-back-within-subframe-hash.html

  • history/HistoryItem.cpp: (WebCore::HistoryItem::hasSameDocuments):
  • history/HistoryItem.h:
  • loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::scheduleHistoryNavigation):
20:37 Changeset [65723] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/LayoutTests

2010-08-19 Kenichi Ishibashi <bashi@google.com>

Reviewed by Kent Tamura.

Improve portability of fast/forms/listbox-selection.html
https://bugs.webkit.org/show_bug.cgi?id=35786

Changed to use script-tests style for improving portability.

  • For sending mouse events to the target element, calculating the item height at runtime instead using Mac-specific metrics.
  • Using mouseDown(), mouseUp(), and keyDown() to send events with modifier keys in a platform-independent way.
  • fast/forms/listbox-selection-expected.txt: Updated to match with new test.
  • fast/forms/listbox-selection.html: Modified to use script-tests style.
  • fast/forms/script-tests/listbox-selection.js: Added.
19:51 Changeset [65722] by commit-queue@webkit.org
  • 4 edits in trunk

2010-08-19 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Antonio Gomes.

[Qt] Fix cursor change propagation
https://bugs.webkit.org/show_bug.cgi?id=44250

No functional change so new tests.

Propagate the setCursor callback to the PageClient via the HostWindow instead of preassuming
the concrete type of the ChromeClient (what was generally wrong and actually incompatible with WebKit2).

  • platform/qt/WidgetQt.cpp: (WebCore::Widget::setCursor):

2010-08-19 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Antonio Gomes.

[Qt] Fix cursor change propagation
https://bugs.webkit.org/show_bug.cgi?id=44250

  • WebCoreSupport/ChromeClientQt.cpp: (WebCore::ChromeClientQt::setCursor): Implemented. Propagete the callback forward to the PageClient.
18:22 Changeset [65721] by rniwa@webkit.org
  • 3 edits
    2 adds
    22 deletes in trunk/LayoutTests

2010-08-19 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

dumpAsMarkup conversion: editing/pasteboard/4242293-1.html and editing/pasteboard/copy-paste-bidi.html
https://bugs.webkit.org/show_bug.cgi?id=44220

Converted the tests.

  • editing/pasteboard/4242293-1-expected.txt: Added.
  • editing/pasteboard/4242293-1.html:
  • editing/pasteboard/copy-paste-bidi-expected.txt: Added.
  • editing/pasteboard/copy-paste-bidi.html:
  • platform/chromium-linux/editing/pasteboard/4242293-1-expected.checksum: Removed.
  • platform/chromium-linux/editing/pasteboard/4242293-1-expected.png: Removed.
  • platform/chromium-linux/editing/pasteboard/copy-paste-bidi-expected.checksum: Removed.
  • platform/chromium-linux/editing/pasteboard/copy-paste-bidi-expected.png: Removed.
  • platform/chromium-win/editing/pasteboard/4242293-1-expected.checksum: Removed.
  • platform/chromium-win/editing/pasteboard/4242293-1-expected.png: Removed.
  • platform/chromium-win/editing/pasteboard/4242293-1-expected.txt: Removed.
  • platform/chromium-win/editing/pasteboard/copy-paste-bidi-expected.checksum: Removed.
  • platform/chromium-win/editing/pasteboard/copy-paste-bidi-expected.png: Removed.
  • platform/chromium-win/editing/pasteboard/copy-paste-bidi-expected.txt: Removed.
  • platform/mac-leopard/editing/pasteboard/4242293-1-expected.checksum: Removed.
  • platform/mac-leopard/editing/pasteboard/4242293-1-expected.png: Removed.
  • platform/mac-leopard/editing/pasteboard/copy-paste-bidi-expected.checksum: Removed.
  • platform/mac-leopard/editing/pasteboard/copy-paste-bidi-expected.png: Removed.
  • platform/mac/editing/pasteboard/4242293-1-expected.checksum: Removed.
  • platform/mac/editing/pasteboard/4242293-1-expected.png: Removed.
  • platform/mac/editing/pasteboard/4242293-1-expected.txt: Removed.
  • platform/mac/editing/pasteboard/copy-paste-bidi-expected.checksum: Removed.
  • platform/mac/editing/pasteboard/copy-paste-bidi-expected.png: Removed.
  • platform/mac/editing/pasteboard/copy-paste-bidi-expected.txt: Removed.
  • platform/qt/editing/pasteboard/4242293-1-expected.txt: Removed.
  • platform/qt/editing/pasteboard/copy-paste-bidi-expected.txt: Removed.
18:05 Changeset [65720] by mrobinson@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-19 Martin Robinson <mrobinson@igalia.com>

[GTK] r65681 broke a couple GTK+ a11y tests
https://bugs.webkit.org/show_bug.cgi?id=44316

  • platform/gtk/Skipped: Skip some newly failing tests.
17:58 Changeset [65719] by jamesr@google.com
  • 2 edits in trunk/LayoutTests

2010-08-19 James Robinson <jamesr@chromium.org>

Unreviewed. Clean up test_expectations now that tables/hittesting/filltable-* pass on Chromium.

  • platform/chromium/test_expectations.txt:
17:47 Changeset [65718] by kinuko@chromium.org
  • 5 edits
    3 adds in trunk/WebKit/chromium

2010-08-19 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Darin Fisher.

Add WebKit API for FileSystem API
https://bugs.webkit.org/show_bug.cgi?id=43151

Add asynchronous FileSystem interface to WebKit API for
FileSystem API.
http://dev.w3.org/2009/dap/file-system/file-dir-sys.html

  • WebKit.gyp:
  • public/WebFileError.h: Added.
  • public/WebFileSystem.h: Added.
  • public/WebFileSystemEntry.h: Added.
  • public/WebFileSystemCallbacks.h: Added.
  • public/WebKitClient.h: (WebKit::WebKitClient::fileSystem):
  • public/WebFrameClient.h: (WebKit::WebFrameClient::openFileSystem):
17:46 Changeset [65717] by vangelis@chromium.org
  • 13 edits
    2 adds
    2 deletes in trunk/WebCore

2010-08-19 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Rearranging the accelerated compositing code such that the
layer rendering logic now lives in the layer classes rather than the
compositor. This lifts the restriction of having one texture per layer
and significantly cleans up the compositor code. This change mostly
resulted in a lot of code moving around files. Noteworthy changes
include:

  • Made LayerChromium a proper based class for layers that mostly handles the updates to the layer properties and superlayer / sublayer updates.
  • Introduced a new layer type, ContentLayerChromium, which handles layers that require a GraphicsContext to render their content.
  • ImageLayerChromium and VideoLayerChromium now derive from ContentLayerChromium as they share the same shader and draw function.
  • Removed TransformLayerChromium as its functionality is now replaced by the base LayerChromium class.
  • Re-arranged the order in which the members of LayerChromium are defined in the header file to form a more reasonable logical grouping.
  • Changed LayerRendererChromium to use the shader creation and drawing methods defined in the layer classes. As a result, a lot of GL code was removed from the implementation file.
  • Eliminated randomly dispersed calls to check for GL errors by a macro (GLC) which allows turning error testing on/off with a single define (DEBUG_GL_CALLS defined in LayerRendererChromium.h)
  • Replaced the previous layer render loop with two traversals of the layer hierarchy, the first to update transforms and opacity values and the second to render the layers. Eliminated global Z sorting of all layers as it was wrong. https://bugs.webkit.org/show_bug.cgi?id=44148

Test: Verified that pages using the compositor and compositor

layout tests have not regressed.

  • WebCore.gypi:
  • platform/graphics/chromium/CanvasLayerChromium.cpp: (WebCore::CanvasLayerChromium::SharedValues::SharedValues): (WebCore::CanvasLayerChromium::SharedValues::~SharedValues): (WebCore::CanvasLayerChromium::updateContents): (WebCore::CanvasLayerChromium::draw):
  • platform/graphics/chromium/CanvasLayerChromium.h: (WebCore::CanvasLayerChromium::SharedValues::canvasShaderProgram): (WebCore::CanvasLayerChromium::SharedValues::shaderSamplerLocation): (WebCore::CanvasLayerChromium::SharedValues::shaderMatrixLocation): (WebCore::CanvasLayerChromium::SharedValues::shaderAlphaLocation): (WebCore::CanvasLayerChromium::SharedValues::initialized):
  • platform/graphics/chromium/ContentLayerChromium.cpp: Added. (WebCore::ContentLayerChromium::SharedValues::SharedValues): (WebCore::ContentLayerChromium::SharedValues::~SharedValues): (WebCore::ContentLayerChromium::create): (WebCore::ContentLayerChromium::ContentLayerChromium): (WebCore::ContentLayerChromium::~ContentLayerChromium): (WebCore::ContentLayerChromium::updateContents): (WebCore::ContentLayerChromium::updateTextureRect): (WebCore::ContentLayerChromium::draw):
  • platform/graphics/chromium/ContentLayerChromium.h: Added. (WebCore::ContentLayerChromium::drawsContent): (WebCore::ContentLayerChromium::SharedValues::contentShaderProgram): (WebCore::ContentLayerChromium::SharedValues::shaderSamplerLocation): (WebCore::ContentLayerChromium::SharedValues::shaderMatrixLocation): (WebCore::ContentLayerChromium::SharedValues::shaderAlphaLocation): (WebCore::ContentLayerChromium::SharedValues::initialized):
  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::GraphicsLayerChromium): (WebCore::GraphicsLayerChromium::updateLayerPreserves3D):
  • platform/graphics/chromium/ImageLayerChromium.cpp: (WebCore::ImageLayerChromium::ImageLayerChromium): (WebCore::ImageLayerChromium::updateContents):
  • platform/graphics/chromium/ImageLayerChromium.h:
  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::loadShader): (WebCore::LayerChromium::SharedValues::SharedValues): (WebCore::LayerChromium::SharedValues::~SharedValues): (WebCore::LayerChromium::LayerChromium): (WebCore::LayerChromium::~LayerChromium): (WebCore::LayerChromium::createShaderProgram): (WebCore::LayerChromium::toGLMatrix): (WebCore::LayerChromium::drawTexturedQuad): (WebCore::LayerChromium::drawDebugBorder): (WebCore::LayerChromium::prepareForDraw):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::getSublayers): (WebCore::LayerChromium::setPosition): (WebCore::LayerChromium::contentsDirty): (WebCore::LayerChromium::drawsContent): (WebCore::LayerChromium::updateContents): (WebCore::LayerChromium::draw): (WebCore::LayerChromium::SharedValues::quadVerticesVbo): (WebCore::LayerChromium::SharedValues::quadElementsVbo): (WebCore::LayerChromium::SharedValues::maxTextureSize): (WebCore::LayerChromium::SharedValues::borderShaderProgram): (WebCore::LayerChromium::SharedValues::borderShaderMatrixLocation): (WebCore::LayerChromium::SharedValues::borderShaderColorLocation): (WebCore::LayerChromium::SharedValues::initialized): (WebCore::LayerChromium::layerRenderer):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::~LayerRendererChromium): (WebCore::LayerRendererChromium::debugGLCall): (WebCore::LayerRendererChromium::useShader): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::createLayerTexture): (WebCore::LayerRendererChromium::updateLayersRecursive): (WebCore::LayerRendererChromium::drawLayersRecursive): (WebCore::LayerRendererChromium::drawLayer): (WebCore::LayerRendererChromium::checkTextureSize): (WebCore::LayerRendererChromium::initializeSharedObjects): (WebCore::LayerRendererChromium::cleanupSharedObjects):
  • platform/graphics/chromium/LayerRendererChromium.h: (WebCore::LayerRendererChromium::projectionMatrix): (WebCore::LayerRendererChromium::layerSharedValues): (WebCore::LayerRendererChromium::contentLayerSharedValues): (WebCore::LayerRendererChromium::canvasLayerSharedValues):
  • platform/graphics/chromium/TransformLayerChromium.cpp: Removed.
  • platform/graphics/chromium/TransformLayerChromium.h: Removed.
  • platform/graphics/chromium/VideoLayerChromium.cpp: (WebCore::VideoLayerChromium::VideoLayerChromium): (WebCore::VideoLayerChromium::updateContents): (WebCore::VideoLayerChromium::createTextureRect):
  • platform/graphics/chromium/VideoLayerChromium.h:
17:40 Changeset [65716] by simon.fraser@apple.com
  • 6 edits in trunk/WebCore

2010-08-19 Simon Fraser <simon.fraser@apple.com>

Reviewed by Sam Weinig.

Clean up some selection code
https://bugs.webkit.org/show_bug.cgi?id=44314

Change the concept of "layout" in SelectionController to "updateCaretRect", renaming
methods accordingly.

Move selection and printing methods in RenderView.h into groups.

No behavioral changes.

  • editing/SelectionController.cpp: (WebCore::SelectionController::SelectionController): (WebCore::SelectionController::setSelection): (WebCore::SelectionController::modify): (WebCore::SelectionController::setCaretRectNeedsUpdate): (WebCore::SelectionController::updateCaretRect): (WebCore::SelectionController::localCaretRect): (WebCore::SelectionController::recomputeCaretRect): Test m_caretRectNeedsUpdate first. (WebCore::SelectionController::invalidateCaretRect):
  • editing/SelectionController.h:
  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollToOffset):
  • rendering/RenderView.h: (WebCore::RenderView::selectionStart): (WebCore::RenderView::selectionEnd): (WebCore::RenderView::printRect): (WebCore::RenderView::setPrintRect):
17:03 Changeset [65715] by kinuko@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-08-19 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Jian Li.

[Chromium] Rolls Chromium DEPS forward to 56564
https://bugs.webkit.org/show_bug.cgi?id=44281

Roll Chromium DEPS forward to 56564 to include WebFileSystem
implementation changes.

  • DEPS:
16:39 Changeset [65714] by commit-queue@webkit.org
  • 6 edits
    5 moves
    5 deletes in trunk/LayoutTests

2010-08-19 Fady Samuel <fsamuel@chromium.org>

Reviewed by Dimitri Glazkov.

Fixed table hit testing layout test issues
https://bugs.webkit.org/show_bug.cgi?id=44204

Moved expectations to the same directory as tests.
Expectations were failing due to inconsistent eventSender behavior:
Switched to using elementFromPoint

  • platform/gtk/tables/hittesting/filltable-emptycells-expected.txt: Removed.
  • platform/gtk/tables/hittesting/filltable-levels-expected.txt: Removed.
  • platform/gtk/tables/hittesting/filltable-outline-expected.txt: Removed.
  • platform/gtk/tables/hittesting/filltable-rtl-expected.txt: Removed.
  • platform/gtk/tables/hittesting/filltable-stress-expected.txt: Removed.
  • tables/hittesting/filltable-emptycells-expected.txt: Renamed from LayoutTests/platform/chromium-linux/tables/hittesting/filltable-emptycells-expected.txt.
  • tables/hittesting/filltable-emptycells.html:
  • tables/hittesting/filltable-levels-expected.txt: Renamed from LayoutTests/platform/chromium-linux/tables/hittesting/filltable-levels-expected.txt.
  • tables/hittesting/filltable-levels.html:
  • tables/hittesting/filltable-outline-expected.txt: Renamed from LayoutTests/platform/chromium-linux/tables/hittesting/filltable-outline-expected.txt.
  • tables/hittesting/filltable-outline.html:
  • tables/hittesting/filltable-rtl-expected.txt: Renamed from LayoutTests/platform/chromium-linux/tables/hittesting/filltable-rtl-expected.txt.
  • tables/hittesting/filltable-rtl.html:
  • tables/hittesting/filltable-stress-expected.txt: Renamed from LayoutTests/platform/chromium-linux/tables/hittesting/filltable-stress-expected.txt.
  • tables/hittesting/filltable-stress.html:
15:52 Changeset [65713] by ddkilzer@apple.com
  • 6 edits in trunk

BUILD FIX: Fix Mac build after Windows WebKit2 changes for Netscape Plug-ins

WebCore:

  • WebCore.exp.in:

(WebCore::ScrollView::contentsToWindow): Added export.

WebKit2:

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::NetscapePlugin::platformPaint): Changed frameRect() to
m_frameRect.

WebKitTools:

  • MiniBrowser/mac/BrowserWindowController.m:

(decidePolicyForNavigationAction): Updated method signature.
(decidePolicyForNewWindowAction): Updated method signature.

15:35 Changeset [65712] by abarth@webkit.org
  • 1 edit
    4 adds in trunk/LayoutTests

2010-08-19 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

<style> inside <noscript> is parsed when JavaScript is enabled.
https://bugs.webkit.org/show_bug.cgi?id=44021

More random bugs fixed by the HTML5 parser. Time to document with a
couple tests. I added both these tests because <noscript> in <body> is
slightly different than <noscript> before <body>.

  • fast/parser/no-style-in-noscript-expected.txt: Added.
  • fast/parser/no-style-in-noscript-with-body-expected.txt: Added.
  • fast/parser/no-style-in-noscript-with-body.html: Added.
  • fast/parser/no-style-in-noscript.html: Added.
15:09 Changeset [65711] by ddkilzer@apple.com
  • 4 edits in trunk

BUILD FIX #3: <http://webkit.org/b/44285> Fix compilation with NETSCAPE_PLUGIN_API disabled

Still trying to make Qt Linux Release minimal buildbot happy.

WebCore:

  • plugins/PluginViewNone.cpp: Compile missing methods for Qt.

WebKit/qt:

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::createPlugin): Wrap code that
handles Netscape plugins in #if ENABLE(NETSCAPE_PLUGIN_API)
and #endif.

14:59 Changeset [65710] by johnnyg@google.com
  • 2 edits in trunk/LayoutTests

2010-08-19 Sam Weinig <sam@webkit.org>

Rubber-stamped by Jon Honeycutt.

Add a few items in the WebKitTestRunner Skipped list.

  • platform/mac-wk2/Skipped:
14:41 Changeset [65709] by weinig@apple.com
  • 2 edits in trunk/LayoutTests

Add a few items in the WebKitTestRunner Skipped list.

Rubber-stamped by Jon Honeycutt.

  • platform/mac-wk2/Skipped:
14:32 Changeset [65708] by ddkilzer@apple.com
  • 2 edits in trunk/WebKit/qt

BUILD FIX: FrameLoaderClientQt.h should include ResourceError.h

  • WebCoreSupport/FrameLoaderClientQt.h: Include ResourceError.h.
14:25 Changeset [65707] by ddkilzer@apple.com
  • 2 edits in trunk/WebCore

BUILD FIX: <http://webkit.org/b/44285> Fix compilation with NETSCAPE_PLUGIN_API disabled

Reviewed by Joseph Pecoraro.

  • plugins/PluginView.cpp: Added #if ENABLE(NETSCAPE_PLUGIN_API)

and #endif macros around the source to fix the Qt Linux Release
Minimal build.

14:23 Changeset [65706] by aroben@apple.com
  • 2 edits in trunk/WebKit2

Send mouse events to windowless plugins on Windows

This seems to have been the last issue blocking us from playing videos
on vimeo.com.

Fixes <http://webkit.org/b/44276> <rdar://problem/8330398> Can't click
play button in vimeo.com Flash video player in WebKit2 on Windows

Reviewed by Sam Weinig.

  • WebProcess/Plugins/Netscape/win/NetscapePluginWin.cpp:

(WebKit::toNP): Added. Converts a WebMouseEvent to an NPEvent. Code
was adapted from WebCore's PluginViewWin.cpp.

(WebKit::NetscapePlugin::platformHandleMouseEvent):
(WebKit::NetscapePlugin::platformHandleMouseEnterEvent):
(WebKit::NetscapePlugin::platformHandleMouseLeaveEvent):
Convert the WebMouseEvent to an NPEvent and send it to the plugin.

14:22 Changeset [65705] by aroben@apple.com
  • 8 edits in trunk/WebKit2

Add support for painting windowless plugins on Windows

This gets the video on the front page of vimeo.com painting (though
you can't make it play because we don't support mouse events yet).

Fixes <http://webkit.org/b/44274> <rdar://problem/8330398> Windowless
plugins don't paint in WebKit2 on Windows

Reviewed by Sam Weinig.

  • WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp:

(WebKit::NPN_GetValue): Tell the plugin that we support windowless
mode on Windows.
(WebKit::NPN_SetValue): Implemented handling of NPPVpluginWindowBool
by calling through to NetscapePlugin::setIsWindowed.

  • WebProcess/Plugins/Netscape/NetscapePlugin.cpp:

(WebKit::NetscapePlugin::NetscapePlugin): Added initialization of
m_isWindowed. It defaults to false on Mac and true on other platforms,
matching WebCore's PluginView.

  • WebProcess/Plugins/Netscape/NetscapePlugin.h: Added m_isWindowed.

(WebKit::NetscapePlugin::setIsWindowed): Added standard setter.

  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::NetscapePlugin::platformPaint): Moved Mac-specific context
translation code here from PluginView::paint.

  • WebProcess/Plugins/Netscape/win/NetscapePluginWin.cpp:

(WebKit::NetscapePlugin::platformPaint): Implemented. Code was ported
from WebCore's PluginViewWin.cpp.

  • WebProcess/Plugins/Plugin.h: Updated the comment for Plugin::paint

to explain the coordinate system of the context.

  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::paint): Changed to translate the context from
document to window coordinates. We were previously trying to translate
to plugin-local coordinates, but this only worked for documents whose
origin was the same as the window's origin (i.e., the main frame). The
plugin takes care of any further translations needed. (Mac translates
the context into plugin-local coordinates, and Windows leaves it in
window coordinates.)

14:22 Changeset [65704] by aroben@apple.com
  • 2 edits in trunk/WebKit2

Implement PluginInfoStore::pluginsDirectories

This makes WebKit2 able to find and load Flash on Windows.

Note that we aren't yet respecting the MozillaPlugins registry key.
That's covered by <http://webkit.org/b/44271>.

Fixes <http://webkit.org/b/43510> <rdar://problem/8273827> WebKit2
should load plugins from the same directories as old-WebKit

Reviewed by Jon Honeycutt and Ada Chan.

  • UIProcess/Plugins/win/PluginInfoStoreWin.cpp:

(WebKit::parseVersionString):
(WebKit::compareVersions):
(WebKit::safariPluginsDirectory):
(WebKit::addMozillaPluginDirectories):
(WebKit::addWindowsMediaPlayerPluginDirectory):
(WebKit::addQuickTimePluginDirectory):
(WebKit::addAdobeAcrobatPluginDirectory):
(WebKit::addMacromediaPluginDirectories):
(WebKit::PluginInfoStore::pluginsDirectories):
Ported (and ever-so-slightly cleaned up) code from WebCore's
PluginDatabaseWin.cpp.

14:22 Changeset [65703] by aroben@apple.com
  • 4 edits in trunk

Call NP_GetEntryPoints before NP_Initialize on Windows

Doing otherwise will cause Flash and QuickTime to crash inside
NP_Initialize.

Fixes <http://webkit.org/b/44270> <rdar://problem/8330393> Crash in
NP_Initialize when loading QuickTime when running
plugins/embed-attributes-setting.html in WebKit2 on Windows

Reviewed by Sam Weinig.

WebKit2:

  • WebProcess/Plugins/Netscape/NetscapePluginModule.cpp:

(WebKit::NetscapePluginModule::tryLoad): On Windows, first call
NP_GetEntryPoints, then NP_Initialize. Do the reverse on Mac to
prevent Silverlight (e.g.) from crashing (see r38858).

WebKitTools:

Test that NP_Initialize and NP_GetEntryPoints are called in the
correct order

The order differs between Mac and Windows (see r38858).

  • DumpRenderTree/TestNetscapePlugIn/main.cpp: Added a CRASH macro and

a boolean to record whether NP_GetEntryPoints has been called.
(NP_Initialize): Crash on Windows if NP_GetEntryPoints hasn't been
called yet. This matches Flash and QuickTime's behavior. Crash on Mac
if NP_GetEntryPoints has been called already. This matches
Silverlight's behavior.

14:21 Changeset [65702] by aroben@apple.com
  • 3 edits
    1 add in trunk/WebKit2

Add NetscapePluginWin.cpp

Fixes <http://webkit.org/b/44269> <rdar://problem/8330391>
NetscapePlugin.h contains functions that should be in
NetscapePluginWin.cpp

Reviewed by Sam Weinig.

  • WebProcess/Plugins/Netscape/NetscapePlugin.h: Moved functions from

here...

  • WebProcess/Plugins/Netscape/win/NetscapePluginWin.cpp: ...to here.
  • win/WebKit2.vcproj: Added NetscapePluginWin.cpp and let VS reorder

the file list.

14:21 Changeset [65701] by aroben@apple.com
  • 2 edits in trunk/WebKitTools

Make build-webkittestrunner build TestNetscapePlugIn on Windows

Fixes <http://webkit.org/b/44268> <rdar://problem/8330388>

Reviewed by Sam Weinig.

  • WebKitTestRunner/WebKitTestRunner.sln: Added TestNetscapePlugIn.

Threaded it in between ImageDiff and InjectedBundleGenerated.

14:13 Changeset [65700] by senorblanco@chromium.org
  • 4 edits in trunk/WebCore

2010-08-19 Stephen White <senorblanco@chromium.org>

Reviewed by Kenneth Russell.

[CHROMIUM] Assert w/canvas image draw
https://bugs.webkit.org/show_bug.cgi?id=44279

Don't use bitmap->pixelRef()->getPixels() directly; prefer
bitmap->pixels() and an SkAutoLockPixels. Also, make sure to set
the backing store state to Software when it was None. This handles
missing draws when the first call is software. Finally, remove some
spurious prepareForSoftwareDraw() calls in addPath() and beginPath().
It's really the fillPath() or strokePath() that matters.

Covered by LayoutTests/fast/canvas/arc360.html and others.

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::addPath): (WebCore::GraphicsContext::beginPath): Remove spurious prepareForSoftwareDraw() calls.
  • platform/graphics/skia/ImageSkia.cpp: (WebCore::drawBitmapGLES2): Lock SkBitmap's pixels, and use ->getPixels().
  • platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::prepareForSoftwareDraw): Switch backing store state to Software when it was None.
14:09 Changeset [65699] by weinig@apple.com
  • 8 edits in trunk/WebKit2

Add modifier key info to policy client functions in WebKit2
<rdar://problem/8185298>
https://bugs.webkit.org/show_bug.cgi?id=44280

Reviewed by Adam Roben.

  • UIProcess/API/C/WKAPICast.h:

Add conversion function for modifiers and rename an incorrectly named
toWK to toRef.

  • UIProcess/API/C/WKPage.h:

Add WKEventModifiers enum.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didReceiveMessage):
(WebKit::WebPageProxy::decidePolicyForNavigationAction):
(WebKit::WebPageProxy::decidePolicyForNewWindowAction):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPolicyClient.cpp:

(WebKit::WebPolicyClient::decidePolicyForNavigationAction):
(WebKit::WebPolicyClient::decidePolicyForNewWindowAction):

  • UIProcess/WebPolicyClient.h:

Pipe through modifier info.

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::modifiersForNavigationAction):
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNewWindowAction):
(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForNavigationAction):
Get modifier info out of the NavigationInfo class in the same manner as
WebKit1.

14:03 Changeset [65698] by andreas.kling@nokia.com
  • 3 edits in trunk/JavaScriptCore

2010-08-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Geoffrey Garen.

JSC: Move the static_cast into to(U)Int32 fast case
https://bugs.webkit.org/show_bug.cgi?id=44037

Do the static_cast<(u)int32_t> inline to avoid the function call overhead
for easily converted values (within (u)int32_t range.)

  • runtime/JSValue.cpp: (JSC::toInt32SlowCase): (JSC::toUInt32SlowCase):
  • runtime/JSValue.h: (JSC::JSValue::toInt32): (JSC::JSValue::toUInt32):
13:51 Changeset [65697] by ddkilzer@apple.com
  • 15 edits in trunk

<http://webkit.org/b/44285> Fix compilation with NETSCAPE_PLUGIN_API disabled

Reviewed by Joseph Pecoraro.

WebCore:

  • WebCore.exp.in:

(WebCore::HTMLPlugInElement::getNPObject): Moved from general
section into ENABLE(NETSCAPE_PLUGIN_API) section.
(WebCore::FrameView::windowClipRectForLayer): Moved from
ENABLE(NETSCAPE_PLUGIN_API) section to general section since
it's used by WebKit2.

  • plugins/PluginView.h: Added #if ENABLE(NETSCAPE_PLUGIN_API)

and #endif macros as needed to make iOS WebKit build with
NETSCAPE_PLUGIN_API disabled.

  • plugins/PluginViewNone.cpp: Ditto.

(WebCore::PluginView::platformGetValueStatic):

WebKit/mac:

  • Plugins/Hosted/HostedNetscapePluginStream.mm: Changed

USE(PLUGIN_HOST_PROCESS) to
USE(PLUGIN_HOST_PROCESS) && ENABLE(NETSCAPE_PLUGIN_API).

  • Plugins/Hosted/NetscapePluginHostManager.mm: Ditto.
  • Plugins/Hosted/NetscapePluginHostProxy.mm: Ditto.
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: Ditto.
  • Plugins/Hosted/ProxyInstance.mm: Ditto.
  • Plugins/Hosted/WebHostedNetscapePluginView.mm: Ditto.
  • WebCoreSupport/WebChromeClient.mm: Ditto.

(WebChromeClient::createWindow):

  • WebCoreSupport/WebFrameLoaderClient.mm: Ditto.

(WebFrameLoaderClient::dispatchCreatePage):

  • WebView/WebHTMLView.mm:

(needsCursorRectsSupportAtPoint): Added #if
ENABLE(NETSCAPE_PLUGIN_API) and #endif macros as needed.

  • WebView/WebView.mm:

(+[WebView _isNodeHaltedPlugin:]): Ditto.
(+[WebView _hasPluginForNodeBeenHalted:]): Ditto.
(+[WebView _restartHaltedPluginForNode:]): Ditto.

13:46 Changeset [65696] by ojan@chromium.org
  • 2 edits in trunk/WebKitTools

2010-08-19 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

install rietveld's upload script from a versioned url
https://bugs.webkit.org/show_bug.cgi?id=44291

autoinstall expects the file at a given URL to never change.
In order to be able to update the file, we need to be able to
change the path.

  • Scripts/webkitpy/thirdparty/init.py:
13:16 Changeset [65695] by johnnyg@google.com
  • 2 edits in trunk/LayoutTests

2010-08-19 John Gregg <johnnyg@google.com>

Unreviewed, chromium test expectations.

Add expected failures for caret-based pixel tests. See bug 44290.

  • platform/chromium/test_expectations.txt:
13:04 Changeset [65694] by ojan@chromium.org
  • 4 edits in trunk/WebKitTools

2010-08-19 Ojan Vafai <ojan@chromium.org>

Reviewed by Adam Barth.

add --webkit_patch_id to reitveld uploads
https://bugs.webkit.org/show_bug.cgi?id=44289

This is the first step in having rietveld store which
patches have been uploaded and getting rid of the
in-rietveld flag.

  • Scripts/webkitpy/common/net/rietveld.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/steps/postcodereview.py:
12:48 Changeset [65693] by johnnyg@google.com
  • 8 edits
    7 adds in trunk/LayoutTests

2010-08-19 John Gregg <johnnyg@google.com>

Unreviewed, chromium rebaselines.

Add new baselines after http://trac.webkit.org/changeset/65665/

  • platform/chromium-linux/svg/custom/pattern-rotate-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/pattern-rotate-expected.png: Added.
  • platform/chromium-linux/svg/transforms: Added.
  • platform/chromium-linux/svg/transforms/text-with-pattern-inside-transformed-html-expected.checksum: Added.
  • platform/chromium-linux/svg/transforms/text-with-pattern-inside-transformed-html-expected.png: Added.
  • platform/chromium-linux/svg/transforms/text-with-pattern-with-svg-transform-expected.checksum: Added.
  • platform/chromium-linux/svg/transforms/text-with-pattern-with-svg-transform-expected.png: Added.
  • platform/chromium-win/svg/custom/pattern-rotate-expected.checksum:
  • platform/chromium-win/svg/custom/pattern-rotate-expected.png:
  • platform/chromium-win/svg/transforms/text-with-pattern-inside-transformed-html-expected.checksum:
  • platform/chromium-win/svg/transforms/text-with-pattern-inside-transformed-html-expected.png:
  • platform/chromium-win/svg/transforms/text-with-pattern-with-svg-transform-expected.checksum:
  • platform/chromium-win/svg/transforms/text-with-pattern-with-svg-transform-expected.png:
  • platform/chromium/test_expectations.txt
12:32 Changeset [65692] by eric@webkit.org
  • 37 edits
    14 adds in trunk

2010-08-16 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Make DocumentParser safer to use
https://bugs.webkit.org/show_bug.cgi?id=43055

This also includes a test from
https://bugs.webkit.org/show_bug.cgi?id=42099

These tests were kindly provided by Nate Chapin and Adam Barth.

  • fast/frames/document-write-in-iframe-onload-expected.txt: Copied from LayoutTests/editing/selection/5136696-expected.txt.
  • fast/frames/document-write-in-iframe-onload.html: Added.
  • fast/frames/resources/set-src-to-javascript-url.html: Added.
  • fast/frames/set-parent-src-synchronously-expected.txt: Added.
  • fast/frames/set-parent-src-synchronously.html: Added.
  • fast/parser/document-close-iframe-load-expected.txt: Added.
  • fast/parser/document-close-iframe-load.html: Added.
  • fast/parser/document-close-nested-iframe-load-expected.txt: Added.
  • fast/parser/document-close-nested-iframe-load.html: Added.
  • fast/parser/iframe-sets-parent-to-javascript-url-expected.txt: Added.
  • fast/parser/iframe-sets-parent-to-javascript-url.html: Added.
  • fast/parser/resources/set-parent-to-javascript-url.html: Added.

2010-08-17 Eric Seidel <eric@webkit.org>

Reviewed by Adam Barth.

Make DocumentParser safer to use
https://bugs.webkit.org/show_bug.cgi?id=43055

Make DocumentParser and its subclasses RefCounted, and protect
HTMLDocumentParser during parsing. It's possible for a parser to
get deleted if certain actions (e.g., a document.write()) occur
synchronously.

The original version of this patch was written by Nate Chapin.

DocumentParser doesn't actually have to be fully RefCounted, since
the only two things which should ever hold a reference to it are
Document and DocumentParser itself. However using RefCounted and
RefPtr was easier/cleaner than inventing a custom ref() scheme.

This deploys a new "detach()" method throughout the parsing
framework. detach() causes the parser to disconnect from the
document so that no further modifications will be made to the
document while any possible DocumentParser stacks are unwound.

The irony of this patch is that the new detach() system is never
used, since Document always outlives the DocumentParser in all of
our layout tests. There is an ASSERT in ~Document() to verify
that the DocumentParser will not outlive the Document.

However I expect that we will soon either find new test cases, or change
the architecture in such a way that DocumentParser will outlive
Document. At which point, the detach() plumbing will be crucial.
Right now detach() serves as a safe-guard against use-after-free bugs
for any case where DocumentParser does outlive the Document.

This also fixes test cases attached to:
https://bugs.webkit.org/show_bug.cgi?id=42099

Tests: fast/frames/document-write-in-iframe-onload.html

fast/frames/set-parent-src-synchronously.html
fast/parser/document-close-iframe-load.html
fast/parser/document-close-nested-iframe-load.html
fast/parser/iframe-sets-parent-to-javascript-url.html

  • dom/Document.cpp:
    • Added a new detachParser() call to be used anywhere we used to call m_parser.clear(). There is an ASSERT in ~DocumentParser which ensures that we get this right.

(WebCore::Document::removedLastRef):
(WebCore::Document::~Document):
(WebCore::Document::createParser):
(WebCore::Document::detachParser):
(WebCore::Document::cancelParsing):
(WebCore::Document::implicitOpen):

  • removed redundant m_parser.clear()

(WebCore::Document::implicitClose):

  • dom/Document.h:
  • dom/DocumentParser.cpp: (WebCore::DocumentParser::~DocumentParser):
    • ASSERT that callers always call detach() before destruction.
    • This ASSERT might prove too cumbersome, but for now it's useful.

(WebCore::DocumentParser::detach):

  • dom/DocumentParser.h:
  • dom/RawDataDocumentParser.h:
  • dom/XMLDocumentParser.cpp: (WebCore::XMLDocumentParser::finish):
    • Add a FIXME explaining part of the reason why stopParsing() and detach() are separate concepts.
  • dom/XMLDocumentParser.h: (WebCore::XMLDocumentParser::create):
  • dom/XMLDocumentParserLibxml2.cpp: (WebCore::XMLDocumentParser::parseDocumentFragment):
  • dom/XMLDocumentParserQt.cpp: (WebCore::XMLDocumentParser::parseDocumentFragment):
  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::detach): (WebCore::HTMLConstructionSite::dispatchDocumentElementAvailableIfNeeded):
  • html/HTMLConstructionSite.h:
  • html/HTMLDocument.cpp: (WebCore::HTMLDocument::createParser):
  • html/HTMLDocument.h:
  • html/HTMLDocumentParser.cpp:
    • We need to protect(this) before calling into any code which might cause the parser to be destroyed.

(WebCore::HTMLDocumentParser::~HTMLDocumentParser):
(WebCore::HTMLDocumentParser::detach):
(WebCore::HTMLDocumentParser::resumeParsingAfterYield):
(WebCore::HTMLDocumentParser::pumpTokenizer):
(WebCore::HTMLDocumentParser::insert):
(WebCore::HTMLDocumentParser::append):
(WebCore::HTMLDocumentParser::end):
(WebCore::HTMLDocumentParser::finish):
(WebCore::HTMLDocumentParser::notifyFinished):
(WebCore::HTMLDocumentParser::executeScriptsWaitingForStylesheets):
(WebCore::HTMLDocumentParser::parseDocumentFragment):

  • html/HTMLDocumentParser.h: (WebCore::HTMLDocumentParser::create):
  • html/HTMLScriptRunner.cpp: (WebCore::HTMLScriptRunner::detach): (WebCore::HTMLScriptRunner::executeParsingBlockingScript): (WebCore::HTMLScriptRunner::executeScript): (WebCore::HTMLScriptRunner::executeScriptsWaitingForStylesheets): (WebCore::HTMLScriptRunner::runScript):
  • html/HTMLScriptRunner.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::detach): (WebCore::HTMLTreeBuilder::passTokenToLegacyParser): (WebCore::HTMLTreeBuilder::finished):
  • html/HTMLTreeBuilder.h:
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::createParser):
  • html/HTMLViewSourceDocument.h:
  • html/HTMLViewSourceParser.cpp: (WebCore::HTMLViewSourceParser::HTMLViewSourceParser):
  • html/HTMLViewSourceParser.h: (WebCore::HTMLViewSourceParser::create):
  • loader/FTPDirectoryDocument.cpp: (WebCore::FTPDirectoryDocumentParser::create): (WebCore::FTPDirectoryDocument::createParser):
  • loader/FTPDirectoryDocument.h:
  • loader/ImageDocument.cpp: (WebCore::ImageDocumentParser::create): (WebCore::ImageDocumentParser::ImageDocumentParser): (WebCore::ImageDocument::createParser):
  • loader/ImageDocument.h:
  • loader/MediaDocument.cpp: (WebCore::MediaDocumentParser::create): (WebCore::MediaDocument::createParser):
  • loader/MediaDocument.h:
  • loader/PluginDocument.cpp: (WebCore::PluginDocumentParser::create): (WebCore::PluginDocument::createParser):
  • loader/PluginDocument.h:
  • loader/SinkDocument.cpp: (WebCore::SinkDocumentParser::create): (WebCore::SinkDocument::createParser):
  • loader/SinkDocument.h:
  • loader/TextDocument.cpp: (WebCore::TextDocumentParser::create): (WebCore::TextDocument::createParser): (WebCore::createTextDocumentParser):
  • loader/TextDocument.h:
12:29 Changeset [65691] by ddkilzer@apple.com
  • 2 edits in trunk/WebCore

Fix compilation of SelectElement.cpp with ARROW_KEYS_POP_MENU == 0

Reviewed by Simon Fraser.

  • dom/SelectElement.cpp:

(WebCore::SelectElement::menuListDefaultEventHandler): Added
UNUSED_PARAM() macro for htmlForm argument when
ARROW_KEYS_POP_MENU defined as zero.

12:07 Changeset [65690] by laszlo.1.gombos@nokia.com
  • 2 edits in trunk/WebCore

2010-08-19 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Disable XSLT for production configuration
https://bugs.webkit.org/show_bug.cgi?id=37445

Introduce a new CONFIG option to maintain stable configuration for
the QtWebKit port and disable XSLT for the stable configuration
because of bug 37445.

No new tests as there is no new functionality.

  • features.pri:
11:59 Changeset [65689] by zmo@google.com
  • 13 edits
    2 adds in trunk

2010-08-18 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Implement strict type checking in js bindings for WebGL functions
https://bugs.webkit.org/show_bug.cgi?id=44202

Test: fast/canvas/webgl/bad-arguments-test.html

  • bindings/js/JSWebGLRenderingContextCustom.cpp: Add type check for wrapper types and DOMString and throw error in case of mismatch. (WebCore::JSWebGLRenderingContext::getAttachedShaders): (WebCore::JSWebGLRenderingContext::getProgramParameter): (WebCore::JSWebGLRenderingContext::getShaderParameter): (WebCore::JSWebGLRenderingContext::getUniform): (WebCore::dataFunctionf): (WebCore::dataFunctioni): (WebCore::dataFunctionMatrix):
  • bindings/scripts/CodeGeneratorJS.pm: Ditto.
  • bindings/scripts/CodeGeneratorV8.pm: Ditto.
  • bindings/v8/custom/V8WebGLRenderingContextCustom.cpp: Ditto. (WebCore::V8WebGLRenderingContext::getAttachedShadersCallback): (WebCore::V8WebGLRenderingContext::getProgramParameterCallback): (WebCore::V8WebGLRenderingContext::getShaderParameterCallback): (WebCore::V8WebGLRenderingContext::getUniformCallback): (WebCore::vertexAttribAndUniformHelperf): (WebCore::uniformHelperi): (WebCore::uniformMatrixHelper):
  • html/canvas/WebGLRenderingContext.idl: Add attribute StrictTypeChecking for WebGL functions.

2010-08-18 Zhenyao Mo <zmo@google.com>

Reviewed by Kenneth Russell.

Implement strict type checking in js bindings for WebGL functions
https://bugs.webkit.org/show_bug.cgi?id=44202

  • fast/canvas/webgl/bad-arguments-test-expected.txt: Added.
  • fast/canvas/webgl/bad-arguments-test.html: Added.
  • fast/canvas/webgl/error-reporting-expected.txt: Fix bugs due to strict type checking in the js bindings.
  • fast/canvas/webgl/error-reporting.html: ditto.
  • fast/canvas/webgl/gl-object-get-calls-expected.txt: ditto.
  • fast/canvas/webgl/gl-object-get-calls.html: ditto.
  • fast/canvas/webgl/null-object-behaviour-expected.txt: ditto.
  • fast/canvas/webgl/null-object-behaviour.html: ditto.
11:45 Changeset [65688] by commit-queue@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-08-19 Vincent Scheib <scheib@chromium.org>

Reviewed by David Levin.

Fixing previous Mac only build breakage with TilingDataTest.cpp
https://bugs.webkit.org/show_bug.cgi?id=44223

TilingDataTest.cpp included the wrong header, which had a compile
option around the definition of the WebCore namespace. Without the
namespace defined, error. Fixed by correctly including TilingData.h,
which has no conditional compilation.

  • WebKit.gyp:
  • tests/TilingDataTest.cpp:
11:44 Changeset [65687] by johnnyg@google.com
  • 2 edits in trunk/LayoutTests

2010-08-19 John Gregg <johnnyg@google.com>

Unreviewed, test expectations.

Add some new exceptions for SVG failures, see
https://bugs.webkit.org/show_bug.cgi?id=44278

  • platform/chromium/test_expectations.txt:
11:32 WebKit Team edited by kbr@google.com
Adjusting my (kbr's) status from committer to reviewer. (diff)
11:28 Changeset [65686] by tony@chromium.org
  • 2 edits in trunk/BugsSite

2010-08-19 Tony Chang <tony@chromium.org>

Reviewed by Adam Barth.

webkit-patch barfed on upload with a new image test result
https://bugs.webkit.org/show_bug.cgi?id=39097

  • PrettyPatch/PrettyPatch.rb: don't use full path to git
11:25 Changeset [65685] by kbr@google.com
  • 2 edits in trunk/WebKitTools

2010-08-19 Kenneth Russell <kbr@google.com>

Reviewed by David Levin.

Adjust my status from committer to reviewer
https://bugs.webkit.org/show_bug.cgi?id=44277

  • Scripts/webkitpy/common/config/committers.py:
11:13 Changeset [65684] by eric.carlson@apple.com
  • 3 edits in trunk/LayoutTests

2010-08-19 Eric Carlson <eric.carlson@apple.com>

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=33671
media/video-loop.html fails intermittently on GTK and Chromium bots

Rewrite test to make it less timing dependent.

  • media/video-loop-expected.txt:
  • media/video-loop.html:
10:56 Changeset [65683] by simon.fraser@apple.com
  • 2 edits in trunk/WebCore

2010-08-19 Simon Fraser <simon.fraser@apple.com>

Fix Chromium build.

  • svg/SVGElement.cpp: (WebCore::SVGElement::attributeChanged):
10:35 Changeset [65682] by johnnyg@google.com
  • 3 edits in trunk/LayoutTests

2010-08-19 John Gregg <johnnyg@google.com>

Unreviewed; test expectations.

Update chromium-mac expectations for zoom-mask-with-percentages.svg after
http://trac.webkit.org/changeset/65665/

  • platform/chromium-mac/svg/zoom/page/zoom-mask-with-percentages-expected.checksum:
  • platform/chromium-mac/svg/zoom/page/zoom-mask-with-percentages-expected.png:
10:19 Changeset [65681] by simon.fraser@apple.com
  • 19 edits in trunk/WebCore

2010-08-19 Simon Fraser <simon.fraser@apple.com>

Reviewed by Nikolas Zimmermann.

HTMLElement::isContentEditable() can cause an updateStyleIfNeeded() to happen in the middle of layout
https://bugs.webkit.org/show_bug.cgi?id=21834
<rdar://problem/8093653&8261394>

If we're in the middle of layout, or painting, and something causes updateStyleIfNeeded() to
get called, then we can end up entering recalcStyle() during layout or painting. This is bad
because it can create/destry the renderers and RenderLayers which are in use by layout/painting.
This is the cause of a number of random crashers, some of which show up more frequently
in content which uses accelerated compositing.

The changes here:

  1. Add an assertion in Document::updateStyleIfNeeded() that we are not laying out or painting.
  2. Remove calls to updateStyleIfNeeded() in editing and caret painting code
  3. Pass along information to CTM and BBox-related SVG methods to indicate whether it's safe to update style.

Tested by new assertions and existing tests.

  • dom/Document.cpp: (WebCore::Document::updateStyleIfNeeded): New assertion that we are not mid-layout or painting. (WebCore::command): Call updateStyleIfNeeded() to ensure that subsequent calls to isContentEditable() return the correct result.
  • dom/Element.cpp: (WebCore::Element::focus): Move the supportsFocus() call to after style has been updated.
  • editing/SelectionController.cpp: (WebCore::SelectionController::localCaretRect): (WebCore::SelectionController::caretRepaintRect): (WebCore::SelectionController::paintCaret):
  • editing/SelectionController.h: (WebCore::SelectionController::localCaretRectForPainting): When painting, use localCaretRectForPainting() which does not update style. Make localCaretRect() non-const so allowing it to update style without ugly casts.
  • html/HTMLElement.cpp: (WebCore::HTMLElement::isContentEditable): Don't call updateStyleIfNeeded() here. (WebCore::HTMLElement::isContentRichlyEditable): Ditto. (WebCore::HTMLElement::contentEditable): Ditto.
  • page/FrameView.h: (WebCore::FrameView::isMidLayout): New accessor, used for asserting.
  • rendering/RenderPath.cpp: (WebCore::fillAndStrokePath): Pass DisallowStyleUpdate to getScreenCTM since we are painting.
  • rendering/RenderSVGResourceContainer.cpp: (WebCore::RenderSVGResourceContainer::transformOnNonScalingStroke): This is only called when painting, so use DisallowStyleUpdate.
  • svg/SVGElement.cpp: (WebCore::SVGElement::attributeChanged): Changes to the style attribute should not have side effects, since a call to Element::getAttribute() is allowed to result in a call to setAttribute() for the style attribute. To avoid updateStyleIfNeeded() during painting, this must not cause SVG to do extra work.
  • svg/SVGLocatable.cpp: Pass StyleUpdateStrategy down to these methods to indicate whether it's OK to update style. (WebCore::SVGLocatable::getBBox): (WebCore::SVGLocatable::computeCTM): (WebCore::SVGLocatable::getTransformToElement):
  • svg/SVGLocatable.h: (WebCore::SVGLocatable::):
  • svg/SVGStyledLocatableElement.cpp: (WebCore::SVGStyledLocatableElement::getBBox): (WebCore::SVGStyledLocatableElement::getCTM): (WebCore::SVGStyledLocatableElement::getScreenCTM):
  • svg/SVGStyledLocatableElement.h:
  • svg/SVGStyledTransformableElement.cpp: (WebCore::SVGStyledTransformableElement::getCTM): (WebCore::SVGStyledTransformableElement::getScreenCTM): (WebCore::SVGStyledTransformableElement::getBBox):
  • svg/SVGStyledTransformableElement.h:
  • svg/SVGTextElement.cpp: (WebCore::SVGTextElement::getBBox): (WebCore::SVGTextElement::getCTM): (WebCore::SVGTextElement::getScreenCTM):
  • svg/SVGTextElement.h:
10:16 Changeset [65680] by rniwa@webkit.org
  • 2 edits in trunk/WebCore

2010-08-19 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Range, EAnnotateForInterchange, and EAbsoluteURLs should be member variables of MarkupAccumulator
https://bugs.webkit.org/show_bug.cgi?id=44229

No new tests are added since this is a clean up.

  • editing/markup.cpp: (WebCore::MarkupAccumulator::MarkupAccumulator): Added shouldResolveURLs, shouldAnnotate, and range. (WebCore::MarkupAccumulator::shouldResolveURLs): Private inline accessor for m_shouldResolveURLs. (WebCore::MarkupAccumulator::shouldAnnotate): Private inline accessor for m_shouldAnnotate. (WebCore::MarkupAccumulator::appendEndTag): Renamed local variable result to markup. (WebCore::MarkupAccumulator::appendStartTag): Removed range, annotate, shouldResolveURLs from the function arguments. (WebCore::MarkupAccumulator::wrapWithNode): Ditto. (WebCore::MarkupAccumulator::appendStartMarkup): Ditto. (WebCore::serializeNodes): Ditto and uses MarkupAccumulator. (WebCore::createMarkup): Uses MarkupAccumulator. (WebCore::serializeNodesWithNamespaces): Ditto.
10:04 Changeset [65679] by tony@chromium.org
  • 1 edit in trunk/BugsSite/ChangeLog

2010-08-19 Tony Chang <tony@chromium.org>

Unreviewed, setting svn:eol-style native on ChangeLog.

09:54 Changeset [65678] by tony@chromium.org
  • 1 edit in trunk/BugsSite/ChangeLog

2010-08-19 Tony Chang <tony@chromium.org>

Unreviewed. Just removing \r's from ChangeLog.

09:46 Changeset [65677] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: do not dump inspector errors into the inspected window console.
https://bugs.webkit.org/show_bug.cgi?id=44272

  • inspector/front-end/inspector.js: (WebInspector.reportProtocolError):
09:11 Changeset [65676] by jochen@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-08-19 Jochen Eisinger <jochen@chromium.org>

Reviewed by Pavel Feldman.

concatenated_devtools_js should depend on inspector_protocol_sources
https://bugs.webkit.org/show_bug.cgi?id=44255

  • WebKit.gyp:
08:47 Changeset [65675] by weinig@apple.com
  • 5 edits in trunk/WebKit2

Add ability to get all the keys from a WKDictionaryRef
https://bugs.webkit.org/show_bug.cgi?id=44221

Reviewed by Adam Roben.

  • Shared/ImmutableDictionary.cpp:

(WebKit::ImmutableDictionary::keys):

  • Shared/ImmutableDictionary.h:

Add keys function.

  • UIProcess/API/C/WKDictionary.cpp:

(WKDictionaryCopyKeys):

  • UIProcess/API/C/WKDictionary.h:

Wrap keys function.

08:44 Changeset [65674] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-08-19 Pavel Podivilov <podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: activate/deactivate breakpoints using v8 debugger flag.
https://bugs.webkit.org/show_bug.cgi?id=44112

  • src/js/DebuggerScript.js: ():
08:19 Changeset [65673] by jorlow@chromium.org
  • 3 edits in trunk/WebCore

2010-08-19 Jeremy Orlow <jorlow@chromium.org>

Chromium build fix.

Use ASSERT_UNUSED on variables that aren't used besides by the ASSERT.

  • storage/IDBDatabaseBackendImpl.cpp: (WebCore::IDBDatabaseBackendImpl::createObjectStore): (WebCore::IDBDatabaseBackendImpl::removeObjectStore): (WebCore::IDBDatabaseBackendImpl::loadObjectStores):
  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::IDBObjectStoreBackendImpl::get): (WebCore::IDBObjectStoreBackendImpl::put): (WebCore::IDBObjectStoreBackendImpl::remove): (WebCore::IDBObjectStoreBackendImpl::createIndex): (WebCore::IDBObjectStoreBackendImpl::removeIndex): (WebCore::IDBObjectStoreBackendImpl::openCursor): (WebCore::IDBObjectStoreBackendImpl::loadIndexes):
08:06 Changeset [65672] by commit-queue@webkit.org
  • 5 edits in trunk

2010-08-19 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: expose Panel.onSelectionChanged in extension API
https://bugs.webkit.org/show_bug.cgi?id=44171

  • inspector/extensions-api-expected.txt:
  • inspector/extensions-expected.txt:

2010-08-19 Andrey Kosyakov <caseq@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: expose Panel.onSelectionChanged in extension API
https://bugs.webkit.org/show_bug.cgi?id=44171

  • inspector/front-end/ExtensionAPI.js: (injectedExtensionAPI.PanelImpl): (injectedExtensionAPI):
08:04 Changeset [65671] by pfeldman@chromium.org
  • 3 edits in trunk/WebKitTools

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: remove injected script fetcher from test harness.
https://bugs.webkit.org/show_bug.cgi?id=44266

  • DumpRenderTree/chromium/DRTDevToolsAgent.cpp:
  • DumpRenderTree/chromium/DRTDevToolsAgent.h:
07:53 Changeset [65670] by andreip@google.com
  • 32 edits
    17 copies
    5 adds in trunk

2010-08-19 Andrei Popescu <andreip@google.com>

Reviewed by Jeremy Orlow.

[IndexedDB] Abort idle IDBTransactions when the JS context they were created in finishes execution.
https://bugs.webkit.org/show_bug.cgi?id=44101

  • storage/indexeddb/resources/shared.js: (unexpectedErrorCallback):
  • storage/indexeddb/script-tests/transaction-basics.js: Added. (abortCallback): (createTransactionCallback): (test):
  • storage/indexeddb/transaction-basics-expected.txt: Added.
  • storage/indexeddb/transaction-basics.html: Added.

2010-08-19 Andrei Popescu <andreip@google.com>

Reviewed by Jeremy Orlow.

[IndexedDB] Abort idle IDBTransactions when the JS context they were created in finishes execution.
https://bugs.webkit.org/show_bug.cgi?id=44101

This change also introduces the TransactionCoordinator, which will be used
to schedule transactions for execution.

Test: storage/indexeddb/transaction-basics.html

  • WebCore.gypi:
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventTargetToV8Object):
  • bindings/v8/V8Proxy.cpp: (WebCore::V8Proxy::runScript): (WebCore::V8Proxy::callFunction): (WebCore::V8Proxy::didLeaveScriptContext):
  • bindings/v8/V8Proxy.h:
  • dom/EventTarget.cpp: (WebCore::EventTarget::toIDBTransaction):
  • dom/EventTarget.h:
  • page/PageGroup.h: (WebCore::PageGroup::hasIDBFactory):
  • storage/IDBAbortEvent.cpp: Added. (WebCore::IDBAbortEvent::create): (WebCore::IDBAbortEvent::IDBAbortEvent): (WebCore::IDBAbortEvent::~IDBAbortEvent):
  • storage/IDBAbortEvent.h: Added. (WebCore::IDBAbortEvent::isIDBAbortEvent):
  • storage/IDBDatabase.cpp: (WebCore::IDBDatabase::transaction):
  • storage/IDBDatabaseBackendImpl.cpp: (WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl): (WebCore::IDBDatabaseBackendImpl::transaction):
  • storage/IDBDatabaseBackendImpl.h: (WebCore::IDBDatabaseBackendImpl::create):
  • storage/IDBDatabaseBackendInterface.h:
  • storage/IDBFactoryBackendImpl.cpp: (WebCore::IDBFactoryBackendImpl::IDBFactoryBackendImpl): (WebCore::IDBFactoryBackendImpl::open): (WebCore::IDBFactoryBackendImpl::abortPendingTransactions):
  • storage/IDBFactoryBackendImpl.h:
  • storage/IDBFactoryBackendInterface.h:
  • storage/IDBPendingTransactionMonitor.cpp: Added. (WebCore::IDBPendingTransactionMonitor::addPendingTransaction): (WebCore::IDBPendingTransactionMonitor::removePendingTransaction): (WebCore::IDBPendingTransactionMonitor::clearPendingTransactions): (WebCore::IDBPendingTransactionMonitor::pendingTransactions):
  • storage/IDBPendingTransactionMonitor.h: Added.
  • storage/IDBTransaction.cpp: (WebCore::IDBTransaction::IDBTransaction): (WebCore::IDBTransaction::onAbort): (WebCore::IDBTransaction::id): (WebCore::IDBTransaction::stop): (WebCore::IDBTransaction::timerFired):
  • storage/IDBTransaction.h:
  • storage/IDBTransactionBackendInterface.h:
  • storage/IDBTransactionCallbacks.h: Added. (WebCore::IDBTransactionCallbacks::~IDBTransactionCallbacks):
  • storage/IDBTransactionCoordinator.cpp: Added. (WebCore::IDBTransactionBackendImpl::~IDBTransactionBackendImpl): (WebCore::IDBTransactionBackendImpl::create): (WebCore::IDBTransactionBackendImpl::IDBTransactionBackendImpl): (WebCore::IDBTransactionBackendImpl::objectStore): (WebCore::IDBTransactionBackendImpl::mode): (WebCore::IDBTransactionBackendImpl::scheduleTask): (WebCore::IDBTransactionBackendImpl::abort): (WebCore::IDBTransactionBackendImpl::sqliteDatabase): (WebCore::IDBTransactionBackendImpl::id): (WebCore::IDBTransactionBackendImpl::setCallbacks): (WebCore::IDBTransactionCoordinator::IDBTransactionCoordinator): (WebCore::IDBTransactionCoordinator::~IDBTransactionCoordinator): (WebCore::IDBTransactionCoordinator::createTransaction): (WebCore::IDBTransactionCoordinator::abort):
  • storage/IDBTransactionCoordinator.h: Added. (WebCore::IDBTransactionCoordinator::create):

2010-08-19 Andrei Popescu <andreip@google.com>

Reviewed by Jeremy Orlow.

[IndexedDB] Abort idle IDBTransactions when the JS context they were created in finishes execution.
https://bugs.webkit.org/show_bug.cgi?id=44101

  • WebKit.gyp:
  • public/WebIDBDatabase.h: (WebKit::WebIDBDatabase::transaction):
  • public/WebIDBFactory.h: (WebKit::WebIDBFactory::abortPendingTransactions):
  • public/WebIDBTransaction.h: Added. (WebKit::WebIDBTransaction::~WebIDBTransaction): (WebKit::WebIDBTransaction::mode): (WebKit::WebIDBTransaction::objectStore): (WebKit::WebIDBTransaction::abort): (WebKit::WebIDBTransaction::id): (WebKit::WebIDBTransaction::setCallbacks):
  • public/WebIDBTransactionCallbacks.h: Added. (WebKit::WebIDBTransactionCallbacks::~WebIDBTransactionCallbacks): (WebKit::WebIDBTransactionCallbacks::onAbort): (WebKit::WebIDBTransactionCallbacks::id):
  • src/IDBDatabaseProxy.cpp: (WebCore::IDBDatabaseProxy::transaction):
  • src/IDBFactoryBackendProxy.cpp: (WebCore::IDBFactoryBackendProxy::abortPendingTransactions):
  • src/IDBFactoryBackendProxy.h:
  • src/IDBTransactionBackendProxy.cpp: Added. (WebCore::IDBTransactionBackendProxy::create): (WebCore::IDBTransactionBackendProxy::IDBTransactionBackendProxy): (WebCore::IDBTransactionBackendProxy::~IDBTransactionBackendProxy): (WebCore::IDBTransactionBackendProxy::objectStore): (WebCore::IDBTransactionBackendProxy::mode): (WebCore::IDBTransactionBackendProxy::abort): (WebCore::IDBTransactionBackendProxy::scheduleTask): (WebCore::IDBTransactionBackendProxy::sqliteDatabase): (WebCore::IDBTransactionBackendProxy::id): (WebCore::IDBTransactionBackendProxy::setCallbacks):
  • src/IDBTransactionBackendProxy.h: Added.
  • src/IDBTransactionCallbacksProxy.cpp: Added. (WebCore::IDBTransactionCallbacksProxy::create): (WebCore::IDBTransactionCallbacksProxy::IDBTransactionCallbacksProxy): (WebCore::IDBTransactionCallbacksProxy::~IDBTransactionCallbacksProxy): (WebCore::IDBTransactionCallbacksProxy::onAbort): (WebCore::IDBTransactionCallbacksProxy::id):
  • src/IDBTransactionCallbacksProxy.h: Added.
  • src/WebDOMStringList.cpp: (WebKit::WebDOMStringList::length):
  • src/WebIDBDatabaseImpl.cpp: (WebKit::WebIDBDatabaseImpl::transaction):
  • src/WebIDBDatabaseImpl.h:
  • src/WebIDBFactoryImpl.cpp: (WebKit::WebIDBFactoryImpl::abortPendingTransactions):
  • src/WebIDBFactoryImpl.h:
  • src/WebIDBTransactionCallbacksImpl.cpp: Added. (WebCore::WebIDBTransactionCallbacksImpl::WebIDBTransactionCallbacksImpl): (WebCore::WebIDBTransactionCallbacksImpl::~WebIDBTransactionCallbacksImpl): (WebCore::WebIDBTransactionCallbacksImpl::onAbort): (WebCore::WebIDBTransactionCallbacksImpl::id):
  • src/WebIDBTransactionCallbacksImpl.h: Added.
  • src/WebIDBTransactionImpl.cpp: Added. (WebKit::WebIDBTransactionImpl::WebIDBTransactionImpl): (WebKit::WebIDBTransactionImpl::~WebIDBTransactionImpl): (WebKit::WebIDBTransactionImpl::mode): (WebKit::WebIDBTransactionImpl::objectStore): (WebKit::WebIDBTransactionImpl::abort): (WebKit::WebIDBTransactionImpl::id): (WebKit::WebIDBTransactionImpl::setCallbacks):
  • src/WebIDBTransactionImpl.h: Added. (WebKit::):
07:47 Changeset [65669] by pfeldman@chromium.org
  • 8 edits in trunk

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: There is no need in resource-based InjectedScript.js source.
Now that we populate front-end after its onload handler, we don't need
to install injected script early. Exposing injected script source on the WebCore
level here.
https://bugs.webkit.org/show_bug.cgi?id=44029

  • inspector/InjectedScriptHost.h: (WebCore::InjectedScriptHost::injectedScriptSource):

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: There is no need in resource-based InjectedScript.js source.
Now that we populate front-end after its onload handler, I don't think we need
to install injected script early.
https://bugs.webkit.org/show_bug.cgi?id=44029

  • public/WebDevToolsAgentClient.h:
  • src/WebDevToolsAgentImpl.cpp: (WebKit::): (WebKit::WebDevToolsAgentImpl::attach): (WebKit::WebDevToolsAgentImpl::frontendLoaded):

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: There is no need in resource-based InjectedScript.js source.
Now that we populate front-end after its onload handler, we don't need
to install injected script early. Exposing injected script source on the WebCore
level here.
https://bugs.webkit.org/show_bug.cgi?id=44029

  • DumpRenderTree/chromium/DRTDevToolsAgent.cpp: (DRTDevToolsAgent::runtimePropertyChanged): (DRTDevToolsAgent::delayedFrontendLoaded): (DRTDevToolsAgent::frontendLoaded):
  • DumpRenderTree/chromium/DRTDevToolsAgent.h:
07:47 Changeset [65668] by girish@forwardbias.in
  • 2 edits in trunk/WebKit/qt

[Qt] Inject wmode=opaque for both QWebView and QGraphicsWebView on Maemo5
as Flash XEmbed support is flaky.

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

Reviewed by Kenneth Christiansen.

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::createPlugin):

07:38 Changeset [65667] by jorlow@chromium.org
  • 16 edits in trunk

2010-08-18 Jeremy Orlow <jorlow@chromium.org>

Implement persistance for IndexedDB ObjectStores
https://bugs.webkit.org/show_bug.cgi?id=44164

Remove old object stores before running test.

  • storage/indexeddb/resources/shared.js: (deleteAllObjectStores):
  • storage/indexeddb/script-tests/objectstore-basics.js: (openSuccess):
  • storage/indexeddb/script-tests/open-cursor.js: (openSuccess):

2010-08-18 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Implement persistance for IndexedDB ObjectStores
https://bugs.webkit.org/show_bug.cgi?id=44164

Get rid of the in-memory storage of ObjectStores and instead
use SQL backed memory.

Existing tests give most of the coverage. Will update the manual
test in the next patch (which will also re-work database.description).

  • manual-tests/indexed-database.html: Get rid of race where you can click before the page loads.
  • platform/sql/SQLiteStatement.cpp: Add two helper functions (WebCore::SQLiteStatement::bindInt): (WebCore::SQLiteStatement::isColumnNull):
  • platform/sql/SQLiteStatement.h:
  • storage/IDBCursorBackendImpl.cpp:
  • storage/IDBDatabaseBackendImpl.cpp: (WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl): (WebCore::IDBDatabaseBackendImpl::createObjectStore): (WebCore::IDBDatabaseBackendImpl::removeObjectStore): (WebCore::IDBDatabaseBackendImpl::importObjectStores):
  • storage/IDBDatabaseBackendImpl.h: (WebCore::IDBDatabaseBackendImpl::sqliteDatabase):
  • storage/IDBFactoryBackendImpl.cpp: (WebCore::createTables):
  • storage/IDBIndexBackendImpl.cpp: (WebCore::IDBIndexBackendImpl::IDBIndexBackendImpl): (WebCore::IDBIndexBackendImpl::sqliteDatabase):
  • storage/IDBIndexBackendImpl.h: (WebCore::IDBIndexBackendImpl::create):
  • storage/IDBObjectStoreBackendImpl.cpp: (WebCore::IDBObjectStoreBackendImpl::IDBObjectStoreBackendImpl): (WebCore::whereClause): (WebCore::bindKey): (WebCore::IDBObjectStoreBackendImpl::get): (WebCore::IDBObjectStoreBackendImpl::put): (WebCore::IDBObjectStoreBackendImpl::remove): (WebCore::IDBObjectStoreBackendImpl::createIndex): (WebCore::IDBObjectStoreBackendImpl::removeIndex): (WebCore::IDBObjectStoreBackendImpl::openCursor): (WebCore::IDBObjectStoreBackendImpl::importIndexes): (WebCore::IDBObjectStoreBackendImpl::sqliteDatabase):
  • storage/IDBObjectStoreBackendImpl.h: (WebCore::IDBObjectStoreBackendImpl::create): (WebCore::IDBObjectStoreBackendImpl::database):
06:30 Changeset [65666] by kov@webkit.org
  • 13 edits in trunk

LayoutTests

2010-08-19 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Kenneth Rohde Christiansen.

[Gtk] Enable view mode media feature layout test
https://bugs.webkit.org/show_bug.cgi?id=43278

Unskip tests that are now passing.

  • platform/gtk/Skipped:

WebKit/gtk

2010-08-19 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Kenneth Rohde Christiansen.

[Gtk] Enable view mode media feature layout test
https://bugs.webkit.org/show_bug.cgi?id=43278

Add the view-mode property, to control the WebCore page's viewMode.

  • docs/webkitgtk-sections.txt:
  • webkit/webkitwebview.cpp: (webkit_web_view_get_property): (webkit_web_view_set_property): (webkit_web_view_class_init): (webkit_web_view_set_view_mode): (webkit_web_view_get_view_mode):
  • webkit/webkitwebview.h:

WebKitTools

2010-08-19 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Kenneth Rohde Christiansen.

[Gtk] Enable view mode media feature layout test
https://bugs.webkit.org/show_bug.cgi?id=43278

Add view mode media feature testing infrastructure, and add
placeholder methods for the other platforms'
LayoutTestControllers.

  • DumpRenderTree/LayoutTestController.cpp: (setViewModeMediaFeatureCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setViewModeMediaFeature):
  • DumpRenderTree/mac/LayoutTestControllerMac.mm: (LayoutTestController::setViewModeMediaFeature):
  • DumpRenderTree/win/LayoutTestControllerWin.cpp: (LayoutTestController::setViewModeMediaFeature):
  • DumpRenderTree/wx/LayoutTestControllerWx.cpp: (LayoutTestController::setViewModeMediaFeature):
05:41 Changeset [65665] by zimmermann@webkit.org
  • 76 edits
    16 adds in trunk

2010-08-19 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Pattern is rasterized
https://bugs.webkit.org/show_bug.cgi?id=41396

SVG pattern size changed when resizing browser
https://bugs.webkit.org/show_bug.cgi?id=38704

Incorrect pattern tilling
https://bugs.webkit.org/show_bug.cgi?id=41603

Apply same fixes to <pattern> that <mask> recently received. Build the tile image in absolute coordinates,
to avoid pixelation, when the target element is scaled. Also fixes problems when zooming into patterns.

Clamp ImageBuffer sizes to RenderSVGRoots viewport, now that gradient/pattern and mask buffers are created
in absolute coordinates -> no more huge image buffer allocations, that could fail.

Drop overflow="visible" support for <pattern>, which complicates the code a lot. Neither Opera nor Firefox
support this, and SVG 1.1 2nd Edition says that the rendering behaviour is "undefined".

Tests: svg/batik/paints/patternRegions-positioned-objects.svg

svg/custom/pattern-incorrect-tiling.svg
svg/custom/pattern-no-pixelation.svg
svg/transforms/text-with-mask-with-svg-transform.svg

  • rendering/PaintInfo.h: (WebCore::PaintInfo::applyTransform): Don't alter the repaint rect if it's infinite (only affects SVG). (WebCore::PaintInfo::infiniteRect): Moved from RenderLayer into PaintInfo, as applyTransform() needs it.
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::calculateClipRects): s/ClipRects::infiniteRect/PaintInfo::infiniteRect/
  • rendering/RenderLayer.h: Moved infiniteRect() to PaintInfo.
  • rendering/RenderLayerBacking.cpp: (WebCore::clipBox): s/ClipRects::infiniteRect/PaintInfo::infiniteRect/
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::clippedByAncestor): Ditto.
  • rendering/RenderSVGResourceGradient.cpp: (WebCore::createMaskAndSwapContextForTextGradient): Clamp image buffer size, use new SVGImageBufferTools helper methods for that. (WebCore::clipToTextMask): Adapt to renames/code changes in SVGImageBufferTools. (WebCore::RenderSVGResourceGradient::applyResource): Ditto.
  • rendering/RenderSVGResourceMasker.cpp: (WebCore::RenderSVGResourceMasker::applyResource): lamp image buffer size, use new SVGImageBufferTools helper methods for that. (WebCore::RenderSVGResourceMasker::drawContentIntoMaskImage): Adapt to renames/code changes in SVGImageBufferTools.
  • rendering/RenderSVGResourceMasker.h:
  • rendering/RenderSVGResourcePattern.cpp: (WebCore::RenderSVGResourcePattern::applyResource): Rewrite to create tile image buffers in absolute coordinates, avoids pixelation artefacts. (WebCore::calculatePatternBoundaries): (WebCore::RenderSVGResourcePattern::buildTileImageTransform): (WebCore::RenderSVGResourcePattern::createTileImage):
  • rendering/RenderSVGResourcePattern.h:
  • rendering/SVGImageBufferTools.cpp: (WebCore::SVGImageBufferTools::transformationToOutermostSVGCoordinateSystem): Helper method, concating all localToParentTransforms() up until RenderSVGRoot is reached. (WebCore::SVGImageBufferTools::createImageBuffer): Simplified. (WebCore::SVGImageBufferTools::clipToImageBuffer): Rename absoluteTargetRect to clampedAbsoluteTargetRect. (WebCore::SVGImageBufferTools::roundedImageBufferSize): New helper method to centralize FloatSize -> IntSize rounding, when creating image buffers. (WebCore::SVGImageBufferTools::clampedAbsoluteTargetRectForRenderer): New helper methods to clamp an image buffer rect against the viewport.
  • rendering/SVGImageBufferTools.h:
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderSupport::renderSubtreeToImage): Use PaintInfo::infiniteRect() as repaint rect, to avoid any culling. (WebCore::SVGRenderSupport::findTreeRootObject): Renamed from svgRootTreeObject, and expose it. (WebCore::SVGRenderSupport::layoutChildren): s/svgRootTreeObject/findTreeRootObject/.
  • rendering/SVGRenderSupport.h:
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::setCurrentTranslate): Be sure to relayout on invalidation, otherwhise resources are not updated, and we have to update them, as they depend on the absolute transform.

2010-08-19 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Pattern is rasterized
https://bugs.webkit.org/show_bug.cgi?id=41396

SVG pattern size changed when resizing browser
https://bugs.webkit.org/show_bug.cgi?id=38704

Incorrect pattern tilling
https://bugs.webkit.org/show_bug.cgi?id=41603

Update pattern results after fixing the pixelation issues.

  • platform/mac-leopard/svg/batik/paints/patternRegions-expected.checksum:
  • platform/mac-leopard/svg/batik/paints/patternRegions-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-08-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-08-b-expected.png:
  • platform/mac/svg/batik/paints/patternRegions-positioned-objects-expected.checksum: Added.
  • platform/mac/svg/batik/paints/patternRegions-positioned-objects-expected.png: Added.
  • platform/mac/svg/batik/paints/patternRegions-positioned-objects-expected.txt: Added.
  • platform/mac/svg/batik/text/textEffect3-expected.checksum:
  • platform/mac/svg/batik/text/textEffect3-expected.png:
  • platform/mac/svg/custom/absolute-sized-content-with-resources-expected.checksum:
  • platform/mac/svg/custom/absolute-sized-content-with-resources-expected.png:
  • platform/mac/svg/custom/grayscale-gradient-mask-expected.checksum:
  • platform/mac/svg/custom/grayscale-gradient-mask-expected.png:
  • platform/mac/svg/custom/js-late-gradient-and-object-creation-expected.checksum:
  • platform/mac/svg/custom/js-late-gradient-and-object-creation-expected.png:
  • platform/mac/svg/custom/js-late-gradient-creation-expected.checksum:
  • platform/mac/svg/custom/js-late-gradient-creation-expected.png:
  • platform/mac/svg/custom/js-late-pattern-and-object-creation-expected.checksum:
  • platform/mac/svg/custom/js-late-pattern-and-object-creation-expected.png:
  • platform/mac/svg/custom/js-late-pattern-creation-expected.checksum:
  • platform/mac/svg/custom/js-late-pattern-creation-expected.png:
  • platform/mac/svg/custom/non-scaling-stroke-expected.checksum:
  • platform/mac/svg/custom/non-scaling-stroke-expected.png:
  • platform/mac/svg/custom/pattern-incorrect-tiling-expected.checksum: Added.
  • platform/mac/svg/custom/pattern-incorrect-tiling-expected.png: Added.
  • platform/mac/svg/custom/pattern-incorrect-tiling-expected.txt: Added.
  • platform/mac/svg/custom/pattern-no-pixelation-expected.checksum: Added.
  • platform/mac/svg/custom/pattern-no-pixelation-expected.png: Added.
  • platform/mac/svg/custom/pattern-no-pixelation-expected.txt: Added.
  • platform/mac/svg/custom/pattern-rotate-expected.checksum:
  • platform/mac/svg/custom/pattern-rotate-expected.png:
  • platform/mac/svg/custom/stroked-pattern-expected.checksum:
  • platform/mac/svg/custom/stroked-pattern-expected.png:
  • platform/mac/svg/custom/text-rotated-gradient-expected.checksum:
  • platform/mac/svg/custom/text-rotated-gradient-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-height-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-height-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-maskContentUnits-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-maskContentUnits-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-width-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-width-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-x-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-x-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-y-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-dom-y-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-height-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-height-prop-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-maskContentUnits-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-maskContentUnits-prop-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-width-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-width-prop-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-x-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-x-prop-expected.png:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-y-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGMaskElement-svgdom-y-prop-expected.png:
  • platform/mac/svg/dynamic-updates/SVGPatternElement-dom-patternContentUnits-attr-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGPatternElement-dom-patternContentUnits-attr-expected.png:
  • platform/mac/svg/dynamic-updates/SVGPatternElement-svgdom-patternContentUnits-prop-expected.checksum:
  • platform/mac/svg/dynamic-updates/SVGPatternElement-svgdom-patternContentUnits-prop-expected.png:
  • platform/mac/svg/text/selection-background-color-expected.checksum:
  • platform/mac/svg/text/selection-background-color-expected.png:
  • platform/mac/svg/transforms/text-with-mask-with-svg-transform-expected.checksum: Added.
  • platform/mac/svg/transforms/text-with-mask-with-svg-transform-expected.png: Added.
  • platform/mac/svg/transforms/text-with-mask-with-svg-transform-expected.txt: Added.
  • platform/mac/svg/transforms/text-with-pattern-inside-transformed-html-expected.checksum:
  • platform/mac/svg/transforms/text-with-pattern-inside-transformed-html-expected.png:
  • platform/mac/svg/transforms/text-with-pattern-with-svg-transform-expected.checksum:
  • platform/mac/svg/transforms/text-with-pattern-with-svg-transform-expected.png:
  • platform/mac/svg/zoom/page/zoom-mask-with-percentages-expected.checksum:
  • platform/mac/svg/zoom/page/zoom-mask-with-percentages-expected.png:
  • svg/batik/paints/patternRegions-positioned-objects.svg: Added.
  • svg/custom/pattern-incorrect-tiling.svg: Added.
  • svg/custom/pattern-no-pixelation.svg: Added.
  • svg/transforms/text-with-mask-with-svg-transform.svg: Added.
  • svg/transforms/text-with-pattern-inside-transformed-html.xhtml:
05:35 Changeset [65664] by ossy@webkit.org
  • 1 edit
    32 adds in trunk/LayoutTests

Unreviewed.

[Qt] Add platform specific expected results for new passing tests.
Compared to Mac png files and Mac expected files.

  • platform/qt/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
  • platform/qt/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
  • platform/qt/fast/canvas/setWidthResetAfterForcedRender-expected.txt: Added.
  • platform/qt/fast/table/early-table-layout-expected.checksum: Added.
  • platform/qt/fast/table/early-table-layout-expected.png: Added.
  • platform/qt/fast/table/early-table-layout-expected.txt: Added.
  • platform/qt/http/tests/misc/slow-loading-image-in-pattern-expected.checksum: Added.
  • platform/qt/http/tests/misc/slow-loading-image-in-pattern-expected.png: Added.
  • platform/qt/http/tests/misc/slow-loading-image-in-pattern-expected.txt: Added.
  • platform/qt/svg/W3C-SVG-1.1/filters-turb-02-f-expected.checksum: Added.
  • platform/qt/svg/W3C-SVG-1.1/filters-turb-02-f-expected.png: Added.
  • platform/qt/svg/W3C-SVG-1.1/filters-turb-02-f-expected.txt: Added.
  • platform/qt/svg/custom/absolute-sized-content-with-resources-expected.checksum: Added.
  • platform/qt/svg/custom/absolute-sized-content-with-resources-expected.png: Added.
  • platform/qt/svg/custom/absolute-sized-content-with-resources-expected.txt: Added.
  • platform/qt/svg/custom/js-update-stop-linked-gradient-expected.checksum: Added.
  • platform/qt/svg/custom/js-update-stop-linked-gradient-expected.png: Added.
  • platform/qt/svg/custom/js-update-stop-linked-gradient-expected.txt: Added.
  • platform/qt/svg/in-html: Added.
  • platform/qt/svg/in-html/circle-expected.checksum: Added.
  • platform/qt/svg/in-html/circle-expected.png: Added.
  • platform/qt/svg/in-html/circle-expected.txt: Added.
  • platform/qt/svg/zoom/page/zoom-mask-with-percentages-expected.checksum: Added.
  • platform/qt/svg/zoom/page/zoom-mask-with-percentages-expected.png: Added.
  • platform/qt/svg/zoom/page/zoom-mask-with-percentages-expected.txt: Added.
  • platform/qt/tables/layering: Added.
  • platform/qt/tables/layering/paint-test-layering-1-expected.checksum: Added.
  • platform/qt/tables/layering/paint-test-layering-1-expected.png: Added.
  • platform/qt/tables/layering/paint-test-layering-1-expected.txt: Added.
  • platform/qt/tables/layering/paint-test-layering-2-expected.checksum: Added.
  • platform/qt/tables/layering/paint-test-layering-2-expected.png: Added.
  • platform/qt/tables/layering/paint-test-layering-2-expected.txt: Added.
05:28 Changeset [65663] by pfeldman@chromium.org
  • 4 edits in trunk/WebKit/chromium

2010-08-19 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Chromium DevTools: remove setRuntimeFeatureEnabled API method that
is no longer used.
https://bugs.webkit.org/show_bug.cgi?id=44237

  • public/WebDevToolsAgent.h:
  • src/WebDevToolsAgentImpl.cpp:
  • src/WebDevToolsAgentImpl.h:
05:19 Changeset [65662] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-19 Ademar de Souza Reis Jr <ademar.reis@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Remove outdated reference to http/tests/history/redirect-*.pl tests

[Qt] DRT keepWebHistory is not being called in redirect-30*.pl tests
https://bugs.webkit.org/show_bug.cgi?id=34212

Those tests have been renamed on
http://trac.webkit.org/changeset/65340

And are passing on the bot since then.

  • platform/qt/Skipped:
04:00 Changeset [65661] by alex@webkit.org
  • 7 edits in trunk/WebCore

2010-08-19 Alejandro G. Castro <alex@igalia.com>

Reviewed by Dirk Schulze.

[GTK] The size of the shadow image uses the standard deviation
size instead of the blur radius
https://bugs.webkit.org/show_bug.cgi?id=40793

The kernelSize variable was renamed to radius and recalculated
considering the CSS3 specification
http://www.w3.org/TR/css3-background/#the-box-shadow, and the
visual result of other browsers. The HTML5 canvas shadow standard
deviation calculation that was used, was not appropiate for the
blur distance specified in the CSS3.

  • platform/graphics/GraphicsContext.h:
  • platform/graphics/cairo/FontCairo.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::calculateShadowBufferDimensions): Changed the calculation, now we use the parameter in the style directly as recomended in the CSS3 standard. (WebCore::drawPathShadow): (WebCore::drawBorderlessRectShadow): (WebCore::GraphicsContext::createPlatformShadow): We get the standard deviation from the radius using the new function and we create the filter with that deviation.
  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::draw):
  • platform/graphics/filters/FEGaussianBlur.cpp: (WebCore::FEGaussianBlur::calculateStdDeviation): Added this function that gets the standard deviation from the blur radius. Required in the CSS3 case where we have this radio and we need the deviation to initialize the algorithm.
  • platform/graphics/filters/FEGaussianBlur.h:
01:21 Changeset [65660] by philn@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-19 Philippe Normand <pnormand@igalia.com>

Unreviewed, skip crashing test.

[GStreamer] multiple <video> elements + PulseAudio are crash-prone
https://bugs.webkit.org/show_bug.cgi?id=44235

  • platform/gtk/Skipped: Skip media/video-layer-crash.html
01:16 Changeset [65659] by andreas.kling@nokia.com
  • 2 edits in trunk/WebCore

2010-08-19 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Avoid unnecessary string copy in TextCodecQt::decode()
https://bugs.webkit.org/show_bug.cgi?id=44232

  • platform/text/qt/TextCodecQt.cpp: (WebCore::TextCodecQt::decode): Use String::append(const UChar*, int) to avoid creating a temporary string.
01:04 Changeset [65658] by philn@webkit.org
  • 24 edits
    4 adds in trunk

Reviewed by Gustavo Noronha Silva.

[GStreamer] GTK XOverlay support in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=39474

  • configure.ac: gst-interfaces link support needed for use of GstXOverlay interface.
  • GNUmakefile.am: Added new FullscreenVideoController files in the build.

LayoutTests:

Reviewed by Gustavo Noronha Silva.

[GStreamer] GTK XOverlay support in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=39474

Rebaselined tests showing the video controls containing the new
fullscreen button.

  • platform/gtk/media/controls-after-reload-expected.txt:
  • platform/gtk/media/controls-drag-timebar-expected.txt:
  • platform/gtk/media/controls-strict-expected.txt:
  • platform/gtk/media/controls-styling-expected.txt:
  • platform/gtk/media/video-controls-rendering-expected.txt:

WebCore:

Reviewed by Gustavo Noronha Silva.

[GStreamer] GTK XOverlay support in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=39474

GStreamerGWorld now catches synchronous messages coming from the
sinks and creates a GTK window where the video can be overlayed if
fullscreen display is requested using enterFullscreen.

  • GNUmakefile.am:
  • platform/graphics/gstreamer/GStreamerGWorld.cpp: (WebCore::gstGWorldSyncMessageCallback): (WebCore::GStreamerGWorld::GStreamerGWorld): (WebCore::GStreamerGWorld::~GStreamerGWorld): (WebCore::GStreamerGWorld::enterFullscreen): (WebCore::GStreamerGWorld::exitFullscreen): (WebCore::GStreamerGWorld::setWindowOverlay):
  • platform/graphics/gstreamer/GStreamerGWorld.h: (WebCore::GStreamerGWorld::pipeline): (WebCore::GStreamerGWorld::platformVideoWindow):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
  • platform/graphics/gstreamer/PlatformVideoWindow.h: Added. (WebCore::PlatformVideoWindow::createWindow): (WebCore::PlatformVideoWindow::window): (WebCore::PlatformVideoWindow::videoWindowId):
  • platform/graphics/gstreamer/PlatformVideoWindowGtk.cpp: Added. (PlatformVideoWindow::PlatformVideoWindow):

WebKt/gtk:

Reviewed by Gustavo Noronha Silva.

[GStreamer] GTK XOverlay support in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=39474

New FullscreenVideoController object, private in the webview. It
is created when the user presses the fullscreen button of the
video controls. Video is displayed in fullscreen, controls are
displayed when the user moves the mouse and when the video is
paused. There's also basic keyboard shortcuts support: F/f to
leave fullscreen, space to toggle play/pause and up/down to
control volume.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::supportsFullscreenForNode): (WebKit::ChromeClient::enterFullscreenForNode): (WebKit::ChromeClient::exitFullscreenForNode):
  • WebCoreSupport/ChromeClientGtk.h:
  • WebCoreSupport/FullscreenVideoController.cpp: Added. (hideHudCallback): (onFullscreenGtkMotionNotifyEvent): (onFullscreenGtkActiveNotification): (onFullscreenGtkConfigureEvent): (onFullscreenGtkDestroy): (togglePlayPauseActivated): (exitFullscreenActivated): (progressBarUpdateCallback): (timeScaleButtonPressed): (timeScaleButtonReleased): (timeScaleValueChanged): (volumeValueChanged): (playerVolumeChangedCallback): (playerMuteChangedCallback): (FullscreenVideoController::FullscreenVideoController): (FullscreenVideoController::~FullscreenVideoController): (FullscreenVideoController::setMediaElement): (FullscreenVideoController::gtkConfigure): (FullscreenVideoController::showHud): (FullscreenVideoController::hideHud): (onFullscreenGtkKeyPressEvent): (FullscreenVideoController::enterFullscreen): (FullscreenVideoController::updateHudPosition): (FullscreenVideoController::exitOnUserRequest): (FullscreenVideoController::exitFullscreen): (FullscreenVideoController::canPlay): (FullscreenVideoController::play): (FullscreenVideoController::pause): (FullscreenVideoController::playStateChanged): (FullscreenVideoController::togglePlay): (FullscreenVideoController::volume): (FullscreenVideoController::muted): (FullscreenVideoController::setVolume): (FullscreenVideoController::volumeChanged): (FullscreenVideoController::muteChanged): (FullscreenVideoController::currentTime): (FullscreenVideoController::setCurrentTime): (FullscreenVideoController::duration): (FullscreenVideoController::percentLoaded): (FullscreenVideoController::beginSeek): (FullscreenVideoController::doSeek): (FullscreenVideoController::endSeek): (timeToString): (FullscreenVideoController::updateHudProgressBar): (FullscreenVideoController::createHud):
  • WebCoreSupport/FullscreenVideoController.h: Added. (FullscreenVideoController::mediaElement):
  • webkit/webkitprivate.cpp: (webkit_web_view_enter_fullscreen): (webkit_web_view_exit_fullscreen):
  • webkit/webkitprivate.h:

WebKit/gtk/po:

Reviewed by Gustavo Noronha Silva.

[GStreamer] GTK XOverlay support in GStreamerGWorld
https://bugs.webkit.org/show_bug.cgi?id=39474

  • POTFILES: Added new file that contains strings to extract.
00:45 Changeset [65657] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2010-08-19 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

REGRESSION(r64303): dumpAsMarkup doesn't print the output without layoutTestController
https://bugs.webkit.org/show_bug.cgi?id=44225

Have notifyDone append Markup._container to document.body even if window.layoutTestController is undefined.

  • resources/dump-as-markup.js: (Markup.notifyDone):
00:18 Changeset [65656] by commit-queue@webkit.org
  • 13 edits
    3 adds in trunk/WebCore

2010-08-19 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Add idl and mock classes for FileWriter.
https://bugs.webkit.org/show_bug.cgi?id=44075

No new tests, since there's no new functionality.

New do-nothing classes, but the real IDL:

  • FileWriter.idl:
  • FileWriter.h:
  • FileWriter.cpp:

The build file changes to include the above:

  • CMakeLists.txt:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:

Added FileWriter event names and required EventTarget changes.

  • dom/EventNames.h:
  • dom/EventTarget.cpp: (WebCore::EventTarget::toFileWriter):
  • dom/EventTarget.h:
Note: See TracTimeline for information about the timeline view.