WebKit-https.git
7 years agoAnother unreviewed build fix for Mac.
alexis.menard@openbossa.org [Tue, 24 Apr 2012 13:50:57 +0000 (13:50 +0000)]
Another unreviewed build fix for Mac.

* css/CSSStyleSelector.h:
(CSSStyleSelector):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115047 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[EFL] Enable Web Timing
commit-queue@webkit.org [Tue, 24 Apr 2012 13:48:27 +0000 (13:48 +0000)]
[EFL] Enable Web Timing
https://bugs.webkit.org/show_bug.cgi?id=84705

Patch by Christophe Dumez <christophe.dumez@intel.com> on 2012-04-24
Reviewed by Tony Gentilcore.

.:

Enable Web Timing in EFL port for for performance profiling and
improvement.

* Source/cmake/OptionsEfl.cmake:

Tools:

Enable web timing by default whenever build-webkit is used.

* Scripts/build-webkit:

LayoutTests:

Unskip the test cases that require Web Timing now that it is enabled
by default in EFL port.

* platform/efl/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115046 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed build fix attempt for Mac.
alexis.menard@openbossa.org [Tue, 24 Apr 2012 13:44:08 +0000 (13:44 +0000)]
Unreviewed build fix attempt for Mac.

* rendering/style/RenderStyle.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115045 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 13:28:10 +0000 (13:28 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=65063

Unmark as failing on Mac 10.6 css2.1/t0905-c414-flt-01-d-g.html.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115044 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoRename CSSStyleApplyProperty class to StyleBuilder.
alexis.menard@openbossa.org [Tue, 24 Apr 2012 13:28:06 +0000 (13:28 +0000)]
Rename CSSStyleApplyProperty class to StyleBuilder.
https://bugs.webkit.org/show_bug.cgi?id=84711

Reviewed by Antti Koivisto.

As per discussion on bug https://bugs.webkit.org/show_bug.cgi?id=42764
the name is a bit confusing. This first patch rename the class and a
following one will rename the files. We also drop the CSS prefix of the
class to keep CSS prefixed classes for CSSOM types.

No new tests : renaming, no behavior changes should appear.

* css/CSSStyleApplyProperty.cpp:
(WebCore::ApplyPropertyExpanding::applyInheritValue):
(WebCore::ApplyPropertyExpanding::applyInitialValue):
(WebCore::ApplyPropertyExpanding::applyValue):
(WebCore::StyleBuilder::sharedStyleBuilder):
(WebCore::StyleBuilder::StyleBuilder):
* css/CSSStyleApplyProperty.h:
(WebCore):
(StyleBuilder):
* css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::CSSStyleSelector):
(WebCore::CSSStyleSelector::collectMatchingRulesForList):
* css/CSSStyleSelector.h:
(CSSStyleSelector):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115043 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 13:21:03 +0000 (13:21 +0000)]
[Chromium] Unreviewed test expectations update.

