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

Timeline



Sep 22, 2011:

11:59 PM Changeset in webkit [95786] by fpizlo@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

GetScopedVar should have value profiling
​https://bugs.webkit.org/show_bug.cgi?id=68676

Reviewed by Oliver Hunt.

Added GetScopedVar value profiling and predictin propagation.
Added GetScopeChain to CSE.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::predict):

  • dfg/DFGNode.h:

(JSC::DFG::Node::hasPrediction):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):
(JSC::DFG::Propagator::getScopeChainLoadElimination):
(JSC::DFG::Propagator::performNodeCSE):

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::emit_op_get_scoped_var):

10:58 PM Changeset in webkit [95785] by Csaba Osztrogonác
  • 3 edits in trunk/LayoutTests

[Qt] Unreviewed gardening, update expected file after r95745.

  • platform/qt/editing/deleting/merge-whitespace-pre-expected.png:
  • platform/qt/editing/deleting/merge-whitespace-pre-expected.txt:
10:53 PM Changeset in webkit [95784] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed fix after r95764, update expected fail to expected pass.

  • ietestcenter/Javascript/15.3.4.5-0-2-expected.txt:
9:54 PM Changeset in webkit [95783] by tony@chromium.org
  • 5 edits
    3 adds in trunk

implement -webkit-flex-order
​https://bugs.webkit.org/show_bug.cgi?id=67432

Reviewed by Ojan Vafai.

Source/WebCore:

flex-order can be an int (including negative), but we disallow the two
smallest values so we can put the numbers into a hash set.

Also, create two iterators: one that goes in render tree order (we use
this for the first pass and to collect the possible flex order values)
and one that goes in flex order.

Test: css3/flexbox/flex-order.html

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseValue):

  • rendering/RenderFlexibleBox.cpp:

(WebCore::FlexOrderHashTraits::emptyValue):
(WebCore::FlexOrderHashTraits::constructDeletedValue):
(WebCore::FlexOrderHashTraits::isDeletedValue):
(WebCore::RenderFlexibleBox::TreeOrderIterator::TreeOrderIterator): A simple iterator

that goes in render tree order.

(WebCore::RenderFlexibleBox::TreeOrderIterator::next):
(WebCore::RenderFlexibleBox::TreeOrderIterator::reset):
(WebCore::RenderFlexibleBox::TreeOrderIterator::flexOrderValues):
(WebCore::RenderFlexibleBox::FlexOrderIterator::FlexOrderIterator): An iterator that

goes in flex-order order. Creating this involves sorting, so only create it once
and pass it around.

(WebCore::RenderFlexibleBox::FlexOrderIterator::first):
(WebCore::RenderFlexibleBox::FlexOrderIterator::next):
(WebCore::RenderFlexibleBox::FlexOrderIterator::reset):
(WebCore::RenderFlexibleBox::layoutHorizontalBlock):
(WebCore::RenderFlexibleBox::computePreferredLogicalWidth):
(WebCore::RenderFlexibleBox::runFreeSpaceAllocationAlgorithmInlineDirection):
(WebCore::RenderFlexibleBox::layoutAndPlaceChildrenInlineDirection):

  • rendering/RenderFlexibleBox.h:

LayoutTests:

  • css3/flexbox/flex-order-expected.png: Added.
  • css3/flexbox/flex-order-expected.txt: Added.
  • css3/flexbox/flex-order.html: Added.
9:10 PM Changeset in webkit [95782] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Use AffineTransform scale functions in ShadowBlur::adjustBlurRadius
​https://bugs.webkit.org/show_bug.cgi?id=68667

Reviewed by Simon Fraser.

  • platform/graphics/ShadowBlur.cpp:

(WebCore::ShadowBlur::adjustBlurRadius): Use AffineTransform::xScale and
AffineTransform::yScale instead of the custom code here that seems to do
the same thing.

9:00 PM Changeset in webkit [95781] by dpranke@chromium.org
  • 2 edits in trunk/Tools

enable NRWT for run-webkit-tests on Lion
​https://bugs.webkit.org/show_bug.cgi?id=68673

Reviewed by Eric Seidel.

  • Scripts/run-webkit-tests:

(useNewRunWebKitTests):

8:35 PM Changeset in webkit [95780] by commit-queue@webkit.org
  • 50 edits in trunk/LayoutTests

Rebaseline for bug 65583 (path based border radius drawing on skia) part 6
​https://bugs.webkit.org/show_bug.cgi?id=68671

Patch by Ben Wells <​benwells@chromium.org> on 2011-09-22
Reviewed by James Robinson.

  • platform/chromium-linux/tables/mozilla/marvin/: 9 changes
  • platform/chromium-linux/tables/mozilla_expected_failures/marvin/: 14 changes
  • platform/chromium-linux/transforms/svg-vs-css-expected.png:
  • platform/chromium-win/tables/mozilla/marvin/: 9 changes
  • platform/chromium-win/tables/mozilla_expected_failures/marvin/: 14 changes
  • platform/chromium-win/transforms/svg-vs-css-expected.png:
  • platform/chromium/test_expectations.txt:
8:27 PM Changeset in webkit [95779] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

PPC build fix, part 3.

  • runtime/Executable.cpp:

(JSC::FunctionExecutable::compileForConstructInternal):

8:21 PM Changeset in webkit [95778] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Remove unneeded type conversion from background bleed code
​https://bugs.webkit.org/show_bug.cgi?id=68669

Reviewed by Geoffrey Garen.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::backgroundRectAdjustedForBleedAvoidance): Use ceil instead of ceilf
and eliminate superflous conversion to float and use of FloatSize. Also added
a "why" comment.

8:00 PM Changeset in webkit [95777] by Darin Adler
  • 6 edits in trunk/Source/WebCore

Refactor checks for antialiasing lines to share a single function
​https://bugs.webkit.org/show_bug.cgi?id=68666

Reviewed by Dan Bernstein.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintColumnRules): Call shouldAntialiasLines.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintBorder): Ditto.
(WebCore::RenderBoxModelObject::shouldAntialiasLines): Added.

  • rendering/RenderBoxModelObject.h: Added shouldAntialiasLines function.
  • rendering/RenderInline.cpp:

(WebCore::RenderInline::paintOutlineForLine): Call shouldAntialiasLines.

  • rendering/RenderTableCell.cpp:

(WebCore::RenderTableCell::paintCollapsedBorder): Ditto.

7:41 PM Changeset in webkit [95776] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Remove unused members from LayerChromium.
These members are never set, and never used.
Other members that are set but not used have been left (we may use them
later, e.g. m_opaque).
​https://bugs.webkit.org/show_bug.cgi?id=68297

Patch by Antoine Labour <​piman@chromium.org> on 2011-09-22
Reviewed by James Robinson.

No functional change, no new test needed.

  • platform/graphics/chromium/LayerChromium.cpp:

(WebCore::LayerChromium::LayerChromium):

  • platform/graphics/chromium/LayerChromium.h:
7:40 PM Changeset in webkit [95775] by annacc@chromium.org
  • 2 edits in trunk/LayoutTests

Adding testTracks() function to video-test.js
​https://bugs.webkit.org/show_bug.cgi?id=68643

Reviewed by Eric Carlson.

  • media/video-test.js:

(testTracks):

7:28 PM Changeset in webkit [95774] by commit-queue@webkit.org
  • 5 edits
    6 adds in trunk

Make XSSAuditor extract meaningful snippet from script blocks for comparison
against the URL when checking for reflection. Avoids getting caugh up in
trailing comments.
​https://bugs.webkit.org/show_bug.cgi?id=68094

Patch by Tom Sepez <​tsepez@chromium.org> on 2011-09-22
Reviewed by Adam Barth.

Source/WebCore:

Tests: http/tests/security/xssAuditor/script-tag-with-trailing-comment.html

http/tests/security/xssAuditor/script-tag-with-trailing-comment2.html
http/tests/security/xssAuditor/script-tag-with-trailing-comment3.html

  • html/parser/XSSAuditor.cpp:

(WebCore::XSSAuditor::filterTokenAfterScriptStartTag):
(WebCore::XSSAuditor::extractCodeFragment):

  • html/parser/XSSAuditor.h:

LayoutTests:

  • http/tests/security/xssAuditor/resources/echo-intertag.pl:
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment2-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment2.html: Added.
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment3-expected.txt: Added.
  • http/tests/security/xssAuditor/script-tag-with-trailing-comment3.html: Added.
7:27 PM Changeset in webkit [95773] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[Qt] TouchWebView load test not running
​https://bugs.webkit.org/show_bug.cgi?id=68620

Patch by Gopal Raghavan <​gopal.1.raghavan@nokia.com> on 2011-09-22
Reviewed by Chang Shu.

Load test was not executed since it was missing in pro file. Added tst_load.qml to qmltests.pro

  • UIProcess/API/qt/tests/qmltests/qmltests.pro:
7:16 PM Changeset in webkit [95772] by fpizlo@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Another PPC build fix.

  • runtime/Executable.cpp:
  • runtime/Executable.h:
6:56 PM Changeset in webkit [95771] by commit-queue@webkit.org
  • 55 edits
    2 moves
    1 add in trunk/LayoutTests

Rebaseline for bug 65583 (path based border radius drawing on skia) part 5
​https://bugs.webkit.org/show_bug.cgi?id=68608

Patch by Ben Wells <​benwells@chromium.org> on 2011-09-22
Reviewed by James Robinson.

  • platform/chromium-linux/css2.1/: 14 changes
  • platform/chromium-linux/editing/execCommand/5138441-expected.png:
  • platform/chromium-linux/editing/inserting/editing-empty-divs-expected.png:
  • platform/chromium-linux/editing/selection/transformed-selection-rects-expected.png:
  • platform/chromium-linux/fast/: 9 changes
  • platform/chromium-win/css2.1/: 14 changes
  • platform/chromium-win/editing/execCommand/5138441-expected.png:
  • platform/chromium-win/editing/inserting/editing-empty-divs-expected.png:
  • platform/chromium-win/editing/selection/transformed-selection-rects-expected.png:
  • platform/chromium-win/fast/: 10 changes
  • platform/chromium/test_expectations.txt:
  • platform/win-xp/editing/selection/transformed-selection-rects-expected.txt: Renamed from LayoutTests/platform/win-wk2/editing/selection/transformed-selection-rects-expected.txt.
  • platform/win/editing/selection/transformed-selection-rects-expected.txt: Renamed from LayoutTests/platform/chromium-win/editing/selection/transformed-selection-rects-expected.txt.
6:42 PM Changeset in webkit [95770] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk/Tools

Add unit test for existing StringBuilder
​https://bugs.webkit.org/show_bug.cgi?id=67080

Patch by Xianzhu Wang <​wangxianzhu@chromium.org> on 2011-09-22
Reviewed by Darin Adler.

  • TestWebKitAPI/TestWebKitAPI.gypi:
  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
  • TestWebKitAPI/Tests/WTF/StringBuilder.cpp: Added.

(operator<<):
(TestWebKitAPI::expectBuilderContent):
(TestWebKitAPI::expectEmpty):
(TestWebKitAPI::TEST):

6:33 PM WebKit Team edited by dino@apple.com
Moving me around (diff)
6:33 PM AddingFeatures created by dino@apple.com
Figured it was worth documenting how to add a feature
6:29 PM Changeset in webkit [95769] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Updated expectations and filed WK68674.

  • platform/chromium/test_expectations.txt:
6:21 PM WikiStart edited by dino@apple.com
Adding link to new ENABLE page (diff)
6:12 PM Changeset in webkit [95768] by Nate Chapin
  • 16 edits in trunk/Source/WebCore

Remove didReceiveAuthenticationChallenge() from SubresourceLoaderClient.
Instead, add a load-specific policy for showing the user authentication
challenge down to ResourceLoaderOptions and enforce it in ResourceLoader.
​https://bugs.webkit.org/show_bug.cgi?id=65330

Reviewed by Alexey Proskuryakov.

No new tests, refactor only.

  • loader/DocumentThreadableLoader.cpp:
  • loader/DocumentThreadableLoader.h:
  • loader/MainResourceLoader.cpp:
  • loader/NetscapePlugInStreamLoader.cpp:
  • loader/ResourceLoadScheduler.h:
  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::didReceiveAuthenticationChallenge):

For resource types that always send a challenge to the embedder,
this patch doesn't change anything. For those that don't, we will
always try to continue without credentials when they are forbidden
and the platform supports it.
When continuing without credentials was initially implemented in
DocumentThreadableLoader, we sent the ThreadableLoaderClient a didFail(),
then canceled the SubresourceLoader. This was necessary because of the
quirks of ThreadableLoader cancellation (we sever the client/loader connections
before the load actually cancels), but a simple didFail() should suffice at
the ResourceLoader layer.

  • loader/ResourceLoaderOptions.h:
  • loader/SubresourceLoader.cpp:
  • loader/SubresourceLoader.h:
  • loader/SubresourceLoaderClient.h:
  • loader/cache/CachedResource.cpp:
  • loader/cache/CachedResourceLoader.cpp:
  • loader/cache/CachedResourceLoader.h:
  • loader/icon/IconLoader.cpp: The ResourceLoader implementation of

didReceiveAuthenticationChallege means that IconLoader will now
try to continue with credentials on platforms that support it,
rather than just canceling outright. We still will never prompt
for authentication for icons.

  • loader/icon/IconLoader.h:
5:43 PM Changeset in webkit [95767] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[EFL] Use ewk_view to navigate in history instead of ewk_history.
​https://bugs.webkit.org/show_bug.cgi?id=68455

Patch by Raphael Kubo da Costa <​kubo@profusion.mobi> on 2011-09-22
Reviewed by Antonio Gomes.

BackForwardItem::invoke() was using ewk_history functions to navigate
in history; however, this will only call WebCore::BackForwardListImpl
and not move between pages at all. We now call ewk_view_navigate and
its siblings to make sure the navigation actually happens.

This should make tests like fast/dom/navigation-type-back-forward.html
stop timing out.

  • DumpRenderTree/efl/WorkQueueItemEfl.cpp:

(BackForwardItem::invoke):

5:38 PM Changeset in webkit [95766] by dino@apple.com
  • 13 edits in trunk

Add ENABLE_CSS_FILTERS
​https://bugs.webkit.org/show_bug.cgi?id=68652

Reviewed by Simon Fraser.

  • Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:
  • Source/WebCore/Configurations/FeatureDefines.xcconfig:
  • Source/WebKit/mac/Configurations/FeatureDefines.xcconfig:
  • Source/WebKit2/Configurations/FeatureDefines.xcconfig:
  • WebKitLibraries/win/tools/vsprops/FeatureDefines.vsprops:
  • WebKitLibraries/win/tools/vsprops/FeatureDefinesCairo.vsprops:
  • Tools/Scripts/build-webkit:
5:02 PM Changeset in webkit [95765] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Make sure to update scrollbars in ScrollView::setFrameRect
​https://bugs.webkit.org/show_bug.cgi?id=68663

Reviewed by Sam Weinig.

Call updateScrollbars after setting the frame rect. This was previously done by the
call to setBoundsSize from RenderWidget::setWidgetGeometry but setBoundsSize was removed
completely in ​http://trac.webkit.org/changeset/95725.

  • platform/ScrollView.cpp:

(WebCore::ScrollView::setFrameRect):

4:57 PM Changeset in webkit [95764] by barraclough@apple.com
  • 5 edits in trunk

Function.prototype.bind.length shoudl be 1.

Rubber stamped by Olier Hunt.

Source/JavaScriptCore:

  • runtime/FunctionPrototype.cpp:

(JSC::FunctionPrototype::addFunctionProperties):

LayoutTests:

  • fast/js/function-bind-expected.txt:
  • fast/js/script-tests/function-bind.js:
4:24 PM Changeset in webkit [95763] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

PPC build fix.

  • bytecode/CodeBlock.h:
4:22 PM Changeset in webkit [95762] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Windows build fix pt. 2

4:05 PM Changeset in webkit [95761] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Windows build fix pt. 1

3:58 PM Changeset in webkit [95760] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][Mac]Unreviewed gardening.

New Qt specific results were added by r95723, but they fail on Qt-Mac platform

  • platform/qt-mac/Skipped: Skip css3/selectors3 tests.
3:49 PM Changeset in webkit [95759] by Simon Fraser
  • 11 edits
    1 copy
    1 add in trunk/Source/WebKit2

Add APIObjects for Size, Point, Rect
​https://bugs.webkit.org/show_bug.cgi?id=68644

Reviewed by Sam Weinig.

Make APIObjects for size, point and rect so that they can be
used in WKArrayRefs.

Make some inline helper methods to create WKSize, WKPoint and WKRect
types.

  • GNUmakefile.am:
  • Shared/API/c/WKBase.h:
  • Shared/API/c/WKGeometry.cpp: Copied from Source/WebKit2/Shared/API/c/WKGeometry.h.

(WKSizeGetTypeID):
(WKPointGetTypeID):
(WKRectGetTypeID):
(WKPointCreate):
(WKSizeCreate):
(WKRectCreate):
(WKSizeGetValue):
(WKPointGetValue):
(WKRectGetValue):

  • Shared/API/c/WKGeometry.h:

(WKPoint::WKPointMake):
(WKPoint::WKSizeMake):
(WKPoint::WKRectMake):

  • Shared/API/c/WKSharedAPICast.h:
  • Shared/APIObject.h:
  • Shared/UserMessageCoders.h:

(WebKit::UserMessageEncoder::baseEncode):
(WebKit::UserMessageDecoder::baseDecode):

  • Shared/WebGeometry.h: Added.

(WebKit::WebSize::create):
(WebKit::WebSize::size):
(WebKit::WebSize::WebSize):
(WebKit::WebSize::type):
(WebKit::WebPoint::create):
(WebKit::WebPoint::point):
(WebKit::WebPoint::WebPoint):
(WebKit::WebPoint::type):
(WebKit::WebRect::create):
(WebKit::WebRect::rect):
(WebKit::WebRect::WebRect):
(WebKit::WebRect::type):

  • WebKit2.pro:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebKit2API.pri:
  • win/WebKit2.vcproj:
3:42 PM Changeset in webkit [95758] by fpizlo@apple.com
  • 13 edits in trunk/Source/JavaScriptCore

DFG JIT does not support to_primitive or strcat
​https://bugs.webkit.org/show_bug.cgi?id=68582

Reviewed by Darin Adler.

This adds functional support for to_primitive and strcat. It focuses
on minimizing the amount of code emitted on to_primitive (if we know
that it is a primitive or can speculate cheaply, then we omit the
slow path) and on keeping the implementation of strcat simple while
leveraging whatever optimizations we have already. In particular,
unlike the Call and Construct nodes which require extending the size
of the DFG's callee registers, StrCat takes advantage of the fact
that no JS code can run while StrCat is in progress and uses a
scratch buffer, rather than the register file, to store the list of
values to concatenate. This was done mainly to keep the code simple,
but there are probably other benefits to keeping call frame sizes
down. Essentially, this patch ensures that the presence of an
op_strcat does not mess up any other optimizations we might do while
ensuring that if you do execute it, it'll work about as well as you'd
expect.

When combined with the previous patch for integer division, this is a
14% speed-up on Kraken. Without it, it would have been a 2% loss.

  • assembler/AbstractMacroAssembler.h:

(JSC::AbstractMacroAssembler::TrustedImmPtr::TrustedImmPtr):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGJITCodeGenerator.h:

(JSC::DFG::JITCodeGenerator::callOperation):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::exitSpeculativeWithOSR):

  • dfg/DFGNode.h:
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):
(JSC::DFG::Propagator::performNodeCSE):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::JSGlobalData):
(JSC::JSGlobalData::~JSGlobalData):

  • runtime/JSGlobalData.h:

(JSC::JSGlobalData::scratchBufferForSize):

3:35 PM Changeset in webkit [95757] by jonlee@apple.com
  • 4 edits in trunk/Source/WebCore

Progress control gets cropped on the bottom
​https://bugs.webkit.org/show_bug.cgi?id=68302
<rdar://problem/10069915>

Reviewed by Kent Tamura.

As it turns out the smaller control type does not get rendered either, so this patch
fixes both.

  • manual-tests/dom/progressbar.html: Altered to show both sizes of controls
  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm: Added methods to inflate the drawing rect

(WebCore::RenderThemeMac::progressBarSizes):
(WebCore::RenderThemeMac::progressBarMargins):
(WebCore::RenderThemeMac::minimumProgressBarHeight):
(WebCore::RenderThemeMac::paintProgressBar): Inflate the rect based on minimum desired
control height and glow margin.

3:28 PM Changeset in webkit [95756] by hyatt@apple.com
  • 5 edits
    3 adds in trunk

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

Make matchedEndLine smart enough to not match lines that have moved to new
regions with different available content logical widths. When this happens, we go ahead and treat
the line as failing to match.

Reviewed by Anders Carlsson.

