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

Timeline



Sep 15, 2013:

11:59 PM Changeset in webkit [155837] by akling@apple.com
  • 5 edits in trunk/Source/WebCore

CTTE: RenderSVGInlineText always has a Text node.
<​https://webkit.org/b/121409>

Reviewed by Antti Koivisto.

This renderer is never anonymous and always has a corresponding Text node.
Replaced node() with a textNode() reference getter.

Only one null check was harmed in the making of this patch.

11:59 PM Changeset in webkit [155836] by Carlos Garcia Campos
  • 2 edits in releases/WebKitGTK/webkit-2.2

Merge r155517 - autogen.sh: fix removal of autom4te.cache
​https://bugs.webkit.org/show_bug.cgi?id=121150

Reviewed by Carlos Garcia Campos.

The removal of autom4te.cache is wrong, it uses 'rm -f' instead of
'rm -rf' and it relies on an undefined variable.

In addition to that, it should be done after running autoreconf,
which is when it's no longer needed.

  • autogen.sh:
11:58 PM WebKitGTK/2.2.x edited by Carlos Garcia Campos
(diff)
11:58 PM Changeset in webkit [155835] by Carlos Garcia Campos
  • 2 edits in releases/WebKitGTK/webkit-2.2

Merge r155515 - Unquoted $ORIGDIR in autogen.sh
​https://bugs.webkit.org/show_bug.cgi?id=19512

Reviewed by Carlos Garcia Campos.

Quote all directory names. This doesn't mean that all possible
directory names are safe for building webkit, but the configure
script already runs a sanity check.

  • autogen.sh:
11:57 PM WebKitGTK/2.2.x edited by Carlos Garcia Campos
(diff)
10:53 PM Changeset in webkit [155834] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Try to fix the Windows build.

  • dom/make_names.pl: The static_cast here was running into const-correctness issues, so for now change it back to a C-style cast.
10:40 PM Changeset in webkit [155833] by commit-queue@webkit.org
  • 29 edits in trunk/Source/WebCore

Adding "explicit" keyword in rendering related constructors
​https://bugs.webkit.org/show_bug.cgi?id=121398

Patch by Santosh Mahto <​santosh.ma@samsung.com> on 2013-09-15
Reviewed by Andreas Kling.

Adding explicit in below constructor.
All files belong to Source/WebCore/rendering/ path.

  • rendering/AutoTableLayout.h:
  • rendering/ClipPathOperation.h:

(WebCore::ClipPathOperation::ClipPathOperation):
(WebCore::ShapeClipPathOperation::ShapeClipPathOperation):

  • rendering/FixedTableLayout.h:
  • rendering/FloatingObjects.h:

(WebCore::FloatingObject::FloatingObject):

  • rendering/FlowThreadController.h:
  • rendering/HitTestResult.h:
  • rendering/ImageQualityController.h:
  • rendering/InlineBox.h:

(WebCore::InlineBox::InlineBox):
(WebCore::InlineBox::InlineBoxBitfields::InlineBoxBitfields):

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::InlineFlowBox):

  • rendering/InlineTextBox.h:

(WebCore::InlineTextBox::InlineTextBox):

  • rendering/LayoutState.h:
  • rendering/LogicalSelectionOffsetCaches.h:

(WebCore::LogicalSelectionOffsetCaches::LogicalSelectionOffsetCaches):

  • rendering/RenderBoxModelObject.h:
  • rendering/RenderFlexibleBox.h:
  • rendering/RenderFrameSet.h:

(WebCore::FrameEdgeInfo::FrameEdgeInfo):

  • rendering/RenderGeometryMap.h:
  • rendering/RenderImage.h:
  • rendering/RenderImageResourceStyleImage.h:
  • rendering/RenderLayerFilterInfo.h:
  • rendering/RenderMultiColumnSet.h:
  • rendering/RenderNamedFlowThread.h:
  • rendering/RenderReplaced.h:
  • rendering/RenderSearchField.h:
  • rendering/RenderSelectionInfo.h:

(WebCore::RenderSelectionInfoBase::RenderSelectionInfoBase):
(WebCore::RenderBlockSelectionInfo::RenderBlockSelectionInfo):

  • rendering/RenderTableSection.h:
  • rendering/RenderTextControl.h:
  • rendering/RenderTextControlSingleLine.h:
  • rendering/TrailingFloatsRootInlineBox.h:
6:29 PM Changeset in webkit [155832] by kov@webkit.org
  • 10 edits in trunk

Unreviewed make distcheck fix.

Source/JavaScriptCore:

  • GNUmakefile.list.am:

Source/WebCore:

  • GNUmakefile.am:
  • GNUmakefile.list.am:

Source/WebKit2:

  • GNUmakefile.list.am:

Source/WTF:

  • GNUmakefile.list.am:
6:19 PM Changeset in webkit [155831] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore/platform/gtk/po

[l10n] Updated Polish translation of WebKitGTK+
​https://bugs.webkit.org/show_bug.cgi?id=121392

Patch by Piotr Drąg <​piotrdrag@gmail.com> on 2013-09-15
Reviewed by Gustavo Noronha Silva.

  • pl.po: updated.
6:01 PM Changeset in webkit [155830] by Patrick Gansterer
  • 5 edits in trunk/Source/WebKit/win

Remove dependency on CoreFoundation from WebIconDatabase
​https://bugs.webkit.org/show_bug.cgi?id=119387

Reviewed by Brent Fulgham.

  • COMVariantSetter.h:

(COMVariant::operator&):
(COMVariant::operator->):

  • WebIconDatabase.cpp:

(WebIconDatabase::iconDatabaseNotificationUserInfoURLKey):
(postDidAddIconNotification):

  • WebIconDatabase.h:
  • WebView.cpp:

(WebView::notifyDidAddIcon):

5:45 PM Changeset in webkit [155829] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

Fix build with Visual Studio 2012 after r153764.
​https://bugs.webkit.org/show_bug.cgi?id=121406

Reviewed by Brent Fulgham.

  • wtf/MathExtras.h:
5:02 PM Changeset in webkit [155828] by Patrick Gansterer
  • 2 edits in trunk/Source/WebCore

[WIN] Fix build without precompiled header after r154146.

  • platform/win/WebCoreBundleWin.h:
2:07 PM Changeset in webkit [155827] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

CTTE: EllipsisBox owner renderer is always a RenderBlock.
<​https://webkit.org/b/121402>

Reviewed by Antti Koivisto.

Let the constructor reflect this. Also out-of-line it because reasons.

1:51 PM Changeset in webkit [155826] by berto@igalia.com
  • 2 edits in trunk/Source/WTF

Unreviewed GTK gardening.

  • GNUmakefile.list.am: remove FastAllocBase.h, this file disappeared

in r155251.

12:56 PM Changeset in webkit [155825] by weinig@apple.com
  • 44 edits in trunk/Source/WebCore

CTTE: RenderObject's createAnonymous() and setDocumentForAnonymous() should take Document references
​https://bugs.webkit.org/show_bug.cgi?id=121400

Reviewed by Andreas Kling.

The Document used for anonymous renderers is never null.

  • dom/Document.cpp:
  • page/FrameView.cpp:
  • rendering/FlowThreadController.cpp:
  • rendering/RenderBlock.cpp:
  • rendering/RenderBlock.h:
  • rendering/RenderFlexibleBox.cpp:
  • rendering/RenderFlexibleBox.h:
  • rendering/RenderFullScreen.cpp:
  • rendering/RenderFullScreen.h:
  • rendering/RenderImage.cpp:
  • rendering/RenderInline.cpp:
  • rendering/RenderInline.h:
  • rendering/RenderLayer.cpp:
  • rendering/RenderListMarker.cpp:
  • rendering/RenderMultiColumnBlock.cpp:
  • rendering/RenderMultiColumnFlowThread.cpp:
  • rendering/RenderMultiColumnFlowThread.h:
  • rendering/RenderMultiColumnSet.cpp:
  • rendering/RenderMultiColumnSet.h:
  • rendering/RenderNamedFlowThread.cpp:
  • rendering/RenderNamedFlowThread.h:
  • rendering/RenderObject.h:
  • rendering/RenderReplica.cpp:
  • rendering/RenderReplica.h:
  • rendering/RenderRuby.cpp:
  • rendering/RenderRubyBase.cpp:
  • rendering/RenderRubyBase.h:
  • rendering/RenderRubyRun.cpp:
  • rendering/RenderScrollbar.cpp:
  • rendering/RenderScrollbarPart.cpp:
  • rendering/RenderScrollbarPart.h:
  • rendering/RenderTable.cpp:
  • rendering/RenderTableCell.cpp:
  • rendering/RenderTableCell.h:
  • rendering/RenderTableRow.cpp:
  • rendering/RenderTableRow.h:
  • rendering/RenderTableSection.cpp:
  • rendering/RenderText.cpp:
  • rendering/RenderView.cpp:
  • rendering/RenderView.h:
  • rendering/mathml/RenderMathMLBlock.cpp:
  • rendering/mathml/RenderMathMLRow.cpp:
  • rendering/mathml/RenderMathMLScripts.cpp:
12:48 PM Changeset in webkit [155824] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

Make the GDB webkit.py script compatible with Python v3
​https://bugs.webkit.org/show_bug.cgi?id=121395

Reviewed by Ryosuke Niwa.