Unmark as failing on Mac 10.5 the following tests:
    fast/backgrounds/size/backgroundSize15.html
    fast/events/reveal-link-when-focused.html
    fast/frames/content-opacity-1.html
    fast/frames/content-opacity-2.html
    fast/xsl/xslt-extra-content-at-end.xml
    printing/return-from-printing-mode.html

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115042 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 13:03:41 +0000 (13:03 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=82628

Unmark svg/hittest/svg-ellipse-non-scale-stroke.xhtml -- passes for a long time.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115041 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 12:56:48 +0000 (12:56 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=75161

Unmark media/video-poster-blocked-by-willsendrequest.html -- fails no more after FFmpeg roll 132717:133551 in Chromium.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115040 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 12:49:43 +0000 (12:49 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84714

Mark compositing/geometry/empty-embed-rects.html as flaky on Mac 10.6.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115039 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoMerge LocalizedCalendar into LocalizedDate
tkent@chromium.org [Tue, 24 Apr 2012 12:49:03 +0000 (12:49 +0000)]
Merge LocalizedCalendar into LocalizedDate
https://bugs.webkit.org/show_bug.cgi?id=84685

Reviewed by Kentaro Hara.

The functionality of LocalizedCalendar is very related to
LocalizedDate. We should merge them.

No new tests. Just refactoring.

* WebCore.gypi: Remove LocalizedCalendar.h and LocalizedCalenderICU.cpp.
* html/shadow/CalendarPickerElement.cpp:
Include LocalizedDate.h instead of LocalizedCalendar.h.
* platform/text/LocalizedCalendar.h:
Removed. Merged into LocalizedDate.h.
* platform/text/LocalizedCalendarICU.cpp:
Removed. Merged into LocalizedDateICU.h.
* platform/text/LocalizedDate.h:
(WebCore): Move some functions from LocalizedCalendar.h.
* platform/text/LocalizedDateICU.cpp:
Move some functions from LocalizedCalendarICU.cpp.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115038 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 12:41:59 +0000 (12:41 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84712

Mark http/tests/inspector/network/x-frame-options-deny.html as flaky on Vista.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115037 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, fix some more GTK baselines after r114836.
philn@webkit.org [Tue, 24 Apr 2012 12:40:30 +0000 (12:40 +0000)]
Unreviewed, fix some more GTK baselines after r114836.

* platform/gtk/fast/dom/horizontal-scrollbar-in-rtl-expected.txt:
* platform/gtk/fast/dom/importNodeHTML-expected.txt:
* platform/gtk/fast/dom/importNodeXML-expected.txt:
* platform/gtk/fast/dom/inner-text-expected.txt:
* platform/gtk/fast/dom/isindex-001-expected.txt:
* platform/gtk/fast/dom/isindex-002-expected.txt:
* platform/gtk/fast/dom/outerText-expected.txt:
* platform/gtk/fast/dom/prototype-inheritance-2-expected.txt:
* platform/gtk/fast/dom/row-inner-text-expected.txt:
* platform/gtk/fast/dom/rtl-scroll-to-leftmost-and-resize-expected.txt:
* platform/gtk/fast/dom/scroll-reveal-left-overflow-expected.txt:
* platform/gtk/fast/dom/scroll-reveal-top-overflow-expected.txt:
* platform/gtk/fast/dom/tabindex-clamp-expected.txt:
* platform/gtk/fast/dom/vertical-scrollbar-in-rtl-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115036 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Gardening. Skipped a test because JSC doesn't support heap profiling.
ossy@webkit.org [Tue, 24 Apr 2012 12:39:33 +0000 (12:39 +0000)]
[Qt] Gardening. Skipped a test because JSC doesn't support heap profiling.

Patch by János Badics <jbadics@inf.u-szeged.hu> on 2012-04-24
Reviewed by Csaba Osztrogonác.

* platform/qt/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115035 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[EFL] [DRT] Enable passing sputnik tests
commit-queue@webkit.org [Tue, 24 Apr 2012 12:31:51 +0000 (12:31 +0000)]
[EFL] [DRT] Enable passing sputnik tests
https://bugs.webkit.org/show_bug.cgi?id=84600

Unreviewed, unskip sputnik tests which are passing now.

Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-04-24

* platform/efl/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115034 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[EFL] Move tests that require link prefetching to test expectations
commit-queue@webkit.org [Tue, 24 Apr 2012 11:33:22 +0000 (11:33 +0000)]
[EFL] Move tests that require link prefetching to test expectations
https://bugs.webkit.org/show_bug.cgi?id=84673

Unreviewed, test expectations update.

Move tests that require link prefetching to test expectations since it
is disabled by default.

Patch by Christophe Dumez <christophe.dumez@intel.com> on 2012-04-24

* platform/efl/Skipped:
* platform/efl/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115033 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoWeb Inspector: remove pure delegation code from DebuggerPresentationModel
pfeldman@chromium.org [Tue, 24 Apr 2012 11:24:32 +0000 (11:24 +0000)]
Web Inspector: remove pure delegation code from DebuggerPresentationModel
https://bugs.webkit.org/show_bug.cgi?id=84704

Reviewed by Yury Semikhatsky.

Source/WebCore:

This change removes the code that delegates calls to breakpoint manager and exposes the breakpoint
manager in debugger presentation model itself. It allows narrowing dpm dependencies.

* inspector/front-end/BreakpointsSidebarPane.js:
(WebInspector.JavaScriptBreakpointsSidebarPane):
(WebInspector.JavaScriptBreakpointsSidebarPane.prototype._breakpointCheckboxClicked):
(WebInspector.JavaScriptBreakpointsSidebarPane.prototype._breakpointContextMenu):
(WebInspector.JavaScriptBreakpointsSidebarPane.prototype._contextMenu):
* inspector/front-end/DebuggerModel.js:
(WebInspector.DebuggerModel.prototype.debuggerPausedDetails):
(WebInspector.DebuggerModel.prototype.createRawLocationByURL):
(WebInspector.DebuggerModel.prototype.isPaused):
* inspector/front-end/DebuggerPresentationModel.js:
(WebInspector.DebuggerPresentationModel.prototype.get breakpointManager):
(WebInspector.DebuggerPresentationModel.prototype._debuggerPaused):
(WebInspector.DebuggerPresentationModelResourceBinding.prototype._setContentWithInitialContent):
* inspector/front-end/JavaScriptSource.js:
(WebInspector.JavaScriptSource):
(WebInspector.JavaScriptSource.prototype.consoleMessagesCleared):
(WebInspector.JavaScriptSource.prototype.findBreakpoint):
(WebInspector.JavaScriptSource.prototype.setBreakpoint):
(WebInspector.JavaScriptSource.prototype.setBreakpointEnabled):
(WebInspector.JavaScriptSource.prototype.removeBreakpoint):
(WebInspector.JavaScriptSource.prototype.updateBreakpoint):
(WebInspector.JavaScriptSource.prototype.continueToLine):
(WebInspector.JavaScriptSource.prototype.canSetContent):
(WebInspector.JavaScriptSource.prototype.setContent):
(WebInspector.JavaScriptSource.prototype.updateBreakpointsAfterLiveEdit):
* inspector/front-end/JavaScriptSourceFrame.js:
(WebInspector.JavaScriptSourceFrame.prototype.canEditSource):
(WebInspector.JavaScriptSourceFrame.prototype.editContent):
(WebInspector.JavaScriptSourceFrame.prototype._onContentChanged):
(WebInspector.JavaScriptSourceFrame.prototype.populateLineGutterContextMenu.else.editBreakpointCondition.didEditBreakpointCondition):
(WebInspector.JavaScriptSourceFrame.prototype.populateLineGutterContextMenu.else):
(WebInspector.JavaScriptSourceFrame.prototype.populateLineGutterContextMenu):
(WebInspector.JavaScriptSourceFrame.prototype.beforeTextChanged):
(WebInspector.JavaScriptSourceFrame.prototype._getPopoverAnchor):
(WebInspector.JavaScriptSourceFrame.prototype._resolveObjectForPopover.showObjectPopover):
(WebInspector.JavaScriptSourceFrame.prototype._resolveObjectForPopover):
(WebInspector.JavaScriptSourceFrame.prototype._setBreakpoint):
(WebInspector.JavaScriptSourceFrame.prototype._onMouseDown):
* inspector/front-end/ScopeChainSidebarPane.js:
(WebInspector.ScopeChainSidebarPane.prototype.update):
* inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._debuggerPaused.else.didGetUILocation):
(WebInspector.ScriptsPanel.prototype._debuggerPaused):

LayoutTests:

* http/tests/inspector/debugger-test.js:
(initialize_DebuggerTest):
* inspector/debugger/pause-in-inline-script.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115032 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoWeb Inspector: native nodes may have snapshot id less than base snapshot max JS object id
yurys@chromium.org [Tue, 24 Apr 2012 11:23:22 +0000 (11:23 +0000)]
Web Inspector: native nodes may have snapshot id less than base snapshot max JS object id
https://bugs.webkit.org/show_bug.cgi?id=84681

Make sure new nodes whose IDs less than base snapshot max JS object id will
get into snapshot delta.

Reviewed by Pavel Feldman.

Source/WebCore:

Test: inspector/profiler/heap-snapshot-comparison-dom-groups-change.html

* inspector/front-end/HeapSnapshot.js:
(WebInspector.HeapSnapshot.prototype._calculateDiffForClass):
* inspector/front-end/HeapSnapshotDataGrids.js:
(WebInspector.HeapSnapshotSortableDataGrid): Drive-by fix, made recursion counter
a normal field on HeapSnapshotSortableDataGrid.
(WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingEnter):
(WebInspector.HeapSnapshotSortableDataGrid.prototype.recursiveSortingLeave):

LayoutTests:

* inspector/profiler/heap-snapshot-comparison-dom-groups-change-expected.txt: Added.
* inspector/profiler/heap-snapshot-comparison-dom-groups-change.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115031 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, unflag tests passing after r114796.
philn@webkit.org [Tue, 24 Apr 2012 11:08:25 +0000 (11:08 +0000)]
Unreviewed, unflag tests passing after r114796.

* platform/gtk/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115030 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, rolling out r115000.
commit-queue@webkit.org [Tue, 24 Apr 2012 11:03:16 +0000 (11:03 +0000)]
Unreviewed, rolling out r115000.
http://trac.webkit.org/changeset/115000
https://bugs.webkit.org/show_bug.cgi?id=84703

Breaks webkit_unit_tests on Mac in Chromium:
http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6/builds/15157
(Requested by mnaganov on #webkit).

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

Source/WebCore:

* platform/graphics/skia/OpaqueRegionSkia.cpp:
(WebCore::paintIsOpaque):
(WebCore::OpaqueRegionSkia::popCanvasLayer):
(WebCore::OpaqueRegionSkia::didDraw):
(WebCore::OpaqueRegionSkia::didDrawUnbounded):
(WebCore::OpaqueRegionSkia::markRectAsOpaque):
(WebCore::OpaqueRegionSkia::markRectAsNonOpaque):
* platform/graphics/skia/OpaqueRegionSkia.h:
(OpaqueRegionSkia):
(WebCore::OpaqueRegionSkia::CanvasLayerState::CanvasLayerState):
(CanvasLayerState):
* platform/graphics/skia/PlatformContextSkia.cpp:
(WebCore::PlatformContextSkia::restoreLayer):

Source/WebKit/chromium:

* tests/PlatformContextSkiaTest.cpp:
(WebCore):
(WebCore::TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115029 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, GTK test_expectations update.
philn@webkit.org [Tue, 24 Apr 2012 10:53:08 +0000 (10:53 +0000)]
Unreviewed, GTK test_expectations update.

* platform/gtk/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115028 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 10:43:27 +0000 (10:43 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=79454

Mark svg/text/text-rescale.html as flaky on release Linux as well.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115027 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 10:36:51 +0000 (10:36 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84696

Mark fast/workers/storage/interrupt-database.html as flaky on Mac 10.6

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115026 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 10:34:26 +0000 (10:34 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84698

Mark fast/forms/placeholder-position.html as flaky on Linux

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115025 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Gardening
ossy@webkit.org [Tue, 24 Apr 2012 10:18:16 +0000 (10:18 +0000)]
[Qt] Gardening

Patch by Szilard Ledan <szledan@inf.u-szeged.hu> on 2012-04-24
Reviewed by Csaba Osztrogonác.

* platform/qt/Skipped: Skip http/tests/notifications/icon-exists-cancel.html, because it fails from r115011.
* platform/qt/fast/forms/form-collection-radio-node-list-expected.txt: Added after r115009.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115024 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, GTK test_expectations update, mark some tests as
philn@webkit.org [Tue, 24 Apr 2012 10:17:17 +0000 (10:17 +0000)]
Unreviewed, GTK test_expectations update, mark some tests as
failing and remove wrong baselines.

* platform/gtk/fast/events/attempt-scroll-with-no-scrollbars-expected.txt: Removed.
* platform/gtk/fast/events/mouse-relative-position-expected.txt: Removed.
* platform/gtk/fast/events/panScroll-click-hyperlink-expected.txt: Removed.
* platform/gtk/fast/events/selectstart-by-double-triple-clicks-expected.txt: Removed.
* platform/gtk/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115023 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Restore x-position when restoring previous zoom-level.
commit-queue@webkit.org [Tue, 24 Apr 2012 09:41:31 +0000 (09:41 +0000)]
[Qt] Restore x-position when restoring previous zoom-level.
https://bugs.webkit.org/show_bug.cgi?id=84591

Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-04-24
Reviewed by Simon Hausmann.

Tap-to-zoom fits content to width, therefore it is essentional that
the content is refitted to width when zooming out.

This patch pairs horizontal position and scale on the zoom-out stack.

Additionally it fixes a problem with not detecting zoom-out properly
due to floating point errors.

* UIProcess/qt/QtViewportInteractionEngine.cpp:
(WebKit::fuzzyCompare):
(WebKit::QtViewportInteractionEngine::zoomToAreaGestureEnded):
* UIProcess/qt/QtViewportInteractionEngine.h:
(WebKit::QtViewportInteractionEngine::ScaleStackItem::ScaleStackItem):
(QtViewportInteractionEngine):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115022 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 09:40:48 +0000 (09:40 +0000)]
[Chromium] Unreviewed test expectations update.

Fix lint error in the test expectations file.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115021 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, re-add the GTK baselines that were removed during last
philn@webkit.org [Tue, 24 Apr 2012 09:38:57 +0000 (09:38 +0000)]
Unreviewed, re-add the GTK baselines that were removed during last
rebaseline of doom (like r114836 and likely other commits).

* platform/gtk/: Fix empty baselines...
* platform/gtk/editing/selection/5195166-1-expected.txt: Added.
* platform/gtk/fast/html/details-marker-style-expected.txt: Added.
* platform/gtk/fast/images/support-broken-image-delegate-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115020 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 09:34:48 +0000 (09:34 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84689

Mark fast/images/embed-does-not-propagate-dimensions-to-object-ancestor.html as flaky on Mac 10.6

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115019 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[EFL] move tests that require LayoutTestController's layerTreeAsText to test expectations
commit-queue@webkit.org [Tue, 24 Apr 2012 09:32:06 +0000 (09:32 +0000)]
[EFL] move tests that require LayoutTestController's layerTreeAsText to test expectations
https://bugs.webkit.org/show_bug.cgi?id=84671

Unreviewed, test expectations update.

Moving test cases which require layerTreeAsText from Skipped list to
test expectations.

Patch by Christophe Dumez <christophe.dumez@intel.com> on 2012-04-24

* platform/efl/Skipped:
* platform/efl/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115018 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 09:28:02 +0000 (09:28 +0000)]
[Chromium] Unreviewed test expectations update.
https://bugs.webkit.org/show_bug.cgi?id=84688

Mark compositing/animation/computed-style-during-delay.html as flaky on Mac 10.6

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115017 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoConsolidate external stylesheet quirks handling to StyleSheetInternal
antti@apple.com [Tue, 24 Apr 2012 09:17:18 +0000 (09:17 +0000)]
Consolidate external stylesheet quirks handling to StyleSheetInternal
https://bugs.webkit.org/show_bug.cgi?id=84632

Reviewed by Andreas Kling.

Move the repeated code from StyleImportRule and HTMLLinkElement to StyleSheetInternal::parseUserStyleSheet.

Remove a BUILDING_ON_LEOPARD-only quirk.

* css/CSSImportRule.cpp:
(WebCore::StyleRuleImport::setCSSStyleSheet):
* css/CSSParser.cpp:
(WebCore):
(WebCore::CSSParserContext::CSSParserContext):

    Add new settings bits to the CSSParserContext. These are used for external stylesheet parsing quirks.

* css/CSSParserMode.h:
(CSSParserContext):
* css/CSSStyleSheet.cpp:
(WebCore::StyleSheetInternal::parseUserStyleSheet):
(WebCore::StyleSheetInternal::parseString):
(WebCore):
(WebCore::StyleSheetInternal::parseStringAtLine):
* css/CSSStyleSheet.h:
(WebCore):
(StyleSheetInternal):
* html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::setCSSStyleSheet):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115016 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] All of <datalist> tests are passing.
tkent@chromium.org [Tue, 24 Apr 2012 09:16:50 +0000 (09:16 +0000)]
[Chromium] All of <datalist> tests are passing.

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115015 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Chromium] Unreviewed test expectations update.
mnaganov@chromium.org [Tue, 24 Apr 2012 09:09:35 +0000 (09:09 +0000)]
[Chromium] Unreviewed test expectations update.

Mark certain media tests after FFmpeg roll in Chromium:
    media/media-controller-playback.html
    media/track/track-cues-missed.html
    media/track/track-cues-pause-on-exit.html
    media/track/track-cues-sorted-before-dispatch.html
    media/video-colorspace-yuv422.html
    media/video-currentTime-delay.html

* platform/chromium/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115014 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, GTK test_expectations and another media rebaseline
philn@webkit.org [Tue, 24 Apr 2012 09:06:24 +0000 (09:06 +0000)]
Unreviewed, GTK test_expectations and another media rebaseline
after r114957.

* platform/gtk/fast/forms/large-toggle-elements-expected.txt:
* platform/gtk/fast/frames/scrolling-iframe-out-of-viewport-expected.txt:
* platform/gtk/media/audio-controls-rendering-expected.txt:
* platform/gtk/media/audio-repaint-expected.txt:
* platform/gtk/media/controls-strict-expected.txt:
* platform/gtk/media/controls-styling-expected.txt:
* platform/gtk/media/controls-without-preload-expected.txt:
* platform/gtk/media/media-controls-clone-expected.txt:
* platform/gtk/media/media-document-audio-repaint-expected.txt:
* platform/gtk/media/video-controls-rendering-expected.txt:
* platform/gtk/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115013 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Incomplete interaction-engine reset.
commit-queue@webkit.org [Tue, 24 Apr 2012 08:52:45 +0000 (08:52 +0000)]
[Qt] Incomplete interaction-engine reset.
https://bugs.webkit.org/show_bug.cgi?id=84594

Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-04-24
Reviewed by Simon Hausmann.

Reset a few values forgotten in the reset function.

* UIProcess/qt/QtViewportInteractionEngine.cpp:
(WebKit::QtViewportInteractionEngine::reset):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115012 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Move notification icon download out of WebCore
yael.aharon@nokia.com [Tue, 24 Apr 2012 08:35:44 +0000 (08:35 +0000)]
[Qt] Move notification icon download out of WebCore
https://bugs.webkit.org/show_bug.cgi?id=80700

Reviewed by Simon Hausmann.

Source/WebCore:

Remove the code that downloads the notification's icon out of WebCore.
Other ports let the client decide if to download the icon or not.
No new tests. This patch is only removing code.

* notifications/Notification.cpp:
(WebCore::Notification::~Notification):
(WebCore::Notification::show):
(WebCore::Notification::close):
(WebCore::Notification::contextDestroyed):
* notifications/Notification.h:
(Notification):

Source/WebKit/qt:

Remove the code that handles the notification's icon and use the icon url instead.
The "display" event has to be asynchronous, so a timer was added to achieve that.
The platform plugin API and example were updated to reflect this change.
This change does not address the recent changes in web notifications spec.

* Api/qwebkitplatformplugin.h:
(QWebNotificationData):
* WebCoreSupport/NotificationPresenterClientQt.cpp:
(WebCore::NotificationWrapper::NotificationWrapper):
(WebCore::NotificationWrapper::sendDisplayEvent):
(WebCore):
(WebCore::NotificationWrapper::iconUrl):
(WebCore::NotificationPresenterClientQt::show):
(WebCore::NotificationPresenterClientQt::displayNotification):
(WebCore::NotificationPresenterClientQt::sendDisplayEvent):
* WebCoreSupport/NotificationPresenterClientQt.h:
(NotificationWrapper):
(NotificationPresenterClientQt):
* examples/platformplugin/WebNotificationPresenter.cpp:
(WebNotificationWidget::showNotification):
* examples/platformplugin/qwebkitplatformplugin.h:
(QWebNotificationData):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115011 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Unreviewed gardening. Skip failing tests because of new regressions.
ossy@webkit.org [Tue, 24 Apr 2012 08:34:25 +0000 (08:34 +0000)]
[Qt] Unreviewed gardening. Skip failing tests because of new regressions.

* platform/qt-5.0-wk2/Skipped:
* platform/qt-5.0/Skipped:
* platform/qt/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115010 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoRadioNodeList support in HTMLFormElement::elements
commit-queue@webkit.org [Tue, 24 Apr 2012 08:11:32 +0000 (08:11 +0000)]
RadioNodeList support in HTMLFormElement::elements
https://bugs.webkit.org/show_bug.cgi?id=81854

Source/WebCore:

Implement RadioNodeList support spec'ed at
http://www.whatwg.org/specs/web-apps/current-work/multipage/common-dom-interfaces.html#radionodelist

Patch by Rakesh KN <rakesh.kn@motorola.com> on 2012-04-24
Reviewed by Ryosuke Niwa.

Test: fast/forms/form-collection-radio-node-list.html

* CMakeLists.txt:
Added entries for new files.
* DerivedSources.cpp: Ditto.
* DerivedSources.make: Ditto.
* DerivedSources.pri: Ditto.
* GNUmakefile.list.am: Ditto.
* Target.pri: Ditto.
* WebCore.gypi: Ditto.
* WebCore.vcproj/WebCore.vcproj: Ditto.
* WebCore.xcodeproj/project.pbxproj: Ditto.
* bindings/js/JSHTMLCollectionCustom.cpp:
(WebCore::getNamedItems):
Modified to create RadioNodeList object when FormControlCollection has more than
one element of same name/id.
* bindings/scripts/CodeGeneratorJS.pm:
(GenerateImplementation):
Added code to include Node.h and JSNode.h in JSRadioNodeElement.cpp.
* bindings/v8/custom/V8HTMLCollectionCustom.cpp:
(WebCore::getNamedItems):
Modified to create RadioNodeList object when FormControlCollection has more than
one element of same name/id.
* html/CollectionType.h:
Added new FormControls type.
* html/HTMLCollection.cpp:
(WebCore::HTMLCollection::shouldIncludeChildren):
(WebCore::HTMLCollection::isAcceptableElement):
Handle FormControls collection type.
* html/HTMLFormCollection.cpp:
(WebCore::HTMLFormCollection::HTMLFormCollection):
Contruct collection of FormControls type.
* html/RadioNodeList.cpp: Added.
(WebCore):
(WebCore::RadioNodeList::RadioNodeList):
(WebCore::RadioNodeList::~RadioNodeList):
(WebCore::toRadioButtonInputElement):
(WebCore::RadioNodeList::value):
(WebCore::RadioNodeList::setValue):
(WebCore::RadioNodeList::nodeMatches):
* html/RadioNodeList.h: Added.
(WebCore):
(RadioNodeList):
(WebCore::RadioNodeList::create):
RadioNodeList implementation.
* html/RadioNodeList.idl: Added.
Idl for generating RadioNodeList JS/V8 bindings.

LayoutTests:

Patch by Rakesh KN <rakesh.kn@motorola.com> on 2012-04-24
Reviewed by Ryosuke Niwa.

* fast/forms/form-collection-radio-node-list-expected.txt: Added.
* fast/forms/form-collection-radio-node-list.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115009 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, GTK test_expectations update.
philn@webkit.org [Tue, 24 Apr 2012 08:02:28 +0000 (08:02 +0000)]
Unreviewed, GTK test_expectations update.

* platform/gtk/test_expectations.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115008 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoImplement the field cache of ObjcClass in WebKit types
commit-queue@webkit.org [Tue, 24 Apr 2012 07:48:20 +0000 (07:48 +0000)]
Implement the field cache of ObjcClass in WebKit types
https://bugs.webkit.org/show_bug.cgi?id=84667

Patch by Benjamin Poulain <bpoulain@apple.com> on 2012-04-24
Reviewed by Geoffrey Garen.

Every time we get a field through the Objective-C bridge, we waste some time converting
the incoming string to an CFString.

This patch implement the field cache based on WTF types so that we can return directly
in case of a positive match.

When we do not have a match on the Identifier, we allocate the memory and find the valid field
as before.

* bridge/objc/objc_class.h:
(ObjcClass):
* bridge/objc/objc_class.mm:
(JSC::Bindings::deleteMethod):
(Bindings):
(JSC::Bindings::ObjcClass::ObjcClass):
(JSC::Bindings::ObjcClass::fieldNamed):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115007 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, GTK rebaseline after r114957.
philn@webkit.org [Tue, 24 Apr 2012 07:47:13 +0000 (07:47 +0000)]
Unreviewed, GTK rebaseline after r114957.

* platform/gtk/fast/hidpi/video-controls-in-hidpi-expected.txt:
* platform/gtk/media/video-empty-source-expected.txt:
* platform/gtk/media/video-no-audio-expected.txt:
* platform/gtk/media/video-volume-slider-expected.txt:
* platform/gtk/media/video-zoom-controls-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115006 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoIncremental rendering should be unsuppressed after the load event or a configurable...
aestes@apple.com [Tue, 24 Apr 2012 07:43:40 +0000 (07:43 +0000)]
Incremental rendering should be unsuppressed after the load event or a configurable timeout, whichever occurs first.
https://bugs.webkit.org/show_bug.cgi?id=84583

Source/WebCore:

Reviewed by Anders Carlsson.

In high-latency situations, having incremental rendering suppression
enabled makes for a worse experience due to the excessively long time
it takes for the load event to fire. So that clients that opt in to
incremental rendering suppression can mitigate this, add a configurable
timeout in which rendering is re-enabled if the load event has yet to
fire.

* dom/Document.cpp:
(WebCore::Document::Document):
(WebCore::Document::setReadyState): When the document transitions to
the loading state, start a timer; when it transitions to the completed
state, allow visual updates.
(WebCore::Document::setVisualUpdatesAllowed): If visual updates are
allowed, force a repaint. Otherwise, set a flag that suppresses
rendering.
(WebCore::Document::visualUpdatesSuppressionTimerFired): When the timer
fires, allow visual updates.
(WebCore::Document::implicitClose): Remove code that is now called in
setVisualUpdatesAllowed().
* dom/Document.h:
(WebCore::Document::visualUpdatesAllowed):
* page/Settings.cpp:
(WebCore::Settings::Settings):
* page/Settings.h:
(WebCore::Settings::setIncrementalRenderingSuppressionTimeoutInSeconds):
(WebCore::Settings::incrementalRenderingSuppressionTimeoutInSeconds):

Source/WebKit/mac:

Add a private WebPreference for setting the incremental rendering suppression timeout.

Reviewed by Anders Carlsson.

* WebView/WebPreferenceKeysPrivate.h:
* WebView/WebPreferences.mm:
(-[WebPreferences setIncrementalRenderingSuppressionTimeoutInSeconds:]):
(-[WebPreferences incrementalRenderingSuppressionTimeoutInSeconds]):
* WebView/WebPreferencesPrivate.h:
* WebView/WebView.mm:
(-[WebView _preferencesChanged:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115005 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed. Fix incorrect test results for <datalist>
tkent@chromium.org [Tue, 24 Apr 2012 06:19:24 +0000 (06:19 +0000)]
Unreviewed. Fix incorrect test results for <datalist>

* fast/forms/datalist/datalist-expected.txt: Remove fallback content.
* fast/forms/datalist/datalist-nonoption-child-expected.txt: ditto.
* platform/chromium-linux/fast/forms/datalist/input-list-expected.png: Removed.
* platform/chromium-mac/fast/forms/datalist/input-list-expected.png: Removed.
* platform/chromium-mac/fast/forms/datalist/input-list-expected.txt: Removed.
* platform/chromium-win/fast/forms/datalist/input-list-expected.png: Removed.
* platform/chromium-win/fast/forms/datalist/input-list-expected.txt: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115001 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium] Image masks are considered opaque incorrectly
danakj@chromium.org [Tue, 24 Apr 2012 06:12:41 +0000 (06:12 +0000)]
[chromium] Image masks are considered opaque incorrectly
https://bugs.webkit.org/show_bug.cgi?id=84275

Reviewed by Adrienne Walker.

Source/WebCore:

Match the behaviour of SkCanvas layers more closely while tracking
opaque paints. SkCanvas layers actually act as a separate device
(ie. pixels) and when the layer is popped off, the pixels are copied
down to the layer below.

While we can use the total clip to decide what pixels the the
drawing operation will affect in the final device, the blending
down through layers needs to consider each layer carefully.

In this case the image mask is drawn into a layer which is copied
down using the DestinationIn operation. Since the layer contains
non-opaque pixels, the DestinationIn copy can destroy opaque
areas in the next layer. We add OpaqueRegionSkia::FillByCopy to
distinguish the case where we are copying a block of pixels, and the
alpha values are essentially unknown.

Unit test: PlatformContextSkiaTest.trackImageMask
           PlatformContextSkiaTest.trackImageMaskWithOpaqueRect

* platform/graphics/skia/OpaqueRegionSkia.cpp:
(WebCore::paintIsOpaque):
(WebCore::OpaqueRegionSkia::applyOpaqueRegionFromLayer):
(WebCore::OpaqueRegionSkia::pushCanvasLayer):
(WebCore::OpaqueRegionSkia::popCanvasLayer):
(WebCore::OpaqueRegionSkia::didDrawRect):
(WebCore::OpaqueRegionSkia::didDrawPath):
(WebCore::OpaqueRegionSkia::didDrawPoints):
(WebCore::OpaqueRegionSkia::didDrawBounded):
(WebCore::OpaqueRegionSkia::didDraw):
(WebCore::OpaqueRegionSkia::didDrawUnbounded):
(WebCore::OpaqueRegionSkia::markRectAsOpaque):
(WebCore::OpaqueRegionSkia::markRectAsNonOpaque):
(WebCore::OpaqueRegionSkia::markAllAsNonOpaque):
* platform/graphics/skia/OpaqueRegionSkia.h:
(OpaqueRegionSkia):
(CanvasLayerState):
* platform/graphics/skia/PlatformContextSkia.cpp:
(WebCore::PlatformContextSkia::restoreLayer):

Source/WebKit/chromium:

* tests/PlatformContextSkiaTest.cpp:
(WebCore):
(WebCore::TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@115000 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoShow the format indicator in a date field
tkent@chromium.org [Tue, 24 Apr 2012 05:57:33 +0000 (05:57 +0000)]
Show the format indicator in a date field
https://bugs.webkit.org/show_bug.cgi?id=83872

Reviewed by Hajime Morita.

Source/WebCore:

Enable the fixed placeholder feature for the date type, and show
the editable date format as the fixed placeholder.
The format string is created by the following steps.
1. Obtain a pattern string from ICU
  http://icu-project.org/apiref/icu4c/classSimpleDateFormat.html#details
2. Replace a sequence of 'y', 'Y', 'M', or 'd' in the pattern with
  a natural language string such as "year", "month", or "day".

Test: fast/forms/date/date-fixed-placeholder.html

* html/DateInputType.cpp:
(WebCore::DateInputType::supportsPlaceholder): Added. Returns true.
(WebCore::DateInputType::usesFixedPlaceholder): Added. Returns true.
(WebCore::DateInputType::fixedPlaceholder):
Added. Calls localizedDateFormatText() provided by LocalizedDate.h.
* html/DateInputType.h:
(DateInputType): Add declarations of new override functions.
* platform/LocalizedStrings.h:
(WebCore): Add dateFormat{Year,Month,DayInMonth}Text()
* platform/text/ICULocale.cpp:
(WebCore::isICUYearSymbol): A function to improve redability.
(WebCore::isICUMonthSymbol): ditto.
(WebCore::isICUDayInMonthSymbol): ditto.
(WebCore::localizeFormat): Step 2 of the above description.
(WebCore::ICULocale::initializeLocalizedDateFormatText):
Step 1 of the above description.
(WebCore::ICULocale::localizedDateFormatText):
* platform/text/ICULocale.h:
(ICULocale): Add new functions and a data member.
* platform/text/LocalizedDate.h:
(WebCore): Add localizedDateFormatText().
* platform/text/LocalizedDateICU.cpp:
(WebCore::localizedDateFormatText):
Just call ICULocale::localizedDateFormatText().

Source/WebKit/chromium:

* src/LocalizedStrings.cpp:
(WebCore::dateFormatYearText): Added a stub.
(WebCore::dateFormatMonthText): ditto.
(WebCore::dateFormatDayInMonthText): ditto.

LayoutTests:

* fast/forms/date/date-fixed-placeholder-expected.txt: Added.
* fast/forms/date/date-fixed-placeholder.html: Added.
* platform/chromium/test_expectations.txt:
Mark date-appearance.html FAIL because of render dump change.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114999 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoGetting baselines for new test landed in http://trac.webkit.org/changeset/114973
dimich@chromium.org [Tue, 24 Apr 2012 05:34:04 +0000 (05:34 +0000)]
Getting baselines for new test landed in trac.webkit.org/changeset/114973
Not reviewed.

* platform/chromium-linux-x86/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-linux/fast/html/details-marker-style-expected.png: Added.
* platform/chromium-linux/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-mac-leopard/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-mac-snowleopard/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-mac/fast/html/details-marker-style-expected.png: Added.
* platform/chromium-mac/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-win-vista/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-win-xp/fast/html/details-marker-style-expected.txt: Added.
* platform/chromium-win/fast/html/details-marker-style-expected.png: Added.
* platform/chromium-win/fast/html/details-marker-style-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114998 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to v8Array()
haraken@chromium.org [Tue, 24 Apr 2012 05:12:14 +0000 (05:12 +0000)]
[V8] Pass Isolate to v8Array()
https://bugs.webkit.org/show_bug.cgi?id=84295

Reviewed by Nate Chapin.

The final objective is to pass Isolate around in V8 bindings.
This patch passes the Isolate to v8Array().

(GenerateNormalAttrGetter):
(NativeToJSValue):
* bindings/scripts/test/V8/V8TestObj.cpp:
(WebCore::TestObjV8Internal::sequenceAttrAttrGetter):
(WebCore::TestObjV8Internal::methodReturningSequenceCallback):
* bindings/v8/V8Binding.h:
(WebCore::v8Array):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114997 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to GetNamedProperty()
haraken@chromium.org [Tue, 24 Apr 2012 05:07:55 +0000 (05:07 +0000)]
[V8] Pass Isolate to GetNamedProperty()
https://bugs.webkit.org/show_bug.cgi?id=84277

Reviewed by Nate Chapin.

The objective is to pass Isolate around in V8 bindings.
This patch passes Isolate to GetNamedProperty().

(GenerateHeader):
* bindings/v8/V8DOMWindowShell.cpp:
(WebCore::getter):
* bindings/v8/custom/V8HTMLDocumentCustom.cpp:
(WebCore::V8HTMLDocument::GetNamedProperty):
(WebCore::V8HTMLDocument::allAccessorGetter):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114996 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoLayout Test platform/chromium/compositing/filters/background-filter-blur-outsets...
danakj@chromium.org [Tue, 24 Apr 2012 04:27:28 +0000 (04:27 +0000)]
Layout Test platform/chromium/compositing/filters/background-filter-blur-outsets.html is failing
https://bugs.webkit.org/show_bug.cgi?id=84072

Reviewed by Adrienne Walker.

Use onload and layoutTestController.display() to ensure the page is
loaded and compositor is ready before setting the background blur.

Also, smaller blur for faster test bots.

* platform/chromium-linux/platform/chromium/compositing/filters/background-filter-blur-outsets-expected.png:
* platform/chromium-linux/platform/chromium/compositing/filters/background-filter-blur-outsets-expected.txt:
* platform/chromium/compositing/filters/background-filter-blur-off-axis.html:
* platform/chromium/compositing/filters/background-filter-blur-outsets.html:
* platform/chromium/compositing/filters/background-filter-blur.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114995 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium] DomStorage event handling mods.
michaeln@google.com [Tue, 24 Apr 2012 04:19:50 +0000 (04:19 +0000)]
[chromium] DomStorage event handling mods.
https://bugs.webkit.org/show_bug.cgi?id=84387

Source/WebCore:

Add a few simple inline getters to allow the source Document of a storage
event to be identified given a pointer to the source StorageArea so those
Documents can be excluded by the event dispatching logic.

Reviewed by Dimitri Glazkov.

No new tests. No new functionality or change in behavior.

* page/DOMWindow.h:
(WebCore::DOMWindow::optionalSessionStorage): simple inline getter to avoid construction where possible
(WebCore::DOMWindow::optionalLocalStorage): ditto
* storage/Storage.h:
(WebCore::Storage::area): simple inline getter

Source/WebKit/chromium:

Events are currently handled inconsistently. The dispatch of some events are
initiated from within webkit/webcore, the dispatch of other events are initiated
from the outside via the WebKit::WebStorageEventDispatcher interface. The existing
WebStorageEventDispatcher is not expressive enough to handle initiation all
events from the outside. There's a chunk of nearly replicated code in there
that shouldn't be. The existing code has several FIXMEs related to making
this better.

The current state of things in webkit/webcore is also blocking development of some
overall performance improvements to chromium's implemention of this feature (getting
rid of sync ipcs for each access and adding a renderer-side caching layer).

To facilitate the perf improvements, this patch adds WebKit API to allow all
DomStorage events to be initiated from the outside. This is needed because
there will be an async latency between setting an item and receiving the
'oldValue' from the main browser process which is required to raise the
mutation event.

This is the first of a multi-sided sequence of patches to make this transition.
wkpatch 1: add the new wider WebKit API and impl
crpatch 2: start using the new wider WebKit API
wkpatch 3: cleanup, delete the old WebKit API and impl
xxxxxxx 4/5: cleanup, remove extra params from new API whose only purpose was to allow the transition

Reviewed by Dimitri Glazkov.

* public/WebStorageEventDispatcher.h: Add new API to dispatch events.
* public/WebStorageNamespace.h: Add new API to test for namespace equality.
* src/StorageAreaProxy.cpp:
(WebCore::StorageAreaProxy::dispatchLocalStorageEvent): implementation of the new API
(WebCore::StorageAreaProxy::dispatchSessionStorageEvent):  implementation of the new API
(WebCore::StorageAreaProxy::IsEventSource): a helper method
(WebCore::StorageAreaProxy::FindPageWithSessionStorageNamespace): a helper method
* src/StorageAreaProxy.h:
* src/StorageEventDispatcherImpl.cpp: added a FIXME comment to delete soon
* src/StorageNamespaceProxy.cpp:
(WebCore::StorageNamespaceProxy::IsSameNamespace): plumbing to call the new API
* src/StorageNamespaceProxy.h:
* src/WebStorageEventDispatcherImpl.cpp:
(WebKit::WebStorageEventDispatcher::dispatchLocalStorageEvent): plumbing to call the new impl
(WebKit::WebStorageEventDispatcher::dispatchSessionStorageEvent): plumbing to call the new impl
* src/WebStorageEventDispatcherImpl.h: added a FIXME comment to delete soon

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114993 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoChange ImageData to reference Uint8ClampedArray rather than CanvasPixelArray
kbr@google.com [Tue, 24 Apr 2012 03:43:31 +0000 (03:43 +0000)]
Change ImageData to reference Uint8ClampedArray rather than CanvasPixelArray
https://bugs.webkit.org/show_bug.cgi?id=73011

Reviewed by Oliver Hunt.

Source/WebCore:

Changed ImageData to contain Uint8ClampedArray instead of ByteArray
per current HTML5 spec.

In order to avoid introducing large data copies, it was necessary to
switch a few other places in WebKit to use Uint8ClampedArray at the
same time, most notably the FilterEffect implementation, portions of
ImageBuffer, and a couple of places in the image encoders.

Note that the change in method name from "get" to "item" occurred
because JSC's autogenerated indexed getters reference a method
called "item". If desired, a convenience method could be
introduced to avoid the renamings in the FilterEffect classes.

With these changes, ByteArray and some custom code in JavaScriptCore
actually become unreferenced, and can be deleted. This deletion will
be done separately in Bug 83655, to keep the size of this patch
manageable.

Test: fast/canvas/imagedata-contains-uint8clampedarray.html

* bindings/js/JSImageDataCustom.cpp:
(WebCore::toJS):
* bindings/js/SerializedScriptValue.cpp:
(WebCore::CloneSerializer::dumpIfTerminal):
(WebCore::CloneDeserializer::readTerminal):
* bindings/v8/SerializedScriptValue.cpp:
* bindings/v8/custom/V8ImageDataCustom.cpp:
(WebCore::toV8):
* bridge/qt/qt_pixmapruntime.cpp:
(JSC::Bindings::QtPixmapToImageDataMethod::invoke):
* html/ImageData.cpp:
(WebCore::ImageData::create):
(WebCore::ImageData::ImageData):
* html/ImageData.h:
(ImageData):
(WebCore::ImageData::data):
* html/ImageData.idl:
* html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::createEmptyImageData):
(WebCore::CanvasRenderingContext2D::getImageData):
(WebCore::CanvasRenderingContext2D::putImageData):
* platform/graphics/GraphicsContext3D.cpp:
(WebCore::GraphicsContext3D::extractImageData):
* platform/graphics/ImageBuffer.cpp:
(WebCore::ImageBuffer::genericConvertToLuminanceMask):
* platform/graphics/ImageBuffer.h:
* platform/graphics/ShadowBlur.cpp:
(WebCore::ShadowBlur::blurShadowBuffer):
* platform/graphics/cairo/ImageBufferCairo.cpp:
(WebCore::getImageData):
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
* platform/graphics/cg/ImageBufferCG.cpp:
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
(WebCore::ImageBuffer::toDataURL):
(WebCore::ImageDataToDataURL):
* platform/graphics/cg/ImageBufferDataCG.cpp:
(WebCore::ImageBufferData::getData):
(WebCore::ImageBufferData::putData):
* platform/graphics/cg/ImageBufferDataCG.h:
(ImageBufferData):
* platform/graphics/filters/FEBlend.cpp:
(WebCore::FEBlend::platformApplySoftware):
* platform/graphics/filters/FEColorMatrix.cpp:
(WebCore::effectType):
(WebCore::FEColorMatrix::platformApplySoftware):
* platform/graphics/filters/FEComponentTransfer.cpp:
(WebCore::FEComponentTransfer::platformApplySoftware):
* platform/graphics/filters/FEComposite.cpp:
(WebCore::FEComposite::platformArithmeticSoftware):
(WebCore::FEComposite::platformApplySoftware):
* platform/graphics/filters/FEComposite.h:
* platform/graphics/filters/FEConvolveMatrix.cpp:
(WebCore::setDestinationPixels):
(WebCore::FEConvolveMatrix::fastSetInteriorPixels):
(WebCore::FEConvolveMatrix::fastSetOuterPixels):
(WebCore::FEConvolveMatrix::platformApplySoftware):
* platform/graphics/filters/FEConvolveMatrix.h:
(PaintingData):
* platform/graphics/filters/FECustomFilter.cpp:
(WebCore::FECustomFilter::platformApplySoftware):
(WebCore::FECustomFilter::bindProgramAndBuffers):
* platform/graphics/filters/FECustomFilter.h:
(JSC):
(FECustomFilter):
* platform/graphics/filters/FEDisplacementMap.cpp:
(WebCore::FEDisplacementMap::platformApplySoftware):
* platform/graphics/filters/FEDropShadow.cpp:
(WebCore::FEDropShadow::platformApplySoftware):
* platform/graphics/filters/FEGaussianBlur.cpp:
(WebCore::boxBlur):
(WebCore::FEGaussianBlur::platformApplyGeneric):
(WebCore::FEGaussianBlur::platformApply):
(WebCore::FEGaussianBlur::platformApplySoftware):
* platform/graphics/filters/FEGaussianBlur.h:
(PlatformApplyParameters):
(FEGaussianBlur):
* platform/graphics/filters/FELighting.cpp:
(WebCore::FELighting::LightingData::topLeft):
(WebCore::FELighting::LightingData::topRow):
(WebCore::FELighting::LightingData::topRight):
(WebCore::FELighting::LightingData::leftColumn):
(WebCore::FELighting::LightingData::interior):
(WebCore::FELighting::LightingData::rightColumn):
(WebCore::FELighting::LightingData::bottomLeft):
(WebCore::FELighting::LightingData::bottomRow):
(WebCore::FELighting::LightingData::bottomRight):
(WebCore::FELighting::inlineSetPixel):
(WebCore::FELighting::drawLighting):
(WebCore::FELighting::platformApplySoftware):
* platform/graphics/filters/FELighting.h:
(LightingData):
(FELighting):
* platform/graphics/filters/FEMorphology.cpp:
(WebCore::FEMorphology::platformApplyGeneric):
(WebCore::FEMorphology::platformApplySoftware):
* platform/graphics/filters/FEMorphology.h:
(PaintingData):
* platform/graphics/filters/FETurbulence.cpp:
(WebCore::FETurbulence::fillRegion):
(WebCore::FETurbulence::platformApplySoftware):
* platform/graphics/filters/FETurbulence.h:
(FillRegionParameters):
(FETurbulence):
* platform/graphics/filters/FilterEffect.cpp:
(WebCore::FilterEffect::forceValidPreMultipliedPixels):
(WebCore::FilterEffect::asUnmultipliedImage):
(WebCore::FilterEffect::asPremultipliedImage):
(WebCore::FilterEffect::copyImageBytes):
(WebCore::FilterEffect::copyUnmultipliedImage):
(WebCore::FilterEffect::copyPremultipliedImage):
(WebCore::FilterEffect::createUnmultipliedImageResult):
(WebCore::FilterEffect::createPremultipliedImageResult):
* platform/graphics/filters/FilterEffect.h:
(FilterEffect):
* platform/graphics/filters/arm/FEGaussianBlurNEON.h:
(WebCore::FEGaussianBlur::platformApplyNeon):
* platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
(WebCore::GraphicsContext3D::paintRenderingResultsToImageData):
* platform/graphics/qt/ImageBufferQt.cpp:
(WebCore::getImageData):
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
* platform/graphics/skia/ImageBufferSkia.cpp:
(WebCore::getImageData):
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
* platform/graphics/texmap/TextureMapperGL.cpp:
* platform/graphics/wince/ImageBufferWinCE.cpp:
(WebCore::getImageData):
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
* platform/graphics/wx/ImageBufferWx.cpp:
(WebCore::ImageBuffer::getUnmultipliedImageData):
(WebCore::ImageBuffer::getPremultipliedImageData):
(WebCore::ImageBuffer::putByteArray):
* platform/image-encoders/skia/JPEGImageEncoder.cpp:
(WebCore::JPEGImageEncoder::encode):
* platform/image-encoders/skia/PNGImageEncoder.cpp:
(WebCore::PNGImageEncoder::encode):
* platform/image-encoders/skia/WEBPImageEncoder.cpp:
(WebCore::WEBPImageEncoder::encode):

Source/WebKit/chromium:

* src/GraphicsContext3DChromium.cpp:
(WebCore::GraphicsContext3DPrivate::paintRenderingResultsToImageData):
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::doPixelReadbackToCanvas):

Source/WTF:

* wtf/ArrayBuffer.h:
(ArrayBuffer):
(WTF::ArrayBuffer::create):
(WTF::ArrayBuffer::createUninitialized):
(WTF):
(WTF::ArrayBufferContents::tryAllocate):
* wtf/TypedArrayBase.h:
(WTF::TypedArrayBase::createUninitialized):
(TypedArrayBase):
* wtf/Uint8ClampedArray.h:
(Uint8ClampedArray):
(WTF::Uint8ClampedArray::createUninitialized):
(WTF):
(WTF::Uint8ClampedArray::zeroFill):

LayoutTests:

* fast/canvas/imagedata-contains-uint8clampedarray-expected.txt: Added.
* fast/canvas/imagedata-contains-uint8clampedarray.html: Added.
* fast/dom/Window/script-tests/postmessage-clone.js:
* fast/dom/Window/window-postmessage-clone-expected.txt:
* platform/chromium/fast/dom/Window/window-postmessage-clone-expected.txt:
* platform/gtk/fast/dom/Window/window-postmessage-clone-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114992 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium] Remove deleted files from WebCore.gypi and fix typo
commit-queue@webkit.org [Tue, 24 Apr 2012 03:39:32 +0000 (03:39 +0000)]
[chromium] Remove deleted files from WebCore.gypi and fix typo
https://bugs.webkit.org/show_bug.cgi?id=84651

Patch by Ryan Sleevi <rsleevi@chromium.org> on 2012-04-23
Reviewed by Ryosuke Niwa.

No new tests. No behavior change.

* WebCore.gypi:
Remove references to PlatformMouseEventQt.cpp and WheelEventQt.cpp. Add a missing comma.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114991 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, fix wrong paths of js-test-*.js.
tkent@chromium.org [Tue, 24 Apr 2012 03:29:00 +0000 (03:29 +0000)]
Unreviewed, fix wrong paths of js-test-*.js.
https://bugs.webkit.org/show_bug.cgi?id=82871

* fast/forms/datalist/input-list.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114990 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Fix issue with trying to access a constructor in a frame that has been removed
arv@chromium.org [Tue, 24 Apr 2012 03:21:48 +0000 (03:21 +0000)]
[V8] Fix issue with trying to access a constructor in a frame that has been removed
https://bugs.webkit.org/show_bug.cgi?id=84640

Reviewed by Kentaro Hara.

Source/WebCore:

This regressed in r113250. Now we do what we did before and return undefined if the frame
does not have a context.

Test: fast/dom/constructor-in-removed-frame.html

* bindings/v8/V8DOMWrapper.cpp:
(WebCore::V8DOMWrapper::constructorForType):

LayoutTests:

* fast/dom/constructor-in-removed-frame-expected.txt: Added.
* fast/dom/constructor-in-removed-frame.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114989 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoOscillator::setWaveTable() should not reset oscillator phase
crogers@google.com [Tue, 24 Apr 2012 03:19:36 +0000 (03:19 +0000)]
Oscillator::setWaveTable() should not reset oscillator phase
https://bugs.webkit.org/show_bug.cgi?id=84647

Reviewed by Kenneth Russell.

* Modules/webaudio/Oscillator.cpp:
(WebCore::Oscillator::setWaveTable):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114988 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoCrash when the page with a calendar picker is scrolled
tkent@chromium.org [Tue, 24 Apr 2012 03:15:08 +0000 (03:15 +0000)]
Crash when the page with a calendar picker is scrolled
https://bugs.webkit.org/show_bug.cgi?id=84287

Reviewed by Hajime Morita.

Use the same ownership model as WebPopupMenuImpl's.

* src/WebPagePopupImpl.cpp:
(WebKit::WebPagePopupImpl::close):
Clear m_widgetClient to avoid furthur access to it. deref instead of delete.
(WebKit::WebPagePopupImpl::closePopup):
Do not call closeWidgetSoon() if close() was already called.
(WebKit::WebPagePopup::create):
Add a reference. Add explanation of the ownership.
* src/WebPagePopupImpl.h:
(WebPagePopupImpl): Make this RefCounted.  Make the destuctor public.
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::WebViewImpl): No need to clear m_pagePopup explicitly.
(WebKit::WebViewImpl::openPagePopup): Need to use .get() because m_pagePopup is a RefPtr.
(WebKit::WebViewImpl::closePagePopup): ditto.
(WebKit::WebViewImpl::hidePopups): ditto.
* src/WebViewImpl.h: Make m_pagePopup a RefPtr.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114987 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8][Refactoring] Remove V8Proxy::throwSyntaxError()
haraken@chromium.org [Tue, 24 Apr 2012 02:11:07 +0000 (02:11 +0000)]
[V8][Refactoring] Remove V8Proxy::throwSyntaxError()
https://bugs.webkit.org/show_bug.cgi?id=84627

