Context Navigation

← Previous Period
Next Period →

Timeline

Nov 1, 2012:

11:35 PM Changeset in webkit [133268] by shinyak@chromium.org

6 edits
2 adds in trunk

[Shadow] Element should have getter and setter of attribute 'pseudo'
https://bugs.webkit.org/show_bug.cgi?id=100831

Reviewed by Hajime Morita.

Source/WebCore:

We expose 'pseudo' attribute in Element. When nothing is assigned to 'pseudo', it should return null.

Test: fast/dom/shadow/pseudo-attribute.html

dom/Element.cpp:

(WebCore::Element::pseudo):
(WebCore):
(WebCore::Element::setPseudo):

dom/Element.h:

(Element):

dom/Element.idl:
html/HTMLAttributeNames.in:

LayoutTests:

fast/dom/shadow/pseudo-attribute-expected.txt: Added.
fast/dom/shadow/pseudo-attribute.html: Added.

11:29 PM Changeset in webkit [133267] by commit-queue@webkit.org

2 edits in trunk/Tools

[EFL][WK2] Make File Chooser dialog modal in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=100962

Patch by Christophe Dumez <Christophe Dumez> on 2012-11-01
Reviewed by Gyuyoung Kim.

Make File Chooser dialog modal in MiniBrowser and make
sure it is closed and freed when its parent window is
closed.

MiniBrowser/efl/main.c:

(close_file_picker):
(on_filepicker_parent_deletion):
(on_file_chooser_request):

10:54 PM Changeset in webkit [133266] by commit-queue@webkit.org

11 edits in trunk

[EFL][DRT] Support Geolocation
https://bugs.webkit.org/show_bug.cgi?id=97427

Source/WebKit:

Patch by Seonae Kim <sunaeluv.kim@samsung.com> on 2012-11-01
Reviewed by Gyuyoung Kim.

Add a mock directory to use GeolocationClientMock in DumpRenderTree.

PlatformEfl.cmake:

Source/WebKit/efl:

Patch by Seonae Kim <sunaeluv.kim@samsung.com> on 2012-11-01
Reviewed by Gyuyoung Kim.

Implement methods to use GeolocationClientMock in DumpRenderTree.
This client attaches to the page whenever a view is created.

WebCoreSupport/DumpRenderTreeSupportEfl.cpp:

(DumpRenderTreeSupportEfl::attachClientToPage):
(DumpRenderTreeSupportEfl::resetGeolocationClientMock):
(DumpRenderTreeSupportEfl::setMockGeolocationPermission):
(DumpRenderTreeSupportEfl::setMockGeolocationPosition):
(DumpRenderTreeSupportEfl::setMockGeolocationPositionUnavailableError):
(DumpRenderTreeSupportEfl::numberOfPendingGeolocationPermissionRequests):

WebCoreSupport/DumpRenderTreeSupportEfl.h:

Tools:

Patch by Seonae Kim <sunaeluv.kim@samsung.com> on 2012-11-01
Reviewed by Gyuyoung Kim.

Implement methods to use GeolocationClientMock in DumpRenderTree.
This client attaches to the page whenever a view is created.

DumpRenderTree/efl/DumpRenderTreeChrome.cpp:

(DumpRenderTreeChrome::createView):
(DumpRenderTreeChrome::resetDefaultsToConsistentValues):

DumpRenderTree/efl/TestRunnerEfl.cpp:

(TestRunner::setMockGeolocationPosition):
(TestRunner::setMockGeolocationPositionUnavailableError):
(TestRunner::setGeolocationPermission):
(TestRunner::numberOfPendingGeolocationPermissionRequests):

LayoutTests:

Patch by Seonae Kim <sunaeluv.kim@samsung.com> on 2012-11-01
Reviewed by Gyuyoung Kim.

Change the comment of Geolocation test cases.

platform/efl/TestExpectations:

9:41 PM Changeset in webkit [133265] by mark.lam@apple.com

2 edits in trunk/Source/JavaScriptCore

A llint workaround for a toolchain issue.
https://bugs.webkit.org/show_bug.cgi?id=101012.

Reviewed by Michael Saboff.

llint/LowLevelInterpreter.asm:
- use a local label to workaround the toolchain issue with undeclared global labels.

8:09 PM Changeset in webkit [133264] by Simon Fraser

100 edits in trunk/LayoutTests

New image results for compositing tests. Cross-platform image results seem
impossible to maintain, so these are all Mac-specific.

platform/mac/compositing/animation/busy-indicator-expected.png:
platform/mac/compositing/animation/state-at-end-event-transform-layer-expected.png:
platform/mac/compositing/color-matching/image-color-matching-expected.png:
platform/mac/compositing/color-matching/pdf-image-match-expected.png:
platform/mac/compositing/compositing-visible-descendant-expected.png:
platform/mac/compositing/direct-image-compositing-expected.png:
platform/mac/compositing/flat-with-transformed-child-expected.png:
platform/mac/compositing/framesets/composited-frame-alignment-expected.png:
platform/mac/compositing/generated-content-expected.png:
platform/mac/compositing/geometry/abs-position-inside-opacity-expected.png:
platform/mac/compositing/geometry/clipping-foreground-expected.png:
platform/mac/compositing/geometry/composited-html-size-expected.png:
platform/mac/compositing/geometry/fixed-in-composited-expected.png:
platform/mac/compositing/geometry/fixed-position-composited-page-scale-down-expected.png:
platform/mac/compositing/geometry/fixed-position-composited-page-scale-expected.png:
platform/mac/compositing/geometry/fixed-position-expected.png:
platform/mac/compositing/geometry/fixed-position-iframe-composited-page-scale-down-expected.png:
platform/mac/compositing/geometry/fixed-position-iframe-composited-page-scale-expected.png:
platform/mac/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png:
platform/mac/compositing/geometry/fixed-position-transform-composited-page-scale-expected.png:
platform/mac/compositing/geometry/foreground-layer-expected.png:
platform/mac/compositing/geometry/horizontal-scroll-composited-expected.png:
platform/mac/compositing/geometry/layer-due-to-layer-children-deep-expected.png:
platform/mac/compositing/geometry/layer-due-to-layer-children-expected.png:
platform/mac/compositing/geometry/limit-layer-bounds-overflow-repaint-expected.png:
platform/mac/compositing/geometry/outline-change-expected.png:
platform/mac/compositing/geometry/partial-layout-update-expected.png:
platform/mac/compositing/geometry/repaint-foreground-layer-expected.png:
platform/mac/compositing/geometry/root-layer-update-expected.png:
platform/mac/compositing/geometry/tall-page-composited-expected.png:
platform/mac/compositing/geometry/transfrom-origin-on-zero-size-layer-expected.png:
platform/mac/compositing/geometry/vertical-scroll-composited-expected.png:
platform/mac/compositing/geometry/video-fixed-scrolling-expected.png:
platform/mac/compositing/geometry/video-opacity-overlay-expected.png:
platform/mac/compositing/iframes/composited-iframe-alignment-expected.png:
platform/mac/compositing/iframes/iframe-copy-on-scroll-expected.png:
platform/mac/compositing/iframes/iframe-in-composited-layer-expected.png:
platform/mac/compositing/images/content-image-change-expected.png:
platform/mac/compositing/images/direct-image-background-color-expected.png:
platform/mac/compositing/layer-creation/fixed-position-scroll-expected.png:
platform/mac/compositing/layer-creation/spanOverlapsCanvas-expected.png:
platform/mac/compositing/layers-inside-overflow-scroll-expected.png:
platform/mac/compositing/masks/direct-image-mask-expected.png:
platform/mac/compositing/masks/masked-ancestor-expected.png:
platform/mac/compositing/masks/multiple-masks-expected.png:
platform/mac/compositing/masks/simple-composited-mask-expected.png:
platform/mac/compositing/overflow/ancestor-overflow-expected.png:
platform/mac/compositing/overflow/fixed-position-ancestor-clip-expected.png:
platform/mac/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/mac/compositing/overflow/overflow-positioning-expected.png:
platform/mac/compositing/overflow/overflow-scaled-descendant-overlapping-expected.png:
platform/mac/compositing/overflow/overflow-scroll-expected.png:
platform/mac/compositing/overflow/parent-overflow-expected.png:
platform/mac/compositing/overflow/scroll-ancestor-update-expected.png:
platform/mac/compositing/overflow/scrollbar-painting-expected.png:
platform/mac/compositing/overflow/theme-affects-visual-overflow-expected.png:
platform/mac/compositing/plugins/composited-plugin-expected.png:
platform/mac/compositing/reflections/animation-inside-reflection-expected.png:
platform/mac/compositing/reflections/compositing-change-inside-reflection-expected.png:
platform/mac/compositing/reflections/deeply-nested-reflections-expected.png:
platform/mac/compositing/reflections/load-video-in-reflection-expected.png:
platform/mac/compositing/reflections/masked-reflection-on-composited-expected.png:
platform/mac/compositing/reflections/nested-reflection-anchor-point-expected.png:
platform/mac/compositing/reflections/nested-reflection-animated-expected.png:
platform/mac/compositing/reflections/nested-reflection-expected.png:
platform/mac/compositing/reflections/nested-reflection-mask-change-expected.png:
platform/mac/compositing/reflections/nested-reflection-on-overflow-expected.png:
platform/mac/compositing/reflections/nested-reflection-opacity-expected.png:
platform/mac/compositing/reflections/nested-reflection-size-change-expected.png:
platform/mac/compositing/reflections/nested-reflection-transformed-expected.png:
platform/mac/compositing/reflections/nested-reflection-transformed2-expected.png:
platform/mac/compositing/reflections/nested-reflection-transition-expected.png:
platform/mac/compositing/reflections/reflection-in-composited-expected.png:
platform/mac/compositing/reflections/reflection-on-composited-expected.png:
platform/mac/compositing/reflections/reflection-opacity-expected.png:
platform/mac/compositing/reflections/reflection-ordering-expected.png:
platform/mac/compositing/reflections/reflection-positioning-expected.png:
platform/mac/compositing/reflections/reflection-positioning2-expected.png:
platform/mac/compositing/reflections/simple-composited-reflections-expected.png:
platform/mac/compositing/reflections/transform-inside-reflection-expected.png:
platform/mac/compositing/repaint/become-overlay-composited-layer-expected.png:
platform/mac/compositing/repaint/composited-document-element-expected.png:
platform/mac/compositing/repaint/content-into-overflow-expected.png:
platform/mac/compositing/repaint/layer-repaint-expected.png:
platform/mac/compositing/repaint/layer-repaint-rects-expected.png:
platform/mac/compositing/repaint/opacity-between-absolute-expected.png:
platform/mac/compositing/repaint/opacity-between-absolute2-expected.png:
platform/mac/compositing/repaint/overflow-into-content-expected.png:
platform/mac/compositing/repaint/same-size-invalidation-expected.png:
platform/mac/compositing/scaling/tiled-layer-recursion-expected.png:
platform/mac/compositing/self-painting-layers-expected.png:
platform/mac/compositing/shadows/shadow-drawing-expected.png:
platform/mac/compositing/sibling-positioning-expected.png:
platform/mac/compositing/text-on-large-layer-expected.png:
platform/mac/compositing/transitions/scale-transition-no-start-expected.png:
platform/mac/compositing/transitions/singular-scale-transition-expected.png:
platform/mac/compositing/webgl/webgl-background-color-expected.png:
platform/mac/compositing/webgl/webgl-no-alpha-expected.png:
platform/mac/compositing/webgl/webgl-reflection-expected.png:

8:00 PM Changeset in webkit [133263] by Simon Fraser

9 edits in trunk/LayoutTests

Rebaseline some MathML results, and mark compositing/iframes/overlapped-nested-iframes.html
as flakey because it is.

platform/mac/TestExpectations:
platform/mac/mathml/presentation/mo-stretch-expected.txt:
platform/mac/mathml/presentation/row-alignment-expected.png:
platform/mac/mathml/presentation/row-alignment-expected.txt:
platform/mac/mathml/presentation/subsup-expected.png:
platform/mac/mathml/presentation/subsup-expected.txt:
platform/mac/mathml/presentation/tables-expected.png:
platform/mac/mathml/presentation/tables-expected.txt:

7:53 PM Changeset in webkit [133262] by roger_fong@apple.com

3 edits in trunk/Source/WebCore

Build fix. http://trac.webkit.org/changeset/133252 broke the Windows build.
https://bugs.webkit.org/show_bug.cgi?id=101008

Reviewed by fischman@chromium.org.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::clearMediaPlayer):

html/HTMLMediaElement.h:

(HTMLMediaElement):

7:50 PM Changeset in webkit [133261] by senorblanco@chromium.org

1 edit
1 move
4 adds in trunk/LayoutTests

[Chromium] Unreviewed gardening.

platform/chromium/TestExpectations:

platform/chromium-mac-lion/svg/dynamic-updates/SVGUseElement-dom-requiredFeatures-expected.png: Added.
platform/efl-wk1/svg/dynamic-updates/SVGUseElement-dom-requiredFeatures-expected.png: Renamed from LayoutTests/platform/efl/svg/dynamic-updates/SVGUseElement-dom-requiredFeatures-expected.png.

7:44 PM Changeset in webkit [133260] by rniwa@webkit.org

3 edits in trunk/Tools

Remove tests no longer needed after r133229.

Scripts/webkitpy/performance_tests/perftest_unittest.py:

(TestPerfTestFactory.test_inspector_test):

Scripts/webkitpy/performance_tests/perftestsrunner_unittest.py:

(test_collect_tests_with_skipped_list):

7:42 PM Changeset in webkit [133259] by Dave Barton

2 edits
12 adds in trunk/LayoutTests

Unreviewed mathml rebaseline.

platform/chromium-linux-x86/mathml: Added.
platform/chromium-linux-x86/mathml/xHeight-expected.png: Added.
platform/chromium-linux-x86/mathml/xHeight-expected.txt: Added.
platform/chromium-linux/mathml: Added.
platform/chromium-linux/mathml/xHeight-expected.png: Added.
platform/chromium-linux/mathml/xHeight-expected.txt: Added.
platform/chromium-win-xp/mathml: Added.
platform/chromium-win-xp/mathml/xHeight-expected.png: Added.
platform/chromium-win-xp/mathml/xHeight-expected.txt: Added.
platform/chromium-win/mathml: Added.
platform/chromium-win/mathml/xHeight-expected.png: Added.
platform/chromium-win/mathml/xHeight-expected.txt: Added.
platform/chromium/TestExpectations:

7:32 PM Changeset in webkit [133258] by keishi@webkit.org

3 edits
2 copies in branches/chromium/1312

Merge 132985 - F4 inside <input type=time> should not open calendar picker
https://bugs.webkit.org/show_bug.cgi?id=100730

Reviewed by Kent Tamura.

Source/WebCore:

We lacked the check to see if the picker indicator is visible.

Test: fast/forms/time-multiple-fields/time-multiple-fields-open-picker-key-bindings.html

html/BaseMultipleFieldsDateAndTimeInputType.cpp:

(WebCore::BaseMultipleFieldsDateAndTimeInputType::handleKeydownEvent): Check if picker indicator is visible.

LayoutTests:

fast/forms/time-multiple-fields/time-multiple-fields-open-picker-key-bindings-expected.txt: Added.
fast/forms/time-multiple-fields/time-multiple-fields-open-picker-key-bindings.html: Added.

TBR=keishi@webkit.org
Review URL: https://codereview.chromium.org/11369041

7:22 PM Changeset in webkit [133257] by keishi@webkit.org

8 edits in branches/chromium/1312

Merge 133011 - Calendar picker can flicker when opened from the suggestion picker
https://bugs.webkit.org/show_bug.cgi?id=100816

Reviewed by Kent Tamura.

Source/WebCore:

Calendar picker was opening before the hiding completed.

No new tests. Added test to date-suggestion-picker-key-operations.html.

Resources/pagepopups/pickerCommon.js: Added didHide event.
Resources/pagepopups/suggestionPicker.js:

(SuggestionPicker.prototype.selectEntry): Use didHide event so we don't open the calendar picker prematurely.
(SuggestionPicker._handleWindowDidHide): When the window finished hiding, open the calendar picker.

LayoutTests:

fast/forms/resources/picker-common.js:

(openPicker): Fix mistake.

platform/chromium-win/platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations-expected.txt:
platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations-expected.txt:
platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations.html: Test that didHide event fires after the entry was selected and before the calendar picker opened.

TBR=keishi@webkit.org
Review URL: https://codereview.chromium.org/11364040

6:59 PM Changeset in webkit [133256] by senorblanco@chromium.org

3 edits in trunk/Source/WebCore

Unreviewed, rolling out r133244.
http://trac.webkit.org/changeset/133244
https://bugs.webkit.org/show_bug.cgi?id=100973

Broke Chromium Mac (clang) builds.

bindings/v8/DOMDataStore.cpp:

(WebCore::DOMDataStore::DOMDataStore):

bindings/v8/IntrusiveDOMWrapperMap.h:

(WebCore::DOMNodeWrapperMap::weakCallback):

6:43 PM Changeset in webkit [133255] by oliver@apple.com

5 edits in trunk/Source/JavaScriptCore

Remove GlobalObject constant register that is typically unused
https://bugs.webkit.org/show_bug.cgi?id=101005

Reviewed by Geoffrey Garen.

The GlobalObject constant register is frequently allocated even when it
is not used, it is also getting in the way of some other optimisations.

bytecode/CodeBlock.cpp:

(JSC::CodeBlock::CodeBlock):

bytecode/CodeBlock.h:

(CodeBlock):

bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):

dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseResolveOperations):

6:42 PM Changeset in webkit [133254] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Mark another flaky test.

platform/chromium/TestExpectations:

6:40 PM Changeset in webkit [133253] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Simplify the code of returning JSValue in "BlackBerry::WebKit::WebPagePrivate::executeJavaScript"
https://bugs.webkit.org/show_bug.cgi?id=100941

Patch by Peter Wang <peter.wang@torchmobile.com.cn> on 2012-11-01
Reviewed by Yong Li.

Remove the unnecessary steps, just directly deal with returned JSValue.

Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::executeJavaScript):
(BlackBerry::WebKit::WebPagePrivate::executeJavaScriptInIsolatedWorld):

6:38 PM Changeset in webkit [133252] by fischman@chromium.org

4 edits
1 add in trunk

HTMLMediaPlayer should free m_player when src is set/changed
https://bugs.webkit.org/show_bug.cgi?id=99647

Reviewed by Eric Carlson.

ManualTests/media-players-are-dropped-on-error.html: Added.

Various scenarios are tested to make sure players aren't
leaked in different ways for each of them.

Source/WebCore:

New ManualTest added; manual since leaking media players doesn't have layoutTestController-visible effects.

html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::parseAttribute): clearMediaPlayer() when src is set/changed
(WebCore::HTMLMediaElement::userCancelledLoad): use new clearMediaPlayer() helper
(WebCore::HTMLMediaElement::clearMediaPlayer): clear m_player and associated timers/flags
(WebCore):
(WebCore::HTMLMediaElement::createMediaPlayer): whitespace-only change

html/HTMLMediaElement.h: new method: createMediaPlayer().

(HTMLMediaElement):

6:36 PM Changeset in webkit [133251] by senorblanco@chromium.org

4 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.

platform/chromium-win/compositing/geometry/vertical-scroll-composited-expected.png:
platform/chromium-win/platform/chromium/compositing/huge-layer-rotated-expected.png:
platform/chromium-win/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png:

6:23 PM Changeset in webkit [133250] by andersca@apple.com

2 edits in trunk/Source/WebKit2

Reformat the annotationStyle initializer to appease update-webkit-localizable-strings.

Rubber-stamped by Dan Bernstein.

WebProcess/Plugins/PDF/PDFPlugin.mm:

6:22 PM Changeset in webkit [133249] by tsepez@chromium.org

3 edits
6 adds in trunk

XSS blocker false positive when page contains <iframe src="">
https://bugs.webkit.org/show_bug.cgi?id=93416

Reviewed by Adam Barth.

Source/WebCore:

Treat src="" and src="about:blank" as safe resources even if they appear
in the query parameters.

Tests: http/tests/security/xssAuditor/iframe-injection-allowed-2.html

http/tests/security/xssAuditor/iframe-injection-allowed-3.html
http/tests/security/xssAuditor/iframe-injection-allowed.html

html/parser/XSSAuditor.cpp:

(WebCore::XSSAuditor::isLikelySafeResource):

LayoutTests:

http/tests/security/xssAuditor/iframe-injection-allowed-2-expected.txt: Added.
http/tests/security/xssAuditor/iframe-injection-allowed-2.html: Added.
http/tests/security/xssAuditor/iframe-injection-allowed-3-expected.txt: Added.
http/tests/security/xssAuditor/iframe-injection-allowed-3.html: Added.
http/tests/security/xssAuditor/iframe-injection-allowed-expected.txt: Added.
http/tests/security/xssAuditor/iframe-injection-allowed.html: Added.

6:19 PM Changeset in webkit [133248] by commit-queue@webkit.org

3 edits
2 adds in trunk

Fix assertion failure in RenderGeometryMap::absoluteRect when frame scale != 1.0
https://bugs.webkit.org/show_bug.cgi?id=100912

Patch by Tien-Ren Chen <trchen@chromium.org> on 2012-11-01
Reviewed by Simon Fraser.

Frame scale will add transformation to RenderView, so fixed position doesn't
get propagated up to the viewport by RenderGeometryMap. This is handled
correctly in RenderView::mapLocalToContainer, causing the assertion to fail.
This patch corrects RenderGeometryMap::mapToAbsolute to handle the RenderView
transformation case.

A layout test is added to catch this issue. The test will crash debug build
without this patch.

Source/WebCore:

Test: compositing/geometry/fixed-position-composited-page-scale-scroll.html

rendering/RenderGeometryMap.cpp:

(WebCore::RenderGeometryMap::mapToAbsolute):

LayoutTests:

compositing/geometry/fixed-position-composited-page-scale-scroll-expected.txt: Added.
compositing/geometry/fixed-position-composited-page-scale-scroll.html: Added.

6:17 PM Changeset in webkit [133247] by commit-queue@webkit.org

4 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Add a prompt to enable global location services
https://bugs.webkit.org/show_bug.cgi?id=100992

PR 227897

Patch by Otto Derek Cheung <otcheung@rim.com> on 2012-11-01
Reviewed by Rob Buis.

Adding a dialog to let the user know location services isn't enabled.
Currently, the browser app does not check whether the global setting for
location service is enabled. This patch adds a dialog to allow the user
to open the settings app and change that setting. This infobar will only
show once per session. If the user fails to turn on location services after
the first prompt, all geolocation requests will automatically fail.

Also, adapting GeolocationClientBlackBerry to the removal of GeoTracker.
The class now talks directly to the geo handler singleton.

Api/WebPageClient.h:
WebCoreSupport/GeolocationClientBlackBerry.cpp:

(GeolocationClientBlackBerry::GeolocationClientBlackBerry):
(GeolocationClientBlackBerry::geolocationDestroyed):
(GeolocationClientBlackBerry::startUpdating):
(GeolocationClientBlackBerry::stopUpdating):
(GeolocationClientBlackBerry::requestPermission):
(GeolocationClientBlackBerry::setEnableHighAccuracy):

WebCoreSupport/GeolocationClientBlackBerry.h:

(WebCore::GeolocationClientBlackBerry::requiresHighAccuracy):
(GeolocationClientBlackBerry):

5:59 PM Changeset in webkit [133246] by thakis@chromium.org

2 edits in trunk/Tools

[chromium] Try to find ninja if it's not in the path already
https://bugs.webkit.org/show_bug.cgi?id=100988

Reviewed by Ryosuke Niwa.

Similar to the gclient detection logic in update-webkit-chromium

Scripts/webkitdirs.pm:

(buildChromiumNinja):

5:41 PM Changeset in webkit [133245] by Simon Fraser

2884 edits
222 adds
18 deletes in trunk/LayoutTests

Rebaseline tests listed in Mac TestExpectations as needing new baselines, other than those that
are failing ref tests.

Removed relevant lines from TestExpectations.

Carry along some new image results for the directories which were rebaselined.

Rubber-stamped by Tim Horton

5:24 PM Changeset in webkit [133244] by abarth@webkit.org

3 edits in trunk/Source/WebCore

[V8] Generalize NodeWrapperMap to be able to handle other sorts of keys
https://bugs.webkit.org/show_bug.cgi?id=100973

Reviewed by Kentaro Hara.

This is the first step towards using intrusive DOM wrapper maps more
widely in WebCore (see
http://lists.webkit.org/pipermail/webkit-dev/2012-November/022686.html
for more context).

bindings/v8/DOMDataStore.cpp:

(WebCore::DOMDataStore::DOMDataStore):

bindings/v8/IntrusiveDOMWrapperMap.h:

(WebCore):
(WebCore::IntrusiveDOMWrapperMap::weakCallback):

5:07 PM Changeset in webkit [133243] by eric@webkit.org

2 edits in trunk/Tools

Add Glenn Adams to the Contributor list
https://bugs.webkit.org/show_bug.cgi?id=100982

Reviewed by Adam Barth.

I CC him enough on bugs it would help me to have him listed here.

Scripts/webkitpy/common/config/committers.py:

5:03 PM Changeset in webkit [133242] by Alexandru Chiculita

6 edits in trunk/Source/WebCore

[CSS Shaders] CustomFilterOperation should be converted to ValidatedCustomFilterOperation before using it
https://bugs.webkit.org/show_bug.cgi?id=100533

Reviewed by Dean Jackson.

Added the code that converts a CustomFilterOperation to a ValidatedCustomFilterOperation.
Both the software path and the composited one will use this operation instead. There will be
no need to check the shader in platform code anymore.

No new tests, already covered by existing custom filter tests.

platform/graphics/filters/ValidatedCustomFilterOperation.cpp:

(WebCore::ValidatedCustomFilterOperation::ValidatedCustomFilterOperation):

platform/graphics/filters/ValidatedCustomFilterOperation.h:

(WebCore):
(WebCore::ValidatedCustomFilterOperation::create):
(ValidatedCustomFilterOperation):
(WebCore::ValidatedCustomFilterOperation::validatedProgram):
(WebCore::ValidatedCustomFilterOperation::parameters):
(WebCore::ValidatedCustomFilterOperation::meshRows):
(WebCore::ValidatedCustomFilterOperation::meshColumns):
(WebCore::ValidatedCustomFilterOperation::meshType):
(WebCore::ValidatedCustomFilterOperation::operator==):

rendering/FilterEffectRenderer.cpp:

(WebCore::createCustomFilterEffect):
(WebCore::FilterEffectRenderer::build):

rendering/RenderLayer.cpp:

(WebCore::RenderLayer::ensureBacking):
(WebCore::RenderLayer::clearBacking):
(WebCore::RenderLayer::styleChanged): updateOrRemoveFilterClients needs to be called before the composited
layer is updated. Otherwise the composited layer will never see a loaded filter in the first call.
(WebCore):
(WebCore::RenderLayer::isCSSCustomFilterEnabled):
(WebCore::RenderLayer::computeFilterOperations):
(WebCore::RenderLayer::updateOrRemoveFilterClients): Split updateOrRemoveFilterEffect into 2 functions.
This one is supposed to add the clients needed to load network resources.
(WebCore::RenderLayer::updateOrRemoveFilterEffectRenderer): Figures out if a software fallback is needed
and creates a FilterEffectRenderer.

rendering/RenderLayer.h:

(RenderLayer):

4:58 PM Changeset in webkit [133241] by commit-queue@webkit.org

7 edits
4 adds in trunk

[CSS Shaders] Get rid of internal tex coord attribute
https://bugs.webkit.org/show_bug.cgi?id=94358

Patch by Max Vujovic <mvujovic@adobe.com> on 2012-11-01
Reviewed by Dean Jackson.

Source/WebCore:

Remove the internal css_a_texCoord attribute that WebKit added to shaders in order to
sample the element texture by texture coordinate.

Now, the WebKit-added sampling code can leverage a_texCoord if the author defined it, or
WebKit can add its own a_texCoord definition to the author's shader.

Note that vertex attributes are read-only in GLSL. Also, note that we already reject the
shader if the author did not define a_texCoord with the correct type. Essentially, if
a_texCoord exists in the author's validated shader, we are guaranteed that it's the correct
type and that its value is unmodified.

Test: css3/filters/custom/custom-filter-a-tex-coord-optional.html

platform/graphics/filters/CustomFilterCompiledProgram.cpp:

(WebCore::CustomFilterCompiledProgram::CustomFilterCompiledProgram):

Remove the references to m_internalTexCoordAttribLocation.

(WebCore::CustomFilterCompiledProgram::initializeParameterLocations): Ditto.

platform/graphics/filters/CustomFilterCompiledProgram.h: Ditto.
platform/graphics/filters/CustomFilterRenderer.cpp:

(WebCore::CustomFilterRenderer::bindProgramAndBuffers): Ditto.
(WebCore::CustomFilterRenderer::unbindVertexAttributes): Ditto.

platform/graphics/filters/CustomFilterValidatedProgram.cpp:

(WebCore::CustomFilterValidatedProgram::CustomFilterValidatedProgram):

Pass the set of symbols found in the author's shaders to the rewriteMixVertexShader
method.

(WebCore::CustomFilterValidatedProgram::rewriteMixVertexShader):

If the author didn't define a_texCoord, add it to the end of the author's vertex
shader, but before the shader's new main function. As before, the new main function
will pass the texture coordinate to the fragment shader via the css_v_texCoord varying.

platform/graphics/filters/CustomFilterValidatedProgram.h:

(WebCore):

Add a forward declaration for ANGLEShaderSymbol.

(CustomFilterValidatedProgram):

Update the method prototype for rewriteMixVertexShader.

LayoutTests:

Add a test which verifies that a custom filter executes regardless of whether the author
defines a_texCoord in the vertex shader. We check this because the implementation uses
the author's a_texCoord definition if it exists. If it doesn't exist, the implementation
adds its own a_texCoord definition to the author's shader.

css3/filters/custom/custom-filter-a-tex-coord-optional-expected.html: Added.
css3/filters/custom/custom-filter-a-tex-coord-optional.html: Added.
css3/filters/resources/a-tex-coord-defined.vs: Added.
css3/filters/resources/a-tex-coord-undefined.vs: Added.

4:36 PM Changeset in webkit [133240] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed gardening. Suppressing an image failure after
r133224.

platform/chromium/TestExpectations:

4:23 PM Changeset in webkit [133239] by crogers@google.com

3 edits in trunk/Source/WebCore

Ensure that AudioNode deletion is synchronized with a stable state of the rendering graph
https://bugs.webkit.org/show_bug.cgi?id=100994

Reviewed by Kenneth Russell.

In some rare cases it has been observed that nodes are getting deleted in the main thread
during an audio rendering quantum where the dirty inputs and outputs have not yet been cleaned
via calls to handleDirtyAudioSummingJunctions() and handleDirtyAudioNodeOutputs().
This was possible because nodes marked for deletion with markForDeletion() could be picked
up in a subsequent call to deleteMarkedNodes() before the render quantum has finished and
handlePostRenderTasks() has had a chance to reconcile these marked nodes and clean the dirty state.
The solution is to manage the marked nodes in a separate vector which only gets copied to another
vector truly eligible for deletion which is synchronized in handlePostRenderTasks().

Modules/webaudio/AudioContext.cpp:

(WebCore::AudioContext::markForDeletion):
(WebCore::AudioContext::scheduleNodeDeletion):
(WebCore::AudioContext::deleteMarkedNodes):

Modules/webaudio/AudioContext.h:

(AudioContext):

3:58 PM Changeset in webkit [133238] by rniwa@webkit.org

2 edits in trunk/Source/WebCore

Build fix after r133224 as suggested by Enrica.
Confirmed the test failures go away on Chromium Mac locally.

page/EventHandler.cpp:

(WebCore::EventHandler::updateSelectionForMouseDrag):

3:58 PM Changeset in webkit [133237] by roger_fong@apple.com

2 edits in trunk/LayoutTests

Unreviewed. Skip fast/xmlhttprequest/xmlhttprequest-recursive-sync-event.html.
It is inherently flaky: https://bugs.webkit.org/show_bug.cgi?id=100117

platform/win/TestExpectations:

3:51 PM Changeset in webkit [133236] by Lucas Forschler

6 edits
2 copies in branches/safari-536.28-branch

Merged r124491. <rdar://problem/12536449>

3:45 PM Changeset in webkit [133235] by Lucas Forschler

4 edits
2 copies
1 add in branches/safari-536.28-branch

Merged r124258. <rdar://problem/12536444>

3:33 PM Changeset in webkit [133234] by jer.noble@apple.com

2 edits in trunk/Tools

Add Python-LLDB support for WTF::Vector types.
https://bugs.webkit.org/show_bug.cgi?id=100980

Reviewed by Anders Carlsson.

Add a synthetic child provider for WTF::Vector types.

lldb/lldb_webkit.py:

(lldb_init_module): Add both the summary and synthetic providers for WTF::Vector
(WTFVector_SummaryProvider): Wrapper around WTFVectorProvider
(WTFVectorProvider):
(WTFVectorProvider.init):
(WTFVectorProvider.num_children): Return m_size+2 children to account for synthesized children.
(WTFVectorProvider.get_child_index): Synthesize child indexes for each entry in the Vector.
(WTFVectorProvider.get_child_at_index): Synthesize children for each entry in the Vector.
(WTFVectorProvider.update): Retrieve state from the underlying Vector.
(WTFVectorProvider.has_children): Vectors always have children.

3:29 PM Changeset in webkit [133233] by ap@apple.com

14 edits in trunk/Source

Rename HAVE(NETWORK_CFDATA_ARRAY_CALLBACK) to USE(NETWORK_CFDATA_ARRAY_CALLBACK)

Rubber-stamped by Joe Pecoraro.

All CFNetwork based platforms have this now, but not all use it at the moment.

loader/ResourceLoader.h:
loader/SubresourceLoader.h:
loader/cf/SubresourceLoaderCF.cpp:
loader/mac/ResourceLoaderMac.mm:
platform/SharedBuffer.cpp: (WebCore::SharedBuffer::clear): (WebCore::SharedBuffer::buffer): (WebCore::SharedBuffer::getSomeData):
platform/SharedBuffer.h:
platform/cf/SharedBufferCF.cpp:
platform/network/ResourceHandle.h:
platform/network/ResourceHandleClient.h: (ResourceHandleClient):
platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::willCacheResponse): (WebCore::ResourceHandle::createCFURLConnection):
platform/network/mac/ResourceHandleMac.mm:

3:26 PM Changeset in webkit [133232] by ap@apple.com

7 edits in trunk/Source

Fix HAVE(NETWORK_CFDATA_ARRAY_CALLBACK) build
https://bugs.webkit.org/show_bug.cgi?id=100979

Reviewed by Joseph Pecoraro.

loader/ResourceBuffer.cpp: (WebCore::ResourceBuffer::append):
loader/ResourceBuffer.h: Added a version of append() that takes CFDataRef, passing it directly to SharedBuffer.

loader/mac/ResourceLoaderMac.mm: (WebCore::ResourceLoader::didReceiveDataArray): InspectorInstrumentation now takes a length. Unfortunately, we don't have encodedLength here, meaning that the results will likely be incorrect.

platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::willCacheResponse): Removed a version that needlessly used deprecated API. (WebCore::ResourceHandle::createCFURLConnection): This code assumed that HAVE(NETWORK_CFDATA_ARRAY_CALLBACK) and USE(PROTECTION_SPACE_AUTH_CALLBACK) always come together. Changed to check for these separately.

3:23 PM Changeset in webkit [133231] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r124156. <rdar://problem/12536490>

3:18 PM Changeset in webkit [133230] by Lucas Forschler

2 edits in branches/safari-536.28-branch/Source/WebCore

Merged r124089. <rdar://problem/12536443>

2:59 PM Changeset in webkit [133229] by zoltan@webkit.org

2 edits in trunk/Tools

Remove PageLoad directory entry from PerformanceTests pattern_map
https://bugs.webkit.org/show_bug.cgi?id=100989

Reviewed by Ryosuke Niwa.

Since we removed PageLoad directory from PerformanceTests we don't need this entry in pattern_map.