Source/WebCore:

Added new tests in fast/regions.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::lineWidthForPaginatedLineChanged):

  • rendering/RenderBlock.h:

Enhanced to take an optional delta, so that you can check a new position without having to move
the line box.

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::checkPaginationAndFloatsAtEndLine):
New function that refactors checking for line width changes from region movement as well as the
float checks. Since the float checks were duplicated twice, this is a nice refactoring.

(WebCore::RenderBlock::matchedEndLine):
Changed to call the new helper function that will check both floats and pagination.

LayoutTests:

  • fast/regions/webkit-flow-inlines-dynamic.html: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-dynamic-expected.png: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-dynamic-expected.txt: Added.
3:13 PM Changeset in webkit [95755] by andersca@apple.com
  • 2 edits in trunk/Tools

EventSenderProxy::mouseUp should call -[WKView mouseUp:] instead of -[WKView mouseDown:]
​https://bugs.webkit.org/show_bug.cgi?id=68660

Reviewed by Adam Barth.

  • WebKitTestRunner/mac/EventSenderProxy.mm:

(WTR::EventSenderProxy::mouseUp):

3:02 PM Changeset in webkit [95754] by fpizlo@apple.com
  • 7 edits in trunk/Source/JavaScriptCore

DFG JIT should support integer division
​https://bugs.webkit.org/show_bug.cgi?id=68597

Reviewed by Darin Adler.

This adds support for ArithDiv speculating integer, and speculating
that the result is integer (i.e. remainder = 0).

This is a 4% win on Kraken and a 1% loss on V8.

  • bytecode/CodeBlock.h:
  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::makeDivSafe):
(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGNode.h:

(JSC::DFG::Node::hasArithNodeFlags):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateArithNodeFlags):
(JSC::DFG::Propagator::propagateNodePredictions):
(JSC::DFG::Propagator::fixupNode):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • jit/JITArithmetic.cpp:

(JSC::JIT::emit_op_div):

2:55 PM Changeset in webkit [95753] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Implement put_scoped_var in the DFG jit
​https://bugs.webkit.org/show_bug.cgi?id=68653

Reviewed by Gavin Barraclough.

Naive implementation of put_scoped_var. Same story as the
get_scoped_var implementation, although I've hoisted scope
object acquisition into a separate dfg node. Ideally in the
future we would reuse the resolved scope chain object, but
for now we don't.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGNode.h:

(JSC::DFG::Node::hasScopeChainDepth):
(JSC::DFG::Node::scopeChainDepth):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

2:43 PM Changeset in webkit [95752] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

Skip test that need new baselines.

  • platform/mac/Skipped:
2:22 PM Changeset in webkit [95751] by barraclough@apple.com
  • 58 edits
    5 adds in trunk

Implement Function.prototype.bind
​https://bugs.webkit.org/show_bug.cgi?id=26382

Reviewed by Sam Weinig.

Source/JavaScriptCore:

This patch provides a basic functional implementation
for Function.bind. It should (hopefully!) be fully
functionally correct, and the bound functions can be
called to quickly (since they are a subclass of
JSFunction, not InternalFunction), but we'll probably
want to follow up with some optimization work to keep
bound calls in JIT code.

(JSC::JITThunks::hostFunctionStub):

  • jit/JITStubs.h:
  • jsc.cpp:

(GlobalObject::addFunction):

  • runtime/CommonIdentifiers.h:
  • runtime/ConstructData.h:
  • runtime/Executable.h:

(JSC::NativeExecutable::NativeExecutable):

  • runtime/FunctionPrototype.cpp:

(JSC::FunctionPrototype::addFunctionProperties):
(JSC::functionProtoFuncBind):

  • runtime/FunctionPrototype.h:
  • runtime/JSBoundFunction.cpp: Added.

(JSC::boundFunctionCall):
(JSC::boundFunctionConstruct):
(JSC::JSBoundFunction::create):
(JSC::JSBoundFunction::hasInstance):
(JSC::JSBoundFunction::getOwnPropertySlot):
(JSC::JSBoundFunction::getOwnPropertyDescriptor):
(JSC::JSBoundFunction::JSBoundFunction):
(JSC::JSBoundFunction::finishCreation):

  • runtime/JSBoundFunction.h: Added.

(JSC::JSBoundFunction::targetFunction):
(JSC::JSBoundFunction::boundThis):
(JSC::JSBoundFunction::boundArgs):
(JSC::JSBoundFunction::createStructure):

  • runtime/JSFunction.cpp:

(JSC::JSFunction::create):
(JSC::JSFunction::finishCreation):
(JSC::createDescriptorForThrowingProperty):
(JSC::JSFunction::getOwnPropertySlot):

  • runtime/JSFunction.h:
  • runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::getHostFunction):

  • runtime/JSGlobalData.h:
  • runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::reset):
(JSC::JSGlobalObject::visitChildren):

  • runtime/JSGlobalObject.h:

(JSC::JSGlobalObject::boundFunctionStructure):

  • runtime/Lookup.cpp:

(JSC::setUpStaticFunctionSlot):

Source/WebCore:

Test: fast/js/function-bind.html

  • bindings/js/JSDOMBinding.cpp:

(WebCore::objectToStringFunctionGetter):

  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::nonCachingStaticFunctionGetter):

  • bindings/js/JSHistoryCustom.cpp:

(WebCore::nonCachingStaticBackFunctionGetter):
(WebCore::nonCachingStaticForwardFunctionGetter):
(WebCore::nonCachingStaticGoFunctionGetter):

  • bindings/js/JSLocationCustom.cpp:

(WebCore::nonCachingStaticReplaceFunctionGetter):
(WebCore::nonCachingStaticReloadFunctionGetter):
(WebCore::nonCachingStaticAssignFunctionGetter):

  • Function::create no longer requires functionStructure() to be passed.

LayoutTests:

We now pass Function.bind tests.

  • fast/js/Object-getOwnPropertyNames-expected.txt:
  • fast/js/basic-strict-mode-expected.txt:
  • fast/js/function-bind-expected.txt: Added.
  • fast/js/function-bind.html: Added.
  • fast/js/mozilla/strict/15.3.4.5-expected.txt:
  • fast/js/script-tests/function-bind.js: Added.
  • ietestcenter/Javascript/15.2.3.3-4-38-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-0-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-0-2-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-13.b-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-13.b-2-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-13.b-3-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-13.b-4-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-13.b-5-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-15-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-15-2-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-16-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-2-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-3-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-4-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-5-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-6-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-7-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-8-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-2-9-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-8-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-8-2-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-9-1-expected.txt:
  • ietestcenter/Javascript/15.3.4.5-9-2-expected.txt:
1:50 PM Changeset in webkit [95750] by hyatt@apple.com
  • 11 edits
    1 add
    1 delete in trunk

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

Make determineStartPosition smart enough to not skip over clean lines that have moved to a new
region with a different available content logical width. When this happens, we go ahead and treat
the line as dirty.

Reviewed by Sam Weinig.

Source/WebCore:

Covered by existing fast/regions tests.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutBlock):
Disable region fitting if we detect that the regions all have uniform widths. In this case we know
the content width can't vary, so there's no reason to waste time worrying about it.

(WebCore::RenderBlock::computeLogicalLocationForFloat):
Pull the content width checking code into computeLogicalLocationForFloat, since we've incorporated
region fitting into the fixed offsets now rather than the line functions.

(WebCore::RenderBlock::positionNewFloats):
Changed because more code moved from here into computeLogicalLocationForFloat.

(WebCore::RenderBlock::adjustForRegionFittingIfNeeded):
No changes. Just moved the function above the left offset function.

(WebCore::RenderBlock::logicalLeftOffsetForContent):
(WebCore::RenderBlock::logicalRightOffsetForContent):
Added. These new functions give the fixed left and right offsets for lines in a particular region.
When no regions are present or when all the regions have a uniform width, they are identical
to the versions of the functions that take no arguments.

(WebCore::RenderBlock::logicalRightOffsetForLine):
Changed to no longer do region fitting, since that's built into logicalRightOffsetForContent now.

(WebCore::RenderBlock::lineWidthForPaginatedLineChanged):
New function that takes a root line box and compares its cached old content width with the width
available at the line's new location.

  • rendering/RenderBlock.h:

(WebCore::RenderBlock::logicalRightOffsetForLine):
(WebCore::RenderBlock::logicalLeftOffsetForLine):
Modified to call the logicalXXXOffsetForContent functions that take a block direction position.

(WebCore::RenderBlock::logicalRightOffsetForContent):
(WebCore::RenderBlock::logicalLeftOffsetForContent):
Added the new functions that take a position so that they can fit to the correct region.

(WebCore::RenderBlock::availableLogicalWidthForContent):
New helper function that calls right - left (similar to the corresponding line function that includes
floats).

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::determineStartPosition):
Patched to check if a line has moved to a place with a different available content width. If so, the line
is dirtied so that line layout can re-run.

  • rendering/RootInlineBox.cpp:

(WebCore::RootInlineBox::RootInlineBox):
(WebCore::RootInlineBox::alignBoxesInBlockDirection):
Sets the paginated line width from the current block logical height.

  • rendering/RootInlineBox.h:

(WebCore::RootInlineBox::paginationStrut):
(WebCore::RootInlineBox::setPaginationStrut):
Fixed paginationStrut to be LayoutUnit instead of int.

(WebCore::RootInlineBox::paginatedLineWidth):
(WebCore::RootInlineBox::setPaginatedLineWidth):
Added paginatedLineWidth getter/setter for caching the content width for a given root line.

LayoutTests:

  • platform/mac/fast/regions/webkit-flow-floats-inside-regions-bounds-expected.png:
  • platform/mac/fast/regions/webkit-flow-floats-inside-regions-bounds-expected.txt:
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-expected.png:
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-expected.txt:
1:37 PM Changeset in webkit [95749] by andersca@apple.com
  • 5 edits in trunk/Source/WebCore

Remove Widget::beforeMouseDown and Widget::afterMouseDown
​https://bugs.webkit.org/show_bug.cgi?id=68570

Reviewed by Darin Adler.

These two member functions were originally added to avoid crashes due to removing NSViews while they were
being tracked by AppKit. Since they were added, we've moved away from NSViews for form controls, and the bugs in
AppKit that lead to crashes have been fixed.

This patch was rolled out in r95743 because it broke a bunch of tests. The patch accidentally removed setting back
m_sendingEventToSubview to false.

  • page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::passMouseDownEventToWidget):

  • platform/Widget.h:
  • platform/mac/WidgetMac.mm:

(WebCore::Widget::Widget):
(WebCore::Widget::removeFromSuperview):

  • rendering/RenderWidget.cpp:

Change widgetHierarchyUpdateSuspendCount to be an unsigned integer instead of a size_t.

12:59 PM Changeset in webkit [95748] by rniwa@webkit.org
  • 48 edits in branches/chromium/874

Revert 93001 - Apple-style-span class seems unnecessary
​https://bugs.webkit.org/show_bug.cgi?id=12248

Reviewed by Justin Garcia.

Source/WebCore:

Stop generating span or font elements with class="Apple-style-span" given WebKit's editing component
no longer depends on Apple-style-span since r92823 removed the dependency of copy and paste code on
style spans. WebKit continues to recognize Apple style spans to remove them.

Also renamed isSpanWithoutAttributesOrUnstyleStyleSpan to isSpanWithoutAttributesOrUnstyle*d*StyleSpan.

  • editing/ApplyStyleCommand.cpp:

(WebCore::isLegacyAppleStyleSpan): Renamed from isStyleSpan.
(WebCore::isSpanWithoutAttributesOrUnstyledStyleSpan): Renamed from isSpanWithoutAttributesOr*Unstyle*StyleSpan.
(WebCore::createFontElement): No longer adds class="Apple-style-span".
(WebCore::createStyleSpanElement): Ditto.
(WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): Calls isSpanWithoutAttributesOrUnstyleStyleSpan
instead of isUnstyledStyleSpan since there won't be any Apple style spans.
(WebCore::dummySpanAncestorForNode):
(WebCore::ApplyStyleCommand::cleanupUnstyledAppleStyleSpans): Ditto.
(WebCore::ApplyStyleCommand::removeEmbeddingUpToEnclosingBlock): Ditto.
(WebCore::ApplyStyleCommand::removeImplicitlyStyledElement):
(WebCore::ApplyStyleCommand::removeCSSStyle):

  • editing/ApplyStyleCommand.h:
  • editing/EditingStyle.cpp:

(WebCore::EditingStyle::removeStyleFromRulesAndContext): Ditto.

  • editing/ReplaceSelectionCommand.cpp:

(WebCore::ReplaceSelectionCommand::removeRedundantStylesAndKeepStyleSpanInline): Calls isLegacyAppleStyleSpan instead
of isStyleSpan.
(WebCore::handleStyleSpansBeforeInsertion): Ditto.
(WebCore::ReplaceSelectionCommand::handleStyleSpans): Ditto.
(WebCore::ReplaceSelectionCommand::doApply): Ditto.

  • editing/markup.cpp:

(WebCore::StyledMarkupAccumulator::appendStyleNodeOpenTag): No longer adds class="Apple-style-span".

LayoutTests:

Rebaselined the tests because spans in these tests lost no longer have class="Apple-style-span".

  • editing/deleting/delete-br-012-expected.txt:
  • editing/deleting/delete-line-break-between-paragraphs-with-same-style-expected.txt:
  • editing/deleting/delete-select-all-001-expected.txt:
  • editing/execCommand/16049-expected.txt:
  • editing/execCommand/5685604-1.html:
  • editing/execCommand/delete-image-in-anchor-expected.txt:
  • editing/execCommand/hilitecolor-expected.txt:
  • editing/execCommand/modifyForeColorByCharacter-expected.txt:
  • editing/execCommand/query-font-size-expected.txt:
  • editing/execCommand/remove-format-multiple-elements-expected.txt:
  • editing/execCommand/script-tests/remove-format-multiple-elements.js:
  • editing/execCommand/script-tests/toggle-text-decorations.js:
  • editing/execCommand/script-tests/toggle-unlink.js:
  • editing/execCommand/toggle-style-2-expected.txt:
  • editing/execCommand/toggle-text-decorations-expected.txt:
  • editing/execCommand/toggle-unlink-expected.txt:
  • editing/inserting/5994480-2-expected.txt:
  • editing/pasteboard/5065605-expected.txt:
  • editing/pasteboard/copy-null-characters-expected.txt:
  • editing/pasteboard/copy-text-with-backgroundcolor-expected.txt:
  • editing/pasteboard/data-transfer-items-expected.txt:
  • editing/pasteboard/do-not-copy-unnecessary-styles-2-expected.txt:
  • editing/pasteboard/do-not-copy-unnecessary-styles-expected.txt:
  • editing/pasteboard/onpaste-text-html-expected.txt:
  • editing/pasteboard/paste-text-with-style-2-expected.txt:
  • editing/style/apply-font-size-to-multiple-nodes-expected.txt:
  • editing/style/block-style-004-expected.txt:
  • editing/style/block-style-005-expected.txt:
  • editing/style/block-style-006-expected.txt:
  • editing/style/fontsize-1-expected.txt:
  • editing/style/fore-color-by-name-expected.txt:
  • editing/style/highlight-insert-paragraph-expected.txt:
  • editing/style/inline-style-container-expected.txt:
  • editing/style/inline-style-extend-run-expected.txt:
  • editing/style/invalid-font-size-expected.txt:
  • editing/style/push-down-font-styles-expected.txt:
  • editing/style/script-tests/inline-style-container.js:
  • editing/style/script-tests/inline-style-extend-run.js:
  • editing/style/script-tests/push-down-font-styles.js:
  • editing/style/style-text-node-without-editable-parent-expected.txt:
  • fast/events/ondrop-text-html-expected.txt:
  • platform/chromium-win/editing/inserting/5994480-2-expected.txt:
  • platform/gtk/editing/inserting/5994480-2-expected.txt:

TBR=​rniwa@webkit.org
Review URL: ​http://codereview.chromium.org/7973008

12:41 PM Changeset in webkit [95747] by ap@apple.com
  • 8 edits in trunk/Source/WebKit2

[WK2] UIProcess should check that WebProcess isn't sending unexpected file: URLs to it
​https://bugs.webkit.org/show_bug.cgi?id=68573

Reviewed by Anders Carlsson.

Re-landing with a slightly less aggressive check.

12:38 PM Changeset in webkit [95746] by robert@webkit.org
  • 1 edit
    3 adds in trunk/LayoutTests

Platform-specific results for test introduced by r95726

  • platform/gtk/fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks-expected.txt: Added.
  • platform/mac/fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks-expected.txt: Added.
  • platform/qt/fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks-expected.txt: Added.
12:25 PM Changeset in webkit [95745] by rniwa@webkit.org
  • 10 edits
    17 deletes in trunk

Unreviewed, rolling out r95335 and r95645.
​http://trac.webkit.org/changeset/95335
​http://trac.webkit.org/changeset/95645
​https://bugs.webkit.org/show_bug.cgi?id=68649

Wrong fix (Requested by rniwa on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-09-22

Source/WebCore:

  • editing/DeleteSelectionCommand.cpp:

(WebCore::DeleteSelectionCommand::saveTypingStyleState):

  • editing/EditingStyle.cpp:

(WebCore::EditingStyle::removeStyleAddedByNode):

  • editing/EditingStyle.h:
  • editing/ReplaceSelectionCommand.cpp:

(WebCore::ReplaceSelectionCommand::removeRedundantStylesAndKeepStyleSpanInline):

  • editing/markup.cpp:

(WebCore::StyledMarkupAccumulator::serializeNodes):
(WebCore::ancestorToRetainStructureAndAppearance):

  • editing/markup.h:

LayoutTests:

  • editing/deleting/merge-paragraph-from-address-expected.txt: Removed.
  • editing/deleting/merge-paragraph-from-address.html: Removed.
  • editing/deleting/merge-paragraph-from-h6-expected.txt: Removed.
  • editing/deleting/merge-paragraph-from-h6-with-style-expected.txt: Removed.
  • editing/deleting/merge-paragraph-from-h6-with-style.html: Removed.
  • editing/deleting/merge-paragraph-from-h6.html: Removed.
  • editing/deleting/merge-paragraph-from-listing-expected.txt: Removed.
  • editing/deleting/merge-paragraph-from-listing.html: Removed.
  • editing/deleting/merge-paragraph-from-p-with-style-expected.txt: Removed.
  • editing/deleting/merge-paragraph-from-p-with-style.html: Removed.
  • editing/deleting/merge-paragraph-into-h1-expected.txt: Removed.
  • editing/deleting/merge-paragraph-into-h1-style-expected.txt: Removed.
  • editing/deleting/merge-paragraph-into-h1-with-style-expected.txt: Removed.
  • editing/deleting/merge-paragraph-into-h1-with-style.html: Removed.
  • editing/deleting/merge-paragraph-into-h1.html: Removed.
  • editing/deleting/merge-paragraph-into-pre-expected.txt: Removed.
  • editing/deleting/merge-paragraph-into-pre.html: Removed.
  • platform/mac/editing/deleting/merge-whitespace-pre-expected.png:
  • platform/mac/editing/deleting/merge-whitespace-pre-expected.txt:
12:14 PM Changeset in webkit [95744] by robert@webkit.org
  • 7 edits in trunk/LayoutTests

Unreviewed, update expected results after r95726

  • platform/chromium-win/css2.1/20110323/abspos-non-replaced-width-margin-000-expected.png:
  • platform/chromium-win/css2.1/20110323/abspos-replaced-width-margin-000-expected.png:
  • platform/gtk/css2.1/20110323/abspos-non-replaced-width-margin-000-expected.txt:
  • platform/gtk/css2.1/20110323/abspos-replaced-width-margin-000-expected.txt:
  • platform/qt/css2.1/20110323/abspos-non-replaced-width-margin-000-expected.txt:
  • platform/qt/css2.1/20110323/abspos-replaced-width-margin-000-expected.txt:
12:12 PM QtWebKitSecurity edited by Ademar Reis
(diff)
12:11 PM QtWebKitSecurity edited by Ademar Reis
(diff)
12:11 PM QtWebKitSecurity edited by Ademar Reis
(diff)
11:55 AM Changeset in webkit [95743] by rniwa@webkit.org
  • 5 edits in trunk/Source/WebCore

Revert r95673 since it caused hundreds of tests to fail on Leopard / Snow Leopard.

  • page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::passMouseDownEventToWidget):

  • platform/Widget.h:
  • platform/mac/WidgetMac.mm:

(WebCore::Widget::Widget):
(WebCore::Widget::removeFromSuperview):
(WebCore::Widget::beforeMouseDown):
(WebCore::Widget::afterMouseDown):

  • rendering/RenderWidget.cpp:
11:51 AM Changeset in webkit [95742] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Implement get_scoped_var in the DFG
​https://bugs.webkit.org/show_bug.cgi?id=68640