GDB fails to load the printing helpers from the webkit.py script if Python v3 is used. To avoid that,
uses of xrange() in the script are replaced with range(), and the print statements are changed into
the print() function invocations.

  • gdb/webkit.py:

(guess_string_length):
(ustring_to_string):
(lstring_to_string):
(WTFCStringPrinter.to_string):
(PrintPathToRootCommand.invoke):

12:19 PM Changeset in webkit [155823] by weinig@apple.com
  • 6 edits in trunk/Source/WebCore

CTTE: ContentData::createRenderer() should take references
​https://bugs.webkit.org/show_bug.cgi?id=121399

Reviewed by Andreas Kling.

  • Convert createRenderer(Document*, RenderStyle*) to createRenderer(Document&, RenderStyle&)
  • Do a drive by final-ing of ContentData derived classes.
  • dom/PseudoElement.cpp:

(WebCore::PseudoElement::didAttachRenderers):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::createAnonymous):

  • rendering/RenderImage.h:
  • rendering/style/ContentData.cpp:

(WebCore::ImageContentData::createRenderer):
(WebCore::TextContentData::createRenderer):
(WebCore::CounterContentData::createRenderer):
(WebCore::QuoteContentData::createRenderer):

  • rendering/style/ContentData.h:
12:10 PM Changeset in webkit [155822] by Alan Bujtas
  • 2 edits in trunk/LayoutTests

Attempt to fix a local flakiness on Lion.

Rubber stamped by Andreas Kling.

  • fast/history/history-back-while-pdf-in-pagecache.html:
12:00 PM Changeset in webkit [155821] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

CTTE: RenderFrameSet is never anonymous.
<​https://webkit.org/b/121396>

Reviewed by Antti Koivisto.

This renderer is never anonymous, and always has a corresponding HTMLFrameSetElement.
Replaced element() with a frameSetElement().

11:53 AM Changeset in webkit [155820] by fpizlo@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

Deoptimize deoptimization: make DFGOSRExitCompiler64.cpp more hackable
​https://bugs.webkit.org/show_bug.cgi?id=121374

Reviewed by Geoffrey Garen.

This reduces the size of DFGOSRExitCompiler64.cpp by almost 50%, and makes it
super easy to add new recovery kinds. For recoveries that involve reboxing, it
allows you to keep most of the code common between the on-stack and in-reg
cases: they all get funneled through the "load from scratch buffer, convert,
and then store to stack" logic.

This opens up a bunch of possibilities. It'll make adding Int48 much easier,
and it probably will come in handy as we do various DFG stack layout changes in
support of the FTL.

  • bytecode/ValueRecovery.h:

(JSC::ValueRecovery::dumpInContext):
(JSC::ValueRecovery::dump):

  • dfg/DFGOSRExitCompiler.cpp:

(JSC::DFG::shortOperandsDump):

  • dfg/DFGOSRExitCompiler64.cpp:

(JSC::DFG::OSRExitCompiler::compileExit):

11:47 AM Changeset in webkit [155819] by weinig@apple.com
  • 9 edits in trunk/Source/WebCore

CTTE: Node subclasses should take a Document by reference in their constructor (Part 10)
​https://bugs.webkit.org/show_bug.cgi?id=121397

Reviewed by Andreas Kling.

Convert the bindings code to pass a Document reference for named constructors.

  • bindings/js/JSImageConstructor.cpp:

(WebCore::constructImage):

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateConstructorDefinition):

  • html/HTMLAudioElement.cpp:

(WebCore::HTMLAudioElement::createForJSConstructor):

  • html/HTMLAudioElement.h:
  • html/HTMLImageElement.cpp:

(WebCore::HTMLImageElement::createForJSConstructor):

  • html/HTMLImageElement.h:
  • html/HTMLOptionElement.cpp:

(WebCore::HTMLOptionElement::createForJSConstructor):

  • html/HTMLOptionElement.h:
11:25 AM Changeset in webkit [155818] by weinig@apple.com
  • 5 edits in trunk/Source/WebCore

CTTE: Node subclasses should take a Document by reference in their constructor (Part 9)
​https://bugs.webkit.org/show_bug.cgi?id=121394

Reviewed by Andreas Kling.

Remove the 'constructorTakesDocumentReference' property. It is the default now.

  • dom/make_names.pl:

Always use a Document reference.

  • html/HTMLTagNames.in:
  • mathml/mathtags.in:
  • svg/svgtags.in:

Remove the 'constructorTakesDocumentReference' property.

11:12 AM Changeset in webkit [155817] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

RenderWidget doesn't need to cache a FrameView pointer.
<​https://webkit.org/b/121158>

Reviewed by Antti Koivisto.

The FrameView can be found through RenderObject::view().frameView().
This way also gets you a reference, so no need for null-checking.

11:08 AM Changeset in webkit [155816] by Carlos Garcia Campos
  • 2 edits in releases/WebKitGTK/webkit-2.2/Source/WebKit2

Merge r155814 - [GTK] The web inspector is not rendered when docked in WebKit2 when AC is used
​https://bugs.webkit.org/show_bug.cgi?id=121381

Reviewed by Martin Robinson.

When accelerated compositing results are rendered we return early
from webkitWebViewBaseDraw and other WebView children are not
drawn. We need to chain up the draw signal handler, so that the
GtkContainer propagates the draw to all its children.

  • UIProcess/API/gtk/WebKitWebViewBase.cpp:

(webkitWebViewBaseDraw):

11:06 AM Changeset in webkit [155815] by weinig@apple.com
  • 185 edits in trunk/Source/WebCore

CTTE: Node subclasses should take a Document by reference in their constructor (Part 8)
​https://bugs.webkit.org/show_bug.cgi?id=121393

Reviewed by Andreas Kling.

Converts the following to take a Document reference:

  • All of SVG
10:41 AM Changeset in webkit [155814] by Carlos Garcia Campos
  • 2 edits in trunk/Source/WebKit2

[GTK] The web inspector is not rendered when docked in WebKit2 when AC is used
​https://bugs.webkit.org/show_bug.cgi?id=121381

Reviewed by Martin Robinson.

When accelerated compositing results are rendered we return early
from webkitWebViewBaseDraw and other WebView children are not
drawn. We need to chain up the draw signal handler, so that the
GtkContainer propagates the draw to all its children.

  • UIProcess/API/gtk/WebKitWebViewBase.cpp:

(webkitWebViewBaseDraw):

10:10 AM Changeset in webkit [155813] by weinig@apple.com
  • 16 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (Part 7)
​https://bugs.webkit.org/show_bug.cgi?id=121389

Reviewed by Andreas Kling.

Converts the following to take a Document reference:

  • Element
  • MathMLElement
  • MathMLInlineContainerElement
  • MathMLMathElement
  • MathMLTextElement
7:58 AM Changeset in webkit [155812] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

[WIN] Fix build without precompiled header after r155454.

  • wtf/win/GDIObject.h: Added missing windows.h include.
7:13 AM Changeset in webkit [155811] by akling@apple.com
  • 9 edits in trunk/Source/WebCore

CTTE: RenderMedia and RenderVideo are never anonymous.
<​https://webkit.org/b/121388>

Reviewed by Sam Weinig.

Codify the following:

  • RenderMedia always has an HTMLMediaElement.
  • RenderVideo always has an HTMLVideoElement.

None of these renderers are ever anonymous, so delete element() and provide
strongly typed reference getters instead.

7:12 AM Changeset in webkit [155810] by akling@apple.com
  • 9 edits
    5 adds in trunk/Source/WebCore

Unreviewed, rolling out r155809.
​http://trac.webkit.org/changeset/155809
​https://bugs.webkit.org/show_bug.cgi?id=121388

Accidentally removed files, let's redo this.

7:06 AM Changeset in webkit [155809] by akling@apple.com
  • 9 edits
    5 deletes in trunk/Source/WebCore

CTTE: RenderMedia and RenderVideo are never anonymous.
<​https://webkit.org/b/121388>

Reviewed by Sam Weinig.

Codify the following:

  • RenderMedia always has an HTMLMediaElement.
  • RenderVideo always has an HTMLVideoElement.

None of these renderers are ever anonymous, so delete element() and provide
strongly typed reference getters instead.

6:31 AM Changeset in webkit [155808] by weinig@apple.com
  • 47 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (Part 6)
​https://bugs.webkit.org/show_bug.cgi?id=121387

Reviewed by Andreas Kling.

Converts the following to take a Document reference:

  • Attr
  • CDATASection
  • CharacterData
  • Comment
  • DOMImplementation
  • DocumentFragment
  • DocumentType
  • EntityReference
  • Notation
  • ProcessingInstruction
  • ShadowRoot
  • TemplateContentDocumentFragment
  • Text
5:59 AM Changeset in webkit [155807] by akling@apple.com
  • 2 edits in trunk/Source/WebCore

Remove RenderObject::clearNode().
<​https://webkit.org/b/121386>

Reviewed by Anders Carlsson.

This was used by ref-counted RenderWidgets after getting removed from their
parent renderer but kept alive by an external ref. We have no need for this
awkward state anymore, so remove clearNode().

5:52 AM Changeset in webkit [155806] by akling@apple.com
  • 21 edits in trunk/Source/WebCore

CTTE: RenderWidgets are never anonymous.
<​https://webkit.org/b/121385>