Scripts/webkitpy/performance_tests/perftest.py:

(PerfTestFactory):

2:59 PM Changeset in webkit [133228] by rwlbuis@webkit.org

3 edits
3 adds in trunk

[BlackBerry] Add more form validation strings
https://bugs.webkit.org/show_bug.cgi?id=100978

Reviewed by Yong Li.

PR 210072

Source/WebCore:

Implement these two methods properly.

platform/blackberry/LocalizedStringsBlackBerry.cpp:

(WebCore::validationMessageRangeUnderflowText):
(WebCore::validationMessageRangeOverflowText):

LayoutTests:

Add the result we expect after adding translations for range over and underflow.

platform/blackberry/fast/forms/number/number-validation-message-expected.txt: Added.

2:48 PM Changeset in webkit [133227] by Alexandru Chiculita

14 edits in trunk/Source

[CSS Shaders] Move MeshBoxType out of CustomFilterOperation
https://bugs.webkit.org/show_bug.cgi?id=100782

Reviewed by Dean Jackson.

Source/WebCore:

Moved CustomFilter::MeshBoxType enum to the CustomFilterConstants.h file and renamed it to CustomFilterMeshBoxType.
This way we can reuse it without including the CustomFilterOperation file.

No new tests, just refactoring code.

css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
(WebCore::CSSPrimitiveValue::operator CustomFilterMeshBoxType):

css/StyleResolver.cpp:

(WebCore::StyleResolver::createCustomFilterOperation):

platform/graphics/filters/CustomFilterArrayParameter.h: Adding AnimationUtilities.h for WebCore::blend, looks like

CustomFilterOperation already included AnimationsUtilities.h indirectly.

platform/graphics/filters/CustomFilterConstants.h: Added CustomFilterMeshBoxType.
platform/graphics/filters/CustomFilterNumberParameter.h:
platform/graphics/filters/CustomFilterOperation.cpp:

(WebCore::CustomFilterOperation::CustomFilterOperation):

platform/graphics/filters/CustomFilterOperation.h:

(WebCore::CustomFilterOperation::create):
(WebCore::CustomFilterOperation::meshBoxType):
(CustomFilterOperation):

platform/graphics/filters/CustomFilterRenderer.cpp:

(WebCore::CustomFilterRenderer::create):
(WebCore::CustomFilterRenderer::CustomFilterRenderer):

platform/graphics/filters/CustomFilterRenderer.h: Also removed the need to include CustomFilterOperation.h. Some other

headers were needed to replace the files that were automatically incldued because of that file.
(WebCore):
(CustomFilterRenderer):

platform/graphics/filters/FECustomFilter.cpp:

(WebCore::FECustomFilter::FECustomFilter):
(WebCore::FECustomFilter::create):

platform/graphics/filters/FECustomFilter.h:

(FECustomFilter):

Source/WebKit2:

Renamed the CustomFilterOperation::MeshBoxType to CustomFilterMeshBoxType.

Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:

(CoreIPC::::decode):

1:57 PM Changeset in webkit [133226] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed; marking another inspector test as flaky.

platform/chromium/TestExpectations:

1:57 PM Changeset in webkit [133225] by aroben@webkit.org

2 edits in trunk/Source/WebKit/mac

[WK1] Fixed-position elements jiggle up and down slightly during scrolling on a Retina display
https://bugs.webkit.org/show_bug.cgi?id=100957

Reviewed by Simon Fraser.

WebCore doesn't yet support subpixel scrolling. WebKit2 forces
scrolling to always be integral. Now WebKit1 forces this as well.

I'm not sure how to write a test for this.

WebView/WebDynamicScrollBarsView.mm:

(shouldRoundScrollOrigin): Returns YES if there are any position:fixed
or position:sticky elements in the page.
(-[WebDynamicScrollBarsView scrollClipView:toPoint:]): Round the
scroll origin to the nearest pixel if needed.

1:54 PM Changeset in webkit [133224] by enrica@apple.com

19 edits
2 adds in trunk

Part2 of: Extend -webkit-user-select with new value "all"
<rdar://problem/10161404>
https://bugs.webkit.org/show_bug.cgi?id=91912

Reviewed by Ryosuke Niwa.

Source/WebCore:

The new value "all" for -webkit-user-select property gives content none-or-all selection option.
The patch was originally prepared by Alice Cheng but the approach has been changed.
The idea is to treat these elements like non editable, meaning that we should skip over them entirely
when moving the cursor and a deletion should delete the element and all its descentants at once.
The key change is in Node::rendererIsEditable where we now return false if the element style is
userSelect: all. The other change is in the way we create the selection on mouse click and dragging
over the element. In both cases we force the selection to extend over the entire element with
the user-select: all attribute.
This is currently enabled only for the Mac port.

Test: editing/selection/user-select-all-selection.html

dom/Node.cpp: Added a parameter to isContentEditable to behave differently

when called from JavaScript. Internally isContentEditable returns false on
nodes with user-select: all style.
(WebCore::Node::isContentEditable):
(WebCore::Node::isContentRichlyEditable):
(WebCore::Node::rendererIsEditable):
(WebCore::Node::shouldUseInputMethod):
(WebCore::Node::willRespondToMouseClickEvents):

dom/Node.h:

(WebCore::Node::rendererIsEditable):
(WebCore::Node::rendererIsRichlyEditable):

dom/Position.cpp:

(WebCore::Position::nodeIsUserSelectAll): Added.
(WebCore::Position::rootUserSelectAllForNode): Added.

dom/Position.h: Added static functions described above.
editing/ApplyStyleCommand.cpp:

(WebCore::ApplyStyleCommand::removeInlineStyleFromElement): Added parameter to
isContentEditable() call.
(WebCore::ApplyStyleCommand::surroundNodeRangeWithElement): Added parameter to
isContentEditable() call.

editing/DeleteFromTextNodeCommand.cpp:

(WebCore::DeleteFromTextNodeCommand::doApply): Added parameter to
isContentEditable() call.

editing/FrameSelection.cpp:

(WebCore::adjustForwardPositionForUserSelectAll): New helper function.
(WebCore::adjustBackwardPositionForUserSelectAll): New helper function.
(WebCore::FrameSelection::modifyExtendingRight):
(WebCore::FrameSelection::modifyExtendingForward):
(WebCore::FrameSelection::modifyExtendingLeft):
(WebCore::FrameSelection::modifyExtendingBackward):
(WebCore::FrameSelection::modify):
(WebCore::CaretBase::invalidateCaretRect): Added parameter to
isContentEditable() call.

editing/InsertNodeBeforeCommand.cpp:

(WebCore::InsertNodeBeforeCommand::doApply): Ditto.
(WebCore::InsertNodeBeforeCommand::doUnapply): Ditto.

editing/RemoveNodeCommand.cpp:

(WebCore::RemoveNodeCommand::doApply): Ditto.

editing/visible_units.cpp:

(WebCore::startOfParagraph): We should not consider a paragraph break and element
with user-select: all style, like we do at the border of editability.
(WebCore::endOfParagraph): Ditto.

page/EventHandler.cpp:

(WebCore::EventHandler::updateSelectionForMouseDownDispatchingSelectStart): Create a selection
around the element whose style is user-select: all.
(WebCore::EventHandler::updateSelectionForMouseDrag): Ditto.

rendering/RootInlineBox.cpp:

(WebCore::RootInlineBox::selectionState): Fixed a bug uncovered during this work.
If the selection starts in one of the leaf boxes and after we encounter one with SelectionNone,
we should return the selection state as SelectionBoth, assuming we went past the end selection.
This avoids doing an incorrect gap filling for the selection highlighting.

LayoutTests:

Testing moving and extending selections with the new CSS propertyvalue, with mouse movements
and with the keyboard.
Updated test expectations for all the platforms, since this is only enabled for the Mac port.

editing/selection/user-select-all-selection-expected.txt: Added.
editing/selection/user-select-all-selection.html: Added.
platform/chromium/TestExpectations:
platform/qt/TestExpectations:
platform/gtk/TestExpectations:
platform/win/TestExpectations:
platform/efl/TestExpectations:

1:50 PM Changeset in webkit [133223] by senorblanco@chromium.org

2 edits in trunk/Source/WebCore

IndexedDB: Fix Windows build by re-adding a #include
https://bugs.webkit.org/show_bug.cgi?id=100986

Patch by Alec Flett <alecflett@chromium.org> on 2012-11-01
Reviewed by Stephen White.

Fix #include that broke windows.

Modules/indexeddb/IDBCallbacks.h:

1:49 PM Changeset in webkit [133222] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged 123131. '<rdar://problem/12536524>'

1:42 PM Changeset in webkit [133221] by Dave Barton

8 edits in trunk

REGRESSION (r128837): mathml/presentation/subsup.xhtml became flaky
https://bugs.webkit.org/show_bug.cgi?id=97390

Reviewed by Ojan Vafai.

Source/WebCore:

In a RenderMathMLSubSup, we need to set the base's wrapper's style so that baseHeight
in layout() will be the base's true height, without any flexbox stretching.

Tested by existing tests.

rendering/mathml/RenderMathMLSubSup.cpp:

(WebCore::RenderMathMLSubSup::fixAnonymousStyles):
(WebCore::RenderMathMLSubSup::addChild):
(WebCore::RenderMathMLSubSup::styleDidChange):

rendering/mathml/RenderMathMLSubSup.h:

(RenderMathMLSubSup):

LayoutTests:

platform/chromium/TestExpectations:
platform/efl/TestExpectations:
platform/gtk/TestExpectations:
platform/mac/TestExpectations:

1:35 PM Changeset in webkit [133220] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r123128. <rdar://problem/12536420>

1:30 PM Changeset in webkit [133219] by Lucas Forschler

5 edits
2 copies in branches/safari-536.28-branch

Merged r123062. <rdar://problem/12516377>

1:30 PM Changeset in webkit [133218] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed gardening. Mark an inspector test as flaky.

platform/chromium/TestExpectations:

1:25 PM Changeset in webkit [133217] by senorblanco@chromium.org

2 edits
1 delete in trunk/LayoutTests

[chromium] Unreviewed gardening.

platform/chromium-linux-x86/compositing/reflections/nested-reflection-anchor-point-expected.png: Removed.
platform/chromium-mac/compositing/reflections/nested-reflection-anchor-point-expected.png:

1:23 PM Changeset in webkit [133216] by senorblanco@chromium.org

3 edits
2 deletes in trunk/LayoutTests

[chromium] Unreviewed gardening. Move baselines for a couple of
compositing tests to platform-mac, so MountainLion can use them too.

platform/chromium-mac-lion/compositing/geometry/vertical-scroll-composited-expected.png: Removed.
platform/chromium-mac-lion/platform/chromium/compositing/huge-layer-rotated-expected.png: Removed.
platform/chromium-mac/compositing/geometry/vertical-scroll-composited-expected.png:
platform/chromium-mac/platform/chromium/compositing/huge-layer-rotated-expected.png:

1:13 PM Changeset in webkit [133215] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r122755. <rdar://problem/12536436>

12:41 PM Changeset in webkit [133214] by Antti Koivisto

4 edits in trunk/Source/WebCore

REGRESSION (r132941): attribute modification 10% performance regression
https://bugs.webkit.org/show_bug.cgi?id=100873

Reviewed by Ojan Vafai.

Don't do the class change finding by mutating SpaceSplitString. It is slow. Instead use a bit vector
to mark the unchanged classes

css/StyleResolver.cpp:

(WebCore):

css/StyleResolver.h:

(WebCore::StyleResolver::hasSelectorForAttribute):
(WebCore):
(WebCore::StyleResolver::hasSelectorForClass):
(WebCore::StyleResolver::hasSelectorForId):

Inlined these and moved value validity testing to clients.

dom/Element.cpp:

(WebCore::checkNeedsStyleInvalidationForIdChange):
(WebCore):
(WebCore::Element::attributeChanged):

Clean up id testing too.

(WebCore::checkNeedsStyleInvalidationForClassChange):

Use bit vector for marking seen values. Avoids allocations and reffing.

(WebCore::Element::classAttributeChanged):

Don't test if style is already invalid.

12:31 PM Changeset in webkit [133213] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed gardening. Mark yet another test as flaky.

platform/chromium/TestExpectations:

12:30 PM Changeset in webkit [133212] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed gardening. Mark a test as flaky.

platform/chromium/TestExpectations:

12:16 PM Changeset in webkit [133211] by jer.noble@apple.com

1 edit
2 adds in trunk/Tools

Add LLDB-Python scripts to support WebKit data types.
https://bugs.webkit.org/show_bug.cgi?id=100898

Reviewed by Anders Carlsson.

Add Summary providers for common WebKit string types.

lldb/lldb_webkit.py: Added.

(lldb_init_module): Register these providers with the debugger.
(WTFString_SummaryProvider): Wrapper around WTFStringProvider.
(WTFStringImpl_SummaryProvider): Wrapper around WTFStringImplProvider.
(WTFAtomicString_SummaryProvider): Ditto.
(guess_string_length): Walk through the string looking for a null-byte(s).
(ustring_to_string): Walk through the string, extracting two-byte values, and concatenate them.
(lstring_to_string): Walk through the string, extracting byte values, and concatenate them.
(WTFStringImplProvider):
(WTFStringImplProvider.init):
(WTFStringImplProvider.get_length): Extract the m_length variable.
(WTFStringImplProvider.get_data8): Extract the m_data8 variable.
(WTFStringImplProvider.get_data16): Extract the m_data16 variable.
(WTFStringImplProvider.to_string): Pass the appropriate data variable to {u,l}string_to_string.
(WTFStringImplProvider.is_8bit): Check the m_hashAndFlags variable.
(WTFStringProvider):
(WTFStringProvider.init):
(WTFStringProvider.stringimpl): Extract the m_impl variable.
(WTFStringProvider.get_length): Pass through to WTFStringImplProvider.
(WTFStringProvider.to_string): Ditto.

12:13 PM Changeset in webkit [133210] by Lucas Forschler

4 edits
2 copies in branches/safari-536.28-branch

Merged r122278. <rdar://problem/12536506>

12:08 PM Changeset in webkit [133209] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r121930. <rdar://problem/12536519>

12:03 PM Changeset in webkit [133208] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r121491. <rdar://problem/12536496>

12:02 PM Changeset in webkit [133207] by rniwa@webkit.org

2 edits in trunk/Source/WebCore

[Mac] Crash in Range::editingStartPosition
https://bugs.webkit.org/show_bug.cgi?id=100972

Reviewed by Enrica Casucci.

Add a missing null pointer check. toNormalizedRange() can return null here, in which case the call to
pastLastNode() will crash.

No new tests since determining the exact condition under which we go through the said code path and
toNormalRange returns null is hard.

editing/mac/EditorMac.mm:

(WebCore::Editor::fontForSelection):

12:01 PM Changeset in webkit [133206] by danakj@chromium.org

2 edits in trunk/LayoutTests

Unreviewed, remove failing expectations after rebaseline.

platform/chromium/TestExpectations:

11:57 AM Changeset in webkit [133205] by Beth Dakin

21 edits
4 adds in trunk

.: https://bugs.webkit.org/show_bug.cgi?id=100917
There should be a way to dump the scrolling tree from the layout tests

Reviewed by Simon Fraser.

Source/autotools/symbols.filter:

Source/WebCore: https://bugs.webkit.org/show_bug.cgi?id=100917
There should be a way to dump the scrolling tree from the layout tests

Reviewed by Simon Fraser.

New Internals call to dump the scrolling state tree as text. Calls through to the
ScrollingCoordinator.

testing/Internals.cpp:

(WebCore::Internals::scrollingStateTreeAsText):
(WebCore):

testing/Internals.h:
testing/Internals.idl:
WebCore.exp.in:
page/Page.cpp:

(WebCore::Page::scrollingStateTreeAsText):
(WebCore):

page/Page.h:

(Page):

ScrollingCoordinator tells the rootStateNode to dump the tree as text.

page/scrolling/mac/ScrollingCoordinatorMac.h:

(ScrollingCoordinatorMac):

page/scrolling/mac/ScrollingCoordinatorMac.mm:

(WebCore::ScrollingCoordinatorMac::scrollingStateTreeAsText):
(WebCore):

page/scrolling/ScrollingCoordinator.cpp:

(WebCore::ScrollingCoordinator::scrollingStateTreeAsText):
(WebCore):

page/scrolling/ScrollingCoordinator.h:

(ScrollingCoordinator):

dumpNode() dumps generic stuff for each node, and dumpProperties will dump the
properties that are specific to different types of nodes.

page/scrolling/ScrollingStateNode.cpp:

(WebCore::ScrollingStateNode::writeIndent):
(WebCore):
(WebCore::ScrollingStateNode::dumpNode):
(WebCore::ScrollingStateNode::scrollingStateTreeAsText):

page/scrolling/ScrollingStateNode.h:

(WebCore):
(ScrollingStateNode):

page/scrolling/ScrollingStateScrollingNode.cpp:

(WebCore::ScrollingStateScrollingNode::dumpProperties):
(WebCore):

page/scrolling/ScrollingStateScrollingNode.h:

(ScrollingStateScrollingNode):

Source/WebKit2: https://bugs.webkit.org/show_bug.cgi?id=100917
There should be a way to dump the scrolling tree from the layout tests

Reviewed by Simon Fraser.

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

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=100917
There should be a way to dump the scrolling tree from the layout tests

Reviewed by Simon Fraser.

platform/mac/tiled-drawing/scrolling-tree-after-scroll-expected.txt: Added.
platform/mac/tiled-drawing/scrolling-tree-after-scroll.html: Added.
platform/mac/tiled-drawing/scrolling-tree-slow-scrolling-expected.txt: Added.
platform/mac/tiled-drawing/scrolling-tree-slow-scrolling.html: Added.

11:56 AM Changeset in webkit [133204] by danakj@chromium.org

12 edits
2 moves
11 adds
1 delete in trunk/LayoutTests

Rebaseline after compositor migration to gfx types.
http://crbug.com/147395

Unreviewed rebaselines.

platform/chromium-linux/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png:
platform/chromium-linux/compositing/geometry/vertical-scroll-composited-expected.png:
platform/chromium-linux/compositing/reflections/nested-reflection-anchor-point-expected.png:
platform/chromium-linux/platform/chromium/compositing/huge-layer-rotated-expected.png:
platform/chromium-linux/transforms/3d/point-mapping/3d-point-mapping-3-expected.png:
platform/chromium-linux/transforms/3d/point-mapping/3d-point-mapping-preserve-3d-expected.png:
platform/chromium-mac-lion/compositing/geometry/vertical-scroll-composited-expected.png: Added.
platform/chromium-mac-lion/compositing/reflections/nested-reflection-anchor-point-expected.png:
platform/chromium-mac-lion/platform/chromium/compositing/huge-layer-rotated-expected.png: Added.
platform/chromium-mac-lion/transforms/3d/point-mapping/3d-point-mapping-3-expected.png: Added.
platform/chromium-mac-lion/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png: Added.
platform/chromium-mac-lion/transforms/3d/point-mapping/3d-point-mapping-preserve-3d-expected.png: Added.
platform/chromium-mac-snowleopard/compositing/reflections/nested-reflection-anchor-point-expected.png:
platform/chromium-mac-snowleopard/transforms/3d/point-mapping/3d-point-mapping-3-expected.png:
platform/chromium-mac-snowleopard/transforms/3d/point-mapping/3d-point-mapping-origins-expected.png:
platform/chromium-mac-snowleopard/transforms/3d/point-mapping/3d-point-mapping-preserve-3d-expected.png:
platform/chromium-win-xp/compositing/reflections/nested-reflection-anchor-point-expected.png: Removed.
platform/efl-wk1/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png: Renamed from LayoutTests/platform/efl/compositing/geometry/fixed-position-transform-composited-page-scale-down-expected.png.
platform/efl-wk1/compositing/reflections/nested-reflection-anchor-point-expected.png: Renamed from LayoutTests/platform/efl/compositing/reflections/nested-reflection-anchor-point-expected.png.

11:56 AM Changeset in webkit [133203] by rjkroege@chromium.org

3 edits
4 adds in trunk/Source/Platform

[chromium] Provide WebKit API interface for platform gesture curves
https://bugs.webkit.org/show_bug.cgi?id=100675

Reviewed by James Robinson.

Define interfaces to permit the Chromium embedder to provide its own
implementation of a gesture animation curve and add an entry point
to Platform.h that permits WebKit to obtain an instance of an embedder-
provided gesture curve.

No tests needed in this CL: only interfaces are being added.

Platform.gypi:
chromium/public/Platform.h:

(WebKit):
(Platform):
(WebKit::Platform::createFlingAnimationCurve):

chromium/public/WebGestureCurve.h: Added.

(WebKit):
(WebGestureCurve):
(WebKit::WebGestureCurve::~WebGestureCurve):

chromium/public/WebGestureCurveTarget.h: Added.

(WebKit):
(WebGestureCurveTarget):
(WebKit::WebGestureCurveTarget::~WebGestureCurveTarget):

chromium/src/WebActiveGestureAnimation.cpp: Added.

(WebKit):
(WebKit::WebActiveGestureAnimation::createAtAnimationStart):
(WebKit::WebActiveGestureAnimation::createWithTimeOffset):
(WebKit::WebActiveGestureAnimation::~WebActiveGestureAnimation):
(WebKit::WebActiveGestureAnimation::WebActiveGestureAnimation):
(WebKit::WebActiveGestureAnimation::animate):

chromium/src/WebActiveGestureAnimation.h: Added.

(WebKit):
(WebActiveGestureAnimation):

11:53 AM Changeset in webkit [133202] by kbr@google.com

3 edits in trunk/Source/WebCore

Add RGB to supported destination formats of ImageBuffer::copyToPlatformTexture
https://bugs.webkit.org/show_bug.cgi?id=100971

Reviewed by Stephen White.

The ImageBuffer::copyToPlatformTexture restriction added in
r132965 was too restrictive. Allow RGB textures to prevent
performance regressions.

html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::texImage2D):

platform/graphics/ImageBuffer.h:

(ImageBuffer):

11:41 AM Changeset in webkit [133201] by danakj@chromium.org

2 edits in trunk/LayoutTests

Unreviewed, marking tests as failing preemptively.
http://crbug.com/147395

platform/chromium/TestExpectations:

11:40 AM Changeset in webkit [133200] by peter@chromium.org

7 edits in trunk/Tools

[Chromium-Android] Apache doesn't properly clean up ipc semaphores after a layout test run
https://bugs.webkit.org/show_bug.cgi?id=100950

Reviewed by Dirk Pranke.

When a test run would fail to complete due to an exception in one of
the workers, the HTTP server wouldn't get a chance to gracefully shut
down. This caused too much IPC semaphores to be left on the server,
causing Apache to fail to start in subsequent runs.

By unifying the Android-specific code with other ports, we no longer
fail to call the ChromiumPort/Base setup_test_run() and clean_up_test_run()
methods either. Furthermore, the number_of_servers argument for starting
the HTTP server is now available as well.

Because not all tests require an HTTP server, it's not guaranteed that
it will be started. Android depends on this, so add a new method to Port
and override it for Android: requires_http_server().

Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:

(LayoutTestRunner.run_tests):

Scripts/webkitpy/layout_tests/controllers/manager.py:

(Manager._run_tests):

Scripts/webkitpy/layout_tests/port/base.py:

(Port.to.requires_http_server):

Scripts/webkitpy/layout_tests/port/base_unittest.py:

(PortTest.test_dont_require_http_server):

Scripts/webkitpy/layout_tests/port/chromium_android.py:

(ChromiumAndroidPort.requires_http_server):
(ChromiumAndroidPort.start_http_server):

Scripts/webkitpy/layout_tests/port/chromium_android_unittest.py:

(ChromiumAndroidPortTest.test_must_require_http_server):

11:37 AM Changeset in webkit [133199] by commit-queue@webkit.org

5 edits in trunk/Source

[BlackBerry] Web page view state should be preserved for pages loaded from page cache
https://bugs.webkit.org/show_bug.cgi?id=100694
Internal PR: 220488

Internally reviewed by Lianghui Chen, Joe Mason
Patch by Michael Matovsky <mmatovsky@rim.com> on 2012-11-01
Reviewed by Rob Buis.

The web page view specific metadata should be preserved for web pages loaded from page cache (by using back/forward).
This requires web page view state to be saved in and restored from the corresponding page cache history item.

Source/WebCore:

No new tests for platform specific internal change.

history/blackberry/HistoryItemViewState.h:

(HistoryItemViewState):

Source/WebKit/blackberry:

Api/WebPageClient.h:
WebCoreSupport/FrameLoaderClientBlackBerry.cpp:

(WebCore::FrameLoaderClientBlackBerry::saveViewStateToItem):
(WebCore::FrameLoaderClientBlackBerry::restoreViewState):

11:36 AM Changeset in webkit [133198] by danakj@chromium.org

2 edits in trunk/Source/Platform

[chromium] Allow implicit conversion between gfx::Vector2d and WebSize
https://bugs.webkit.org/show_bug.cgi?id=100553

Reviewed by James Robinson.

The gfx::Vector2d class is used in chromium for defining distances. The
IntSize class is used equivalently inside WebCore. This lets us convert
between vectors and sizes along the API boundary.

chromium/public/WebSize.h:

(WebKit::WebSize::WebSize):
(WebSize):
(WebKit::WebSize::operator=):
(WebKit::WebSize::operator gfx::Vector2d):

11:28 AM Changeset in webkit [133197] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Unreviewed gardening. Mark a test as flaky.

platform/chromium/TestExpectations:

11:12 AM Changeset in webkit [133196] by roger_fong@apple.com

2 edits in trunk/LayoutTests

Unreviewed. Skip fast/workers/worker-domurl.html on Windows. ENABLE_BLOB not enabled.

platform/win/TestExpectations:

11:07 AM Changeset in webkit [133195] by alecflett@chromium.org

43 edits in trunk/Source

IndexedDB: add methods to support id-based backend APIs
https://bugs.webkit.org/show_bug.cgi?id=100425

Reviewed by Tony Chang.

Source/WebCore:

First half of refactor involves adding a number of methods that
are int64_t-based rather than String-based. As a part of this, the
IDB*Metadata structs and the backend objectStore/index maps all
use int64_t as keys, rather than String.

In addition, there were a number of cleanups that came out of the
refactor:

The list of object stores active in a transaction is now maintained by the frontend IDBTransaction rather than the backend IDBTransactionBackendImpl. This also had a simplifying rippling effect through other call signatures.

I was able to remove an apparently old FIXME from IDBTransactionBackendImpl::objectStore and replace it with an ASSERT.

IDBObjectStoreBackendImpl's IndexWriter class got a little simpler since the id is now easily available in the IDBIndexMetadata.

A number of methods got simpler in their int64_t versions, specifically dropping a number of ExceptionCodes.

There is also some glue code
(getIndexId/getIndexIds/getObjectStoreId) that will go away with
the 2nd half of this: https://bugs.webkit.org/show_bug.cgi?id=100425

No new tests, no new functionality as this is just a refactor.

Modules/indexeddb/IDBDatabase.cpp:

(WebCore::IDBDatabase::objectStoreNames):
(WebCore::IDBDatabase::createObjectStore):
(WebCore::IDBDatabase::deleteObjectStore):
(WebCore::IDBDatabase::transaction):

Modules/indexeddb/IDBDatabaseBackendImpl.cpp:

(WebCore::IDBDatabaseBackendImpl::metadata):
(WebCore::IDBDatabaseBackendImpl::createObjectStore):
(WebCore::IDBDatabaseBackendImpl::objectStore):
(WebCore::IDBDatabaseBackendImpl::getObjectStoreId):
(WebCore):
(WebCore::IDBDatabaseBackendImpl::deleteObjectStore):
(WebCore::IDBDatabaseBackendImpl::transaction):
(WebCore::IDBDatabaseBackendImpl::loadObjectStores):
(WebCore::IDBDatabaseBackendImpl::removeObjectStoreFromMap):
(WebCore::IDBDatabaseBackendImpl::addObjectStoreToMap):

Modules/indexeddb/IDBDatabaseBackendImpl.h:

(WebCore::IDBDatabaseBackendImpl::deleteObjectStore):
(IDBDatabaseBackendImpl):

Modules/indexeddb/IDBDatabaseBackendInterface.h:

(IDBDatabaseBackendInterface):

Modules/indexeddb/IDBIndex.h:

(WebCore::IDBIndex::id):
(WebCore::IDBIndex::openKeyCursor):

Modules/indexeddb/IDBIndexBackendInterface.h:
Modules/indexeddb/IDBMetadata.h:

(WebCore::IDBIndexMetadata::IDBIndexMetadata):
(IDBIndexMetadata):
(IDBObjectStoreMetadata):
(WebCore::IDBObjectStoreMetadata::containsIndex):
(IDBDatabaseMetadata):
(WebCore::IDBDatabaseMetadata::IDBDatabaseMetadata):
(WebCore::IDBDatabaseMetadata::containsObjectStore):

Modules/indexeddb/IDBObjectStore.cpp:

(WebCore::IDBObjectStore::indexNames):
(WebCore::IDBObjectStore::put):
(WebCore::IDBObjectStore::createIndex):
(WebCore::IDBObjectStore::index):
(WebCore::IDBObjectStore::deleteIndex):

Modules/indexeddb/IDBObjectStoreBackendImpl.cpp:

(WebCore::IDBObjectStoreBackendImpl::put):
(WebCore::IDBObjectStoreBackendImpl::putWithIndexKeys):
(WebCore):
(WebCore::makeIndexWriters):
(WebCore::IDBObjectStoreBackendImpl::setIndexKeys):
(WebCore::IDBObjectStoreBackendImpl::setIndexesReady):
(WebCore::IDBObjectStoreBackendImpl::setIndexesReadyInternal):
(WebCore::IDBObjectStoreBackendImpl::putInternal):
(WebCore::IDBObjectStoreBackendImpl::deleteInternal):
(WebCore::IDBObjectStoreBackendImpl::createIndex):
(WebCore::IDBObjectStoreBackendImpl::index):
(WebCore::IDBObjectStoreBackendImpl::getIndexId):
(WebCore::IDBObjectStoreBackendImpl::getIndexIds):
(WebCore::IDBObjectStoreBackendImpl::deleteIndex):
(WebCore::IDBObjectStoreBackendImpl::loadIndexes):
(WebCore::IDBObjectStoreBackendImpl::removeIndexFromMap):
(WebCore::IDBObjectStoreBackendImpl::addIndexToMap):

Modules/indexeddb/IDBObjectStoreBackendImpl.h:

(IDBObjectStoreBackendImpl):

Modules/indexeddb/IDBObjectStoreBackendInterface.h:
Modules/indexeddb/IDBOpenDBRequest.cpp:

(WebCore::IDBOpenDBRequest::onUpgradeNeeded):

Modules/indexeddb/IDBRequest.cpp:

(WebCore::IDBRequest::onSuccess):
(WebCore::IDBRequest::uncaughtExceptionInEventHandler):

Modules/indexeddb/IDBTransaction.cpp:

(WebCore::IDBTransaction::create):
(WebCore::IDBTransaction::IDBTransaction):
(WebCore::IDBTransaction::objectStore):

Modules/indexeddb/IDBTransaction.h:

(IDBTransaction):

Modules/indexeddb/IDBTransactionBackendImpl.cpp:

(WebCore::IDBTransactionBackendImpl::create):
(WebCore::IDBTransactionBackendImpl::IDBTransactionBackendImpl):
(WebCore::IDBTransactionBackendImpl::objectStore):
(WebCore):
(WebCore::IDBTransactionBackendImpl::scheduleTask):

Modules/indexeddb/IDBTransactionBackendImpl.h:

(IDBTransactionBackendImpl):

Modules/indexeddb/IDBTransactionBackendInterface.h:

Source/WebKit/chromium:

Chromium Proxy and WebIDB* support for new int64_t-based methods.

public/WebIDBDatabase.h:

(WebKit::WebIDBDatabase::deleteObjectStore):
(WebKit::WebIDBDatabase::transaction):

public/WebIDBObjectStore.h:

(WebKit::WebIDBObjectStore::put):
(WebKit::WebIDBObjectStore::setIndexKeys):
(WebKit::WebIDBObjectStore::setIndexesReady):
(WebKit::WebIDBObjectStore::index):
(WebKit::WebIDBObjectStore::deleteIndex):

public/WebIDBTransaction.h:

(WebKit::WebIDBTransaction::objectStore):

src/IDBDatabaseBackendProxy.cpp:

(WebKit::IDBDatabaseBackendProxy::deleteObjectStore):
(WebKit):
(WebKit::IDBDatabaseBackendProxy::transaction):

src/IDBDatabaseBackendProxy.h:

(IDBDatabaseBackendProxy):

src/IDBObjectStoreBackendProxy.cpp:

(WebKit::IDBObjectStoreBackendProxy::put):
(WebKit):
(WebKit::IDBObjectStoreBackendProxy::setIndexKeys):
(WebKit::IDBObjectStoreBackendProxy::setIndexesReady):
(WebKit::IDBObjectStoreBackendProxy::index):
(WebKit::IDBObjectStoreBackendProxy::deleteIndex):

src/IDBObjectStoreBackendProxy.h:

(IDBObjectStoreBackendProxy):

src/IDBTransactionBackendProxy.cpp:

(WebKit::IDBTransactionBackendProxy::objectStore):

src/IDBTransactionBackendProxy.h:

(IDBTransactionBackendProxy):

src/WebIDBMetadata.cpp:

(WebKit::WebIDBMetadata::WebIDBMetadata):
(WebKit::WebIDBMetadata::operator IDBDatabaseMetadata):

src/WebIDBTransactionImpl.cpp:

(WebKit::WebIDBTransactionImpl::objectStore):
(WebKit):

src/WebIDBTransactionImpl.h:
tests/IDBDatabaseBackendTest.cpp:

10:48 AM Changeset in webkit [133194] by abarth@webkit.org

4 edits in trunk/Source/WebCore

[V8] The DOMWrapperVisitor abstraction is no longer needed
https://bugs.webkit.org/show_bug.cgi?id=100965

Reviewed by Kentaro Hara.

This patch removes the DOMWrapperVisitor interface because it is no
longer needed. As a consequence, DOMWrapperMaps no longer need to
support enumeration, and we can move more DOM objects to use the faster
intrusive wrappers.

There was one remaining user of DOMWrapperVisitor in the
ScriptProfiler, which I've moved over to enumerating objects directly
from V8, similar to a function above it in the same file.

bindings/v8/DOMWrapperMap.h:

(WebCore):
(DOMWrapperMap):

bindings/v8/IntrusiveDOMWrapperMap.h:
bindings/v8/ScriptProfiler.cpp:

(WebCore::ScriptProfiler::visitNodeWrappers):
(WebCore::ScriptProfiler::visitExternalArrays):

10:46 AM Changeset in webkit [133193] by mkwst@chromium.org

14 edits
2 adds in trunk

CSP 1.0: Warn when old-style directives encountered.
https://bugs.webkit.org/show_bug.cgi?id=100883

Reviewed by Adam Barth.

Source/WebCore:

In Mozilla's pre-W3C-spec implementation, a few directives are
implemented that were either renamed, reworked, or removed from CSP 1.0.
This patch adds special warning messages for three of those directives
to set developer expectations correctly.

Test: http/tests/security/contentSecurityPolicy/source-list-parsing-deprecated.html

page/ContentSecurityPolicy.cpp:

(WebCore::CSPDirectiveList::parseDirective):
(WebCore::CSPDirectiveList::addDirective):
(WebCore::ContentSecurityPolicy::reportUnsupportedDirective):

page/ContentSecurityPolicy.h:

Rename 'reportUnrecognizedDirectives' to
'reportUnsupportedDirectives', and teach it to give more descriptive
error messages when encountering 'allow', 'options', and
'policy-uri'.

LayoutTests:

http/tests/security/contentSecurityPolicy/directive-parsing-01-expected.txt:
http/tests/security/contentSecurityPolicy/directive-parsing-multiple-headers-expected.txt:
http/tests/security/contentSecurityPolicy/inline-script-blocked-goofy-expected.txt:
http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url-expected.txt:
http/tests/security/contentSecurityPolicy/javascript-url-allowed-expected.txt:
http/tests/security/contentSecurityPolicy/javascript-url-blocked-expected.txt:
http/tests/security/contentSecurityPolicy/script-src-in-iframe-expected.txt:
http/tests/security/contentSecurityPolicy/script-src-none-expected.txt:
http/tests/security/contentSecurityPolicy/source-list-parsing-deprecated-expected.txt: Added.
http/tests/security/contentSecurityPolicy/source-list-parsing-deprecated.html: Added.
platform/chromium/http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url-expected.txt:
platform/chromium/http/tests/security/contentSecurityPolicy/javascript-url-blocked-expected.txt:

10:45 AM Changeset in webkit [133192] 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 2012-11-01

DEPS:

10:42 AM Changeset in webkit [133191] by pfeldman@chromium.org

3 edits in trunk/Source/WebKit/chromium

Web Inspector: expose Inspector.detached event factory to the embedder.
https://bugs.webkit.org/show_bug.cgi?id=100967

Reviewed by Vsevolod Vlasov.

That way embedder will be able to send messages on behalf of the backend being destructed.

public/WebDevToolsAgent.h:

(WebDevToolsAgent):

src/WebDevToolsAgentImpl.cpp:

(SerializingFrontendChannel):
(WebKit::SerializingFrontendChannel::sendMessageToFrontend):
(WebKit::WebDevToolsAgent::inspectorDetachedEvent):
(WebKit):
(WebKit::WebDevToolsAgent::workerDisconnectedFromWorkerEvent):
(WebKit::WebDevToolsAgent::disconnectEventAsText):

10:31 AM Changeset in webkit [133190] by tony@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed, marking http/tests/xmlhttprequest/xmlhttprequest-test-send-flag.html
as slow on Linux Debug based on the flakiness dashboard.

platform/chromium/TestExpectations:

10:05 AM Changeset in webkit [133189] by pfeldman@chromium.org

5 edits in trunk/Source/WebCore

Web Inspector: Update CodeMirror to v3
https://bugs.webkit.org/show_bug.cgi?id=99319

Reviewed by Vsevolod Vlasov.

Updated to ToT v3.

inspector/front-end/CodeMirrorTextEditor.js:

(WebInspector.CodeMirrorTextEditor):
(WebInspector.CodeMirrorTextEditor.prototype._gutterClick):
(WebInspector.CodeMirrorTextEditor.prototype.addBreakpoint):
(WebInspector.CodeMirrorTextEditor.prototype.removeBreakpoint):
(WebInspector.CodeMirrorTextEditor.prototype.setExecutionLine):
(WebInspector.CodeMirrorTextEditor.prototype.clearExecutionLine):
(WebInspector.CodeMirrorTextEditor.prototype.highlightLine):
(WebInspector.CodeMirrorTextEditor.prototype.clearLineHighlight):
(WebInspector.CodeMirrorTextEditor.prototype._change):

inspector/front-end/cm/cmdevtools.css:

(.CodeMirror):
(.CodeMirror-linenumber):
(.cm-breakpoint):

inspector/front-end/cm/codemirror.css:

(.CodeMirror):
(.CodeMirror-scroll):
(.CodeMirror-lines):
(.CodeMirror pre):
(.CodeMirror-scrollbar-filler):
(.CodeMirror-gutters):
(.CodeMirror-linenumbers):
(.CodeMirror-linenumber):
(.CodeMirror pre.CodeMirror-cursor):
(.CodeMirror pre.CodeMirror-secondarycursor):
(.cm-keymap-fat-cursor pre.CodeMirror-cursor):
(.cm-keymap-fat-cursor pre.CodeMirror-cursor:not(#nonsense_id)):
(.CodeMirror pre.CodeMirror-cursor.CodeMirror-overwrite):
(.cm-s-default .cm-keyword):
(.cm-s-default .cm-atom):
(.cm-s-default .cm-number):
(.cm-s-default .cm-def):
(.cm-s-default .cm-variable):
(.cm-s-default .cm-variable-2):
(.cm-s-default .cm-variable-3):
(.cm-s-default .cm-property):
(.cm-s-default .cm-operator):
(.cm-s-default .cm-comment):
(.cm-s-default .cm-string):
(.cm-s-default .cm-string-2):
(.cm-s-default .cm-meta):
(.cm-s-default .cm-error):
(.cm-s-default .cm-qualifier):
(.cm-s-default .cm-builtin):
(.cm-s-default .cm-bracket):
(.cm-s-default .cm-tag):
(.cm-s-default .cm-attribute):
(.cm-s-default .cm-header):
(.cm-s-default .cm-quote):
(.cm-s-default .cm-hr):
(.cm-s-default .cm-link):
(.cm-header, .cm-strong):
(.cm-em):
(.cm-emstrong):
(.cm-link):
(.cm-invalidchar):
(div.CodeMirror span.CodeMirror-matchingbracket):
(div.CodeMirror span.CodeMirror-nonmatchingbracket):
(.CodeMirror-sizer):
(.CodeMirror-vscrollbar, .CodeMirror-hscrollbar, .CodeMirror-scrollbar-filler):
(.CodeMirror-vscrollbar):
(.CodeMirror-hscrollbar):
(.CodeMirror-gutter):
(.CodeMirror-gutter-elt):
(.CodeMirror-linebackground):
(.CodeMirror-linewidget):
(.CodeMirror-measure):
(.CodeMirror-measure pre):
(.CodeMirror-selected):
(.CodeMirror-focused .CodeMirror-selected):
(.CodeMirror span):

inspector/front-end/cm/codemirror.js:

(window.CodeMirror.):
(window.CodeMirror):

inspector/front-end/utilities.js:

9:51 AM Changeset in webkit [133188] by commit-queue@webkit.org

3 edits in trunk/Source

Source/WebCore: [BlackBerry] Update BB10 form theme.
https://bugs.webkit.org/show_bug.cgi?id=100760

Patch by Tiancheng Jiang <tijiang@rim.com> on 2012-11-01
Reviewed by Rob Buis.

RIM PR 235194.

Check img pointer is null, if so, do early return.

platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore::drawControl):
(WebCore::drawThreeSlice):
(WebCore::drawNineSlice):

Source/WebKit/chromium: [Chromium] debug builds: Use after free under ~PageOverlay()
https://bugs.webkit.org/show_bug.cgi?id=100930

Patch by Vsevolod Vlasov <vsevik@chromium.org> on 2012-11-01
Reviewed by Pavel Feldman.

Changed members order in PageOverlay so that GraphicsLayer is destroyed before GraphicsLayerClient.

src/PageOverlay.h:

(PageOverlay):

9:47 AM Changeset in webkit [133187] by abarth@webkit.org

4 edits in trunk/Source/WebCore

[V8] The V8DOMMap visitors are no longer needed
https://bugs.webkit.org/show_bug.cgi?id=100963

Reviewed by Kentaro Hara.

This patch inlines visitAllDOMNodes into its one caller (and removes
one layer of visitor adaptor abstraction).

bindings/v8/ScriptProfiler.cpp:

(WebCore::ScriptProfiler::visitNodeWrappers):

bindings/v8/V8DOMMap.cpp:
bindings/v8/V8DOMMap.h:

(WebCore):

9:46 AM Changeset in webkit [133186] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[BlackBerry] Update BB10 form theme.
https://bugs.webkit.org/show_bug.cgi?id=100760

Patch by Tiancheng Jiang <tijiang@rim.com> on 2012-11-01
Reviewed by Rob Buis.

RIM PR 235194.

Check img pointer is null, if so, do early return.

platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore::drawControl):
(WebCore::drawThreeSlice):
(WebCore::drawNineSlice):

9:45 AM Changeset in webkit [133185] by wjmaclean@chromium.org

2 edits in trunk/Source/Platform

[chromium] Make WebLayerTreeView::adjustEventPointForPinchZoom pure virtual.
https://bugs.webkit.org/show_bug.cgi?id=100875

Reviewed by James Robinson.

This CL removes the default implementation for adjustEventPointForPinchZoom(), which
was necessary for landing the original patch in https://bugs.webkit.org/show_bug.cgi?id=100542.

chromium/public/WebLayerTreeView.h:

(WebLayerTreeView):

9:24 AM Changeset in webkit [133184] by leoyang@rim.com

2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Use clipped background rect in backing store
https://bugs.webkit.org/show_bug.cgi?id=100960

Reviewed by Yong Li.
Reviewed internally by Arvid Nilsson.

We calculated clipped background rect but didn't really use it when painting
the background. Use the correct rect to avoid ASSERT failure in platform code.

Api/BackingStore.cpp:

(BlackBerry::WebKit::BackingStorePrivate::paintDefaultBackground):

9:11 AM Changeset in webkit [133183] by eae@chromium.org

9 edits
2 deletes in trunk/LayoutTests

Unreviewed chromium windows rebaseline.

platform/chromium-linux-x86/svg/zoom: Removed.
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.txt: Removed.
platform/chromium-win/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/chromium-win/compositing/overflow/scroll-ancestor-update-expected.png:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.txt:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-absolute-size-expected.txt:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.png:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.txt:

8:55 AM Changeset in webkit [133182] by commit-queue@webkit.org

6 edits in trunk/Source/WebKit2

Coordinated Graphics: Clean up headers of Coordinated Graphics.
https://bugs.webkit.org/show_bug.cgi?id=100907

Patch by Huang Dongsung <luxtella@company100.net> on 2012-11-01
Reviewed by Noam Rosenthal.

This patch performs:

Remove unused methods.
Change public methods to private if needed.
Put virtual, OVERRIDE and explicit keywords if needed.

UIProcess/CoordinatedGraphics/LayerTreeCoordinatorProxy.h:

(LayerTreeCoordinatorProxy):

UIProcess/CoordinatedGraphics/LayerTreeRenderer.h:

(LayerTreeRenderer):

WebProcess/WebPage/CoordinatedGraphics/CoordinatedGraphicsLayer.cpp:
WebProcess/WebPage/CoordinatedGraphics/CoordinatedGraphicsLayer.h:

(CoordinatedGraphicsLayer):
(WebCore::CoordinatedGraphicsLayer::fixedToViewport):
(WebCore::CoordinatedGraphicsLayer::setMaskTarget):

WebProcess/WebPage/CoordinatedGraphics/LayerTreeCoordinator.h:

(LayerTreeCoordinator):

8:51 AM Changeset in webkit [133181] by eae@chromium.org

18 edits
1 add
5 deletes in trunk/LayoutTests

Unreviewed chromium rebaseline for r133172.

platform/chromium-linux-x86/svg/zoom/page: Removed.
platform/chromium-linux-x86/svg/zoom/page/zoom-svg-as-relative-image-expected.txt: Removed.
platform/chromium-mac-lion/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/chromium-mac-lion/compositing/overflow/scroll-ancestor-update-expected.png:
platform/chromium-mac-lion/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png: Added.
platform/chromium-mac-lion/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
platform/chromium-mac-snowleopard/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/chromium-mac-snowleopard/compositing/overflow/scroll-ancestor-update-expected.png:
platform/chromium-mac-snowleopard/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png:
platform/chromium-mac-snowleopard/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
platform/chromium-mac/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/chromium-mac/compositing/overflow/scroll-ancestor-update-expected.png:
platform/chromium-mac/svg/as-image/svg-non-integer-scaled-image-expected.png:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.txt:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-absolute-size-expected.txt:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.png:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.txt:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
platform/chromium-mac/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.txt:
platform/efl/compositing/overflow/overflow-compositing-descendant-expected.png: Removed.
platform/efl/compositing/overflow/scroll-ancestor-update-expected.png: Removed.
platform/efl/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png: Removed.
platform/efl/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.png: Removed.
platform/efl/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png: Removed.

8:41 AM Changeset in webkit [133180] by reed@google.com

2 edits in trunk/LayoutTests

Add suppressions for images for new skia roll
https://bugs.webkit.org/show_bug.cgi?id=100961

Reviewed by NOBODY.

New skia roll improves clipped antialias hairlines, and this means a few images need to be rebaselined.
crbug.com/148638

platform/chromium/TestExpectations:

8:40 AM Changeset in webkit [133179] by aroben@webkit.org

2 edits in trunk/Tools

Crash beneath WKRelease after failed load in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=100954

Reviewed by Sam Weinig.

There were just some simple retain/release bugs.

MiniBrowser/mac/WK2BrowserWindowController.m:

(-[WK2BrowserWindowController updateTextFieldFromURL:]):
(-[WK2BrowserWindowController updateProvisionalURLForFrame:]):
Don't try to release null URLs.

(-[WK2BrowserWindowController updateCommittedURLForFrame:]): Don't
release a URL for which we don't have ownership.

8:33 AM Changeset in webkit [133178] by abarth@webkit.org

2 edits in trunk/Source/WebCore

Unreviewed. Build fix.

bindings/v8/V8GCController.cpp:

8:32 AM Changeset in webkit [133177] by commit-queue@webkit.org

13 edits
4 adds in trunk/Source

[EFL][WK2] Add Ewk_Auth_Request API
https://bugs.webkit.org/show_bug.cgi?id=100858

Patch by Christophe Dumez <Christophe Dumez> on 2012-11-01
Reviewed by Kenneth Rohde Christiansen.

Source/WebCore:

Provide an AuthenticationClient for EFL port in WebCore so
that we can handle authentication similarly to other ports
of WebKit.

No new tests, no behavior change for layout tests.

PlatformEfl.cmake: Add AuthenticationChallengeSoup.cpp to CMake.
loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::didReceiveAuthenticationChallenge):

platform/network/ResourceHandle.h:

(ResourceHandle): Have ResourceHandle subclass AuthenticationClient like
most of the other ports.

platform/network/soup/AuthenticationChallengeSoup.cpp: Fix initialization

of previousFailureCount member. It should be 1 if we are retrying
authentication and 0 otherwise, not the opposite.
(WebCore::AuthenticationChallenge::AuthenticationChallenge):

platform/network/soup/ResourceHandleSoup.cpp: Provide implementation for EFL

port of AuthenticationClient methods.
(WebCore::WebCoreSynchronousLoader::didReceiveAuthenticationChallenge):
(WebCoreSynchronousLoader):
(WebCore):
(WebCore::ResourceHandle::didReceiveAuthenticationChallenge):
(WebCore::ResourceHandle::receivedRequestToContinueWithoutCredential):
(WebCore::ResourceHandle::receivedCredential):
(WebCore::ResourceHandle::receivedCancellation):
(WebCore::authenticateCallback):
(WebCore::ResourceHandle::defaultSession):

Source/WebKit2:

Add Ewk_Auth_Request API for the client to handle
HTTP authentication.

PlatformEfl.cmake:
UIProcess/API/efl/EWebKit2.h:
UIProcess/API/efl/EwkViewCallbacks.h:

(EwkViewCallbacks):

UIProcess/API/efl/ewk_auth_request.cpp: Added.

(Ewk_Auth_Request::Ewk_Auth_Request):
(Ewk_Auth_Request::~Ewk_Auth_Request):
(Ewk_Auth_Request::suggestedUsername):
(Ewk_Auth_Request::realm):
(Ewk_Auth_Request::host):
(Ewk_Auth_Request::cancel):
(Ewk_Auth_Request::authenticate):
(Ewk_Auth_Request::isRetrying):
(ewk_auth_request_ref):
(ewk_auth_request_unref):
(ewk_auth_request_suggested_username_get):
(ewk_auth_request_cancel):
(ewk_auth_request_authenticate):
(ewk_auth_request_retrying_get):
(ewk_auth_request_realm_get):
(ewk_auth_request_host_get):

UIProcess/API/efl/ewk_auth_request.h: Added.
UIProcess/API/efl/ewk_auth_request_private.h: Added.

(WebKit):
(Ewk_Auth_Request):
(Ewk_Auth_Request::create):

UIProcess/API/efl/ewk_view.h:
UIProcess/API/efl/tests/test_ewk2_auth_request.cpp: Added corresponding API tests.

(serverCallback):
(onAuthenticationRequest):
(TEST_F):
(onResourceLoadResponse):

UIProcess/efl/PageLoadClientEfl.cpp:

(WebKit::PageLoadClientEfl::didReceiveAuthenticationChallengeInFrame):
(WebKit):
(WebKit::PageLoadClientEfl::PageLoadClientEfl):

UIProcess/efl/PageLoadClientEfl.h:

(PageLoadClientEfl):

8:19 AM Changeset in webkit [133176] by jocelyn.turcotte@digia.com

2 edits in trunk/Source/WebKit2

Fix the Mac breakage
https://bugs.webkit.org/show_bug.cgi?id=100383

Reviewed by Kenneth Rohde Christiansen.

acceleratedCompositingForFixedPositionEnabled needs to be enabled
for fixed layout to work efficiently but it's wrong to force it
to false if fixed layout isn't enabled.

Separate hard requirements for fixed layout from rendering optimizations
and guard the later with USE(COORDINATED_GRAPHICS).
Also add an early return to make sure that ports always having a false
useFixedLayout creation parameter won't get their settings adjusted.

WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::setUseFixedLayout):

8:17 AM Changeset in webkit [133175] by jocelyn.turcotte@digia.com

4 edits in trunk/Source

WebIconDatabase: Properly clean up on destruction
https://bugs.webkit.org/show_bug.cgi?id=100237

Reviewed by Brady Eidson.

Source/WebCore:

It's now possible that the IconDatabase gets destroyed since WebKit2
isn't using it as a singleton.

Check that the database was properly closed rather than asserting that
it won't get deleted.

loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::~IconDatabase):

Source/WebKit2:

UIProcess/WebIconDatabase.cpp:

(WebKit::WebIconDatabase::invalidate):
Make sure that IconDatabaseBase::iconDatabase won't return a dangling pointer.

8:16 AM Changeset in webkit [133174] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Web Inspector: make component subitems use parent color in native memory snapshots.
https://bugs.webkit.org/show_bug.cgi?id=100876

A component subitems use the color of the component itself if the color
is not explicitly specified for the subitem.

Patch by Alexei Filippov <alph@chromium.org> on 2012-11-01
Reviewed by Yury Semikhatsky.

inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.NativeSnapshotNode.prototype._createSizeCell):
(WebInspector.MemoryBlockViewProperties._initialize):
(WebInspector.MemoryBlockViewProperties._forMemoryBlock):

8:06 AM Changeset in webkit [133173] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Remove some no-longer-flaky tests
from TestExpectations.

platform/chromium/TestExpectations:

8:00 AM Changeset in webkit [133172] by eae@chromium.org

38 edits
2 adds in trunk

[subpixel] Change intrinsicSize to LayoutUnit
https://bugs.webkit.org/show_bug.cgi?id=99104

Reviewed by Levi Weintraub.

Source/WebCore:

Change RenderReplaced and intrinsicSize to LayoutUnit to avoid
rounding problems when zooming/scaling. Also change imageSize to
LayoutUnit as it can return a scaled size.

Test: fast/sub-pixel/tiled-canvas-elements.html

html/ImageDocument.cpp:

(WebCore::ImageDocumentParser::finish):

loader/cache/CachedImage.cpp:

(WebCore::CachedImage::imageSizeForRenderer):

loader/cache/CachedImage.h:

(CachedImage):

platform/graphics/FractionalLayoutSize.h:

(FractionalLayoutSize):
(WebCore::FractionalLayoutSize::scale):
(WebCore::FractionalLayoutSize::clampToMinimumSize):

rendering/RenderBox.h:

(WebCore::RenderBox::intrinsicSize):

rendering/RenderHTMLCanvas.cpp:

(WebCore::RenderHTMLCanvas::canvasSizeChanged):

rendering/RenderImage.cpp:

(WebCore::RenderImage::imageChanged):
(WebCore::RenderImage::updateIntrinsicSizeIfNeeded):
(WebCore::RenderImage::paintReplaced):
(WebCore::RenderImage::minimumReplacedHeight):

rendering/RenderImage.h:

(RenderImage):

rendering/RenderImageResource.h:

(WebCore::RenderImageResource::imageSize):

rendering/RenderImageResourceStyleImage.h:
rendering/RenderReplaced.cpp:

(WebCore::RenderReplaced::RenderReplaced):
(WebCore::RenderReplaced::computeAspectRatioInformationForRenderBox):
(WebCore::RenderReplaced::computeReplacedLogicalWidth):
(WebCore::RenderReplaced::computeReplacedLogicalHeight):

rendering/RenderReplaced.h:

(RenderReplaced):
(WebCore::RenderReplaced::minimumReplacedHeight):
(WebCore::RenderReplaced::setIntrinsicSize):

rendering/RenderVideo.cpp:

(WebCore::RenderVideo::updateIntrinsicSize):
(WebCore::RenderVideo::calculateIntrinsicSize):
(WebCore::RenderVideo::videoBox):
(WebCore::RenderVideo::minimumReplacedHeight):

rendering/RenderVideo.h:

(RenderVideo):

rendering/style/StyleCachedImage.cpp:

(WebCore::StyleCachedImage::imageSize):

rendering/style/StyleCachedImage.h:

(StyleCachedImage):

rendering/style/StyleCachedImageSet.cpp:

(WebCore::StyleCachedImageSet::imageSize):

rendering/style/StyleCachedImageSet.h:

(StyleCachedImageSet):

rendering/style/StyleGeneratedImage.cpp:

(WebCore::StyleGeneratedImage::imageSize):
(WebCore::StyleGeneratedImage::computeIntrinsicDimensions):

rendering/style/StyleGeneratedImage.h:

(StyleGeneratedImage):

rendering/style/StyleImage.h:

(StyleImage):

rendering/style/StylePendingImage.h:

LayoutTests:

Add test for tiled replaced elements and update existing expectations as needed.

fast/sub-pixel/tiled-canvas-elements-expected.txt: Added.
fast/sub-pixel/tiled-canvas-elements.html: Added.
platform/chromium-linux/compositing/overflow/overflow-compositing-descendant-expected.png:
platform/chromium-linux/compositing/overflow/scroll-ancestor-update-expected.png:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.png:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-absolute-size-2-expected.txt:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-absolute-size-expected.txt:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.png:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.png:
platform/chromium-linux/svg/zoom/page/zoom-svg-through-object-with-percentage-size-expected.txt:
platform/chromium-win/svg/as-image/svg-non-integer-scaled-image-expected.png:
platform/chromium-win/svg/zoom/page/zoom-svg-through-object-with-auto-size-expected.txt:
platform/chromium/fast/reflections/reflection-with-zoom-expected.txt:
platform/chromium/svg/as-image/svg-non-integer-scaled-image-expected.txt:
platform/chromium/svg/zoom/page/zoom-svg-as-image-expected.txt:
platform/chromium/svg/zoom/page/zoom-svg-as-relative-image-expected.txt:

7:56 AM Changeset in webkit [133171] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. New reftest from
http://trac.webkit.org/changeset/133146 doesn't seem to work on
MacOS 10.8.

platform/chromium/TestExpectations:

7:56 AM Changeset in webkit [133170] by kenneth@webkit.org

2 edits in trunk/Source/WebKit2

[EFL] Fix event mapping for WebKit2 with coordinated graphics
https://bugs.webkit.org/show_bug.cgi?id=100956

Reviewed by Alexis Menard.

Apply scale after translation.

UIProcess/API/efl/ewk_view.cpp:

(toWebContentTransform):

7:52 AM Changeset in webkit [133169] by abarth@webkit.org

8 edits in trunk/Source/WebCore

[V8] Unify the V8GCController visitors
https://bugs.webkit.org/show_bug.cgi?id=100897

Reviewed by Eric Seidel.

After this patch, we use a single visitor for all DOM wrappers,
regardless of type. We also visit all the wrappers in one pass by
calling v8::V8::VisitHandlesWithClassIds directly rather than via
visitAllDOMNodes.

This patch also introduces a wrapper class ID for non-Node DOM objects.
Previously, only DOM nodes had a class ID.

bindings/v8/IntrusiveDOMWrapperMap.h:
bindings/v8/ScriptProfiler.cpp:

(WebCore::retainedDOMInfo):
(WebCore::ScriptProfiler::initialize):

bindings/v8/V8DOMMap.cpp:

(WebCore::visitAllDOMNodes):

bindings/v8/V8DOMWrapper.cpp:

(WebCore::V8DOMWrapper::setJSWrapperForDOMNode):

bindings/v8/V8DOMWrapper.h:

(WebCore::V8DOMWrapper::setJSWrapperForDOMObject):

bindings/v8/V8GCController.cpp:

(WebCore::GCHandleVisitor::notifyFinished):
(GCHandleVisitor):
(WebCore::V8GCController::majorGCPrologue):

bindings/v8/WrapperTypeInfo.h:

(WebCore):

7:44 AM Changeset in webkit [133168] by mkwst@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.

Skipping http/tests/security/contentSecurityPolicy/object-src-none-blocked.html
as it's consistently leaking state into the next text (object-src-url-allowed.html).
Filed https://bugs.webkit.org/show_bug.cgi?id=100955 to follow up.

platform/chromium/TestExpectations:

7:26 AM Changeset in webkit [133167] by senorblanco@chromium.org

15 edits
3 deletes in trunk/Source

Unreviewed, rolling out r133143.
http://trac.webkit.org/changeset/133143
https://bugs.webkit.org/show_bug.cgi?id=96894

Causing content_browsertests failures

Source/WebCore:

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
dom/DeviceOrientationClient.h:

(DeviceOrientationClient):

dom/DeviceOrientationController.cpp:

(WebCore::DeviceOrientationController::DeviceOrientationController):
(WebCore):
(WebCore::DeviceOrientationController::~DeviceOrientationController):
(WebCore::DeviceOrientationController::timerFired):
(WebCore::DeviceOrientationController::addListener):
(WebCore::DeviceOrientationController::removeListener):
(WebCore::DeviceOrientationController::removeAllListeners):
(WebCore::DeviceOrientationController::suspendEventsForAllListeners):
(WebCore::DeviceOrientationController::resumeEventsForAllListeners):
(WebCore::DeviceOrientationController::didChangeDeviceOrientation):
(WebCore::DeviceOrientationController::supplementName):

dom/DeviceOrientationController.h:

(WebCore):
(DeviceOrientationController):
(WebCore::DeviceOrientationController::isActive):
(WebCore::DeviceOrientationController::client):
(WebCore::DeviceOrientationController::from):

dom/Document.cpp:

(WebCore::Document::suspendActiveDOMObjects):
(WebCore::Document::resumeActiveDOMObjects):

loader/EmptyClients.h:

(WebCore::EmptyDeviceOrientationClient::startUpdating):
(WebCore::EmptyDeviceOrientationClient::stopUpdating):

page/DOMWindow.cpp:

(WebCore::DOMWindow::addEventListener):
(WebCore::DOMWindow::removeEventListener):
(WebCore::DOMWindow::removeAllEventListeners):

page/DeviceClient.h: Removed.
page/DeviceController.cpp: Removed.
page/DeviceController.h: Removed.

Source/WebKit/qt:

WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setMockDeviceOrientation):

7:25 AM Changeset in webkit [133166] by rwlbuis@webkit.org

4 edits in trunk/Tools

[BlackBerry] Fix editing/text-iterator/findString-expected.html fail
https://bugs.webkit.org/show_bug.cgi?id=100902

Reviewed by Yong Li.

PR 210039

Fix the test my using the Page::findString method instead of our own in-page searcher.
This also syncs some of the changes we did regarding String usage.

DumpRenderTree/blackberry/DumpRenderTree.cpp:

(BlackBerry::WebKit::DumpRenderTree::dumpFramesAsText):
(BlackBerry::WebKit::dumpHistoryItem):
(BlackBerry::WebKit::dumpBackForwardListForWebView):
(BlackBerry::WebKit::DumpRenderTree::dump):
(BlackBerry::WebKit::DumpRenderTree::didFinishDocumentLoadForFrame):
(BlackBerry::WebKit::DumpRenderTree::didReceiveTitleForFrame):
(BlackBerry::WebKit::DumpRenderTree::runJavaScriptAlert):

DumpRenderTree/blackberry/DumpRenderTreeBlackBerry.h:

(DumpRenderTree):

DumpRenderTree/blackberry/TestRunnerBlackBerry.cpp:

(TestRunner::setMockGeolocationError):
(TestRunner::layerTreeAsText):
(TestRunner::findString):
(TestRunner::setStorageDatabaseIdleInterval):

7:24 AM Changeset in webkit [133165] by rakuco@webkit.org

1 delete in trunk/LayoutTests/fast/css3-text-decoration

Remove empty directory after r132043.

7:22 AM Changeset in webkit [133164] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit/chromium

[chromium] Avoid attempting to defer commits when they are already deferred
https://bugs.webkit.org/show_bug.cgi?id=100913

Patch by James Robinson <jamesr@chromium.org> on 2012-11-01
Reviewed by Adrienne Walker.

In threaded compositing mode, it's possible to load and unload FrameViews without any
intervening invalidations. In this case, we enter setIsAcceleratedCompositingActive
with commits already deferred. There's no need to defer again and doing so trips
an assertion in the implementation.

src/WebViewImpl.cpp:

(WebKit::WebViewImpl::setIsAcceleratedCompositingActive):
(WebKit):

7:17 AM Changeset in webkit [133163] by commit-queue@webkit.org

2 edits in trunk/Tools

[EFL][WK2] Add File Chooser dialog support to MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=100822

Patch by Christophe Dumez <Christophe Dumez> on 2012-11-01
Reviewed by Kenneth Rohde Christiansen.

Add very basic file picker dialog to MiniBrowser so that
the user can select a file from the file system when
clicking an input element of type "file".

MiniBrowser/efl/main.c:

(close_file_picker):
(on_filepicker_deletion):
(on_fileselector_done):
(on_file_chooser_request):
(window_create):

7:17 AM Changeset in webkit [133162] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[EFL][AC]Free GL resources allocated by GraphicsContext3DEfl.
https://bugs.webkit.org/show_bug.cgi?id=100923.

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-11-01
Reviewed by Kenneth Rohde Christiansen.

GraphicsContext3DEfl creates FBO's, textures and render buffer's, but doesn't free them.
This patch makes sure that GraphicsContext3dEfl frees all the GL resources allocated by it.

platform/graphics/efl/GraphicsContext3DEfl.cpp:

(WebCore::GraphicsContext3D::~GraphicsContext3D):

7:14 AM Changeset in webkit [133161] by kenneth@webkit.org

7 edits in trunk/Source/WebKit2

[EFL] Correct our use of the coordinated graphics
https://bugs.webkit.org/show_bug.cgi?id=100947

Reviewed by Alexis Menard.

Replace the display(...) call with a regular update() on the view.
In the coordinated graphics case this updates our view using
paintToCurrentGLContext with our viewmodel matrix and clips it to
the viewport. This avoid the need to manually joining update regions.

Unneeded update calls have also been removed and the resizing
logic has been improved.

UIProcess/API/efl/EwkViewImpl.cpp:

(EwkViewImpl::displayTimerFired):
(EwkViewImpl::update):

UIProcess/API/efl/EwkViewImpl.h:

(WebCore):
(EwkViewImpl):
(EwkViewImpl::clearEvasGLSurface):

UIProcess/API/efl/ewk_view.cpp:

(_ewk_view_smart_calculate):

UIProcess/efl/PageClientImpl.cpp:

(WebKit::PageClientImpl::setViewNeedsDisplay):

UIProcess/efl/PageViewportControllerClientEfl.cpp:

(WebKit::PageViewportControllerClientEfl::setVisibleContentsRect):
(WebKit::PageViewportControllerClientEfl::didChangeContentsSize):
(WebKit::PageViewportControllerClientEfl::didChangeVisibleContents):

UIProcess/efl/PageViewportControllerClientEfl.h:

(PageViewportControllerClientEfl):

7:10 AM Changeset in webkit [133160] by rakuco@webkit.org

3 edits in trunk/Source/WebCore

Fix StylePropertySet/ElementAttributeData custom allocation in debug builds.
<http://webkit.org/b/100753>

Unreviewed debug bot crash fix after r133138.

There's additional padding after StylePropertySet and ElementAttributeData
in 64-bit debug builds since there are additional members in RefCountedBase.
Use 'sizeof(ImmutableFoo) - sizeof(void*)' as the base size of ImmutableFoo.

Patch by Andreas Kling <kling@webkit.org> on 2012-11-01

css/StylePropertySet.cpp:

(WebCore::sizeForImmutableStylePropertySetWithPropertyCount):

dom/ElementAttributeData.cpp:

(WebCore::sizeForImmutableElementAttributeDataWithAttributeCount):
(WebCore::ElementAttributeData::createImmutable):
(WebCore::ElementAttributeData::reportMemoryUsage):

7:05 AM Changeset in webkit [133159] 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 2012-11-01

DEPS:

6:54 AM Changeset in webkit [133158] by schenney@chromium.org

2 edits in trunk/Source/WebCore

FEImage::m_document is never cleared. Why not?
https://bugs.webkit.org/show_bug.cgi?id=99243

Reviewed by Dirk Schulze.

Adding a comment to explain why the failure to clear m_document is not a problem.

No new tests because no code change at all.

svg/graphics/filters/SVGFEImage.h:

(FEImage): Add a comment on the lifetime of m_document.

6:50 AM Changeset in webkit [133157] by commit-queue@webkit.org

3 edits in trunk/Source/WebCore

Web Inspector: Timeline: make "addRecord" unambiguous
https://bugs.webkit.org/show_bug.cgi?id=100761

Patch by Eugene Klyuchnikov <eustas.bug@gmail.com> on 2012-11-01
Reviewed by Pavel Feldman.

TimelinePresentationModel.addRecord accepts two parameters:
record and parentRecord. parentRecord is always root record.
Make this explicit by removing parentRecord parameter.

inspector/front-end/TimelinePanel.js: Removed parameter.
inspector/front-end/TimelinePresentationModel.js:

(WebInspector.TimelinePresentationModel.prototype.addRecord):
Used root recoed instead of parent record.

6:47 AM Changeset in webkit [133156] by charles.wei@torchmobile.com.cn

2 edits in trunk/Source/WebCore

[BlackBerry] Browser prematurely sends wrong credentials
https://bugs.webkit.org/show_bug.cgi?id=100585

Reviewed by George Staikos.

Don't send credentials to the server before been challenged.

platform/network/blackberry/NetworkManager.cpp:

(WebCore::NetworkManager::startJob):

6:39 AM Changeset in webkit [133155] by schenney@chromium.org

3 edits
3 adds in trunk

SVG as an image may recreate the renderer on zoom
https://bugs.webkit.org/show_bug.cgi?id=99508

Reviewed by Abhishek Arya.

Source/WebCore:

The SVGImage code, when SVG is used in <img> tags, caches the renderer
at the start of the painting method and re-uses the pointer at the end
of the method. However, when the page is zoomed the renderer may be
detached mid-method, thus leaving a stray pointer. The fix is to
re-fetch the pointer after the zooms.

Test: svg/as-image/img-zoom-svg-stylesheet.html

svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::drawSVGToImageBuffer): Re-fetch the renderer after
the zoom operations.

LayoutTests:

Test and ref-test-result for zooming of SVG content when loaded as-image, and the svg is actually a css file.

svg/as-image/img-zoom-svg-stylesheet-expected.html: Added.
svg/as-image/img-zoom-svg-stylesheet.html: Added.
svg/as-image/resources/svg-stylesheet-for-zoom.svg: Added.

6:27 AM Changeset in webkit [133154] by yael@webkit.org

2 edits in trunk/Tools

Unreviewed. Change my email address.

Scripts/webkitpy/common/config/committers.py:

6:18 AM Changeset in webkit [133153] by jocelyn.turcotte@digia.com

6 edits in trunk

[Qt] Double clicks/taps aren't passed down to the page
https://bugs.webkit.org/show_bug.cgi?id=100949

Reviewed by Kenneth Rohde Christiansen.

Source/WebKit2:

Since we are passing double click events as press events and that
they are preceeded by a real press event, double clicks are passed
down to the page as triple clicks.

Ignore double click events and let WebCore::EventHandler handle it
through press and release events.

Shared/qt/WebEventFactoryQt.cpp:

(WebKit::webEventTypeForEvent):

UIProcess/API/qt/qquickwebview.cpp:

(QQuickWebViewPrivate::handleMouseEvent):

UIProcess/qt/QtWebPageEventHandler.cpp:

(WebKit::QtWebPageEventHandler::handleInputEvent):