Reviewed by Gavin Barraclough.

Naive implementation of get_scoped_var in the DFG. Essentially this
is the bare minimum required to get correct behaviour, so there's no
load/store coalescing or type profiling involved, even though these
would be wins. No impact on SunSpider or V8.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGNode.h:

(JSC::DFG::Node::hasVarNumber):
(JSC::DFG::Node::hasScopeChainDepth):
(JSC::DFG::Node::scopeChainDepth):

  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

11:31 AM Changeset in webkit [95741] by hyatt@apple.com
  • 1 edit
    24 adds
    11 deletes in trunk/LayoutTests

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

Fix the layout test fast/regions directory. Remove platform-specific results that were
incorrectly checked in to cross-platform locations and add missing pixel test results.

Reviewed by Beth Dakin.

  • fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt: Removed.
  • fast/regions/flows-dependency-dynamic-remove-expected.txt: Removed.
  • fast/regions/flows-dependency-same-flow-expected.txt: Removed.
  • fast/regions/render-region-renderer-expected.txt: Removed.
  • fast/regions/webkit-flow-floats-inside-regions-bounds-expected.txt: Removed.
  • fast/regions/webkit-flow-inlines-inside-regions-bounds-expected.txt: Removed.
  • fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-expected.txt: Removed.
  • fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-rl-expected.txt: Removed.
  • fast/regions/webkit-flow-renderer-expected.txt: Removed.
  • fast/regions/webkit-flow-renderer-layer-expected.txt: Removed.
  • fast/regions/webkit-flow-renderer-nested-expected.txt: Removed.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-added-expected.png: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-inserted-expected.txt: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-dynamically-removed-expected.png: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-expected.png: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-with-dyn-index-expected.png: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-with-index-dom-expected.png: Added.
  • platform/mac/fast/regions/content-flowed-into-regions-with-index-expected.png: Added.
  • platform/mac/fast/regions/flows-dependency-dynamic-remove-expected.png: Added.
  • platform/mac/fast/regions/flows-dependency-dynamic-remove-expected.txt: Added.
  • platform/mac/fast/regions/flows-dependency-same-flow-expected.png: Added.
  • platform/mac/fast/regions/flows-dependency-same-flow-expected.txt: Added.
  • platform/mac/fast/regions/no-split-line-box-expected.png: Added.
  • platform/mac/fast/regions/render-region-renderer-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-floats-inside-regions-bounds-expected.png: Added.
  • platform/mac/fast/regions/webkit-flow-floats-inside-regions-bounds-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-expected.png: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-expected.png: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-rl-expected.png: Added.
  • platform/mac/fast/regions/webkit-flow-inlines-inside-regions-bounds-vertical-rl-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-renderer-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-renderer-layer-expected.txt: Added.
  • platform/mac/fast/regions/webkit-flow-renderer-nested-expected.txt: Added.
11:12 AM Changeset in webkit [95740] by hyatt@apple.com
  • 3 edits in trunk/Source/WebCore

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

Make RenderFlowThread cache whether or not it has regions of varying widths. This will
be relevant for performance as we begin adding code to do custom block painting and
layout based off regions not having the same width.

Reviewed by Dan Bernstein and Adam Roben.

  • rendering/RenderFlowThread.cpp:

(WebCore::RenderFlowThread::RenderFlowThread):
(WebCore::RenderFlowThread::layout):

  • rendering/RenderFlowThread.h:
11:03 AM Changeset in webkit [95739] by nduca@chromium.org
  • 2 edits in branches/chromium/874/Source/WebCore

Merge 94587 - [chromium] REGRESSION(94353): requestAnimationFrame not throttled in compositing path
​https://bugs.webkit.org/show_bug.cgi?id=67621

CCSingleThreadProxy should not perform layout when called
via the compositeImmediately path. Doing so makes it look
like frame rate is unbounded.

Reviewed by James Robinson.

  • platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:

(WebCore::CCSingleThreadProxy::compositeAndReadback):
(WebCore::CCSingleThreadProxy::compositeImmediately):
(WebCore::CCSingleThreadProxy::commitIfNeeded):

  • platform/graphics/chromium/cc/CCSingleThreadProxy.h:

TBR=​nduca@chromium.org
Review URL: ​http://codereview.chromium.org/7995010

11:02 AM QtWebKitReleases edited by Ademar Reis
(diff)
11:02 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
11:01 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
11:01 AM Changeset in webkit [95738] by andersca@apple.com
  • 3 edits in trunk/Source/WebCore

FrameView::invalidateRect and FrameView::setFrameRect shouldn't take LayoutRects
​https://bugs.webkit.org/show_bug.cgi?id=68639

Reviewed by David Hyatt.

FrameView::invalidateRect and FrameView::setFrameRect override Widget and ScrollView
member functions that take IntRects, and Widget coordinates are currently defined in terms of
IntRects (and probably always should since we want them to be pixel-aligned).

  • page/FrameView.cpp:

(WebCore::FrameView::invalidateRect):
(WebCore::FrameView::setFrameRect):

  • page/FrameView.h:
10:56 AM QtWebKitPackaging edited by Ademar Reis
(diff)
10:54 AM QtWebKitReleases edited by Ademar Reis
(diff)
10:54 AM QtWebKitRelease21 edited by Ademar Reis
(diff)
10:53 AM QtWebKitRelease21 edited by Ademar Reis
(diff)
10:52 AM Changeset in webkit [95737] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix (fixes warning about too many parens).

  • rendering/RenderBox.cpp:

(WebCore::RenderBox::computePositionedLogicalWidthReplaced):

10:51 AM Changeset in webkit [95736] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Widget::frameRect shouldn't be virtual
​https://bugs.webkit.org/show_bug.cgi?id=68637

Reviewed by Sam Weinig.

Nobody overrides Widget::frameRect and nobody should.

  • platform/Widget.h:
10:48 AM Changeset in webkit [95735] by Adam Roben
  • 1 edit
    1 delete in trunk/Tools

Remove FindSafari

It isn't used or built anymore.

Fixes <​http://webkit.org/b/68628>

Reviewed by Darin Adler.

  • FindSafari: Removed this directory and its contents.
10:48 AM Changeset in webkit [95734] by Adam Roben
  • 7 edits in trunk

Remove FindSafari from all our .sln files

It isn't used anymore, so there's no point in building it.

Part of <​http://webkit.org/b/68628> Remove FindSafari

Reviewed by Steve Falkenburg.

Source/JavaScriptCore:

Source/WebKit/win:

  • WebKit.vcproj/WebKit.sln: Also relinearized the build order while I was at it.

Tools:

  • DumpRenderTree/DumpRenderTree.sln:
  • WebKitTestRunner/WebKitTestRunner.sln:
10:47 AM Changeset in webkit [95733] by Adam Roben
  • 3 edits
    1 delete in trunk/Tools

Remove all uses of FindSafari

It isn't really needed anymore.

Fixes <​http://webkit.org/b/68626>.

Reviewed by Darin Adler.

  • Scripts/old-run-webkit-tests: Moved read/writeRegistryString from here to webkitdirs.pm.
  • Scripts/run-webkit-nightly.cmd: Removed. This used to be used to launch nightly builds of

WebKit, but now we use WebKit.exe for that.

  • Scripts/webkitdirs.pm:

(installedSafariPath): Changed to use readRegistryString instead of invoking FindSafari to
do the same thing.

(readRegistryString):
(writeRegistryString):
Moved here from old-run-webkit-tests.

10:47 AM Changeset in webkit [95732] by Adam Roben
  • 2 edits in trunk/Tools

Use the "direct object" form of system() to run WebKit.exe

This prevents the shell from parsing (and possibly splitting, if it contains spaces) the
path to WebKit.exe.

Fixes <​http://webkit.org/b/68623> run-safari doesn't work if there are spaces in the path to
%WEBKITOUTPUTDIR%

Reviewed by Steve Falkenburg.

  • Scripts/webkitdirs.pm:

(runSafari):

10:46 AM Changeset in webkit [95731] by Adam Roben
  • 2 edits in trunk/Tools

Quote the WEBKITLIBRARIESDIR path before passing it to the shell/cygpath

Fixes <​http://webkit.org/b/68621> build-webkit goes totally haywire on Windows if there are
spaces in the path to the WebKit source tree

Reviewed by John Sullivan.

  • Scripts/webkitdirs.pm:

(setupCygwinEnv):

10:46 AM Changeset in webkit [95730] by Adam Roben
  • 2 edits in trunk/Source/WebKit/win

Add testRegExp.vcproj to WebKit.sln

Fixes <​http://webkit.org/b/68403> Windows bots don't build testRegExp.vcproj

Reviewed by Brian Weinstein.

  • WebKit.vcproj/WebKit.sln: Added testRegExp. Made it build just after jsc, just like it

does when built as part of JavaScriptCore.sln.

10:40 AM Changeset in webkit [95729] by vestbo@webkit.org
  • 23 edits in trunk

[Qt] Use same DPI for application font as rest of app in DRT and WTR

QApplication will initialize the default application font based
on the application DPI at construction time, but we then override
the application DPI using QX11Info (hard-coding it to 96 for
consistency). This hard-coding is not reflected in the application
font, so we explicitly have update the font ourselves.

The 6 test results that are updated were originally produced with
a DPI of 75, as this is the default fallback DPI when a QFont is
constructed before QApplication. This was wrong, and the results
are updated to match a DPI of 96.

Reviewed by Csaba Osztrogonác.

10:36 AM Changeset in webkit [95728] by commit-queue@webkit.org
  • 6 edits
    2 adds in trunk

Ref protect shaders in V8WebGLRenderingContext::getAttachedShadersCallback
​https://bugs.webkit.org/show_bug.cgi?id=68630

Patch by Sergey Glazunov <​serg.glazunov@gmail.com> on 2011-09-22
Reviewed by Adam Barth.

Source/WebCore:

Test: fast/canvas/webgl/shader-deleted-by-accessor.html

  • bindings/js/JSWebGLRenderingContextCustom.cpp:

(WebCore::JSWebGLRenderingContext::getAttachedShaders):

  • bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:

(WebCore::V8WebGLRenderingContext::getAttachedShadersCallback):

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::getAttachedShaders):

  • html/canvas/WebGLRenderingContext.h:

LayoutTests:

  • fast/canvas/webgl/shader-deleted-by-accessor-expected.txt: Added.
  • fast/canvas/webgl/shader-deleted-by-accessor.html: Added.
10:30 AM Changeset in webkit [95727] by robert@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

Unreviewed, platform-specific results for r95721.

  • platform/mac/fast/css/outline-narrowLine-expected.txt: Added.
  • platform/gtk/fast/css/outline-narrowLine-expected.txt: Added.
10:16 AM Changeset in webkit [95726] by robert@webkit.org
  • 9 edits
    11 moves
    3 adds
    11 deletes in trunk

CSS 2.1 failure: abspos-non-replaced-width-margin-000, abspos-replaced-width-margin-000
​https://bugs.webkit.org/show_bug.cgi?id=47148

Reviewed by David Hyatt.

Source/WebCore:

  • rendering/RenderBlock.h:
  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::startOffsetForLine): Return offset from right, rather than offset from left + logicalwidth

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::startAlignedOffsetForLine):

Return the correct alignment for RTL blocks too. Looks like neither FF or Opera do this correctly.
Covered by new test left-right-center-inline-alignment-in-ltr-and-rtl-blocks.html

  • rendering/RenderBox.cpp:

(WebCore::computeInlineStaticDistance):

  • The CSS 2.1 tests now require UAs to use the parent block's direction (rather than the containing block's direction) when computing static distance. The container block's direction is still used in the other width and offset related calculations.
  • In the context of this function, the logicalRight value is just an element that will be used later to calculate the correct logicalLeft position for the RTL block. So whereas an LTR block can substract containerBlock->borderLogicalLeft() in this function directly, an RTL block has to add it here so that will be later subtracted from availableSpace to get logicalLeft in computePositionedLogicalWidthUsing().

(WebCore::RenderBox::computePositionedLogicalWidth):

  • Update comments and use the container block's direction except when calculating the static distance.
  • Remove the notion of a quirks mode in this function and for calculating the position of replaced and non-replaced blocks generally. This is in line with both FF4 and Opera - neither of which have a quirks mode for them.

(WebCore::RenderBox::computePositionedLogicalWidthUsing):

  • Update comments and use the container block's direction except when calculating the static distance.

(WebCore::RenderBox::computePositionedLogicalHeight):

  • Update comments.

(WebCore::RenderBox::computePositionedLogicalWidthReplaced):

  • Update comments and use the container block's direction except when calculating the static distance.
  • If the containing block is RTL, always assume that values are over-constrained.

(WebCore::RenderBox::computePositionedLogicalHeightReplaced):

  • Update comments.

LayoutTests:

  • platform/chromium-linux/css2.1/20110323/abspos-non-replaced-width-margin-000-expected.png:
  • platform/chromium-linux/css2.1/20110323/abspos-replaced-width-margin-000-expected.png: New results for CSS 2.1 reference tests. These match exactly the reference test results.
  • fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks.html: Added.
  • platform/chromium-linux/fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks-expected.png: Added.
  • platform/chromium-linux/fast/inline/left-right-center-inline-alignment-in-ltr-and-rtl-blocks-expected.txt: Added. New test for left,right and center alignment of inline LTR and RTL blocks. FF and Opera don't pass this test.
  • fast/block/positioning/absolute-position-direction-strict.html: Removed.
  • fast/block/positioning/absolute-position-direction.html: Renamed from LayoutTests/fast/block/positioning/absolute-position-direction-quirk.html.
  • platform/chromium-linux/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/chromium-linux/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/chromium-linux/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/chromium-win/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/chromium-win/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/chromium-win/fast/block/positioning/absolute-position-direction-expected.txt: Renamed from LayoutTests/platform/chromium-win/fast/block/positioning/absolute-position-direction-quirk-expected.txt.
  • platform/chromium-win/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/chromium-win/fast/block/positioning/absolute-position-direction-strict-expected.txt: Removed.
  • platform/gtk/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/gtk/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/gtk/fast/block/positioning/absolute-position-direction-expected.txt: Renamed from LayoutTests/platform/gtk/fast/block/positioning/absolute-position-direction-quirk-expected.txt.
  • platform/gtk/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/gtk/fast/block/positioning/absolute-position-direction-strict-expected.txt: Removed.
  • platform/mac-leopard/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/mac-leopard/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/mac-leopard/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/mac/css2.1/20110323/abspos-non-replaced-width-margin-000-expected.txt:
  • platform/mac/css2.1/20110323/abspos-replaced-width-margin-000-expected.txt:
  • platform/mac/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/mac/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/mac/fast/block/positioning/absolute-position-direction-expected.txt: Renamed from LayoutTests/platform/mac/fast/block/positioning/absolute-position-direction-quirk-expected.txt.
  • platform/mac/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/mac/fast/block/positioning/absolute-position-direction-strict-expected.txt: Removed.
  • platform/qt/fast/block/positioning/absolute-position-direction-expected.png: Renamed from LayoutTests/platform/qt/fast/block/positioning/absolute-position-direction-quirk-expected.png.
  • platform/qt/fast/block/positioning/absolute-position-direction-expected.txt: Renamed from LayoutTests/platform/qt/fast/block/positioning/absolute-position-direction-quirk-expected.txt.
  • platform/qt/fast/block/positioning/absolute-position-direction-strict-expected.png: Removed.
  • platform/qt/fast/block/positioning/absolute-position-direction-strict-expected.txt: Removed. These tests are superseded by abspos-non-replaced-width-margin-000.htm. Using the parent() block's direction to compute static distance is now a strict requirement for CSS 2.1, rather than an IE quirk.
10:09 AM Changeset in webkit [95725] by andersca@apple.com
  • 16 edits in trunk/Source

Get rid of Widget::setBoundsSize
​https://bugs.webkit.org/show_bug.cgi?id=68491

Reviewed by Sam Weinig.

This basically reverts ​http://trac.webkit.org/changeset/79167 and ​http://trac.webkit.org/changeset/75897
which were added to make zoom gestures work with WebKit1, but since we don't support zoom gestures in WebKit1
anymore, it's better for code cleanliness to get rid of Widget::setBoundsSize and the associated code. While this
will unfortunately break transformed iframes in WebKit1, but Simon says that it's an acceptable tradeoff.

Source/WebCore:

  • WebCore.exp.in:
  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::open):

  • page/FrameView.cpp:

(WebCore::FrameView::create):

  • platform/ScrollView.cpp:

(WebCore::ScrollView::visibleContentRect):
(WebCore::ScrollView::updateScrollbars):
(WebCore::ScrollView::wheelEvent):

  • platform/ScrollView.h:
  • platform/Widget.cpp:
  • platform/Widget.h:

(WebCore::Widget::resize):

  • platform/chromium/PopupContainer.cpp:

(WebCore::PopupContainer::refresh):

  • platform/gtk/ScrollViewGtk.cpp:

(WebCore::ScrollView::visibleContentRect):

  • platform/mac/WidgetMac.mm:

(WebCore::Widget::paint):

  • rendering/RenderWidget.cpp:

(WebCore::RenderWidget::setWidgetGeometry):
(WebCore::RenderWidget::setWidget):
(WebCore::RenderWidget::updateWidgetPosition):

  • rendering/RenderWidget.h:

Source/WebKit/mac:

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView updateAndSetWindow]):

  • WebView/WebFrameView.mm:
9:33 AM Changeset in webkit [95724] by hyatt@apple.com
  • 5 edits in trunk

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

Floats pushed to next page, column or region don't reposition properly if the amount of
available logical width at the new position changes. Refactor the code so that we can
run the float placement algorithm again when this happens.

Source/WebCore:

Covered by an existing regions test that exposes the issue.

Reviewed by Adam Roben.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::computeLogicalLocationForFloat):
(WebCore::RenderBlock::positionNewFloats):

  • rendering/RenderBlock.h:

LayoutTests:

Reviewed by Adam Roben.

  • fast/regions/webkit-flow-floats-inside-regions-bounds-expected.txt:
9:31 AM Changeset in webkit [95723] by Csaba Osztrogonác
  • 2 edits
    1532 adds in trunk/LayoutTests

[Qt] Unreviewed. Add Qt specific expected results for css3/selectors3 tests.
All of them pass, they are compared to mac png results manually.

  • platform/qt/Skipped:
  • platform/qt/css3/selectors3/ [...]: Added.
9:18 AM Changeset in webkit [95722] by leandrogracia@chromium.org
  • 5 edits
    2 adds in trunk

Source/WebCore: Fire TextInput events on speech input, but not set/add any inputMethod attribute.
​https://bugs.webkit.org/show_bug.cgi?id=60451

Reviewed by Ryosuke Niwa.

Introduce code to dispatch TextInput events when text is introduced via
the speech input feature. Handle also the cases where speech input is
started programmatically from the LayoutTestController.

Test: fast/speech/input-ontextinput-event.html

  • dom/TextEventInputType.h:
  • html/shadow/TextControlInnerElements.cpp:

(WebCore::InputFieldSpeechButtonElement::setRecognitionResult):

LayoutTests: Testing text events triggered by speech input.
​https://bugs.webkit.org/show_bug.cgi?id=60451

Reviewed by Ryosuke Niwa.

  • fast/speech/input-ontextinput-event-expected.txt: Added.
  • fast/speech/input-ontextinput-event.html: Added.
  • fast/speech/input-text-speechstart.html:
9:14 AM Changeset in webkit [95721] by robert@webkit.org
  • 3 edits
    5 adds in trunk

Patch by Jason Liu <​jason.liu@torchmobile.com.cn> on 2011-09-20
Reviewed by Simon Fraser.

CSS outline property on a narrow element has rabbit ear artifacts
​https://bugs.webkit.org/show_bug.cgi?id=36793

Patch by Jason Liu <​jason.liu@torchmobile.com.cn>
Patch updated by Robert Hogan <​robert@webkit.org>

Source/WebCore:

Test: fast/css/outline-narrowLine.html

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::paintOutlineForLine):

LayoutTests:

  • fast/css/outline-narrowLine.html: Added.
  • platform/qt/fast/css/outline-narrowLine-expected.txt: Added.
  • platform/qt/fast/css/outline-narrowLine-expected.png: Added.
  • platform/chromium-linux/fast/css/outline-narrowLine-expected.txt: Added.
  • platform/chromium-linux/fast/css/outline-narrowLine-expected.png: Added.
9:08 AM Changeset in webkit [95720] by Martin Robinson
  • 2 edits in trunk/Source/WebCore

[GTK]Popup drop-down menu contains extra empty spaces at beginning and
occupies whole screen when items in popup reaches the height of display device.
​https://bugs.webkit.org/show_bug.cgi?id=68533

When implementing the GTK+ menu position function, set pushIn to false, which tell
GTK+ not create a menu that is as big as the entire screen.