Reviewed by Nate Chapin.

This patch removes V8Proxy::throwSyntaxError()
since it is no longer used.

No tests. No change in behavior.

* bindings/v8/V8Proxy.cpp:
* bindings/v8/V8Proxy.h:
(V8Proxy):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114986 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoEnable parallel GC for BlackBerry.
staikos@webkit.org [Tue, 24 Apr 2012 02:07:52 +0000 (02:07 +0000)]
Enable parallel GC for BlackBerry.
https://bugs.webkit.org/show_bug.cgi?id=84633

Reviewed by Antonio Gomes.

* wtf/Platform.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114985 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[BlackBerry] Build failure when DEBUG_FAT_FINGER is enabled
charles.wei@torchmobile.com.cn [Tue, 24 Apr 2012 02:01:05 +0000 (02:01 +0000)]
[BlackBerry] Build failure when DEBUG_FAT_FINGER is enabled
https://bugs.webkit.org/show_bug.cgi?id=84563

Reviewed by Antonio Gomes.

Fat Finger log code uses obsolete IntRect::toString() for logging,
which was first implemented interally and removed later.

* WebKitSupport/FatFingers.cpp:
(BlackBerry::WebKit::FatFingers::findIntersectingRegions):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114984 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium] Use TextureLayerChromium for WebGL content instead of a dedicated layer...
jamesr@google.com [Tue, 24 Apr 2012 02:00:47 +0000 (02:00 +0000)]
[chromium] Use TextureLayerChromium for WebGL content instead of a dedicated layer type
https://bugs.webkit.org/show_bug.cgi?id=84311