Tools:

Touch mocking shouldn't transform double click events as well,
this adds an extra touch update between clicks which prevents
double-tap detection from triggering on web pages.

Tested on the mobile version of Google maps.

MiniBrowser/qt/MiniBrowserApplication.cpp:

(MiniBrowserApplication::notify):

6:16 AM Changeset in webkit [133152] by pfeldman@chromium.org

5 edits in trunk/Source/WebCore

Web Inspector: introduce Inspector.detached message containing termination cause.
https://bugs.webkit.org/show_bug.cgi?id=100948

Reviewed by Yury Semikhatsky.

Now protocol clients have more information to process connection termination.

English.lproj/localizedStrings.js:
inspector/Inspector.json:
inspector/front-end/InspectorFrontendHostStub.js:

(WebInspector.RemoteDebuggingTerminatedScreen):

inspector/front-end/inspector.js:

(WebInspector.loaded.WebInspector.socket.onopen):
(WebInspector.loaded):
(WebInspector.detached):

5:27 AM Changeset in webkit [133151] by vsevik@chromium.org

9 edits in trunk

Web Inspector: Restoring breakpoints for formatted UISourceCode should be triggered by BreakpointManager, not ScriptsPanel.
https://bugs.webkit.org/show_bug.cgi?id=100593

Reviewed by Pavel Feldman.

Source/WebCore:

BreakpointManager now listen for FormattedChanged event and restores formatted breakpoints on it.
Removed now redundant callback from UISourceCode.setFormatted().

inspector/front-end/BreakpointManager.js:

(WebInspector.BreakpointManager.prototype._restoreBreakpoints):
(WebInspector.BreakpointManager.prototype._uiSourceCodeAdded):
(WebInspector.BreakpointManager.prototype._uiSourceCodeFormatted):
(WebInspector.BreakpointManager.prototype._uiSourceCodeRemoved):

inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel.prototype._addUISourceCode):
(WebInspector.ScriptsPanel.prototype._revealExecutionLine):
(WebInspector.ScriptsPanel.prototype._toggleFormatSource):

inspector/front-end/UISourceCode.js:

(WebInspector.UISourceCode.prototype._fireContentAvailable):
(WebInspector.UISourceCode.prototype.setFormatted.formattedChanged):
(WebInspector.UISourceCode.prototype.setFormatted.didGetContent):
(WebInspector.UISourceCode.prototype.setFormatted):

LayoutTests:

inspector/debugger/script-formatter-breakpoints.html:
inspector/debugger/script-formatter-console.html:
inspector/debugger/script-formatter-search-expected.txt:
inspector/debugger/script-formatter-search.html:

5:17 AM Changeset in webkit [133150] by yurys@chromium.org

24 edits in trunk

Web Inspector: Output code evaluated in the console the same as console.log
https://bugs.webkit.org/show_bug.cgi?id=100695

Reviewed by Pavel Feldman.

Source/WebCore:

Added an option to return object preview from evaluation commands in the protocol.
The option is used to generate preview for console eval results.

inspector/InjectedScript.cpp:

(WebCore::InjectedScript::evaluate):
(WebCore::InjectedScript::callFunctionOn):
(WebCore::InjectedScript::evaluateOnCallFrame):

inspector/InjectedScript.h:

(InjectedScript):

inspector/InjectedScriptSource.js:

(.):

inspector/Inspector.json:
inspector/InspectorDebuggerAgent.cpp:

(WebCore::InspectorDebuggerAgent::evaluateOnCallFrame):

inspector/InspectorDebuggerAgent.h:

(InspectorDebuggerAgent):

inspector/InspectorRuntimeAgent.cpp:

(WebCore::InspectorRuntimeAgent::evaluate):
(WebCore::InspectorRuntimeAgent::callFunctionOn):

inspector/InspectorRuntimeAgent.h:

(InspectorRuntimeAgent):

inspector/front-end/ConsoleMessage.js:

(WebInspector.ConsoleMessageImpl.prototype.useArrayPreviewInFormatter):
(WebInspector.ConsoleMessageImpl.prototype._formatParameterAsArray):

inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleCommandResult.prototype.useArrayPreviewInFormatter):

inspector/front-end/DebuggerModel.js:

(WebInspector.DebuggerModel.prototype.evaluateOnSelectedCallFrame):
(WebInspector.DebuggerModel.CallFrame.prototype.evaluate):

inspector/front-end/ExtensionServer.js:

(WebInspector.ExtensionServer.prototype.evaluate):

inspector/front-end/JavaScriptSourceFrame.js:

(WebInspector.JavaScriptSourceFrame.prototype._resolveObjectForPopover):

inspector/front-end/RemoteObject.js:

(WebInspector.RemoteObject.prototype.):
(WebInspector.RemoteObject.prototype.callFunction):
(WebInspector.RemoteObject.prototype.callFunctionJSON):

inspector/front-end/RuntimeModel.js:

(WebInspector.RuntimeModel.prototype.evaluate):
(WebInspector.RuntimeModel.prototype.evaluated):

inspector/front-end/WatchExpressionsSidebarPane.js:

(WebInspector.WatchExpressionsSection.prototype.update):

LayoutTests:

Rebaselined some tests after object preview is started beeing used for
console eval results.

inspector/console/command-line-api-expected.txt:
inspector/console/command-line-api-inspect-expected.txt:
inspector/console/console-eval-global-expected.txt:
inspector/console/console-eval-undefined-override-expected.txt:
platform/chromium/inspector/console/console-eval-undefined-override-expected.txt:

4:35 AM Changeset in webkit [133149] by vsevik@chromium.org

2 edits in trunk/Source/WebCore

Web Inspector: Remove obsolete code from JavaScriptSourceFrame
https://bugs.webkit.org/show_bug.cgi?id=100594

Reviewed by Yury Semikhatsky.

Removed obsolete WorkingCopyCommitted listener.
Removed redundant code that removes breakpoints from _innerSetContent.

inspector/front-end/JavaScriptSourceFrame.js:

(WebInspector.JavaScriptSourceFrame):
(WebInspector.JavaScriptSourceFrame.prototype._innerSetContent):

4:32 AM Changeset in webkit [133148] by kling@webkit.org

2 edits in trunk/Source/WebCore

Update average StylePropertySet size estimation.
<http://webkit.org/b/100940>

Reviewed by Antti Koivisto.

Use sizeForImmutableStylePropertySetWithPropertyCount(2) as the average StylePropertySet
size to keep it in sync with the changed object memory layout.

css/StylePropertySet.cpp:

(WebCore::sizeForImmutableStylePropertySetWithPropertyCount):
(WebCore::StylePropertySet::createImmutable):
(WebCore::StylePropertySet::averageSizeInBytes):
(WebCore::StylePropertySet::reportMemoryUsage):

4:22 AM Changeset in webkit [133147] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[EFL][Qt][AC] Remove an unnecessary connection to X-Server.
https://bugs.webkit.org/show_bug.cgi?id=100628.

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-11-01
Reviewed by Kenneth Rohde Christiansen.

With Changset: https://bugs.webkit.org/show_bug.cgi?id=100523
GraphicsSurfacePrivate should always use the Display returned by offscreenwindow.
However, in GraphicsSurfacePrivate constructor we call XOpenDisplay before asking
display from offscreenwindow.
This patch removes unnecessary call to XOpenDisplay made in GraphicsSurfacePrivate.

platform/graphics/surfaces/qt/GraphicsSurfaceGLX.cpp:

(WebCore::GraphicsSurfacePrivate::GraphicsSurfacePrivate):

4:17 AM Changeset in webkit [133146] by mihnea@adobe.com

3 edits
2 adds in trunk

[CSSRegions]Former auto-height regions should not ignore their defined height
https://bugs.webkit.org/show_bug.cgi?id=100749

Reviewed by Julien Chaffraix.

Source/WebCore:

When a region with height auto has its height defined, we should also clear the override logical content height.
Otherwise, the region will use the wrong height when laying out content from the associated render flow thread.

Test: fast/regions/autoheight-definedheight-changenotdetected.html

rendering/RenderRegion.cpp:

(WebCore::RenderRegion::updateRegionHasAutoLogicalHeightFlag):

LayoutTests:

Add a test in which we flow content into a region chain with 2 regions, the first one having height auto.
After a first layout, the region with height auto has an override logical content height, which should be
cleared when the first region has its height changed from auto to defined.

fast/regions/autoheight-definedheight-changenotdetected-expected.html: Added.
fast/regions/autoheight-definedheight-changenotdetected.html: Added.

4:06 AM Changeset in webkit [133145] by yurys@chromium.org

2 edits in trunk/Source/WebCore

Memory instrumentation: do not report memory occupied by RenderObjects referenced from CSSImageGeneratorValue
https://bugs.webkit.org/show_bug.cgi?id=100934

Reviewed by Alexander Pavlov.

css/CSSImageGeneratorValue.cpp:

(WTF): Skip rederences to RenderObjects from CSSImageGeneratorValue when collecting
memory usage data.

4:04 AM Changeset in webkit [133144] by commit-queue@webkit.org

3 edits in trunk/Source/WebKit2

[AC][EFL][WK2] Rename PageViewportControllerClientEfl::m_pageViewportController to m_controller
https://bugs.webkit.org/show_bug.cgi?id=100933

Patch by Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com> on 2012-11-01
Reviewed by Kenneth Rohde Christiansen.

PageViewportControllerClientEfl::m_pageViewportController has to be renamed to m_controller
to keep consistency with other ports. Assertinons were also added.

UIProcess/efl/PageViewportControllerClientEfl.cpp:

(WebKit::PageViewportControllerClientEfl::PageViewportControllerClientEfl):
(WebKit::PageViewportControllerClientEfl::updateViewportSize):
(WebKit::PageViewportControllerClientEfl::setVisibleContentsRect):
(WebKit::PageViewportControllerClientEfl::didResumeContent):
(WebKit::PageViewportControllerClientEfl::setController):

UIProcess/efl/PageViewportControllerClientEfl.h:

(PageViewportControllerClientEfl):

3:38 AM Changeset in webkit [133143] by kihong.kwon@samsung.com

15 edits
3 adds in trunk/Source

Add DeviceController base-class to remove duplication of DeviceXXXControler
https://bugs.webkit.org/show_bug.cgi?id=96894

Reviewed by Hajime Morita.

Source/WebCore:

Add DeviceController which is extracted from DeviceOrientationController to remove duplication.
And soon-to-be-added DeviceMotionController and ProximityController.

Covered by existing tests.

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
dom/DeviceOrientationClient.h:
dom/DeviceOrientationController.cpp:

Remove member functions to move to DeviceController.

addListener(), removeListener(), removeAllListeners(), isActive()

(WebCore::DeviceOrientationController::DeviceOrientationController):
(WebCore::DeviceOrientationController::didChangeDeviceOrientation):
(WebCore::DeviceOrientationController::client):
(WebCore::DeviceOrientationController::hasLastData):
(WebCore::DeviceOrientationController::getLastEvent):
(WebCore::DeviceOrientationController::from):
(WebCore):

dom/DeviceOrientationController.h:

(WebCore):
(WebCore::DeviceOrientationController::~DeviceOrientationController):
(DeviceOrientationController):

dom/Document.cpp:

Remove suspendEventsForAllListeners() and resumeEventsForAllListeners() function calls.
These calls can be made by checking activeDOMObjectsAreSuspended() and activeDOMObjectsAreStopped() before dispatchEvent.
(WebCore::Document::suspendActiveDOMObjects):
(WebCore::Document::resumeActiveDOMObjects):

loader/EmptyClients.h:

(EmptyDeviceClient):
(WebCore::EmptyDeviceClient::startUpdating):
(WebCore::EmptyDeviceClient::stopUpdating):
(WebCore):

page/DOMWindow.cpp:

(WebCore::DOMWindow::addEventListener):
(WebCore::DOMWindow::removeEventListener):
(WebCore::DOMWindow::removeAllEventListeners):

page/DeviceClient.h: Added.

(WebCore):
(DeviceClient):
(WebCore::DeviceClient::~DeviceClient):

page/DeviceController.cpp: Added.

DeviceController has extracted functions from DeviceOrientationController and DeviceMotionController.

addDeviceEventListener(), removeDeviceEventlistener(), removeAllDeviceEventListeners(), dispatchDeviceEvent()

All kind of device event controller which has DeviceClient can be inherited from DeviceController.
(WebCore):
(WebCore::DeviceController::DeviceController):
(WebCore::DeviceController::addDeviceEventListener):
(WebCore::DeviceController::removeDeviceEventListener):
(WebCore::DeviceController::removeAllDeviceEventListeners):
(WebCore::DeviceController::dispatchDeviceEvent):
(WebCore::DeviceController::fireDeviceEvent):

page/DeviceController.h: Added.

(WebCore):
(DeviceController):
(WebCore::DeviceController::~DeviceController):
(WebCore::DeviceController::isActive):
(WebCore::DeviceController::client):
(WebCore::DeviceController::hasLastData):
(WebCore::DeviceController::getLastEvent):

Source/WebKit/qt:

Change client() to deviceOrientationClient() to get DeviceOrientationClient*.

WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setMockDeviceOrientation):

3:17 AM Changeset in webkit [133142] by podivilov@chromium.org

1 edit
1 add in trunk/LayoutTests

[chromium] Add new baseline for exif-orientation-css.html on linux.

platform/chromium-linux/platform/chromium/virtual/deferred/fast/images/exif-orientation-css-expected.png: Added.

2:28 AM Changeset in webkit [133141] by yurys@chromium.org

9 edits in trunk/Source

Memory instrumentation: report memory occupied by ResourceRequest instead of its base ResourceRequestBase
https://bugs.webkit.org/show_bug.cgi?id=100497

Reviewed by Alexander Pavlov.

Source/WebCore:

Added memory reporting method to chromium implementation of ResourceRequest.

platform/network/ResourceRequestBase.cpp:

(WebCore::ResourceRequestBase::reportMemoryUsageBase): Renamed reportMemoryUsage
on ResourceRequestBase to reportMemoryUsageBase and made it protected. I'd
rather make ResourceRequestBase::reportMemoryUsage virtual and override it
in the descendant but ResourceRequestBase doesn't have any virtual methods
and shouldn't be used directly(ResourceRequest should be used instead).

platform/network/ResourceRequestBase.h:

(ResourceRequestBase):

platform/network/chromium/ResourceRequest.cpp:

(WebCore::ResourceRequest::reportMemoryUsage):
(WebCore):

platform/network/chromium/ResourceRequest.h:

(ResourceRequest):

Source/WebKit/chromium:

Updated MemoryInstrumentationTest.ImageObserver so that it doesn't fail
when instrumentation of objects referenced by CachedImage changes.

tests/MemoryInstrumentationTest.cpp:

Source/WTF:

wtf/MemoryInstrumentation.h:

(WTF::MemoryClassInfo::addPrivateBuffer): do not report memory usage for empty buffer.

2:08 AM Changeset in webkit [133140] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

[EFL] Gardening after r133056
https://bugs.webkit.org/show_bug.cgi?id=100929

Unreviewed gardening.

Newly added compositing/tiling tests are failing on EFL port due to missing AC support.

Patch by Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com> on 2012-11-01

platform/efl/TestExpectations:

2:00 AM Changeset in webkit [133139] by podivilov@chromium.org

2 edits
1 add in trunk/LayoutTests

[chromium] Unreviewed, two fast/images/exif-orientation-* tests now pass.

platform/chromium-win/platform/chromium/virtual/deferred/fast/images/exif-orientation-css-expected.png: Added.
platform/chromium/TestExpectations:

1:55 AM Changeset in webkit [133138] by kling@webkit.org

5 edits in trunk/Source/WebCore

Pack immutable StylePropertySets harder on 64-bit.
<http://webkit.org/b/100753>
<rdar://problem/12599155>

Reviewed by Antti Koivisto.

Move away from using CSSProperty as internal storage for immutable StylePropertySets.
Instead use two arrays, one for property metadata (ID, shorthand ID, !important, ...)
and one for the CSSValue pointers. This saves 4 bytes per property on 64-bit.

Old object layout:

Ref count (4 bytes)
Metadata (4 bytes)
CSSProperty [N] (16 bytes each)

New object layout:

Ref count (4 bytes)
Metadata (4 bytes)
CSSValue* [N] (8 bytes each)
StylePropertyMetadata [N] (4 bytes each)

901kB progress on Membuster3 (22% overall reduction in StylePropertySet memory.)

The CSSProperty class sticks around for now, it's still used in mutable StylePropertySets
and by the StylePropertySet constructors.

css/CSSProperty.cpp:
css/CSSProperty.h:

(CSSProperty):
(WebCore::CSSProperty::CSSProperty):
(WebCore::CSSProperty::id):
(WebCore::CSSProperty::shorthandID):
(WebCore::CSSProperty::isImportant):
(WebCore::CSSProperty::metadata):

Break the bitfield from CSSProperty out into a StylePropertyMetadata class (actually a union.)

css/StylePropertySet.cpp:

(WebCore::immutableStylePropertySetSize):

Updated size calculation for immutable StylePropertySets, 1/4 smaller!

(WebCore::ImmutableStylePropertySet::ImmutableStylePropertySet):
(WebCore::ImmutableStylePropertySet::~ImmutableStylePropertySet):
(WebCore::MutableStylePropertySet::MutableStylePropertySet):
(WebCore::StylePropertySet::mergeAndOverrideOnConflict):
(WebCore::StylePropertySet::reportMemoryUsage):
(WebCore::StylePropertySet::PropertyReference::cssName):
(WebCore::StylePropertySet::PropertyReference::cssText):

css/StylePropertySet.h:

(WebCore::StylePropertySet::PropertyReference::id):
(WebCore::StylePropertySet::PropertyReference::shorthandID):
(WebCore::StylePropertySet::PropertyReference::isImportant):
(WebCore::StylePropertySet::PropertyReference::isInherited):
(WebCore::StylePropertySet::PropertyReference::isImplicit):
(PropertyReference):
(WebCore::StylePropertySet::PropertyReference::value):
(WebCore::StylePropertySet::PropertyReference::toCSSProperty):
(WebCore::StylePropertySet::PropertyReference::propertyMetadata):
(WebCore::StylePropertySet::PropertyReference::propertyValue):
(StylePropertySet):
(ImmutableStylePropertySet):
(WebCore::StylePropertySet::immutableValueArray):
(WebCore::StylePropertySet::immutableMetadataArray):

Refactored internal storage for StylePropertySet.

1:53 AM Changeset in webkit [133137] by podivilov@chromium.org

2 edits in trunk/LayoutTests

2012-11-01 Pavel Podivilov <podivilov@google.com>

[chromium] Unreviewed, remove several tests that started to pass from TestExpectations.

platform/chromium/TestExpectations:

1:04 AM Changeset in webkit [133136] by commit-queue@webkit.org

7 edits
7 deletes in trunk/Source/WebKit2

Unreviewed, rolling out r133126.
http://trac.webkit.org/changeset/133126
https://bugs.webkit.org/show_bug.cgi?id=100925

This made EFL API test break on EFL buildbots (Requested by
gyuyoung on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-11-01

PlatformEfl.cmake:
UIProcess/API/efl/EWebKit2.h:
UIProcess/API/efl/ewk_context.cpp:

(Ewk_Context::Ewk_Context):

UIProcess/API/efl/ewk_context.h:
UIProcess/API/efl/ewk_context_private.h:

(Ewk_Context):

UIProcess/API/efl/ewk_security_origin.cpp: Removed.
UIProcess/API/efl/ewk_security_origin.h: Removed.
UIProcess/API/efl/ewk_security_origin_private.h: Removed.
UIProcess/API/efl/ewk_storage_manager.cpp: Removed.
UIProcess/API/efl/ewk_storage_manager.h: Removed.
UIProcess/API/efl/ewk_storage_manager_private.h: Removed.
UIProcess/API/efl/tests/test_ewk2_context.cpp:
UIProcess/API/efl/tests/test_ewk2_storage_manager.cpp: Removed.

12:49 AM WebKitGTK/WebKit2Roadmap edited by Carlos Garcia Campos

g-i done (diff)

12:41 AM Changeset in webkit [133135] by fpizlo@apple.com

6 edits in trunk/Source/JavaScriptCore

DFG optimized string access code should be enabled
https://bugs.webkit.org/show_bug.cgi?id=100825

Reviewed by Oliver Hunt.

Removes prediction checks from the parser.

Fixes the handling of array mode refinement for strings. I.e. we don't do any refinement - we already know it's going to be a string. We could revisit this in the future, but for now the DFG lacks the ability to handle any array modes other than Array::String for string intrinsics, so this is as good as it gets.

Removes uses of isBlahSpeculation for checking if a mode is already checked. isBlahSpeculation implicitly checks if the SpeculatedType is not BOTTOM ("empty"), which breaks for checking if a mode is already checked since a mode may already be "checked" in the sense that we've proven that the code is unreachable.

~1% speed-up on V8v7, mostly from a speed-up on crypto, which uses string
intrinsics in one of the hot functions.

bytecode/SpeculatedType.h:

(JSC::speculationChecked):
(JSC):

dfg/DFGArrayMode.cpp:

(JSC::DFG::ArrayMode::alreadyChecked):

dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleIntrinsic):

dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileGetCharCodeAt):

12:31 AM Changeset in webkit [133134] by tkent@chromium.org

13 edits in trunk/Source

Remove unused Locale::parseDateTime
https://bugs.webkit.org/show_bug.cgi?id=100910

Reviewed by Kentaro Hara.

Source/WebCore:

For date/time input types, InputType::convertFromVisibleValue is never
called. convertFromVisibleValue is called when an inner editable node is
updated. However input elements don't have such editable nodes if
ENABLE_INPUT_MULTIPLE_FIELDS_UI is enabled, and a user can't edit the
inner editable node otherwise because we open date/time pickers when a
user try to change the field value.

We had used convertFromVisibleValue for input[type=date] with an old UI.

No new tests because of no behavior changes.

html/BaseDateAndTimeInputType.cpp:

(WebCore::BaseDateAndTimeInputType::convertFromVisibleValue):
Remove Locale::parseDateTiem and put ASSERT_NOT_REACHED.

platform/text/PlatformLocale.h: Remove parseDateTime.

platform/text/LocaleICU.h: Remove parseDateTime and a bogus comment.
platform/text/LocaleICU.cpp: Remove parseDateTime.

platform/text/LocaleNone.cpp: Remove parseDateTime.

platform/text/mac/LocaleMac.h: Remove parseDateTime.
platform/text/mac/LocaleMac.mm: Ditto.
platform/text/win/LocaleWin.h:
Remove parseDateTime, its helpers, and m_baseYear.
Remove appendNumber, appendTwoDigitsNumber, appendFourDigitsNumber, and formatDate. They don't exist.
platform/text/win/LocaleWin.cpp:

Remove the above functions.
(WebCore::LocaleWin::LocaleWin): Remove m_baseYear iniitalization.

Source/WebKit/chromium:

tests/LocaleMacTest.cpp: Remove tests for parseDateTime.
tests/LocaleWinTest.cpp: Ditto.

12:26 AM Changeset in webkit [133133] by commit-queue@webkit.org

7 edits in trunk/Source/WebKit2

[WK2] Add APIs to get/set encoding detector
https://bugs.webkit.org/show_bug.cgi?id=100066

Patch by Yuni Jeong <yhnet.jung@samsung.com> on 2012-11-01
Reviewed by Gyuyoung Kim.

Add setting APIs for encoding dector and a unit test.

Shared/WebPreferencesStore.h:

(WebKit):

UIProcess/API/C/WKPreferences.cpp:

(WKPreferencesSetUsesEncodingDetector):
(WKPreferencesGetUsesEncodingDetector):

UIProcess/API/C/WKPreferences.h:
UIProcess/API/efl/ewk_settings.cpp:

(ewk_setting_uses_encoding_detector_set):
(ewk_setting_uses_encoding_detector_get):

UIProcess/API/efl/ewk_settings.h:
UIProcess/API/efl/tests/test_ewk2_settings.cpp:

(TEST_F):

12:01 AM Changeset in webkit [133132] by keishi@webkit.org

2 edits in trunk/Source/WebKit/chromium

[Chromium] Page popup shows up at wrong locations when root view is scrolled
https://bugs.webkit.org/show_bug.cgi?id=100830

Reviewed by Kent Tamura.

We need to account for the scroll position when calculating the root view rectangle in the screen.

src/DateTimeChooserImpl.cpp:

(WebKit::DateTimeChooserImpl::writeDocument):

Oct 31, 2012:

11:55 PM Changeset in webkit [133131] by mark.lam@apple.com

3 edits in trunk/Source/JavaScriptCore

Sparse array size threshold should be increased to 100000
https://bugs.webkit.org/show_bug.cgi?id=100827

Patch by Filip Pizlo <fpizlo@apple.com> on 2012-10-31
Reviewed by Oliver Hunt.

This enables the use of contiguous arrays in programs that previously
couldn't use them. And I so far can't see any examples of this being
a downside. To the extent that there is a downside, it ought to be
addressed by GC: https://bugs.webkit.org/show_bug.cgi?id=100828

runtime/ArrayConventions.h:

(JSC):

11:20 PM Changeset in webkit [133130] by Nate Chapin

6 edits in trunk/Source/WebCore

Remove some CachedResource::Status's in favor of looking at CachedResource::m_error
https://bugs.webkit.org/show_bug.cgi?id=100901

Reviewed by Adam Barth.

No new tests, refactor only.

inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::buildObjectForFrameTree):

loader/DocumentThreadableLoader.cpp:

(WebCore::DocumentThreadableLoader::notifyFinished):

loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadInSameDocument):

loader/SubresourceLoader.cpp:

(WebCore::SubresourceLoader::didFail):
(WebCore::SubresourceLoader::willCancel):

loader/cache/CachedResource.cpp:

(WebCore::CachedResource::stopLoading):

loader/cache/CachedResource.h:

(WebCore::CachedResource::wasCanceled):
(WebCore::CachedResource::errorOccurred):
(WebCore::CachedResource::loadFailedOrCanceled):

11:09 PM Changeset in webkit [133129] by fpizlo@apple.com

2 edits in trunk/Source/JavaScriptCore

Sparse array size threshold should be increased to 100000
https://bugs.webkit.org/show_bug.cgi?id=100827

Reviewed by Oliver Hunt.

runtime/ArrayConventions.h:

(JSC):

10:26 PM Changeset in webkit [133128] by Lucas Forschler

4 edits in trunk/Source

Versioning.

10:21 PM Changeset in webkit [133127] by Lucas Forschler

1 copy in tags/Safari-537.17

New Tag.

10:19 PM Changeset in webkit [133126] by commit-queue@webkit.org

7 edits
7 adds in trunk/Source/WebKit2

[EFL][WK2] Add ewk_security_origin and ewk_storage_manager APIs
https://bugs.webkit.org/show_bug.cgi?id=92827

Patch by Jihye Kang <jye.kang@samsung.com> on 2012-10-31
Reviewed by Gyuyoung Kim.

Add ewk_security_origin APIs to get security origin.
These APIs will be used to get host, protocol and port of security
origin.
Add ewk_storage_manager to manage web storage and
ewk_storage_manager_origins_get to get origins list of web storage.
Add Unit tests for ewk_storage_manager and ewk_security_origin APIs.

PlatformEfl.cmake:
UIProcess/API/efl/EWebKit2.h:
UIProcess/API/efl/ewk_context.cpp:

(Ewk_Context::storageManager):
(ewk_context_storage_manager_get):

UIProcess/API/efl/ewk_context.h:
UIProcess/API/efl/ewk_context_private.h:

(Ewk_Context):

UIProcess/API/efl/ewk_security_origin.cpp: Added.

(Ewk_Security_Origin::Ewk_Security_Origin):
(Ewk_Security_Origin::host):
(Ewk_Security_Origin::protocol):
(Ewk_Security_Origin::port):
(ewk_security_origin_ref):
(ewk_security_origin_unref):
(ewk_security_origin_host_get):
(ewk_security_origin_protocol_get):
(ewk_security_origin_port_get):

UIProcess/API/efl/ewk_security_origin.h: Added.
UIProcess/API/efl/ewk_security_origin_private.h: Added.

(Ewk_Security_Origin):
(Ewk_Security_Origin::create):

UIProcess/API/efl/ewk_storage_manager.cpp: Added.

(Ewk_Storage_Manager::Ewk_Storage_Manager):
(Ewk_Storage_Manager::getStorageOrigins):
(Ewk_Storage_Manager::createOriginList):
(_Ewk_Storage_Origins_Async_Get_Context):
(_Ewk_Storage_Origins_Async_Get_Context::_Ewk_Storage_Origins_Async_Get_Context):
(getStorageOriginsCallback):
(ewk_storage_manager_origins_get):

UIProcess/API/efl/ewk_storage_manager.h: Added.
UIProcess/API/efl/ewk_storage_manager_private.h: Added.

(Ewk_Storage_Manager):
(Ewk_Storage_Manager::create):

UIProcess/API/efl/tests/test_ewk2_context.cpp:

(TEST_F):

UIProcess/API/efl/tests/test_ewk2_storage_manager.cpp: Added.

(getStorageOriginsCallback):
(TEST_F):

9:44 PM Changeset in webkit [133125] by senorblanco@chromium.org

12 edits in trunk/Source

Unreviewed, rolling out r133122.
http://trac.webkit.org/changeset/133122
https://bugs.webkit.org/show_bug.cgi?id=99083

Broke Chromium Win, Android, ChromeOS builds

Source/Platform:

chromium/public/WebContentLayerClient.h:

(WebContentLayerClient):

Source/WebCore:

platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::setContentsOpaque):
(WebCore::GraphicsLayerChromium::paint):

platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.cpp:

(WebCore::OpaqueRectTrackingContentLayerDelegate::paintContents):

platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.h:

(OpaqueRectTrackingContentLayerDelegate):

Source/WebKit/chromium:

src/LinkHighlight.cpp:

(WebKit::LinkHighlight::paintContents):

src/LinkHighlight.h:

(LinkHighlight):

src/NonCompositedContentHost.cpp:

(WebKit::NonCompositedContentHost::NonCompositedContentHost):
(WebKit::NonCompositedContentHost::setOpaque):
(WebKit::NonCompositedContentHost::paintContents):

src/NonCompositedContentHost.h:

(NonCompositedContentHost):

tests/OpaqueRectTrackingContentLayerDelegateTest.cpp:

(WebCore::TEST_F):

9:34 PM Changeset in webkit [133124] by tasak@google.com

3 edits
2 adds in trunk

[Refactoring] Move initial style setting for ProgressValueElement from attach method to createShadowSubtree method in HTMLProgressElement.
https://bugs.webkit.org/show_bug.cgi?id=83664

Reviewed by Hajime Morita.

Source/WebCore:

The original code updates inline style during attach(). However,
the attach would be invoked from Element::recalcStyle()'s reattach().
If updating inline styles during the reattach(), style related flags,
i.e. childNeedsStyleRecalc, and needsStyleRecalc are cleared after the
reattach(). So the inline styles are not updated in next
setNeedsStyleRecalc, because ProgressValueElement has already had
InlineStyleChange (existingChanegType == InlineStyleChange) and
markAncestorsWithChildNeedsStyleRecalc is not invoked.

Test: fast/dom/HTMLProgressElement/progress-bar-set-value.html

html/HTMLProgressElement.cpp:

(WebCore::HTMLProgressElement::attach):
Copied updateFromElement from didElementStateChange. If removing the
update, indeterminate-progress-001.html and progress-element.html
under fast/dom/HTMLProgressElement will fail. We still need attach()
and updateFromElement. To remove the attach(), need more refactoring,
i.e. investigating where attach() is invoked from and modifying all
related codes.
(WebCore::HTMLProgressElement::createShadowSubtree):
Initialize m_value by indeterminate-position. The value is default
value of progress element.

LayoutTests:

fast/dom/HTMLProgressElement/progress-bar-set-value-expected.html: Added.
fast/dom/HTMLProgressElement/progress-bar-set-value.html: Added.

9:25 PM Changeset in webkit [133123] by hayato@chromium.org

3 edits in trunk/Source/WebCore

Make resolveReprojection() defined in ComposedShadowTreeWalker.cpp callable from outside.
https://bugs.webkit.org/show_bug.cgi?id=100832

Reviewed by Dimitri Glazkov.

InsertionPoint.h now defines resolveReprojection() so that it can be called from outside.

No new tests as no new functionality.

dom/ComposedShadowTreeWalker.cpp:
html/shadow/InsertionPoint.h:

(WebCore::shadowOfParentForDistribution):
(WebCore):
(WebCore::resolveReprojection):

9:16 PM Changeset in webkit [133122] by alokp@chromium.org

12 edits in trunk/Source

[chromium] Pass canPaintLCDText to WebContentLayerClient::paintContents
https://bugs.webkit.org/show_bug.cgi?id=99083

Reviewed by James Robinson.

Use LCD text setting passed to WebContentLayerClient::paintContents instead of turning it off for all composited layers.

Source/Platform:

chromium/public/WebContentLayerClient.h:

(WebContentLayerClient):
(WebKit::WebContentLayerClient::paintContents):

Source/WebCore:

No new tests needed. This patch does not change anything functionally.

platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::setContentsOpaque):
(WebCore::GraphicsLayerChromium::paint):

platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.cpp:

(WebCore::OpaqueRectTrackingContentLayerDelegate::paintContents):

platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.h:

(OpaqueRectTrackingContentLayerDelegate):

Source/WebKit/chromium:

src/LinkHighlight.cpp:

(WebKit::LinkHighlight::paintContents):

src/LinkHighlight.h:

(LinkHighlight):

src/NonCompositedContentHost.cpp:

(WebKit::NonCompositedContentHost::NonCompositedContentHost):
(WebKit::NonCompositedContentHost::setOpaque):
(WebKit::NonCompositedContentHost::paintContents):

src/NonCompositedContentHost.h:

(NonCompositedContentHost):

tests/OpaqueRectTrackingContentLayerDelegateTest.cpp:

(WebCore::TEST_F):

9:14 PM Changeset in webkit [133121] by dpranke@chromium.org

3 edits in trunk/Tools

NRWT: lint warnings don't show up by default
https://bugs.webkit.org/show_bug.cgi?id=100914

Reviewed by Ryosuke Niwa.

NRWT wasn't printing any warnings if you had errors in your
TestExpectations files; this means you would only notice this
with --lint-test-files, which is probably bad. I'm not sure
when this requested, but perhaps when we changed the
warnings to not be fatal if you weren't linting the files.

Now they're still not fatal, but at least we print them.

Also fix one warning message where we were printing the test name twice.

Scripts/webkitpy/layout_tests/models/test_expectations.py:

(TestExpectationsModel._already_seen_better_match):
(TestExpectations._report_warnings):

Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py:

(MiscTests.test_parse_warnings_are_logged_if_not_in_lint_mode(self): Added.

8:33 PM Changeset in webkit [133120] by tdanderson@chromium.org

2 edits in trunk/Tools

Missing stderr output when using run-webkit-tests
https://bugs.webkit.org/show_bug.cgi?id=100709

Reviewed by Dirk Pranke.

Appends #EOF to the end of the stderr output only at the end of a test,
not possibly part-way through. This will allow all of the stderr output
to appear on the results page after running run-webkit-tests.

DumpRenderTree/chromium/TestEventPrinter.cpp:

(TestEventPrinter::handleTextFooter):
(TestEventPrinter::handleTestFooter):

8:21 PM Changeset in webkit [133119] by cevans@google.com

3 edits in trunk/Source/WebCore

RenderArena has a memory leak and poor efficiency
https://bugs.webkit.org/show_bug.cgi?id=100893

Reviewed by Eric Seidel.

1) Avoid memory leak that persists for the Document lifetime by
increasing recycled size buckets up to 1024. It was previously 400,
and sizeof(RenderNamedFlowThread) / sizeof(RenderSVGText) both blew this
quota. An assert was added to prevent this happening again.

2) Fix the size of the recyled size bucket array on 64-bit. We only
need 8 byte granularity on 64-bit, but we had 4.

3) Try and pass power-of-two sizes to the underlying malloc() call, so
that we're space efficient. We now take Arena metadata into account.