Reviewed by Anders Carlsson.

Codify the following:

  • RenderWidget always has an HTMLFrameOwnerElement.
  • RenderEmbeddedObject always has an HTMLFrameOwnerElement.
  • RenderSnapshottedPlugIn always has an HTMLPlugInImageElement.
  • RenderFrame always has an HTMLFrameElement.
  • RenderIFrame always has an HTMLIFrameElement.

None of these renderers are ever anonymous, so delete element() and provide
strongly typed reference getters instead.

5:51 AM Changeset in webkit [155805] by akling@apple.com
  • 2 edits in trunk/Source/WTF

Remove Ref(const T&) constructor.
<​https://webkit.org/b/121384>

Reviewed by Anders Carlsson.

This constructor was added as a stopgap measure until we got better support
for move semantics in Vector.

  • wtf/Ref.h:
5:07 AM WebKitGTK/2.2.x edited by berto@igalia.com
(diff)
4:42 AM WebKitGTK/2.0.x edited by berto@igalia.com
(diff)
3:35 AM Changeset in webkit [155804] by akling@apple.com
  • 2 edits in trunk/Source/WTF

Unreviewed, fix ridiculous bug in WeakPtr.

I made operator!() null check the pointer to the WeakReference. :|

Caught by plugins/destroy-on-setwindow.html on bots.

  • wtf/WeakPtr.h:

(WTF::WeakPtr::operator!):

2:29 AM Changeset in webkit [155803] by zandobersek@gmail.com
  • 4 edits in trunk/Tools

[webkitpy] perf profiler attaches to WKTR after the WebProcess was launched
​https://bugs.webkit.org/show_bug.cgi?id=119631

Reviewed by Ryosuke Niwa.

The perf profiler is attached to the test runner after the test runner has been launched,
but in the case of WebKitTestRunner, this is done after the WebProcess was spawned by the
UIProcess, meaning that WebProcess operations will not be profiled.

To avoid that, the perf profiler is changed to provide a wrapper command for the test runner.
This is done in the scope of already present support for wrapper scripts (such as valgrind)
that are settable through the --wrapper option. This causes the perf profiler to correctly
continue profiling the WebProcess after it was spawned.

When the profiler is launched, a wait process is launched with the profiler's pid as the
argument. This ensures we can check that the profiler exited normally. If it did not, its
output can be corrupted and can cause problems when that output is examined. The after-exit
report is avoided in that case.

  • Scripts/webkitpy/common/system/profiler.py:

(Profiler.wrapper_arguments): The method returns an empty list by default.
(Perf.init): Remove unnecessary member variables.
(Perf.attach_to_pid): Launch the wait process with the profiler's pid as the argument.
(Perf.wrapper_arguments): Used instead of attaching the profiler to the profiled process' pid,
the method returns a list of arguments that should be used as a wrapper for the test runner.
The arguments specify the 'perf record' command, with the output file and the instruction to
construct the call graph given as well.
(Perf._first_ten_lines_of_profile): Prettify the output by avoiding the unnecessary trailing
whitespace in the profile lines.
(Perf.profile_after_exit): Kill the profiler process through its pid if the wait process is still
running. If it is running even after that, or if the profiler process produced a non-zero exit
code, the method reports the profiling failure and returns. Otherwise it continues on reporting
the top 10 lines of the profile and how to examine the profile further.

  • Scripts/webkitpy/port/driver.py:

(Driver._command_wrapper): The method is now a member of the Driver class, returning the wrapper
as specified by the --wrapper option if given, or the profiler wrapper if the profiler is present,
or an empty list otherwise.
(Driver.cmd_line):

  • Scripts/webkitpy/port/driver_unittest.py:

(DriverTest.make_port): Adjusted so mock host or options objects can be specified
for the testing-specific Port object.
(DriverTest._assert_wrapper): Pass a MockOptions object with the wrapper value to the make_port method,
as the Driver._command_wrapper now operates on the option's value directly rather than accepting it through
a parameter.
(DriverTest.test_profiler_as_wrapper): Provide a test case to check the correct profiler wrapper is used
when profiling on the Linux platform with the perf profiler.

1:51 AM Changeset in webkit [155802] by Darin Adler
  • 16 edits in trunk/Source/WebCore

Use FINAL instead of virtualChildren trick in render tree classes
​https://bugs.webkit.org/show_bug.cgi?id=121373

Reviewed by Andreas Kling.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::updateFirstLetterStyle): Use children instead of
virtualChildren.

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::splitAnonymousBoxesAroundChild): Ditto.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::moveChildTo): Ditto.

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::addChild): Ditto.
(WebCore::RenderObject::removeChild): Ditto.
(WebCore::RenderObject::handleDynamicFloatPositionChange): Ditto.
(WebCore::RenderObject::willBeDestroyed): Ditto.
(WebCore::RenderObject::removeFromRenderFlowThreadRecursive): Ditto.

  • rendering/RenderBlock.h: Changed firstChild and lastChild to use

m_children directly instead of a function. Made the children function
a final virtual override. Deleted the virtualChildren override. Added
override keywords to other virtual functions.

  • rendering/RenderEmbeddedObject.h: Ditto.
  • rendering/RenderFrameSet.h: Ditto.
  • rendering/RenderInline.h: Ditto.
  • rendering/RenderMedia.h: Ditto.
  • rendering/RenderTableCol.h: Ditto.
  • rendering/RenderTableRow.h: Ditto.
  • rendering/RenderTableSection.h: Ditto.
  • rendering/svg/RenderSVGContainer.h: Ditto.
  • rendering/svg/RenderSVGRoot.h: Ditto.
  • rendering/RenderObject.h:

(WebCore::RenderObject::firstChild): Use children instead of virtualChildren.
(WebCore::RenderObject::lastChild): Ditto.
(WebCore::RenderObject::children): Renamed from virtualChildren.
(WebCore::RenderObject::canHaveChildren): Use children instead of virtualChildren.

1:50 AM Changeset in webkit [155801] by Darin Adler
  • 15 edits in trunk/Source/WebCore

Shrink factory functions
​https://bugs.webkit.org/show_bug.cgi?id=121378

Reviewed by Andreas Kling.

This makes the functions that builds four different types of per-element tables
all have smaller code size.

  • DerivedSources.make: Tweaked how HTML and SVG wrapper factories are made a bit.

It was OK, but a little sloppy.

  • dom/Document.cpp:

(WebCore::Document::createElement): Changed to call the createElement functions
in the factories. Also did some style improvement.

  • dom/make_names.pl:

(printConstructorSignature): Make constructors take a Document& instead of
a Document*.
(printConstructorInterior): Updated for Document& instead of Document*.
(printFunctionTable): Renamed from printFunctionInits, because this now
creates table entries. The actual code to add each function is now done
in a loop instead.
(printNamesHeaderFile): Made the table const.
(printNamesCppFile): Made the table const.
(printDefinitions): Generate a table, table entries, and a loop that calls
createQualifiedName, rather than an "unrolled loop".
(printFactoryCppFile): Tweaked indentation. Updated to take Document& instead
of Document*. Generate a table, table entries, and a loop that builds a map,
rather than an "unrolled" loop.
(printFactoryHeaderFile): Eliminated the unused createElement member function
that was part of a dream of factory objects in a namespace registry, giving
the name createElement to the static member function so it would be less
redundant. Also made the function takes a Document& instead of Document*.
Also made the default "not created by parser", since the parser is the simpler
use case, outnumbered by the non-parser uses, at least at the moment. Also,
use nullptr instead of 0.
(printWrapperFactoryCppFile): Generate a table, table entries, and a loop
that builds a map rather than an "unrolled" loop.

  • editing/htmlediting.cpp:

(WebCore::createHTMLElement): Put the null check here that used to be in
HTMLElementFactory::createElement, and call with a reference instead of a
pointer. Also don't need to pass "0, false" since "not created by the parser"
is now the default.

  • html/HTMLDocument.cpp:

(WebCore::HTMLDocument::createElement): Ditto.

  • html/HTMLElement.cpp:

(WebCore::populateEventNameForAttributeLocalNameMap): Changed to use an
AtomicStringImpl* as the key instead of AtomicString, since all the strings
come from the local names of tags, and so don't need to be ref'd. This saves
a bit of reference count churn when building the map and is the same pattern
used in some maps in the make_names.pl script above.
(WebCore::HTMLElement::parseAttribute): Updated for change above.

  • html/HTMLObjectElement.cpp:

(WebCore::isRecognizedTagName): More const.

  • html/parser/HTMLConstructionSite.cpp:

(WebCore::HTMLConstructionSite::createHTMLElement): Changed to call the
HTMLElementFactory::createElement function under its new name with a reference
rather than a pointer.

  • html/parser/HTMLIdentifier.cpp:

(WebCore::nameForIndex): More const.
(WebCore::HTMLIdentifier::addNames): More const.

  • html/parser/HTMLTreeBuilder.cpp:

(WebCore::mapLoweredLocalNameToName): More const.
(WebCore::adjustSVGTagNameCase): Eliminated local variable so we would not have
to utter type with more const.
(WebCore::adjustAttributes): Ditto.
(WebCore::addNamesWithPrefix): More const. Also tweaked to use a bit more reference,
and a bit less pointer. Also used array instead of pointer for argument since they
are synonyms and this use is more array-like and const array is slightly easier to
understand than the double const pointer.
(WebCore::adjustForeignAttributes): Eliminated local variable so we would not have
to utter type with more const.

  • html/track/TextTrackCue.cpp:

(WebCore::TextTrackCue::copyWebVTTNodeToDOMTree): Pass a reference to
WebVTTElement::createEquivalentHTMLElement.

  • html/track/WebVTTElement.cpp:

(WebCore::WebVTTElement::WebVTTElement): Take a reference instead of a pointer.
(WebCore::WebVTTElement::create): Ditto.
(WebCore::WebVTTElement::cloneElementWithoutAttributesAndChildren): Pass a
reference instead of a pointer to create.
(WebCore::WebVTTElement::createEquivalentHTMLElement): Take a reference instead
of a pointer, and update uses of HTMLElementFactory::createElement. Also made
this return null when a bad node type is passed in, which should never happen.

  • html/track/WebVTTElement.h: Use Document& instead of Document*. Removed unused

and undefined create/constructor overloads.

  • html/track/WebVTTParser.cpp:

(WebCore::WebVTTParser::constructTreeFromToken): Pass document reference rather than
pointer to WebVTTElement::create.

1:21 AM Changeset in webkit [155800] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

Get rid of ref-counting on RenderArena.
<​https://webkit.org/b/121379>

Reviewed by Darin Adler.

This was used to guard the arena during ref-counted RenderWidget teardown.

1:18 AM Changeset in webkit [155799] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Fix Qt build.

  • css/CSSSelector.cpp: (WebCore::populatePseudoTypeByNameMap): Removed stray line that was breaking the build for any platform with !ENABLE(VIDEO_TRACK).
1:15 AM Changeset in webkit [155798] by akling@apple.com
  • 3 edits in trunk/Source/WebCore

Restore two-pass mechanism for FrameView::updateEmbeddedObjects().
<​https://webkit.org/b/121380>

Reviewed by Darin Adler.

Restore the code to make a second pass over the embedded objects needing
an update in case more objects were added to the set during the first pass.

1:08 AM Changeset in webkit [155797] by commit-queue@webkit.org
  • 15 edits
    20 adds
    2 deletes in trunk

Implement the mmultiscripts tag
​https://bugs.webkit.org/show_bug.cgi?id=99618

Patch by Frédéric Wang <​fred.wang@free.fr> on 2013-09-15
Reviewed by Chris Fleizach.

Source/WebCore:

Tests: mathml/invalid-scripts-crash.html

mathml/presentation/multiscripts-equivalence.html
mathml/presentation/multiscripts-noscripts.html
mathml/presentation/multiscripts-positions.html
mathml/presentation/scripts-base-alignment.html
mathml/presentation/scripts-horizontal-alignment.html
mathml/presentation/scripts-vertical-alignment.html
mathml/scripts-addChild.html
mathml/scripts-removeChild.html

This relies on the existing msub/msup/msubsup code to implement the mmultiscripts tag. This also improves dynamic addition/removal of children for these MathML elements and adds a specific style for invalid children, so that they render like an merror tag. Finally, this fixes a bad memory access in the Accessibility render object of msubsup.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.vcxproj/WebCore.vcxproj:
  • WebCore.vcxproj/WebCore.vcxproj.filters:
  • WebCore.xcodeproj/project.pbxproj:
  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::isMathSubscriptSuperscript):
(WebCore::AccessibilityRenderObject::mathSuperscriptObject):

  • css/mathml.css:

(mo, mrow, mfenced, mfrac, msub, msup, msubsup, mmultiscripts, mprescripts, none, munder, mover, munderover, msqrt, mroot):
(mover > :last-child, munderover > :last-child):
(msub > * + *, msup > * + *, msubsup > * + *, mmultiscripts > * + *, munder > * + *, mover > * + *, munderover > * + *):
(merror, msub > * + * + *, msup > * + * + *, msubsup > * + * + * + *, msub > mprescripts, msup > mprescripts, msubsup > mprescripts, msub > none, msup > none, msubsup > none, mmultiscripts > mprescripts ~ mprescripts, mmultiscripts > mprescripts ~ mprescripts ~ *):

  • mathml/MathMLInlineContainerElement.cpp:

(WebCore::MathMLInlineContainerElement::createRenderer):

  • mathml/mathtags.in:
  • rendering/mathml/RenderMathMLBlock.h:

(WebCore::RenderMathMLBlock::isRenderMathMLScripts):
(WebCore::RenderMathMLBlock::isRenderMathMLScriptsWrapper):

  • rendering/mathml/RenderMathMLScripts.cpp: Added.

(WebCore::isMPrescripts):
(WebCore::RenderMathMLScripts::RenderMathMLScripts):
(WebCore::RenderMathMLScripts::base):
(WebCore::RenderMathMLScripts::fixAnonymousStyleForSubSupPair):
(WebCore::RenderMathMLScripts::fixAnonymousStyles):
(WebCore::RenderMathMLScripts::addChildInternal):
(WebCore::RenderMathMLScripts::removeChildInternal):
(WebCore::RenderMathMLScripts::addChild):
(WebCore::RenderMathMLScripts::removeChild):
(WebCore::RenderMathMLScripts::styleDidChange):
(WebCore::RenderMathMLScripts::unembellishedOperator):
(WebCore::RenderMathMLScripts::layout):
(WebCore::RenderMathMLScripts::firstLineBoxBaseline):
(WebCore::RenderMathMLScriptsWrapper::createAnonymousWrapper):
(WebCore::RenderMathMLScriptsWrapper::addChildInternal):
(WebCore::RenderMathMLScriptsWrapper::addChild):
(WebCore::RenderMathMLScriptsWrapper::removeChildInternal):
(WebCore::RenderMathMLScriptsWrapper::removeChild):

  • rendering/mathml/RenderMathMLScripts.h: Added.

(WebCore::RenderMathMLScriptsWrapper::RenderMathMLScriptsWrapper):
(WebCore::RenderMathMLScriptsWrapper::renderName):
(WebCore::RenderMathMLScriptsWrapper::isRenderMathMLScriptsWrapper):
(WebCore::toRenderMathMLScriptsWrapper):
(WebCore::RenderMathMLScripts::isRenderMathMLScripts):
(WebCore::RenderMathMLScripts::renderName):
(WebCore::toRenderMathMLScripts):

  • rendering/mathml/RenderMathMLSubSup.cpp: Removed.
  • rendering/mathml/RenderMathMLSubSup.h: Removed.

LayoutTests:

Add many tests for script msub/msup/msubsup/mmultiscripts:

  • invalid markup
  • equivalence between mmultiscripts without scripts and mrow
  • equivalence between mmultiscripts and msub/msup/msubsup
  • position of scripts in mmultiscripts
  • baseline alignment of msub/msup/msubsup elements
  • horizontal/vertical alignment of scripts
  • adding/removing children with the DOM
  • TestExpectations: Some remaining MathML pixel tests are broken by this patch.
  • mathml/invalid-scripts-crash-expected.txt: Added.
  • mathml/invalid-scripts-crash.html: Added.
  • mathml/presentation/multiscripts-equivalence-expected.html: Added.
  • mathml/presentation/multiscripts-equivalence.html: Added.
  • mathml/presentation/multiscripts-noscripts-expected.html: Added.
  • mathml/presentation/multiscripts-noscripts.html: Added.
  • mathml/presentation/multiscripts-positions-expected.html: Added.
  • mathml/presentation/multiscripts-positions.html: Added.
  • mathml/presentation/scripts-base-alignment-expected.html: Added.
  • mathml/presentation/scripts-base-alignment.html: Added.
  • mathml/presentation/scripts-horizontal-alignment-expected.html: Added.
  • mathml/presentation/scripts-horizontal-alignment.html: Added.
  • mathml/presentation/scripts-vertical-alignment-expected.html: Added.
  • mathml/presentation/scripts-vertical-alignment.html: Added.
  • mathml/scripts-addChild-expected.html: Added.
  • mathml/scripts-addChild.html: Added.
  • mathml/scripts-removeChild-expected.html: Added.
  • mathml/scripts-removeChild.html: Added.
  • platform/mac/accessibility/mathml-multiscript-expected.txt: Reference updated.
12:27 AM Changeset in webkit [155796] by akling@apple.com
  • 13 edits
    1 delete in trunk/Source

Get rid of ref-counting on RenderWidget.
<​https://webkit.org/b/121357>

Reviewed by Darin Adler.

Source/WebCore:

Instead of RenderView tracking RenderWidgets, have FrameView track Widgets
that are currently in the render tree.

To protect ourselves during NPAPI tomfoolery, we now let RenderWidget hand
out weak pointers through a createWeakPtr() method so call sites can monitor
the renderer for deletion without having to take shared ownership.

This works out quite nicely since instead of keeping a limping object alive
for a little longer just so we can call methods on it (to accomplish nothing),
we're forced to check right away if it's gone, and take immediate action.

De-virtualized RenderObject::destroy() since it's no longer needed for
RenderWidget to defer destruction.

  • page/FrameView.cpp:

(WebCore::FrameView::layout):
(WebCore::FrameView::repaintFixedElementsAfterScrolling):

Call updateWidgetPositions() on FrameView instead of RenderView.

(WebCore::FrameView::updateEmbeddedObject):