Reviewed by Adrienne Walker.

Source/Platform:

Remove some unused getters.

* chromium/public/WebExternalTextureLayer.h:
(WebExternalTextureLayer):

Source/WebCore:

WebGL content's interface with the compositor can be expressed in terms of two basic operations:
1.) Do some stuff
2.) Provide a texture ID for the compositor to use

This implements (1) in DrawingBufferChromium and uses the already-existing TextureLayerChromium to handle (2).
This cuts down the compositor interface significantly and allows for more code reuse in exchange for more code
in TextureLayerChromium to handle the features WebGL needs.

* WebCore.gypi:
* platform/graphics/chromium/DrawingBufferChromium.cpp:
(WebCore::DrawingBuffer::~DrawingBuffer):
(DrawingBufferPrivate):
(WebCore::DrawingBufferPrivate::DrawingBufferPrivate):
(WebCore::DrawingBufferPrivate::~DrawingBufferPrivate):
(WebCore::DrawingBufferPrivate::layer):
(WebCore):
(WebCore::DrawingBuffer::platformLayer):
* platform/graphics/chromium/LayerRendererChromium.cpp:
* platform/graphics/chromium/TextureLayerChromium.h:
(TextureLayerChromiumClient):
(WebCore::TextureLayerChromiumClient::~TextureLayerChromiumClient):
(WebCore):
(TextureLayerChromium):
(WebCore::TextureLayerChromium::clearClient):
* platform/graphics/chromium/WebGLLayerChromium.cpp: Removed.
* platform/graphics/chromium/WebGLLayerChromium.h: Removed.
* platform/graphics/gpu/DrawingBuffer.h:
(WebCore):
(WebCore::DrawingBuffer::graphicsContext3D):
(DrawingBuffer):