4) Double the default RenderArena size allocation to 8192 bytes. Even
for a render of a trivial text file, 4096 bytes is not enough to prevent
extra calls into the underlying malloc() for more arena pool.

rendering/RenderArena.cpp:

(WebCore::RenderArena::RenderArena): Adjust arena size so that we pass on the page-sized multiple to the underlying malloc() implementation.
(WebCore::RenderArena::allocate):
(WebCore::RenderArena::free): Assert that the allocation size is handled by our recycling buckets.

rendering/RenderArena.h:

(WebCore): Maintain free buckets up to 1024 bytes to avoid memory leak.
(RenderArena): Double the default allocation size and handle 64-bit systems more efficiently.

8:16 PM Changeset in webkit [133118] 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 2012-10-31

DEPS:

8:11 PM Changeset in webkit [133117] by dpranke@chromium.org

2 edits in trunk/LayoutTests

Unreviewed, remove entries for tests no longer failing.

platform/chromium/TestExpectations:

8:05 PM Changeset in webkit [133116] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Mark a crashy test.

platform/chromium/TestExpectations:

7:57 PM Changeset in webkit [133115] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.

platform/chromium/TestExpectations:

7:49 PM Changeset in webkit [133114] by dpranke@chromium.org

2 edits in trunk/LayoutTests

Unreviewed chromium mac ml expectation updating.

platform/chromium-mac-mountainlion/TestExpectations:

7:37 PM Changeset in webkit [133113] by abarth@webkit.org

9 edits in trunk

[V8] Garbage collection should use opaque roots rather than implicit references
https://bugs.webkit.org/show_bug.cgi?id=100707

Reviewed by Kentaro Hara.

Source/WebCore:

This patch replaces visitDOMWrapper with opaqueRootForGC. The
former used to inform V8 of implicit relationships between wrapper
objects on a per-wrapper basis. That meant that we needed to know which
DOMDataStore a given wrapper was in during garbage collection.

After this patch, we now use object groups rather than implicit
references to inform V8 of these relationships. That has two benefits:

1) We no longer need to know which DOMDataStore a wrapper belongs

because we don't need to find the exact source wrapper for the
implicit connection.

2) We can now handle more complicated implicit relationships, for

example when some of the intervening objects haven't had their
JavaScript wrappers created yet.

This patch also unlocks to paths of future development:
A) Fixing the remaining failures in fast/dom/gc-9.html
B) Enumerating DOM wrappers entirely from V8 rather than from the

DOMWrapperMaps (so that we can move more object towards using
IntrusiveDOMWrapperMaps, which aren't enumerable from WebCore).

bindings/scripts/CodeGeneratorV8.pm:

(NeedsCustomOpaqueRootForGC):
(GenerateOpaqueRootForGC):
(GenerateHeader):
(GenerateImplementation):

bindings/v8/V8GCController.cpp:

(ImplicitConnection):
(WebCore::ImplicitConnection::ImplicitConnection):
(WebCore::ImplicitConnection::root):
(WebCore::ImplicitConnection::wrapper):
(WebCore):
(WebCore::operator<):
(WrapperGrouper):
(WebCore::WrapperGrouper::WrapperGrouper):
(WebCore::WrapperGrouper::addToGroup):
(WebCore::WrapperGrouper::keepAlive):
(WebCore::WrapperGrouper::apply):
(WebCore::ObjectVisitor::ObjectVisitor):
(WebCore::ObjectVisitor::visitDOMWrapper):
(ObjectVisitor):
(WebCore::V8GCController::opaqueRootForGC):
(WebCore::NodeVisitor::NodeVisitor):
(WebCore::NodeVisitor::visitNodeWrapper):
(NodeVisitor):
(WebCore::V8GCController::majorGCPrologue):

bindings/v8/V8GCController.h:

(WebCore):
(V8GCController):

bindings/v8/WrapperTypeInfo.h:

(WebCore):
(WebCore::WrapperTypeInfo::opaqueRootForGC):
(WrapperTypeInfo):

bindings/v8/custom/V8NodeListCustom.cpp:

(WebCore::V8NodeList::opaqueRootForGC):

bindings/v8/custom/V8SpeechRecognitionResultCustom.cpp:

(WebCore::V8SpeechRecognitionResult::opaqueRootForGC):

LayoutTests:

Test progression.

platform/chromium/fast/dom/gc-9-expected.txt:

7:08 PM Changeset in webkit [133112] by senorblanco@chromium.org

43 edits in trunk/Source

Unreviewed, rolling out r133107.
http://trac.webkit.org/changeset/133107
https://bugs.webkit.org/show_bug.cgi?id=100425

Source/WebCore:

Broke compile on Chromium Win.

Modules/indexeddb/IDBCallbacks.h:

(IDBCallbacks):

Modules/indexeddb/IDBDatabase.cpp:

(WebCore::IDBDatabase::objectStoreNames):
(WebCore::IDBDatabase::createObjectStore):
(WebCore::IDBDatabase::deleteObjectStore):
(WebCore::IDBDatabase::transaction):

Modules/indexeddb/IDBDatabaseBackendImpl.cpp:

(WebCore::IDBDatabaseBackendImpl::metadata):
(WebCore::IDBDatabaseBackendImpl::createObjectStore):
(WebCore::IDBDatabaseBackendImpl::objectStore):
(WebCore::IDBDatabaseBackendImpl::deleteObjectStore):
(WebCore::IDBDatabaseBackendImpl::transaction):
(WebCore::IDBDatabaseBackendImpl::loadObjectStores):
(WebCore::IDBDatabaseBackendImpl::removeObjectStoreFromMap):
(WebCore::IDBDatabaseBackendImpl::addObjectStoreToMap):

Modules/indexeddb/IDBDatabaseBackendImpl.h:

(IDBDatabaseBackendImpl):

Modules/indexeddb/IDBDatabaseBackendInterface.h:

(IDBDatabaseBackendInterface):

Modules/indexeddb/IDBFactory.cpp:
Modules/indexeddb/IDBFactory.h:
Modules/indexeddb/IDBIndex.h:

(WebCore::IDBIndex::openKeyCursor):

Modules/indexeddb/IDBIndexBackendInterface.h:
Modules/indexeddb/IDBKeyPath.cpp:
Modules/indexeddb/IDBMetadata.h:

(WebCore):
(IDBDatabaseMetadata):
(WebCore::IDBDatabaseMetadata::IDBDatabaseMetadata):
(IDBObjectStoreMetadata):
(WebCore::IDBIndexMetadata::IDBIndexMetadata):
(IDBIndexMetadata):

Modules/indexeddb/IDBObjectStore.cpp:

(WebCore::IDBObjectStore::indexNames):
(WebCore::IDBObjectStore::put):
(WebCore::IDBObjectStore::createIndex):
(WebCore::IDBObjectStore::index):
(WebCore::IDBObjectStore::deleteIndex):

Modules/indexeddb/IDBObjectStoreBackendImpl.cpp:

(WebCore::IDBObjectStoreBackendImpl::putWithIndexKeys):
(WebCore):
(WebCore::makeIndexWriters):
(WebCore::IDBObjectStoreBackendImpl::setIndexKeys):
(WebCore::IDBObjectStoreBackendImpl::setIndexesReady):
(WebCore::IDBObjectStoreBackendImpl::setIndexesReadyInternal):
(WebCore::IDBObjectStoreBackendImpl::putInternal):
(WebCore::IDBObjectStoreBackendImpl::deleteInternal):
(WebCore::IDBObjectStoreBackendImpl::createIndex):
(WebCore::IDBObjectStoreBackendImpl::index):
(WebCore::IDBObjectStoreBackendImpl::deleteIndex):
(WebCore::IDBObjectStoreBackendImpl::loadIndexes):
(WebCore::IDBObjectStoreBackendImpl::removeIndexFromMap):
(WebCore::IDBObjectStoreBackendImpl::addIndexToMap):

Modules/indexeddb/IDBObjectStoreBackendImpl.h:

(IDBObjectStoreBackendImpl):

Modules/indexeddb/IDBObjectStoreBackendInterface.h:
Modules/indexeddb/IDBOpenDBRequest.cpp:

(WebCore::IDBOpenDBRequest::onUpgradeNeeded):

Modules/indexeddb/IDBRequest.cpp:

(WebCore::IDBRequest::onSuccess):
(WebCore::IDBRequest::uncaughtExceptionInEventHandler):

Modules/indexeddb/IDBTransaction.cpp:

(WebCore::IDBTransaction::create):
(WebCore::IDBTransaction::IDBTransaction):
(WebCore::IDBTransaction::objectStore):

Modules/indexeddb/IDBTransaction.h:

(IDBTransaction):

Modules/indexeddb/IDBTransactionBackendImpl.cpp:

Modules/indexeddb/IDBTransactionBackendImpl.h:

(IDBTransactionBackendImpl):

Modules/indexeddb/IDBTransactionBackendInterface.h:
Modules/indexeddb/IDBTransactionCoordinator.h:

Source/WebKit/chromium:

Broke compile on Chromium Win

public/WebIDBDatabase.h:

(WebKit::WebIDBDatabase::transaction):

public/WebIDBObjectStore.h:

(WebKit::WebIDBObjectStore::index):

public/WebIDBTransaction.h:

(WebKit::WebIDBTransaction::objectStore):

src/IDBDatabaseBackendProxy.cpp:
src/IDBDatabaseBackendProxy.h:

(IDBDatabaseBackendProxy):

src/IDBObjectStoreBackendProxy.cpp:

(WebKit::IDBObjectStoreBackendProxy::setIndexesReady):

src/IDBObjectStoreBackendProxy.h:

(IDBObjectStoreBackendProxy):

src/IDBTransactionBackendProxy.cpp:
src/IDBTransactionBackendProxy.h:

(IDBTransactionBackendProxy):

src/WebIDBCallbacksImpl.cpp:
src/WebIDBDatabaseImpl.cpp:
src/WebIDBDatabaseImpl.h:

(WebIDBDatabaseImpl):

src/WebIDBMetadata.cpp:

(WebKit::WebIDBMetadata::WebIDBMetadata):
(WebKit::WebIDBMetadata::operator IDBDatabaseMetadata):

src/WebIDBObjectStoreImpl.cpp:
src/WebIDBObjectStoreImpl.h:

(WebIDBObjectStoreImpl):

src/WebIDBTransactionImpl.cpp:
src/WebIDBTransactionImpl.h:
tests/IDBDatabaseBackendTest.cpp:

7:00 PM Changeset in webkit [133111] by ryuan.choi@samsung.com

2 edits in trunk/LayoutTests

[EFL] Move jquery/traversing.html to flaky list
https://bugs.webkit.org/show_bug.cgi?id=100908

Unreviewed gardening.

jquery/traversing.html sometimes crashes and sometimes passes, so needs to be moved to the flaky list.

Patch by KyungTae Kim <ktf.kim@samsung.com> on 2012-10-31

platform/efl/TestExpectations:

6:59 PM Changeset in webkit [133110] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.

platform/chromium/TestExpectations:

6:59 PM Changeset in webkit [133109] by andersca@apple.com

4 edits in trunk/Source

Fix build.
<rdar://problem/12612207>.

Reviewed by Sam Weinig.

Source/WebKit/mac:

Panels/WebAuthenticationPanel.m:

(-[WebAuthenticationPanel loadNib]):

Source/WTF:

wtf/CheckedArithmetic.h:

6:55 PM Changeset in webkit [133108] by roger_fong@apple.com

2 edits in trunk/LayoutTests

Unreviewed. fast\text\international\text-combine-image-test-expected.txt needs some 1 pixel off rebaselines on Windows.

platform/win/fast/text/international/text-combine-image-test-expected.txt:

6:21 PM Changeset in webkit [133107] by alecflett@chromium.org

43 edits in trunk/Source

IndexedDB: add methods to support id-based backend APIs
https://bugs.webkit.org/show_bug.cgi?id=100425

Reviewed by Tony Chang.

Source/WebCore:

In addition, there were a number of cleanups that came out of the
refactor:

The list of object stores active in a transaction is now maintained by the frontend IDBTransaction rather than the backend IDBTransactionBackendImpl. This also had a simplifying rippling effect through other call signatures.

I was able to remove an apparently old FIXME from IDBTransactionBackendImpl::objectStore and replace it with an ASSERT.

IDBObjectStoreBackendImpl's IndexWriter class got a little simpler since the id is now easily available in the IDBIndexMetadata.

A number of methods got simpler in their int64_t versions, specifically dropping a number of ExceptionCodes.

There is also some glue code
(getIndexId/getIndexIds/getObjectStoreId) that will go away with
the 2nd half of this: https://bugs.webkit.org/show_bug.cgi?id=100425

No new tests, no new functionality as this is just a refactor.

Modules/indexeddb/IDBDatabase.cpp:

(WebCore::IDBDatabase::objectStoreNames):
(WebCore::IDBDatabase::createObjectStore):
(WebCore::IDBDatabase::deleteObjectStore):
(WebCore::IDBDatabase::transaction):

Modules/indexeddb/IDBDatabaseBackendImpl.cpp:

Modules/indexeddb/IDBDatabaseBackendImpl.h:

(WebCore::IDBDatabaseBackendImpl::deleteObjectStore):
(IDBDatabaseBackendImpl):

Modules/indexeddb/IDBDatabaseBackendInterface.h:

(IDBDatabaseBackendInterface):

Modules/indexeddb/IDBIndex.h:

(WebCore::IDBIndex::id):
(WebCore::IDBIndex::openKeyCursor):

Modules/indexeddb/IDBIndexBackendInterface.h:
Modules/indexeddb/IDBMetadata.h:

Modules/indexeddb/IDBObjectStore.cpp:

(WebCore::IDBObjectStore::indexNames):
(WebCore::IDBObjectStore::put):
(WebCore::IDBObjectStore::createIndex):
(WebCore::IDBObjectStore::index):
(WebCore::IDBObjectStore::deleteIndex):

Modules/indexeddb/IDBObjectStoreBackendImpl.cpp:

Modules/indexeddb/IDBObjectStoreBackendImpl.h:

(IDBObjectStoreBackendImpl):

Modules/indexeddb/IDBObjectStoreBackendInterface.h:
Modules/indexeddb/IDBOpenDBRequest.cpp:

(WebCore::IDBOpenDBRequest::onUpgradeNeeded):

Modules/indexeddb/IDBRequest.cpp:

(WebCore::IDBRequest::onSuccess):
(WebCore::IDBRequest::uncaughtExceptionInEventHandler):

Modules/indexeddb/IDBTransaction.cpp:

(WebCore::IDBTransaction::create):
(WebCore::IDBTransaction::IDBTransaction):
(WebCore::IDBTransaction::objectStore):

Modules/indexeddb/IDBTransaction.h:

(IDBTransaction):

Modules/indexeddb/IDBTransactionBackendImpl.cpp:

Modules/indexeddb/IDBTransactionBackendImpl.h:

(IDBTransactionBackendImpl):

Modules/indexeddb/IDBTransactionBackendInterface.h:

Source/WebKit/chromium:

Chromium Proxy and WebIDB* support for new int64_t-based methods.

public/WebIDBDatabase.h:

(WebKit::WebIDBDatabase::deleteObjectStore):
(WebKit::WebIDBDatabase::transaction):

public/WebIDBObjectStore.h:

public/WebIDBTransaction.h:

(WebKit::WebIDBTransaction::objectStore):

src/IDBDatabaseBackendProxy.cpp:

(WebKit::IDBDatabaseBackendProxy::deleteObjectStore):
(WebKit):
(WebKit::IDBDatabaseBackendProxy::transaction):

src/IDBDatabaseBackendProxy.h:

(IDBDatabaseBackendProxy):

src/IDBObjectStoreBackendProxy.cpp:

src/IDBObjectStoreBackendProxy.h:

(IDBObjectStoreBackendProxy):

src/IDBTransactionBackendProxy.cpp:

(WebKit::IDBTransactionBackendProxy::objectStore):

src/IDBTransactionBackendProxy.h:

(IDBTransactionBackendProxy):

src/WebIDBMetadata.cpp:

(WebKit::WebIDBMetadata::WebIDBMetadata):
(WebKit::WebIDBMetadata::operator IDBDatabaseMetadata):

src/WebIDBTransactionImpl.cpp:

(WebKit::WebIDBTransactionImpl::objectStore):
(WebKit):

src/WebIDBTransactionImpl.h:
tests/IDBDatabaseBackendTest.cpp:

6:21 PM Changeset in webkit [133106] by commit-queue@webkit.org

2 edits
2 adds in trunk/LayoutTests

[CSS Exclusions] Test that dynamically setting shape-inside causes relayout in child elements
https://bugs.webkit.org/show_bug.cgi?id=97721

Patch by Bear Travis <betravis@adobe.com> on 2012-10-31
Reviewed by Dirk Schulze.

Adding a test that dynamically sets shape-inside on a block containing multiple block children.

fast/exclusions/resources/simple-rectangle.js:

(createRectangleTest): Modifying test to work with elements with existing content.
(createRectangleTestResult): Modifying test expectation to work with elements with existing content.

fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic-expected.html: Added.
fast/exclusions/shape-inside/shape-inside-multiple-blocks-dynamic.html: Added.

6:12 PM Changeset in webkit [133105] by roger_fong@apple.com

2 edits in trunk/LayoutTests

Unreviewed. Skip another failing drag and drop test.
Drag & Drop doesn't work correctly in DRT <rdar://5621244>

platform/win/TestExpectations:

6:02 PM Changeset in webkit [133104] by roger_fong@apple.com

2 edits in trunk/LayoutTests

Unreviewed. Skipping http/tests/xmlhttprequest/timeout/xmlhttprequest-timeout-synconmain.html.
Not supported in CFNetwork yet. https://bugs.webkit.org/show_bug.cgi?id=100349.

platform/win/TestExpectations:

5:59 PM Changeset in webkit [133103] by mark.lam@apple.com

7 edits
3 adds in trunk

A JSC printf (support for %J+s and %b).
https://bugs.webkit.org/show_bug.cgi?id=100566.

Reviewed by Michael Saboff.

Source/JavaScriptCore:

Added VMInspector::printf(), fprintf(), sprintf(), and snprintf().

%b prints ints as boolean TRUE (non-zero) or FALSE (zero).
%Js prints a WTF::String* like a %s prints a char*. Also works for 16bit WTF::Strings (prints wchar_t* using %S).
'+' is a modifier meaning 'use verbose mode', and %J+s is an example of its use.

JavaScriptCore.xcodeproj/project.pbxproj:
interpreter/VMInspector.cpp:

(FormatPrinter):
(JSC::FormatPrinter::~FormatPrinter):
(JSC::FormatPrinter::print):
(JSC::FormatPrinter::printArg):
(JSC::FormatPrinter::printWTFString):
(JSC::FileFormatPrinter::FileFormatPrinter):
(JSC::FileFormatPrinter::printArg):
(JSC::StringFormatPrinter::StringFormatPrinter):
(JSC::StringFormatPrinter::printArg):
(JSC::StringNFormatPrinter::StringNFormatPrinter):
(JSC::StringNFormatPrinter::printArg):
(JSC::VMInspector::fprintf):
(JSC::VMInspector::printf):
(JSC::VMInspector::sprintf):
(JSC::VMInspector::snprintf):

interpreter/VMInspector.h:

(VMInspector):

Source/WebCore:

Added forwarding header for VMInspector.h.

No new tests needed for this.

ForwardingHeaders/interpreter/VMInspector.h: Added.

Tools:

Added regression tests for VMInspector::fprintf() sprintf() and snprintf().

TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
TestWebKitAPI/Tests/JavaScriptCore: Added.
TestWebKitAPI/Tests/JavaScriptCore/VMInspector.cpp: Added.

5:59 PM Changeset in webkit [133102] by staikos@webkit.org

4 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Crash on html popup form control when element is deleted
https://bugs.webkit.org/show_bug.cgi?id=100869

Reviewed by Yong Li.

Make the element pointers RefPtr so they don't go away before the popup
is closed.

WebCoreSupport/ColorPickerClient.h:

(ColorPickerClient):

WebCoreSupport/DatePickerClient.h:

(DatePickerClient):

WebCoreSupport/SelectPopupClient.h:

(SelectPopupClient):

5:39 PM Changeset in webkit [133101] by crogers@google.com

6 edits
3 adds in trunk

Implement optional arguments in AudioBufferSourceNode start() method
https://bugs.webkit.org/show_bug.cgi?id=100894

Reviewed by Kenneth Russell.

Source/WebCore:

The start() method should be able to take 1, 2, or 3 arguments, optionally supporting
offset and duration. Currently, only 1 and 3 arguments are supported.

Test: webaudio/audiobuffersource-start.html

Modules/webaudio/AudioBufferSourceNode.cpp:

(WebCore::AudioBufferSourceNode::renderFromBuffer):
(WebCore::AudioBufferSourceNode::startGrain):
(WebCore):

Modules/webaudio/AudioBufferSourceNode.h:

(AudioBufferSourceNode):

Modules/webaudio/AudioBufferSourceNode.idl:

LayoutTests:

webaudio/audiobuffersource-loop-comprehensive.html:
webaudio/audiobuffersource-start-expected.txt: Added.
webaudio/audiobuffersource-start.html: Added.
webaudio/resources/audiobuffersource-testing.js: Added.

(createTestBuffer):
(checkSingleTest):
(checkAllTests):

5:31 PM Changeset in webkit [133100] by benjamin@webkit.org

2 edits in trunk/Source/WTF

Add an optimized version of copyLCharsFromUCharSource for ARM
https://bugs.webkit.org/show_bug.cgi?id=94886

Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-10-31
Reviewed by Gavin Barraclough.

Michael Saboff added a SIMD version of copyLCharsFromUCharSource() in r125846.

This patch a similar optimization for ARMv7 by using the interleaved load/store available
in the NEON extension.

The performance gains:
-10000 characters: ~3.5 times faster.
-20 characters (2 vectors): ~55% faster.
-15 characters (1 vector): ~21% faster.
-3 characters (no vector, pure overhead): ~10% slower.

wtf/text/ASCIIFastPath.h:

(WTF::copyLCharsFromUCharSource):

5:09 PM Changeset in webkit [133099] by Alexandru Chiculita

2 edits in trunk/Source/WebKit2

CustomFilterOperation parameters list is not decoded correctly in CoordinatedGraphicsArgumentCoders
https://bugs.webkit.org/show_bug.cgi?id=100889

Reviewed by Noam Rosenthal.

The custom filters parameters list was initialized with a specific size and then
the decoder was expected to fill it with all the decoded parameters. The decoder was
using the append command which added the paramters at the end, so it doubled the size
of the vector leaving empty spaces at the beginning of the list. Fixed it by writting
directly into the allocated space.

No new tests, the code path is not activated yet.

Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:

(CoreIPC::::decode):

5:06 PM Changeset in webkit [133098] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.
fast/js/post-message-numeric-property.html is flaky in Release as well.

platform/chromium/TestExpectations:

5:03 PM Changeset in webkit [133097] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit/blackberry

[BLACKBERRY] Custom WebWorks context menus don't always work.
https://bugs.webkit.org/show_bug.cgi?id=100891

Patch by Genevieve Mak <gmak@rim.com> on 2012-10-31
Reviewed by Rob Buis.

Walk up the dom tree until we find the webworks attribute or run out
of parent nodes.
PR# 227732

Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::webContext):

5:03 PM Changeset in webkit [133096] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. svg/custom/font-face-simple.svg
seems to be failing on Windows (wrong font).

platform/chromium/TestExpectations:

4:41 PM Changeset in webkit [133095] by mkwst@chromium.org

105 edits in trunk

Implement the canonical "Content-Security-Policy" header.
https://bugs.webkit.org/show_bug.cgi?id=96765

Reviewed by Adam Barth.

Source/WebCore:

The CSP 1.0 specification defines the "Content-Security-Policy" header
as the canonical mechanism of defining a resource's security policy. Up
through this patch, we've implemented the functionality behind a prefix
in order to ensure compatibility with the standard once it's released as
a recommendation. Both the specification and WebKit's implementation are
far enough along in that process that it makes sense to support the
unprefixed header for sites that wish to opt-in to CSP 1.0.

As discussed on public-webappsec[1], we'll keep the experimental 1.1
features behind the prefixed header ('X-WebKit-CSP') until that standard
is far enough along to justify moving them out to the canonical header.

This patch defines the 'Content-Security-Policy' header for all ports,
just as the 'X-WebKit-CSP' header is currently supported on all ports.
Ports that have not opted-in to the CSP_NEXT flag will see exactly the
same behavior with both headers. Ports that have opted-in will see much
of CSP 1.1's current definition on the prefixed header, and CSP 1.0 on
the canonical header.

The functionality in this change is covered by the changes made to
existing tests. No expectations changed, only the headers that are sent.

dom/Document.cpp:

(WebCore::Document::processHttpEquiv):

Add canonical header support to 'meta' element definitions.

loader/FrameLoader.cpp:

(WebCore::FrameLoader::didBeginDocument):

Add canonical header support to FrameLoader.

page/ContentSecurityPolicy.cpp:

(WebCore::CSPDirectiveList::headerType):

The ContentSecurityPolicy::HeaderType enum now has four values:
prefixed/report-only, unprefixed/report-only, prefixed/enforce, and
unprefixed/enforce. Instead of creating logic to output the proper
type based on internal flags, CSPDirectiveList now saves the value
provided at creation time, and returns it via this method.

(CSPDirectiveList):
(WebCore::CSPDirectiveList::CSPDirectiveList):

The constructor now accepts a type, which is stored on the object.
It also stores a new internal variable, 'm_experimental', which
defines whether or not experimental features ought to be available.
These features are still locked behind the CSP_NEXT flag, but that
might not be the case forever.

(WebCore::CSPDirectiveList::create):

The static constructor wrapper now passes the type into the real
constructor, which also now handles setting its internal variables.

(WebCore::CSPDirectiveList::parse):

'parse()' is given the header, so it makes sense to store it here as
well, rather than in the create wrapper.

(WebCore::CSPDirectiveList::addDirective):

1.1 directives remain locked behind CSP_NEXT, but now also require
that 'm_experimental' is set, signaling usage of the prefixed header
and an implicit opt-in to 1.1.

page/ContentSecurityPolicy.h:

Added two new types to the HeaderTypes enum: PrefixedReportOnly, and
PrefixedEnforcePolicy. These map to 'X-WebKitCSP-Report-Only' and
'X-WebKit-CSP', respectively.

LayoutTests:

http/tests/security/contentSecurityPolicy/1.1/plugintypes-invalid.html:
http/tests/security/contentSecurityPolicy/1.1/plugintypes-url-01.html:
http/tests/security/contentSecurityPolicy/1.1/plugintypes-url-02.html:
http/tests/security/contentSecurityPolicy/1.1/scriptnonce-invalidnonce.html:
http/tests/security/contentSecurityPolicy/1.1/scriptnonce-scriptsrc-blocked.html:
http/tests/security/contentSecurityPolicy/1.1/scriptnonce-separators-allowed.html:

Updating these 1.1 tests along with the multiple-iframe-*.js test
"framework" to ensure that the experimental prefixed header is sent.

http/tests/security/contentSecurityPolicy/blob-urls-match-self.html:
http/tests/security/contentSecurityPolicy/combine-multiple-policies.html:
http/tests/security/contentSecurityPolicy/connect-src-eventsource-allowed.html:
http/tests/security/contentSecurityPolicy/connect-src-eventsource-blocked.html:
http/tests/security/contentSecurityPolicy/connect-src-websocket-allowed.html:
http/tests/security/contentSecurityPolicy/connect-src-websocket-blocked.html:
http/tests/security/contentSecurityPolicy/connect-src-xmlhttprequest-allowed.html:
http/tests/security/contentSecurityPolicy/connect-src-xmlhttprequest-blocked.html:
http/tests/security/contentSecurityPolicy/default-src-inline-allowed.html:
http/tests/security/contentSecurityPolicy/default-src-inline-blocked.html:
http/tests/security/contentSecurityPolicy/duplicate-directive.html:
http/tests/security/contentSecurityPolicy/eval-allowed.html:
http/tests/security/contentSecurityPolicy/eval-blocked-in-about-blank-iframe.html:
http/tests/security/contentSecurityPolicy/eval-blocked.html:
http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-allowed.html:
http/tests/security/contentSecurityPolicy/eval-scripts-setInterval-blocked.html:
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-allowed.html:
http/tests/security/contentSecurityPolicy/eval-scripts-setTimeout-blocked.html:
http/tests/security/contentSecurityPolicy/filesystem-urls-match-self.html:
http/tests/security/contentSecurityPolicy/frame-src-about-blank-allowed-by-default.html:
http/tests/security/contentSecurityPolicy/frame-src-about-blank-allowed-by-scheme.html:
http/tests/security/contentSecurityPolicy/frame-src-allowed.html:
http/tests/security/contentSecurityPolicy/frame-src-blocked.html:
http/tests/security/contentSecurityPolicy/function-constructor-allowed.html:
http/tests/security/contentSecurityPolicy/function-constructor-blocked.html:
http/tests/security/contentSecurityPolicy/iframe-inside-csp.html:
http/tests/security/contentSecurityPolicy/image-allowed.html:
http/tests/security/contentSecurityPolicy/image-blocked.html:
http/tests/security/contentSecurityPolicy/image-full-host-wildcard-allowed.html:
http/tests/security/contentSecurityPolicy/image-host-wildcard-allowed.html:
http/tests/security/contentSecurityPolicy/injected-inline-script-allowed.html:
http/tests/security/contentSecurityPolicy/injected-inline-script-blocked.html:
http/tests/security/contentSecurityPolicy/injected-inline-style-allowed.html:
http/tests/security/contentSecurityPolicy/injected-inline-style-blocked.html:
http/tests/security/contentSecurityPolicy/inline-script-allowed.html:
http/tests/security/contentSecurityPolicy/inline-script-blocked-goofy.html:
http/tests/security/contentSecurityPolicy/inline-script-blocked-javascript-url.html:
http/tests/security/contentSecurityPolicy/inline-script-blocked.html:
http/tests/security/contentSecurityPolicy/inline-style-allowed.html:
http/tests/security/contentSecurityPolicy/inline-style-attribute-allowed.html:
http/tests/security/contentSecurityPolicy/inline-style-attribute-blocked.html:
http/tests/security/contentSecurityPolicy/inline-style-attribute-on-html.html:
http/tests/security/contentSecurityPolicy/inline-style-blocked.html:
http/tests/security/contentSecurityPolicy/media-src-allowed.html:
http/tests/security/contentSecurityPolicy/media-src-blocked.html:
http/tests/security/contentSecurityPolicy/object-src-no-url-allowed.html:
http/tests/security/contentSecurityPolicy/object-src-no-url-blocked.html:
http/tests/security/contentSecurityPolicy/object-src-url-allowed.html:
http/tests/security/contentSecurityPolicy/object-src-url-blocked.html:
http/tests/security/contentSecurityPolicy/policy-does-not-affect-child.html:
http/tests/security/contentSecurityPolicy/register-bypassing-scheme.html:
http/tests/security/contentSecurityPolicy/report-and-enforce.html:
http/tests/security/contentSecurityPolicy/report-blocked-uri-cross-origin.html:
http/tests/security/contentSecurityPolicy/report-blocked-uri.html:
http/tests/security/contentSecurityPolicy/report-only-from-header.php:
http/tests/security/contentSecurityPolicy/report-only.html:
http/tests/security/contentSecurityPolicy/report-uri.html:
http/tests/security/contentSecurityPolicy/resources/echo-iframe.pl:
http/tests/security/contentSecurityPolicy/resources/echo-multiple-headers.pl:

s/X-WebKit-CSP/Content-Security-Policy/g

http/tests/security/contentSecurityPolicy/resources/echo-object-data.pl:
http/tests/security/contentSecurityPolicy/resources/echo-script-src.pl:

Reworking these two scripts in order to support sending both
the experimental header and the canonical header, as required.

http/tests/security/contentSecurityPolicy/resources/event-handler.pl:
http/tests/security/contentSecurityPolicy/resources/generate-csp-report.html:
http/tests/security/contentSecurityPolicy/resources/javascript-url.pl:
http/tests/security/contentSecurityPolicy/resources/mixed-content-with-csp.html:

s/X-WebKit-CSP/Content-Security-Policy/g

http/tests/security/contentSecurityPolicy/resources/multiple-iframe-plugin-test.js:

(testExperimentalPolicy):
(test):
(testImpl.iframe.onload):
(testImpl):

http/tests/security/contentSecurityPolicy/resources/multiple-iframe-test.js:

(testPreescapedPolicy):
(testExperimentalPolicy):
(test):
(testImpl.iframe.onload):
(testImpl):

Reworking these two "frameworks" in order to support sending both
the experimental header and the canonical header, as required.

http/tests/security/contentSecurityPolicy/resources/sandbox.php:
http/tests/security/contentSecurityPolicy/resources/sandboxed-eval.php:
http/tests/security/contentSecurityPolicy/sandbox-allow-scripts-subframe.html:
http/tests/security/contentSecurityPolicy/sandbox-allow-scripts.html:
http/tests/security/contentSecurityPolicy/sandbox-empty-subframe.html:
http/tests/security/contentSecurityPolicy/sandbox-empty.html:
http/tests/security/contentSecurityPolicy/script-src-overrides-default-src.html:
http/tests/security/contentSecurityPolicy/shared-worker-connect-src-allowed.html:
http/tests/security/contentSecurityPolicy/shared-worker-connect-src-blocked.html:
http/tests/security/contentSecurityPolicy/source-list-parsing-malformed-meta.html:
http/tests/security/contentSecurityPolicy/srcdoc-doesnt-bypass-script-src.html:
http/tests/security/contentSecurityPolicy/style-allowed.html:
http/tests/security/contentSecurityPolicy/style-blocked.html:
http/tests/security/contentSecurityPolicy/worker-connect-src-allowed.html:
http/tests/security/contentSecurityPolicy/worker-connect-src-blocked.html:
http/tests/security/contentSecurityPolicy/worker-eval-blocked.html:
http/tests/security/contentSecurityPolicy/worker-function-function-blocked.html:
http/tests/security/contentSecurityPolicy/worker-script-src.html:
http/tests/security/contentSecurityPolicy/worker-set-timeout-blocked.html:
http/tests/security/contentSecurityPolicy/xsl-allowed.php:
http/tests/security/contentSecurityPolicy/xsl-blocked.php:
http/tests/security/contentSecurityPolicy/xsl-img-blocked.php:
http/tests/security/contentSecurityPolicy/xsl-unaffected-by-style-src-1.php:
http/tests/security/contentSecurityPolicy/xsl-unaffected-by-style-src-2.php:

s/X-WebKit-CSP/Content-Security-Policy/g

4:27 PM Changeset in webkit [133094] by commit-queue@webkit.org

7 edits
6 adds in trunk

[BlackBerry] Adding window.external to our port
https://bugs.webkit.org/show_bug.cgi?id=100868

Patch by Otto Derek Cheung <otcheung@rim.com> on 2012-10-31
Reviewed by Rob Buis.

Source/WebKit:

Adding ExternalExtension classes to the BB cmake file.

PlatformBlackBerry.cmake:

Source/WebKit/blackberry:

Attaching the window.external interface to the frame when
it loads up.

When called, the external interface methods call stubs declared
in ChromeClientBlackBerry. The method passes in the url of the search
provider and the url of the frame. Cross-site check has to be done in the
implementation PR.

WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::addSearchProvider):
(WebCore):
(WebCore::ChromeClientBlackBerry::isSearchProviderInstalled):

WebCoreSupport/ChromeClientBlackBerry.h:

(ChromeClientBlackBerry):

WebCoreSupport/ExternalExtension.cpp: Added.

(addSearchProviderMethod):
(IsSearchProviderInstalledMethod):
(externalExtensionInitialize):
(externalExtensionFinalize):

WebCoreSupport/ExternalExtension.h: Added.

(WebCore):

WebCoreSupport/FrameLoaderClientBlackBerry.cpp:

(WebCore::FrameLoaderClientBlackBerry::dispatchDidClearWindowObjectInWorld):

LayoutTests:

Adding BlackBerry platform specific DRT tests for window.external interface.

platform/blackberry/external/search-provider/search-provider-installed-expected.txt: Added.
platform/blackberry/external/search-provider/search-provider-installed.html: Added.