Patch by Wajahat Siddiqui <​mdwajahatali.siddiqui@motorola.com> on 2011-09-22
Reviewed by Martin Robinson.

  • platform/gtk/GtkPopupMenu.cpp:

(WebCore::GtkPopupMenu::menuPositionFunction): Set pushIn to false in the menu position function.

8:42 AM Changeset in webkit [95719] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-09-22

  • DEPS:
7:48 AM Changeset in webkit [95718] by bulach@chromium.org
  • 1 edit
    4 adds in trunk/LayoutTests

[chromium] Rebaseline following r95715
​https://bugs.webkit.org/show_bug.cgi?id=68619

Unreviewed, rebaseline.

  • platform/chromium-cg-mac/fast/events/scale-and-scroll-iframe-body-expected.png: Added.
  • platform/chromium-cg-mac/fast/events/scale-and-scroll-iframe-window-expected.png: Added.
  • platform/chromium-mac/fast/events/scale-and-scroll-iframe-body-expected.png: Added.
  • platform/chromium-mac/fast/events/scale-and-scroll-iframe-window-expected.png: Added.
6:54 AM Changeset in webkit [95717] by hclam@chromium.org
  • 2 edits in trunk/Source/WebCore

Define export symbol correctly due to rename of pageScaleFactor to
frameScaleFactor.

Not reviewed. Build fix.

  • WebCore.exp.in:
6:43 AM Changeset in webkit [95716] by vsevik@chromium.org
  • 4 edits in trunk/Source/WebCore

Web Inspector: Console is always scrolled to the top
​https://bugs.webkit.org/show_bug.cgi?id=67483

Reviewed by Pavel Feldman.

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype.wasShown):
(WebInspector.ConsoleView.prototype.storeScrollPositions):
(WebInspector.ConsoleView.prototype.restoreScrollPositions):
(WebInspector.ConsoleView.prototype.onResize):
(WebInspector.ConsoleView.prototype._immediatelyScrollIntoView):
(WebInspector.ConsoleView.prototype._cancelScheduledScrollIntoView):
(WebInspector.ConsoleView.prototype._consoleCleared):

  • inspector/front-end/Drawer.js:

(WebInspector.Drawer.prototype.hide):
(WebInspector.Drawer.prototype.resize):
(WebInspector.Drawer.prototype._startStatusBarDragging):
(WebInspector.Drawer.prototype._statusBarDragging):

  • inspector/front-end/inspector.js:

(WebInspector.animateStyle):
(WebInspector.animateStyle.forceComplete):

6:30 AM Changeset in webkit [95715] by hclam@chromium.org
  • 11 edits
    12 adds in trunk

Source/WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=68081
Fix scroll in page scaling mode.

Reviewed by Simon Fraser.

The following things are done in this patch:

  1. Rename Frame::pageScaleFactor() to Frame::frameScaleFactor() to better reflect its purpose.
  2. Frame::frameScaleFactor() returns the scale factor of this frame with respect to the container. So for the main frame it'll return the scale factor of the page, inner frames will return 1.0
  3. scrollBy(), scrollX() and scrollY() to take into account of the frame scale factor, causing incorrect coordinates reported to Javascript.

Tests: fast/events/scale-and-scroll-body.html

fast/events/scale-and-scroll-iframe-body.html
fast/events/scale-and-scroll-iframe-window.html
fast/events/scale-and-scroll-window.html

  • html/HTMLBodyElement.cpp:

(WebCore::adjustForZoom):
(WebCore::HTMLBodyElement::setScrollLeft):
(WebCore::HTMLBodyElement::setScrollTop):

  • loader/HistoryController.cpp:

(WebCore::HistoryController::saveScrollPositionAndViewStateToItem):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::scrollX):
(WebCore::DOMWindow::scrollY):
(WebCore::DOMWindow::scrollTo):

  • page/Frame.cpp:

(WebCore::Frame::frameScaleFactor): Return page's scale factor if this is

main frame, otherwise 1.0

  • page/Frame.h:
  • page/FrameView.cpp:

(WebCore::FrameView::applyOverflowToViewport):
(WebCore::FrameView::scrollXForFixedPosition):
(WebCore::FrameView::scrollYForFixedPosition):

Source/WebKit2: ​https://bugs.webkit.org/show_bug.cgi?id=68081
Fix scroll in page scaling mode.

Reviewed by Simon Fraser.

  • WebProcess/Plugins/PluginView.cpp: Rename pageScaleFactor to frameScaleFactor.

(WebKit::PluginView::handleEvent):
(WebKit::PluginView::viewGeometryDidChange):
(WebKit::PluginView::clipRectInWindowCoordinates):

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=68081
Add layout test for page scale and scrolling.

Reviewed by Simon Fraser.

Add tests to cover the following cases:

  1. Scale and scroll window
  2. Scale and scroll document.body
  3. Scale and scroll iframe's contentWindow
  4. Scale and scroll iframe's contentDocument.body
  • fast/events/scale-and-scroll-body-expected.png: Added.
  • fast/events/scale-and-scroll-body-expected.txt: Added.
  • fast/events/scale-and-scroll-body.html: Added.
  • fast/events/scale-and-scroll-iframe-body-expected.png: Added.
  • fast/events/scale-and-scroll-iframe-body-expected.txt: Added.
  • fast/events/scale-and-scroll-iframe-body.html: Added.
  • fast/events/scale-and-scroll-iframe-window-expected.png: Added.
  • fast/events/scale-and-scroll-iframe-window-expected.txt: Added.
  • fast/events/scale-and-scroll-iframe-window.html: Added.
  • fast/events/scale-and-scroll-window-expected.png: Added.
  • fast/events/scale-and-scroll-window-expected.txt: Added.
  • fast/events/scale-and-scroll-window.html: Added.
6:25 AM Changeset in webkit [95714] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

CSS: Implement 'font' property in CSSComputedStyle.
​https://bugs.webkit.org/show_bug.cgi?id=66666

Patch by Andreas Kling <​kling@webkit.org> on 2011-09-22
Reviewed by Antti Koivisto.

Source/WebCore:

Test: fast/css/getComputedStyle/computed-style-font.html

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::fontFamilyFromStyle): Factored out of getPropertyCSSValue().
This function will always return a CSSValueList, as opposed to the previous
implementation which would return a CSSPrimitiveValue if there was only one family.
getCSSPropertyValue() will return the first element of the list for 1-element lists
to preserve the current behavior of getPropertyCSSValue('font-family').

(WebCore::lineHeightFromStyle): Factored out of getPropertyCSSValue().
(WebCore::fontSizeFromStyle): Ditto.
(WebCore::fontStyleFromStyle): Ditto.
(WebCore::fontVariantFromStyle): Ditto.
(WebCore::fontWeightFromStyle): Ditto.

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added
implementation of the computed 'font' property (CSSPropertyFont.)

LayoutTests:

  • fast/css/getComputedStyle/computed-style-font-expected.txt: Added.
  • fast/css/getComputedStyle/computed-style-font.html: Added.
  • fast/css/getComputedStyle/script-tests/computed-style-font.js: Added.

(computedFont):
(computedFontCSSValue):

6:05 AM Changeset in webkit [95713] by kov@webkit.org
  • 2 edits in trunk/Source/WebKit/gtk

Remove useless empty WebKit namespace.

Patch by Gustavo Noronha Silva <​gustavo.noronha@collabora.com> on 2011-09-22
Reviewed by Xan Lopez.

  • webkit/webkitglobals.cpp:

(webkitInit):

6:01 AM Changeset in webkit [95712] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Unreivewed gardening.

  • platform/qt-wk2/Skipped: Skip new failing test introduced in r95645.
5:50 AM Changeset in webkit [95711] by Csaba Osztrogonác
  • 64 edits in trunk/LayoutTests

[Qt] 69 tests failing after ​http://trac.webkit.org/changeset/92375
​https://bugs.webkit.org/show_bug.cgi?id=65721

Unreviewed gardening. Update Qt 4.8 specific expected files after r95706.

  • platform/qt-4.8/ [...]: Updated.
5:43 AM Changeset in webkit [95710] by jesus@webkit.org
  • 4 edits in trunk/Source

[Qt] Redirection overflow errors have wrong error domain
​https://bugs.webkit.org/show_bug.cgi?id=68612

Reviewed by Andreas Kling.

Source/WebCore:

Redirection overflow currently puts the URL hostname in the errorDomain field.
Now redirection overflow errors will be classified as "HTTP" errors.

  • platform/network/qt/QNetworkReplyHandler.cpp:

(WebCore::QNetworkReplyHandler::redirect):

Source/WebKit2:

Redirection overflow currently puts the URL hostname in the errorDomain field
and therefore are treated as a QWebError::EngineError. From now on they
will be treated as QWebError::HttpError, since their error domain was modified
to "HTTP" in QNetworkReplyHandler::redirect().

  • UIProcess/API/qt/qweberror.cpp:

(QWebError::type):

5:23 AM Changeset in webkit [95709] by Csaba Osztrogonác
  • 1 edit
    2 adds in trunk/LayoutTests

[Qt] Unreviewed gardening. Add Qt specific expected files for new test introduced in r95671.

  • platform/qt/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/qt/fast/table/quote-text-around-iframe-expected.txt: Added.
4:50 AM Changeset in webkit [95708] by Csaba Osztrogonác
  • 8 edits in trunk/Source/WebKit2

REGRESSION(r95679): It broke everything.
​https://bugs.webkit.org/show_bug.cgi?id=68611

Unreviewed rolling out r95679 and r95690, because it broke everything.

  • UIProcess/API/mac/WKView.mm:

(maybeCreateSandboxExtensionFromPasteboard):
(-[WKView performDragOperation:]):

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::didPerformClientRedirect):
(WebKit::WebContext::didPerformServerRedirect):
(WebKit::WebContext::didUpdateHistoryTitle):
(WebKit::WebContext::getPluginPath):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::reattachToWebProcessWithItem):
(WebKit::WebPageProxy::initializeSandboxExtensionHandle):
(WebKit::WebPageProxy::loadURL):
(WebKit::WebPageProxy::loadURLRequest):
(WebKit::WebPageProxy::loadHTMLString):
(WebKit::WebPageProxy::loadAlternateHTMLString):
(WebKit::WebPageProxy::goForward):
(WebKit::WebPageProxy::goBack):
(WebKit::WebPageProxy::goToBackForwardItem):
(WebKit::WebPageProxy::didStartProvisionalLoadForFrame):
(WebKit::WebPageProxy::didReceiveServerRedirectForProvisionalLoadForFrame):
(WebKit::WebPageProxy::didSameDocumentNavigationForFrame):
(WebKit::WebPageProxy::decidePolicyForNavigationAction):
(WebKit::WebPageProxy::decidePolicyForNewWindowAction):
(WebKit::WebPageProxy::decidePolicyForResponse):
(WebKit::WebPageProxy::didInitiateLoadForResource):
(WebKit::WebPageProxy::didSendRequestForResource):
(WebKit::WebPageProxy::didReceiveResponseForResource):
(WebKit::WebPageProxy::missingPluginButtonClicked):

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

(WebKit::WebProcessProxy::WebProcessProxy):
(WebKit::WebProcessProxy::addBackForwardItem):

  • UIProcess/WebProcessProxy.h:
  • UIProcess/cf/WebPageProxyCF.cpp:

(WebKit::WebPageProxy::restoreFromSessionStateData):

4:05 AM Changeset in webkit [95707] by Csaba Osztrogonác
  • 2 edits in trunk/Source/JavaScriptCore

32-bit call code clobbers the function cell tag
​https://bugs.webkit.org/show_bug.cgi?id=68606

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-09-22
Reviewed by Csaba Osztrogonác.

This is a minimalistic fix: it simply emits code to restore the
cell tag on the slow path, if we know that we failed due to
emitCallIfNotType.

  • jit/JITCall32_64.cpp:

(JSC::JIT::compileOpCallVarargsSlowCase):
(JSC::JIT::compileOpCallSlowCase):

3:23 AM Changeset in webkit [95706] by Csaba Osztrogonác
  • 134 edits
    2 adds in trunk/LayoutTests

[Qt] 69 tests failing after ​http://trac.webkit.org/changeset/92375
​https://bugs.webkit.org/show_bug.cgi?id=65721

Unreviewed gardening. Update Qt specific expected files.

3:17 AM Changeset in webkit [95705] by bulach@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Update test expectation following r95685
​https://bugs.webkit.org/show_bug.cgi?id=68604

Unreviewed, update test expectation.

  • platform/chromium/test_expectations.txt:
3:05 AM Changeset in webkit [95704] by Csaba Osztrogonác
  • 13 edits
    1 add in trunk/LayoutTests

[Qt] 11 tests failed after r95671 because the tests are updated.
​https://bugs.webkit.org/show_bug.cgi?id=68596

Update Qt specific expected files.

  • platform/qt/fast/css-generated-content/table-cell-before-content-expected.png:
  • platform/qt/fast/css-generated-content/table-cell-before-content-expected.txt:
  • platform/qt/fast/table/table-cell-after-child-in-block-expected.txt:
  • platform/qt/fast/table/table-cell-before-after-content-around-table-block-expected.txt:
  • platform/qt/fast/table/table-cell-before-after-content-around-table-expected.txt:
  • platform/qt/fast/table/table-cell-before-child-in-block-expected.txt:
  • platform/qt/fast/table/table-row-after-child-in-block-expected.txt:
  • platform/qt/fast/table/table-row-before-after-content-around-block-expected.txt:
  • platform/qt/fast/table/table-row-before-after-content-around-table-expected.txt:
  • platform/qt/fast/table/table-row-before-child-in-block-expected.txt:
  • platform/qt/fast/table/table-row-style-not-updated-with-after-content-expected.png: Added.
  • platform/qt/fast/table/table-row-style-not-updated-with-after-content-expected.txt:
2:29 AM Changeset in webkit [95703] by loki@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] 11 tests failed after r95671 because the tests are updated.
​https://bugs.webkit.org/show_bug.cgi?id=68596

Unreviewed gardening. Typo fixes.

Patch by Oliver Varga <​voliver@inf.u-szeged.hu> on 2011-09-22

  • platform/qt/Skipped:
12:53 AM Changeset in webkit [95702] by jamesr@google.com
  • 15 edits in trunk/Source

Unreviewed, rolling out r95699.
​http://trac.webkit.org/changeset/95699
​https://bugs.webkit.org/show_bug.cgi?id=67417

Makes many chromium compositor tests crash

Source/WebCore:

  • platform/CrossThreadCopier.h:
  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::~LayerRendererChromium):

  • platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:

(WebCore::CCHeadsUpDisplay::enabled):

  • platform/graphics/chromium/cc/CCLayerImpl.cpp:

(WebCore::CCLayerImpl::CCLayerImpl):
(WebCore::CCLayerImpl::~CCLayerImpl):

  • platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::CCLayerTreeHost):
(WebCore::CCLayerTreeHost::commitTo):
(WebCore::CCLayerTreeHost::setNeedsRedraw):

  • platform/graphics/chromium/cc/CCLayerTreeHost.h:

(WebCore::CCSettings::CCSettings):

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::~CCLayerTreeHostImpl):

  • platform/graphics/chromium/cc/CCProxy.h:
  • platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:

(WebCore::CCSingleThreadProxy::finishAllRendering):
(WebCore::CCSingleThreadProxy::setNeedsCommit):
(WebCore::CCSingleThreadProxy::commitIfNeeded):

  • platform/graphics/chromium/cc/CCThreadProxy.cpp:

(WebCore::CCThreadProxy::CCThreadProxy):
(WebCore::CCThreadProxy::~CCThreadProxy):
(WebCore::CCThreadProxy::compositeAndReadback):
(WebCore::CCThreadProxy::finishAllRendering):
(WebCore::CCThreadProxy::isStarted):
(WebCore::CCThreadProxy::initializeLayerRenderer):
(WebCore::CCThreadProxy::setNeedsCommit):
(WebCore::CCThreadProxy::setNeedsCommitAndRedraw):
(WebCore::CCThreadProxy::setNeedsRedraw):
(WebCore::CCThreadProxy::start):
(WebCore::CCThreadProxy::stop):
(WebCore::CCThreadProxy::beginFrameAndCommitOnCCThread):
(WebCore::CCThreadProxy::beginFrameAndCommit):
(WebCore::CCThreadProxy::commitOnCCThread):
(WebCore::CCThreadProxy::drawLayersOnCCThread):
(WebCore::CCThreadProxy::setNeedsCommitOnCCThread):
(WebCore::CCThreadProxy::setNeedsCommitAndRedrawOnCCThread):
(WebCore::CCThreadProxy::setNeedsRedrawOnCCThread):

  • platform/graphics/chromium/cc/CCThreadProxy.h:

Source/WebKit/chromium:

  • tests/CCLayerTreeHostTest.cpp:

(WTF::CCLayerTreeHostTest::doBeginTest):
(WTF::TEST_F):

  • tests/TreeSynchronizerTest.cpp:

(WebCore::TEST):

12:40 AM Changeset in webkit [95701] by Philippe Normand
  • 11 edits
    2 adds in trunk/LayoutTests

Unreviewed, GTK rebaseline after r95671.

  • platform/gtk/fast/css-generated-content/table-cell-before-content-expected.txt:
  • platform/gtk/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/gtk/fast/table/quote-text-around-iframe-expected.txt: Added.
  • platform/gtk/fast/table/table-cell-after-child-in-block-expected.txt:
  • platform/gtk/fast/table/table-cell-before-after-content-around-table-block-expected.txt:
  • platform/gtk/fast/table/table-cell-before-after-content-around-table-expected.txt:
  • platform/gtk/fast/table/table-cell-before-child-in-block-expected.txt:
  • platform/gtk/fast/table/table-row-after-child-in-block-expected.txt:
  • platform/gtk/fast/table/table-row-before-after-content-around-block-expected.txt:
  • platform/gtk/fast/table/table-row-before-after-content-around-table-expected.txt:
  • platform/gtk/fast/table/table-row-before-child-in-block-expected.txt:
  • platform/gtk/fast/table/table-row-style-not-updated-with-after-content-expected.txt:
12:35 AM Changeset in webkit [95700] by reni@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] 11 tests failed after r95671.
​https://bugs.webkit.org/show_bug.cgi?id=68596

Unreviewed gardening.

Patch by Oliver Varga <​voliver@inf.u-szeged.hu> on 2011-09-22

  • platform/qt/Skipped:
12:22 AM Changeset in webkit [95699] by jamesr@google.com
  • 15 edits in trunk/Source

[chromium] Make CCThreadProxy draw
​https://bugs.webkit.org/show_bug.cgi?id=67417

Source/WebCore:

Update the CCThreadProxy to correctly implement the CCProxy
interface, do all the right committing and updating steps, and
draw a picture on the screen.

Patch by Nat Duca <​nduca@chromium.org> on 2011-09-22
Reviewed by James Robinson.

  • platform/graphics/IntRect.h:
  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::~LayerRendererChromium):

  • platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:

(WebCore::CCHeadsUpDisplay::enabled):

  • platform/graphics/chromium/cc/CCLayerImpl.cpp:

(WebCore::CCLayerImpl::CCLayerImpl):
(WebCore::CCLayerImpl::~CCLayerImpl):

  • platform/graphics/chromium/cc/CCLayerTreeHost.cpp:

(WebCore::CCLayerTreeHost::CCLayerTreeHost):
(WebCore::CCLayerTreeHost::commitTo):
(WebCore::CCLayerTreeHost::commitComplete):
(WebCore::CCLayerTreeHost::setNeedsRedraw):

  • platform/graphics/chromium/cc/CCLayerTreeHost.h:
  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::~CCLayerTreeHostImpl):

  • platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:

(WebCore::CCSingleThreadProxy::finishAllRendering):
(WebCore::CCSingleThreadProxy::setNeedsCommit):
(WebCore::CCSingleThreadProxy::commitIfNeeded):

  • platform/graphics/chromium/cc/CCThreadProxy.cpp:

(WebCore::CCThreadProxy::CCThreadProxy):
(WebCore::CCThreadProxy::~CCThreadProxy):
(WebCore::CCThreadProxy::compositeAndReadback):
(WebCore::CCThreadProxy::drawLayersAndReadbackOnCCThread):
(WebCore::CCThreadProxy::finishAllRendering):
(WebCore::CCThreadProxy::isStarted):
(WebCore::CCThreadProxy::setNeedsCommit):
(WebCore::CCThreadProxy::setNeedsCommitAndRedraw):
(WebCore::CCThreadProxy::setNeedsRedraw):
(WebCore::CCThreadProxy::start):
(WebCore::CCThreadProxy::stop):
(WebCore::CCThreadProxy::finishAllRenderingOnCCThread):
(WebCore::CCThreadProxy::createBeginFrameAndCommitTaskOnCCThread):
(WebCore::CCThreadProxy::beginFrameAndCommit):
(WebCore::CCThreadProxy::commitOnCCThread):
(WebCore::CCThreadProxy::scheduleDrawTaskOnCCThread):
(WebCore::CCThreadProxy::drawLayersAndPresentOnCCThread):
(WebCore::CCThreadProxy::drawLayersOnCCThread):
(WebCore::CCThreadProxy::updateSchedulerStateOnCCThread):

  • platform/graphics/chromium/cc/CCThreadProxy.h:

Source/WebKit/chromium:

Disable CCLayerTreeHostTest temporarily. Will re-enable
with ​https://bugs.webkit.org/show_bug.cgi?id=67418

Patch by Nat Duca <​nduca@chromium.org> on 2011-09-22
Reviewed by James Robinson.

  • tests/CCLayerTreeHostTest.cpp:

Sep 21, 2011:

9:06 PM Changeset in webkit [95698] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit/chromium

Implement WebKit side of IDBFactory::getDatabaseNames
​https://bugs.webkit.org/show_bug.cgi?id=68037

Patch by Joshua Bell <​jsbell@chromium.org> on 2011-09-21
Reviewed by Tony Chang.

Interface changes and stub implementations for
new IndexedDB IDBFactory.getDatabaseNames. This is
part one of a two-sided patch.

  • public/WebIDBCallbacks.h:

(WebKit::WebIDBCallbacks::onSuccess):

  • public/WebIDBFactory.h:

(WebKit::WebIDBFactory::getDatabaseNames):

8:29 PM Changeset in webkit [95697] by Beth Dakin
  • 7 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=67415
Text drawn via -webkit-background-clip:text is blurry at device scale factors >1.0
-and corresponding-
<rdar://problem/10060379>

Reviewed by Darin Adler.

New helper function RenderBoxModelObject scales the mask image by the
deviceScaleFactor to get an image of the appropriate resolution. It also scales
the image's GraphicsContext so that the clip is set up on the same scale. Back in
paintFillLayerExtended() we still clip the image to the original maskRect to get
everything scaled back to the appropriate size.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::createDeviceScaledImageBuffer):
(WebCore::RenderBoxModelObject::paintFillLayerExtended):

Make the deviceScaleFactor convenience function just a namespace-level function in
Page rather than a static member or Page.

  • page/Page.cpp:

(WebCore::deviceScaleFactor):

  • page/Page.h:


Pre-existing callers of Page::deviceScaleFactor(Frame*) must now use
WebCore::deviceScaleFactor(Frame*)

  • editing/DeleteButtonController.cpp:

(WebCore::DeleteButtonController::createDeletionUI):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::imageSizeForError):
(WebCore::RenderImage::paintReplaced):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::drawPlatformResizerImage):

7:14 PM Changeset in webkit [95696] by inferno@chromium.org
  • 6 edits
    4 adds in trunk/LayoutTests

Unreviewed. Chromium rebaselines for r95671.

  • platform/chromium-linux/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/chromium-mac/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/chromium-win/fast/css-generated-content/table-cell-before-content-expected.txt:
  • platform/chromium-win/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/chromium-win/fast/table/quote-text-around-iframe-expected.txt: Added.
  • platform/chromium-win/fast/table/table-cell-after-child-in-block-expected.txt:
  • platform/chromium-win/fast/table/table-cell-before-child-in-block-expected.txt:
  • platform/chromium-win/fast/table/table-row-after-child-in-block-expected.txt:
  • platform/chromium-win/fast/table/table-row-before-child-in-block-expected.txt:
7:04 PM Changeset in webkit [95695] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebCore

[CG] ImageBufferData::getData has an invariant comparison in the inner part of a loop which doesn't get optimized out
​https://bugs.webkit.org/show_bug.cgi?id=68588
<rdar://problem/10164955>

Reviewed by Simon Fraser.

Factor the unmultiplied check out of the inner loop, resulting in a
speed bump in ImageBufferData::getData.

No new tests, performance improvement.

  • platform/graphics/cg/ImageBufferDataCG.cpp:

(WebCore::ImageBufferData::getData):

6:52 PM Changeset in webkit [95694] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

:hover selector fails when hovering over a child select element with size attribute
​https://bugs.webkit.org/show_bug.cgi?id=67594

Patch by Sameer Patil <​mkrp87@motorola.com> on 2011-09-21
Reviewed by Darin Adler.

Source/WebCore:

Test: fast/css/hover-affects-ancestor.html

This patch addresses the problem of deactivation of the ancestor element :hover rules
when hovered its contained elements.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::updateHoverActiveState):

LayoutTests:

  • fast/css/hover-affects-ancestor-expected.txt: Added.
  • fast/css/hover-affects-ancestor.html: Added.
6:46 PM Changeset in webkit [95693] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Add missing addPtr->add32 mapping for X86.

Rubber stamped by Sam Weinig.

  • assembler/MacroAssembler.h:

(JSC::MacroAssembler::addPtr):

6:10 PM Changeset in webkit [95692] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Add missing addDouble for AbsoluteAddress to X86

Rubber stamped by Geoff Garen.

  • assembler/MacroAssemblerX86.h:

(JSC::MacroAssemblerX86::addDouble):

  • assembler/X86Assembler.h:

(JSC::X86Assembler::addsd_mr):
(JSC::X86Assembler::cvtsi2sd_rr):
(JSC::X86Assembler::cvtsi2sd_mr):

6:06 PM Changeset in webkit [95691] by commit-queue@webkit.org
  • 34 edits
    12 adds in trunk/LayoutTests

Rebaseline for bug 65583 (path based border radius drawing on skia) part 4
​https://bugs.webkit.org/show_bug.cgi?id=68339

Patch by Ben Wells <​benwells@chromium.org> on 2011-09-21
Reviewed by James Robinson.

  • platform/chromium-linux/css1/box_properties/border_style-expected.png:
  • platform/chromium-linux/css1/box_properties/border_style_inline-expected.png:
  • platform/chromium-linux/css2.1/: 14 changes
  • platform/chromium-linux/fast/borders/border-radius-inset-outset-expected.png: Added.
  • platform/chromium-linux/fast/borders/border-styles-split-expected.png: Added.
  • platform/chromium-linux/fast/writing-mode/border-styles-vertical-lr-expected.png: Added.
  • platform/chromium-linux/fast/writing-mode/border-styles-vertical-rl-expected.png: Added.
  • platform/chromium-win/css1/box_properties/border_style-expected.png:
  • platform/chromium-win/css1/box_properties/border_style_inline-expected.png:
  • platform/chromium-win/css2.1/: 14 changes
  • platform/chromium-win/fast/borders/border-radius-inset-outset-expected.png: Added.
  • platform/chromium-win/fast/borders/border-radius-inset-outset-expected.txt: Added.
  • platform/chromium-win/fast/borders/border-styles-split-expected.png: Added.
  • platform/chromium-win/fast/borders/border-styles-split-expected.txt: Added.
  • platform/chromium-win/fast/writing-mode/border-styles-vertical-lr-expected.png: Added.
  • platform/chromium-win/fast/writing-mode/border-styles-vertical-lr-expected.txt: Added.
  • platform/chromium-win/fast/writing-mode/border-styles-vertical-rl-expected.png: Added.
  • platform/chromium-win/fast/writing-mode/border-styles-vertical-rl-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
6:01 PM Changeset in webkit [95690] by ap@apple.com
  • 2 edits in trunk/Source/WebKit2

[WK2] UIProcess should check that WebProcess isn't sending unexpected file: URLs to it
​https://bugs.webkit.org/show_bug.cgi?id=68573

Unreviewed follow-up fix.

  • UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::checkURLReceivedFromWebProcess): Just like we allow null URLs, also allow empty strings.
5:17 PM Changeset in webkit [95689] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[Chromium] Protect message ports from being deleted in V8MessageEvent::portsAccessorGetter
​https://bugs.webkit.org/show_bug.cgi?id=68584

Patch by Sergey Glazunov <​serg.glazunov@gmail.com> on 2011-09-21
Reviewed by Adam Barth.

Source/WebCore:

Test: fast/dom/message-port-deleted-by-accessor.html

  • bindings/v8/custom/V8MessageEventCustom.cpp:

(WebCore::V8MessageEvent::portsAccessorGetter):

LayoutTests:

  • fast/dom/message-port-deleted-by-accessor-expected.txt: Added.
  • fast/dom/message-port-deleted-by-accessor.html: Added.
5:05 PM Changeset in webkit [95688] by levin@chromium.org
  • 3 edits
    6 adds
    1 delete in trunk/LayoutTests

[chromium] Rebaselines for passing tests and expectation updates/narrowing.

  • platform/chromium-cg-mac-leopard/editing/inserting/editable-inline-element-expected.png: Added.
  • platform/chromium-mac-leopard/css2.1/t090204-display-change-01-b-ao-expected.png: Added.
  • platform/chromium-mac-leopard/editing/inserting/editable-inline-element-expected.png: Added.
  • platform/chromium-mac/css2.1/t090204-display-change-01-b-ao-expected.png:
  • platform/chromium-mac/css2.1/t090204-display-change-01-b-ao-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
5:04 PM Changeset in webkit [95687] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Build fix following fix for bug #68586.

  • jit/JIT.cpp:
  • jit/JITInlineMethods.h:
4:58 PM Changeset in webkit [95686] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Add back protection against the NSView going away while handling mouseDown
​https://bugs.webkit.org/show_bug.cgi?id=68585

Reviewed by Darin Adler.

Add calls to RenderWidget::suspendWidgetHierarchyUpdates() and RenderWidget::resumeWidgetHierarchyUpdates(),
as a followup fix to ​https://bugs.webkit.org/show_bug.cgi?id=68570.

  • page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::passMouseDownEventToWidget):

4:50 PM Changeset in webkit [95685] by jchaffraix@webkit.org
  • 21 edits
    10 adds in trunk

Crash in RenderBox::paintMaskImages when GraphicsContext's painting is disabled
​https://bugs.webkit.org/show_bug.cgi?id=68133

Reviewed by Darin Adler.

.:

  • Source/autotools/symbols.filter: Added the mangled symbols needed for window.internals

Source/WebCore:

Tests: fast/css/webkit-mask-crash-fieldset-legend.html

fast/css/webkit-mask-crash-figure.html
fast/css/webkit-mask-crash-table.html
fast/css/webkit-mask-crash-td-2.html
fast/css/webkit-mask-crash-td.html

GraphicsContext::getCTM crashes if called with a GraphicsContext that has painting
disabled. RenderBox::paintMaskImages would thus crash if called in this condition.

This change just modifies the different GraphicsContext::getCTM method to bail early
if painting is disabled on the GraphicsContext. The rest of the change is exposing
paintControlTints that exposes this.

  • WebCore.exp.in: Added symbols of the newly export window.internals function.
  • page/FrameView.cpp:

(WebCore::FrameView::updateControlTints): Split this function in 2 so that
I can expose the internal paintControlTints.

(WebCore::FrameView::paintControlTints):
This is the one exposed to Internals as we want to be testable regardless of
whether the platform supports control tints.

  • page/FrameView.h: Added paintControlTints.
  • testing/Internals.cpp:

(WebCore::Internals::paintControlTints):

  • testing/Internals.h:
  • testing/Internals.idl:

Added a way to force a fake painting so that we can easily reproduce the bugs.

  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/skia/GraphicsContextSkia.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/wince/GraphicsContextWinCE.cpp:

(WebCore::GraphicsContext::getCTM):

  • platform/graphics/wx/GraphicsContextWx.cpp:

(WebCore::GraphicsContext::getCTM):
Fixed all our back-end to exit early if painting is disabled.

Source/WebKit2:

  • win/WebKit2.def:
  • win/WebKit2CFLite.def:

Exported the new FrameView::paintControlTints function.

LayoutTests:

Those tests checks that we do not crash when calling internals.paintControlTints.

  • platform/mac/Skipped: Skipped 2 tests as they are hitting an ASSERT unrelated to

this change on Mac.

  • fast/css/webkit-mask-crash-fieldset-legend-expected.txt: Added.
  • fast/css/webkit-mask-crash-fieldset-legend.html: Added.
  • fast/css/webkit-mask-crash-figure-expected.txt: Added.
  • fast/css/webkit-mask-crash-figure.html: Added.
  • fast/css/webkit-mask-crash-table-expected.txt: Added.
  • fast/css/webkit-mask-crash-table.html: Added.
  • fast/css/webkit-mask-crash-td-2-expected.txt: Added.
  • fast/css/webkit-mask-crash-td-2.html: Added.
  • fast/css/webkit-mask-crash-td-expected.txt: Added.
  • fast/css/webkit-mask-crash-td.html: Added.
4:49 PM Changeset in webkit [95684] by barraclough@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

DFG JIT should be able to compile op_throw
​https://bugs.webkit.org/show_bug.cgi?id=68571

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-09-21
Reviewed by Geoffrey Garen.

This compiles op_throw in the simplest way possible: it's an OSR
point back to the old JIT. This is a good step towards increasing
coverage, particularly on Kraken, but it's neutral because the
same functions that do throw also use some other unsupported
opcodes.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGNode.h:
  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

4:46 PM Changeset in webkit [95683] by fpizlo@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

DFG JIT should be able to compile op_throw
​https://bugs.webkit.org/show_bug.cgi?id=68571

Reviewed by Geoffrey Garen.

This compiles op_throw in the simplest way possible: it's an OSR
point back to the old JIT. This is a good step towards increasing
coverage, particularly on Kraken, but it's neutral because the
same functions that do throw also use some other unsupported
opcodes.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

  • dfg/DFGNode.h:
  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

4:44 PM Changeset in webkit [95682] by commit-queue@webkit.org
  • 4 edits in trunk

IndexedDB: compare strings without decoding
​https://bugs.webkit.org/show_bug.cgi?id=68554

Patch by Joshua Bell <​jsbell@chromium.org> on 2011-09-21
Reviewed by Tony Chang.

Resolves a FIXME in IndexedDB that was also identified as a hotspot
during profiling. Yields a small performance improvement.

  • Source/WebCore/storage/IDBLevelDBCoding.cpp:
  • Source/WebCore/storage/IDBLevelDBCoding.h:
  • Source/WebKit/chromium/tests/IDBLevelDBCodingTest.cpp:
4:36 PM Changeset in webkit [95681] by fpizlo@apple.com
  • 24 edits
    3 adds in trunk/Source/JavaScriptCore

DFG should support continuous optimization
​https://bugs.webkit.org/show_bug.cgi?id=68329

Reviewed by Geoffrey Garen.

This adds the ability to reoptimize a code block if speculation
failures happen frequently. 6% speed-up on Kraken, 1% slow-down
on V8, neutral on SunSpider.

(JSC::CodeBlock::CodeBlock):
(JSC::ProgramCodeBlock::jettison):
(JSC::EvalCodeBlock::jettison):
(JSC::FunctionCodeBlock::jettison):
(JSC::CodeBlock::shouldOptimizeNow):
(JSC::CodeBlock::dumpValueProfiles):

  • bytecode/CodeBlock.h:
  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::getStrongPrediction):

  • dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::exitSpeculativeWithOSR):
(JSC::DFG::JITCompiler::compileEntry):
(JSC::DFG::JITCompiler::compileBody):

  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::noticeOSREntry):

  • dfg/DFGOSREntry.cpp:

(JSC::DFG::prepareOSREntry):

  • dfg/DFGOSREntry.h:

(JSC::DFG::getOSREntryDataBytecodeIndex):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • heap/ConservativeRoots.cpp:

(JSC::ConservativeRoots::ConservativeRoots):
(JSC::ConservativeRoots::~ConservativeRoots):
(JSC::DummyMarkHook::mark):
(JSC::ConservativeRoots::genericAddPointer):
(JSC::ConservativeRoots::genericAddSpan):
(JSC::ConservativeRoots::add):

  • heap/ConservativeRoots.h:
  • heap/Heap.cpp:

(JSC::Heap::addJettisonCodeBlock):
(JSC::Heap::markRoots):

  • heap/Heap.h:
  • heap/JettisonedCodeBlocks.cpp: Added.

(JSC::JettisonedCodeBlocks::JettisonedCodeBlocks):
(JSC::JettisonedCodeBlocks::~JettisonedCodeBlocks):
(JSC::JettisonedCodeBlocks::addCodeBlock):
(JSC::JettisonedCodeBlocks::clearMarks):
(JSC::JettisonedCodeBlocks::deleteUnmarkedCodeBlocks):
(JSC::JettisonedCodeBlocks::traceCodeBlocks):

  • heap/JettisonedCodeBlocks.h: Added.

(JSC::JettisonedCodeBlocks::mark):

  • interpreter/RegisterFile.cpp:

(JSC::RegisterFile::gatherConservativeRoots):

  • interpreter/RegisterFile.h:
  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/Executable.cpp:

(JSC::jettisonCodeBlock):
(JSC::EvalExecutable::jettisonOptimizedCode):
(JSC::ProgramExecutable::jettisonOptimizedCode):
(JSC::FunctionExecutable::jettisonOptimizedCodeForCall):
(JSC::FunctionExecutable::jettisonOptimizedCodeForConstruct):

  • runtime/Executable.h:

(JSC::FunctionExecutable::jettisonOptimizedCodeFor):

  • wtf/BitVector.h: Added.

(WTF::BitVector::BitVector):
(WTF::BitVector::~BitVector):
(WTF::BitVector::operator=):
(WTF::BitVector::size):
(WTF::BitVector::ensureSize):
(WTF::BitVector::resize):
(WTF::BitVector::clearAll):
(WTF::BitVector::get):
(WTF::BitVector::set):
(WTF::BitVector::clear):
(WTF::BitVector::bitsInPointer):
(WTF::BitVector::maxInlineBits):
(WTF::BitVector::byteCount):
(WTF::BitVector::makeInlineBits):
(WTF::BitVector::OutOfLineBits::numBits):
(WTF::BitVector::OutOfLineBits::numWords):
(WTF::BitVector::OutOfLineBits::bits):
(WTF::BitVector::OutOfLineBits::create):
(WTF::BitVector::OutOfLineBits::destroy):
(WTF::BitVector::OutOfLineBits::OutOfLineBits):
(WTF::BitVector::isInline):
(WTF::BitVector::outOfLineBits):
(WTF::BitVector::resizeOutOfLine):
(WTF::BitVector::bits):

4:32 PM Changeset in webkit [95680] by annacc@chromium.org
  • 2 edits in trunk/Tools

Update committers.py with a new contributor contact
​https://bugs.webkit.org/show_bug.cgi?id=68565

Reviewed by Eric Carlson.

  • Scripts/webkitpy/common/config/committers.py:
4:25 PM Changeset in webkit [95679] by ap@apple.com
  • 8 edits in trunk/Source/WebKit2

[WK2] UIProcess should check that WebProcess isn't sending unexpected file: URLs to it
​https://bugs.webkit.org/show_bug.cgi?id=68573

Reviewed by Anders Carlsson.

  • UIProcess/API/mac/WKView.mm: (maybeCreateSandboxExtensionFromPasteboard): Return a boolean, telling the caller whether an extension actually needed to be created (-[WKView performDragOperation:]): Tell process proxy when the process is going to get universal file read sandbox extension.
  • UIProcess/WebContext.cpp: (WebKit::WebContext::didPerformClientRedirect): Check the URLs. (WebKit::WebContext::didPerformServerRedirect): Ditto. (WebKit::WebContext::didUpdateHistoryTitle): Ditto. (WebKit::WebContext::getPluginPath): Ditto. Also, properly parse the URL - we can never assume that a string coming from WebProcess is a ParsedURLString.
  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::reattachToWebProcessWithItem): Tell process proxy when the process is going to get universal file read sandbox extension. (WebKit::WebPageProxy::maybeInitializeSandboxExtensionHandle): Changed to return a boolean, telling the caller whether an extension actually needed to be created. (WebKit::WebPageProxy::loadURL): Tell process proxy about extension. (WebKit::WebPageProxy::loadURLRequest): Ditto. (WebKit::WebPageProxy::loadHTMLString): Tell process proxy if a file URL was used as a base one for a string. In this case, WebKit2 assumes that WebProcess has access to a subdirectory, (typically, one where error page resources live), and can load from it. (WebKit::WebPageProxy::loadAlternateHTMLString): Ditto. (WebKit::WebPageProxy::goForward): Tell process proxy about extension. (WebKit::WebPageProxy::goBack): Tell process proxy about extension. (WebKit::WebPageProxy::goToBackForwardItem): Tell process proxy about extension. (WebKit::WebPageProxy::didStartProvisionalLoadForFrame): Check the URL. (WebKit::WebPageProxy::didReceiveServerRedirectForProvisionalLoadForFrame): Ditto. (WebKit::WebPageProxy::didSameDocumentNavigationForFrame): Ditto. (WebKit::WebPageProxy::decidePolicyForNavigationAction): Ditto. (WebKit::WebPageProxy::decidePolicyForNewWindowAction): Ditto. (WebKit::WebPageProxy::decidePolicyForResponse): Ditto. (WebKit::WebPageProxy::didInitiateLoadForResource): Ditto. (WebKit::WebPageProxy::didSendRequestForResource): Ditto. (WebKit::WebPageProxy::didReceiveResponseForResource): Ditto. (WebKit::WebPageProxy::missingPluginButtonClicked): Ditto.
  • UIProcess/WebPageProxy.h: Changed initializeSandboxExtensionHandle() to return a bool, and renamed to maybeInitializeSandboxExtensionHandle (matching WKView counterpart).
  • UIProcess/WebProcessProxy.cpp: (WebKit::WebProcessProxy::WebProcessProxy): Initialize m_mayHaveUniversalFileReadSandboxExtension. It's going to be true if we ever granted an extension for "/". (WebKit::WebProcessProxy::willLoadHTMLStringWithBaseURL): Remember the path, we should expect that WebProcess will load subresources from it. (WebKit::WebProcessProxy::checkURLReceivedFromWebProcess): Check that it's reasonable to expect WebProcess send us a URL like this. (WebKit::WebProcessProxy::addBackForwardItem): Check the URLs.
  • UIProcess/WebProcessProxy.h: Added data members remembering what to expect from this process.
  • UIProcess/cf/WebPageProxyCF.cpp: (WebKit::WebPageProxy::restoreFromSessionStateData): Tell process proxy when the process is going to get universal file read sandbox extension.