Source/WebKit/chromium:

* WebKit.gypi:
* src/WebExternalTextureLayer.cpp:
(WebKit::WebExternalTextureLayer::create):
* src/WebLayer.cpp:
* tests/WebGLLayerChromiumTest.cpp: Removed.
* tests/WebLayerTest.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114983 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoAdd notice about running xcode-select before using Xcode 4.3.2 command line tools
dbates@webkit.org [Tue, 24 Apr 2012 01:50:35 +0000 (01:50 +0000)]
Add notice about running xcode-select before using Xcode 4.3.2 command line tools

Rubber-stamped by Eric Seidel.

Before you can use the Xcode 4.3.2 command line tools, you must either run
sudo xcode-select -switch /Applications/Xcode.app or set the environment variable
DEVELOPER_DIR to point to the Xcode developer tools directory.

* building/tools.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114982 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoIndexedDB: remove IDB(Index|ObjectStore)BackendImpl::get/getKey for IDBKey
commit-queue@webkit.org [Tue, 24 Apr 2012 01:50:07 +0000 (01:50 +0000)]
IndexedDB: remove IDB(Index|ObjectStore)BackendImpl::get/getKey for IDBKey
https://bugs.webkit.org/show_bug.cgi?id=84285

Patch by Alec Flett <alecflett@chromium.org> on 2012-04-23
Reviewed by Dimitri Glazkov.

Source/WebCore:

No new tests required, this code is no longer called.

This is old code supporting IDBKey-based lookup, supplanted by
IDBKeyRange. Now that chromium IPC/proxy code has landed,
these functions can be removed.

* Modules/indexeddb/IDBIndexBackendInterface.h:
(IDBIndexBackendInterface):
* Modules/indexeddb/IDBObjectStoreBackendInterface.h:
(IDBObjectStoreBackendInterface):

Source/WebKit/chromium:

This is old code supporting IDBKey-based lookup, supplanted by
IDBKeyRange. Now that chromium IPC/proxy code has landed,
these functions can be removed.

* public/WebIDBIndex.h:
* public/WebIDBObjectStore.h:
* src/IDBIndexBackendProxy.cpp:
* src/IDBIndexBackendProxy.h:
(IDBIndexBackendProxy):
* src/IDBObjectStoreBackendProxy.cpp:
* src/IDBObjectStoreBackendProxy.h:
(IDBObjectStoreBackendProxy):
* src/WebIDBIndexImpl.cpp:
* src/WebIDBIndexImpl.h:
(WebIDBIndexImpl):
* src/WebIDBObjectStoreImpl.cpp:
* src/WebIDBObjectStoreImpl.h:
(WebIDBObjectStoreImpl):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114981 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoREGRESSION(113604): [Soup] Some pages that use synchronous XMLHttpRequests freeze...
mrobinson@webkit.org [Tue, 24 Apr 2012 01:33:00 +0000 (01:33 +0000)]
REGRESSION(113604): [Soup] Some pages that use synchronous XMLHttpRequests freeze the browser
https://bugs.webkit.org/show_bug.cgi?id=84560

Reviewed by Xan Lopez.

When kicking off a synchronous XMLHttpRequest, add one to the connection
limit. This ensures that when a page starts a synchronous request, while
already at the connection limit the request will not deadlock.

* platform/network/soup/ResourceHandleSoup.cpp:
(WebCore::WebCoreSynchronousLoader::WebCoreSynchronousLoader): Accept a new SoupSession
argument so that we can get the correct SoupSession for the networking context. Bump
the connection limit.
(WebCore::WebCoreSynchronousLoader::~WebCoreSynchronousLoader): Decrement the connection limit.
(WebCore::WebCoreSynchronousLoader::adjustMaxConnections): Added this helper.
(WebCoreSynchronousLoader): Added a new SoupSession member.
(WebCore::sessionFromContext): Added this helper.
(WebCore::ResourceHandleInternal::soupSession): Use the new sessionFromContext helper.
(WebCore::ResourceHandle::loadResourceSynchronously): Pass the SoupSession from the NetworkingContext
to the synchronous loader.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114980 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoExplicitly mention Xcode version 4.3.2 in Mac debug instructions
dbates@webkit.org [Tue, 24 Apr 2012 01:27:39 +0000 (01:27 +0000)]
Explicitly mention Xcode version 4.3.2 in Mac debug instructions

Rubber-stamped by Eric Seidel.

The build location instructions at the top of page building/{debug-mac-uiprocess.html, debug.html}
are specific to Xcode 4.3.2.

* building/debug-mac-uiprocess.html: Substitute "Xcode 4.3.2" for "Xcode 4".
* building/debug.html: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114978 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, rolling out r114965.
jer.noble@apple.com [Tue, 24 Apr 2012 01:25:43 +0000 (01:25 +0000)]
Unreviewed, rolling out r114965.
http://trac.webkit.org/changeset/114965
https://bugs.webkit.org/show_bug.cgi?id=84665

Four new resource load delegate test failures (Requested by
jernoble on #webkit).

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

Source/WebCore:

* loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::didFinishLoading):
(WebCore::SubresourceLoader::didFail):
(WebCore::SubresourceLoader::willCancel):
(WebCore::SubresourceLoader::releaseResources):

LayoutTests:

* http/tests/cache/cancel-during-failure-crash-expected.txt: Removed.
* http/tests/cache/cancel-during-failure-crash.html: Removed.
* platform/chromium-mac/security/block-test-expected.txt:
* platform/chromium-win/security/block-test-expected.txt:
* platform/gtk/security/block-test-expected.txt:
* platform/mac/security/block-test-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114976 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoSource/WebCore: https://bugs.webkit.org/show_bug.cgi?id=84649
bdakin@apple.com [Tue, 24 Apr 2012 01:12:18 +0000 (01:12 +0000)]
Source/WebCore: https://bugs.webkit.org/show_bug.cgi?id=84649
RenderMathMLOperator currently ignores font families, fails
to use Stix

Reviewed by Dan Bernstein.

This patch makes RenderMathMLOperator honor the font-family
list. This means that by default, Stix glyphs will now be
used for operators just like for other MathML content.
Unfortunately, just doing that resulted in a bug because of
the fragile hardcoded glyph sizes. The Stix vertical bar
glyph is much smaller than the code assumed any glyphs would
be. That code should be re-written, but in the meantime, I
put a fix in place to try to make it work for small glyphs.

These new functions should be used instead of accessing
gGlyphHeight or gGlyphLineHeight directly. If the glyph is
smaller than the expected hardcoded value, then the actual
glyph height is used, and a new line height based on the
glyph height is used.
(WebCore::RenderMathMLOperator::glyphHeightForCharacter):
(WebCore::RenderMathMLOperator::lineHeightForCharacter):

Use the style's FontDescription to honor the font family
list and the style's FontSelector to honor @font-face.
Also use glyphHeightForCharacter() instead of gGlyphHeight.
And now that createGlyph() takes a lineHeight, use
lineHeightForCharacter() to determine the right value.
(WebCore::RenderMathMLOperator::updateFromElement):

size was an ambiguous variable name, so I renamed it to the
more accurate maxHeightForRenderer. This function also now
takes a lineHeight instead of using gGlyphLineHeight. And
again, use the style's FontDescription and FontSelector.
(WebCore::RenderMathMLOperator::createStackableStyle):

Re-name size to maxHeightForRenderer, and take lineHeight as
a parameter.
(WebCore::RenderMathMLOperator::createGlyph):
* rendering/mathml/RenderMathMLOperator.h:
(RenderMathMLOperator):

LayoutTests: https://bugs.webkit.org/show_bug.cgi?id=84649
RenderMathMLOperator currently ignores font families, fails to use Stix

Reviewed by Dan Bernstein.

 Metrics are slightly different now that Stix glyphs are being used for operators.
* platform/mac/mathml/presentation/fenced-expected.txt:
* platform/mac/mathml/presentation/fenced-mi-expected.txt:
* platform/mac/mathml/presentation/mo-expected.txt:
* platform/mac/mathml/presentation/mo-stretch-expected.txt:
* platform/mac/mathml/presentation/over-expected.txt:
* platform/mac/mathml/presentation/row-alignment-expected.txt:
* platform/mac/mathml/presentation/row-expected.txt:
* platform/mac/mathml/presentation/sub-expected.txt:
* platform/mac/mathml/presentation/subsup-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114975 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoReturn value from executed script in Chromium.
commit-queue@webkit.org [Tue, 24 Apr 2012 01:10:25 +0000 (01:10 +0000)]
Return value from executed script in Chromium.
https://bugs.webkit.org/show_bug.cgi?id=79851

Patch by Eriq Augustine <eaugusti@chromium.org> on 2012-04-23
Reviewed by Adam Barth.

Source/WebCore:

Providing a varaiant of evaluateScriptInIsolatedWorld that
returns the value of the evaluated script.

Test: platform/chromium/http/tests/misc/execute-and-return-value.html

* bindings/v8/ScriptController.cpp:
(WebCore::ScriptController::evaluateInIsolatedWorld):
* bindings/v8/ScriptController.h:
(ScriptController):
* bindings/v8/V8Proxy.cpp:
(WebCore::V8Proxy::evaluateInIsolatedWorld):
* bindings/v8/V8Proxy.h:
(V8Proxy):

Source/WebKit/chromium:

Providing a variant of evaluateScriptInIsolatedWorld that
returns the value of the evaluated script.

* public/WebFrame.h:
(WebFrame):
* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::executeScriptInIsolatedWorldAndReturnValues):
(WebKit):
* src/WebFrameImpl.h:
(WebKit):
(WebFrameImpl):