4:20 PM Changeset in webkit [133093] by senorblanco@chromium.org

9 edits
4 deletes in trunk/LayoutTests

[chromium] Win rebaselines for media tests. Unreviewed.

platform/chromium-win/media/video-aspect-ratio-expected.png:
platform/chromium-win/media/video-colorspace-yuv420-expected.png:
platform/chromium-win/media/video-colorspace-yuv422-expected.png:
platform/chromium-win/media/video-layer-crash-expected.png:
platform/chromium-win/media/video-transformed-expected.png:
platform/chromium-win/media/video-zoom-controls-expected.png:
platform/chromium-win/media/video-zoom-expected.png:
platform/chromium-win/platform/chromium/media/video-frame-size-change-expected.png:
platform/efl/media/video-aspect-ratio-expected.png: Removed.
platform/efl/media/video-layer-crash-expected.png: Removed.
platform/efl/media/video-transformed-expected.png: Removed.
platform/efl/media/video-zoom-expected.png: Removed.

4:09 PM Changeset in webkit [133092] by roger_fong@apple.com

2 edits in trunk/Source/WebCore

Change PopupMenu positioning on Windows such that behaviour on multiple monitors matches Windows standards.
https://bugs.webkit.org/show_bug.cgi?id=100317

Reviewed by Sam Weinig.

The existing code determines which screen the popup menu "belongs" to by determining which screen the owning application's hwnd belongs to,
where ownership is determined by how much of the hwnd is on which screen.
To match what most Windows applications do, the owning screen should be whichever screen the drop down button belongs to.
To determine which screen an element belongs to in Windows we need to pass in an hwnd for that element.
However, since the drop down button is something that WebKit renders there is no hwnd.

To remedy this issue, we can temporarily move the popup menu's hwnd to match the position and size of the button,
determine the correct screen, and then eventually move it back to the correct final position after the rest of
the calculations have been completed. This is all done in the same function call so no rendering of the popup menu occurs
between the temporary and final positionings.

There's not really a good way of testing popup menus except manually, they're separate hwnds created outside of the WebView.

platform/win/PopupMenuWin.cpp:

(WebCore::monitorFromHwnd):
(WebCore):
(WebCore::PopupMenuWin::show):
(WebCore::PopupMenuWin::calculatePositionAndSize):

3:50 PM Changeset in webkit [133091] by simonjam@chromium.org

4 edits
1 delete in branches/chromium/1312

Revert 129070 - [Chromium] Disable resource load scheduling
https://bugs.webkit.org/show_bug.cgi?id=97131

Reviewed by Nate Chapin.

Source/WebCore:

We'll use Chrome's network stack for scheduling instead.

No new tests.

loader/ResourceLoadScheduler.cpp:
loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::preload):

LayoutTests:

The preload scanner loads resources sooner and these tests are affected.

platform/chromium/http/tests/css/link-css-disabled-value-with-slow-loading-sheet-in-error-expected.txt: Added.
platform/chromium/http/tests/inspector/network/network-initiator-expected.txt: Added.

TBR=simonjam@chromium.org
Review URL: https://codereview.chromium.org/11367020

3:50 PM Changeset in webkit [133090] by Lucas Forschler

16 edits
2 adds in branches/safari-536.28-branch

Merge 121003. '<rdar://problem/12536517>'

3:40 PM Changeset in webkit [133089] by mark.lam@apple.com

2 edits in trunk/Source/JavaScriptCore

64-bit llint PC offset can be negative: using an unsigned shift is a bug.
https://bugs.webkit.org/show_bug.cgi?id=100896.

Reviewed by Filip Pizlo.

Fixed the PC offset divisions in the 64-bit llint asm to use rshift instead of urshift.

llint/LowLevelInterpreter64.asm:

3:34 PM Changeset in webkit [133088] by Lucas Forschler

6 edits
2 copies in branches/safari-536.28-branch

Merged r121001. <rdar://problem/12504451>

3:32 PM Changeset in webkit [133087] by dpranke@chromium.org

2 edits in trunk/LayoutTests

Unreviewed, chromium-mac-mountainlion expectation updating.

platform/chromium-mac-mountainlion/TestExpectations:

3:27 PM Changeset in webkit [133086] by thakis@chromium.org

2 edits in branches/chromium/1312/Source/WebCore

Merge 132961 (merge bug: http://crbug.com/158840) - Fix potential overflow in jpeg exif reader. Found by aedla@google.com.
https://bugs.webkit.org/show_bug.cgi?id=100320

Reviewed by Eric Seidel.

Adding more than 1 element past an array is undefined, so don't do it.

No test, since in practice ifd will just overflow and end - ifd will
become much larget than 2 and the if (end - ifd < 2) a few lines
down will catch that case.

platform/image-decoders/jpeg/JPEGImageDecoder.cpp:

(WebCore::readImageOrientation):

TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/11361023

3:27 PM Changeset in webkit [133085] by commit-queue@webkit.org

4 edits
3 adds in trunk

Apply transformation for highlights on non-composited contents
https://bugs.webkit.org/show_bug.cgi?id=100438

Patch by Tien-Ren Chen <trchen@chromium.org> on 2012-10-31
Reviewed by Adam Barth.

Non-composited content host layer "steals" the contents on the
RenderView layer, including the highlight layer.
The transformation for regular contents are applied by the paint
function of the non-composited content host however the highlight
uses its own WebLayer. Need to apply the same transformation to
the highlight layer as well.

Source/WebKit/chromium:

src/LinkHighlight.cpp:

(WebKit::LinkHighlight::computeEnclosingCompositingLayer):

src/LinkHighlight.h:

(LinkHighlight):

LayoutTests:

platform/chromium-linux/compositing/gestures/gesture-tapHighlight-simple-scaled-document.html: Added.
platform/chromium-linux/platform/chromium-linux/compositing/gestures/gesture-tapHighlight-simple-scaled-document-expected.png: Added.
platform/chromium-linux/platform/chromium-linux/compositing/gestures/gesture-tapHighlight-simple-scaled-document-expected.txt: Added.

3:22 PM Changeset in webkit [133084] by commit-queue@webkit.org

26 edits
5 adds in trunk

Added viewport at-rule to the CSS parser and tokenizer
https://bugs.webkit.org/show_bug.cgi?id=95961

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Enable CSS Device Adaptation by default on EFL.

Source/cmake/OptionsEfl.cmake:

Source/WebCore:

Add tokens and grammar rules to parse @-webkit-viewport blocks. Also add
the newly parsed rule to the rule list.

This parser now implements the following part of the CSS Device Adaptation
specification: http://www.w3.org/TR/css-device-adapt/#syntax

Test: css3/device-adapt/viewport-at-rule-parsing.html

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
css/CSSGrammar.y.in:
css/CSSParser.cpp:

(WebCore::CSSParser::CSSParser):
(WebCore::CSSParser::detectAtToken):
(WebCore):
(WebCore::CSSParser::createViewportRule):

css/CSSParser.h:

(CSSParser):
(WebCore::CSSParser::markViewportRuleBodyStart):
(WebCore::CSSParser::markViewportRuleBodyEnd):
(WebCore::CSSParser::inViewport):

These methods are needed by the next patch validating the properties.
Some viewport properties are common to other rules but have different
semantics, and accepts different keywords. The validation needs to be
done in a different code path.

css/CSSPropertySourceData.h:
css/CSSRule.cpp:

(WebCore):
(WebCore::CSSRule::cssText):
(WebCore::CSSRule::destroy):
(WebCore::CSSRule::reattach):
(WebCore::CSSRule::reportMemoryUsage):

css/CSSRule.h:

(CSSRule):
(WebCore::CSSRule::isViewportRule):

css/StyleRule.cpp:

(WebCore::StyleRuleBase::reportMemoryUsage):
(WebCore::StyleRuleBase::destroy):
(WebCore::StyleRuleBase::copy):
(WebCore::StyleRuleBase::createCSSOMWrapper):
(WebCore):
(WebCore::StyleRuleViewport::StyleRuleViewport):
(WebCore::StyleRuleViewport::~StyleRuleViewport):
(WebCore::StyleRuleViewport::mutableProperties):
(WebCore::StyleRuleViewport::setProperties):
(WebCore::StyleRuleViewport::reportDescendantMemoryUsage):

css/StyleRule.h:

(StyleRuleBase):
(WebCore::StyleRuleBase::isViewportRule):
(WebCore):
(StyleRuleViewport):
(WebCore::StyleRuleViewport::create):
(WebCore::StyleRuleViewport::properties):
(WebCore::StyleRuleViewport::copy):

css/WebKitCSSViewportRule.cpp: Added.

(WebCore):
(WebCore::WebKitCSSViewportRule::WebKitCSSViewportRule):
(WebCore::WebKitCSSViewportRule::~WebKitCSSViewportRule):
(WebCore::WebKitCSSViewportRule::style):
(WebCore::WebKitCSSViewportRule::cssText):
(WebCore::WebKitCSSViewportRule::reattach):
(WebCore::WebKitCSSViewportRule::reportDescendantMemoryUsage):

css/WebKitCSSViewportRule.h: Added.

(WebCore):

Tools:

Enable CSS Device Adaptation by default on EFL.

Scripts/webkitperl/FeatureList.pm:

LayoutTests:

Added a test that detects if the @-webkit-viewport rule is being
parsed and skip the tests on all ports since the feature is disabled
by default for instance.

Note that this test doesn't do property validation, which will be
added in the next patch of the series.

css3/device-adapt/viewport-at-rule-parsing-expected.txt: Added.
css3/device-adapt/viewport-at-rule-parsing.html: Added.
platform/chromium/TestExpectations:
platform/gtk/TestExpectations:
platform/mac/TestExpectations:
platform/qt/TestExpectations:
platform/win/TestExpectations:
platform/wincairo/TestExpectations:

3:13 PM Changeset in webkit [133083] by commit-queue@webkit.org

3 edits
8 adds in trunk

[CSS Shaders] Validate types of built-in uniforms
https://bugs.webkit.org/show_bug.cgi?id=98974

Patch by Max Vujovic <mvujovic@adobe.com> on 2012-10-31
Reviewed by Dean Jackson.

Source/WebCore:

Reject shaders which define built-in uniforms with the wrong type. For example, we reject a
shader with the GLSL code "uniform float u_textureSize;" because u_textureSize should be a
vec2, not a float.

Relevant Spec Section:
https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#shader-uniform-variables

Test: css3/filters/custom/invalid-custom-filter-uniform-types.html

platform/graphics/filters/CustomFilterValidatedProgram.cpp:

(WebCore):
(WebCore::builtInUniformNameToTypeMap):
(WebCore::validateSymbols):

LayoutTests:

Add a test to verify that shaders which define built-in uniforms with the wrong type do not
execute. Add shaders for each built-in uniform in CSS Custom Filters.

Relevant Spec Section:
https://dvcs.w3.org/hg/FXTF/raw-file/tip/filters/index.html#shader-uniform-variables

css3/filters/custom/invalid-custom-filter-uniform-types-expected.html: Added.
css3/filters/custom/invalid-custom-filter-uniform-types.html: Added.
css3/filters/resources/invalid-type-u-mesh-box.fs: Added.
css3/filters/resources/invalid-type-u-mesh-size.fs: Added.
css3/filters/resources/invalid-type-u-projection-matrix.fs: Added.
css3/filters/resources/invalid-type-u-texture-size.fs: Added.
css3/filters/resources/invalid-type-u-tile-size.fs: Added.
css3/filters/resources/invalid-type-uniform-array.fs: Added.

3:10 PM Changeset in webkit [133082] by tdanderson@chromium.org

4 edits in trunk

[touchadjustment] touch-links-two-finger-tap test pass incorrectly
https://bugs.webkit.org/show_bug.cgi?id=100619

Reviewed by Antonio Gomes.

Tools:

Store the touch area when sending a GestureTwoFingerTap event.

DumpRenderTree/chromium/TestRunner/src/EventSender.cpp:

(WebTestRunner):
(WebTestRunner::EventSender::gestureEvent):

LayoutTests:

Make use of the gesture's touch area when testing the correct targeting of elements.

touchadjustment/touch-links-two-finger-tap.html:

3:08 PM Changeset in webkit [133081] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. More flaky fallout from
http://trac.webkit.org/changeset/133069.

platform/chromium/TestExpectations:

2:56 PM Changeset in webkit [133080] by Lucas Forschler

3 edits
2 copies in branches/safari-536.28-branch

Merged r118249. <rdar://problem/12536403>

2:49 PM Changeset in webkit [133079] by Lucas Forschler

15 edits
4 copies
2 deletes in branches/safari-536.28-branch

Merged r117971. <rdar://problem/12536368>

2:36 PM Changeset in webkit [133078] by commit-queue@webkit.org

9 edits in trunk

Unreviewed, rolling out r133044.
http://trac.webkit.org/changeset/133044
https://bugs.webkit.org/show_bug.cgi?id=100888

Hits an ASSERT in the isolatedWorlds tests (Requested by
abarth on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-10-31

Source/WebCore:

bindings/scripts/CodeGeneratorV8.pm:

(NeedsToVisitDOMWrapper):
(GenerateVisitDOMWrapper):
(GenerateHeader):
(GenerateImplementation):

bindings/v8/V8GCController.cpp:

(WebCore::ObjectVisitor::ObjectVisitor):
(WebCore::ObjectVisitor::visitDOMWrapper):
(ObjectVisitor):
(WebCore::rootForGC):
(ImplicitConnection):
(WebCore::ImplicitConnection::ImplicitConnection):
(WebCore::ImplicitConnection::root):
(WebCore::ImplicitConnection::wrapper):
(WebCore):
(WebCore::operator<):
(WebCore::NodeVisitor::NodeVisitor):
(WebCore::NodeVisitor::visitNodeWrapper):
(NodeVisitor):
(WebCore::NodeVisitor::applyGrouping):
(WebCore::V8GCController::majorGCPrologue):

bindings/v8/V8GCController.h:

(V8GCController):

bindings/v8/WrapperTypeInfo.h:

(WebCore):
(WebCore::WrapperTypeInfo::visitDOMWrapper):
(WrapperTypeInfo):

bindings/v8/custom/V8NodeListCustom.cpp:

(WebCore::V8NodeList::visitDOMWrapper):

bindings/v8/custom/V8SpeechRecognitionResultCustom.cpp:

(WebCore::V8SpeechRecognitionResult::visitDOMWrapper):

LayoutTests:

platform/chromium/fast/dom/gc-9-expected.txt:

2:30 PM Changeset in webkit [133077] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Updating TestExpectations for
recent failures.

platform/chromium/TestExpectations:

2:28 PM Changeset in webkit [133076] by dpranke@chromium.org

2 edits in trunk/Tools

nrwt is not killing stuck DRTs on chromium win
https://bugs.webkit.org/show_bug.cgi?id=100886

Unreviewed, build fix.

In the original implementation of NRWT for Chromium, we used
to assume that DRT would always manage to exit on its own and
so NRWT wouldn't attempt to kill DRT if it was stuck (this was the
"hung worker" problem, and meant that if DRT hung, NRWT would hang).

Eventually, we merged the chromium code with the other ports and
now will always try to shut down DRTs if a test times out, but as
part of that merge we left in a code path where we wouldn't
shut the driver down on windows. This was resulting in stray
DRTs being left around, and NRWT sometimes not exiting cleanly as
a result.

No unit tests added because this was a windows-only code path (hence
hard to test) and that is now gone.

Scripts/webkitpy/layout_tests/port/server_process.py:

(ServerProcess.stop):

2:19 PM Changeset in webkit [133075] by weinig@apple.com

21 edits in trunk/Source/WebKit2

Convert the rest of the encoders to take the encoder as a reference
https://bugs.webkit.org/show_bug.cgi?id=100821

Reviewed by Anders Carlsson.

Platform/CoreIPC/ArgumentCoders.h:

(CoreIPC::SimpleArgumentCoder::encode):

Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:

(CoreIPC::::encode):
(CoreIPC::encodeTimingFunction):

Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.h:
Shared/PrintInfo.cpp:

(WebKit::PrintInfo::encode):

Shared/SessionState.cpp:
Shared/UserMessageCoders.h:

(WebKit::UserMessageDecoder::baseDecode):

Shared/WebCoreArgumentCoders.cpp:

(CoreIPC::::encode):

Shared/WebCoreArgumentCoders.h:
Shared/WebGeolocationPosition.cpp:

(WebKit::WebGeolocationPosition::Data::encode):

Shared/WebLayerTreeInfo.cpp:

(WebKit::WebLayerInfo::encode):

Shared/gtk/ArgumentCodersGtk.cpp:

(CoreIPC::encodeImage):
(CoreIPC::encodeDataObject):
(CoreIPC::::encode):
(CoreIPC::encodeGKeyFile):
(CoreIPC::encode):

Shared/gtk/ArgumentCodersGtk.h:
Shared/mac/WebCoreArgumentCodersMac.mm:

(CoreIPC::::encodePlatformData):
(CoreIPC::::encode):

Shared/qt/ArgumentCodersQt.cpp:

(CoreIPC::::encode):

Shared/qt/ArgumentCodersQt.h:

(CoreIPC):

Shared/qt/WebCoreArgumentCodersQt.cpp:

(CoreIPC::::encodePlatformData):

Shared/soup/WebCoreArgumentCodersSoup.cpp:

(CoreIPC::::encodePlatformData):

Shared/win/WebCoreArgumentCodersWin.cpp:

(CoreIPC::::encodePlatformData):

WebProcess/WebPage/DecoderAdapter.cpp:

(WebKit::DecoderAdapter::decodeBytes):

WebProcess/WebPage/EncoderAdapter.cpp:

(WebKit::EncoderAdapter::encodeBytes):

1:54 PM Changeset in webkit [133074] by pdr@google.com

17 edits in trunk

Cache animationMode() in SVG animations.
https://bugs.webkit.org/show_bug.cgi?id=99694

Reviewed by Eric Seidel.

Source/WebCore:

This patch caches animationMode() which accounted for 25% of the CPU time of a simple
animation benchmark! This is the 3rd patch for WK99694.

Background: SVGAnimationElement contains most of the shared animation logic for <animate>,
<set>, <animateColor>, <animateTransform>, and <animateMotion>. <animateMotion> is the only
animation element that can depend on other elements in the page. For example:
<path id="mypath" d="M0 0 L 100 100Z"/> 
<rect x="0" y="0" width="100" height="100" fill="green">

<animateMotion dur="6s" repeatCount="indefinite">

<mpath xlink:href="#mypath"/>

</animateMotion>

</rect>
See: http://www.w3.org/TR/SVG/single-page.html#animate-AnimateMotionElement

animationMode depends on several properties of an animation element: the set tag, whether
the animation has a path, and the "values", "to", and "by" attributes. animationMode() was
a hot function for two reasons:
1) hasAttribute(SVGNames::valuesAttr) is expensive because we are unable to use

fastHasAttribute. This is cacheable by simply calculating the animationMode when the
"values" attribute changes.

2) In the <animateMotion> case, determining if a path is empty or changed is expensive.

AnimateMotion can have a path attribute, or it can have <mpath> children that reference
a (possibly non-existent) <path> element elsewhere in the page. Before this patch we did
this path lookup on every animation loop in case something changed. After this patch we
only update AnimateMotion's path when it changes.

A previous patch, http://trac.webkit.org/changeset/132847, laid the groundwork for tracking
<path> changes in <mpath>. This patch adds <mpath> to our resource tracking infrastructure
to track when target <path>s change, instead of looking this up every time.

This refactoring is covered by existing tests.
svg/animations/mozilla/animateMotion-mpath-targetChange-1.svg fails after this patch
because our element dependency tracking has a bug with duplicate ids; see WK99893.

svg/SVGAnimateMotionElement.cpp:

(WebCore::SVGAnimateMotionElement::parseAttribute):
(WebCore::SVGAnimateMotionElement::updateAnimationPath):

animationPath() has been changed to updateAnimationPath() and should only be called
when the animation path (path attribute, or mpath's referenced path) has changed.

(WebCore::SVGAnimateMotionElement::buildTransformForProgress):
(WebCore::SVGAnimateMotionElement::updateAnimationMode):

If an animationPath exists, we use PathAnimation, otherwise we fall back to the normal
updateAnimationMode() codepath.

(WebCore):

svg/SVGAnimateMotionElement.h:

(SVGAnimateMotionElement):

svg/SVGAnimationElement.cpp:

(WebCore::SVGAnimationElement::SVGAnimationElement):
(WebCore::SVGAnimationElement::isSupportedAttribute):
(WebCore::SVGAnimationElement::parseAttribute):

The from, by, and to attributes have been added so that we can track when they change
and update the animation mode. Similarly for when the values attribute changes.

(WebCore::SVGAnimationElement::updateAnimationMode):

svg/SVGAnimationElement.h:

(WebCore::SVGAnimationElement::animationMode):
(SVGAnimationElement):
(WebCore::SVGAnimationElement::setAnimationMode):
(WebCore::SVGAnimationElement::calculateDistance):

svg/SVGMPathElement.cpp:

(WebCore::SVGMPathElement::buildPendingResource):

It would be nice to move all the duplicated buildPendingResource() logic into a central
place (SVGURIReference?) but for now it is copied. This function is nearly identical to
SVGFEImageElement::buildPendingResource.

(WebCore):
(WebCore::SVGMPathElement::clearResourceReferences):
(WebCore::SVGMPathElement::insertedInto):
(WebCore::SVGMPathElement::removedFrom):
(WebCore::SVGMPathElement::svgAttributeChanged):
(WebCore::SVGMPathElement::targetPathChanged):
(WebCore::SVGMPathElement::notifyParentOfPathChange):

svg/SVGMPathElement.h:

(SVGMPathElement):

svg/SVGPathElement.cpp:

When a <path>'s path changes, we need to notify any dependent <mpath> elements. This is
typically handled with RenderSVGResource::markForLayoutAndParentResourceInvalidation
but for the special-case of <mpath> we only need to track when the path's "d" attribute
changes so invalidateMPathDependencies() has been added.

(WebCore::SVGPathElement::svgAttributeChanged):
(WebCore::SVGPathElement::invalidateMPathDependencies):
(WebCore):
(WebCore::SVGPathElement::insertedInto):
(WebCore::SVGPathElement::removedFrom):

svg/SVGPathElement.h:

(SVGPathElement):

svg/SVGSetElement.cpp:

(WebCore::SVGSetElement::SVGSetElement):
(WebCore::SVGSetElement::updateAnimationMode):
(WebCore):

svg/SVGSetElement.h:

(SVGSetElement):

svg/animation/SVGSMILElement.cpp:

(WebCore::SVGSMILElement::isSupportedAttribute):

LayoutTests:

platform/chromium/TestExpectations:
platform/mac/TestExpectations:

1:49 PM Changeset in webkit [133073] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Updating TestExpectations for
recent failures.

platform/chromium/TestExpectations:

1:45 PM Changeset in webkit [133072] by Lucas Forschler

12 edits
6 copies
3 deletes in branches/safari-536.28-branch

Merged r117865. <rdar://problem/12536400>

1:42 PM Changeset in webkit [133071] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening.

platform/chromium/TestExpectations:

1:38 PM Changeset in webkit [133070] by Lucas Forschler

8 edits
15 copies in branches/safari-536.28-branch

Merged r117482. <rdar://problem/12536395>

1:35 PM Changeset in webkit [133069] by fischman@chromium.org

4 edits in trunk

[chromium] DRT and WTR should clear the cache between tests
https://bugs.webkit.org/show_bug.cgi?id=93195

Reviewed by Tony Chang.

Tools:

This change makes chromium DRT match the GTK+, Qt, and EFL ports.

DumpRenderTree/chromium/TestShell.cpp:

(TestShell::resetTestController): added a WebCache::clear() call to achieve the goal of the bug.

LayoutTests:

media/video-poster-blocked-by-willsendrequest.html: updated list of still-busted ports.

1:11 PM Changeset in webkit [133068] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Unreviewed gardening. Mark a few tests as crashing.

platform/chromium/TestExpectations:

1:11 PM Changeset in webkit [133067] by Beth Dakin

6 edits in trunk/Source/WebCore

https://bugs.webkit.org/show_bug.cgi?id=100879
ScrollingStateNode::cloneAndResetNode() should not be virtual

Reviewed by Simon Fraser.

cloneAndResetNode() is currently pure virtual and implemented only
in ScrollingStateScrollingNode. However, all of the work that it
does at this time is stuff that a generic ScrollingStateNode could
do. We should move this implementation to the base class so that it
does not need to be duplicated for future node types.

This patch also re-names cloneAndResetNode() to cloneAndReset()
and correspondingly re-names cloneAndResetChildNodes() to
cloneAndResetChildren().

Finally the patch also changes the copy constructors of both of these
classes to take a const reference instead of a pointer.

page/scrolling/ScrollingStateNode.cpp:

(WebCore::ScrollingStateNode::ScrollingStateNode):
(WebCore::ScrollingStateNode::cloneAndReset):
(WebCore):
(WebCore::ScrollingStateNode::cloneAndResetChildren):

page/scrolling/ScrollingStateNode.h:

(ScrollingStateNode):

page/scrolling/ScrollingStateScrollingNode.cpp:

(WebCore::ScrollingStateScrollingNode::ScrollingStateScrollingNode):
(WebCore):

page/scrolling/ScrollingStateScrollingNode.h:

(ScrollingStateScrollingNode):

page/scrolling/ScrollingStateTree.cpp:

(WebCore::ScrollingStateTree::commit):

1:07 PM Changeset in webkit [133066] by tsepez@chromium.org

7 edits
5 copies
2 moves
3 adds in trunk

Source/WebCore: Malformed X-XSS-Protection headers not reported.
https://bugs.webkit.org/show_bug.cgi?id=100538

Reviewed by Adam Barth.

Re-writes X-XSS-Protection header parser to be more particular, and to
return better information on error.

Tests: http/tests/security/xssAuditor/malformed-xss-protection-header-1.html

http/tests/security/xssAuditor/malformed-xss-protection-header-2.html
http/tests/security/xssAuditor/malformed-xss-protection-header-4.html
http/tests/security/xssAuditor/xss-protection-parsing-02.html

html/parser/XSSAuditor.cpp:

(WebCore::XSSAuditor::init):
Detect error return code and log console message with details

platform/network/HTTPParsers.cpp:

(WebCore):
(WebCore::skipWhiteSpace):
Use safe less-than comparsion in case called with pos already out of range.
(WebCore::skipToken):
Fix comparison to properly reject substrings at end of input. Prevent advancing
returned position when match fails, so that this may someday be used to match
optional tokens.
(WebCore::parseXSSProtectionHeader):
Return detailled error status. Avoid needless string copy.

platform/network/HTTPParsers.h:

Add new error returns for x-xss-protection header parser.

LayoutTests: Malformed X-XSS-Protection headers not reported
https://bugs.webkit.org/show_bug.cgi?id=100538

Reviewed by Adam Barth.

http/tests/security/xssAuditor/malformed-xss-protection-header-expected.txt: Removed.
http/tests/security/xssAuditor/malformed-xss-protection-header.html: Removed.
http/tests/security/xssAuditor/malformed-xss-protection-header-1.html: Copied from LayoutTests/http/tests/security/xssAuditor/malformed-xss-protection-header.html.
http/tests/security/xssAuditor/malformed-xss-protection-header-expected-1.txt: Copied from LayoutTests/http/tests/security/xssAuditor/malformed-xss-protection-header-expected.txt.

renamed to -1 to match convention.

http/tests/security/xssAuditor/malformed-xss-protection-header-2-expected.txt: Added.
http/tests/security/xssAuditor/malformed-xss-protection-header-2.html: Added.
http/tests/security/xssAuditor/malformed-xss-protection-header-3-expected.txt: Added.
http/tests/security/xssAuditor/malformed-xss-protection-header-3.html: Added.
http/tests/security/xssAuditor/malformed-xss-protection-header-4-expected.txt: Added.
http/tests/security/xssAuditor/malformed-xss-protection-header-4.html: Added.

New test to cover new error message cases.

http/tests/security/xssAuditor/resources/echo-intertag.pl:

Added more x-xss-protection header values to its set of values.

http/tests/security/xssAuditor/xss-protection-parsing-01.html:

Converted for interface change to echo_intertag.pl

http/tests/security/xssAuditor/xss-protection-parsing-02-expected.txt: Added.
http/tests/security/xssAuditor/xss-protection-parsing-02.html: Added.

New test for testing x-xss-protect: 0; (allow trailing semicolon).

1:02 PM Changeset in webkit [133065] by kbr@google.com

1 edit in branches/chromium/1312/Source/WebCore/html/canvas/WebGLRenderingContext.cpp

Merge 132980 - Set the initial scissor box for WebGL
https://bugs.webkit.org/show_bug.cgi?id=100804

Patch by Gregg Tavares <gman@google.com> on 2012-10-30
Reviewed by Kenneth Russell.

The initial scissor box needs to be set for WebGL

No new tests as no new functionality.

html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::initializeNewContext):

TBR=commit-queue@webkit.org
Review URL: https://codereview.chromium.org/11366017

1:01 PM Changeset in webkit [133064] by dpranke@chromium.org

3 edits in trunk/Tools

Fix typos introduced in r133061.
Unreviewed, build fix.

BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js:
Scripts/webkitpy/tool/servers/reflectionhandler.py:

(ReflectionHandler._serve_file):

12:50 PM Changeset in webkit [133063] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Fix incorrect test names landed in
http://trac.webkit.org/changeset/133056

Unreviewed.

platform/chromium/TestExpectations:

12:43 PM Changeset in webkit [133062] by senorblanco@chromium.org

2 edits
1 move in trunk/LayoutTests

[chromium] New linux baseline for basic-inputs-expected.png, due to
crrev.com/164985.

Unreviewed.

platform/chromium-linux/fast/forms/basic-inputs-expected.png:
platform/efl-wk1/fast/forms/basic-inputs-expected.png: Renamed from LayoutTests/platform/efl/fast/forms/basic-inputs-expected.png.

12:36 PM Changeset in webkit [133061] by dpranke@chromium.org

9 edits in trunk/Tools

garden-o-matic should work for local results
https://bugs.webkit.org/show_bug.cgi?id=100805

Reviewed by Adam Barth.

This is the next step in making garden-o-matic replace
'webkit-patch rebaseline-server' and be able to deal with local
results. garden-o-matic now accepts a '--results-directory' option
that will point to a layout-test-results directory. At the moment
this will only work if you are also filtering to a single bot/
platform at a time (we can't handle multiple bots' worth of local
results, but that is a logical next step).

BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/config.js:
BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js:
Scripts/webkitpy/tool/commands/gardenomatic.py:

(GardenOMatic.init):
(GardenOMatic.execute):

Scripts/webkitpy/tool/commands/rebaseline.py:

(AbstractRebaseliningCommand):
(RebaselineTest.init):
(RebaselineTest._rebaseline_test):
(RebaselineTest._rebaseline_test_and_update_expectations):
(RebaselineTest.execute):
(AbstractParallelRebaselineCommand._rebaseline_commands):
(AbstractParallelRebaselineCommand._rebaseline):
(RebaselineJson.init):

Scripts/webkitpy/tool/commands/rebaseline_unittest.py:

(_BaseTestCase):
(test_rebaseline_updates_expectations_file):
(test_rebaseline_does_not_include_overrides):
(test_rebaseline_test):
(test_rebaseline_test_and_print_scm_changes):
(test_rebaseline_and_copy_test):
(test_rebaseline_and_copy_test_no_existing_result):
(test_rebaseline_and_copy_test_with_lion_result):
(test_rebaseline_and_copy_no_overwrite_test):
(test_rebaseline_test_internal_with_move_overwritten_baselines_to):
(TestRebaselineJson.test_rebaseline_all):
(TestRebaselineJson.test_rebaseline_debug):
(TestRebaselineJson.test_move_overwritten):
(TestRebaselineJson.test_no_optimize):
(TestRebaselineExpectations.test_rebaseline_expectations):

Scripts/webkitpy/tool/servers/gardeningserver.py:

(GardeningHTTPRequestHandler.rebaselineall):
(GardeningHTTPRequestHandler):
(GardeningHTTPRequestHandler.localresult):

Scripts/webkitpy/tool/servers/reflectionhandler.py:

(ReflectionHandler.do_POST):
(ReflectionHandler):
(ReflectionHandler.do_HEAD):

12:34 PM Changeset in webkit [133060] by dpranke@chromium.org

14 edits in trunk/Tools

garden-o-matic should support gardening a single port and specifying how to deal with overwritten baselines
https://bugs.webkit.org/show_bug.cgi?id=100563

Reviewed by Ojan Vafai.

This patch starts the process of making garden-o-matic more useful for
bringing up a single new port in two ways. The goal is to fully
obsolete webkit-patch rebaseline-server.

First, it adds support for the --platform arguments to garden-o-matic
(so that the UI will default to the right cluster of bots), and if the platform arg
specifies a single bot, filters the results down to just that bot.

Second, it adds support for --move-overwritten-baselines and changes
the implementation of the move_overwritten_baselines setting in builders.py
so that it is only used if this flag is provided.

Future patches will make it possible to use local copies of the build
results (rather than going to the bots) and other optimizations to make
it faster to review lots of changes on a single bot.

BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/garden-o-matic.html:
BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/config.js:

(.):

Scripts/webkitpy/layout_tests/port/builders.py:
Scripts/webkitpy/tool/commands/gardenomatic.py:

(GardenOMatic):
(GardenOMatic.execute):

Scripts/webkitpy/tool/commands/rebaseline.py:

(AbstractRebaseliningCommand):
(AbstractRebaseliningCommand.init):
(RebaselineTest):
(OptimizeBaselines):
(AnalyzeBaselines):
(AbstractParallelRebaselineCommand):
(AbstractParallelRebaselineCommand._rebaseline_commands):
(AbstractParallelRebaselineCommand._rebaseline):
(Rebaseline.init):

Scripts/webkitpy/tool/servers/gardeningserver.py:

(GardeningHTTPServer.url):
(GardeningHTTPRequestHandler.rebaselineall):

12:29 PM Changeset in webkit [133059] by commit-queue@webkit.org

4 edits in trunk/Source

Enable webkit2 introspection
https://bugs.webkit.org/show_bug.cgi?id=94313

Patch by Jesse van den Kieboom <jessevdk@gnome.org> on 2012-10-31
Reviewed by Martin Robinson.

This patch enables generation of introspection information for webkit2
in the same was as it is done for webkit. The resulting gir and
typelib file are named WebKit2-3.0.{gir,typelib}.

GNUmakefile.am:
UIProcess/API/gtk/WebKitBackForwardList.cpp:

12:20 PM Changeset in webkit [133058] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[Chromium] Adjust expectations for float-wrap-zoom on mac.
Unreviewed.

platform/chromium/TestExpectations:

12:17 PM Changeset in webkit [133057] by commit-queue@webkit.org

7 edits
5 adds in trunk/Source/WebKit2

[EFL][WK2] Add Ewk_File_Chooser_Request API
https://bugs.webkit.org/show_bug.cgi?id=100818

Patch by Christophe Dumez <Christophe Dumez> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Add Ewk_File_Chooser_Request API so that the client
can handle file chooser requests (e.g. Show a file
picker dialog) when the user clicks on an input
element of type "file".

PlatformEfl.cmake:
UIProcess/API/efl/EWebKit2.h:
UIProcess/API/efl/EwkViewCallbacks.h:

(EwkViewCallbacks):

UIProcess/API/efl/ewk_file_chooser_request.cpp: Added.

(Ewk_File_Chooser_Request::Ewk_File_Chooser_Request):
(Ewk_File_Chooser_Request::~Ewk_File_Chooser_Request):
(Ewk_File_Chooser_Request::allowMultipleFiles):
(Ewk_File_Chooser_Request::acceptedMIMETypes):
(Ewk_File_Chooser_Request::cancel):
(Ewk_File_Chooser_Request::chooseFiles):
(ewk_file_chooser_request_ref):
(ewk_file_chooser_request_unref):
(ewk_file_chooser_request_allow_multiple_files_get):
(ewk_file_chooser_request_accepted_mimetypes_get):
(ewk_file_chooser_request_cancel):
(ewk_file_chooser_request_files_choose):
(ewk_file_chooser_request_file_choose):