Turn null checking of embedded object's element backpointer into an
assertion. This will eventually go away completely once that renderer
can return a HTMLFrameOwnerElement&.

Use WeakPtr to check for renderer destruction following the call out
to updateWidget().

(WebCore::FrameView::updateEmbeddedObjects):

Slap a WidgetHierarchyUpdatesSuspensionScope guard on this function
to defer Widget updates until all the updateEmbeddedObject calls are
done. This avoids RenderWidget::setWidget() having to handle 'this'
disappearing from underneath.

Also use a ListHashSet with a null sentinel to avoid looping forever.

(WebCore::FrameView::performPostLayoutTasks):

Only call updateEmbeddedObjects() once since that function no longer
operates in chunks.

(WebCore::FrameView::notifyWidgetsInAllFrames):

Call notifyWidgets() on FrameView instead of RenderView.

(WebCore::FrameView::didAddWidgetToRenderTree):
(WebCore::FrameView::willRemoveWidgetFromRenderTree):

Added. These are called by RenderWidget when a Widget is being
added or removed from a RenderWidget.

(WebCore::collectWidgets):

Helper to collect raw Widget pointers into a Vector and ref them.

(WebCore::FrameView::updateWidgetPositions):

Moved here from RenderView. This function holds a ref on all the
attached Widgets and calls RenderWidget::updateWidgetPosition() on
their corresponding renderers.

(WebCore::FrameView::notifyWidgets):

Moved here from RenderView. Holds a ref on all the widgets while
calling Widget::notifyWidget() on each one.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::scrollTo):

Call updateWidgetPositions() on FrameView instead of RenderView.

  • rendering/RenderObject.h:

De-virtualized destroy().

  • rendering/RenderView.cpp:
  • rendering/RenderView.h:

Moved a bunch of things to FrameView. Made protected section private
since nothing inherits from RenderView.

  • rendering/RenderWidget.h:

(WebCore::RenderWidget::createWeakPtr):

Added a WeakPtr factory for clients that want to monitor this object
for destruction.

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::RenderWidget):
(WebCore::RenderWidget::willBeDestroyed):
(WebCore::RenderWidget::~RenderWidget):

Removed ref counting.
Removed registration with RenderView in ctor/willBeDestroyed.

(WebCore::RenderWidget::setWidgetGeometry):

Monitor the RenderWidget itself through a WeakPtr and check on it
after each step that we're still alive. In that case just bail out.

(WebCore::RenderWidget::setWidget):

Register the incoming/outgoing Widget with the FrameView.
Use a WeakPtr to check on 'this' after updateWidgetGeometry().

(WebCore::RenderWidget::updateWidgetPosition):

Use a WeakPtr to check on 'this' after updateWidgetGeometry().

  • GNUmakefile.list.am:
  • rendering/RenderWidgetProtector.h:

Removed.

Source/WTF:

  • wtf/WeakPtr.h:

(WTF::WeakPtr::operator!):

Add operator! to WeakPtr.

Sep 14, 2013:

10:23 PM Changeset in webkit [155795] by weinig@apple.com
  • 81 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (Part 5)
​https://bugs.webkit.org/show_bug.cgi?id=121375

Reviewed by Andreas Kling.

  • dom/make_names.pl:

Add support for a top level 'constructorTakesDocumentReference' property to
force using a Document reference for the whole factory.

  • html/HTMLTagNames.in:

Replace per-tag 'constructorTakesDocumentReference' with a top level declaration
now that all of the HTML tag names are supported.

Converts the following to take a Document reference:

  • HTMLContentElement
  • HTMLElement
  • HTMLFrameElementBase
  • HTMLFrameOwnerElement
  • HTMLPlugInElement
  • HTMLPlugInImageElement
  • HTMLUnknownElement
  • InsertionPoint
  • StyledElement
  • TextTrackCueBox
  • TextTrackCueGenericBoxElement
7:25 PM Changeset in webkit [155794] by eric.carlson@apple.com
  • 8 edits in trunk

MediaStream API: Update RTCDataChannel
​https://bugs.webkit.org/show_bug.cgi?id=121102

Reviewed by Sam Weinig.

Source/WebCore:

Based on ​https://chromium.googlesource.com/chromium/blink/+/c6975c41956acded7cf0363012d7d6b69d0c6d96,
and ​https://chromium.googlesource.com/chromium/blink/+/214dab0bd6385f573c918ba5ce58a5aa206ce186.

No new tests, existing tests updated.

  • Modules/mediastream/RTCDataChannel.cpp:

(WebCore::RTCDataChannel::create): Take an options Dictionary.
(WebCore::RTCDataChannel::didChangeReadyState): String -> AtomicString. Early return if the

state hasn't changed.

(WebCore::RTCDataChannel::binaryType): String -> AtomicString.
(WebCore::RTCDataChannel::setBinaryType): String -> AtomicString.

  • Modules/mediastream/RTCDataChannel.h:
  • Modules/mediastream/RTCDataChannel.idl:
  • Modules/mediastream/RTCPeerConnection.cpp:

(WebCore::RTCPeerConnection::createDataChannel): Take an options struct.

  • platform/mediastream/RTCDataChannelHandler.h:
  • platform/mediastream/RTCPeerConnectionHandler.h:

(WebCore::RTCDataChannelInit::RTCDataChannelInit): Take an options struct.

LayoutTests:

  • fast/mediastream/RTCPeerConnection-datachannel-expected.txt:
  • fast/mediastream/RTCPeerConnection-datachannel.html:
5:57 PM Changeset in webkit [155793] by fpizlo@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

It should be easy to add new nodes that do OSR forward rewiring in both DFG and FTL
​https://bugs.webkit.org/show_bug.cgi?id=121371

Reviewed by Sam Weinig.

Forward rewiring is a tricky part of OSR that handles the following:

a: Something(...)

SetLocal(@a, locX)

b: Int32ToDouble(@a)
c: SomethingThatExits(@b)

<no further uses of @a or @b>

Note that at @c, OSR will think that locX->@a, but @a will be dead. So it must be
smart enough to find @b, which contains an equivalent value. It must do this for
any identity functions we support. Currently we support four such functions.

Currently the code for doing this is basically duplicated between the DFG and the
FTL. Also both versions of the code have some really weirdly written logic for
picking the "best" identity function to use.

We should fix this by simply having a way to ask "is this node an identity
function, and if so, then how good is it?" Then both the DFG and FTL could use
this and have no hard-wired knowledge of those identity functions.

While we're at it, this also changes some terminology because I found the use of
the word "needs" confusing. Note that this retains the somewhat confusing behavior
that we don't search all possible forward/backward uses. We only search one step
in each direction. This is because we only need to handle cases that FixupPhase
and the parser insert. All other code that tries to insert intermediate conversion
nodes should ensure to Phantom the original node. For example, the following
transformation is illegal:

Before:

x: SomethingThatExits(@a)


After:

w: Conversion(@a)
x: SomethingThatExits(@w)


The correct form of that transformation is one of these:

Correct #1:

v: DoAllChecks(@a) exit here
w: Conversion(@a)
x: Something(@w) no exit


Correct #2:

w: Conversion(@a)
x: SomethingThatExits(@w)
y: Phantom(@a)


Correct #3:

w: Conversion(@a)
x: SomethingThatExits(@w, @a)


Note that we use #3 for some heap accesses, but of course it requires that the
node you're using has an extra slot for a "dummy" use child.

Broadly speaking though, such transformations should be relegated to something
below DFG IR, like LLVM IR.

  • dfg/DFGNodeType.h:

(JSC::DFG::forwardRewiringSelectionScore):
(JSC::DFG::needsOSRForwardRewiring):

  • dfg/DFGVariableEventStream.cpp:

(JSC::DFG::VariableEventStream::reconstruct):

  • ftl/FTLLowerDFGToLLVM.cpp:

(JSC::FTL::LowerDFGToLLVM::addExitArgumentForNode):

5:50 PM Changeset in webkit [155792] by eric.carlson@apple.com
  • 5 edits in trunk/Source/WebCore

MediaStream API: Update RTCDataChannel to match the specification
​https://bugs.webkit.org/show_bug.cgi?id=120889

Patch by Thiago de Barros Lacerda <​thiago.lacerda@openbossa.org> on 2013-09-14
Reviewed by Eric Carlson

Merged from ​https://chromium.googlesource.com/chromium/blink/+/c3862b0a83e20fc8b1f770c7e4a886a7cceb80d2
by Tommy Widenflycht.

According to WebRTC specification, RTCDataChannel must have the following new attributes:
boolean ordered
unsigned short maxRetransmitTime
unsigned short maxRetransmits
DOMString protocol
boolean negotiated
unsigned short id

and the following one was deprecated:
boolean reliable

Test updates will be landed with ​https://webkit.org/b/121102.

  • Modules/mediastream/RTCDataChannel.cpp:

(WebCore::RTCDataChannel::ordered):
(WebCore::RTCDataChannel::maxRetransmitTime):
(WebCore::RTCDataChannel::maxRetransmits):
(WebCore::RTCDataChannel::protocol):
(WebCore::RTCDataChannel::negotiated):
(WebCore::RTCDataChannel::id):

  • Modules/mediastream/RTCDataChannel.h:
  • Modules/mediastream/RTCDataChannel.idl:
  • platform/mediastream/RTCDataChannelHandler.h:
5:39 PM Changeset in webkit [155791] by weinig@apple.com
  • 96 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (Part 4)