Tools:

Adding a function to LayoutTestController that evaluates a script in
an isolated wold an returns the result of the evaluation.

* DumpRenderTree/LayoutTestController.cpp:
(evaluateScriptInIsolatedWorldAndReturnValue):
(LayoutTestController::staticFunctions):
* DumpRenderTree/LayoutTestController.h:
(LayoutTestController):
* DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/chromium/LayoutTestController.cpp:
(LayoutTestController::LayoutTestController):
(LayoutTestController::setXSSAuditorEnabled):
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/chromium/LayoutTestController.h:
(LayoutTestController):
* DumpRenderTree/efl/LayoutTestControllerEfl.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/mac/LayoutTestControllerMac.mm:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/qt/LayoutTestControllerQt.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/qt/LayoutTestControllerQt.h:
(LayoutTestController):
* DumpRenderTree/win/LayoutTestControllerWin.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):
* DumpRenderTree/wx/LayoutTestControllerWx.cpp:
(LayoutTestController::evaluateScriptInIsolatedWorldAndReturnValue):

LayoutTests:

Testing executing a script and getting the result of that script.

* platform/chromium/http/tests/misc/execute-and-return-value-expected.txt: Added.
* platform/chromium/http/tests/misc/execute-and-return-value.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114974 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoRenderDetailsMarker should draw the triangle inside the content box
tkent@chromium.org [Tue, 24 Apr 2012 00:57:30 +0000 (00:57 +0000)]
RenderDetailsMarker should draw the triangle inside the content box
https://bugs.webkit.org/show_bug.cgi?id=84557

Reviewed by Hajime Morita.

Source/WebCore:

RenderDetailsMarker didn't support cases in which a marker has
borders and/or padings, or non-square size.

Test: fast/html/details-marker-style.html

* rendering/RenderDetailsMarker.cpp:
(WebCore::RenderDetailsMarker::getPath):
Scale by contentWidth() and contentHeight().
(WebCore::RenderDetailsMarker::paint):
Move the trianle origin by the ammount of left/top borders and paddings.

LayoutTests:

* fast/html/details-marker-style.html: Added.
* platform/mac-snowleopard/fast/html/details-marker-style-expected.png: Added.
* platform/mac-snowleopard/fast/html/details-marker-style-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114973 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to toV8() (Part6)
haraken@chromium.org [Tue, 24 Apr 2012 00:52:26 +0000 (00:52 +0000)]
[V8] Pass Isolate to toV8() (Part6)
https://bugs.webkit.org/show_bug.cgi?id=84273

Reviewed by Nate Chapin.

The objective is to pass Isolate to all toV8()s.
Since there are a lot of toV8()s, I'll make the change
step by step. This patch passes Isolate to toV8() in
several custom bindings.

No tests. No change in behavior.

* bindings/v8/custom/V8HTMLAllCollectionCustom.cpp:
(WebCore::getNamedItems):
(WebCore::getItem):
(WebCore::V8HTMLAllCollection::namedPropertyGetter):
(WebCore::V8HTMLAllCollection::itemCallback):
(WebCore::V8HTMLAllCollection::namedItemCallback):
(WebCore::V8HTMLAllCollection::callAsFunctionCallback):
* bindings/v8/custom/V8HTMLCollectionCustom.cpp:
(WebCore::getNamedItems):
(WebCore::V8HTMLCollection::namedPropertyGetter):
(WebCore::V8HTMLCollection::namedItemCallback):
(WebCore::toV8):
* bindings/v8/custom/V8SVGPathSegCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8StyleSheetCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8StyleSheetListCustom.cpp:
(WebCore::V8StyleSheetList::namedPropertyGetter):
* bindings/v8/custom/V8TrackEventCustom.cpp:
(WebCore::V8TrackEvent::trackAccessorGetter):
* bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:
(WebCore::toV8Object):
(WebCore::getObjectParameter):
(WebCore::V8WebGLRenderingContext::getAttachedShadersCallback):
(WebCore::V8WebGLRenderingContext::getExtensionCallback):
(WebCore::V8WebGLRenderingContext::getFramebufferAttachmentParameterCallback):
(WebCore::V8WebGLRenderingContext::getParameterCallback):
(WebCore::V8WebGLRenderingContext::getProgramParameterCallback):
(WebCore::V8WebGLRenderingContext::getShaderParameterCallback):
(WebCore::V8WebGLRenderingContext::getUniformCallback):
* bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
(WebCore::V8XMLHttpRequest::responseAccessorGetter):
* bindings/v8/custom/V8XSLTProcessorCustom.cpp:
(WebCore::V8XSLTProcessor::transformToFragmentCallback):
(WebCore::V8XSLTProcessor::transformToDocumentCallback):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114972 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoMove the content of LocalizedCalendarICU.cpp and LocalizedDateICU.cpp to ICULocale...
tkent@chromium.org [Tue, 24 Apr 2012 00:47:22 +0000 (00:47 +0000)]
Move the content of LocalizedCalendarICU.cpp and LocalizedDateICU.cpp to ICULocale.cpp.
https://bugs.webkit.org/show_bug.cgi?id=84568

Reviewed by Hajime Morita.

Move it to ICULocal.cpp because we want to share UDateFormat
object in LocalizedDate functions and LocalizedCalendar functions,
and it becomes easier to cache the UDateFormat object, and making
unit tests for these functions easier.

No new tests because of no bahevior changes.

* platform/text/ICULocale.cpp:
(WebCore::ICULocale::ICULocale): Initialize new members.
(WebCore::ICULocale::~ICULocale): Delete m_shortDateFormat.
(WebCore::ICULocale::initializeShortDateFormat):
Moved from creteShortDateFormatter of LocalizedDateICU.cpp.
(WebCore::ICULocale::parseLocalizedDate):
Moved from LocalizedDate.cpp, use m_shortDateFormat.
(WebCore::ICULocale::formatLocalizedDate): ditto.
(WebCore::ICULocale::createLabelVector):
Moved from LocalizedCalendarICU.cpp, use m_shortDateFormat.
(WebCore::createFallbackMonthLabels): Moved from LocalizedCalendarICU.cpp.
(WebCore::createFallbackWeekDayShortLabels): ditto.
(WebCore::ICULocale::initializeCalendar):
lazy initialization of m_monthLabels, m_weekDayShortLabels, and m_firstDayOfWeek.
(WebCore::ICULocale::monthLabels):
(WebCore::ICULocale::weekDayShortLabels):
(WebCore::ICULocale::firstDayOfWeek):
* platform/text/ICULocale.h:
(ICULocale): Add declarations.
* platform/text/LocalizedCalendarICU.cpp:
(WebCore::monthLabels): Use ICULocale::currentLocale().
(WebCore::weekDayShortLabels): ditto.
(WebCore::firstDayOfWeek): ditto.
* platform/text/LocalizedDateICU.cpp:
(WebCore::parseLocalizedDate): ditto.
(WebCore::formatLocalizedDate): ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114971 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoMove AudioDestinationChromium FIFO class to its own class.
commit-queue@webkit.org [Tue, 24 Apr 2012 00:39:20 +0000 (00:39 +0000)]
Move AudioDestinationChromium FIFO class to its own class.
https://bugs.webkit.org/show_bug.cgi?id=84058

Patch by Raymond Toy <rtoy@google.com> on 2012-04-23
Reviewed by Chris Rogers.

Source/WebCore:

Current tests cover the changes.

* WebCore.gypi: Update with new files.
* platform/audio/AudioPullFIFO.cpp: Copied from Source/WebKit/chromium/src/AudioDestinationChromium.cpp.
(WebCore): Renamed old class the AudioPullFIFO.
(WebCore::AudioPullFIFO::AudioPullFIFO):
(WebCore::AudioPullFIFO::consume):
(WebCore::AudioPullFIFO::findWrapLengths):
(WebCore::AudioPullFIFO::fillBuffer):
* platform/audio/AudioPullFIFO.h: Added.
(WebCore):
(AudioPullFIFO):
(WebCore::AudioPullFIFO::updateIndex):

Source/WebKit/chromium:

Remove private FIFO class and use new AudioPullFIFO class.
* src/AudioDestinationChromium.cpp:
(WebCore::AudioDestinationChromium::AudioDestinationChromium):
* src/AudioDestinationChromium.h:
(AudioDestinationChromium):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114970 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoDFG OSR exit should ensure that all variables have been initialized
fpizlo@apple.com [Tue, 24 Apr 2012 00:36:46 +0000 (00:36 +0000)]
DFG OSR exit should ensure that all variables have been initialized
https://bugs.webkit.org/show_bug.cgi?id=84653
<rdar://problem/11258183>

Source/JavaScriptCore:

Reviewed by Gavin Barraclough.

Initialize all uncaptured dead variables to undefined on OSR exit.

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::ValueSource::dump):
(JSC::DFG::SpeculativeJIT::compile):
(JSC::DFG::SpeculativeJIT::computeValueRecoveryFor):
* dfg/DFGSpeculativeJIT.h:

LayoutTests:

Reviewed by Gavin Barraclough.

* fast/js/dfg-dead-variable-on-exit-expected.txt: Added.
* fast/js/dfg-dead-variable-on-exit.html: Added.
* fast/js/script-tests/dfg-dead-variable-on-exit.js: Added.
(foo):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114969 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, test expectations update.
zmo@google.com [Tue, 24 Apr 2012 00:19:25 +0000 (00:19 +0000)]
Unreviewed, test expectations update.

* platform/chromium/test_expectations.txt

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114967 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoSource/WebCore: REGRESSION (r100311): YummySoup app crashes when trying to print
japhet@chromium.org [Tue, 24 Apr 2012 00:14:05 +0000 (00:14 +0000)]
Source/WebCore: REGRESSION (r100311): YummySoup app crashes when trying to print
https://bugs.webkit.org/show_bug.cgi?id=83918

Reviewed by Alexey Proskuryakov.

Test: http/tests/xmlhttprequest/cancel-during-failure-crash.html

* loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::didFinishLoading):
(WebCore::SubresourceLoader::didFail):
(WebCore::SubresourceLoader::willCancel): Now that we might enter the function if we're already Finishing,
    some cleanup needs to be done in the Initialized state only.
(WebCore::SubresourceLoader::releaseResources): Move requesting counting decrement and laodDone() to
    the finishing functions.

LayoutTests: Test for https://bugs.webkit.org/show_bug.cgi?id=83918.

Reviewed by Alexey Proskuryakov.

* http/tests/cache/cancel-during-failure-crash-expected.txt: Added.
* http/tests/cache/cancel-during-failure-crash.html: Added.
* platform/chromium-mac/security/block-test-expected.txt:
* platform/chromium-win/security/block-test-expected.txt:
* platform/gtk/security/block-test-expected.txt:
* platform/mac/security/block-test-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114965 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed. Split LayoutTests/ChangeLog.
haraken@chromium.org [Mon, 23 Apr 2012 23:57:40 +0000 (23:57 +0000)]
Unreviewed. Split LayoutTests/ChangeLog.

* ChangeLog-2012-04-23: Copied from LayoutTests/ChangeLog.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114964 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium] Visualize accelerated compositor rects.
shawnsingh@chromium.org [Mon, 23 Apr 2012 23:52:01 +0000 (23:52 +0000)]
[chromium] Visualize accelerated compositor rects.
https://bugs.webkit.org/show_bug.cgi?id=79400

Reviewed by Adrienne Walker.

No tests, all the code added is only debugging code.

This patch adds support for visualizing three different types of
rects on the chromium heads-up display:
1. updateRects that indicate what was painted/uploaded to a resource
2. propertyChangedRects that indicate layers that has property changes
3. surfaceDamageRects that are the accumulation of updateRects and
   propertyChangedRects, indicating what pixels on the screen have
   actually changed.  (Surface damage also accounts for newly exposed
   areas but that is not explicitly visualized.)

In addition to adding support for visualizing this, the
CCHeadsUpDisplay was significantly refactored, separating the FPS
Counter functionality into a different class, so that the heads-up
display is all about visualizing annotations, and those
annotations (frame rate, debug rects) are logged separately.

* WebCore.gypi:
* platform/graphics/chromium/cc/CCDebugRectHistory.cpp: Added.
(WebCore):
(WebCore::CCDebugRectHistory::CCDebugRectHistory):
(WebCore::CCDebugRectHistory::enabled):
(WebCore::CCDebugRectHistory::saveDebugRectsForCurrentFrame):
(WebCore::CCDebugRectHistory::savePaintRects):
(WebCore::CCDebugRectHistory::savePropertyChangedRects):
(WebCore::CCDebugRectHistory::saveSurfaceDamageRects):
* platform/graphics/chromium/cc/CCDebugRectHistory.h: Added.
(WebCore):
(WebCore::CCDebugRect::CCDebugRect):
(CCDebugRect):
(CCDebugRectHistory):
(WebCore::CCDebugRectHistory::create):
(WebCore::CCDebugRectHistory::debugRects):
* platform/graphics/chromium/cc/CCFrameRateCounter.cpp: Added.
(WebCore):
(WebCore::safeMod):
(WebCore::CCFrameRateCounter::frameIndex):
(WebCore::CCFrameRateCounter::CCFrameRateCounter):
(WebCore::CCFrameRateCounter::markBeginningOfFrame):
(WebCore::CCFrameRateCounter::markEndOfFrame):
(WebCore::CCFrameRateCounter::isBadFrameInterval):
(WebCore::CCFrameRateCounter::isBadFrame):
(WebCore::CCFrameRateCounter::getAverageFPSAndStandardDeviation):
(WebCore::CCFrameRateCounter::timeStampOfRecentFrame):
* platform/graphics/chromium/cc/CCFrameRateCounter.h: Added.
(WebCore):
(CCFrameRateCounter):
(WebCore::CCFrameRateCounter::create):
(WebCore::CCFrameRateCounter::currentFrameNumber):
(WebCore::CCFrameRateCounter::timeStampHistorySize):
* platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
(WebCore):
(WebCore::CCHeadsUpDisplay::enabled):
(WebCore::CCHeadsUpDisplay::showDebugRects):
(WebCore::CCHeadsUpDisplay::draw):
(WebCore::CCHeadsUpDisplay::drawHudContents):
(WebCore::CCHeadsUpDisplay::drawFPSCounter):
(WebCore::CCHeadsUpDisplay::drawFPSCounterText):
(WebCore::CCHeadsUpDisplay::drawDebugRects):
* platform/graphics/chromium/cc/CCHeadsUpDisplay.h:
(WebCore):
(CCHeadsUpDisplay):
(WebCore::CCHeadsUpDisplay::CCHeadsUpDisplay):
* platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore::CCSettings::CCSettings):
(CCSettings):
* platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
(WebCore::CCLayerTreeHostImpl::drawLayers):
(WebCore::CCLayerTreeHostImpl::swapBuffers):
* platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
(WebCore):
(WebCore::CCLayerTreeHostImpl::fpsCounter):
(WebCore::CCLayerTreeHostImpl::debugRectHistory):
(CCLayerTreeHostImpl):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114963 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoCut dependency on RenderLayer::scrollRectToVisible outside rendering
jchaffraix@webkit.org [Mon, 23 Apr 2012 23:49:34 +0000 (23:49 +0000)]
Cut dependency on RenderLayer::scrollRectToVisible outside rendering
https://bugs.webkit.org/show_bug.cgi?id=84607