4:06 PM Changeset in webkit [95678] by jer.noble@apple.com
  • 9 edits in trunk/Source

Add settings to control the availability of the Web Audio API to WebKit and WebKit2.
​https://bugs.webkit.org/show_bug.cgi?id=68382

Source/WebCore:

Reviewed by Darin Adler.

Only create a AudioContext object if the Web Audio feature is runtime-enabled in Settings.

No new tests, as this feature is not yet enabled by default.

  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::webkitAudioContext):

Source/WebKit/mac:

Reviewed by Darin Adler.

Initialize the webAudioEnabled preference to NO by default.

  • WebView/WebPreferences.mm:

(+[WebPreferences initialize]):

Source/WebKit2:

Add support for the webAudioEnabled preference in WebKit2.

Reviewed by Darin Adler.

  • Shared/WebPreferencesStore.h: Add WebAudioEnabled getter and setter macro.
  • UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetWebAudioEnabled): Added.
(WKPreferencesGetWebAudioEnabled): Added.

  • UIProcess/API/C/WKPreferences.h:
  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences): Update the webAudioEnabled preference.

3:54 PM Changeset in webkit [95677] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

[chromium] Code cleanup in gesture recognizer unit tests
​https://bugs.webkit.org/show_bug.cgi?id=68561

Fix some style nits in the gesture recognizer unit tests.

Patch by Robert Kroeger <​rjkroege@chromium.org> on 2011-09-21
Reviewed by David Levin.

  • tests/InnerGestureRecognizerTest.cpp:

(InspectableGestureRecognizerChromium::stubEdgeFunction):
(BuildablePlatformTouchPoint::BuildablePlatformTouchPoint):
(TEST_F):

3:43 PM Changeset in webkit [95676] by barraclough@apple.com
  • 7 edits in trunk/Source/JavaScriptCore

Should support value profiling on CPU(X86)
​https://bugs.webkit.org/show_bug.cgi?id=68575

Reviewed by Sam Weinig.

Fix verbose profiling in ToT (SlowCaseProfile had been
partially renamed to RareCaseProfile), add in-memory
bucket counter for CPU(X86), move JIT::m_canBeOptimized
out of the DFG_JIT ifdef.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::resetRareCaseProfiles):
(JSC::CodeBlock::dumpValueProfiles):

  • bytecode/CodeBlock.h:
  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::makeSafe):

  • jit/JIT.cpp:

(JSC::JIT::privateCompileSlowCases):
(JSC::JIT::privateCompile):

  • jit/JIT.h:
  • jit/JITInlineMethods.h:

(JSC::JIT::emitValueProfilingSite):

3:41 PM Changeset in webkit [95675] by andersca@apple.com
  • 7 edits in trunk/Source

Remove checks for Leopard-only Objective-C APIs
​https://bugs.webkit.org/show_bug.cgi?id=68579

Reviewed by Sam Weinig.

We can always assume that OBJC_API_VERSION is at least 2, it's been 2 since Leopard.

../WebCore:

  • bridge/objc/objc_class.mm:

(JSC::Bindings::ObjcClass::methodsNamed):
(JSC::Bindings::ObjcClass::fieldNamed):

  • bridge/objc/objc_instance.mm:

(allocateAutoReleasePool):

  • bridge/objc/objc_runtime.mm:

(JSC::Bindings::ObjcField::ObjcField):

  • page/mac/EventHandlerMac.mm:

../WebKit/mac:

  • Misc/WebNSObjectExtras.h:
3:33 PM Changeset in webkit [95674] by hyatt@apple.com
  • 1 edit in trunk/LayoutTests/ChangeLog

Fix resolve-Changelog's strange butchery of the LayoutTests/Changelog.

3:20 PM Changeset in webkit [95673] by andersca@apple.com
  • 5 edits in trunk/Source/WebCore

Remove Widget::beforeMouseDown and Widget::afterMouseDown
​https://bugs.webkit.org/show_bug.cgi?id=68570

Reviewed by Sam Weinig.

These two member functions were originally added to avoid crashes due to removing NSViews while they were
being tracked by AppKit. Since they were added, we've moved away from NSViews for form controls, and the bugs in
AppKit that lead to crashes have been fixed.

  • page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::passMouseDownEventToWidget):

  • platform/Widget.h:
  • platform/mac/WidgetMac.mm:

(WebCore::Widget::Widget):
(WebCore::Widget::removeFromSuperview):

  • rendering/RenderWidget.cpp:

Change widgetHierarchyUpdateSuspendCount to be an unsigned integer instead of a size_t.

3:17 PM Changeset in webkit [95672] by fpizlo@apple.com
  • 10 edits in trunk/Source/JavaScriptCore

DFG does not support compiling functions as constructors
​https://bugs.webkit.org/show_bug.cgi?id=68500

Reviewed by Oliver Hunt.

This adds support for compiling constructors to the DFG. It's a
1% speed-up on V8, mostly due to a 6% speed-up on early-boyer.
It's also a 13% win on access-binary-trees, but it's neutral in
the SunSpider and Kraken averages.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGCapabilities.h:

(JSC::DFG::mightCompileFunctionForConstruct):
(JSC::DFG::canCompileOpcode):

  • dfg/DFGNode.h:
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • dfg/DFGPropagator.cpp:

(JSC::DFG::Propagator::propagateNodePredictions):
(JSC::DFG::Propagator::performNodeCSE):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compile):

  • runtime/Executable.cpp:

(JSC::FunctionExecutable::compileOptimizedForConstruct):
(JSC::FunctionExecutable::compileForConstructInternal):

  • runtime/Executable.h:

(JSC::FunctionExecutable::compileForConstruct):
(JSC::FunctionExecutable::compileFor):
(JSC::FunctionExecutable::compileOptimizedFor):

3:05 PM Changeset in webkit [95671] by inferno@chromium.org
  • 16 edits
    3 adds in trunk

Source/WebCore: Not use anonymousContainer on beforeChild calculation
and let later code handle it. Fix style update on block-level
anonymous objects.
​https://bugs.webkit.org/show_bug.cgi?id=67656

Reviewed by David Hyatt.

Test: fast/table/quote-text-around-iframe.html

  • rendering/RenderTable.cpp:

(WebCore::RenderTable::addChild):

  • rendering/RenderTableCell.cpp:

(WebCore::RenderTableCell::styleDidChange):

  • rendering/RenderTableRow.cpp:

(WebCore::RenderTableRow::addChild):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::propagateStyleToAnonymousChildren):

LayoutTests: Not use anonymousContainer on beforeChild calculation
and let later code handle it.
​https://bugs.webkit.org/show_bug.cgi?id=67656

Reviewed by David Hyatt.

  • fast/table/quote-text-around-iframe.html: Added.
  • fast/table/table-row-style-not-updated-with-after-content-expected.txt:
  • platform/mac/fast/table/quote-text-around-iframe-expected.png: Added.
  • platform/mac/fast/table/quote-text-around-iframe-expected.txt: Added.

and lot of style propagation rebaselines.

2:56 PM Changeset in webkit [95670] by hyatt@apple.com
  • 4 edits
    3 adds in trunk

Source/WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=68562

Pagination struts don't update properly when collapsing margin estimates are wrong.

Refactor the pagination adjustments that happen during block layout to allow for
a 3-pass layout system instead of just a 2-pass layout system. The basic idea is
to come up with an estimate that includes collapsing, clearance and pagination.

If the estimate only with collapsing and clearance was wrong, then we lay out again
using the correct collapsing margin values and clearance before applying pagination breaks.

After this second layout, the normal code that runs when unpaginated compares estimates
and lays out a third time to account for adjustments as a result of pagination movement.

Reviewed by Sam Weinig.

Added fast/regions/text-regions-split-small-pagination.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::estimateLogicalTopPosition):
(WebCore::RenderBlock::layoutBlockChild):
(WebCore::RenderBlock::adjustBlockChildForPagination):

  • rendering/RenderBlock.h:

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

Pagination struts don't update properly when collapsing margin estimates are wrong.

Add a new regions result to test the bug fix.

Reviewed by Sam Weinig.

  • fast/regions/text-region-split-small-pagination.html: Added.
  • platform/mac/fast/regions/text-region-split-small-pagination-expected.png: Added.
  • platform/mac/fast/regions/text-region-split-small-pagination-expected.txt: Added.
2:21 PM Changeset in webkit [95669] by aa@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

Remove old and busted context notification APIs
​https://bugs.webkit.org/show_bug.cgi?id=68381

Reviewed by David Levin.

  • public/WebFrameClient.h:
  • src/FrameLoaderClientImpl.cpp:

(WebKit::FrameLoaderClientImpl::didCreateScriptContext):
(WebKit::FrameLoaderClientImpl::willReleaseScriptContext):

1:39 PM Changeset in webkit [95668] by jamesr@google.com
  • 2 edits in trunk/Source/WebCore

[chromium] SharedGraphicsContext3D's context should not be destroyed during atexit
​https://bugs.webkit.org/show_bug.cgi?id=68558

Reviewed by Stephen White.

The SharedGraphicsContext3D's internal context is lazily created and then leaked intentionally, but since the
last ref is in a function-static RefPtr<> it ends up being destroyed at process exit time. This is bad, since
by the time we get this far in process shutdown we can't actually cleanly run the destruction logic and we
crash. Since we are explicitly leaking this for the lifetime of the process there's no point in keeping it in a
RefPtr<>

  • platform/graphics/gpu/SharedGraphicsContext3D.cpp:

(WebCore::SharedGraphicsContext3D::get):

1:13 PM Changeset in webkit [95667] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

[Chromium] Protect the Frame in V8HTMLDocument::openCallback
​https://bugs.webkit.org/show_bug.cgi?id=68555

Patch by Sergey Glazunov <​serg.glazunov@gmail.com> on 2011-09-21
Reviewed by Nate Chapin.

Source/WebCore:

Test: fast/dom/frame-deleted-in-document-open.html

  • bindings/v8/custom/V8HTMLDocumentCustom.cpp:

(WebCore::V8HTMLDocument::openCallback):

LayoutTests:

  • fast/dom/frame-deleted-in-document-open-expected.txt: Added.
  • fast/dom/frame-deleted-in-document-open.html: Added.
12:59 PM Changeset in webkit [95666] by barraclough@apple.com
  • 16 edits in trunk/Source/JavaScriptCore

Replace jsFunctionVPtr compares with a type check on the Structure.
​https://bugs.webkit.org/show_bug.cgi?id=68557

Reviewed by Oliver Hunt.

This will permit calls to still optimize to subclasses of JSFunction
that have the correct type (but a different C++ vptr).

This patch stops passing the globalData into numerous functions.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::isFunctionConstant):
(JSC::DFG::Graph::valueOfFunctionConstant):

  • dfg/DFGJITCompiler.h:

(JSC::DFG::JITCompiler::isFunctionConstant):
(JSC::DFG::JITCompiler::valueOfFunctionConstant):

  • dfg/DFGOperations.cpp:
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JIT.h:
  • jit/JITCall.cpp:

(JSC::JIT::compileOpCallVarargs):
(JSC::JIT::compileOpCallSlowCase):

  • jit/JITCall32_64.cpp:

(JSC::JIT::compileOpCallVarargs):
(JSC::JIT::compileOpCallSlowCase):

  • jit/JITInlineMethods.h:

(JSC::JIT::emitJumpIfNotType):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/Executable.h:

(JSC::isHostFunction):

  • runtime/JSFunction.h:

(JSC::JSFunction::createStructure):

  • runtime/JSObject.cpp:

(JSC::JSObject::put):
(JSC::JSObject::putWithAttributes):

  • runtime/JSObject.h:

(JSC::getJSFunction):
(JSC::JSObject::putDirect):
(JSC::JSObject::putDirectWithoutTransition):

  • runtime/JSType.h:
12:49 PM Changeset in webkit [95665] by leandro@webkit.org
  • 3 edits in trunk

Unreviewed. Remove unused ENABLE_AS_IMAGE flag after r95234 from
Options{Efl,WinCE}.cmake.

  • Source/cmake/OptionsEfl.cmake:
  • Source/cmake/OptionsWinCE.cmake:
12:47 PM Changeset in webkit [95664] by adamk@chromium.org
  • 8 edits
    2 adds in trunk

Clean up CSS Counter code
​https://bugs.webkit.org/show_bug.cgi?id=68492

Reviewed by Tony Chang.

Source/WebCore:

Several cleanups/fixes:

  • Make Counter::listStyle() return the actual list style string, rather than empty string.
  • Handle a specified list style of "none" correctly when generating cssText and in Counter::listStyle().
  • Use StringBuilder when generating cssText.

Test: fast/css/counters/getCounterValue.html

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseCounterContent):
Simplify processing (moving conversion to CSSStyleSelector),
pass a CSS_IDENT as the listStyle to Counter::create.

  • css/CSSPrimitiveValue.cpp:

(WebCore::CSSPrimitiveValue::cssText):
Use StringBuilder.

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyProperty):
Convert from a CSS_IDENT to an EListStyleType.

  • css/Counter.h:

(WebCore::Counter::listStyleIdent):
Renamed from listStyleNumber to match its new functionality.

LayoutTests:

Added tests for setting the list style to "none"
as well as tests for getCounterValue.

  • fast/css/counters/counter-cssText-expected.txt:
  • fast/css/counters/counter-cssText.html:
  • fast/css/counters/getCounterValue-expected.txt: Added.
  • fast/css/counters/getCounterValue.html: Added.
12:34 PM Changeset in webkit [95663] by ggaren@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

Removed WTFTHREADDATA_MULTITHREADED, making it always true
​https://bugs.webkit.org/show_bug.cgi?id=68549

Reviewed by Darin Adler.

Another part of making threads exist in WebKit.

  • wtf/WTFThreadData.cpp:
  • wtf/WTFThreadData.h:

(WTF::wtfThreadData):

12:33 PM Changeset in webkit [95662] by cmarrin@apple.com
  • 2 edits in trunk/Source/WebCore

2011-09-21 Chris Marrin <​cmarrin@apple.com>

Check in WebCore.vcproj file unchanged but written like MSDev likes it

Unreviewed.

  • WebCode.vcproj/WebCore.vcproj:
12:30 PM Changeset in webkit [95661] by mitz@apple.com
  • 8 edits in trunk/Source/WebKit2

Use the prefix "WK" instead of "Web" for Objective-C classes defined in WebKit2.

Reviewed by Anders Carlsson.

Renamed WebInspectorProxyObjCAdapter to WKWebInspectorProxyObjCAdapter.
Renamed WebInspectorWKView to WKWebInspectorWKView.
Renamed WebFindIndicatorWindowAnimation to WKFindIndicatorWindowAnimation.
Renamed WebFindIndicatorView to WKFindIndicatorView.
Renamed WebUserDataWrapper to WKUserDataWrapper.

Updated the build phase checking for inappropriate Objective-C class names to disallow the prefix
"Web" and only allow "WK".

  • UIProcess/API/mac/FindIndicatorWindow.h:
  • UIProcess/API/mac/FindIndicatorWindow.mm:

(WebKit::FindIndicatorWindow::setFindIndicator):
(WebKit::FindIndicatorWindow::startFadeOutTimerFired):

  • UIProcess/WebInspectorProxy.h:
  • UIProcess/mac/WebContextMenuProxyMac.mm:

(-[WKMenuTarget forwardContextMenuAction:]):
(WebKit::nsMenuItemVector):

  • UIProcess/mac/WebInspectorProxyMac.mm:

(WebKit::WebInspectorProxy::platformCreateInspectorPage):
(WebKit::WebInspectorProxy::platformOpen):

  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/FullScreen/mac/WebFullScreenManagerMac.mm:

(WebKit::WebFullScreenManagerMac::WebFullScreenManagerMac):

12:24 PM Changeset in webkit [95660] by commit-queue@webkit.org
  • 18 edits in trunk

[WK2] [Mac] Implement a more-complete MouseDown/MouseUp/MouseMoveTo functions for WebKit2 EventSender
​https://bugs.webkit.org/show_bug.cgi?id=68108

This patch implements the event sender mouse events through WebKit2 UIProcess which is
closer to the real simulation than a WebProcess-only approach. The patch only supports Mac
platform as the first step so the existing code is still kept for other platforms.

Source/WebKit2:

Patch by Chang Shu <​cshu@webkit.org> on 2011-09-21
Reviewed by Darin Adler.

  • UIProcess/API/C/WKPage.cpp:

(WKPageSetShouldSendEventsSynchronously):

  • UIProcess/API/C/WKPagePrivate.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::handleMouseEvent):
(WebKit::WebPageProxy::handleKeyboardEvent):

  • UIProcess/WebPageProxy.h:

(WebKit::WebPageProxy::setShouldSendEventsSynchronously):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::mouseEventSyncForTesting):

  • WebProcess/WebPage/WebPage.h:
  • WebProcess/WebPage/WebPage.messages.in:

Tools:

Patch by Chang Shu <​cshu@webkit.org> on 2011-09-21
Reviewed by Darin Adler.

  • WebKitTestRunner/EventSenderProxy.h:

(WTR::EventSenderProxy::EventSenderProxy):
(WTR::EventSenderProxy::leapForward):
(WTR::EventSenderProxy::currentEventTime):

  • WebKitTestRunner/InjectedBundle/EventSendingController.cpp:

(WTR::EventSendingController::EventSendingController):
(WTR::EventSendingController::mouseDown):
(WTR::EventSendingController::mouseUp):
(WTR::EventSendingController::mouseMoveTo):
(WTR::EventSendingController::leapForward):
(WTR::EventSendingController::keyDown):

  • WebKitTestRunner/InjectedBundle/EventSendingController.h:
  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::didReceiveSynchronousMessageFromInjectedBundle):

  • WebKitTestRunner/mac/EventSenderProxy.mm:

(WTR::eventTypeForMouseButtonAndAction):
(WTR::operator==):
(WTR::EventSenderProxy::updateClickCountForButton):
(WTR::EventSenderProxy::mouseDown):
(WTR::EventSenderProxy::mouseUp):
(WTR::EventSenderProxy::mouseMoveTo):
(WTR::EventSenderProxy::keyDown):

  • WebKitTestRunner/qt/EventSenderProxyQt.cpp:

(WTR::EventSenderProxy::keyDown):
(WTR::EventSenderProxy::mouseDown):
(WTR::EventSenderProxy::mouseUp):
(WTR::EventSenderProxy::mouseMoveTo):

LayoutTests:

Unskip the passed tests.

Patch by Chang Shu <​cshu@webkit.org> on 2011-09-21
Reviewed by Darin Adler.

  • platform/mac-wk2/Skipped:
  • platform/wk2/Skipped:
12:17 PM Changeset in webkit [95659] by demarchi@webkit.org
  • 28 edits in trunk/Source/WebKit/efl

[EFL] Revert pointer operator coding style in ewk
​https://bugs.webkit.org/show_bug.cgi?id=68231

Reviewed by Antonio Gomes.

The WebKit coding style mandates that "(...) in C++ code - Both pointer
types and reference types should be written with no space between the
type name and the * or &.". Until now EFL port was using the
coding-style of the library for platform code instead of WebKit's.

We are now trying to be more compliant with WebKit coding-style so it's
easier for other reviewers to review EFL code. The first step in this
direction is to move the '*' and '&' of pointers and references to the
right place.

This patch was generated with 'uncrustify' with the configure file
attached to the respective but report. There was one fix that had to be
manually made in order to keep the 'check-webkit-style' happy: a
function definition with a meaningless name. Other functions like this
exist but they will be fixed by a later patch.