​https://bugs.webkit.org/show_bug.cgi?id=121372

Reviewed by Eric Carlson.

Converts the following to take a Document reference:

  • HTMLFormControlElement
  • HTMLFormControlElementWithState
  • HTMLMapElement
  • HTMLMarqueeElement
  • HTMLMediaElement
  • HTMLMenuElement
  • HTMLMetaElement
  • HTMLMeterElement
  • HTMLOListElement
  • HTMLObjectElement
  • HTMLOptGroupElement
  • HTMLOptionElement
  • HTMLOutputElement
  • HTMLParagraphElement
  • HTMLParamElement
  • HTMLProgressElement
  • HTMLScriptElement
  • HTMLSelectElement
  • HTMLSourceElement
  • HTMLSpanElement
  • HTMLStyleElement
  • HTMLSummaryElement
  • HTMLTableCellElement
  • HTMLTableElement
  • HTMLTableRowElement
  • HTMLTableSectionElement
  • HTMLTemplateElement
  • HTMLTextAreaElement
  • HTMLTextFormControlElement
  • HTMLTitleElement
  • HTMLTrackElement
  • HTMLUListElement
  • InlineStyleSheetOwner
  • LabelableElement
  • All the MediaControl elements
5:27 PM Changeset in webkit [155790] by Joseph Pecoraro
  • 7 edits in trunk/Source/WebCore

XMLViewer_js and XMLViewer_css should be minified
​https://bugs.webkit.org/show_bug.cgi?id=121334

Reviewed by Timothy Hatcher.

  • CMakeLists.txt:
  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:
  • xml/XMLViewer.css:
  • xml/XMLViewer.js:
5:16 PM Changeset in webkit [155789] by eric.carlson@apple.com
  • 3 edits in trunk/Source/WebCore

[MediaStream] remove MediaStream.label
​https://bugs.webkit.org/show_bug.cgi?id=121337

Reviewed by Sam Weinig.

No new tests or modified tests, this attribute was apparently never tested.

  • Modules/mediastream/MediaStream.h:
  • Modules/mediastream/MediaStream.idl:
2:35 PM Changeset in webkit [155788] by mitz@apple.com
  • 2 edits in trunk/Tools

[mac] Check For Framework Include Consistency build phase checks the wrong framework
​https://bugs.webkit.org/show_bug.cgi?id=121215

Reviewed by Mark Rowe.

  • Scripts/check-for-webkit-framework-include-consistency: Replaced hard-coded paths to

WebKit.framework’s Headers and PrivateHeaders directories with the current target’s
public and private headers folder paths. Also tweaked the error messages’ format to make
Xcode recognize them as errors.

2:33 PM Changeset in webkit [155787] by mitz@apple.com
  • 3 edits in trunk/Source/WebKit2

Remove non-Apple cruft from the installed WKBase.h
​https://bugs.webkit.org/show_bug.cgi?id=121370

Reviewed by Darin Adler.

  • Configurations/WebKit2.xcconfig: Set INSTALLHDRS_SCRIPT_PHASE to YES.
  • WebKit2.xcodeproj/project.pbxproj: Added a script build phase that runs unifdef on

the installed WKBase.h with APPLE defined and various BUILDING_* and WTF_USE_* symbols
undefined.

2:33 PM Changeset in webkit [155786] by weinig@apple.com
  • 99 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (Part 3)
​https://bugs.webkit.org/show_bug.cgi?id=121368

Reviewed by Darin Adler.

Converts the following to take a Document reference:

  • DeleteButton
  • DetailsContentElement
  • DetailsMarkerControl
  • DetailsSummaryElement
  • HTMLBDIElement
  • HTMLBRElement
  • HTMLBaseElement
  • HTMLBaseFontElement
  • HTMLButtonElement
  • HTMLCanvasElement
  • HTMLDListElement
  • HTMLDataListElement
  • HTMLDetailsElement
  • HTMLDirectoryElement
  • HTMLDivElement
  • HTMLEmbedElement
  • HTMLFieldSetElement
  • HTMLFontElement
  • HTMLFormElement
  • HTMLFrameElement
  • HTMLFrameSetElement
  • HTMLHRElement
  • HTMLHeadElement
  • HTMLHeadingElement
  • HTMLHtmlElement
  • HTMLIFrameElement
  • HTMLImageElement
  • HTMLInputElement
  • HTMLKeygenElement
  • HTMLLIElement
  • HTMLLabelElement
  • HTMLLegendElement
  • HTMLLinkElement
  • HTMLModElement
  • HTMLPreElement
  • HTMLQuoteElement
  • HTMLTableCaptionElement
  • HTMLTableColElement
  • ImageDocumentElement
  • KeygenSelectElement
  • SummaryContentElement
  • UploadButtonElement
2:11 PM Changeset in webkit [155785] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

REGRESSION(r155228): Call to DragData::asFragment() nullifies PassRefPtr<Range> in documentFragmentFromDragData
​https://bugs.webkit.org/show_bug.cgi?id=121359

Reviewed by Darin Adler.

  • page/DragController.cpp:

(WebCore::documentFragmentFromDragData): Pass the naked pointer as the PassRefPtr<Range> argument to the DragData::asFragment()
call. The naked pointer will get wrapped and protected by a new PassRefPtr. Passing the original PassRefPtr will also create
a new PassRefPtr, but will leak the reference of the original one, leaving its pointer nullified.

1:59 PM Changeset in webkit [155784] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

Unreviewed GTK build fix after r155774.

  • bindings/scripts/CodeGeneratorGObject.pm:

(SkipFunction): Temporarily skip generation of bindings for the Console::profile() and Console::profileEnd() methods
as they're not correctly generated for the moment.

1:21 PM Changeset in webkit [155783] by fpizlo@apple.com
  • 5 edits in trunk/Source/JavaScriptCore

Rename IntegerBranch/IntegerCompare to Int32Branch/Int32Compare.

Rubber stamped by Mark Hahnenberg.

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePeepHoleInt32Branch):
(JSC::DFG::SpeculativeJIT::compilePeepHoleBranch):
(JSC::DFG::SpeculativeJIT::compare):
(JSC::DFG::SpeculativeJIT::compileStrictEq):

  • dfg/DFGSpeculativeJIT.h:
  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compileInt32Compare):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compileInt32Compare):

1:15 PM Changeset in webkit [155782] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Shrink the nameToPseudoTypeMap function
​https://bugs.webkit.org/show_bug.cgi?id=121367

Reviewed by Andreas Kling.

  • css/CSSSelector.cpp:

(WebCore::populatePseudoTypeByNameMap): Replaces the old nameToPseudoTypeMap
function. Use a table instead of lots of globals and unrolled code to set up
the map.
(WebCore::CSSSelector::parsePseudoType): Updated to use the code above, and
also use a different style that's tighter.

1:11 PM Changeset in webkit [155781] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Shrink the listMarkerText function
​https://bugs.webkit.org/show_bug.cgi?id=121364

Reviewed by Andreas Kling.

Changed code to use StringBuilder, which is a better fit for what we are
doing here. But mainly, inlining was out of hand, so hit that with the
NEVER_INLINE stick. Might be worth revisiting this to optimize further for
speed by making StringBuilder better at this.

  • rendering/RenderListMarker.cpp:

(WebCore::toRoman): NEVER_INLINE and use StringBuilder.
(WebCore::toAlphabeticOrNumeric): Use StringBuilder.
(WebCore::toSymbolic): NEVER_INLINE and use StringBuilder.
(WebCore::toAlphabetic): Ditto.
(WebCore::toNumeric): Ditto.
(WebCore::toHebrewUnder1000): NEVER_INLINE.
(WebCore::toHebrew): NEVER_INLINE and use StringBuilder.
(WebCore::toArmenianUnder10000): NEVER_INLINE.
(WebCore::toArmenian): NEVER_INLINE and use StringBuilder.
(WebCore::toGeorgian): NEVER_INLINE and use StringBuilder.
(WebCore::toCJKIdeographic): NEVER_INLINE and use StringBuilder.
(WebCore::listMarkerText): Use StringBuilder.

12:51 PM Changeset in webkit [155780] by weinig@apple.com
  • 14 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their
constructor (HTMLAnchorElement, HTMLAppletElement, HTMLAreaElement, HTMLAudioElement)
​https://bugs.webkit.org/show_bug.cgi?id=121365

Reviewed by Andreas Kling.

  • editing/CreateLinkCommand.cpp:

(WebCore::CreateLinkCommand::doApply):

  • editing/UnlinkCommand.cpp:

(WebCore::UnlinkCommand::doApply):

  • html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::HTMLAnchorElement):
(WebCore::HTMLAnchorElement::create):

  • html/HTMLAnchorElement.h:
  • html/HTMLAppletElement.cpp:

(WebCore::HTMLAppletElement::HTMLAppletElement):
(WebCore::HTMLAppletElement::create):

  • html/HTMLAppletElement.h:
  • html/HTMLAreaElement.cpp:

(WebCore::HTMLAreaElement::HTMLAreaElement):
(WebCore::HTMLAreaElement::create):

  • html/HTMLAreaElement.h:
  • html/HTMLAudioElement.cpp:

(WebCore::HTMLAudioElement::HTMLAudioElement):
(WebCore::HTMLAudioElement::create):
(WebCore::HTMLAudioElement::createForJSConstructor):

  • html/HTMLAudioElement.h:
  • html/HTMLTagNames.in:
  • html/HTMLViewSourceDocument.cpp:

(WebCore::HTMLViewSourceDocument::addLink):

  • page/DragController.cpp:

(WebCore::documentFragmentFromDragData):

12:48 PM Changeset in webkit [155779] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

Get INPUT_SPEECH compiling for Nix, EFL and GTK
​https://bugs.webkit.org/show_bug.cgi?id=121362

r154257 was not applied completely in HTMLInputElement.cpp and there was a wrong method signature
in TextControlInnerElements.
Also adding the files needed to build INPUT_SPEECH to each build system files.

Patch by Thiago de Barros Lacerda <​thiago.lacerda@openbossa.org> on 2013-09-14
Reviewed by Andreas Kling.

No new tests needed.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::parseAttribute):

  • html/shadow/TextControlInnerElements.h:
12:21 PM Changeset in webkit [155778] by berto@igalia.com
  • 2 edits in trunk/Source/WebCore

Missing allow-none introspection annotation for DomDocument.evaluate method
​https://bugs.webkit.org/show_bug.cgi?id=118310

Reviewed by Martin Robinson.

Add (allow-none) introspection annotation to parameters that can
be NULL.

  • bindings/scripts/CodeGeneratorGObject.pm:

(GenerateFunction):

12:16 PM Changeset in webkit [155777] by weinig@apple.com
  • 7 edits in trunk/Source/WebCore

[CTTE] Node subclasses should take a Document by reference in their constructor (HTMLBodyElement)
​https://bugs.webkit.org/show_bug.cgi?id=121363

Reviewed by Andreas Kling.

  • dom/make_names.pl:

(defaultTagPropertyHash):
(printConstructorInterior):
Add a new property, 'constructorTakesDocumentReference', to indicate that the class
has been converted to take a Document by reference. Once all classes are converted, this
will go away.

  • html/HTMLTagNames.in:

Add the 'constructorTakesDocumentReference' property to the <body> tag as an initial
step.

  • html/HTMLBodyElement.cpp:

(WebCore::HTMLBodyElement::HTMLBodyElement):
(WebCore::HTMLBodyElement::create):

  • html/HTMLBodyElement.h:

Modify the constructor and create functions of HTMLBodyElement to take a Document reference.

  • editing/markup.cpp:

(WebCore::createFragmentFromMarkup):
(WebCore::createFragmentForTransformToFragment):

  • html/HTMLViewSourceDocument.cpp:

(WebCore::HTMLViewSourceDocument::createContainingTable):
Update callers of HTMLBodyElement::create() to pass a Document reference.

12:14 PM Changeset in webkit [155776] by Darin Adler
  • 2 edits in trunk/Source/WebCore

stage

12:03 PM Changeset in webkit [155775] by Darin Adler
  • 5 edits in trunk/Source

Add NeverDestroyed::get
​https://bugs.webkit.org/show_bug.cgi?id=121358

Reviewed by Anders Carlsson.

Source/WebCore:

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::parseAttribute): Use auto and the new get
function to make this code less repetitive.

Source/WTF:

  • wtf/NeverDestroyed.h: Removed code to make this non-copyable, since we also

use a macro to accomplish that.
(WTF::NeverDestroyed::get): Added. Useful in conjuction with auto to avoid
having to repeat the type when we want to put one of these into a reference.

11:54 AM WebKitGTK/2.2.x edited by berto@igalia.com
(diff)
11:52 AM WebKitGTK/2.2.x edited by berto@igalia.com
(diff)
11:29 AM Changeset in webkit [155774] by weinig@apple.com
  • 11 edits
    1 delete in trunk/Source/WebCore

There is no need for any custom bindings for Console
​https://bugs.webkit.org/show_bug.cgi?id=121338

Reviewed by Timothy Hatcher.

  • GNUmakefile.list.am:
  • Target.pri:
  • UseJSC.cmake:
  • WebCore.vcxproj/WebCore.vcxproj:
  • WebCore.vcxproj/WebCore.vcxproj.filters:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:

Remove JSConsoleCustom.cpp from project files.

  • bindings/js/JSConsoleCustom.cpp:

Removed.

  • page/Console.cpp:

(WebCore::Console::profile):
(WebCore::Console::profileEnd):

  • page/Console.h:

Swap the order of the arguments to appease the bindings generator.

  • page/Console.idl:

Add the [CallWith=ScriptState] attribute to pass the ExecState
and [TreatNullAs=NullString, TreatUndefinedAs=NullString] to get
the right stringification.

11:09 AM WebKitGTK/2.0.x edited by berto@igalia.com
(diff)
11:07 AM Changeset in webkit [155773] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

Build fix after r155770.

  • UIProcess/API/mac/WKViewPrivate.h: Added WKBase.h import.
11:03 AM Changeset in webkit [155772] by berto@igalia.com
  • 2 edits in trunk/Tools

[GTK] Fails to compile if the build directory is in a different partition
​https://bugs.webkit.org/show_bug.cgi?id=121296

Reviewed by Martin Robinson.

Instead of using os.symlink() to install the gtk-doc templates,
use os.link() first and fall back to os.symlink() if it fails.

  • gtk/gtkdoc.py:

(GTKDoc._copy_doc_files_to_output_dir.copy_file_replacing_existing):

10:26 AM Changeset in webkit [155771] by andersca@apple.com
  • 8 edits in trunk/Source/WebCore

AnimationBase::m_animation is never null
​https://bugs.webkit.org/show_bug.cgi?id=121360

Reviewed by Darin Adler.

Enforce this by using references instead of pointers.

  • page/animation/AnimationBase.cpp:
  • page/animation/AnimationBase.h:
  • page/animation/CompositeAnimation.cpp:
  • page/animation/ImplicitAnimation.cpp:
  • page/animation/ImplicitAnimation.h:
  • page/animation/KeyframeAnimation.cpp:
  • page/animation/KeyframeAnimation.h:
10:06 AM Changeset in webkit [155770] by mitz@apple.com
  • 23 edits
    3 copies in trunk/Source/WebKit2

[mac] Public header WKBase.h imports a private header, WKBaseMac.h
​https://bugs.webkit.org/show_bug.cgi?id=121216

Reviewed by Darin Adler.

  • Shared/API/c/WKBase.h: Moved definitions of enums and WK_EXPORT and WK_INLINE macros

out of this files.

  • Shared/API/c/WKDeclarationSpecifiers.h: Copied from Source/WebKit2/Shared/API/c/WKBase.h.

Contains definitions of WK_EXPORT and WK_INLINE.

  • Shared/API/c/WKSharedAPICast.h: Added #includes.
  • Shared/API/c/WKUserContentInjectedFrames.h: Copied from Source/WebKit2/Shared/API/c/WKBase.h.

Contains enum definition.

  • Shared/API/c/WKUserScriptInjectionTime.h: Copied from Source/WebKit2/Shared/API/c/WKBase.h.

Contains enum definition.

  • UIProcess/API/C/WKPageGroup.h: Added #includes.
  • UIProcess/API/mac/WKBrowsingContextController.h: Removed WKBase.h import from this public

header, replacing it with WKDeclarationSpecifiers.h import.

  • UIProcess/API/mac/WKBrowsingContextControllerPrivate.h: Added WKBase.h import.
  • UIProcess/API/mac/WKBrowsingContextGroup.h: Removed WKBase.h import from this public

header, added necessary public imports.

  • UIProcess/API/mac/WKBrowsingContextGroupPrivate.h: Added WKBase.h import.
  • UIProcess/API/mac/WKBrowsingContextLoadDelegate.h: Removed WKBase.h import from public

header.

  • UIProcess/API/mac/WKConnection.h: Ditto. Added WKDeclarationSpecifiers.h import.
  • UIProcess/API/mac/WKConnectionInternal.h: Added WKBase.h import.
  • UIProcess/API/mac/WKProcessGroup.h: Removed WKBase.h import from public header, added

WKDeclarationSpecifiers.h import.

  • UIProcess/API/mac/WKProcessGroupPrivate.h: Added WKBase.h import.
  • UIProcess/API/mac/WKView.h: Removed WKBase.h import from public header, added

WKDeclarationSpecifiers.h import.

  • WebKit2.xcodeproj/project.pbxproj: Added new public header files, made WKBase.h private.
  • WebProcess/InjectedBundle/API/mac/WKDOMNode.h: Removed WKBase.h import from public header,

added WKDeclarationSpecifiers.h import.

  • WebProcess/InjectedBundle/API/mac/WKDOMNodePrivate.h: Added WKBase.h import.
  • WebProcess/InjectedBundle/API/mac/WKDOMRange.h: Removed WKBase.h import from public

header, added WKDeclarationSpecifiers.h import.

  • WebProcess/InjectedBundle/API/mac/WKDOMRangePrivate.h: Added WKBase.h import.
  • WebProcess/InjectedBundle/API/mac/WKDOMTextIterator.h: Removed WKBase.h import from public

header, added WKDeclarationSpecifiers.h import.

  • WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextController.h: Ditto.
  • WebProcess/InjectedBundle/API/mac/WKWebProcessPlugInBrowserContextControllerInternal.h:

Added WKBase.h import.