Reviewed by Simon Fraser.

Source/WebCore:

Layering fix only, there should be no change behavior.

Because we don't have an accessor on RenderObject, a lot of the code needs
to know about RenderLayer. This is not necessary and exposes RenderLayer to
objects that shouldn't know about it.

This patch adds a RenderObject::scrollRectToVisible with the ad-hoc explanation
as to why it isn't on RenderBox (scrolling is a RenderBox concept).

* WebCore.exp.in:
* WebCore.order:
Updated to expose the new method.

* dom/Element.cpp:
(WebCore::Element::scrollIntoView):
(WebCore::Element::scrollIntoViewIfNeeded):
(WebCore::Element::updateFocusAppearance):
* editing/Editor.cpp:
(WebCore::Editor::findStringAndScrollToVisible):
* editing/FrameSelection.cpp:
(WebCore::FrameSelection::revealSelection):
* page/FrameView.cpp:
(WebCore::FrameView::scrollToAnchor):
Updated those call sites to use the new function.

* rendering/RenderLayer.h:
Removed ScrollBehavior.h #include and default argument values
as we are always called through RenderObject now.

* rendering/RenderObject.cpp:
(WebCore::RenderObject::scrollRectToVisible):
* rendering/RenderObject.h:
Added a new function that just forwards to the enclosing layer
if any. We return whether we actually tried to scroll to match
some call sites expectations.

Source/WebKit/mac:

* WebView/WebFrame.mm:
(-[WebFrame _scrollDOMRangeToVisible:]):
Updated to use RenderObject::scrollRectToVisible and not depend on RenderLayer.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114962 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoframebuffer binding should not be changed after canvas resize or compositing
zmo@google.com [Mon, 23 Apr 2012 23:22:03 +0000 (23:22 +0000)]
framebuffer binding should not be changed after canvas resize or compositing
https://bugs.webkit.org/show_bug.cgi?id=84609

Reviewed by Kenneth Russell.

Source/WebCore:

Test: fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize.html

* html/canvas/WebGLRenderingContext.cpp: set framebuffer binding to DrawingBuffer.
(WebCore):
(WebCore::WebGLRenderingContext::bindFramebuffer):
(WebCore::WebGLRenderingContext::deleteFramebuffer):
(WebCore::WebGLRenderingContext::loseContextImpl):
* platform/graphics/cairo/DrawingBufferCairo.cpp: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/DrawingBufferChromium.cpp: Ditto.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/WebGLLayerChromium.cpp: Recover framebuffer binding after update().
(WebCore::WebGLLayerChromium::update):
* platform/graphics/gpu/DrawingBuffer.cpp: Add a function to restore framebuffer binding.
(WebCore::DrawingBuffer::restoreFramebufferBinding):
(WebCore):
* platform/graphics/gpu/DrawingBuffer.h: Ditto.
(WebCore::DrawingBuffer::setTexture2DBinding):
(DrawingBuffer):
(WebCore::DrawingBuffer::setFramebufferBinding):
* platform/graphics/gpu/mac/DrawingBufferMac.mm: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/gpu/qt/DrawingBufferQt.cpp: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):

LayoutTests:

* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize-expected.txt: Added.
* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize.html: Added.
* fast/canvas/webgl/resources/webgl-test-utils.js: Sync with khronos side (partial)
(WebGLTestUtils.):
(WebGLTestUtils):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114961 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agorun-perf-tests fail intermittently with an exception
dpranke@chromium.org [Mon, 23 Apr 2012 23:13:13 +0000 (23:13 +0000)]
run-perf-tests fail intermittently with an exception
https://bugs.webkit.org/show_bug.cgi?id=83532

Reviewed by Ryosuke Niwa.

This bulletproofs port.get_option('child_processes')
against the case where the option wasn't set.

* Scripts/webkitpy/layout_tests/port/mac.py:
(MacPort._get_crash_log):
* Scripts/webkitpy/layout_tests/port/mac_unittest.py:
(test_get_crash_log):
(test_get_crash_log.fake_time_cb):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114960 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoCall instruction for the baseline JIT stores origin info in wrong callframe
oliver@apple.com [Mon, 23 Apr 2012 23:07:45 +0000 (23:07 +0000)]
Call instruction for the baseline JIT stores origin info in wrong callframe
https://bugs.webkit.org/show_bug.cgi?id=84645

Reviewed by Gavin Barraclough.

Source/JavaScriptCore:

The baseline JIT was updating the wrong callframe when making a call.  If the
call failed during dispatch (unable to perform codegen, calling a non-object)
we would attempt to use this information, but it would be completely wrong.

* jit/JITCall.cpp:
(JSC::JIT::compileOpCall):
* jit/JITCall32_64.cpp:
(JSC::JIT::compileOpCall):

LayoutTests:

Directly force LLInt, Baseline, and DFG JITs to handle the path where
a call fails during dispatch.

* fast/js/script-tests/stack-trace.js:
(callNonCallable):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114959 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Mac] WebProcess should empty cache on a background thread/block.
kling@webkit.org [Mon, 23 Apr 2012 22:52:13 +0000 (22:52 +0000)]
[Mac] WebProcess should empty cache on a background thread/block.
<http://webkit.org/b/84619>
<rdar://problem/10668689>

Reviewed by Anders Carlsson.

Move the removeAllCachedResponses call to a dispatch queue and wait for it on exit.