UIProcess/API/efl/ewk_file_chooser_request.h: Added.
UIProcess/API/efl/ewk_file_chooser_request_private.h: Added.

(WebKit):
(Ewk_File_Chooser_Request):
(Ewk_File_Chooser_Request::create):
(Ewk_File_Chooser_Request::wasHandled):

UIProcess/API/efl/ewk_view.h:
UIProcess/API/efl/tests/resources/file_chooser.html: Added.
UIProcess/API/efl/tests/test_ewk2_file_chooser_request.cpp: Added corresponding

unit tests.
(onFileChooserRequest):
(compareStrings):
(freeStringList):
(TEST_F):

UIProcess/efl/PageUIClientEfl.cpp:

(WebKit::PageUIClientEfl::runOpenPanel):
(WebKit):
(WebKit::PageUIClientEfl::PageUIClientEfl):

UIProcess/efl/PageUIClientEfl.h:

(PageUIClientEfl):

12:01 PM Changeset in webkit [133056] by Simon Fraser

8 edits
8 adds in trunk

REGRESSION (tile cache layers): bits of tiled layers are missing with certain 3D transforms
https://bugs.webkit.org/show_bug.cgi?id=100808
<rdar://problem/12562541>

Reviewed by Dean Jackson.

Source/WebCore:

When projecting rects down into transformed layers, the projection can fail with severe
3D rotations if the computed w component in TransformationMatrix::projectPoint() is negative.
In this case we already clamp, but the fact that we clamped doesn't make it out to
GraphicsLayerCA::computeVisibleRect() which resulted in incorrect visible rects being
computed.

Fix by propagating the fact that clamping occurred out of the TransformState functions
which can clamp. In computeVisibleRect(), simply consider the entire layer bounds
to be visible if clamping occurred.

Tests: compositing/tiling/rotated-tiled-clamped.html

compositing/tiling/rotated-tiled-preserve3d-clamped.html

platform/graphics/ca/GraphicsLayerCA.cpp:

(WebCore::GraphicsLayerCA::computeVisibleRect): If either the applyTransform()
or the state.mappedQuad() clamped, use our bounds as the visible rect.

platform/graphics/transforms/TransformState.cpp:

(WebCore::TransformState::applyTransform): Pass out clamping state.
(WebCore::TransformState::flatten): Ditto.
(WebCore::TransformState::mappedPoint): Ditto.
(WebCore::TransformState::mappedQuad): Ditto.
(WebCore::TransformState::flattenWithTransform): Ditto. No need to initialize
wasClamped, since this function is internal.

platform/graphics/transforms/TransformState.h:

(TransformState):

platform/graphics/transforms/TransformationMatrix.cpp:

(WebCore::TransformationMatrix::projectQuad): If any point projection clamped,
set the flag to say we clamped.

platform/graphics/transforms/TransformationMatrix.h:

(TransformationMatrix):

LayoutTests:

Two tests for visible rect computation under clamping, one of which tests 'applyWasClamped',
and the other 'mapWasClamped'.

compositing/tiling/rotated-tiled-clamped-expected.png: Added.
compositing/tiling/rotated-tiled-clamped-expected.txt: Added.
compositing/tiling/rotated-tiled-clamped.html: Added.
compositing/tiling/rotated-tiled-preserve3d-clamped-expected.png: Added.
compositing/tiling/rotated-tiled-preserve3d-clamped-expected.txt: Added.
compositing/tiling/rotated-tiled-preserve3d-clamped.html: Added.
platform/chromium/TestExpectations:
platform/mac/compositing/tiling/rotated-tiled-clamped-expected.txt: Added.
platform/mac/compositing/tiling/rotated-tiled-preserve3d-clamped-expected.txt: Added.

12:01 PM Changeset in webkit [133055] by dpranke@chromium.org

3 edits in trunk/Tools

test-webkitpy: fix running modules and classes on the command line
https://bugs.webkit.org/show_bug.cgi?id=100787

Reviewed by Eric Seidel.

Handling modules and classes are arguments on the command line
was broken due to the way we were parsing test names looking for
parallel and serial tests.

Scripts/webkitpy/test/main.py:

(Tester._test_names):
(_Loader):

Scripts/webkitpy/test/main_unittest.py:

(TestStubs.integration_test_empty):
(TestStubs):
(TestStubs.serial_test_empty):
(TestStubs.serial_integration_test_empty):
(TesterTest._find_test_names):
(TesterTest):
(TesterTest.test_individual_names_are_not_run_twice):
(TesterTest.test_integration_tests_are_not_found_by_default):
(TesterTest.test_integration_tests_are_found):

11:51 AM Changeset in webkit [133054] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Change bubble message style to BB10 UX spec.
https://bugs.webkit.org/show_bug.cgi?id=100862

Patch by Tiancheng Jiang <tijiang@rim.com> on 2012-10-31
Reviewed by Rob Buis.

RIM PR 198108
Internal Reviewed by Otto Cheung.
No new tests.

css/themeBlackBerry.css:

(::-webkit-validation-bubble-message): Added.
(::-webkit-validation-bubble-arrow): Added.
(::-webkit-validation-bubble-heading): Added.

11:47 AM Changeset in webkit [133053] by mkwst@chromium.org

10 edits in trunk/Source

Prefer document->addConsoleMessage to document->domWindow->console->addMessage.
https://bugs.webkit.org/show_bug.cgi?id=100850

Reviewed by Adam Barth.

For historical reasons, a few places in WebCore talk to Console directly
via 'document()->domWindow()->console()->addMessage(...)'. This is more
safely wrapped by calling 'addConsoleMessage' on the Document itself.

No visible functionality should change; we'll simply avoid potential
null dereferences in the future.

Source/WebCore:

html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::validateInteractively):

html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::printWarningToConsole):

loader/FrameLoader.cpp:

(WebCore::FrameLoader::reportLocalLoadFailed):

loader/MixedContentChecker.cpp:

(WebCore::MixedContentChecker::logWarning):

loader/appcache/ApplicationCacheGroup.cpp:

(WebCore::ApplicationCacheGroup::abort):
(WebCore::ApplicationCacheGroup::didReceiveResponse):
(WebCore::ApplicationCacheGroup::didFinishLoading):
(WebCore::ApplicationCacheGroup::didFail):
(WebCore::ApplicationCacheGroup::didReceiveManifestResponse):
(WebCore::ApplicationCacheGroup::didFinishLoadingManifest):
(WebCore::ApplicationCacheGroup::checkIfLoadIsComplete):

loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::printAccessDeniedMessage):

Source/WebKit/chromium:

src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::addMessageToConsole):

11:45 AM Changeset in webkit [133052] by pfeldman@chromium.org

14 edits in trunk/Source/WebCore

Web Inspector: frame chooser does not work on subsequent inspector open.
https://bugs.webkit.org/show_bug.cgi?id=100771

Reviewed by Yury Semikhatsky.

Make WorkerRuntimeAgent and PageRuntimeAgent register themselves in the instrumenting agents independently
Move pause / run worker handling from InspectorRuntimeAgent into WorkerRuntimeAgent
Remove remains of the front-end reused signal from InspectorAgent
Send execution contexts post clear window object instead of post commit load.

inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::InspectorAgent):
(WebCore::InspectorAgent::clearFrontend):
(WebCore::InspectorAgent::didCommitLoad):

inspector/InspectorAgent.h:

(InspectorAgent):

inspector/InspectorInstrumentation.cpp:

(WebCore):
(WebCore::InspectorInstrumentation::didClearWindowObjectInWorldImpl):
(WebCore::InspectorInstrumentation::willEvaluateWorkerScript):

inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):

inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::restore):

inspector/InspectorRuntimeAgent.cpp:

(WebCore::InspectorRuntimeAgent::InspectorRuntimeAgent):
(WebCore::InspectorRuntimeAgent::~InspectorRuntimeAgent):
(WebCore::InspectorRuntimeAgent::run):
(WebCore::InspectorRuntimeAgent::setScriptDebugServer):

inspector/InspectorRuntimeAgent.h:

(InspectorRuntimeAgent):

inspector/InstrumentingAgents.h:

(WebCore):
(WebCore::InstrumentingAgents::InstrumentingAgents):
(WebCore::InstrumentingAgents::workerRuntimeAgent):
(WebCore::InstrumentingAgents::setWorkerRuntimeAgent):
(InstrumentingAgents):

inspector/PageRuntimeAgent.cpp:

(WebCore::PageRuntimeAgent::PageRuntimeAgent):
(WebCore::PageRuntimeAgent::~PageRuntimeAgent):
(WebCore::PageRuntimeAgent::setFrontend):
(WebCore::PageRuntimeAgent::clearFrontend):
(WebCore::PageRuntimeAgent::enable):
(WebCore::PageRuntimeAgent::didCreateMainWorldContext):

inspector/PageRuntimeAgent.h:

(PageRuntimeAgent):

inspector/WorkerRuntimeAgent.cpp:

(WebCore::WorkerRuntimeAgent::WorkerRuntimeAgent):
(WebCore::WorkerRuntimeAgent::~WorkerRuntimeAgent):
(WebCore::WorkerRuntimeAgent::run):
(WebCore):
(WebCore::WorkerRuntimeAgent::pauseWorkerContext):

inspector/WorkerRuntimeAgent.h:

(WorkerRuntimeAgent):

11:32 AM Changeset in webkit [133051] by dpranke@chromium.org

6 edits in trunk/Tools

webkitpy: refactor common command line arguments for --platform, rebaseline commands
https://bugs.webkit.org/show_bug.cgi?id=100800

Reviewed by Ojan Vafai.

This change shuffles around how we share command-line options
between NRWT and the various webkit-patch rebaselining commands.

There are no functional changes in this patch but it should make
things clearer and will be useful in a couple of upcoming patches.

Scripts/webkitpy/layout_tests/port/init.py:
Scripts/webkitpy/layout_tests/port/factory.py:

(platform_options):
(configuration_options):

Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(parse_args):

Scripts/webkitpy/tool/commands/queries.py:

(PrintExpectations.init):
(PrintBaselines.init):

Scripts/webkitpy/tool/commands/rebaseline.py:

(AbstractRebaseliningCommand):
(AbstractRebaseliningCommand.init):
(RebaselineTest.init):
(OptimizeBaselines.init):
(AnalyzeBaselines.init):
(AbstractParallelRebaselineCommand):
(RebaselineJson.init):
(RebaselineExpectations.init):
(Rebaseline.init):

11:28 AM Changeset in webkit [133050] by commit-queue@webkit.org

2 edits in trunk/Tools

[NRWT] Xvfb driver is leaking application cache files
https://bugs.webkit.org/show_bug.cgi?id=100864

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Application Cache looks for XDG_CACHE_HOME which is now
prepended with DUMPRENDERTREE_TEMP and cleaned up when
the test worker finishes his job.

Scripts/webkitpy/layout_tests/port/xvfbdriver.py:

(XvfbDriver._start):

11:24 AM Changeset in webkit [133049] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

[Qt][EFL][AC][WK2]TextureMapperLayer backing store contents are drawn upside down on screen.
https://bugs.webkit.org/show_bug.cgi?id=100845.

Patch by Kondapally Kalyan <kalyan.kondapally@intel.com> on 2012-10-31
Reviewed by Noam Rosenthal.

WebGL displays the Canvas with (0,0) being the bottom left corner.
In GraphicsSurface::platformPaintToTextureMapper we don't set ShouldFlipTexture
flag before painting to TextureMapper.
It results in backing store contents being drawn upside down on screen.
This patch enables ShouldFlipTexture flag when TextureMapperLayer renders
it's backing store contents into a TextureMapper.

platform/graphics/surfaces/qt/GraphicsSurfaceGLX.cpp:

(WebCore::GraphicsSurface::platformPaintToTextureMapper):

11:18 AM Changeset in webkit [133048] by abarth@webkit.org

2 edits in trunk/Websites/webkit.org

technical-articles.html should link to my talk on how WebKit works
https://bugs.webkit.org/show_bug.cgi?id=100867

Reviewed by Tony Chang.

coding/technical-articles.html:

11:06 AM Changeset in webkit [133047] by noam.rosenthal@nokia.com

7 edits in trunk/Source

[Texmap] Enable filter animations in GraphicsLayerAnimation
https://bugs.webkit.org/show_bug.cgi?id=100318

Reviewed by Kenneth Rohde Christiansen.

Source/WebCore:

Use the same method of animating filters in WebCore to animate filters for TextureMapper.
Added the appropriate methods to GraphicsLayerAnimation and TextureMapperLayer.

Tested by LayoutTests/css3/filters/filter-animation-hw.html and other tests.

platform/graphics/GraphicsLayerAnimation.cpp:

(WebCore):
(WebCore::blendFunc):
(WebCore::applyFilterAnimation):
(WebCore::GraphicsLayerAnimation::applyInternal):

platform/graphics/GraphicsLayerAnimation.h:

(Client):

platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::intermediateSurfaceRect):
(WebCore::TextureMapperLayer::shouldPaintToIntermediateSurface):
(WebCore::TextureMapperLayer::paintRecursive):
(WebCore::TextureMapperLayer::syncAnimations):

platform/graphics/texmap/TextureMapperLayer.h:

(TextureMapperLayer):
(WebCore::TextureMapperLayer::setFilters):
(WebCore::TextureMapperLayer::setAnimatedFilters):

Source/WebKit2:

Encode/decode KeyframeValues of type FilterOperations.

Shared/CoordinatedGraphics/CoordinatedGraphicsArgumentCoders.cpp:

(CoreIPC::::encode):
(CoreIPC::::decode):

11:02 AM Changeset in webkit [133046] by commit-queue@webkit.org

3 edits in trunk/Source/WebKit2

[EFL][WK2][AC] Avoid storing dirty rects in a Vector inside EwkViewImpl
https://bugs.webkit.org/show_bug.cgi?id=100736

Patch by Christophe Dumez <Christophe Dumez> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

No longer store dirty rectangles in a temporary Vector and construct
a WebCore::Region directly from them instead. This avoid having to
iterate over the Vector in EwkViewImpl::displayTimerFired() to construct
a Region object, which should be more efficient.

UIProcess/API/efl/EwkViewImpl.cpp:

(EwkViewImpl::displayTimerFired):
(EwkViewImpl::redrawRegion):

UIProcess/API/efl/EwkViewImpl.h:

(WebCore):
(EwkViewImpl):

10:36 AM Changeset in webkit [133045] by andersca@apple.com

33 edits in trunk/Source/WebKit2

Connection::Client::didReceiveInvalidMessage should take the full message name
https://bugs.webkit.org/show_bug.cgi?id=100788

Reviewed by Sam Weinig.

Change Connection::Client::didReceiveInvalidMessage to take the message receiver name and message name as parameters
and make WebProcessProxy::didReceiveInvalidMessage print out the full message name.

NetworkProcess/NetworkConnectionToWebProcess.cpp:

(WebKit::NetworkConnectionToWebProcess::didReceiveInvalidMessage):

NetworkProcess/NetworkConnectionToWebProcess.h:

(NetworkConnectionToWebProcess):

NetworkProcess/NetworkProcess.cpp:

(WebKit::NetworkProcess::didReceiveInvalidMessage):

NetworkProcess/NetworkProcess.h:

(NetworkProcess):

Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::dispatchMessage):

Platform/CoreIPC/Connection.h:

(Client):

Platform/CoreIPC/StringReference.cpp:

(CoreIPC::StringReference::toString):
(CoreIPC):

Platform/CoreIPC/StringReference.h:

(StringReference):

PluginProcess/PluginProcess.cpp:

(WebKit::PluginProcess::didReceiveInvalidMessage):

PluginProcess/PluginProcess.h:

(PluginProcess):

PluginProcess/WebProcessConnection.cpp:

(WebKit::WebProcessConnection::didReceiveInvalidMessage):

PluginProcess/WebProcessConnection.h:

(WebProcessConnection):

SharedWorkerProcess/SharedWorkerProcess.cpp:

(WebKit::SharedWorkerProcess::didReceiveInvalidMessage):

SharedWorkerProcess/SharedWorkerProcess.h:

(SharedWorkerProcess):

UIProcess/Network/NetworkProcessProxy.cpp:

(WebKit::NetworkProcessProxy::didReceiveInvalidMessage):

UIProcess/Network/NetworkProcessProxy.h:

(NetworkProcessProxy):

UIProcess/Plugins/PluginProcessProxy.cpp:

(WebKit::PluginProcessProxy::didReceiveInvalidMessage):

UIProcess/Plugins/PluginProcessProxy.h:

(PluginProcessProxy):

UIProcess/SharedWorkers/SharedWorkerProcessProxy.cpp:

(WebKit::SharedWorkerProcessProxy::didReceiveInvalidMessage):

UIProcess/SharedWorkers/SharedWorkerProcessProxy.h:

(SharedWorkerProcessProxy):

UIProcess/WebConnectionToWebProcess.cpp:

(WebKit::WebConnectionToWebProcess::didReceiveInvalidMessage):

UIProcess/WebConnectionToWebProcess.h:

(WebConnectionToWebProcess):

UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::didReceiveInvalidMessage):

UIProcess/WebProcessProxy.h:

(WebProcessProxy):

WebProcess/Network/NetworkProcessConnection.cpp:

(WebKit::NetworkProcessConnection::didReceiveInvalidMessage):

WebProcess/Network/NetworkProcessConnection.h:

(NetworkProcessConnection):

WebProcess/Plugins/PluginProcessConnection.cpp:

(WebKit::PluginProcessConnection::didReceiveInvalidMessage):

WebProcess/Plugins/PluginProcessConnection.h:

(PluginProcessConnection):

WebProcess/WebConnectionToUIProcess.cpp:

(WebKit::WebConnectionToUIProcess::didReceiveInvalidMessage):

WebProcess/WebConnectionToUIProcess.h:

(WebConnectionToUIProcess):

WebProcess/WebProcess.cpp:

(WebKit::WebProcess::didReceiveInvalidMessage):

WebProcess/WebProcess.h:

(WebProcess):

10:33 AM Changeset in webkit [133044] by abarth@webkit.org

9 edits in trunk

[V8] Garbage collection should use opaque roots rather than implicit references
https://bugs.webkit.org/show_bug.cgi?id=100707

Reviewed by Kentaro Hara.

Source/WebCore:

After this patch, we now use object groups rather than implicit
references to inform V8 of these relationships. That has two benefits:

1) We no longer need to know which DOMDataStore a wrapper belongs

because we don't need to find the exact source wrapper for the
implicit connection.

2) We can now handle more complicated implicit relationships, for

example when some of the intervening objects haven't had their
JavaScript wrappers created yet.

This patch also unlocks to paths of future development:
A) Fixing the remaining failures in fast/dom/gc-9.html
B) Enumerating DOM wrappers entirely from V8 rather than from the

DOMWrapperMaps (so that we can move more object towards using
IntrusiveDOMWrapperMaps, which aren't enumerable from WebCore).

bindings/scripts/CodeGeneratorV8.pm:

(NeedsCustomOpaqueRootForGC):
(GenerateOpaqueRootForGC):
(GenerateHeader):
(GenerateImplementation):

bindings/v8/V8GCController.cpp:

bindings/v8/V8GCController.h:

(WebCore):
(V8GCController):

bindings/v8/WrapperTypeInfo.h:

(WebCore):
(WebCore::WrapperTypeInfo::opaqueRootForGC):
(WrapperTypeInfo):

bindings/v8/custom/V8NodeListCustom.cpp:

(WebCore::V8NodeList::opaqueRootForGC):

bindings/v8/custom/V8SpeechRecognitionResultCustom.cpp:

(WebCore::V8SpeechRecognitionResult::opaqueRootForGC):

LayoutTests:

Test progression.

platform/chromium/fast/dom/gc-9-expected.txt:

10:21 AM Changeset in webkit [133043] by jsbell@chromium.org

1 edit in branches/chromium/1312/Source/WebCore/Modules/indexeddb/IDBLevelDBBackingStore.cpp

Merge 132848 - IndexedDB: Crash on checking version of corrupt backing store
https://bugs.webkit.org/show_bug.cgi?id=100692

Reviewed by Tony Chang.

If the backing store fails to open (due to corruption, non-writeable disk, etc)
the subsequent schema version check dereferences a null pointer. Fix to only
do the schema check if the database opened.

Chromium tests will be included with crrev.com/11196029

Modules/indexeddb/IDBLevelDBBackingStore.cpp:

(WebCore::IDBLevelDBBackingStore::open):

TBR=jsbell@chromium.org
Review URL: https://codereview.chromium.org/11358013

10:07 AM Changeset in webkit [133042] by senorblanco@chromium.org

2 edits in trunk/LayoutTests

[chromium] Added skipped expectations for new test
svg-remote-element.html with no results.

Unreviewed.

platform/chromium/TestExpectations:

9:48 AM Changeset in webkit [133041] by commit-queue@webkit.org

8 edits
4 adds in trunk/Source

[EFL][WK2][AC] Use smart pointers for Evas_GL types
https://bugs.webkit.org/show_bug.cgi?id=100745

Patch by Christophe Dumez <Christophe Dumez> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Source/WebKit2:

Replace raw pointers to Evas_GL, Evas_GL_Context and
Evas_GL_Surface by smart pointers in EwkViewImpl.

C++ wrappers were added for Evas_GL_Context and
Evas_GL_Surface in order to use OwnPtr. This is needed
because their destroy functions need the Evas_GL.

PlatformEfl.cmake:
UIProcess/API/efl/EvasGLContext.cpp: Added.

(WebKit):
(WebKit::EvasGLContext::EvasGLContext):
(WebKit::EvasGLContext::~EvasGLContext):

UIProcess/API/efl/EvasGLContext.h: Added.

(WebKit):
(EvasGLContext):
(WebKit::EvasGLContext::create):
(WebKit::EvasGLContext::context):

UIProcess/API/efl/EvasGLSurface.cpp: Added.

(WebKit):
(WebKit::EvasGLSurface::EvasGLSurface):
(WebKit::EvasGLSurface::~EvasGLSurface):

UIProcess/API/efl/EvasGLSurface.h: Added.

(WebKit):
(EvasGLSurface):
(WebKit::EvasGLSurface::create):
(WebKit::EvasGLSurface::surface):

UIProcess/API/efl/EwkViewImpl.cpp:

(EwkViewImpl::EwkViewImpl):
(EwkViewImpl::displayTimerFired):
(EwkViewImpl::createGLSurface):
(EwkViewImpl::enterAcceleratedCompositingMode):
(EwkViewImpl::exitAcceleratedCompositingMode):

UIProcess/API/efl/EwkViewImpl.h:

(EwkViewImpl::evasGL):
(EwkViewImpl::evasGLContext):
(EwkViewImpl::evasGLSurface):
(EwkViewImpl::resetEvasGLSurface):
(EwkViewImpl):

UIProcess/API/efl/ewk_view.cpp:

(_ewk_view_smart_calculate):

Source/WTF:

Make OwnPtr usable with Evas_GL type to avoid handling
raw pointers in EFL port.

wtf/OwnPtrCommon.h:

(WTF):

wtf/efl/OwnPtrEfl.cpp:

(WTF):
(WTF::deleteOwnedPtr):

9:46 AM Changeset in webkit [133040] by commit-queue@webkit.org

2 edits in trunk/Source/WebCore

Web Inspector: Add total node to native memory snapshot tree
https://bugs.webkit.org/show_bug.cgi?id=100843

Patch by Alexei Filippov <alph@chromium.org> on 2012-10-31
Reviewed by Yury Semikhatsky.

inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.NativeSnapshotDataGrid):
(WebInspector.NativeSnapshotNode):
(WebInspector.MemoryBlockViewProperties._initialize):

9:45 AM Changeset in webkit [133039] by kbalazs@webkit.org

3 edits in trunk/Source/WebKit2

[Qt][WK2] Fix "Unable to find a renderable master window QQuickView" warnings in tst_QQuickWebView
https://bugs.webkit.org/show_bug.cgi?id=100247

Reviewed by Jocelyn Turcotte.

Make sure we have a visible window for tests that want to render.

UIProcess/API/qt/tests/qquickwebview/tst_qquickwebview.cpp:

(tst_QQuickWebView::showWebView):
(tst_QQuickWebView::removeFromCanvas):
(tst_QQuickWebView::multipleWebViewWindows):
(tst_QQuickWebView::multipleWebViews):
(tst_QQuickWebView::basicRenderingSanity):
(tst_QQuickWebView::transparentWebViews):
Removed some unnecessary resize now that the window have a default size.

UIProcess/API/qt/tests/testwindow.h:

(TestWindow): Resize the window to a default size at construction.

9:42 AM Changeset in webkit [133038] by senorblanco@chromium.org

1 edit
1 add
2 deletes in trunk/LayoutTests

[chromium] New baselines for
fast/images/exif-orientation-css-expected.png. Unreviewed.

platform/chromium-linux-x86/fast/images/exif-orientation-css-expected.png: Removed.
platform/chromium-win/fast/images/exif-orientation-css-expected.png: Added.
platform/chromium/fast/images/exif-orientation-css-expected.png: Removed.

9:32 AM Changeset in webkit [133037] by commit-queue@webkit.org

10 edits
2 adds in trunk

Table with percentage column widths doesn't scale to fill the entire width of a table containing it
https://bugs.webkit.org/show_bug.cgi?id=11645

Patch by Arpita Bahuguna <arpitabahuguna@gmail.com> on 2012-10-31
Reviewed by Julien Chaffraix.

Source/WebCore:

When the inner (or nested) table has auto width and column(s) with
percent width, we should scale our column(s) to the width of the
containing table (unless it has auto width).

Test: fast/table/scale-nested-percent-width-cols.html

rendering/AutoTableLayout.cpp:

(WebCore::shouldScaleColumns):
If the containing table width was of percent type, we were disallowing
our inner or nested table's column from scaling to the size of the parent
table. We should prohibit the scaling of the nested table columns only
if the parent table has auto width.

LayoutTests:

fast/table/scale-nested-percent-width-cols-expected.html: Added.
fast/table/scale-nested-percent-width-cols.html: Added.

Added a ref test verifying that our nested (auto width) table's columns
are scaled and sized appropriately if the containing table has
percent width.

platform/chromium/TestExpectations:
platform/efl/TestExpectations:
platform/gtk/TestExpectations:
platform/mac/TestExpectations:
platform/win/TestExpectations:

Added an entry for fast/table/025.html for these ports. It would need
rebaselining as a result of this fix.

platform/qt/fast/table/025-expected.png:
platform/qt/fast/table/025-expected.txt:

Modified expected results for the existing test on qt port.

In the given test, our innermost nested table has auto width and consits
of three table cells with percent width specified for the first and the
third cells. Its containing table on the other hand is a percent width
table.

Previously, we were not scaling our inner table's column(s) from expanding
to the width of the containing table if it had percent width type. Because
of this our first and last table cells in the innermost table were being
rendered only with the minimum width (4px).

Post this fix, since now our inner table columns shall scale, the first and
the last table cell would thus expand in width (41px each) and hence the
change in the column and the table dimensions.

8:53 AM Changeset in webkit [133036] by commit-queue@webkit.org

2 edits in trunk/Tools

[EFL] DRT doesn't clean up after itself
https://bugs.webkit.org/show_bug.cgi?id=100346

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

The test driver exports an environment variable DUMPRENDERTREE_TEMP
which is used by both DRT and WTR as the folder for storing icons,
databases, application cache, etc, overriding the default XDG_* path.

We don't need to create these folders because in fact, they are never
used and also never cleaned. Things created inside DUMPRENDERTREE_TEMP
are automatically removed after the test run.

Scripts/webkitpy/layout_tests/port/efl.py:

(EflPort.setup_environ_for_server):

8:53 AM Changeset in webkit [133035] by commit-queue@webkit.org

3 edits in trunk/Source/WebKit2

[EFL] WebContext::platformDefaultIconDatabasePath() should return a path to a file
https://bugs.webkit.org/show_bug.cgi?id=100679

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

This patch aligns the EFL implementation with other ports. Before
that, we had an inconsistence: the return value of the getter could
not be used on the setter (the setter expects a full path, not a
directory).

UIProcess/API/efl/ewk_context.cpp:

(Ewk_Context::faviconDatabase):

UIProcess/efl/WebContextEfl.cpp:

(WebKit::WebContext::platformDefaultIconDatabasePath):

8:43 AM Changeset in webkit [133034] by rakuco@webkit.org

2 edits in trunk/LayoutTests

[EFL] More unreviewed pixel expectations gardening.

Accept Missing as a valid result for some ietestcenter tests, and
skip another one of them.

platform/efl/TestExpectations:

8:36 AM Changeset in webkit [133033] by vollick@chromium.org

20 edits
2 adds
15 deletes in trunk

Add support for text-based repaint testing
https://bugs.webkit.org/show_bug.cgi?id=100584

Reviewed by Simon Fraser.

Allows tracked repaint rects to be dumped as text.

Source/autotools/symbols.filter: Exports for:

FrameView::setTracksRepaints(bool)
Frame::trackedRepaintRectsAsText() const

Source/WebCore:

The following layout tests have been converted:

fast/repaint/layer-full-repaint.html
fast/repaint/overflow-flipped-writing-mode-table.html

WebCore.exp.in: Exports for:

FrameView::setTracksRepaints(bool)
Frame::trackedRepaintRectsAsText() const

page/Frame.cpp:

(WebCore::Frame::trackedRepaintRectsAsText):

Returns a string containing information on all tracked repaint rects.

page/FrameView.cpp:

(WebCore::FrameView::setTracksRepaints):

Rather than calling m_trackedRepaintRects.clear() directly, I've
called resetTrackedRepaints(). This will allow us to do more
sophisticated resetting when we start tracking repaint rects for
composited layers.

(WebCore::FrameView::trackedRepaintRectsAsText):

Provides the string returned by Frame::trackedRepaintRectsAsText.

testing/Internals.cpp:

(WebCore::Internals::repaintRectsAsText):

Returns Frame::trackedRepaintRectsAsText.

(WebCore):
(WebCore::Internals::startTrackingRepaints):

Calls FrameView::setTracksRepaints(true)

(WebCore::Internals::stopTrackingRepaints):

Calls FrameView::setTracksRepaints(false)

testing/Internals.h:
testing/Internals.idl: Declarations for:

repaintRectsAsText
startTrackingRepaints
stopTrackingRepaints

Source/WebKit2:

mac/WebKit2.order:
win/WebKit2.def:
win/WebKit2CFLite.def: Exports for:

FrameView::setTracksRepaints(bool)
Frame::trackedRepaintRectsAsText() const

LayoutTests:

fast/repaint/layer-full-repaint-expected.txt: Added.
fast/repaint/layer-full-repaint.html:
fast/repaint/overflow-flipped-writing-mode-table-expected.png: Removed.
fast/repaint/overflow-flipped-writing-mode-table-expected.txt:
fast/repaint/overflow-flipped-writing-mode-table.html:
fast/repaint/resources/repaint.js:
fast/repaint/resources/text-based-repaint.js: Added.

(runRepaintTest):

The function calls internals.startTrackingRepaints() rather than display()
before calling repaintTest(). It also generates a <pre> element and sets
its innerHTML to internals.repaintRectsAsText().

platform/chromium-mac/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/chromium-mac/fast/repaint/overflow-flipped-writing-mode-table-expected.png: Removed.
platform/chromium-win/fast/repaint/overflow-flipped-writing-mode-table-expected.png: Removed.
platform/chromium/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/chromium/fast/repaint/layer-full-repaint-expected.txt: Removed.
platform/efl/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/efl/fast/repaint/layer-full-repaint-expected.txt: Removed.
platform/gtk/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/gtk/fast/repaint/layer-full-repaint-expected.txt: Removed.
platform/gtk/fast/repaint/overflow-flipped-writing-mode-table-expected.png: Removed.
platform/mac/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/mac/fast/repaint/layer-full-repaint-expected.txt: Removed.
platform/qt/fast/repaint/layer-full-repaint-expected.png: Removed.
platform/qt/fast/repaint/layer-full-repaint-expected.txt: Removed.

8:27 AM Changeset in webkit [133032] by senorblanco@chromium.org

1 edit
2 adds in trunk/LayoutTests

New Chromium baselines for css3/filters/composited-reflected.html.

Unreviewed.

platform/chromium-mac/css3/filters/composited-reflected-expected.png: Added.
platform/chromium-win/css3/filters/composited-reflected-expected.png: Added.

8:22 AM Changeset in webkit [133031] by mkwst@chromium.org

4 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Rebaselining tests after r133019.

platform/qt/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
platform/qt/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
platform/qt/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:

8:12 AM Changeset in webkit [133030] by yurys@chromium.org

10 edits
4 adds in trunk

Web Inspector: page crash when pausing in dedicated worker
https://bugs.webkit.org/show_bug.cgi?id=100837

Reviewed by Pavel Feldman.

Source/WebCore:

Pass debugger agent to InjectedScriptHost::init in case of workers.

Test: inspector-protocol/debugger-pause-dedicated-worker.html

inspector/WorkerInspectorController.cpp:

(WebCore::WorkerInspectorController::WorkerInspectorController):

LayoutTests:

Test that Debugger.pause works for dedicated workers. The test is skipped
on all platforms that use JSC as worker debugging is not supported there
yet.

http/tests/inspector-protocol/resources/InspectorTest.js:

(WebInspector.dispatchMessageFromBackend):

inspector-protocol/debugger-pause-dedicated-worker-expected.txt: Added.
inspector-protocol/debugger-pause-dedicated-worker.html: Added.
inspector-protocol/resources/dedicated-worker.js: Added.

(onmessage):
(doWork):

platform/efl/TestExpectations:
platform/gtk/TestExpectations:
platform/mac/TestExpectations:
platform/qt/TestExpectations:
platform/win/TestExpectations:
platform/wincairo/TestExpectations:

8:01 AM Changeset in webkit [133029] by commit-queue@webkit.org

5 edits in trunk

perspective-origin should ignore values with more then two lengths and use the default values
https://bugs.webkit.org/show_bug.cgi?id=100835

Patch by Zoltan Nyul <zoltan.nyul@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Source/WebCore:

Perspective-origin should ignore values with more then two lengths
(http://www.w3.org/TR/css3-transforms/#perspective-origin) and use
the default values instead of parsing it as an unlimited-length list,
of which all but the first two values are ignored. This makes perspective-origin
behave similar to transform-origin which uses default value in this case,
as well as firefox does the same.

Test: transforms/2d/computed-style-origin.html

css/CSSParser.cpp:

(WebCore::CSSParser::parsePerspectiveOrigin):

LayoutTests:

Add test for perspective-origin with more the than two lengths.

transforms/2d/computed-style-origin-expected.txt:
transforms/2d/computed-style-origin.html:

7:59 AM Changeset in webkit [133028] by podivilov@chromium.org

15 edits
3 deletes in trunk/Source

Unreviewed, rolling out r133016.
http://trac.webkit.org/changeset/133016
https://bugs.webkit.org/show_bug.cgi?id=100856

broke compile-webkit on several bots (Requested by podivilov
on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-10-31

Source/WebCore:

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
dom/DeviceOrientationClient.h:

(DeviceOrientationClient):

dom/DeviceOrientationController.cpp:

dom/DeviceOrientationController.h:

(WebCore):
(DeviceOrientationController):
(WebCore::DeviceOrientationController::isActive):
(WebCore::DeviceOrientationController::client):
(WebCore::DeviceOrientationController::from):

dom/Document.cpp:

(WebCore::Document::suspendActiveDOMObjects):
(WebCore::Document::resumeActiveDOMObjects):

loader/EmptyClients.h:

(WebCore::EmptyDeviceOrientationClient::startUpdating):
(WebCore::EmptyDeviceOrientationClient::stopUpdating):

page/DOMWindow.cpp:

(WebCore::DOMWindow::addEventListener):
(WebCore::DOMWindow::removeEventListener):
(WebCore::DOMWindow::removeAllEventListeners):

page/DeviceClient.h: Removed.
page/DeviceController.cpp: Removed.
page/DeviceController.h: Removed.

Source/WebKit/qt:

WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setMockDeviceOrientation):

7:22 AM Changeset in webkit [133027] by rakuco@webkit.org

2 edits
2 deletes in trunk/LayoutTests

[EFL] Unreviewed pixel expectations gardening.

Skip a few ietestcenter/css3/text texts which are rendering
incorrectly, and accept Missing as a valid result for some culling
tests.

platform/efl/TestExpectations:
platform/efl/ietestcenter/css3/text/textshadow-005-expected.png: Removed.
platform/efl/ietestcenter/css3/text/textshadow-005-expected.txt: Removed.

7:13 AM Changeset in webkit [133026] by dominik.rottsches@intel.com

3 edits in trunk/LayoutTests

[EFL][GTK] Unreviewed gardening.

Removing duplicate in EFL file, changing bug number for exif orientation bug in GTK.

platform/efl/TestExpectations:
platform/gtk/TestExpectations:

7:01 AM Changeset in webkit [133025] by dominik.rottsches@intel.com

4 edits in trunk/LayoutTests

[EFL][GTK] Unreviewed gardening.

Exif orientation test marked as failing for GTK & EFL,
On EFL, the vibration IPC issue may cause crash, see bug 100334.

platform/efl-wk2/TestExpectations:
platform/efl/TestExpectations:
platform/gtk/TestExpectations:

7:01 AM Changeset in webkit [133024] by commit-queue@webkit.org

2 edits in trunk/Tools

[WTR] WebKitTestRunner is not cleaning up the icon database
https://bugs.webkit.org/show_bug.cgi?id=100678

Patch by Thiago Marcos P. Santos <thiago.santos@intel.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

The icon database was not being set to the temporary folder created for
the test runner by the test driver, which is removed automatically
after the run.

WebKitTestRunner/TestController.cpp:

(WTR::TestController::initialize):

6:59 AM Changeset in webkit [133023] by rakuco@webkit.org

2 edits in trunk/LayoutTests

[EFL] More unreviewed gardening.

Skip the EXIF-related tests since the feature is not implemented
for the port anyway, and skip the recently-added
fast/images/exif-orientation-image-document.html as well.

platform/efl/TestExpectations:

6:58 AM Changeset in webkit [133022] by rakuco@webkit.org

2 edits in trunk/LayoutTests

[EFL] Skip http/tests/appcache/load-from-appcache-defer-resume-crash.html on WK2.

Just marking it as Failure does not prevent a useless PNG expectation to be generated when running pixel tests.

platform/efl-wk2/TestExpectations:

6:55 AM Changeset in webkit [133021] by Antti Koivisto

3 edits in trunk/Source/WebCore

Remove stray calls to mutableAttributeData()
https://bugs.webkit.org/show_bug.cgi?id=100849

Reviewed by Andreas Kling.

Changing the cached class attribute value does not require mutable attribute data.

dom/Element.cpp:

(WebCore::Element::classAttributeChanged):

dom/ElementAttributeData.h:

(WebCore::ElementAttributeData::clearClass):

Make const like setClass().

6:49 AM Changeset in webkit [133020] by rakuco@webkit.org

2 edits
6 adds in trunk/LayoutTests

[EFL] Unreviewed, add a few more pixel expectations.

platform/efl/css3/flexbox/repaint-during-resize-no-flex-expected.png: Added.
platform/efl/css3/flexbox/repaint-rtl-column-expected.png:
platform/efl/fast/forms/indeterminate-expected.png: Added.
platform/efl/media/video-aspect-ratio-expected.png: Added.
platform/efl/media/video-layer-crash-expected.png: Added.
platform/efl/media/video-transformed-expected.png: Added.
platform/efl/media/video-zoom-expected.png: Added.

6:33 AM Changeset in webkit [133019] by mkwst@chromium.org

11 edits in trunk

X-Frame-Options console message should be associated with a request.
https://bugs.webkit.org/show_bug.cgi?id=100735

Reviewed by Pavel Feldman.

Source/WebCore:

In 99941, we added the possibility to tie console messages to requests,
which enables automatic generation of stack traces, line numbers, etc.
making the error simpler to diagnose for web developers. This patch
uses the piping laid in that patch to improve the console message
generated when a document is blocked by X-Frame-Options.

No new tests; the functionality is covered by changes to existing tests.

dom/Document.cpp:

(WebCore::Document::processHttpEquiv):

Grab the request identifier from the currently active DocumentLoader
and pass it into the console message.

loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::didReceiveResponse):

Grab the request identifier from the MainResourceLoader, and pass it
into the console message.

LayoutTests:

http/tests/inspector/network/x-frame-options-deny-expected.txt:
http/tests/security/XFrameOptions/x-frame-options-deny-expected.txt:
http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:
http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny-expected.txt:
http/tests/security/xssAuditor/meta-tag-http-refresh-x-frame-options-expected.txt:

Update tests with new console message.

6:25 AM Changeset in webkit [133018] by commit-queue@webkit.org

13 edits in trunk/Source/WebCore

Unreviewed, rolling out r133015.
http://trac.webkit.org/changeset/133015
https://bugs.webkit.org/show_bug.cgi?id=100847

broke Apple Win Debug compilation (Requested by podivilov on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-10-31

inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::InspectorAgent):
(WebCore::InspectorAgent::clearFrontend):
(WebCore::InspectorAgent::didCommitLoad):

inspector/InspectorAgent.h:

(WebCore::InspectorAgent::didCommitLoadFired):
(InspectorAgent):

inspector/InspectorInstrumentation.cpp:

(WebCore):
(WebCore::InspectorInstrumentation::didClearWindowObjectInWorldImpl):
(WebCore::InspectorInstrumentation::willEvaluateWorkerScript):
(WebCore::InspectorInstrumentation::runtimeAgentEnabled):

inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):

inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::restore):

inspector/InspectorRuntimeAgent.cpp:

inspector/InspectorRuntimeAgent.h:

(InspectorRuntimeAgent):

inspector/InstrumentingAgents.h:

(WebCore):
(WebCore::InstrumentingAgents::InstrumentingAgents):
(WebCore::InstrumentingAgents::inspectorRuntimeAgent):
(WebCore::InstrumentingAgents::setInspectorRuntimeAgent):
(InstrumentingAgents):

inspector/PageRuntimeAgent.cpp:

inspector/PageRuntimeAgent.h:

(PageRuntimeAgent):

inspector/WorkerRuntimeAgent.cpp:

(WebCore::WorkerRuntimeAgent::WorkerRuntimeAgent):
(WebCore::WorkerRuntimeAgent::~WorkerRuntimeAgent):

inspector/WorkerRuntimeAgent.h:

(WorkerRuntimeAgent):

6:00 AM Changeset in webkit [133017] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

[EFL] Skip EWK2UnitTestBase.ewk_context_vibration_client_callbacks_set API test
https://bugs.webkit.org/show_bug.cgi?id=100839

Unreviewed, Skip EWK2UnitTestBase.ewk_context_vibration_client_callbacks_set
API test since it is failing on the bots.

Patch by Sudarsana Nagineni <sudarsana.nagineni@intel.com> on 2012-10-31

UIProcess/API/efl/tests/test_ewk2_context.cpp:

(TEST_F):

5:55 AM Changeset in webkit [133016] by kihong.kwon@samsung.com

15 edits
3 adds in trunk/Source

Add DeviceController base-class to remove duplication of DeviceXXXControler
https://bugs.webkit.org/show_bug.cgi?id=96894

Reviewed by Hajime Morita.

Source/WebCore:

Add DeviceController which is extracted from DeviceOrientationController to remove duplication.
And soon-to-be-added DeviceMotionController and ProximityController.

Covered by existing tests.

CMakeLists.txt:
GNUmakefile.list.am:
Target.pri:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
dom/DeviceOrientationClient.h:
dom/DeviceOrientationController.cpp:

Remove member functions to move to DeviceController.

addListener(), removeListener(), removeAllListeners(), isActive()

dom/DeviceOrientationController.h:

(WebCore):
(WebCore::DeviceOrientationController::~DeviceOrientationController):
(DeviceOrientationController):

dom/Document.cpp:

loader/EmptyClients.h:

(EmptyDeviceClient):
(WebCore::EmptyDeviceClient::startUpdating):
(WebCore::EmptyDeviceClient::stopUpdating):
(WebCore):

page/DOMWindow.cpp:

(WebCore::DOMWindow::addEventListener):
(WebCore::DOMWindow::removeEventListener):
(WebCore::DOMWindow::removeAllEventListeners):

page/DeviceClient.h: Added.

(WebCore):
(DeviceClient):
(WebCore::DeviceClient::~DeviceClient):

page/DeviceController.cpp: Added.

DeviceController has extracted functions from DeviceOrientationController and DeviceMotionController.

addDeviceEventListener(), removeDeviceEventlistener(), removeAllDeviceEventListeners(), dispatchDeviceEvent()

page/DeviceController.h: Added.

Source/WebKit/qt:

Change client() to deviceOrientationClient() to get DeviceOrientationClient*.

WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setMockDeviceOrientation):

5:41 AM Changeset in webkit [133015] by pfeldman@chromium.org

13 edits in trunk/Source/WebCore

Web Inspector: frame chooser does not work on subsequent inspector open.
https://bugs.webkit.org/show_bug.cgi?id=100771

Reviewed by Yury Semikhatsky.

Make WorkerRuntimeAgent and PageRuntimeAgent register themselves in the instrumenting agents independently
Move pause / run worker handling from InspectorRuntimeAgent into WorkerRuntimeAgent
Remove remains of the front-end reused signal from InspectorAgent
Send execution contexts post clear window object instead of post commit load.

inspector/InspectorAgent.cpp:

(WebCore::InspectorAgent::InspectorAgent):
(WebCore::InspectorAgent::clearFrontend):
(WebCore::InspectorAgent::didCommitLoad):

inspector/InspectorAgent.h:

(InspectorAgent):

inspector/InspectorInstrumentation.cpp:

(WebCore):
(WebCore::InspectorInstrumentation::didClearWindowObjectInWorldImpl):
(WebCore::InspectorInstrumentation::willEvaluateWorkerScript):

inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):

inspector/InspectorPageAgent.cpp:

(WebCore::InspectorPageAgent::restore):

inspector/InspectorRuntimeAgent.cpp:

inspector/InspectorRuntimeAgent.h:

(InspectorRuntimeAgent):

inspector/InstrumentingAgents.h:

(WebCore):
(WebCore::InstrumentingAgents::InstrumentingAgents):
(WebCore::InstrumentingAgents::workerRuntimeAgent):
(WebCore::InstrumentingAgents::setWorkerRuntimeAgent):
(InstrumentingAgents):

inspector/PageRuntimeAgent.cpp:

inspector/PageRuntimeAgent.h:

(PageRuntimeAgent):

inspector/WorkerRuntimeAgent.cpp:

inspector/WorkerRuntimeAgent.h:

(WorkerRuntimeAgent):

5:19 AM Changeset in webkit [133014] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

[WK2][EFL] Assert check need to be changed in InputMethodContextEfl::InputMethodContextEfl
https://bugs.webkit.org/show_bug.cgi?id=100840

Patch by KyungTae Kim <ktf.kim@samsung.com> on 2012-10-31
Reviewed by Kenneth Rohde Christiansen.

Because 'context' is already set to 'm_context', assert check need to be done for m_context.

UIProcess/efl/InputMethodContextEfl.cpp:

(WebKit::InputMethodContextEfl::InputMethodContextEfl):

4:52 AM Changeset in webkit [133013] by rakuco@webkit.org

2 edits in trunk/LayoutTests

[EFL] Unreviewed, unskip media/video-seek-past-end-playing.html on WK2.

It has been passing on the WK2 bot for quite a while, so unskip
and hope for the best.

platform/efl-wk2/TestExpectations:

4:41 AM Changeset in webkit [133012] by kenneth@webkit.org

10 edits in trunk/Source/WebKit2

[WK2/EFL] Make sure all our events are transformed correctly
https://bugs.webkit.org/show_bug.cgi?id=100836

Reviewed by Alexis Menard.

When moving to using the tiled backing store, we need to
transform our events before sending them to the web process.

We needed to do that before as well, as the webkit view
doesn't have to be positioned at (0,0), but it was done in
a hacking way in the WebEvent creating.

Now we generate two transforms, one for transforming points
to the web content, and one for transforming them into the
device screen coordinate system.

This fixes the global position which was wrong before.

Shared/NativeWebMouseEvent.h:

(NativeWebMouseEvent):

Shared/NativeWebTouchEvent.h:

(NativeWebTouchEvent):

Shared/NativeWebWheelEvent.h:

(NativeWebWheelEvent):

Shared/efl/NativeWebMouseEventEfl.cpp:

(WebKit::NativeWebMouseEvent::NativeWebMouseEvent):

Shared/efl/NativeWebTouchEventEfl.cpp:

(WebKit::NativeWebTouchEvent::NativeWebTouchEvent):

Shared/efl/NativeWebWheelEventEfl.cpp:

(WebKit::NativeWebWheelEvent::NativeWebWheelEvent):

Shared/efl/WebEventFactory.cpp:

(WebKit::WebEventFactory::createWebMouseEvent):
(WebKit::WebEventFactory::createWebWheelEvent):
(WebKit::WebEventFactory::createWebTouchEvent):

Shared/efl/WebEventFactory.h:

(WebEventFactory):

UIProcess/API/efl/ewk_view.cpp:

(toDeviceScreenTransform):
(toWebContentTransform):
(_ewk_view_smart_mouse_wheel):
(_ewk_view_smart_mouse_down):
(_ewk_view_smart_mouse_up):
(_ewk_view_smart_mouse_move):
(ewk_view_feed_touch_event):

4:38 AM Changeset in webkit [133011] by keishi@webkit.org

8 edits in trunk

Calendar picker can flicker when opened from the suggestion picker
https://bugs.webkit.org/show_bug.cgi?id=100816

Reviewed by Kent Tamura.

Source/WebCore:

Calendar picker was opening before the hiding completed.

No new tests. Added test to date-suggestion-picker-key-operations.html.

Resources/pagepopups/pickerCommon.js: Added didHide event.
Resources/pagepopups/suggestionPicker.js:

LayoutTests:

fast/forms/resources/picker-common.js:

(openPicker): Fix mistake.

platform/chromium-win/platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations-expected.txt:
platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations-expected.txt:
platform/chromium/fast/forms/suggestion-picker/date-suggestion-picker-key-operations.html: Test that didHide event fires after the entry was selected and before the calendar picker opened.

4:33 AM Changeset in webkit [133010] by rakuco@webkit.org

2 edits in trunk/LayoutTests

[EFL] Unreviewed, unskip fast/forms/select-multiple-elements-with-mouse-drag.html.
https://bugs.webkit.org/show_bug.cgi?id=100248

The test is consistently passing again, and "fixing" the EFL side
is not trivial.

platform/efl-wk2/TestExpectations:

4:26 AM Changeset in webkit [133009] by tkent@chromium.org

2 edits in trunk/Source/WebKit/chromium

Remove code to hide/reshow text caret for PagePopups
https://bugs.webkit.org/show_bug.cgi?id=100834

Reviewed by Hajime Morita.

All of form controls supporting PagePopups don't have a text
caret. We don't need to hide a text caret on opening a PagePopup,
and don't need to reshow it on closing the PagePopup.

We had this code because an old UI of input[type=date] had a text
caret.

src/WebViewImpl.cpp:

(WebKit::WebViewImpl::openPagePopup):
(WebKit::WebViewImpl::closePagePopup):

4:03 AM Changeset in webkit [133008] by mkwst@chromium.org

1 edit in branches/chromium/1312/Source/WebKit/chromium/features.gypi

[Chromium] Unreviewed. Toggling ENABLE_CSP_NEXT flag off for M24.

BUG=158727

3:59 AM Changeset in webkit [133007] by allan.jensen@digia.com

2 edits in trunk/Source/WebCore

Reset binding test result after r132973
https://bugs.webkit.org/show_bug.cgi?id=99178

Unreviewed gardening.

bindings/scripts/test/JS/JSFloat64Array.cpp:

(WebCore::getJSFloat64ArrayConstructorTable):
(WebCore::JSFloat64ArrayConstructor::getOwnPropertySlot):
(WebCore::JSFloat64ArrayConstructor::getOwnPropertyDescriptor):

2:59 AM Changeset in webkit [133006] by mkwst@chromium.org

19 edits
2 adds in trunk

Script run from an isolated world should bypass a page's CSP.
https://bugs.webkit.org/show_bug.cgi?id=97398

Reviewed by Adam Barth.

Source/WebCore:

A page's Content Security Policy currently applies to all resources
loaded, regardless of their source. That generally makes sense, but
proves problematic when considering script run from an isolated
world (Chrome extensions, for instance). These scripts should be allowed
to inject resources into a page's DOM without eiher being restricted by
the page's active CSP, or generating violation reports that spam the
page owner.

Ideally, the isolated world could define its own Content Security Policy
which should be applied to resources it loads. For the moment, this
patch accepts a String that we can parse later on, but only uses it in
a binary way. If a non-empty policy String is provided, we bypass the
main world's CSP checks. If an empty String is provided, the main
world's CSP checks remain active.

Test: http/tests/security/isolatedWorld/bypass-main-world-csp.html

(WebCore::isolatedWorldContentSecurityPolicies):
(WebCore::DOMWrapperWorld::isolatedWorldHasContentSecurityPolicy):
(WebCore::DOMWrapperWorld::setIsolatedWorldContentSecurityPolicy):
(WebCore::DOMWrapperWorld::clearIsolatedWorldContentSecurityPolicy):

bindings/v8/DOMWrapperWorld.h:

(DOMWrapperWorld):

Mechanisms for setting and clearing Content Security Policies from
isolated worlds; implemented in the same HashMappy way as
SecurityOrigin.

bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::shouldBypassMainWorldContentSecurityPolicy):
(WebCore):

bindings/v8/ScriptController.h:

(ScriptController):

Adding a method to ScriptController to query the state of the
current world's Content Security Policy. We'll drop this once we can
apply a policy more directly, but for the moment it's necessary for
the next bit.

loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::canRequest):

Check the new ScriptController method, and only perform CSP checks
when loading resources if we're executing code from the main world,
or an isolated world with no Content Security Policy set.

Source/WebKit/chromium:

public/WebFrame.h:

(WebFrame):

src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::setIsolatedWorldSecurityOrigin):
(WebKit):
(WebKit::WebFrameImpl::setIsolatedWorldContentSecurityPolicy):

src/WebFrameImpl.h:

(WebFrameImpl):

Piping a Content Security Policy through WebFrame, in the same way
we're currently doing for SecurityOrigin. As a drive-by, this also
uses the static method on DOMWrapperWindow to set the security
origin, rather than routing through the frame's ScriptController.

Tools:

DumpRenderTree/chromium/DRTTestRunner.cpp:

(DRTTestRunner::DRTTestRunner):
(DRTTestRunner::setIsolatedWorldContentSecurityPolicy):

DumpRenderTree/chromium/DRTTestRunner.h:

(DRTTestRunner):

Adding a mechanism to set the Content Security Policy for an
isolated world to Chromium's testRunner.

LayoutTests:

http/tests/security/isolatedWorld/bypass-main-world-csp-expected.txt: Added.
http/tests/security/isolatedWorld/bypass-main-world-csp.html: Added.
platform/efl/TestExpectations:
platform/mac/TestExpectations:
platform/qt/TestExpectations:
platform/win/TestExpectations:

Skip the new tests on ports that don't support the new functionality

2:48 AM Changeset in webkit [133005] by podivilov@chromium.org

3 edits
1 move
1 add in trunk/LayoutTests

[chromium] Rebaseline media/video-zoom.html.

platform/chromium-linux/media/video-zoom-expected.png:
platform/chromium-mac-lion/media/video-zoom-expected.png: Added.
platform/chromium-mac-snowleopard/media/video-zoom-expected.png:
platform/efl-wk1/media/video-zoom-expected.png: Renamed from LayoutTests/platform/efl/media/video-zoom-expected.png.

2:44 AM Changeset in webkit [133004] by podivilov@chromium.org

13 edits
6 moves
9 adds in trunk/LayoutTests

[chromium] Rebaseline 7 media/video-* tests on Linux and Mac.

platform/chromium-linux/media/video-aspect-ratio-expected.png:
platform/chromium-linux/media/video-colorspace-yuv420-expected.png:
platform/chromium-linux/media/video-colorspace-yuv422-expected.png:
platform/chromium-linux/media/video-layer-crash-expected.png:
platform/chromium-linux/media/video-transformed-expected.png:
platform/chromium-linux/media/video-zoom-controls-expected.png:
platform/chromium-linux/platform/chromium/media/video-frame-size-change-expected.png:
platform/chromium-mac-lion/media/video-aspect-ratio-expected.png: Added.
platform/chromium-mac-lion/media/video-colorspace-yuv420-expected.png: Added.
platform/chromium-mac-lion/media/video-colorspace-yuv422-expected.png: Added.
platform/chromium-mac-lion/media/video-layer-crash-expected.png: Added.
platform/chromium-mac-lion/media/video-transformed-expected.png: Added.
platform/chromium-mac-lion/media/video-zoom-controls-expected.png: Added.
platform/chromium-mac-lion/platform/chromium/media/video-frame-size-change-expected.png: Added.
platform/chromium-mac-snowleopard/media/video-aspect-ratio-expected.png:
platform/chromium-mac-snowleopard/media/video-colorspace-yuv420-expected.png:
platform/chromium-mac-snowleopard/media/video-colorspace-yuv422-expected.png:
platform/chromium-mac-snowleopard/media/video-layer-crash-expected.png:
platform/chromium-mac-snowleopard/media/video-transformed-expected.png:
platform/efl-wk1/media/video-aspect-ratio-expected.png: Renamed from LayoutTests/platform/efl/media/video-aspect-ratio-expected.png.
platform/efl-wk1/media/video-colorspace-yuv420-expected.png: Renamed from LayoutTests/platform/efl/media/video-colorspace-yuv420-expected.png.
platform/efl-wk1/media/video-colorspace-yuv422-expected.png: Renamed from LayoutTests/platform/efl/media/video-colorspace-yuv422-expected.png.
platform/efl-wk1/media/video-layer-crash-expected.png: Renamed from LayoutTests/platform/efl/media/video-layer-crash-expected.png.
platform/efl-wk1/media/video-transformed-expected.png: Renamed from LayoutTests/platform/efl/media/video-transformed-expected.png.
platform/efl-wk1/media/video-zoom-controls-expected.png: Renamed from LayoutTests/platform/efl/media/video-zoom-controls-expected.png.

2:33 AM Changeset in webkit [133003] by podivilov@chromium.org

2 edits
1 move
2 adds in trunk/LayoutTests

[chromium] Add new baselines for fast/forms/basic-inputs.html and fast/forms/indeterminate.html on linux.

platform/chromium-linux/fast/forms/indeterminate-expected.png:
platform/efl-wk1/fast/forms/indeterminate-expected.png: Renamed from LayoutTests/platform/efl/fast/forms/indeterminate-expected.png.

2:28 AM Changeset in webkit [133002] by podivilov@chromium.org

2 edits in trunk/LayoutTests

2012-10-31 Pavel Podivilov <podivilov@google.com>

[chromium] Unreviewed, mark two touchadjustment tests as failing on mac.

platform/chromium/TestExpectations:

2:10 AM Changeset in webkit [133001] by commit-queue@webkit.org

2 edits in trunk/Source/WebKit2

[WK2] Change the scope of locking in CoreIPC::Connection class.
https://bugs.webkit.org/show_bug.cgi?id=98998

Patch by Byungwoo Lee <bw80.lee@samsung.com> on 2012-10-31
Reviewed by Laszlo Gombos.

Release the m_incomingMessagesLock and m_outgoingMessagesLock mutexes
immediately after the protected resource is no longer modified to
prevent a possible source of a deadlock.

The change allows the EFL port to prevent an actual dead-lock
situation with some upcoming changes.

Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::sendMessage):
(CoreIPC::Connection::enqueueIncomingMessage):

1:58 AM Changeset in webkit [133000] by commit-queue@webkit.org

13 edits
4 adds in trunk

[EFL][DRT] EventSender requires contextClick implementation
https://bugs.webkit.org/show_bug.cgi?id=86091

Patch by Michał Pakuła vel Rutka <Michał Pakuła vel Rutka> on 2012-10-31
Reviewed by Gyuyoung Kim.

Source/WebKit:

Add new unit test file, needed due to ewk API changes.

PlatformEfl.cmake:

Source/WebKit/efl:

Add a pointer to parent menu in Ewk_Context_Menu_Item and a getter
function to allow selecting a context menu item without having
parent menu pointer in DumpRenderTree EventSender.
Three unit tests added due ewk API changes.

ewk/ewk_contextmenu.cpp:

(_Ewk_Context_Menu_Item):
(ewk_context_menu_item_new):
(ewk_context_menu_item_parent_get):
(ewk_context_menu_item_append):

ewk/ewk_contextmenu.h:
ewk/ewk_view.cpp:

(ewk_view_context_menu_get):

ewk/ewk_view.h:
tests/test_ewk_contextmenu.cpp: Added.

(TEST_F):

tests/test_ewk_view.cpp:

(TEST_F):

Tools:

Add contextClick for DumpRenderTreeEventSender basing on WebKit GTK
implementation.

DumpRenderTree/efl/EventSender.cpp:

(getMenuItemTitleCallback):
(setMenuItemTitleCallback):
(menuItemClickCallback):
(contextClickCallback):

LayoutTests:

Move contextClick dependent tests to WebKit2 TestExpectations and
add new expected results.

platform/efl-wk2/TestExpectations:
platform/efl/TestExpectations:
platform/efl/editing/selection/5354455-2-expected.txt: Added.
platform/efl/fast/events/context-no-deselect-expected.txt: Added.
platform/efl/fast/events/context-onmousedown-event-expected.png: Added.

1:41 AM Changeset in webkit [132999] by timothy_horton@apple.com

2 edits in trunk/Source/WebCore

Unreviewed, attempt to fix the release build by not exporting symbols that aren't defined there.

WebCore.exp.in:

1:36 AM Changeset in webkit [132998] by yurys@chromium.org

7 edits in trunk/Source/WebCore

Memory instrumentation: do not report memory occupied by objects referenced using client interfaces
https://bugs.webkit.org/show_bug.cgi?id=100824

Reviewed by Alexander Pavlov.

In many classes we have pointer to a client object. These references should be
reported as weak pointers and the client's memory usage should be reported when
the client is reached from its instrumented owner where we know exact type of
the client. Otherwise we would need to add reportMemoryUsage to all client
interfaces which places additional burden on the client implementers and blurs
the instrumentation boundaries.

inspector/InspectorController.cpp:

(WebCore::InspectorController::reportMemoryUsage):

inspector/InspectorDebuggerAgent.cpp:

(WebCore::InspectorDebuggerAgent::reportMemoryUsage):

inspector/InspectorMemoryAgent.cpp:

(WebCore::InspectorMemoryAgent::reportMemoryUsage):

inspector/InspectorOverlay.cpp:

(WebCore::InspectorOverlay::reportMemoryUsage):

inspector/InspectorResourceAgent.cpp:

(WebCore::InspectorResourceAgent::reportMemoryUsage):

page/Page.cpp:

(WebCore::Page::reportMemoryUsage):

1:18 AM Changeset in webkit [132997] by timothy_horton@apple.com

2 edits in trunk/Source/WebCore

Unreviewed, attempt to fix the build by exporting NoExceptionAssertionChecker symbols.

WebCore.exp.in:

1:15 AM Changeset in webkit [132996] by timothy_horton@apple.com

3 edits in trunk/Source/WebKit2

Unreviewed, fix header sorting in r132992.

WebProcess/Plugins/PDF/PDFPluginAnnotation.mm:
WebProcess/Plugins/PDF/PDFPluginTextAnnotation.mm:

1:14 AM Changeset in webkit [132995] by eric@webkit.org

11 edits in trunk/Source/WebCore

Add non-virtual firstChild/lastChild overrides to RenderBlock and RenderTableCol for a > 30% speedup on table from bug 100304
https://bugs.webkit.org/show_bug.cgi?id=100306

Reviewed by Abhishek Arya.

Presumably this is a speedup for other rendering tests as well. We use firstChild() all over
the rendering code w/o considering that it makes a virtual function call.
Originally I just fixed the one callsite which was showing up on the sample to
use children()->firstChild() directly. However after further thought, this
broader solution seemed the better way to go. The first patch was a 15% win for
this large table, this broader solution was a 30%!? win.

The elephant in the room for this table is that we're walking the whole table
for many nextColumn calls. But that I will solve in a later bug.

rendering/RenderBlock.h:

(WebCore::RenderBlock::firstChild):
(WebCore::RenderBlock::lastChild):
(RenderBlock):

rendering/RenderObject.h:

(RenderObject):

rendering/RenderTableCol.h:

(WebCore::RenderTableCol::firstChild):
(WebCore::RenderTableCol::lastChild):
(RenderTableCol):

1:07 AM Changeset in webkit [132994] by timothy_horton@apple.com

2 edits in trunk/Source/WebKit2

Unreviewed, fix 32bit build breakage.

WebProcess/Plugins/PDF/PDFPluginAnnotation.mm:

(WebKit::PDFPluginAnnotation::updateGeometry):

12:54 AM Changeset in webkit [132993] by Philippe Normand

2 edits in trunk/Source/WebCore

Spatialized panning renders incorrectly with USE_CONCATENATED_IMPULSE_RESPONSES
https://bugs.webkit.org/show_bug.cgi?id=98294

Reviewed by Chris Rogers.

Regenerated the concatenated impulse responses file with
elevations in increasing order per azimuth. In the previous
version of this file the elevations for 315, 330 and 345 were
concatenated at the wrong indexes.

platform/audio/resources/Composite.wav:

12:53 AM Changeset in webkit [132992] by timothy_horton@apple.com

10 edits
7 adds in trunk/Source

Full-page PDFPlugin should support inline form editing
https://bugs.webkit.org/show_bug.cgi?id=100155

Reviewed by Dan Bernstein.

Add form controls corresponding to Text and Choice PDF annotations,
allowing for single- and multi-line text input, and <select>-like input.

Currently, forms are only implemented for full-page PDF documents.

Shared/mac/PDFKitImports.h:

(WebKit): Add pdfAnnotationTextWidgetClass and pdfAnnotationChoiceWidgetClass.

Shared/mac/PDFKitImports.mm:

(WebKit::pdfAnnotationTextWidgetClass): Add pdfAnnotationTextWidgetClass.
(WebKit::pdfAnnotationChoiceWidgetClass): Add pdfAnnotationChoiceWidgetClass.

WebKit2.xcodeproj/project.pbxproj: Add PDFLayerControllerDetails, PDFPluginTextAnnotation, PDFPluginChoiceAnnotation,

and PDFPluginAnnotation.

WebProcess/Plugins/PDF/PDFLayerControllerDetails.h: Added. Extract PDFLayerController details category into a header.
WebProcess/Plugins/PDF/PDFPlugin.h:

(PDFPlugin): Add m_activeAnnotation, to keep track of the active PDFPluginAnnotation, and m_annotationContainer,
the <div> that form controls are inserted into.

WebProcess/Plugins/PDF/PDFPlugin.mm:

(-[WKPDFLayerControllerDelegate pdfLayerController:didChangeActiveAnnotation:]): Forward active annotation changes to PDFPlugin.
(WebKit::PDFPlugin::PDFPlugin): Add a <div> to the PluginDocument <body>, which will contain <input>, <select>, and <textarea> elements
corresponding to the active annotation. This div is made to fit to the body, and is "overflow: hidden" so that form elements scrolled
out of the viewport do not cause the Document to expand to be larger than the Plugin.
(WebKit::PDFPlugin::pdfDocumentDidLoad): Update PDFPlugin's notion of the scroll position immediately after the document loads, in case
the scroll position is constrained by PDFKit.
(WebKit::PDFPlugin::destroy): Tear down any forms for the active annotation.
(WebKit::PDFPlugin::geometryDidChange): Notify the current annotation that it should update its geometry when the Plugin's geometry changes.
(WebKit::PDFPlugin::setScrollOffset): Notify the current annotation that it should update its geometry when we scroll.
(WebKit::PDFPlugin::setActiveAnnotation): Commit changes to the current annotation, if there is one, then make a new PDFPluginAnnotation
corresponding to the newly active annotation, and attach it to our document.

WebProcess/Plugins/PDF/PDFPluginAnnotation.h: Added.

(WebKit::PDFPluginAnnotation::element): Return the Element corresponding to this object.
(WebKit::PDFPluginAnnotation::annotation): Return the PDFAnnotation corresponding to this object.
(WebKit::PDFPluginAnnotation::plugin): Return the Plugin that created this object.
(WebKit::PDFPluginAnnotation::commit): Commit changes from the form to the PDFAnnotation. This is implemented in subclasses.
(WebKit::PDFPluginAnnotation::attach): Attach a PDFPluginAnnotation to its parent Element.
(WebKit::PDFPluginAnnotation::parent): Return the parent Element that we're attach()ed to.
(WebKit::PDFPluginAnnotation::pdfLayerController): Return the pdfLayerController that the PDFAnnotation comes from.
(PDFPluginAnnotationEventListener): Add an event listener to respond to "change" and "blur" events from m_element.

WebProcess/Plugins/PDF/PDFPluginAnnotation.mm: Added.

(WebKit::PDFPluginAnnotation::create): Create the appropriate PDFPluginAnnotation subclass (PDFPluginTextAnnotation or
PDFPluginChoiceAnnotation based on the class of the PDFAnnotation).
(WebKit::PDFPluginAnnotation::attach): Lazily create the HTML form element corresponding to this annotation. Add it to its parent
element. Add event listeners for "change" and "blur".
(WebKit::PDFPluginAnnotation::~PDFPluginAnnotation): Tear down the annotation, by removing event listeners and removing
the form element from its parent element.
(WebKit::PDFPluginAnnotation::updateGeometry): Update the size and position of the form element, given the current PDF scale and scrolling position.
(WebKit::PDFPluginAnnotation::PDFPluginAnnotationEventListener::handleEvent): If we get a blur or change event, we should remove the
form control to match the behavior of PDFKit.

WebProcess/Plugins/PDF/PDFPluginChoiceAnnotation.h: Added.

(WebKit::PDFPluginChoiceAnnotation::choiceAnnotation): Return annotation() with the appropriate cast.

WebProcess/Plugins/PDF/PDFPluginChoiceAnnotation.mm: Added.

(WebKit::PDFPluginChoiceAnnotation::updateGeometry): Update the font size of the choice widget given the current PDF scale.
(WebKit::PDFPluginChoiceAnnotation::commit): Save the value of the currently selected <option> into the PDFAnnotationChoiceWidget.
(WebKit::PDFPluginChoiceAnnotation::createAnnotationElement): Create a <select> element, populated with <option> elements
for the PDFAnnotationChoiceWidget's available choices. Select the correct <option> initially.

WebProcess/Plugins/PDF/PDFPluginTextAnnotation.h: Added.

(WebKit::PDFPluginTextAnnotation::textAnnotation): Return annotation() with the appropriate cast.

WebProcess/Plugins/PDF/PDFPluginTextAnnotation.mm: Added.

(WebKit::cssAlignmentValueForNSTextAlignment): Return the CSS alignment value for a given NSTextAlignment.
(WebKit::PDFPluginTextAnnotation::createAnnotationElement): Create a <textarea> or <input> element, depending on whether the
PDFAnnotationTextWidget supports multiline editing or not.
(WebKit::PDFPluginTextAnnotation::updateGeometry): Update the font size of the text widget given the current PDF scale.
(WebKit::PDFPluginTextAnnotation::commit): Save the current string in the form element into the PDFAnnotationTextWidget.

WebProcess/WebPage/WebPage.cpp:

(WebKit::focusedPluginViewForFrame): Return the PluginView for a Frame only if the Plugin's element is focused.
(WebKit::WebPage::executeEditingCommand): Only forward editing commands to PluginView if the plugin itself is focused.
(WebKit::WebPage::isEditingCommandEnabled): Ditto.
(WebKit::WebPage::validateCommand): Ditto.

WebCore.exp.in: Expose a variety of additional WebCore functionality to WebKit2.
WebCore.xcodeproj/project.pbxproj: Expose some more WebCore headers as private headers for use in WebKit2.

Note: See TracTimeline for information about the timeline view.

Download in other formats:

RSS Feed