9:19 AM Changeset in webkit [155769] by Darin Adler
  • 3 edits in trunk/Source/WebCore

Shrink the function that builds the event listener attribute name map
​https://bugs.webkit.org/show_bug.cgi?id=121354

Reviewed by Andreas Kling.

  • html/HTMLElement.cpp:

(WebCore::populateEventNameForAttributeLocalNameMap): Added. Uses a
table to populate the map.
(WebCore::eventNameForAttributeLocalNameMap): Added. Uses NeverDestroyed
to allocate a map, but does not initialize it.
(WebCore::HTMLElement::parseAttribute): Put code to use map in here.

  • html/HTMLElement.h: Removed eventNameForAttributeName, no longer

a member function.

9:12 AM Changeset in webkit [155768] by berto@igalia.com
  • 10 edits in trunk

[GTK] WebKitGTK+ is linking against libxslt in too many places
​https://bugs.webkit.org/show_bug.cgi?id=121356

Reviewed by Martin Robinson.

Source/WebCore:

Don't use LIBXSLT_CFLAGS when compiling libWebCorePlatform and
libWebCoreGtk, no source files use libxslt there.

  • GNUmakefile.am:

Source/WebKit/gtk:

Don't use LIBXSLT_CFLAGS when compiling libwebkitgtk, no source
files use libxslt there.

The webkit tests don't use libxslt, so we shouldn't link against
it.

  • GNUmakefile.am:

Source/WebKit2:

Don't use LIBXSLT_CFLAGS when compiling libWebCoreGtk2, no source
files use libxslt there.

  • GNUmakefile.am:

Tools:

TestWebCore, WebKitTestRunner and MiniBrowser don't use libxslt,
so we don't need to link against it.

  • MiniBrowser/gtk/GNUmakefile.am:
  • TestWebKitAPI/GNUmakefile.am:
  • WebKitTestRunner/GNUmakefile.am:
3:22 AM Changeset in webkit [155767] by zandobersek@gmail.com
  • 3 edits in trunk/LayoutTests

Unreviewed GTK gardening. Rebaselining after r155324.

  • platform/gtk/fast/forms/placeholder-position-expected.png:
  • platform/gtk/fast/forms/placeholder-position-expected.txt:
3:20 AM Changeset in webkit [155766] by zandobersek@gmail.com
  • 6 edits in trunk/LayoutTests

Unreviewed GTK gardening. Rebaselining after r155408.

  • platform/gtk/fast/forms/input-placeholder-visibility-1-expected.txt:
  • platform/gtk/fast/forms/input-placeholder-visibility-3-expected.txt:
  • platform/gtk/fast/forms/textarea-placeholder-pseudo-style-expected.txt:
  • platform/gtk/fast/forms/textarea-placeholder-visibility-1-expected.txt:
  • platform/gtk/fast/forms/textarea-placeholder-visibility-2-expected.txt:
3:11 AM Changeset in webkit [155765] by zandobersek@gmail.com
  • 19 edits in trunk/LayoutTests

Unreviewed GTK gardening. Rebaselining after r155253.

  • platform/gtk/editing/pasteboard/4989774-expected.txt:
  • platform/gtk/editing/pasteboard/paste-match-style-001-expected.txt:
  • platform/gtk/editing/selection/caret-ltr-2-expected.txt:
  • platform/gtk/editing/selection/caret-ltr-2-left-expected.txt:
  • platform/gtk/editing/selection/caret-ltr-expected.txt:
  • platform/gtk/editing/selection/caret-ltr-right-expected.txt:
  • platform/gtk/editing/selection/caret-rtl-2-expected.txt:
  • platform/gtk/editing/selection/caret-rtl-2-left-expected.txt:
  • platform/gtk/editing/selection/caret-rtl-expected.txt:
  • platform/gtk/editing/selection/caret-rtl-right-expected.txt:
  • platform/gtk/editing/selection/vertical-rl-ltr-extend-line-backward-wrap-expected.txt:
  • platform/gtk/editing/selection/vertical-rl-ltr-extend-line-forward-wrap-expected.txt:
  • platform/gtk/editing/style/style-boundary-005-expected.txt:
  • platform/gtk/fast/forms/formmove3-expected.txt:
  • platform/gtk/fast/forms/input-align-image-expected.txt:
  • platform/gtk/fast/forms/preserveFormDuringResidualStyle-expected.txt:
  • platform/gtk/fast/invalid/003-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug647-expected.txt:
2:47 AM Changeset in webkit [155764] by zandobersek@gmail.com
  • 3 edits in trunk/LayoutTests

Unreviewed GTK gardening.

Skip additional inspector tests that are failing after switching to the new Web Inspector.

  • platform/gtk-wk1/TestExpectations:
  • platform/gtk/TestExpectations:
2:28 AM Changeset in webkit [155763] by Joseph Pecoraro
  • 11 edits
    1 copy
    5 adds in trunk

Web Inspector: InjectedScriptSource_js and InjectedScriptCanvasModuleSource_js should be minified
​https://bugs.webkit.org/show_bug.cgi?id=121183

Reviewed by Timothy Hatcher.

Source/WebCore:

  • CMakeLists.txt:
  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:

Minify files at build time before running xxd.pl.

  • WebCore.xcodeproj/project.pbxproj:
  • inspector/InjectedScriptCanvasModuleSource.js:
  • inspector/InjectedScriptSource.js:

Strip trailing whitespace (touching the files so the derived sources are regenerated).

  • inspector/InspectorOverlayPage.css: Added.
  • inspector/InspectorOverlayPage.html:
  • inspector/InspectorOverlayPage.js: Copied from Source/WebCore/inspector/InspectorOverlayPage.html.

Move JS and CSS to external files. They are inlined and minified at build time.

  • inspector/Scripts/cssmin.py: Added.
  • inspector/Scripts/jsmin.py: Added.

Copied from WebInspectorUI/Scripts.

  • inspector/Scripts/inline-and-minify-stylesheets-and-scripts.py: Added.

Add a script that can inline stylesheets and scripts. For minifying InspectorOverlayPage.html.

LayoutTests:

Now that the file is minified, a line number changed.

  • platform/mac/inspector/console/command-line-api-expected.txt:
2:26 AM Changeset in webkit [155762] by zandobersek@gmail.com
  • 2 edits in trunk/Source/WebCore

Unreviewed, fixing debug GTK builds after r155709.

  • platform/gtk/PasteboardGtk.cpp:

(WebCore::Pasteboard::write): Assert that the PasteboardURL's URL is not empty.

2:15 AM Changeset in webkit [155761] by akling@apple.com
  • 5 edits in trunk/Source/WebCore

CTTE: RenderDetailsMarker is never anonymous.
<​https://webkit.org/b/121355>

Reviewed by Darin Adler.

This renderer is never anonymous, and always has a corresponding DetailsMarkerControl.
Couldn't resist cleaning up the class a little bit, adding OVERRIDE and making
toRenderDetailsMarker() into a reference version.

2:13 AM Changeset in webkit [155760] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

CTTE: RenderRubyAs{Block,Inline} are never anonymous.
<​https://webkit.org/b/121353>

Reviewed by Darin Adler.

Replace element() with existingElement() and spam class with OVERRIDE.
Removed bogus removeLeftoverAnonymousBlock() override in RenderRubyAsInline
since it didn't override anything from a parent class.

2:11 AM Changeset in webkit [155759] by akling@apple.com
  • 4 edits in trunk/Source/WebCore

CTTE: RenderRubyText is never anonymous.
<​https://webkit.org/b/121352>

Reviewed by Darin Adler.

Replace element() with existingElement() and spam class with OVERRIDE.

1:38 AM Changeset in webkit [155758] by berto@igalia.com
  • 2 edits in trunk/Tools

Uses gtk-doc tools even when --disable-gtk-doc passed
​https://bugs.webkit.org/show_bug.cgi?id=91239

Patch by Ross Burton <​ross.burton@intel.com> on 2013-09-14
Reviewed by Martin Robinson.

Don't run generate-gtkdoc on install if gtk-doc is disabled.

  • GNUmakefile.am:
1:36 AM Changeset in webkit [155757] by berto@igalia.com
  • 2 edits in trunk/Tools

[GTK] Fails to compile if the build directory is in a different partition
​https://bugs.webkit.org/show_bug.cgi?id=121296

Reviewed by Carlos Garcia Campos.

Use symlinks instead of hard links to support building on a
different file system.

  • gtk/gtkdoc.py:

(GTKDoc._copy_doc_files_to_output_dir.copy_file_replacing_existing):

12:46 AM Changeset in webkit [155756] by akling@apple.com
  • 8 edits in trunk/Source/WebCore

CTTE: RenderSVGBlock renderers are never anonymous.
<​https://webkit.org/b/121351>

Reviewed by Antti Koivisto.

RenderSVGBlock inherits from RenderBlockFlow, so needs separate treatment
from RenderSVGModelObject.

But like RSMO, these renderers are never anonymous, so add strongly typed
reference getters and do away with some magical type casts.

12:28 AM Changeset in webkit [155755] by akling@apple.com
  • 34 edits in trunk/Source/WebCore

CTTE: RenderSVGModelObject renderers are never anonymous.
<​https://webkit.org/b/121350>

Reviewed by Darin Adler.

Flip the switch, turning RenderSVGModelObject::element() into a reference.

Note: See TracTimeline for information about the timeline view.