* WebProcess/WebProcess.cpp:
(WebKit::WebProcess::WebProcess):
* WebProcess/WebProcess.h:
(WebProcess):
* WebProcess/mac/WebProcessMac.mm:
(WebKit::WebProcess::platformClearResourceCaches):
(WebKit::WebProcess::platformTerminate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114958 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoSimplify volume slider rendering
commit-queue@webkit.org [Mon, 23 Apr 2012 22:41:01 +0000 (22:41 +0000)]
Simplify volume slider rendering
https://bugs.webkit.org/show_bug.cgi?id=82150

Patch by Victor Carbune <vcarbune@adobe.com> on 2012-04-23
Reviewed by Eric Carlson.

Source/WebCore:

Test: media/video-controls-rendering-toggle-display-none.html

* css/mediaControlsChromium.css: Update controls css for Chromium.
(audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel):
(audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button):
(audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container):
* css/mediaControlsGtk.css: Update controls css for GTK.
(audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button):
(audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container):
* css/mediaControlsQuickTime.css: Update controls css for Safari.
(audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button):
(audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container):
(audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):
* html/shadow/MediaControlElements.cpp: Removed particular renderer.
(WebCore):
* html/shadow/MediaControlElements.h:
(MediaControlVolumeSliderContainerElement): Remoed particular renderer.
* html/shadow/MediaControlRootElement.cpp:
(WebCore::MediaControlRootElement::create): Added a div element as a container. Removed extra unused mute button.
* html/shadow/MediaControlRootElementChromium.cpp: Added an extra div
element as a container for the mute button and the volume slider to
easily position them relative to each other.
(WebCore::MediaControlRootElementChromium::create):

LayoutTests:

Added new TEXT baselines for Mac.
Baselines for GTK and Chromium need to be re-generated as well.

* media/video-controls-rendering-toggle-display-none-expected.txt: Added.
* media/video-controls-rendering-toggle-display-none.html: Added.
* platform/chromium/test_expectations.txt: Updated test expectations
* platform/gtk/test_expectations.txt: Updated test expectations.
Commented out a duplicate test marked as CRASH because otherwise cq complains.
* platform/mac/fast/hidpi/video-controls-in-hidpi-expected.txt: Updated.
* platform/mac/media/audio-controls-rendering-expected.txt: Updated.
* platform/mac/media/audio-repaint-expected.txt: Updated.
* platform/mac/media/controls-after-reload-expected.txt: Updated.
* platform/mac/media/controls-strict-expected.txt: Updated.
* platform/mac/media/media-controls-clone-expected.txt: Updated.
* platform/mac/media/video-controls-rendering-expected.txt: Updated.
* platform/mac/media/video-display-toggle-expected.txt: Updated.
* platform/mac/media/video-no-audio-expected.txt: Updated.
* platform/mac/media/video-playing-and-pause-expected.txt: Updated.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114957 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoDFG must keep alive values that it will perform speculations on
fpizlo@apple.com [Mon, 23 Apr 2012 22:36:08 +0000 (22:36 +0000)]
DFG must keep alive values that it will perform speculations on
https://bugs.webkit.org/show_bug.cgi?id=84638
<rdar://problem/11258183>

Source/JavaScriptCore:

Reviewed by Oliver Hunt.

* dfg/DFGNodeType.h:
(DFG):

LayoutTests:

Rubber stamped by Oliver Hunt.

* fast/js/dfg-dead-speculation-expected.txt: Added.
* fast/js/dfg-dead-speculation.html: Added.
* fast/js/script-tests/dfg-dead-speculation.js: Added.
(foo):
(.thingy.f.valueOf):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114956 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[Qt] Add desktop zooming support for QQuickWebView
kbalazs@webkit.org [Mon, 23 Apr 2012 22:33:10 +0000 (22:33 +0000)]
[Qt] Add desktop zooming support for QQuickWebView
https://bugs.webkit.org/show_bug.cgi?id=82337

Unreviewed typo fix after r114900.

* MiniBrowser/qt/BrowserWindow.cpp:
(BrowserWindow::BrowserWindow): Fix the mistyped
value of a zoom level.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114955 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoFix non-LLInt builds by temporarily removing an over-enthusiastic assertion
oliver@apple.com [Mon, 23 Apr 2012 22:07:55 +0000 (22:07 +0000)]
Fix non-LLInt builds by temporarily removing an over-enthusiastic assertion

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114953 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[chromium-android] Get rid of the trick that replaces system fallback font with DejaV...
commit-queue@webkit.org [Mon, 23 Apr 2012 21:59:17 +0000 (21:59 +0000)]
[chromium-android] Get rid of the trick that replaces system fallback font with DejaVuSans.
https://bugs.webkit.org/show_bug.cgi?id=84580

Patch by Hao Zheng <zhenghao@chromium.org> on 2012-04-23
Reviewed by Ojan Vafai.

Android's first fallback font is changed to DroidSansArabic.ttf in JB.
Take the opportunity to get rid of the trick.

* Scripts/webkitpy/layout_tests/port/chromium_android.py:
(ChromiumAndroidPort.start_helper):
(ChromiumAndroidPort._push_fonts):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114951 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, rolling out r109981.
commit-queue@webkit.org [Mon, 23 Apr 2012 21:58:43 +0000 (21:58 +0000)]
Unreviewed, rolling out r109981.
http://trac.webkit.org/changeset/109981
https://bugs.webkit.org/show_bug.cgi?id=84630

Broke Twitter map buddy icon (see
https://bugs.webkit.org/show_bug.cgi?id=84558) (Requested by
thorton on #webkit).

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

Source/WebCore:

* rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::addToOverlapMap):
(WebCore::RenderLayerCompositor::computeCompositingRequirements):

LayoutTests:

* compositing/layer-creation/overlap-empty-layer-expected.txt: Removed.
* compositing/layer-creation/overlap-empty-layer.html: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114950 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoChromium roll 133422:133506.
dimich@chromium.org [Mon, 23 Apr 2012 20:32:33 +0000 (20:32 +0000)]
Chromium roll 133422:133506.
Not reviewed.

* DEPS:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114937 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoCrash in RenderInline::clippedOverflowRectForRepaint for PrintPreview
eae@chromium.org [Mon, 23 Apr 2012 20:17:31 +0000 (20:17 +0000)]
Crash in RenderInline::clippedOverflowRectForRepaint for PrintPreview
https://bugs.webkit.org/show_bug.cgi?id=84300

Reviewed by Simon Fraser.

No new tests, have not been able to come up with a reliable reduction.

* rendering/RenderInline.cpp:
(WebCore::RenderInline::clippedOverflowRectForRepaint):
Add NULL check for containingBlock() as it can return NULL when detached
from the tree.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114936 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoUnreviewed, rolling out r114929.
jer.noble@apple.com [Mon, 23 Apr 2012 20:12:20 +0000 (20:12 +0000)]
Unreviewed, rolling out r114929.
http://trac.webkit.org/changeset/114929
https://bugs.webkit.org/show_bug.cgi?id=84623

Broke 35 canvas/webgl tests (Requested by jernoble on
#webkit).

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

Source/WebCore:

* html/canvas/WebGLRenderingContext.cpp:
(WebCore):
(WebCore::WebGLRenderingContext::bindFramebuffer):
(WebCore::WebGLRenderingContext::deleteFramebuffer):
(WebCore::WebGLRenderingContext::loseContextImpl):
* platform/graphics/cairo/DrawingBufferCairo.cpp:
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/DrawingBufferChromium.cpp:
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/WebGLLayerChromium.cpp:
(WebCore::WebGLLayerChromium::update):
* platform/graphics/gpu/DrawingBuffer.cpp:
* platform/graphics/gpu/DrawingBuffer.h:
(WebCore::DrawingBuffer::setTexture2DBinding):
(DrawingBuffer):
* platform/graphics/gpu/mac/DrawingBufferMac.mm:
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/gpu/qt/DrawingBufferQt.cpp:
(WebCore::DrawingBuffer::DrawingBuffer):

LayoutTests:

* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize-expected.txt: Removed.
* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize.html: Removed.
* fast/canvas/webgl/resources/webgl-test-utils.js:
(WebGLTestUtils):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114935 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate around in V8Collection.h
haraken@chromium.org [Mon, 23 Apr 2012 20:06:11 +0000 (20:06 +0000)]
[V8] Pass Isolate around in V8Collection.h
https://bugs.webkit.org/show_bug.cgi?id=84299

Reviewed by Nate Chapin.

The objective is to pass Isolate around in V8 bindings.
This patch passes Isolate around in V8Collection.h.

No tests. No change in behavior.

* bindings/v8/V8Collection.h:
(WebCore::getV8Object):
(WebCore::getNamedPropertyOfCollection):
(WebCore::collectionNamedPropertyGetter):
(WebCore::getIndexedPropertyOfCollection):
(WebCore::collectionIndexedPropertyGetter):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114934 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agotest-webkitpy is failing
dpranke@chromium.org [Mon, 23 Apr 2012 19:49:02 +0000 (19:49 +0000)]
test-webkitpy is failing
https://bugs.webkit.org/show_bug.cgi?id=84617

Reviewed by Eric Seidel.

* Scripts/webkitpy/layout_tests/port/http_lock_unittest.py:
(HttpLockTestWithRealFileSystem.clean_all_lockfile):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114933 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to toV8() (Part5)
haraken@chromium.org [Mon, 23 Apr 2012 19:38:45 +0000 (19:38 +0000)]
[V8] Pass Isolate to toV8() (Part5)
https://bugs.webkit.org/show_bug.cgi?id=84271

Reviewed by Nate Chapin.

The objective is to pass Isolate to all toV8()s.
Since there are a lot of toV8(), I'll make the change
step by step. This patch passes Isolate to toV8()
in several custom bindings.

No tests. No change in behavior.

* bindings/v8/custom/V8MessageChannelConstructor.cpp:
(WebCore::V8MessageChannel::constructorCallback):
* bindings/v8/custom/V8MessageEventCustom.cpp:
(WebCore::V8MessageEvent::dataAccessorGetter):
(WebCore::V8MessageEvent::portsAccessorGetter):
* bindings/v8/custom/V8MutationCallbackCustom.cpp:
(WebCore::V8MutationCallback::handleEvent):
* bindings/v8/custom/V8NamedNodeMapCustom.cpp:
(WebCore::V8NamedNodeMap::indexedPropertyGetter):
(WebCore::V8NamedNodeMap::namedPropertyGetter):
(WebCore::toV8):
* bindings/v8/custom/V8NodeListCustom.cpp:
(WebCore::V8NodeList::namedPropertyGetter):
* bindings/v8/custom/V8NotificationCenterCustom.cpp:
(WebCore::V8NotificationCenter::createHTMLNotificationCallback):
(WebCore::V8NotificationCenter::createNotificationCallback):
* bindings/v8/custom/V8PerformanceCustom.cpp:
(WebCore::V8Performance::memoryAccessorGetter):
* bindings/v8/custom/V8PopStateEventCustom.cpp:
(WebCore::V8PopStateEvent::stateAccessorGetter):
* bindings/v8/custom/V8SQLTransactionSyncCustom.cpp:
(WebCore::V8SQLTransactionSync::executeSqlCallback):
* bindings/v8/custom/V8SVGPathSegCustom.cpp:
(WebCore::toV8):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114932 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoframebuffer binding should not be changed after canvas resize or compositing
zmo@google.com [Mon, 23 Apr 2012 19:31:22 +0000 (19:31 +0000)]
framebuffer binding should not be changed after canvas resize or compositing
https://bugs.webkit.org/show_bug.cgi?id=84609

Reviewed by Kenneth Russell.

Source/WebCore:

Test: fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize.html

* html/canvas/WebGLRenderingContext.cpp: set framebuffer binding to DrawingBuffer.
(WebCore):
(WebCore::WebGLRenderingContext::bindFramebuffer):
(WebCore::WebGLRenderingContext::deleteFramebuffer):
(WebCore::WebGLRenderingContext::loseContextImpl):
* platform/graphics/cairo/DrawingBufferCairo.cpp: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/DrawingBufferChromium.cpp: Ditto.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/chromium/WebGLLayerChromium.cpp: Recover framebuffer binding after update().
(WebCore::WebGLLayerChromium::update):
* platform/graphics/gpu/DrawingBuffer.cpp: Add a function to restore framebuffer binding.
(WebCore::DrawingBuffer::restoreFramebufferBinding):
(WebCore):
* platform/graphics/gpu/DrawingBuffer.h: Ditto.
(WebCore::DrawingBuffer::setTexture2DBinding):
(DrawingBuffer):
(WebCore::DrawingBuffer::setFramebufferBinding):
* platform/graphics/gpu/mac/DrawingBufferMac.mm: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):
* platform/graphics/gpu/qt/DrawingBufferQt.cpp: initialize m_framebufferBinding.
(WebCore::DrawingBuffer::DrawingBuffer):

LayoutTests:

* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize-expected.txt: Added.
* fast/canvas/webgl/framebuffer-bindings-unaffected-on-resize.html: Added.
* fast/canvas/webgl/resources/webgl-test-utils.js: Sync with khronos side (partial)
(WebGLTestUtils.):
(WebGLTestUtils):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114929 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoSplit up flakiness_dashboard.html into multiple files. This is a step in the direction
ojan@chromium.org [Mon, 23 Apr 2012 19:24:44 +0000 (19:24 +0000)]
Split up flakiness_dashboard.html into multiple files. This is a step in the direction
of making the code more maintainable (e.g. being able to use qunit for testing).

* TestResultServer/static-dashboards/dashboard_base.js:
* TestResultServer/static-dashboards/flakiness_dashboard.css: Added.
* TestResultServer/static-dashboards/flakiness_dashboard.html:
* TestResultServer/static-dashboards/flakiness_dashboard.js: Copied from Tools/TestResultServer/static-dashboards/flakiness_dashboard.html.
* TestResultServer/static-dashboards/flakiness_dashboard_tests.css: Added.
* TestResultServer/static-dashboards/flakiness_dashboard_unittests.js: Renamed from Tools/TestResultServer/static-dashboards/flakiness_dashboard_tests.js.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114928 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to toV8() (Part4)
haraken@chromium.org [Mon, 23 Apr 2012 19:07:14 +0000 (19:07 +0000)]
[V8] Pass Isolate to toV8() (Part4)
https://bugs.webkit.org/show_bug.cgi?id=84269

Reviewed by Nate Chapin.

The objective is to pass Isolate to all toV8()s.
Since there are a lot of toV8()s, I'll make the change
step by step. This patch passes Isolate to toV8() in
several custom bindings.

No tests. No change in behavior.

* bindings/v8/custom/V8HTMLCanvasElementCustom.cpp:
(WebCore::V8HTMLCanvasElement::getContextCallback):
* bindings/v8/custom/V8HTMLCollectionCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8HTMLDocumentCustom.cpp:
(WebCore::V8HTMLDocument::allAccessorGetter):
* bindings/v8/custom/V8HTMLElementCustom.cpp:
(WebCore::toV8Object):
(WebCore::V8HTMLElement::itemValueAccessorGetter):
* bindings/v8/custom/V8HTMLFormElementCustom.cpp:
(WebCore::V8HTMLFormElement::indexedPropertyGetter):
(WebCore::V8HTMLFormElement::namedPropertyGetter):
* bindings/v8/custom/V8HTMLFrameSetElementCustom.cpp:
(WebCore::V8HTMLFrameSetElement::namedPropertyGetter):
* bindings/v8/custom/V8HTMLImageElementConstructor.cpp:
(WebCore::v8HTMLImageElementConstructorCallback):
* bindings/v8/custom/V8HTMLLinkElementCustom.cpp:
(WebCore::V8HTMLLinkElement::sizesAccessorGetter):
* bindings/v8/custom/V8HTMLOptionsCollectionCustom.cpp:
(WebCore::V8HTMLOptionsCollection::indexedPropertyGetter):
* bindings/v8/custom/V8HTMLOutputElementCustom.cpp:
(WebCore::V8HTMLOutputElement::htmlForAccessorGetter):
* bindings/v8/custom/V8HTMLSelectElementCustom.cpp:
(WebCore::V8HTMLSelectElement::indexedPropertyGetter):
* bindings/v8/custom/V8ImageDataCustom.cpp:
(WebCore::toV8):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114927 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years agoClean up subpixel unit handling in hit testing code
eae@chromium.org [Mon, 23 Apr 2012 18:59:59 +0000 (18:59 +0000)]
Clean up subpixel unit handling in hit testing code
https://bugs.webkit.org/show_bug.cgi?id=84496

Reviewed by Eric Seidel.

Fix use of IntRect and LayoutRect in hit testing code in preparation for
subpixel layout.

No new tests, no change in functionality.

* rendering/HitTestResult.cpp:
(WebCore::HitTestResult::rectForPoint):
Revert rectForPoint to IntRect as all call sites converted it to an
IntRect anyway to compare it with an IntPoint or another IntRect.

* rendering/HitTestingTransformState.cpp:
(WebCore::HitTestingTransformState::boundsOfMappedQuad):
* rendering/HitTestingTransformState.h:
Convert boundsOfMappedQuad to LayoutRect as the TransformationMatrix now
has subpixel precision.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114926 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to toV8() (Part3)
haraken@chromium.org [Mon, 23 Apr 2012 18:47:43 +0000 (18:47 +0000)]
[V8] Pass Isolate to toV8() (Part3)
https://bugs.webkit.org/show_bug.cgi?id=84261

Reviewed by Nate Chapin.

The objective is to pass Isolate to all toV8()s.
Since there are a lot of toV8(), I'll make the change
step by step. This patch passes Isolate to toV8()
in several custom bindings.

No tests. No change in behavior.

* bindings/v8/custom/V8DOMWindowCustom.cpp:
(WebCore::V8DOMWindow::openCallback):
(WebCore::V8DOMWindow::indexedPropertyGetter):
(WebCore::V8DOMWindow::namedPropertyGetter):
* bindings/v8/custom/V8DirectoryEntrySyncCustom.cpp:
(WebCore::V8DirectoryEntrySync::getDirectoryCallback):
(WebCore::V8DirectoryEntrySync::getFileCallback):
* bindings/v8/custom/V8DocumentCustom.cpp:
(WebCore::V8Document::evaluateCallback):
(WebCore::V8Document::getCSSCanvasContextCallback):
(WebCore::V8Document::createTouchListCallback):
* bindings/v8/custom/V8DocumentLocationCustom.cpp:
(WebCore::V8Document::locationAccessorGetter):
* bindings/v8/custom/V8EntryCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8EntrySyncCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8EventCustom.cpp:
(WebCore::V8Event::dataTransferAccessorGetter):
(WebCore::V8Event::clipboardDataAccessorGetter):
(WebCore):
* bindings/v8/custom/V8FileReaderCustom.cpp:
(WebCore::V8FileReader::resultAccessorGetter):
* bindings/v8/custom/V8HTMLAllCollectionCustom.cpp:
(WebCore::getItem):
(WebCore::V8HTMLAllCollection::callAsFunctionCallback):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114925 268f45cc-cd09-0410-ab3c-d52691b4dbfc

7 years ago[V8] Pass Isolate to toV8() (Part2)
haraken@chromium.org [Mon, 23 Apr 2012 18:42:51 +0000 (18:42 +0000)]
[V8] Pass Isolate to toV8() (Part2)
https://bugs.webkit.org/show_bug.cgi?id=84259

Reviewed by Nate Chapin.

The objective is to pass Isolate to all toV8()s.
Since there are a lot of toV8(), I'll make the change
step by step. This patch passes Isolate to toV8() in
several custom bindings.

No tests. No change in behavior.

* bindings/v8/custom/V8BlobCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSRuleCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSStyleSheetCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CSSValueCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8CanvasRenderingContext2DCustom.cpp:
(WebCore::toV8Object):
(WebCore::V8CanvasRenderingContext2D::strokeStyleAccessorGetter):
(WebCore::V8CanvasRenderingContext2D::fillStyleAccessorGetter):
* bindings/v8/custom/V8ConsoleCustom.cpp:
(WebCore::V8Console::memoryAccessorGetter):
* bindings/v8/custom/V8CustomSQLStatementErrorCallback.cpp:
(WebCore::V8SQLStatementErrorCallback::handleEvent):
* bindings/v8/custom/V8DOMStringMapCustom.cpp:
(WebCore::toV8):
* bindings/v8/custom/V8DOMTokenListCustom.cpp:
(WebCore::toV8):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114924 268f45cc-cd09-0410-ab3c-d52691b4dbfc