Some nice side-effects of using uncrustify:

  • Whitespace cleanup on comments
  • Function parameters are aligned to the open parenthesis
  • Remove tab characters
12:11 PM Changeset in webkit [95658] by senorblanco@chromium.org
  • 4 edits in trunk/Source/WebCore

[chromium] Optimize ImageBuffer constructor when accelerated
​https://bugs.webkit.org/show_bug.cgi?id=68501

If the accelerated rendering mode is passed in to ImageBuffer, instead
of creating a bitmap PlatformCanvas and subsequently replacing its
device with an SkGpuDevice, create an SkCanvas and SkGpuDevice first,
and fall back to raster creation only failure. This saves on system
RAM for the backing store (and possibly GDI resources).

Reviewed by James Robinson.

Covered by existing tests.

  • platform/graphics/gpu/SharedGraphicsContext3D.cpp:

(WebCore::SharedGraphicsContext3D::get):

  • platform/graphics/gpu/SharedGraphicsContext3D.h:

Rename "SharedGraphicsContext3D::create()" to "get()" to more
accurately reflect its function.

  • platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::createAcceleratedCanvas):
Refactor creation of accelerated ImageBuffer.
(WebCore::ImageBuffer::ImageBuffer):
Attempt creation of accelerated ImageBuffer first, and only fall back
to bitmap-backed canvas creation if it fails.

12:06 PM Changeset in webkit [95657] by dimich@chromium.org
  • 3 edits
    4 copies in branches/chromium/874

Merge 95471 - [Chromium] Crash after magic iframe transfer for Pepper/NaCl plugins.
​https://bugs.webkit.org/show_bug.cgi?id=68267
Make adoptNode() to not enable live iframe transfer when the iframe's subtree contains plugins.

Reviewed by Adam Barth.

Source/WebCore:

Test: fast/frames/iframe-reparenting-embed-elements.html

  • dom/Document.cpp:

(WebCore::Document::adoptNode):

  • html/HTMLFrameElementBase.cpp:

(WebCore::hasPluginElements):
(WebCore::HTMLFrameElementBase::canRemainAliveOnRemovalFromTree):

  • html/HTMLFrameElementBase.h:

LayoutTests:

  • fast/frames/iframe-reparenting-embed-elements-expected.txt: Added.
  • fast/frames/iframe-reparenting-embed-elements.html: Added.
  • fast/frames/resources/iframe-reparenting-embed-frame1.html: Added.
  • fast/frames/resources/iframe-reparenting-embed-iframe.html: Added.

TBR=​dimich@chromium.org
Review URL: ​http://codereview.chromium.org/7980042

11:59 AM Changeset in webkit [95656] by demarchi@webkit.org
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Don't try to free user-provided list
​https://bugs.webkit.org/show_bug.cgi?id=68356

Reviewed by Antonio Gomes.

A coding-style cleanup revealed that we were using EINA_LIST_FREE in a
wrong manner. If we intended to free the data, we should call free()
(or whatever function is required to destroy it). However not only did
it use the wrong EFL api but it also introduced a change in behavior,
freeing user-provided data.

  • ewk/ewk_frame.cpp:

(ewk_frame_feed_touch_event): do not free user-provided data.

11:48 AM Changeset in webkit [95655] by mitz@apple.com
  • 9 edits
    1 copy in trunk

Source/JavaScriptCore: JavaScriptCore Part of: Prevent the WebKit frameworks from defining inappropriately-named Objective-C classes
​https://bugs.webkit.org/show_bug.cgi?id=68451

Reviewed by Darin Adler.

  • JavaScriptCore.xcodeproj/project.pbxproj: Added a script build phase that invokes

check-for-inappropriate-objc-class-names, allowing only class names prefixed with "JS".

Source/WebCore: WebCore part of: Prevent the WebKit frameworks from defining inappropriately-named Objective-C classes
​https://bugs.webkit.org/show_bug.cgi?id=68451

Reviewed by Darin Adler.

  • WebCore.xcodeproj/project.pbxproj: Added a script build phase that invokes

check-for-inappropriate-objc-class-names, allowing only class names prefixed with "DOM" or "Web".

Source/WebKit: WebKit part of: Prevent the WebKit frameworks from defining inappropriately-named Objective-C classes
​https://bugs.webkit.org/show_bug.cgi?id=68451

Reviewed by Darin Adler.

  • WebKit.xcodeproj/project.pbxproj: Added a script build phase that invokes

check-for-inappropriate-objc-class-names, allowing only class names prefixed with "Web" or "_Web".

Source/WebKit2: WebKit2 part of: Prevent the WebKit frameworks from defining inappropriately-named Objective-C classes
​https://bugs.webkit.org/show_bug.cgi?id=68451

Reviewed by Darin Adler.

  • WebKit2.xcodeproj/project.pbxproj: Added a script build phase that invokes

check-for-inappropriate-objc-class-names, allowing only class names prefixed with "WK" or "Web".

Tools: Tools part of: Prevent the WebKit frameworks from defining inappropriately-named Objective-C classes
​https://bugs.webkit.org/show_bug.cgi?id=68451

Reviewed by Darin Adler.

  • Scripts/check-for-inappropriate-objc-class-names: Added. Checks for

Objective-C classes with names not having one of a list of prefixes
passed on the command line.

11:45 AM Changeset in webkit [95654] by mitz@apple.com
  • 3 edits
    2 adds in trunk

<rdar://problem/9768483> REGRESSION: Crash in RenderBlock::removeFloatingObjectsBelow()
​https://bugs.webkit.org/show_bug.cgi?id=68550

Reviewed by Darin Adler.

Source/WebCore:

Test: fast/multicol/null-lastFloat-in-removeFloatingObjectsBelow.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::removeFloatingObjectsBelow): Break out of the while() loop when the
floating object set is empty, which can happen if there were no floats on previous lines
(in which case, lastFloat is 0).

LayoutTests:

  • fast/multicol/null-lastFloat-in-removeFloatingObjectsBelow-expected.txt: Added.
  • fast/multicol/null-lastFloat-in-removeFloatingObjectsBelow.html: Added.
11:38 AM Changeset in webkit [95653] by dslomov@google.com
  • 2 edits in trunk/LayoutTests

Fixed bug number in test expectations.

  • platform/chromium/test_expectations.txt:
11:33 AM Changeset in webkit [95652] by barraclough@apple.com
  • 8 edits in trunk/Source/JavaScriptCore

MacroAssembler fixes.
​https://bugs.webkit.org/show_bug.cgi?id=68494

Reviewed by Sam Weinig.

Add X86-64's 3 operand or32 to other MacroAssembler, fix load32's [const] void* mismatch

  • assembler/MacroAssembler.h:

(JSC::MacroAssembler::orPtr):
(JSC::MacroAssembler::loadPtr):

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::or32):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::or32):

  • assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::or32):

  • assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::or32):
(JSC::MacroAssemblerSH4::load32):

  • assembler/MacroAssemblerX86.h:

(JSC::MacroAssemblerX86::load32):

  • assembler/MacroAssemblerX86_64.h:

(JSC::MacroAssemblerX86_64::load32):

11:18 AM Changeset in webkit [95651] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[GTK] Fix build break when enabling webgl on r95593
​https://bugs.webkit.org/show_bug.cgi?id=68502

Patch by ChangSeok Oh <ChangSeok Oh> on 2011-09-21
Reviewed by Martin Robinson.

Add PassOwnPtr.h to GC3DPrivate.h and remove useless one from GC3DGtk.cpp & GC3DPrivate.cpp

No tests required.

  • platform/graphics/gtk/GraphicsContext3DGtk.cpp:
  • platform/graphics/gtk/GraphicsContext3DPrivate.cpp:
  • platform/graphics/gtk/GraphicsContext3DPrivate.h:
11:14 AM Changeset in webkit [95650] by sfalken@apple.com
  • 2 edits in trunk/Source/WebKit/win

Branch builds don't launch on Windows when run against trunk WebKit
​https://bugs.webkit.org/show_bug.cgi?id=68546

Reviewed by Darin Adler.

  • Interfaces/IWebPreferences.idl: Swap avFoundation and pictographicFontFamily

in IDL for binary compatibility between branch and trunk.

11:01 AM Changeset in webkit [95649] by vsevik@chromium.org
  • 3 edits
    2 adds in trunk

Web Inspector: clear resource agent resource cache upon resource agent's disable.
​https://bugs.webkit.org/show_bug.cgi?id=67995

Reviewed by Pavel Feldman.

Source/WebCore:

Test: http/tests/inspector/network/network-disabling-check-no-memory-leak.html

  • inspector/InspectorResourceAgent.cpp:

(WebCore::InspectorResourceAgent::disable):

LayoutTests:

  • http/tests/inspector/network/network-disabling-check-no-memory-leak-expected.txt: Added.
  • http/tests/inspector/network/network-disabling-check-no-memory-leak.html: Added.
10:58 AM Changeset in webkit [95648] by hyatt@apple.com
  • 9 edits
    2 adds in trunk

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

Make sure that when regions change dimensions we recompute pagination slices.

Reviewed by Simon Fraser.

Source/WebCore:

Added new tests in fast/regions.

  • rendering/LayoutState.cpp:

(WebCore::LayoutState::LayoutState):

  • rendering/LayoutState.h:
  • rendering/RenderFlowThread.cpp:

(WebCore::RenderFlowThread::layout):
(WebCore::RenderFlowThread::renderRegionForLine):
(WebCore::RenderFlowThread::regionLogicalWidthForLine):

  • rendering/RenderRegion.cpp:

(WebCore::RenderRegion::layout):

  • rendering/RenderView.cpp:

(WebCore::RenderView::pushLayoutState):

  • rendering/RenderView.h:

(WebCore::LayoutStateMaintainer::LayoutStateMaintainer):
(WebCore::LayoutStateMaintainer::push):

LayoutTests:

  • fast/regions/text-region-split-after-resize-expected.txt: Added.
  • fast/regions/text-region-split-after-resize.html: Added.
  • fast/regions/webkit-flow-floats-inside-regions-bounds-expected.txt:
10:37 AM Changeset in webkit [95647] by commit-queue@webkit.org
  • 8 edits
    2 adds in trunk

Ensure TestWebKitAPI works on mac, win, chromium-mac and chromium-linux
​https://bugs.webkit.org/show_bug.cgi?id=68206

Patch by Xianzhu Wang <​wangxianzhu@chromium.org> on 2011-09-21
Reviewed by Tony Chang.

Source/WebCore:

Added two new wtf headers into ForwardingHeaders/wtf to make
#include <wtf/...h> cross-platform.
No new tests needed.

  • ForwardingHeaders/wtf/MetaAllocator.h: Added.
  • ForwardingHeaders/wtf/RedBlackTree.h: Added.

Source/WebKit/chromium:

  • WebKit.gyp:

Tools:

  • Scripts/webkitdirs.pm:

(determineBaseProductDir): add support for chromium-mac

  • TestWebKitAPI/Tests/WTF/MetaAllocator.cpp:
  • TestWebKitAPI/Tests/WTF/RedBlackTree.cpp:
10:10 AM Changeset in webkit [95646] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[WinCairo] IconDatabase::defaultIcon always fails for non-CAN_THEME_URL_ICON builds.
​https://bugs.webkit.org/show_bug.cgi?id=67469

Patch by David Delaune <​david.delaune@googlemail.com> on 2011-09-21
Reviewed by Brent Fulgham.

No new functionality added in this change.

  • loader/icon/IconDatabase.cpp: defining CAN_THEME_URL_ICON for the WIN_CAIRO
10:06 AM Changeset in webkit [95645] by rniwa@webkit.org
  • 10 edits
    2 adds in trunk

Span element gets produced using backspace/delete to merge header with paragraph
​https://bugs.webkit.org/show_bug.cgi?id=68413

Reviewed by Darin Adler.

Source/WebCore:

Add p element to the list of elements to retain appearance. Also modified removeStyleFromNode
(and renamed to removeStyleFromRules) not to remove inline style declarations because
inline styles need to stay on copy.

Test: editing/deleting/merge-paragraph-from-p-with-style.html

  • editing/DeleteSelectionCommand.cpp:

(WebCore::DeleteSelectionCommand::saveTypingStyleState):

  • editing/EditingStyle.cpp:

(WebCore::EditingStyle::removeStyleFromRules): Renamed from removeStyleFromNode.

  • editing/EditingStyle.h:
  • editing/markup.cpp:

(WebCore::isBlockNodeToRetainAppearance):
(WebCore::StyledMarkupAccumulator::serializeNodes):

LayoutTests:

Add a test to remove the line break before a p element. WebKit should not preserve
the font color from the style rule but should preserve the font style from inline style declaration.

Also fix merge-paragraph-from-h6* to actually h6 instead of h1.

  • editing/deleting/merge-paragraph-from-h6-expected.txt:
  • editing/deleting/merge-paragraph-from-h6-with-style-expected.txt:
  • editing/deleting/merge-paragraph-from-h6-with-style.html:
  • editing/deleting/merge-paragraph-from-h6.html:
  • editing/deleting/merge-paragraph-from-p-with-style-expected.txt: Added.
  • editing/deleting/merge-paragraph-from-p-with-style.html: Added.
10:00 AM Changeset in webkit [95644] by ojan@chromium.org
  • 7 edits in trunk/LayoutTests

[flexbox] rename classnames and methods in the tests to not assume horizontal
​https://bugs.webkit.org/show_bug.cgi?id=68498

Reviewed by Tony Chang.

  • css3/flexbox/001.html:
  • css3/flexbox/002.html:
  • css3/flexbox/003.html:
  • css3/flexbox/004.html:
  • css3/flexbox/resources/flexbox.js:

(checkFlexBoxen):

  • css3/flexbox/writing-modes.html:
  • platform/mac/Skipped:
9:51 AM Changeset in webkit [95643] by reed@google.com
  • 2 edits in trunk/LayoutTests

need rebaseline for gradient speedup in skia
​https://bugs.webkit.org/show_bug.cgi?id=68531

Reviewed by Stephen White.

  • platform/chromium/test_expectations.txt:
9:48 AM Changeset in webkit [95642] by Philippe Normand
  • 3 edits in trunk/LayoutTests

Unreviewed, move today's skipped flaky tests to test_expectations
as suggested by mrobinson.

  • platform/gtk/Skipped:
  • platform/gtk/test_expectations.txt:
9:45 AM Changeset in webkit [95641] by commit-queue@webkit.org
  • 5 edits
    2 adds in trunk

Report AXValueChanged when value changes in element with role=textbox.
​https://bugs.webkit.org/show_bug.cgi?id=68201

Patch by Alice Boxhall <​aboxhall@chromium.org> on 2011-09-21
Reviewed by Chris Fleizach.

Source/WebCore:

Test: platform/mac/accessibility/textbox-role-reports-notifications.html

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::isARIATextControl):

  • accessibility/AccessibilityObject.h:
  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::contentChanged):
(WebCore::AccessibilityRenderObject::childrenChanged):

LayoutTests:

  • platform/mac/accessibility/textbox-role-reports-notifications-expected.txt: Added.
  • platform/mac/accessibility/textbox-role-reports-notifications.html: Added.
9:39 AM Changeset in webkit [95640] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

[chromium]ClearFocusedNode doesn't clear the cursor or selection in editable div element.
​https://bugs.webkit.org/show_bug.cgi?id=68351

Patch by chandra shekar vallala <​chandra.vallala@gmail.com> on 2011-09-21
Reviewed by Darin Fisher.

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::clearFocusedNode):

9:27 AM Changeset in webkit [95639] by annacc@chromium.org
  • 1 edit
    6 adds in trunk/LayoutTests

New layout tests for the JS bindings related to <track>:

e.g. TextTrack, TextTrackCue, TextTrackCueList, MutableTextTrack.

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

Reviewed by Eric Carlson.

  • media/track/track-mutable-expected.txt: Added.
  • media/track/track-mutable.html: Added.
  • media/track/track-text-track-cue-list-expected.txt: Added.
  • media/track/track-text-track-cue-list.html: Added.
  • media/track/track-text-track-expected.txt: Added.
  • media/track/track-text-track.html: Added.
9:19 AM Changeset in webkit [95638] by kbalazs@webkit.org
  • 1 edit in trunk/LayoutTests/platform/qt-wk2/Skipped

Actually skip the test (forgot in the previous commit). Sorry for the noise.

9:15 AM Changeset in webkit [95637] by kbalazs@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt][WK2] plugins/mouse-click-iframe-to-plugin.html fails since enabled in r95626
​https://bugs.webkit.org/show_bug.cgi?id=68539

Unreviewed gardening.

  • platform/qt-wk2/Skipped: skip the test.
9:13 AM Changeset in webkit [95636] by pfeldman@chromium.org
  • 7 edits in trunk/Source/WebCore

Web Inspector: introduce Page.enable and Page.disable
​https://bugs.webkit.org/show_bug.cgi?id=68530

Otherwise protocol is violating the contract of not sending
notifications by default.

Reviewed by Yury Semikhatsky.

  • inspector/Inspector.json:
8:51 AM Changeset in webkit [95635] by caryclark@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed; new baselines (Skia on Mac)

This separates test failures that are specific to CG Mac from failures
for Skia on Mac. Most of these changes are specific to Leopard, but a
handful add new fail flavors for all Mac variations.

  • LayoutTests/platform/chromium/test_expectations.txt:
8:39 AM Changeset in webkit [95634] by pfeldman@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: add minimal error handling to the DOM breakpoints.
​https://bugs.webkit.org/show_bug.cgi?id=68537

Reviewed by Yury Semikhatsky.

  • inspector/InspectorDOMAgent.h:
  • inspector/InspectorDOMDebuggerAgent.cpp:

(WebCore::domTypeForName):
(WebCore::InspectorDOMDebuggerAgent::setDOMBreakpoint):
(WebCore::InspectorDOMDebuggerAgent::removeDOMBreakpoint):

8:36 AM Changeset in webkit [95633] by Philippe Normand
  • 1 edit
    1 add in trunk/LayoutTests

Unreviewed, GTK baselines for
cross-origin-xsl-redirect-BLOCKED.html which were removed by mistake.

  • platform/gtk/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt: Added.
8:19 AM Changeset in webkit [95632] by loislo@chromium.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: make WebInspector a bit more timeline friendly.
​https://bugs.webkit.org/show_bug.cgi?id=68534

I found that the maximum captured stack frame depth tracked by timeline is not
very convenient because the average depth of call stack considerably more than five.
Another useful trick is console.timeStamp for all backend calls.

Reviewed by Yury Semikhatsky.

  • inspector/CodeGeneratorInspector.pm:
  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel.prototype._toggleTimelineButtonClicked):

8:10 AM Changeset in webkit [95631] by commit-queue@webkit.org
  • 11 edits
    2 adds in trunk/Source

[Qt][WK2] Implement Download support in WebProcess
​https://bugs.webkit.org/show_bug.cgi?id=68153

Patch by Jesus Sanchez-Palencia <​jesus.palencia@openbossa.org> on 2011-09-21
Reviewed by Andreas Kling.

Source/WebCore:

Refactored QNetworkReplyHandler::finish() in order to add
and use the static function QNetworkReplyHandler::errorForReply().
This will be used by our Download implementation in WebKit2 (WebProcess)
when handling ResourceError.

  • platform/network/qt/QNetworkReplyHandler.cpp:

(WebCore::QNetworkReplyHandler::finish):
(WebCore::QNetworkReplyHandler::errorForReply):

  • platform/network/qt/QNetworkReplyHandler.h:
  • platform/network/qt/ResourceRequest.h:

Source/WebKit2:

We implement the necessary functions of Download.h, and our QtFileDownloader
to handle all network communication and call the necessary functions of Download.

We use the download policy for any MIME type not supported by WebKit. This
behaves like Qt non-WebKit2 except that we don't force download when
we encounter Content-Disposition: attachment. We still use the "filename="
field for file name suggestion though.

Based on original patches by: Kimmo Kinnunen <​kimmo.t.kinnunen@nokia.com>,
Jocelyn Turcotte <​jocelyn.turcotte@nokia.com>, Simon Hausmann <​simon.hausmann@nokia.com>
and Zalan Bujtas <​zalan.bujtas@nokia.com>.

  • UIProcess/API/qt/qweberror.cpp: Adding DownloadError

(QWebError::type):

  • UIProcess/API/qt/qweberror.h: Adding DownloadError
  • WebKit2.pro:
  • WebProcess/Downloads/Download.cpp: Adding QtFileDownloader, Qt platform specific

(WebKit::Download::Download):

  • WebProcess/Downloads/Download.h: Adding QtFileDownloader, Qt platform specific
  • WebProcess/Downloads/qt/DownloadQt.cpp:

(WebKit::Download::start):
(WebKit::Download::startWithHandle):
(WebKit::Download::cancel):
(WebKit::Download::platformInvalidate):
(WebKit::Download::didDecideDestination):

  • WebProcess/Downloads/qt/QtFileDownloader.cpp: Added.

(WebKit::QtFileDownloader::QtFileDownloader):
(WebKit::QtFileDownloader::~QtFileDownloader):
(WebKit::QtFileDownloader::determineFilename):
(WebKit::QtFileDownloader::decidedDestination):
(WebKit::QtFileDownloader::abortDownloadWritingAndEmitError):
(WebKit::QtFileDownloader::onReadyRead):
(WebKit::QtFileDownloader::onFinished):
(WebKit::QtFileDownloader::onError):
(WebKit::QtFileDownloader::cancel):

  • WebProcess/Downloads/qt/QtFileDownloader.h: Added.
7:59 AM Changeset in webkit [95630] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip flaky media test on GTK.

  • platform/gtk/Skipped: Skip media/media-blocked-by-beforeload.html
7:58 AM Changeset in webkit [95629] by abarth@webkit.org
  • 2 edits in trunk/Tools

garden-o-matic command should explain that it is starting a local server
​https://bugs.webkit.org/show_bug.cgi?id=68486

Reviewed by Dimitri Glazkov.

In the user study, both participants didn't understand why the
garden-o-matic command appeared to hang. This patch adds an
explaination of what's going on. It also adds a confirmation prompt
before launching the browser to give the user a chance to read the
text. If that's too much of a speed bump, we can iterate.

  • Scripts/webkitpy/tool/commands/gardenomatic.py:
7:48 AM Changeset in webkit [95628] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Syntax error in media/audio-delete-while-step-button-clicked.html
​https://bugs.webkit.org/show_bug.cgi?id=68507

Reviewed by Eric Carlson.

  • media/audio-delete-while-step-button-clicked.html: Some curly

braces were missing in a try/catch statement.

7:28 AM Changeset in webkit [95627] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebKit/efl

[EFL] Fix alphabetical sorting problem
​https://bugs.webkit.org/show_bug.cgi?id=68505

Reviewed by Philippe Normand.

  • ewk/ewk_util.cpp: Fix wrong alphabetical sorting problem.
7:20 AM Changeset in webkit [95626] by commit-queue@webkit.org
  • 7 edits
    1 add in trunk

[Qt] Implement KeyDown function for WebKit2 EventSender.
​https://bugs.webkit.org/show_bug.cgi?id=56485

Tools:

Implement the KeyDown function on Qt platform.

Patch by Chang Shu <​cshu@webkit.org> on 2011-09-21
Reviewed by Andreas Kling.

  • WebKitTestRunner/PlatformWebView.h:
  • WebKitTestRunner/TestController.cpp:

(WTR::TestController::TestController):
(WTR::TestController::didReceiveSynchronousMessageFromInjectedBundle):

  • WebKitTestRunner/qt/EventSenderProxyQt.cpp: Added.

(WTR::getModifiers):
(WTR::EventSenderProxy::keyDown):
(buildModifierFlags):
(EventSenderProxy::keyDown):

  • WebKitTestRunner/qt/PlatformWebViewQt.cpp:

(WTR::PlatformWebView::sendEvent):

  • WebKitTestRunner/qt/WebKitTestRunner.pro:

LayoutTests:

Unskip passed tests.

Patch by Chang Shu <​cshu@webkit.org> on 2011-09-21
Reviewed by Andreas Kling.

  • platform/qt-wk2/Skipped:
7:06 AM Changeset in webkit [95625] by loislo@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Web Inspector: UI performance test for Resource panel first open experience.
​https://bugs.webkit.org/show_bug.cgi?id=68527

Reviewed by Pavel Feldman.

  • inspector/performance/resources/first-open-resources.html: Added.
7:04 AM Changeset in webkit [95624] by loislo@chromium.org
  • 2 edits
    1 add in trunk/LayoutTests

Web Inspector: extend UI perf tests coverage. Elements panel opens first time.
​https://bugs.webkit.org/show_bug.cgi?id=68432

Reviewed by Pavel Feldman.

  • inspector/performance/resources/first-open-elements.html: Added.
  • inspector/performance/resources/performance-test.js:
6:53 AM Changeset in webkit [95623] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip some more flaky SVG tests on GTK.

  • platform/gtk/Skipped:
6:34 AM Changeset in webkit [95622] by jochen@chromium.org
  • 3 edits
    4 copies
    2 moves
    5 adds
    1 delete in trunk/LayoutTests

Unreviewed, fix baselines for cross-origin-xsl-redirect-BLOCKED

  • http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/chromium-cg-mac/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/chromium-mac/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/chromium-win/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/gtk/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt: Removed
  • platform/mac-wk2/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/qt-wk2/Skipped:
  • platform/qt-wk2/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
  • platform/qt/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.png: Removed
  • platform/qt/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt: Removed
  • platform/win-wk2/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt:
6:33 AM Changeset in webkit [95621] by pfeldman@chromium.org
  • 9 edits in trunk/Source/WebCore

Web Inspector: paint box model colors in Metrics sidebar at all times, do not draw box outlines.
​https://bugs.webkit.org/show_bug.cgi?id=68240

Today we paint backgrounds in Metrics box model on hover only - should be painted at all
times for the reference. Outlining boxes is highlight is wrong since outlines are outside
the corresponding box regions. We've seen few reports on that + Firebug does not do borders
for that reason.

Reviewed by Yury Semikhatsky.

  • inspector/DOMNodeHighlighter.cpp:
  • inspector/DOMNodeHighlighter.h:
  • inspector/Inspector.json:
  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::setHighlightDataFromConfig):

  • inspector/front-end/Color.js:
  • inspector/front-end/MetricsSidebarPane.js:

(WebInspector.MetricsSidebarPane.prototype._highlightDOMNode):
(WebInspector.MetricsSidebarPane.prototype._updateMetrics):

  • inspector/front-end/inspector.css:

(.metrics .label):

  • inspector/front-end/inspector.js:

(WebInspector.buildHighlightConfig):

6:14 AM Changeset in webkit [95620] by keishi@webkit.org
  • 1 edit
    5 adds in trunk/LayoutTests

[chromium] rebaseline caused by r95567

  • platform/chromium-cg-mac-leopard/fast/clip/overflow-border-radius-fixed-position-expected.png: Added.
  • platform/chromium-mac-leopard: Added.
  • platform/chromium-mac-leopard/fast: Added.
  • platform/chromium-mac-leopard/fast/clip: Added.
  • platform/chromium-mac-leopard/fast/clip/overflow-border-radius-fixed-position-expected.png: Added.
6:08 AM Changeset in webkit [95619] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Protect against misuse of EventListenerIterator.
​https://bugs.webkit.org/show_bug.cgi?id=68364

Patch by Andreas Kling <​kling@webkit.org> on 2011-09-21
Reviewed by Darin Adler.

In debug mode, keep track of the number of active EventListenerIterators
on an EventListenerMap, and assert that there are no iterators when the
map is being modified.

  • dom/EventListenerMap.cpp:

(WebCore::EventListenerMap::EventListenerMap):
(WebCore::EventListenerMap::clear):
(WebCore::EventListenerMap::add):
(WebCore::EventListenerMap::remove):
(WebCore::EventListenerMap::find):
(WebCore::EventListenerMap::removeFirstEventListenerCreatedFromMarkup):
(WebCore::EventListenerMap::copyEventListenersNotCreatedFromMarkupToTarget):
(WebCore::EventListenerIterator::EventListenerIterator):
(WebCore::EventListenerIterator::~EventListenerIterator):

  • dom/EventListenerMap.h:
6:05 AM Changeset in webkit [95618] by keishi@webkit.org
  • 2 edits
    4 adds
    2 deletes in trunk/LayoutTests

[chromium] rebaseline caused by r95599

  • fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.txt: Added.
  • platform/chromium-mac-leopard: Removed.
  • platform/chromium-mac/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.png: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.png: Added.
  • platform/chromium-win/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.txt: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.txt: Removed.
5:52 AM Changeset in webkit [95617] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, skip 2 GTK flaky tests.

  • platform/gtk/Skipped: Skip

http/tests/websocket/tests/hixie76/error-detect.html and
svg/zoom/page/zoom-svg-through-object-with-auto-size.html

5:13 AM Changeset in webkit [95616] by alexis.menard@openbossa.org
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Unskip some cookies related layout tests.
​https://bugs.webkit.org/show_bug.cgi?id=68484

Reviewed by Csaba Osztrogonác.

Unskip some cookies related layout tests as they seems to work fine with
--iterations 1000.

  • platform/qt-wk2/Skipped:
4:57 AM Changeset in webkit [95615] by rgabor@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt][Mac] Unreviewed gardening after r95601.

  • platform/qt-mac/Skipped: Paint the Qt-Mac bot green.
4:50 AM Changeset in webkit [95614] by pfeldman@chromium.org
  • 5 edits in trunk/Source/WebCore

Web Inspector: minor fixes in the DOM domain of the protocol.
​https://bugs.webkit.org/show_bug.cgi?id=68517

Reviewed by Yury Semikhatsky.

  • inspector/Inspector.json:
  • inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::setAttributesAsText):

  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/DOMAgent.js:

(WebInspector.DOMNode.prototype.setAttribute):

4:45 AM Changeset in webkit [95613] by pfeldman@chromium.org
  • 6 edits in trunk/Source/WebCore

Web Inspector: limit the use of "Resource" term in the protocol.
​https://bugs.webkit.org/show_bug.cgi?id=68250

Reviewed by Yury Semikhatsky.

  • inspector/Inspector.json:
  • inspector/InspectorResourceAgent.cpp:

(WebCore::InspectorResourceAgent::markResourceAsCached):
(WebCore::InspectorResourceAgent::didLoadResourceFromMemoryCache):
(WebCore::InspectorResourceAgent::setExtraHTTPHeaders):
(WebCore::InspectorResourceAgent::getResponseBody):

  • inspector/InspectorResourceAgent.h:
  • inspector/front-end/ExtensionServer.js:

(WebInspector.ExtensionServer.prototype._onAddRequestHeaders):

  • inspector/front-end/NetworkManager.js:

(WebInspector.NetworkManager.prototype.requestContent):
(WebInspector.NetworkDispatcher.prototype.requestServedFromCache):
(WebInspector.NetworkDispatcher.prototype.requestServedFromMemoryCache):

4:39 AM Changeset in webkit [95612] by commit-queue@webkit.org
  • 8 edits
    12 deletes in trunk

Unreviewed, rolling out r95608.
​http://trac.webkit.org/changeset/95608
​https://bugs.webkit.org/show_bug.cgi?id=68521

breaks GTK and Windows builds. Please check EWS next time.
(Requested by philn-tp on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-09-21

Source/WebCore:

  • html/HTMLBodyElement.cpp:

(WebCore::adjustForZoom):
(WebCore::HTMLBodyElement::setScrollLeft):
(WebCore::HTMLBodyElement::setScrollTop):

  • loader/HistoryController.cpp:

(WebCore::HistoryController::saveScrollPositionAndViewStateToItem):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::scrollX):
(WebCore::DOMWindow::scrollY):
(WebCore::DOMWindow::scrollTo):

  • page/Frame.cpp:

(WebCore::Frame::pageScaleFactor):

  • page/Frame.h:
  • page/FrameView.cpp:

(WebCore::FrameView::applyOverflowToViewport):
(WebCore::FrameView::scrollXForFixedPosition):
(WebCore::FrameView::scrollYForFixedPosition):

LayoutTests:

  • fast/events/scale-and-scroll-body-expected.png: Removed.
  • fast/events/scale-and-scroll-body-expected.txt: Removed.
  • fast/events/scale-and-scroll-body.html: Removed.
  • fast/events/scale-and-scroll-iframe-body-expected.png: Removed.
  • fast/events/scale-and-scroll-iframe-body-expected.txt: Removed.
  • fast/events/scale-and-scroll-iframe-body.html: Removed.
  • fast/events/scale-and-scroll-iframe-window-expected.png: Removed.
  • fast/events/scale-and-scroll-iframe-window-expected.txt: Removed.
  • fast/events/scale-and-scroll-iframe-window.html: Removed.
  • fast/events/scale-and-scroll-window-expected.png: Removed.
  • fast/events/scale-and-scroll-window-expected.txt: Removed.
  • fast/events/scale-and-scroll-window.html: Removed.
4:37 AM Changeset in webkit [95611] by pfeldman@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: console panel's scroller is not properly aligned.
​https://bugs.webkit.org/show_bug.cgi?id=68448

Reviewed by Yury Semikhatsky.

  • inspector/front-end/inspector.css:

(#console-view):
(#console-messages):
(#drawer #console-messages):

4:25 AM Changeset in webkit [95610] by Philippe Normand
  • 2 edits
    4 adds in trunk/LayoutTests

Unreviewed, GTK baseslines for 2 new tests and skip a flaky svg test.

  • platform/gtk/Skipped: Skip svg/W3C-SVG-1.1-SE/filters-image-05-f.svg
  • platform/gtk/fast/clip/overflow-border-radius-fixed-position-expected.png: Added.
  • platform/gtk/fast/clip/overflow-border-radius-fixed-position-expected.txt: Added.
  • platform/gtk/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.png: Added.
  • platform/gtk/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.txt: Added.
4:16 AM Changeset in webkit [95609] by rgabor@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt][WK2] http/tests/security/cross-origin-xsl-redirect-BLOCKED.html fails
​https://bugs.webkit.org/show_bug.cgi?id=68515

  • platform/qt-wk2/Skipped: Skip http/tests/security/cross-origin-xsl-redirect-BLOCKED.html.
4:01 AM Changeset in webkit [95608] by hclam@chromium.org
  • 8 edits
    12 adds in trunk

Source/WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=68081
Fix scroll in page scaling mode.

Reviewed by Simon Fraser.

The following things are done in this patch:

  1. Rename Frame::pageScaleFactor() to Frame::frameScaleFactor() to better reflect its purpose.
  2. Frame::frameScaleFactor() returns the scale factor of this frame with respect to the container. So for the main frame it'll return the scale factor of the page, inner frames will return 1.0
  3. scrollBy(), scrollX() and scrollY() to take into account of the frame scale factor, causing incorrect coordinates reported to Javascript.

Tests: fast/events/scale-and-scroll-body.html

fast/events/scale-and-scroll-iframe-body.html
fast/events/scale-and-scroll-iframe-window.html
fast/events/scale-and-scroll-window.html

  • html/HTMLBodyElement.cpp:

(WebCore::adjustForZoom):
(WebCore::HTMLBodyElement::setScrollLeft):
(WebCore::HTMLBodyElement::setScrollTop):

  • loader/HistoryController.cpp:

(WebCore::HistoryController::saveScrollPositionAndViewStateToItem):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::scrollX):
(WebCore::DOMWindow::scrollY):
(WebCore::DOMWindow::scrollTo):

  • page/Frame.cpp:

(WebCore::Frame::frameScaleFactor): Return page's scale factor if this is

main frame, otherwise 1.0

  • page/Frame.h:
  • page/FrameView.cpp:

(WebCore::FrameView::applyOverflowToViewport):
(WebCore::FrameView::scrollXForFixedPosition):
(WebCore::FrameView::scrollYForFixedPosition):

LayoutTests: ​https://bugs.webkit.org/show_bug.cgi?id=68081
Add layout test for page scale and scrolling.

Reviewed by Simon Fraser.

Add tests to cover the following cases:

  1. Scale and scroll window
  2. Scale and scroll document.body
  3. Scale and scroll iframe's contentWindow
  4. Scale and scroll iframe's contentDocument.body
  • fast/events/scale-and-scroll-body-expected.png: Added.
  • fast/events/scale-and-scroll-body-expected.txt: Added.
  • fast/events/scale-and-scroll-body.html: Added.
  • fast/events/scale-and-scroll-iframe-body-expected.png: Added.
  • fast/events/scale-and-scroll-iframe-body-expected.txt: Added.
  • fast/events/scale-and-scroll-iframe-body.html: Added.
  • fast/events/scale-and-scroll-iframe-window-expected.png: Added.
  • fast/events/scale-and-scroll-iframe-window-expected.txt: Added.
  • fast/events/scale-and-scroll-iframe-window.html: Added.
  • fast/events/scale-and-scroll-window-expected.png: Added.
  • fast/events/scale-and-scroll-window-expected.txt: Added.
  • fast/events/scale-and-scroll-window.html: Added.
3:16 AM Changeset in webkit [95607] by mrowe@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9890932> "Open PDF in Preview" on two PDFs of the same name does nothing the second time

The logic in PDFViewController::pathToPDFOnDisk that deals with constructing a unique
file name if a file of the suggested name already exists was incorrect. It was appending
the template used by mkstemps to the directory name component of the path rather than
using it as a prefix on the filename portion of the path. This resulted in a template path
like WebKitPDFs-ABCDEFXXXXXX-/foo.pdf where WebKitPDFs-ABCDEF/XXXXXX-foo.pdf was expected.

Reviewed by Jon Honeycutt.

  • UIProcess/API/mac/PDFViewController.mm:

(WebKit::PDFViewController::pathToPDFOnDisk): Append the template as a new path component,
then append the suggested file name to that component.

2:49 AM Changeset in webkit [95606] by Philippe Normand
  • 4 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline, skip some flaky worker tests and 2
failing http tests.

  • platform/gtk/Skipped:
  • platform/gtk/editing/deleting/regional-indicators-expected.txt:
  • platform/gtk/fast/text/justify-ideograph-complex-expected.txt:
2:19 AM Changeset in webkit [95605] by Philippe Normand
  • 3 edits
    1 add in trunk/LayoutTests

Unreviewed, GTK rebaseline and skip and a11y failing test.

  • platform/gtk/Skipped: Skip accessibility/aria-hidden-updates-alldescendants.html
  • platform/gtk/editing/deleting/merge-whitespace-pre-expected.txt:

Rebaseline after r95335.

  • platform/gtk/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt: Added.
1:59 AM Changeset in webkit [95604] by abecsi@webkit.org
  • 18 edits in trunk/Source

[Qt] Remove Qt specific code from css/SelectorChecker.cpp
​https://bugs.webkit.org/show_bug.cgi?id=67702

Reviewed by Csaba Osztrogonác.

Source/WebCore:

No new tests needed.

  • css/SelectorChecker.cpp: Move Qt-specific code to the Qt-specific WebPlatformStrategies.

(WebCore::SelectorChecker::determineLinkStateSlowCase):

  • platform/VisitedLinkStrategy.h:

Source/WebKit/mac:

  • WebCoreSupport/WebPlatformStrategies.h:
  • WebCoreSupport/WebPlatformStrategies.mm:

(WebPlatformStrategies::isLinkVisited):

Source/WebKit/qt:

  • WebCoreSupport/WebPlatformStrategies.cpp: Check the Qt global history here, instead of a platform independent file.

(WebPlatformStrategies::isLinkVisited):

  • WebCoreSupport/WebPlatformStrategies.h:

Source/WebKit/win:

  • WebCoreSupport/WebPlatformStrategies.cpp: Modify to match new prototype.

(WebPlatformStrategies::isLinkVisited):

  • WebCoreSupport/WebPlatformStrategies.h: Ditto.

Source/WebKit/wince:

  • WebCoreSupport/PlatformStrategiesWinCE.cpp:

(PlatformStrategiesWinCE::addVisitedLink):

  • WebCoreSupport/PlatformStrategiesWinCE.h:

Source/WebKit2:

  • WebProcess/WebCoreSupport/WebPlatformStrategies.cpp: Modify to match new prototype.

(WebKit::WebPlatformStrategies::isLinkVisited):
(WebKit::WebPlatformStrategies::addVisitedLink):

  • WebProcess/WebCoreSupport/WebPlatformStrategies.h: Ditto.
1:41 AM Changeset in webkit [95603] by Philippe Normand
  • 2 edits in trunk/LayoutTests

Unreviewed, GTK rebaseline after r95566.

  • platform/gtk/mathml/xHeight-expected.txt:
12:09 AM Changeset in webkit [95602] by morrita@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed Skipped list update for meter-element-repaint-on-update-value.html,
which needs windows

  • platform/win/Skipped:
12:01 AM Changeset in webkit [95601] by Csaba Osztrogonác
  • 3 edits
    6 adds in trunk/LayoutTests

[Qt] Unreviewed gardening.

  • platform/qt-mac/Skipped:
  • platform/qt-wk2/Skipped:
  • platform/qt/fast/clip/overflow-border-radius-fixed-position-expected.png: Added.
  • platform/qt/fast/clip/overflow-border-radius-fixed-position-expected.txt: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.png: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-repaint-on-update-value-expected.txt: Added.
  • platform/qt/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.png: Added.
  • platform/qt/http/tests/security/cross-origin-xsl-redirect-BLOCKED-expected.txt: Added.
Note: See TracTimeline for information about the timeline view.