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

Timeline



Jun 9, 2012:

11:36 PM Changeset in webkit [119922] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Make WebGL mark draws for compositing even if the draw count is zero
​https://bugs.webkit.org/show_bug.cgi?id=88718

Patch by Gregg Tavares <​gman@google.com> on 2012-06-09
Reviewed by Kenneth Russell.

No new tests as no new functionality.

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::drawArrays):
(WebCore::WebGLRenderingContext::drawElements):

11:31 PM Changeset in webkit [119921] by dominicc@chromium.org
  • 8 edits
    14 deletes in trunk

[Chromium] Remove JavaScriptCore dependencies from gyp
​https://bugs.webkit.org/show_bug.cgi?id=88510

Reviewed by Adam Barth.

Chromium doesn't support JSC any more and there doesn't seem to be
a strong interest in using GYP as the common build system in other
ports.

Source/JavaScriptCore:

  • JavaScriptCore.gyp/JavaScriptCore.gyp: WebCore still depends on YARR interpreter.
  • JavaScriptCore.gypi: Only include YARR source.
  • gyp/JavaScriptCore.gyp: Removed.
  • gyp/gtk.gyp: Removed.

Source/WebCore:

No new functionality => No new tests.

  • gyp/WebCore.gyp: Removed.
  • gyp/copy-forwarding-and-icu-headers.sh: Removed.
  • gyp/copy-inspector-resources.sh: Removed.
  • gyp/generate-derived-sources.sh: Removed.
  • gyp/generate-webcore-export-file-generator.sh: Removed.
  • gyp/gtk.gyp: Removed.
  • gyp/run-if-exists.sh: Removed.
  • gyp/streamline-inspector-source.sh: Removed.
  • gyp/update-info-plist.sh: Removed.

Source/WTF:

  • WTF.gyp/WTF.gyp:

Tools:

  • DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
  • Scripts/generate-project-files: Removed.
7:45 PM Changeset in webkit [119920] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Remove a Chromium test expectation for the test that has been rolled out in r119918.

  • platform/chromium/TestExpectations:
6:08 PM Changeset in webkit [119919] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Refactor and improve code style in TextResourceDecoder to prepare for buffering improvement
​https://bugs.webkit.org/show_bug.cgi?id=88566

Reviewed by Alexey Proskuryakov.

  • loader/TextResourceDecoder.cpp:

(WebCore::bytesEqual): Added helper function to make comparing against sequences of bytes
easier to read at the call site.
(WebCore::KanjiCode::judge): Changed to use bytesEqual in a few places, merged multiple if
statements, got rid of else after goto.
(WebCore::TextResourceDecoder::checkForCSSCharset): Changed to use early return instead of
nesting the whole function. Changed to use bytesEqual.
(WebCore::skipComment): Changed to use early return for more cases. Changed to use bytesEqual.
(WebCore::TextResourceDecoder::checkForHeadCharset): Changed to use bytesEqual.
(WebCore::TextResourceDecoder::decode): Changed to use emptyString() instead of "", since
the former is more efficient.

4:03 PM Changeset in webkit [119918] by commit-queue@webkit.org
  • 21 edits
    4 deletes in trunk

Unreviewed, rolling out r118618 and r119353.
​http://trac.webkit.org/changeset/118618
​http://trac.webkit.org/changeset/119353
​https://bugs.webkit.org/show_bug.cgi?id=88720

Caused at least 30 different crashes on ClusterFuzz (Requested
by inferno-sec on #webkit).

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

Source/WebCore:

  • loader/SubresourceLoader.cpp:

(WebCore::SubresourceLoader::checkForHTTPStatusCodeError):

  • loader/cache/CachedCSSStyleSheet.cpp:

(WebCore::CachedCSSStyleSheet::allClientsRemoved):

  • loader/cache/CachedFont.cpp:

(WebCore::CachedFont::allClientsRemoved):

  • loader/cache/CachedFont.h:

(WebCore::CachedFontClient::resourceClientType):

  • loader/cache/CachedImage.cpp:

(WebCore::CachedImage::removeClientForRenderer):
(WebCore):
(WebCore::CachedImage::allClientsRemoved):
(WebCore::CachedImage::lookupOrCreateImageForRenderer):

  • loader/cache/CachedImage.h:

(CachedImage):
(WebCore::CachedImageClient::resourceClientType):

  • loader/cache/CachedRawResource.cpp:

(WebCore::CachedRawResource::allClientsRemoved):
(WebCore):

  • loader/cache/CachedRawResource.h:

(CachedRawResource):
(WebCore::CachedRawResourceClient::resourceClientType):

  • loader/cache/CachedResource.cpp:

(WebCore::CachedResource::removeClient):

  • loader/cache/CachedResource.h:

(WebCore::CachedResource::allClientsRemoved):

  • loader/cache/CachedResourceClient.h:

(WebCore::CachedResourceClient::resourceClientType):

  • loader/cache/CachedSVGDocument.h:

(WebCore::CachedSVGDocumentClient::resourceClientType):

  • loader/cache/CachedScript.cpp:

(WebCore::CachedScript::allClientsRemoved):

  • loader/cache/CachedStyleSheetClient.h:

(WebCore::CachedStyleSheetClient::resourceClientType):

  • rendering/style/StyleCachedImage.cpp:

(WebCore::StyleCachedImage::removeClient):

  • rendering/style/StyleCachedImageSet.cpp:

(WebCore::StyleCachedImageSet::removeClient):

  • svg/graphics/SVGImageCache.cpp:

(WebCore::SVGImageCache::~SVGImageCache):
(WebCore::SVGImageCache::removeRendererFromCache):
(WebCore::SVGImageCache::setRequestedSizeAndScales):
(WebCore::SVGImageCache::requestedSizeAndScales):
(WebCore::SVGImageCache::lookupOrCreateBitmapImageForRenderer):

  • svg/graphics/SVGImageCache.h:

(WebCore):
(SVGImageCache):

LayoutTests:

  • http/tests/cache/cancel-in-progress-load-expected.txt: Removed.
  • http/tests/cache/cancel-in-progress-load.html: Removed.
  • http/tests/misc/write-while-waiting.html:
  • svg/as-image/svg-image-leak-cached-data-expected.txt: Removed.
  • svg/as-image/svg-image-leak-cached-data.html: Removed.
2:38 PM Changeset in webkit [119917] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Fix Chromium test expectations after my previous commit (r119916).

  • platform/chromium/TestExpectations:
1:57 PM Changeset in webkit [119916] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Spring CRASH cleanup of Chromium test expectations.
Removed a whole bunch of bogus CRASH test expectations.

  • platform/chromium/TestExpectations:
12:58 PM Changeset in webkit [119915] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

Remove bogus CRASH expectations from skipped tests.
There's no point in specifying test expectation when tests are skipped.

  • platform/chromium/TestExpectations:
12:37 PM Changeset in webkit [119914] by fmalita@chromium.org
  • 3 edits
    4 adds in trunk

Fixed-position foreignObject descendants should be relative to the foreignObject viewport
​https://bugs.webkit.org/show_bug.cgi?id=88547

Reviewed by Abhishek Arya.

Source/WebCore:

Tests: svg/foreignObject/fO-fixed-position-crash.html

svg/foreignObject/fixed-position-expected.svg
svg/foreignObject/fixed-position.svg

Fixed position elements are currently registered with the top level
RenderView even when embedded within an SVG foreignOject. This patch
changes containingBlock() & container() to return the containing
foreignObject renderer instead.

The new foreignObject fixed position behavior matches that of current
FireFox and Opera versions and is consistent with the spec:
​http://www.w3.org/TR/CSS2/visuren.html#fixed-positioning
​http://www.w3.org/TR/SVG/coords.html#EstablishingANewViewport

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::containingBlock):
(WebCore::RenderObject::container):

LayoutTests:

  • svg/foreignObject/fO-fixed-position-crash-expected.txt: Added.
  • svg/foreignObject/fO-fixed-position-crash.html: Added.
  • svg/foreignObject/fixed-position-expected.svg: Added.
  • svg/foreignObject/fixed-position.svg: Added.
12:35 PM Changeset in webkit [119913] by abarth@webkit.org
  • 7 edits
    3 adds in trunk

Can't use eval in iframes sanbdoxed via CSP header
​https://bugs.webkit.org/show_bug.cgi?id=88450

Reviewed by Mihai Parparita.

Source/WebCore:

The initial empty document in a frame inherits the security context of
its parent (including the CSP policy). When we load the real document,
in some cases we'll do a "secure transition" to the new document. That
means that we leave the global object in place in case the parent
document has created any properties that it expects will be visible to
the new document.

If the parent document has a CSP policy that blocks eval, the "no eval"
bit will be set on the global object of the initial document. When we
perform a "secure transition" to the new document, we'll keep the bit,
which is wrong. In this patch, we reset the bit by always enabling
eval when clearing the context, regardless of whether we're performing
a "secure transition".

Test: http/tests/security/contentSecurityPolicy/iframe-inside-csp.html

  • bindings/js/ScriptController.cpp:

(WebCore::ScriptController::enableEval):
(WebCore):

  • bindings/js/ScriptController.h:

(ScriptController):

  • bindings/v8/ScriptController.cpp:

(WebCore::ScriptController::enableEval):
(WebCore):
(WebCore::ScriptController::disableEval):

  • bindings/v8/ScriptController.h:

(ScriptController):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::clear):

LayoutTests:

  • http/tests/security/contentSecurityPolicy/iframe-inside-csp-expected.txt: Added.
  • http/tests/security/contentSecurityPolicy/iframe-inside-csp.html: Added.
  • http/tests/security/contentSecurityPolicy/resources/sandboxed-eval.php: Added.
12:14 PM Changeset in webkit [119912] by commit-queue@webkit.org
  • 3 edits in trunk/LayoutTests

[EFL] skip new tests added in r119883
​https://bugs.webkit.org/show_bug.cgi?id=88716

Unreviewed EFL gardening. Skip new tests added in r119883.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-09

  • platform/efl/Skipped:
  • platform/efl/TestExpectations:
11:42 AM Changeset in webkit [119911] by commit-queue@webkit.org
  • 4 edits
    3 adds in trunk

The value in Access-Control-Allow-Origin is not being matched correctly for CORS-enabled requests
​https://bugs.webkit.org/show_bug.cgi?id=88139

Patch by Pablo Flouret <​pablof@motorola.com> on 2012-06-09
Reviewed by Adam Barth.

Source/WebCore:

Compare a request's origin with the value given in any
Access-Control-Allow-Origin headers in an exact, case-sensitive manner,
instead of using SecurityOrigin::isSameSchemeHostPort(). Per step 3 of
the resource sharing check algorithm in
​http://dvcs.w3.org/hg/cors/raw-file/tip/Overview.html#resource-sharing-check

Test: http/tests/xmlhttprequest/origin-exact-matching.html

  • loader/CrossOriginAccessControl.cpp:

(WebCore::passesAccessControlCheck):

LayoutTests:

  • http/tests/xmlhttprequest/origin-exact-matching-expected.txt: Added.
  • http/tests/xmlhttprequest/origin-exact-matching.html: Added.
  • http/tests/xmlhttprequest/resources/basic-auth/access-control-auth-basic.php: Get rid of a trailing slash in the origin.
  • http/tests/xmlhttprequest/resources/origin-exact-matching-iframe.html: Added.
11:26 AM Changeset in webkit [119910] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL] Skip tests that rely on pathToLocalResource() due to regression in r119788
​https://bugs.webkit.org/show_bug.cgi?id=88714

Unreviewed EFL gardening. Skip tests that rely on pathToLocalResource()
due to regression in r119788.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-09

  • platform/efl/TestExpectations:
10:34 AM Changeset in webkit [119909] by ggaren@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Unreviewed, rolling back in part2 of r118646.

This patch removes eager finalization.

Weak pointer finalization should be lazy
​https://bugs.webkit.org/show_bug.cgi?id=87599

Reviewed by Sam Weinig.

  • heap/Heap.cpp:

(JSC::Heap::collect): Don't finalize eagerly -- we'll do it lazily.

  • heap/MarkedBlock.cpp:

(JSC::MarkedBlock::sweep): Do sweep weak sets when sweeping a block,
since we won't get another chance.

  • heap/MarkedBlock.h:

(JSC::MarkedBlock::sweepWeakSet):

  • heap/MarkedSpace.cpp:

(MarkedSpace::WeakSetSweep):

  • heap/MarkedSpace.h:

(JSC::MarkedSpace::sweepWeakSets): Removed now-unused code.

10:11 AM Changeset in webkit [119908] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[Qt][Texmap] All layers with backingStore are opaque when using TextureMapperGL.
​https://bugs.webkit.org/show_bug.cgi?id=88703

The bug originated from StillImage that is not override
Image::currentFrameHasAlpha(), so it always returns false.
StillImage is used by ImageBuffer, and if Texmap draws contents on ImageBuffer,
TextureMapperBackingStore thinks contents does not have alpha.

Covered by existing compositing tests.

Patch by Huang Dongsung <​luxtella@company100.net> on 2012-06-09
Reviewed by Noam Rosenthal.

  • platform/graphics/qt/StillImageQt.cpp:

(WebCore::StillImage::currentFrameHasAlpha):
(WebCore):

  • platform/graphics/qt/StillImageQt.h:

(StillImage):

10:01 AM Changeset in webkit [119907] by commit-queue@webkit.org
  • 7 edits
    3 adds in trunk

Basic support for timestamps within a TextTrackCue
​https://bugs.webkit.org/show_bug.cgi?id=88187

Implemented support for timestamps within a TextTrackCue.
This enables rendering functionality for Karaoke and Paint-on captions.

Patch by Victor Carbune <​victor@rosedu.org> on 2012-06-09
Reviewed by Eric Carlson.

Source/WebCore:

Test: media/track/track-cue-rendering-inner-timestamps.html

  • css/mediaControls.css: Remove the background shadow pseudo-id and

added two others, which can be further used for styling past / future
contents of the cue.
(video::-webkit-media-text-track-past-nodes): Used to style elements
within the cue that are before the current movie time.
(video::-webkit-media-text-track-future-nodes): Used to style elements
within the cue that are after the current movie time.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::updateActiveTextTrackCues): Regardless of whether
the active set changed or not, the current cues need to be informed of the current
movie time.

  • html/track/TextTrackCue.cpp:

(WebCore::TextTrackCue::TextTrackCue): Initiliazed past and future containers.
(WebCore::TextTrackCue::getCueAsHTML): Checked whether the current cue text has
inner timestamps or not, in order to render faster if no timestamps are there.
(WebCore::TextTrackCue::updateDisplayTree): Added method that keeps the past and future
containers up to date, given a current movie time.
(WebCore):
(WebCore::TextTrackCue::getDisplayTree): Replaced the cue background container with the
past cues container. The *internal* display tree of the cue now has two containers,
representing past and future elements.

  • html/track/TextTrackCue.h: Added several variables to support the new functionality.

(TextTrackCue):

  • html/track/WebVTTParser.cpp:

(WebCore::WebVTTParser::constructTreeFromToken): Updated representation for this patch.

  • html/track/WebVTTParser.h:

(WebVTTParser): Made public the collectTimeStamp method, as this is required for
parsing timestamps within a cue as well.

LayoutTests:

  • media/track/captions-webvtt/captions-inner-timestamps.vtt: Added.
  • media/track/track-cue-rendering-inner-timestamps-expected.txt: Added.
  • media/track/track-cue-rendering-inner-timestamps.html: Added.
9:53 AM Changeset in webkit [119906] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Open links in Sources panel by default and fallback to Resources and Network panel otherwise.
​https://bugs.webkit.org/show_bug.cgi?id=88710

Reviewed by Pavel Feldman.

  • inspector/front-end/inspector.js:

(WebInspector.documentClick.followLink):
(WebInspector.documentClick):
(WebInspector._showAnchorLocation):

9:48 AM Changeset in webkit [119905] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Only expand root domain in Sources/Scripts panel
​https://bugs.webkit.org/show_bug.cgi?id=88135

Reviewed by Pavel Feldman.

  • inspector/front-end/NavigatorView.js:

(WebInspector.NavigatorTreeOutline._treeElementsCompare.compare.typeWeight):
(WebInspector.NavigatorTreeOutline._treeElementsCompare):
(WebInspector.NavigatorFolderTreeElement.prototype.onattach):

9:46 AM Changeset in webkit [119904] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Pretty print should respect text editor indent setting.
​https://bugs.webkit.org/show_bug.cgi?id=88713

Reviewed by Pavel Feldman.

  • inspector/front-end/ScriptFormatter.js:
9:29 AM Changeset in webkit [119903] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Fixed the build after r119895.

  • WebCore.xcodeproj/project.pbxproj:
6:18 AM Changeset in webkit [119902] by caseq@chromium.org
  • 4 edits in trunk/Source/WebCore

Web Inspector: [refactoring] rename TimelineVerticalOverview into TimelineFrameOverview
​https://bugs.webkit.org/show_bug.cgi?id=88708

Reviewed by Vsevolod Vlasov.

  • rename TimelineVerticalOverview to TimelineFrameOverview;
  • rename associated fields, parameters, CSS classes etc;
  • rename overview mode designators to match the UI;
  • drop unused CSS rules for old-style mode selector;
  • inspector/front-end/TimelineOverviewPane.js:

(WebInspector.TimelineOverviewPane):
(WebInspector.TimelineOverviewPane.prototype._showEvents):
(WebInspector.TimelineOverviewPane.prototype._showFrames):
(WebInspector.TimelineOverviewPane.prototype._showMemoryGraph):
(WebInspector.TimelineOverviewPane.prototype._setFrameMode):
(WebInspector.TimelineOverviewPane.prototype._onCategoryVisibilityChanged):
(WebInspector.TimelineOverviewPane.prototype._update):
(WebInspector.TimelineOverviewPane.prototype.addFrame):
(WebInspector.TimelineOverviewPane.prototype.zoomToFrame):
(WebInspector.TimelineOverviewPane.prototype._reset):
(WebInspector.TimelineOverviewPane.prototype._onWindowChanged):
(WebInspector.TimelineFrameOverview):

  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel):
(WebInspector.TimelinePanel.prototype._shouldShowFrames):
(WebInspector.TimelinePanel.prototype.revealRecordAt):

  • inspector/front-end/timelinePanel.css:

(.timeline-frame-overview #timeline-overview-grid):
(.timeline-frame-overview .timeline-overview-window):
(.timeline-frame-overview .timeline-overview-dividers-background):
(.timeline-frame-overview #timeline-overview-memory):
(.timeline-frame-overview-status-bar-item.toggled-on .glyph):
(.timeline-frame-overview-bars):
(.timeline.timeline-frame-overview .resources-divider):
(.sidebar-tree-item .timeline-frame-overview-status-bar-item):

5:01 AM Changeset in webkit [119901] by vsevik@chromium.org
  • 10 edits in trunk

IndexedDB: Inspector should handle null, string, and array keyPaths
​https://bugs.webkit.org/show_bug.cgi?id=84303

Reviewed by Pavel Feldman.

Source/WebCore:

Supported different key path types and updated tests to cover each case.
Key path strings are now highlighted in the DataGrid column headers.

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

(WebCore):

  • inspector/front-end/IndexedDBModel.js:

(WebInspector.IndexedDBModel.idbKeyPathFromKeyPath):
(WebInspector.IndexedDBModel.keyPathStringFromIDBKeyPath):
(WebInspector.IndexedDBModel.prototype._loadDatabase.callback):
(WebInspector.IndexedDBModel.prototype._loadDatabase):
(WebInspector.IndexedDBModel.ObjectStore.prototype.get keyPathString):
(WebInspector.IndexedDBModel.Index.prototype.get keyPathString):

  • inspector/front-end/IndexedDBViews.js:

(WebInspector.IDBDataView.prototype._createDataGrid):
(WebInspector.IDBDataView.prototype._keyColumnHeaderFragment):
(WebInspector.IDBDataView.prototype._keyPathStringFragment):
(WebInspector.IDBDataGridNode.prototype.createCell):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.IDBObjectStoreTreeElement.prototype._updateTooltip):
(WebInspector.IDBIndexTreeElement.prototype._updateTooltip):

LayoutTests:

  • http/tests/inspector/indexeddb/database-structure-expected.txt:
  • http/tests/inspector/indexeddb/database-structure.html:
4:49 AM Changeset in webkit [119900] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL] Gardening failure case on EFL build bot
​https://bugs.webkit.org/show_bug.cgi?id=88656

Unreviewed EFL gardening after r119795. jquery/core.html test
is crashing on debug bot and failing with a text diff on
release bot.

Patch by Sudarsana Nagineni <​sudarsana.nagineni@linux.intel.com> on 2012-06-09

  • platform/efl/TestExpectations:
4:25 AM Changeset in webkit [119899] by vsevik@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Increase size limits for resource content cached in InspectorResourceAgent.
​https://bugs.webkit.org/show_bug.cgi?id=88674

Reviewed by Pavel Feldman.

  • inspector/NetworkResourcesData.cpp:
4:23 AM Changeset in webkit [119898] by vsevik@chromium.org
  • 18 edits in trunk

Web Inspector: Limit DebuggerAgent.Location use to DebuggerModel only, introduce WebInspector.RawLocation to be used elsewhere.
​https://bugs.webkit.org/show_bug.cgi?id=87270

Reviewed by Pavel Feldman.

Source/WebCore:

Introduced WebInspector.RawLocation interface, a generic raw location independent from concrete domain (Styles, Debugger).
Made WebInspector.DebuggerModel.Location implement it and switched all debugger related methods to it.

  • inspector/front-end/BreakpointManager.js:

(WebInspector.BreakpointManager.prototype._breakpointResolved):

  • inspector/front-end/CompilerScriptMapping.js:

(WebInspector.CompilerScriptMapping.prototype.rawLocationToUILocation):

  • inspector/front-end/DebuggerModel.js:

(WebInspector.DebuggerModel.Location):
(WebInspector.DebuggerModel.prototype.continueToLocation):
(WebInspector.DebuggerModel.prototype.setBreakpointByScriptLocation):
(WebInspector.DebuggerModel.prototype.setBreakpoint.didSetBreakpoint):
(WebInspector.DebuggerModel.prototype.setBreakpoint):
(WebInspector.DebuggerModel.prototype.setBreakpointBySourceId):
(WebInspector.DebuggerModel.prototype._breakpointResolved):
(WebInspector.DebuggerModel.prototype.createRawLocation):
(WebInspector.DebuggerModel.prototype.createRawLocationByURL):
(WebInspector.DebuggerModel.prototype.setSelectedCallFrame):
(WebInspector.DebuggerModel.prototype.createLiveLocation):
(WebInspector.DebuggerModel.CallFrame.prototype.get location):
(WebInspector.DebuggerModel.CallFrame.prototype.createLiveLocation):

  • inspector/front-end/DebuggerResourceBinding.js:

(WebInspector.DebuggerResourceBinding.setScriptSource.didEditScriptSource):
(WebInspector.DebuggerResourceBinding.setScriptSource):
(WebInspector.DebuggerResourceBinding.prototype.setContent.get if):
(WebInspector.DebuggerResourceBinding.prototype.setContent):
(WebInspector.DebuggerResourceBinding.prototype._setContentWithInitialContent):

  • inspector/front-end/JavaScriptSource.js:

(WebInspector.JavaScriptSource.prototype.uiLocationToRawLocation):

  • inspector/front-end/Linkifier.js:
  • inspector/front-end/ObjectPopoverHelper.js:

(WebInspector.ObjectPopoverHelper.prototype._showObjectPopover.showObjectPopover.):
(WebInspector.ObjectPopoverHelper.prototype._showObjectPopover):

  • inspector/front-end/PresentationConsoleMessageHelper.js:
  • inspector/front-end/RawSourceCode.js:
  • inspector/front-end/ResourceScriptMapping.js:

(WebInspector.ResourceScriptMapping.prototype.rawLocationToUILocation):

  • inspector/front-end/Script.js:

(WebInspector.Script.prototype.rawLocationToUILocation):

  • inspector/front-end/ScriptSnippetModel.js:

(WebInspector.SnippetScriptMapping.prototype.rawLocationToUILocation):

  • inspector/front-end/SourceMapping.js:
  • inspector/front-end/UISourceCode.js:

(WebInspector.RawLocation):

LayoutTests:

  • inspector/debugger/breakpoint-manager.html:
  • inspector/debugger/pause-in-inline-script.html:
3:16 AM Changeset in webkit [119897] by commit-queue@webkit.org
  • 15 edits in trunk/Source

Web Inspector: Add message loop instrumentation to public API and timeline agent
​https://bugs.webkit.org/show_bug.cgi?id=88639

Patch by Eugene Klyuchnikov <​eustas.bug@gmail.com> on 2012-06-09
Reviewed by Vsevolod Vlasov.

Message loop instrumentation will show when the render thread is busy.
That way developer can discover if a render thread business causes low fps, or not.

Source/WebCore:

  • inspector/InspectorClient.h:

(WebCore::InspectorClient::startMessageLoopMonitoring): Starts observer.
(WebCore::InspectorClient::stopMessageLoopMonitoring): Stops observer.

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::InspectorController):
Changed timeline agent constructor parameters.

  • inspector/InspectorInstrumentation.cpp:

(WebCore::InspectorInstrumentation::willProcessTaskImpl):
Forwards observer notification.
(WebCore):
(WebCore::InspectorInstrumentation::didProcessTaskImpl):
Ditto.

  • inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):
(WebCore::InspectorInstrumentation::willProcessTask):
Ditto.
(WebCore):
(WebCore::InspectorInstrumentation::didProcessTask):
Ditto.

  • inspector/InspectorTimelineAgent.cpp:

(WebCore::InspectorTimelineAgent::start):
Starts observer.
(WebCore::InspectorTimelineAgent::stop):
Stops observer.
(WebCore::InspectorTimelineAgent::supportsFrameInstrumentation):
Changed result origin.
(WebCore::InspectorTimelineAgent::willProcessTask):
Noop implementqation.
(WebCore):
(WebCore::InspectorTimelineAgent::didProcessTask):
Noop implementqation.
(WebCore::InspectorTimelineAgent::InspectorTimelineAgent):
Changed constructor parameters.

  • inspector/InspectorTimelineAgent.h:

(WebCore::InspectorTimelineAgent::create):
Ditto.
(InspectorTimelineAgent):

  • inspector/WorkerInspectorController.cpp:

(WebCore::WorkerInspectorController::WorkerInspectorController):
Changed timeline agent constructor parameters.

Source/WebKit/chromium:

  • public/WebDevToolsAgent.h:

(WebDevToolsAgent):

  • public/WebDevToolsAgentClient.h:

(WebDevToolsAgentClient):
(WebKit::WebDevToolsAgentClient::startMessageLoopMonitoring):
Request message loop notifications.
(WebKit::WebDevToolsAgentClient::stopMessageLoopMonitoring):
Cancel message loop notifications.

  • src/InspectorClientImpl.cpp:

(WebKit::InspectorClientImpl::startMessageLoopMonitoring):
Request message loop notifications.
(WebKit):
(WebKit::InspectorClientImpl::stopMessageLoopMonitoring):
Cancel message loop notifications.

  • src/InspectorClientImpl.h:

(InspectorClientImpl):

  • src/WebDevToolsAgentImpl.cpp:

(WebKit::WebDevToolsAgentImpl::startMessageLoopMonitoring):
Request message loop notifications.
(WebKit):
(WebKit::WebDevToolsAgentImpl::stopMessageLoopMonitoring):
Cancel message loop notifications.
(WebKit::WebDevToolsAgentImpl::instrumentWillProcessTask):
Message loop notification.
(WebKit::WebDevToolsAgentImpl::instrumentDidProcessTask):
Ditto.

  • src/WebDevToolsAgentImpl.h:

(WebDevToolsAgentImpl):

3:13 AM Changeset in webkit [119896] by caseq@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: optimize timeline's frame mode overview appearance for 30fps
​https://bugs.webkit.org/show_bug.cgi?id=88556

Reviewed by Vsevolod Vlasov.

  • adjust frame overview scale to 30fps == full height by default;
  • provide a fall-back auto-scale mode when median frame is longer than 1/30s;
  • add horizontal lines for 30 and 60fps;
  • drive-by fix: always use same order of categories in frame bars.
  • inspector/front-end/TimelineOverviewPane.js:

(WebInspector.TimelineVerticalOverview.prototype.update):
(WebInspector.TimelineVerticalOverview.prototype._aggregateFrames):
(WebInspector.TimelineVerticalOverview.prototype._renderBars):
(WebInspector.TimelineVerticalOverview.prototype._drawFPSMarks):
(WebInspector.TimelineVerticalOverview.prototype._renderBar):

2:05 AM Changeset in webkit [119895] by commit-queue@webkit.org
  • 35 edits
    6 adds in trunk

Add UNDO_MANAGER flag
​https://bugs.webkit.org/show_bug.cgi?id=87908

Patch by Sukolsak Sakshuwong <​sukolsak@google.com> on 2012-06-09
Reviewed by Tony Chang.

.:

  • Source/cmake/WebKitFeatures.cmake:

Source/JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

Source/WebCore:

Add undoManager property to document behind the UNDO_MANAGER flag and
add placeholders for some methods of undoManager.

Test: editing/undomanager/document-has-undomanager.html

  • CMakeLists.txt:
  • Configurations/FeatureDefines.xcconfig:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • DerivedSources.pri:
  • GNUmakefile.am:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/Document.cpp:

(WebCore::Document::Document):
(WebCore):
(WebCore::Document::undoManager):

  • dom/Document.h:

(WebCore):
(Document):

  • dom/Document.idl:
  • editing/UndoManager.cpp: Added.

(WebCore):
(WebCore::UndoManager::create):
(WebCore::UndoManager::UndoManager):
(WebCore::UndoManager::undo):
(WebCore::UndoManager::redo):
(WebCore::UndoManager::clearUndo):
(WebCore::UndoManager::clearRedo):

  • editing/UndoManager.h: Added.

(WebCore):
(UndoManager):

  • editing/UndoManager.idl: Added.

Source/WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

  • Configurations/FeatureDefines.xcconfig:

Tools:

  • Scripts/webkitperl/FeatureList.pm:
  • qmake/mkspecs/features/features.pri:

WebKitLibraries:

  • win/tools/vsprops/FeatureDefines.vsprops:
  • win/tools/vsprops/FeatureDefinesCairo.vsprops:

Jun 8, 2012:

11:18 PM Changeset in webkit [119894] by rniwa@webkit.org
  • 3 edits in trunk/Tools

webkit-patch land-safely should set cq? if the patch author is not in committers.py
​https://bugs.webkit.org/show_bug.cgi?id=88689

Reviewed by Dirk Pranke.

When posting a patch on Bugzilla, trun cq+ into cq? if the Bugzilla login is not listed
in committers.py or the contributor cannot commit. If anything, the contributor can still
set cq+ on thier patches manually so I don't think this will be an issue for people who
don't list their Bugzille email on committers.py.

  • Scripts/webkitpy/common/net/bugzilla/bugzilla.py:

(Bugzilla._commit_queue_flag):

  • Scripts/webkitpy/common/net/bugzilla/bugzilla_unittest.py:

(test_commit_queue_flag):
(test_commit_queue_flag.assert_commit_queue_flag):

10:44 PM Changeset in webkit [119893] by leviw@chromium.org
  • 9 edits
    2 adds in trunk

RenderLayer::paintChildLayerIntoColumns doesn't properly pixel snap
​https://bugs.webkit.org/show_bug.cgi?id=88554

Reviewed by Darin Adler.

Source/WebCore:

Column handling code in RenderLayer and RenderBlock was improperly clipping and translating
using floats derived from FractionalLayoutUnits without pixel snapping. RenderLayer also
improperly stored render tree offsets in ints instead of LayoutUnits. This corrects this
issue.

Test: fast/sub-pixel/column-clipping.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintColumnContents):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::paintChildLayerIntoColumns):

LayoutTests:

  • fast/sub-pixel/column-clipping-expected.html: Added.
  • fast/sub-pixel/column-clipping.html: Added.
  • platform/chromium-mac-snowleopard/fast/multicol/nested-columns-expected.png:
  • platform/chromium-mac-snowleopard/fast/multicol/vertical-lr/nested-columns-expected.png:
  • platform/chromium-mac-snowleopard/fast/multicol/vertical-rl/nested-columns-expected.png:
  • platform/chromium-mac/fast/borders/border-antialiasing-expected.png:
  • platform/chromium/TestExpectations:
10:02 PM Changeset in webkit [119892] by rniwa@webkit.org
  • 4 edits in trunk/LayoutTests

Remove FAIL test expectations from EFL, Mac, and Qt ports' test expectations.

  • platform/efl/TestExpectations:
  • platform/mac/TestExpectations:
  • platform/qt/TestExpectations:
8:50 PM Changeset in webkit [119891] by commit-queue@webkit.org
  • 7 edits
    2 adds in trunk

REGRESSION(r116487?): HTMLFormElement::elementsname is empty if the form is detached from the document tree
​https://bugs.webkit.org/show_bug.cgi?id=88632

Patch by Rakesh KN <​rakesh.kn@motorola.com> on 2012-06-08
Reviewed by Ryosuke Niwa.

Source/WebCore:

Update root node of RadioNodeList when the form element is detached from dom tree.

Test: fast/forms/radionodelist-whose-form-element-detached-from-domtree.html

  • dom/Node.cpp:

(WebCore::Node::resetCachedRadioNodeListRootNode):
New function to update the root node of RadioNodeLists to form element from document.

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

(WebCore::HTMLFormElement::removedFrom):
On detach, we update the root node of all RadioNodeLists of this form element.

  • html/RadioNodeList.cpp:

(WebCore::RadioNodeList::setRootElement):
Setter for updating root node.

  • html/RadioNodeList.h:

(RadioNodeList): Ditto.

LayoutTests:

  • fast/forms/radionodelist-whose-form-element-detached-from-domtree-expected.txt: Added.
  • fast/forms/radionodelist-whose-form-element-detached-from-domtree.html: Added.
8:04 PM Changeset in webkit [119890] by dpranke@chromium.org
  • 2 edits in trunk/Tools

remove obsolete NewRunWebKitTests buildbot master factory classes
​https://bugs.webkit.org/show_bug.cgi?id=88693

Reviewed by Ryosuke Niwa.

I believe these aren't used any more ...

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:

(RunWebKitTests.getText2):
(BuildAndTestLeaksFactory):

7:51 PM Changeset in webkit [119889] by rniwa@webkit.org
  • 3 edits in trunk/LayoutTests

Remove all uses of FAIL test expectation from Chromium test expectations.

  • platform/chromium/TestExpectations:
  • platform/gtk/TestExpectations:
7:45 PM Changeset in webkit [119888] by wangxianzhu@chromium.org
  • 3 edits in trunk/Tools

[Chromium-Android] Build DumpRenderTree with Android SDK
​https://bugs.webkit.org/show_bug.cgi?id=88598

Change WebKit side of DRT related code according to some recent changes at chromium side.

Reviewed by Adam Barth.

  • DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
  • Scripts/webkitpy/layout_tests/port/chromium_android.py:

(ChromiumAndroidPort.init):
(ChromiumAndroidPort._path_to_driver):

7:29 PM Changeset in webkit [119887] by commit-queue@webkit.org
  • 21 edits in trunk/Source

[Chromium] Compositor doesn't support translucent root layers.
​https://bugs.webkit.org/show_bug.cgi?id=87821

Patch by David Reveman <​reveman@chromium.org> on 2012-06-08
Reviewed by James Robinson.

Source/Platform:

  • chromium/public/WebLayer.h:

(WebLayer):

  • chromium/public/WebLayerTreeView.h:

(WebLayerTreeView):

Source/WebCore:

Forward the isTransparent WebView setting to WebLayerTreeView,
CCLayerTreeHost and CCLayerTreeHostImpl as hasTransparentBackground.
Use hasTransparentBackground setting to determine how to clear the
root render pass and draw the background. Set opaque flag correctly
on the NonCompositedContentHost's graphics layer to make sure
sub-pixel rendering is not used with a transparent WebView.

Unit tests: CCLayerTreeHostImplTest.hasTransparentBackground

LayerRendererChromiumTest2.opaqueRenderPass
LayerRendererChromiumTest2.transparentRenderPass

  • platform/graphics/chromium/BitmapCanvasLayerTextureUpdater.cpp:

(WebCore::BitmapCanvasLayerTextureUpdater::prepareToUpdate):

  • platform/graphics/chromium/LayerRendererChromium.cpp:

(WebCore::LayerRendererChromium::clearRenderPass):
(WebCore::LayerRendererChromium::drawRenderPass):
(WebCore::LayerRendererChromium::drawBackgroundFilters):

  • platform/graphics/chromium/LayerRendererChromium.h:

(LayerRendererChromium):

  • platform/graphics/chromium/SkPictureCanvasLayerTextureUpdater.cpp:

(WebCore::SkPictureCanvasLayerTextureUpdater::prepareToUpdate):

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

(WebCore::CCLayerTreeHost::CCLayerTreeHost):
(WebCore::CCLayerTreeHost::finishCommitOnImplThread):

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

(WebCore::CCLayerTreeHost::setHasTransparentBackground):
(CCLayerTreeHost):

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

(WebCore::CCLayerTreeHostImpl::CCLayerTreeHostImpl):
(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):

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

(WebCore::CCLayerTreeHostImpl::hasTransparentBackground):
(WebCore::CCLayerTreeHostImpl::setHasTransparentBackground):
(CCLayerTreeHostImpl):

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

(WebCore::CCRenderPass::CCRenderPass):

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

(WebCore::CCRenderPass::hasTransparentBackground):
(WebCore::CCRenderPass::setHasTransparentBackground):
(CCRenderPass):

Source/WebKit/chromium:

  • src/NonCompositedContentHost.cpp:

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

  • src/NonCompositedContentHost.h:

(NonCompositedContentHost):

  • src/WebLayerTreeView.cpp:

(WebKit::WebLayerTreeView::setHasTransparentBackground):
(WebKit):

  • src/WebViewImpl.cpp:

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

  • tests/CCLayerTreeHostImplTest.cpp:
  • tests/LayerRendererChromiumTest.cpp:

(ClearCountingContext):
(ClearCountingContext::ClearCountingContext):
(ClearCountingContext::clear):
(ClearCountingContext::clearCount):
(TEST):

7:26 PM Changeset in webkit [119886] by commit-queue@webkit.org
  • 6 edits
    3 adds in trunk/Source

[chromium] Single thread proxy's animation timer should short circuit if the layer renderer has not been initialized
​https://bugs.webkit.org/show_bug.cgi?id=88668

Patch by Ian Vollick <​vollick@chromium.org> on 2012-06-08
Reviewed by James Robinson.

Source/WebCore:

Unit test: CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer.runSingleThreaded

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

Source/WebKit/chromium:

  • WebKit.gypi:
  • tests/CCLayerTreeHostTest.cpp:

(WTF::CCLayerTreeHostTestWriteLayersRedraw::beginTest):
(WTF::CCLayerTreeHostTestWriteLayersAfterVisible::commitCompleteOnCCThread):
(WTF):
(WTF::CCLayerTreeHostTestLayerAddedWithAnimation::CCLayerTreeHostTestLayerAddedWithAnimation):
(WTF::CCLayerTreeHostTestLayerAddedWithAnimation::beginTest):
(WTF::CCLayerTreeHostTestLayerAddedWithAnimation::didAddAnimation):
(CCLayerTreeHostTestLayerAddedWithAnimation):

  • tests/CCSingleThreadProxyTest.cpp: Added.

(FakeWebGraphicsContext3DMakeCurrentFails):
(FakeWebGraphicsContext3DMakeCurrentFails::makeContextCurrent):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation::CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation::beginTest):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation::animateLayers):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation::didRecreateContext):
(CCSingleThreadProxyTestInitializeLayerRendererFailsAfterAddAnimation::afterTest):
(TEST_F):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::beginTest):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::animateLayers):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::didRecreateContext):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::didAddAnimation):
(CCSingleThreadProxyTestDidAddAnimationBeforeInitializingLayerRenderer::afterTest):

  • tests/CCThreadedTest.cpp: Added.

(WebKitTests):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::create):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::createTexture):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::deleteTexture):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::bindTexture):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::numTextures):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::texture):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::resetTextures):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::numUsedTextures):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::usedTexture):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::resetUsedTextures):
(WebKitTests::CompositorFakeWebGraphicsContext3DWithTextureTracking::CompositorFakeWebGraphicsContext3DWithTextureTracking):
(WebKitTests::TestHooks::createContext):
(WebKitTests::MockLayerTreeHostImpl::create):
(WebKitTests::MockLayerTreeHostImpl::beginCommit):
(WebKitTests::MockLayerTreeHostImpl::commitComplete):
(WebKitTests::MockLayerTreeHostImpl::prepareToDraw):
(WebKitTests::MockLayerTreeHostImpl::drawLayers):
(WebKitTests::MockLayerTreeHostImpl::animateLayers):
(WebKitTests::MockLayerTreeHostImpl::lowFrequencyAnimationInterval):
(WebKitTests::MockLayerTreeHostImpl::MockLayerTreeHostImpl):
(MockLayerTreeHost):
(WebKitTests::MockLayerTreeHost::create):
(WebKitTests::MockLayerTreeHost::createLayerTreeHostImpl):
(WebKitTests::MockLayerTreeHost::MockLayerTreeHost):
(MockLayerTreeHostClient):
(WebKitTests::MockLayerTreeHostClient::create):
(WebKitTests::MockLayerTreeHostClient::MockLayerTreeHostClient):
(TimeoutTask):
(WebKitTests::TimeoutTask::TimeoutTask):
(WebKitTests::TimeoutTask::clearTest):
(WebKitTests::TimeoutTask::~TimeoutTask):
(WebKitTests::TimeoutTask::run):
(BeginTask):
(WebKitTests::BeginTask::BeginTask):
(WebKitTests::BeginTask::~BeginTask):
(WebKitTests::BeginTask::run):
(EndTestTask):
(WebKitTests::EndTestTask::EndTestTask):
(WebKitTests::EndTestTask::~EndTestTask):
(WebKitTests::EndTestTask::clearTest):
(WebKitTests::EndTestTask::run):
(WebKitTests::CCThreadedTest::CCThreadedTest):
(WebKitTests::CCThreadedTest::endTest):
(WebKitTests::CCThreadedTest::endTestAfterDelay):
(WebKitTests::CCThreadedTest::postSetNeedsAnimateToMainThread):
(WebKitTests::CCThreadedTest::postAddAnimationToMainThread):
(WebKitTests::CCThreadedTest::postAddInstantAnimationToMainThread):
(WebKitTests::CCThreadedTest::postSetNeedsCommitToMainThread):
(WebKitTests::CCThreadedTest::postAcquireLayerTextures):
(WebKitTests::CCThreadedTest::postSetNeedsRedrawToMainThread):
(WebKitTests::CCThreadedTest::postSetNeedsAnimateAndCommitToMainThread):
(WebKitTests::CCThreadedTest::postSetVisibleToMainThread):
(WebKitTests::CCThreadedTest::postDidAddAnimationToMainThread):
(WebKitTests::CCThreadedTest::doBeginTest):
(WebKitTests::CCThreadedTest::timeout):
(WebKitTests::CCThreadedTest::scheduleComposite):
(WebKitTests::CCThreadedTest::onEndTest):
(WebKitTests::CCThreadedTest::dispatchSetNeedsAnimate):
(WebKitTests::CCThreadedTest::dispatchAddInstantAnimation):
(WebKitTests::CCThreadedTest::dispatchAddAnimation):
(WebKitTests::CCThreadedTest::dispatchSetNeedsAnimateAndCommit):
(WebKitTests::CCThreadedTest::dispatchSetNeedsCommit):
(WebKitTests::CCThreadedTest::dispatchAcquireLayerTextures):
(WebKitTests::CCThreadedTest::dispatchSetNeedsRedraw):
(WebKitTests::CCThreadedTest::dispatchSetVisible):
(WebKitTests::CCThreadedTest::dispatchSetInvisible):
(WebKitTests::CCThreadedTest::dispatchComposite):
(WebKitTests::CCThreadedTest::dispatchDidAddAnimation):
(WebKitTests::CCThreadedTest::runTest):

  • tests/CCThreadedTest.h: Added.

(WebCore):
(WebKit):
(WebKitTests):
(TestHooks):
(WebKitTests::TestHooks::beginCommitOnCCThread):
(WebKitTests::TestHooks::commitCompleteOnCCThread):
(WebKitTests::TestHooks::prepareToDrawOnCCThread):
(WebKitTests::TestHooks::drawLayersOnCCThread):
(WebKitTests::TestHooks::animateLayers):
(WebKitTests::TestHooks::willAnimateLayers):
(WebKitTests::TestHooks::applyScrollAndScale):
(WebKitTests::TestHooks::updateAnimations):
(WebKitTests::TestHooks::layout):
(WebKitTests::TestHooks::didRecreateContext):
(WebKitTests::TestHooks::didAddAnimation):
(WebKitTests::TestHooks::didCommit):
(WebKitTests::TestHooks::didCommitAndDrawFrame):
(WebKitTests::TestHooks::scheduleComposite):
(WebKitTests::TestHooks::notifyAnimationStarted):
(WebKitTests::TestHooks::notifyAnimationFinished):
(CCThreadedTest):
(WebKitTests::CCThreadedTest::clearTimeout):
(WebKitTests::CCThreadedTest::clearEndTestTask):
(WebKitTests::CCThreadedTest::layerTreeHost):
(CCThreadedTestThreadOnly):
(WebKitTests::CCThreadedTestThreadOnly::runTestThreaded):
(MockLayerTreeHostImpl):
(CompositorFakeWebGraphicsContext3DWithTextureTracking):

7:00 PM Changeset in webkit [119885] by ojan@chromium.org
  • 1 edit
    8 adds in trunk/LayoutTests

Last set of chromium rebaselines missed in r119875.

  • platform/chromium-linux/svg/custom/js-repaint-rect-on-path-with-stroke-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-container-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-polygon-changes-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-polygon-removal-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-transform-addition-expected.png: Added.
  • platform/chromium-linux/svg/custom/js-update-transform-changes-expected.png: Added.
  • platform/chromium-linux/svg/custom/marker-referencePoint-expected.png: Added.
  • platform/chromium-linux/svg/custom/non-scaling-stroke-markers-expected.png: Added.
6:56 PM Changeset in webkit [119884] by commit-queue@webkit.org
  • 7 edits
    4 adds in trunk

Caret is not rendered in empty inline contenteditable elements
​https://bugs.webkit.org/show_bug.cgi?id=85793

Patch by Shezan Baig <​shezbaig.wk@gmail.com> on 2012-06-08
Reviewed by Ryosuke Niwa.

Source/WebCore:

Override localCaretRect in RenderInline. The implementation was almost
identical to localCaretRect in RenderBlock for empty block elements, so
I refactored RenderBlock::localCaretRect and moved the logic to a new
method 'localCaretRectForEmptyElement' in RenderBoxModelObject. The
implementation of 'localCaretRect' in RenderBlock and RenderInline both
use this helper method in RenderBoxModelObject.

Tests: editing/selection/caret-in-empty-inline-1.html

editing/selection/caret-in-empty-inline-2.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::localCaretRect):
Modified to use RenderBoxModelObject::localCaretRectForEmptyElement.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::localCaretRectForEmptyElement):
(WebCore):

  • rendering/RenderBoxModelObject.h:

(RenderBoxModelObject):
Add localCaretRectForEmptyElement helper method.

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::localCaretRect):
(WebCore):

  • rendering/RenderInline.h:

(RenderInline):
Override localCaretRect using localCaretRectForEmptyElement.

LayoutTests:

Add test cases for caret in empty inline.

  • editing/selection/caret-in-empty-inline-1-expected.txt: Added.
  • editing/selection/caret-in-empty-inline-1.html: Added.
  • editing/selection/caret-in-empty-inline-2-expected.txt: Added.
  • editing/selection/caret-in-empty-inline-2.html: Added.
6:51 PM Changeset in webkit [119883] by commit-queue@webkit.org
  • 5 edits
    4 adds in trunk

Source/WebCore: Treat blob: and filesystem: URLs generated via secure origins as secure.
​https://bugs.webkit.org/show_bug.cgi?id=84054

Loading a blob: or filesystem: URL into an iframe or image that's
contained on an HTTPS page shouldn't generate a mixed content warning.
This change adds a SecurityOrigin::isSecure to check both against a
URLs protocol, and the protocol of it's so-called "inner URL" if it's
the type of URL that has such a thing. These sorts of URLs which are
generated from secure sources will themselves be treated as secure.

Patch by Mike West <​mkwst@chromium.org> on 2012-06-08
Reviewed by Adam Barth.

Tests: http/tests/security/mixedContent/blob-url-in-iframe.html

http/tests/security/mixedContent/filesystem-url-in-iframe.html

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::isMixedContent):

  • page/SecurityOrigin.cpp:

(WebCore):
(WebCore::SecurityOrigin::isSecure):

  • page/SecurityOrigin.h:

(SecurityOrigin):

LayoutTests: Excluding blob: and filesystem: schemes from the mixed content check.
​https://bugs.webkit.org/show_bug.cgi?id=84054

Patch by Mike West <​mkwst@chromium.org> on 2012-06-08
Reviewed by Adam Barth.

  • http/tests/security/mixedContent/blob-url-in-iframe-expected.txt: Added.
  • http/tests/security/mixedContent/blob-url-in-iframe.html: Added.
  • http/tests/security/mixedContent/filesystem-url-in-iframe-expected.txt: Added.
  • http/tests/security/mixedContent/filesystem-url-in-iframe.html: Added.
5:56 PM Changeset in webkit [119882] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[Cairo] [TextureMapperGL] Nvidia card does not update WebGL texture when interacting with canvas
​https://bugs.webkit.org/show_bug.cgi?id=87797

Patch by Martin Robinson <​mrobinson@igalia.com> on 2012-06-08
Reviewed by Philippe Normand.

No new tests. WebGL is already covered by WebGL tests and this
bug is only present on certain hardware.

After resolving multisampling before drawing the texture via the
TextureMapper, first rebind the multisampling framebuffer. This ensures
that the texture isn't active in two framebuffers at a time.

  • platform/graphics/cairo/GraphicsContext3DPrivate.cpp:

(WebCore::GraphicsContext3DPrivate::paintToTextureMapper): Rebind the multisampling
framebuffer.

5:46 PM Changeset in webkit [119881] by bfulgham@webkit.org
  • 2 edits in trunk/Source/WebKit2

[WinCairo] Build fix after Windows export definition file change.

  • win/WebKit2CFLite.def: This file needs to be kept in sync with

WebKit2.def (aside from the WebKitInterface exports.)

5:12 PM Changeset in webkit [119880] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Fix crash on PagePopupChromeClient
​https://bugs.webkit.org/show_bug.cgi?id=88675

Patch by Crystal Zhang <​haizhang@rim.com> on 2012-06-08
Reviewed by Antonio Gomes.

PR 163672.

Actually it's the bug inside InputHandler, should delete the old popup and create a new one,
because update() is problematic. Also no need to save pointer in InputHandler.

  • WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::openPagePopup):
(WebCore::ChromeClientBlackBerry::closePagePopup):

  • WebKitSupport/InputHandler.cpp:

(BlackBerry::WebKit::InputHandler::InputHandler):
(BlackBerry::WebKit::InputHandler::~InputHandler):
(BlackBerry::WebKit::InputHandler::openSelectPopup):

  • WebKitSupport/InputHandler.h:

(WebCore):
(InputHandler):

5:07 PM Changeset in webkit [119879] by efidler@rim.com
  • 3 edits
    2 adds in trunk

[BlackBerry] Default font settings are getting deleted
​https://bugs.webkit.org/show_bug.cgi?id=88670

Reviewed by Rob Buis.

Source/WebKit/blackberry:

RIM PR: 163720

WebSettings is overriding the default font settings with "", which deletes them.

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::didChangeSettings):

LayoutTests:

This test crashes if -webkit-body (i.e. WebCore::Settings::standardFontFamily())
is not a valid font family.

  • fonts/valid-standard-font.html: Added.
4:57 PM Changeset in webkit [119878] by ggaren@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

Unreviewed, rolling back in part1 of r118646.

This patch includes everything necessary for lazy finalization, but
keeps eager finalization enabled for the time being.

Weak pointer finalization should be lazy
​https://bugs.webkit.org/show_bug.cgi?id=87599

Reviewed by Sam Weinig.

  • heap/MarkedBlock.cpp:
  • heap/MarkedBlock.h:

(JSC::MarkedBlock::resetAllocator):

  • heap/MarkedSpace.cpp:

(JSC::MarkedSpace::resetAllocators):

  • heap/MarkedSpace.h:

(JSC::MarkedSpace::resetAllocators): Don't force allocator reset anymore.
It will happen automatically when a weak set is swept. It's simpler to
have only one canonical way for this to happen, and it wasn't buying
us anything to do it eagerly.

  • heap/WeakBlock.cpp:

(JSC::WeakBlock::sweep): Don't short-circuit a sweep unless we know
the sweep would be a no-op. If even one finalizer is pending, we need to
run it, since we won't get another chance.

  • heap/WeakSet.cpp:

(JSC::WeakSet::sweep): This loop can be simpler now that
WeakBlock::sweep() does what we mean.
Reset our allocator after a sweep because this is the optimal time to
start trying to recycle old weak pointers.
(JSC::WeakSet::tryFindAllocator): Don't sweep when searching for an
allocator because we've swept already, and forcing a new sweep would be
wasteful.

  • heap/WeakSet.h:

(JSC::WeakSet::shrink): Be sure to reset our allocator after a shrink
because the shrink may have removed the block the allocator was going to
allocate out of.

4:54 PM Changeset in webkit [119877] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

<rdar://problem/11412294> Add appropriate iOS default font settings

Reviewed by Dan Bernstein.

  • page/mac/SettingsMac.mm: (WebCore::Settings::initializeDefaultFontFamilies):
4:41 PM Changeset in webkit [119876] by charles.wei@torchmobile.com.cn
  • 3 edits in trunk/Source/WebCore

IndexedDB: Transactions without any request scheduled should abort itself.
​https://bugs.webkit.org/show_bug.cgi?id=88052

Reviewed by Kentaro Hara.

This is needed for indexedDB to make it work for JSC. We need to clean up all the
pending transactions(transactions without any request scheduled) when leaving Javascript context.

Manually tested using the existing test LayoutTests/storage/indexeddb/tutorial.html

  • bindings/js/JSMainThreadExecState.cpp:

(WebCore):
(WebCore::JSMainThreadExecState::didLeaveScriptContext):

  • bindings/js/JSMainThreadExecState.h:

(WebCore::JSMainThreadExecState::~JSMainThreadExecState):
(JSMainThreadExecState):

4:18 PM Changeset in webkit [119875] by ojan@chromium.org
  • 34 edits
    9 copies
    8 adds
    1 delete in trunk/LayoutTests

Final set of chromium rebaselines for skia change. See crbug.com/131829.

  • platform/chromium-linux/svg/clip-path/clipper-placement-issue-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-update-container-expected.png.
  • platform/chromium-linux/svg/css/arrow-with-shadow-expected.png: Added.
  • platform/chromium-linux/svg/css/background-image-svg-expected.png: Added.
  • platform/chromium-linux/svg/custom/broken-internal-references-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/marker-strokeWidth-changes-expected.png.
  • platform/chromium-linux/svg/custom/dynamic-empty-path-expected.png: Added.
  • platform/chromium-linux/svg/custom/gradient-stop-corner-cases-expected.png: Added.
  • platform/chromium-linux/svg/custom/svgpolyparser-extra-space-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-update-container-expected.png.
  • platform/chromium-linux/svg/custom/use-forward-refs-expected.png: Added.
  • platform/chromium-linux/svg/custom/use-on-clip-path-with-transformation-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-repaint-rect-on-path-with-stroke-expected.png.
  • platform/chromium-linux/svg/text/text-hkern-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-update-container-expected.png.
  • platform/chromium-linux/svg/text/text-hkern-on-vertical-text-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-update-container-expected.png.
  • platform/chromium-linux/svg/text/text-vkern-expected.png: Added.
  • platform/chromium-linux/svg/text/text-vkern-on-horizontal-text-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/marker-strokeWidth-changes-expected.png.
  • platform/chromium-linux/svg/zoom/page/absolute-sized-document-no-scrollbars-expected.png: Added.
  • platform/chromium-linux/svg/zoom/page/relative-sized-document-scrollbars-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-repaint-rect-on-path-with-stroke-expected.png.
  • platform/chromium-linux/svg/zoom/page/zoom-background-image-tiled-expected.png: Added.
  • platform/chromium-linux/svg/zoom/page/zoom-svg-as-relative-image-expected.png: Copied from LayoutTests/platform/chromium-win/svg/custom/js-update-container-expected.png.
  • platform/chromium-win-xp/platform/chromium/compositing/accelerated-drawing/svg-filters-expected.png: Removed.
  • platform/chromium-win/svg/custom/js-late-marker-creation-expected.png:
  • platform/chromium-win/svg/custom/js-repaint-rect-on-path-with-stroke-expected.png:
  • platform/chromium-win/svg/custom/js-update-bounce-expected.png:
  • platform/chromium-win/svg/custom/js-update-container-expected.png:
  • platform/chromium-win/svg/custom/js-update-path-changes-expected.png:
  • platform/chromium-win/svg/custom/js-update-path-removal-expected.png:
  • platform/chromium-win/svg/custom/js-update-polygon-changes-expected.png:
  • platform/chromium-win/svg/custom/js-update-polygon-removal-expected.png:
  • platform/chromium-win/svg/custom/js-update-transform-addition-expected.png:
  • platform/chromium-win/svg/custom/js-update-transform-changes-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-preserveAspectRatio-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-transform-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-viewBox-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-viewBox-transform-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-viewTarget-expected.png:
  • platform/chromium-win/svg/custom/linking-a-03-b-zoomAndPan-expected.png:
  • platform/chromium-win/svg/custom/linking-uri-01-b-expected.png:
  • platform/chromium-win/svg/custom/marker-changes-expected.png:
  • platform/chromium-win/svg/custom/marker-child-changes-css-expected.png:
  • platform/chromium-win/svg/custom/marker-child-changes-expected.png:
  • platform/chromium-win/svg/custom/marker-opacity-expected.png:
  • platform/chromium-win/svg/custom/marker-referencePoint-expected.png:
  • platform/chromium-win/svg/custom/marker-strokeWidth-changes-expected.png:
  • platform/chromium-win/svg/custom/marker-viewBox-changes-expected.png:
  • platform/chromium-win/svg/custom/mask-colorspace-expected.png:
  • platform/chromium-win/svg/custom/mask-on-multiple-objects-expected.png:
  • platform/chromium-win/svg/custom/no-inherited-dashed-stroke-expected.png:
  • platform/chromium-win/svg/custom/non-circular-marker-reference-expected.png:
  • platform/chromium-win/svg/custom/non-scaling-stroke-markers-expected.png:
  • platform/chromium-win/svg/custom/object-sizing-explicit-height-expected.png:
  • platform/chromium-win/svg/custom/object-sizing-explicit-width-height-expected.png:
  • platform/chromium-win/svg/filters/invalidate-on-child-layout-expected.png:
  • platform/chromium-win/svg/foreignObject/text-tref-02-b-expected.png:
  • platform/chromium/TestExpectations:
4:13 PM Changeset in webkit [119874] by Martin Robinson
  • 2 edits in trunk

Fix the GTK+ build when OpenGL is enabled.

  • configure.ac: Fix the build.
4:13 PM Changeset in webkit [119873] by ojan@chromium.org
  • 2 edits in trunk/LayoutTests

viewspec-parser tests still timeout in Chromium debug even when marked slow.

  • platform/chromium/TestExpectations:
4:09 PM Changeset in webkit [119872] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Crash in WebCore::CompositeEditCommand::breakOutOfEmptyListItem
​https://bugs.webkit.org/show_bug.cgi?id=88361

Reviewed by Levi Weintraub.

Use RefPtr instead of raw pointers across DOM mutations.

No tests are added for the lack of a reliable reduction.

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::breakOutOfEmptyListItem):

4:00 PM Changeset in webkit [119871] by commit-queue@webkit.org
  • 6 edits
    4 deletes in trunk

Unreviewed, rolling out r119514.
​http://trac.webkit.org/changeset/119514
​https://bugs.webkit.org/show_bug.cgi?id=88664

Broke JSFiddle.net (Requested by arv on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-06-08

Source/WebCore:

  • bindings/v8/V8DOMWindowShell.cpp:

(WebCore::V8DOMWindowShell::initContextIfNeeded):

  • bindings/v8/WorkerContextExecutionProxy.cpp:

(WebCore::WorkerContextExecutionProxy::initIsolate):

LayoutTests:

  • fast/dom/Window/window-property-shadowing-name-expected.txt:
  • fast/dom/Window/window-property-shadowing-name.html:
  • fast/dom/Window/window-property-shadowing-onclick-expected.txt: Removed.
  • fast/dom/Window/window-property-shadowing-onclick.html: Removed.
  • platform/chromium/fast/dom/Window/window-property-shadowing-name-expected.txt: Removed.
  • platform/chromium/fast/dom/Window/window-property-shadowing-onclick-expected.txt: Removed.
3:35 PM Changeset in webkit [119870] by rniwa@webkit.org
  • 6 edits in trunk/Source/WebCore

Crash in WebCore::InsertParagraphSeparatorCommand::doApply
​https://bugs.webkit.org/show_bug.cgi?id=88108

Reviewed by Levi Weintraub.

Use NodeVector instead of walking through siblings as we mutate the DOM.

No new tests are added since there is no reliable reduction.

  • editing/BreakBlockquoteCommand.cpp:

(WebCore::BreakBlockquoteCommand::doApply):

  • editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::moveRemainingSiblingsToNewParent):
(WebCore):

  • editing/CompositeEditCommand.h:

(CompositeEditCommand):

  • editing/InsertParagraphSeparatorCommand.cpp:

(WebCore::InsertParagraphSeparatorCommand::doApply):

3:24 PM Changeset in webkit [119869] by Martin Robinson
  • 2 edits in trunk

[GTK] build accelerated compositing on by default if OpenGL is present
​https://bugs.webkit.org/show_bug.cgi?id=88677

Reviewed by Alejandro G. Castro.

Build accelerated compositing by default if OpenGL is present, just like WebGL.
Also prevent enabling WebGL if Clutter is turned on.

  • configure.ac: Build AC by default.
2:59 PM Changeset in webkit [119868] by dgrogan@chromium.org
  • 12 edits in trunk/Source

IndexedDB: rename some instances of open to registerFrontendCallbacks
​https://bugs.webkit.org/show_bug.cgi?id=88611

Source/WebCore:

We should still make the change in the WebKit API.

Reviewed by Tony Chang.

No new tests, just a method rename.

  • Modules/indexeddb/IDBDatabase.cpp:

(WebCore::IDBDatabase::registerFrontendCallbacks):

  • Modules/indexeddb/IDBDatabase.h:

(IDBDatabase):

  • Modules/indexeddb/IDBDatabaseBackendImpl.cpp:

(WebCore::IDBDatabaseBackendImpl::registerFrontendCallbacks):

  • Modules/indexeddb/IDBDatabaseBackendImpl.h:

(IDBDatabaseBackendImpl):

  • Modules/indexeddb/IDBDatabaseBackendInterface.h:

(IDBDatabaseBackendInterface):

  • Modules/indexeddb/IDBRequest.cpp:

(WebCore::IDBRequest::onSuccess):

Source/WebKit/chromium:

Reviewed by Tony Chang.

  • src/IDBDatabaseBackendProxy.cpp:

(WebKit::IDBDatabaseBackendProxy::registerFrontendCallbacks):

  • src/IDBDatabaseBackendProxy.h:

(IDBDatabaseBackendProxy):

  • src/WebIDBDatabaseImpl.cpp:

(WebKit::WebIDBDatabaseImpl::open):

  • src/WebIDBDatabaseImpl.h:

(WebIDBDatabaseImpl):

2:37 PM Changeset in webkit [119867] by danakj@chromium.org
  • 5 edits in trunk/Source

[chromium] Skip willDraw() and didDraw() on fully occluded layers
​https://bugs.webkit.org/show_bug.cgi?id=88435

Reviewed by Adrienne Walker.

Source/WebCore:

Current willDraw() is called on all layers with non-empty
visibleLayerRect and non-empty scissorRect. This excludes
layers outside the viewport, but does not exclude occluded
layers. We add a check for occlusion to calculateRenderPasses
in order to avoid willDraw() when it will be culled anyway.

We prevent didDraw() from being called for occluded layers, for
which we did not call didDraw() by holding a vector of layers
for which we did call willDraw(). This lets us avoid storing a
flag on the layers, or computing occlusion again in
didDrawAllLayers.

Unit test: CCLayerTreeHostImplTest.willDrawNotCalledOnOccludedLayer

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

(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
(WebCore::CCLayerTreeHostImpl::prepareToDraw):
(WebCore::CCLayerTreeHostImpl::didDrawAllLayers):

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

(FrameData):
(CCLayerTreeHostImpl):

Source/WebKit/chromium:

  • tests/CCLayerTreeHostImplTest.cpp:
2:37 PM Changeset in webkit [119866] by ojan@chromium.org
  • 462 edits
    4 copies
    9 moves
    10 adds
    221 deletes in trunk/LayoutTests

Second wave of Chromium rebaselines after skia change. See crbug.com/131829.

List of changed expectations deleted because it's too long.

2:30 PM Changeset in webkit [119865] by barraclough@apple.com
  • 8 edits in trunk/Source/JavaScriptCore

Unreviewed roll out r119795.

This broke jquery/core.html

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::emitAllocateBasicJSObject):

  • jit/JITInlineMethods.h:

(JSC::JIT::emitAllocateBasicJSObject):

  • llint/LowLevelInterpreter.asm:
  • runtime/JSGlobalData.h:

(JSGlobalData):

  • runtime/JSGlobalThis.cpp:

(JSC::JSGlobalThis::setUnwrappedObject):

  • runtime/JSObject.cpp:

(JSC::JSObject::visitChildren):
(JSC::JSObject::createInheritorID):

  • runtime/JSObject.h:

(JSObject):
(JSC::JSObject::resetInheritorID):
(JSC):
(JSC::JSObject::offsetOfInheritorID):
(JSC::JSObject::inheritorID):

2:13 PM Changeset in webkit [119864] by mitz@apple.com
  • 1 edit
    2 adds
    2 deletes in trunk/LayoutTests

Added Mac pixel results for canvas-imageSmoothingEnabled-patterns.html, and made the text
results shared between platforms.

  • fast/canvas/canvas-imageSmoothingEnabled-patterns-expected.txt: Added.
  • platform/chromium/fast/canvas/canvas-imageSmoothingEnabled-patterns-expected.txt: Removed.
  • platform/gtk/fast/canvas/canvas-imageSmoothingEnabled-patterns-expected.txt: Removed.
  • platform/mac/fast/canvas/canvas-imageSmoothingEnabled-patterns-expected.png: Added.
2:12 PM Changeset in webkit [119863] by tony@chromium.org
  • 3 edits in trunk/LayoutTests

[chromium] Unreviewed, rebaseline a chromium-win result that changed
due to the various mock scrollbar changes.
​https://bugs.webkit.org/show_bug.cgi?id=88490

  • platform/chromium-win/fast/repaint/fixed-tranformed-expected.png:
  • platform/chromium/TestExpectations:
1:37 PM Changeset in webkit [119862] by tonikitoo@webkit.org
  • 2 edits in trunk/Source/WebCore

EventHandler shouldn't schedule the fake mousemove event timer when scrolling on devices that don't have a mouse
​https://bugs.webkit.org/show_bug.cgi?id=88379

Reviewed by James Robinson.
Patch by Antonio Gomes <​agomes@rim.com>

Paraphrasing Andy Estes:
"In <​http://trac.webkit.org/changeset/119465> we stopped dispatching fake
mousemove events when scrolling on devices that don't support mice. This event
is dispatched on a timer, so one better would be to not even schedule the
timer (rather than scheduling the timer but making it a no-op).

No new tests, as no functionality has changed. We basically do not
start a timer under a given circumstance instead of fire it, it times out,
and we make the callback no-op when under the given circumstance.

It is still possible though that we have the following scenario:

  • Setting::deviceSupportsMouse set as true;
  • we fire the timer;
  • before its callback gets executed (timed out) the setting gets toggled.

So for sakeness, lets keep the check in both places.

  • page/EventHandler.cpp:

(WebCore::EventHandler::dispatchFakeMouseMoveEventSoon):
(WebCore::EventHandler::fakeMouseMoveEventTimerFired):

1:31 PM Changeset in webkit [119861] by jamesr@google.com
  • 4 edits in trunk/Source

[chromium] Clean up some unnecessary LayerChromium.h includes
​https://bugs.webkit.org/show_bug.cgi?id=88599

Reviewed by Adam Barth.

Source/WebCore:

Reworks test code slightly to use WebCore interfaces.

  • testing/Internals.cpp:

(WebCore::Internals::setBackgroundBlurOnNode):

Source/WebKit/chromium:

Removes unused include, update comment.

  • src/WebMediaPlayerClientImpl.cpp:

(WebKit::WebMediaPlayerClientImpl::platformLayer):
(WebKit::WebMediaPlayerClientImpl::paint):

1:13 PM Changeset in webkit [119860] by schenney@chromium.org
  • 1184 edits
    93 adds
    229 deletes in trunk/LayoutTests

WebCore::ImageBuffer.cpp has broken color table code
​https://bugs.webkit.org/show_bug.cgi?id=80321

Unreviewed Chromium test expectations update after changes to SVG filter color conversion.

Full list of changed files excluded to to very large number.

  • platform/chromium/TestExpectations:
1:04 PM Changeset in webkit [119859] by fpizlo@apple.com
  • 1 edit in trunk/Source/JavaScriptCore/ChangeLog

Fix JavaScriptCore/ChangeLog goof, where Geoff's change did not have a date + name + email header

1:02 PM Changeset in webkit [119858] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

PredictedType should be called SpeculatedType
​https://bugs.webkit.org/show_bug.cgi?id=88477

Unreviewed, fix a renaming goof from ​http://trac.webkit.org/changeset/119660.
I accidentally renamed ByteCodeParser::getPrediction to
ByteCodeParser::getSpeculation. That was not the intent. This changes it
back.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::addCall):
(JSC::DFG::ByteCodeParser::getPredictionWithoutOSRExit):
(JSC::DFG::ByteCodeParser::getPrediction):
(JSC::DFG::ByteCodeParser::handleCall):
(JSC::DFG::ByteCodeParser::parseBlock):

12:57 PM Changeset in webkit [119857] by wingo@igalia.com
  • 13 edits in trunk/Source

Explictly mark stubs called by JIT as being internal
​https://bugs.webkit.org/show_bug.cgi?id=88552

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

  • dfg/DFGOSRExitCompiler.h:
  • dfg/DFGOperations.cpp:
  • dfg/DFGOperations.h:
  • jit/HostCallReturnValue.h:
  • jit/JITStubs.cpp:
  • jit/JITStubs.h:
  • jit/ThunkGenerators.cpp:
  • llint/LLIntSlowPaths.h: Mark a bunch of stubs as being

WTF_INTERNAL. Change most calls to SYMBOL_STRING_RELOCATION to
LOCAL_REFERENCE, or GLOBAL_REFERENCE in the case of the wrappers
to truly global symbols.

  • offlineasm/asm.rb: Generate LOCAL_REFERENCE instead of

SYMBOL_STRING_RELOCATION.

Don't rely on weak pointers for eager CodeBlock finalization
​https://bugs.webkit.org/show_bug.cgi?id=88465

Reviewed by Gavin Barraclough.

This is incompatible with lazy weak pointer finalization.

I considered just making CodeBlock finalization lazy-friendly, but it
turns out that the heap is already way up in CodeBlock's business when
it comes to finalization, so I decided to finish the job and move full
responsibility for CodeBlock finalization into the heap.

will build.

  • debugger/Debugger.cpp: Updated for rename.
  • heap/Heap.cpp:

(JSC::Heap::deleteAllCompiledCode): Renamed for consistency. Fixed a bug
where we would not delete code for a code block that had been previously
jettisoned. I don't know if this happens in practice -- I mostly did
this to improve consistency with deleteUnmarkedCompiledCode.

(JSC::Heap::deleteUnmarkedCompiledCode): New function, responsible for
eager finalization of unmarked code blocks.

(JSC::Heap::collect): Updated for rename. Updated to call
deleteUnmarkedCompiledCode(), which takes care of jettisoned DFG code
blocks too.

(JSC::Heap::addCompiledCode): Renamed, since this points to all code
now, not just functions.

  • heap/Heap.h:

(Heap): Keep track of all user code, not just functions. This is a
negligible additional overhead, since most code is function code.

  • runtime/Executable.cpp:

(JSC::*::finalize): Removed these functions, since we don't rely on
weak pointer finalization anymore.

(JSC::FunctionExecutable::FunctionExecutable): Moved linked-list stuff
into base class so all executables can be in the list.

(JSC::EvalExecutable::clearCode):
(JSC::ProgramExecutable::clearCode):
(JSC::FunctionExecutable::clearCode): All we need to do is delete our
CodeBlock -- that will delete all of its internal data structures.

(JSC::FunctionExecutable::clearCodeIfNotCompiling): Factored out a helper
function to improve clarity.

  • runtime/Executable.h:

(JSC::ExecutableBase): Moved linked-list stuff
into base class so all executables can be in the list.

(JSC::NativeExecutable::create):
(NativeExecutable):
(ScriptExecutable):
(JSC::ScriptExecutable::finishCreation):
(JSC::EvalExecutable::create):
(EvalExecutable):
(JSC::ProgramExecutable::create):
(ProgramExecutable):
(FunctionExecutable):
(JSC::FunctionExecutable::create): Don't use a finalizer -- the heap
will call us back to destroy our code block.

(JSC::FunctionExecutable::discardCode): Renamed to clearCodeIfNotCompiling()
for clarity.

(JSC::FunctionExecutable::isCompiling): New helper function, for clarity.

(JSC::ScriptExecutable::clearCodeVirtual): New helper function, since
the heap needs to make polymorphic calls to clear code.

  • runtime/JSGlobalData.cpp:

(JSC::StackPreservingRecompiler::operator()):

  • runtime/JSGlobalObject.cpp:

(JSC::DynamicGlobalObjectScope::DynamicGlobalObjectScope): Updated for
renames.

Source/WTF:

  • wtf/ExportMacros.h (WTF_INTERNAL, HAVE_INTERNAL_VISIBILITY): New

defines. Regardless of what the port does about visibility in
general, for code referenced only from assembly it is useful to
give it internal visibility.

  • wtf/InlineASM.h: Split SYMBOL_STRING_RELOCATION into

LOCAL_REFERENCE and GLOBAL_REFERENCE; the former will try to avoid
indirection if HAVE(INTERNAL_VISIBILITY).

12:49 PM Changeset in webkit [119856] by jsbell@chromium.org
  • 6 edits in trunk/LayoutTests

IndexedDB: Add regression test to verify that legacy constants have been removed
​https://bugs.webkit.org/show_bug.cgi?id=88669

Reviewed by Tony Chang.

  • storage/indexeddb/legacy-constants-expected.txt:
  • storage/indexeddb/mozilla/create-index-unique-expected.txt:
  • storage/indexeddb/mozilla/resources/create-index-unique.js:

(createAndVerifyIndex):

  • storage/indexeddb/resources/constants.js:

(test):

  • storage/indexeddb/resources/legacy-constants.js:

(testObsoleteConstants):

12:25 PM Changeset in webkit [119855] by ojan@chromium.org
  • 1 edit in trunk/LayoutTests/ChangeLog

First wave of Chromium rebaselines after skia rollback. See crbug.com/131829.

List of changed expected files excluded because it's too long.

12:19 PM Changeset in webkit [119854] by ojan@chromium.org
  • 535 edits
    21 copies
    16 moves
    218 adds
    15 deletes in trunk/LayoutTests

First wave of Chromium rebaselines after skia rollback. See crbug.com/131289.

List of changed expected files excluded because it's too long.

12:05 PM Changeset in webkit [119853] by arv@chromium.org
  • 8 edits
    1 copy in trunk

[V8] Keep TextTrackList alive as long as its owner is alive
​https://bugs.webkit.org/show_bug.cgi?id=88541

Reviewed by Adam Barth.

Add a hidden reference from the owner to the text track list.

Source/WebCore:

Covered by existing tests.

  • Target.pri: Add V8TextTrackListCustom.cpp.
  • UseV8.cmake: Ditto.
  • WebCore.gypi: Ditto.
  • bindings/v8/V8HiddenPropertyName.h:

(WebCore):

  • bindings/v8/custom/V8TextTrackListCustom.cpp: Copied from Source/WebCore/bindings/v8/V8HiddenPropertyName.h.

(WebCore):
(WebCore::toV8): When the wrapper is created add a reference to the owner of the TextTrackList to the TextTrackList.

  • html/track/TextTrackList.idl:

LayoutTests:

  • platform/chromium/TestExpectations:
11:53 AM Changeset in webkit [119852] by jochen@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

  • DEPS:
11:44 AM Changeset in webkit [119851] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

Add app_abi option for DumpRenderTree_apk
​https://bugs.webkit.org/show_bug.cgi?id=88626

Patch by Wei James <​james.wei@intel.com> on 2012-06-08
Reviewed by Adam Barth.

  • DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
11:41 AM Changeset in webkit [119850] by commit-queue@webkit.org
  • 20 edits
    2 adds in trunk/Source

[chromium] Accelerated animations should use WebTransformOperations
​https://bugs.webkit.org/show_bug.cgi?id=87686

Patch by Ian Vollick <​vollick@chromium.org> on 2012-06-08
Reviewed by James Robinson.

CCTransformKeyframe new owns a WebTransformOperations rather than a
TransformOperations. LayerChromium's API has been changed so that
LayerChromium::addAnimation should take only a CCActiveAnimation.
GraphicsLayerChromium is new responsible for translating to
WebTransformOperations and creating CCActiveAnimations. Tests that use
the public API (that is, they call addAnimation with KeyframeValueList
and Animation arguments) have been moved to GraphicsLayerChromiumTest.

Source/WebCore:

Unit tests:

GraphicsLayerChromiumTest.createOpacityAnimation
GraphicsLayerChromiumTest.createTransformAnimation
GraphicsLayerChromiumTest.createTransformAnimationWithBigRotation
GraphicsLayerChromiumTest.createTransformAnimationWithSingularMatrix
GraphicsLayerChromiumTest.createReversedAnimation
GraphicsLayerChromiumTest.createAlternatingAnimation
GraphicsLayerChromiumTest.createReversedAlternatingAnimation

  • WebCore.gypi:
  • platform/graphics/chromium/AnimationTranslationUtil.cpp: Added.

(WebCore):
(WebCore::toWebTransformOperations):
(WebCore::appendKeyframe):
(WebCore::CCKeyframedTransformAnimationCurve):
(WebCore::createActiveAnimation):

  • platform/graphics/chromium/AnimationTranslationUtil.h: Added.

(WebCore):

  • platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::addAnimation):

  • platform/graphics/chromium/LayerChromium.cpp:

(WebCore::LayerChromium::addAnimation):

  • platform/graphics/chromium/LayerChromium.h:

(WebCore):
(WebCore::LayerChromium::bounds):
(LayerChromium):

  • platform/graphics/chromium/LinkHighlight.cpp:

(WebCore::LinkHighlight::LinkHighlight):

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

(CCTransformAnimationCurve):

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

(WebCore::CCTransformKeyframe::create):
(WebCore::CCTransformKeyframe::CCTransformKeyframe):
(WebCore::CCTransformKeyframe::value):
(WebCore::CCTransformKeyframe::clone):
(WebCore::CCKeyframedTransformAnimationCurve::getValue):

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

(CCTransformKeyframe):
(CCKeyframedTransformAnimationCurve):

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

(WebCore::CCLayerAnimationController::removeAnimation):
(WebCore):
(WebCore::CCLayerAnimationController::addAnimation):
(WebCore::CCLayerAnimationController::getActiveAnimation):
(WebCore::CCLayerAnimationController::pushNewAnimationsToImplThread):
(WebCore::CCLayerAnimationController::replaceImplThreadAnimations):
(WebCore::CCLayerAnimationController::tickAnimations):

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

(CCLayerAnimationControllerClient):
(CCLayerAnimationController):

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

(WebCore::CCLayerImpl::bounds):

Source/WebKit/chromium:

  • tests/CCAnimationTestCommon.cpp:

(WebCore::addOpacityTransition):
(WebCore::addAnimatedTransform):
(WebKitTests::FakeTransformTransition::getValue):

  • tests/CCAnimationTestCommon.h:
  • tests/CCKeyframedAnimationCurveTest.cpp:
  • tests/CCLayerAnimationControllerTest.cpp:
  • tests/CCLayerTreeHostTest.cpp:

(WTF::CCLayerTreeHostTestLayerAddedWithAnimation::beginTest):

  • tests/GraphicsLayerChromiumTest.cpp:

(MockLayerTreeHostClient):
(WebKitTests):
(MockLayerTreeHost):
(WebKitTests::MockLayerTreeHost::create):
(WebKitTests::MockLayerTreeHost::createLayerTreeHostImpl):
(WebKitTests::MockLayerTreeHost::MockLayerTreeHost):
(GraphicsLayerChromiumTest):
(WebKitTests::GraphicsLayerChromiumTest::GraphicsLayerChromiumTest):
(WebKitTests::GraphicsLayerChromiumTest::~GraphicsLayerChromiumTest):
(WebKitTests::GraphicsLayerChromiumTest::expectTranslateX):
(WebKitTests::TEST_F):

  • tests/LayerChromiumTest.cpp:
11:39 AM Changeset in webkit [119849] by noam.rosenthal@nokia.com
  • 6 edits in trunk/Source/WebCore

[Texmap] Accelerated versions of drop-shadow and blur filters
​https://bugs.webkit.org/show_bug.cgi?id=87695

Reviewed by Kenneth Rohde Christiansen.

Added shaders for blur and drop-shadow effects.
The shaders use sampling of multiple coordinates and averaging them with normal-
distribution to create a fast gaussian blur effect.

Covered by existing tests in css3/filters.

  • platform/graphics/texmap/TextureMapperGL.cpp:

(WebCore::TextureMapperGL::drawFiltered):

Call the prepare function explicitly, with a size argument that's used to calculate
some of the uniform values.

  • platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::intermediateSurfaceRect):

The outsets of the effect have to be considered when calculating the intermediate rect.
Otherwise the resulting image is scaled instead of outsetted.

(WebCore::applyFilters):

  • platform/graphics/texmap/TextureMapperShaderManager.cpp:

(WebCore::StandardFilterProgram::StandardFilterProgram):

Added the new shaders.

(WebCore::StandardFilterProgram::prepare):
(WebCore::TextureMapperShaderManager::getShaderForFilter):

  • platform/graphics/texmap/TextureMapperShaderManager.h:

(StandardFilterProgram):

11:36 AM Changeset in webkit [119848] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Tried to fix the build after r119844.

  • bindings/js/GCController.cpp:

(WebCore::GCController::discardAllCompiledCode):

11:34 AM Changeset in webkit [119847] by ostapenko.viatcheslav@nokia.com
  • 3 edits in trunk/Source/WebKit2

[Qt] [WK2] API tests randomly assert in WebLayerTreeRenderer::assignImageToLayer
​https://bugs.webkit.org/show_bug.cgi?id=88667

Reviewed by Noam Rosenthal.

Move deactivation of WebLayerTreeRenderer from LayerTreeHostProxy::purgeBackingStores
to WebLayerTreeRenderer::purgeGLResources. LayerTreeHostProxy::purgeBackingStores is
invoked on main thread after going through call gate and it is too late to deactivate
renderer here, because some update messages are able to go through.
WebLayerTreeRenderer::purgeGLResources is called directly from render node deallocation
and it is right place to deactivate renderer.

  • UIProcess/LayerTreeHostProxy.cpp:

(WebKit::LayerTreeHostProxy::purgeBackingStores):

  • UIProcess/WebLayerTreeRenderer.cpp:

(WebKit::WebLayerTreeRenderer::purgeGLResources):

11:24 AM Changeset in webkit [119846] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Add COMPILE_ASSERT to StyledElement to ensure it doesn't accidentally grow larger.
​https://bugs.webkit.org/show_bug.cgi?id=88627

Patch by Mike West <​mkwst@chromium.org> on 2012-06-08
Reviewed by Adam Barth.

No change in user-facing behavior.

  • dom/StyledElement.cpp:

(WebCore):

11:19 AM Changeset in webkit [119845] by tony@chromium.org
  • 2 edits
    3 adds in trunk/LayoutTests

[chromium] Unreviewed, rebaseline platform/chromium-linux/fast/text/chromium-linux-fontconfig-renderstyle.html after skia gamma correction
​https://bugs.webkit.org/show_bug.cgi?id=88589

  • platform/chromium-linux/platform/chromium-linux/fast/text/chromium-linux-fontconfig-renderstyle-expected.png: Added.
  • platform/chromium/TestExpectations:
11:17 AM Changeset in webkit [119844] by ggaren@apple.com
  • 9 edits in trunk/Source/JavaScriptCore

Don't rely on weak pointers for eager CodeBlock finalization
​https://bugs.webkit.org/show_bug.cgi?id=88465

Reviewed by Gavin Barraclough.

This is incompatible with lazy weak pointer finalization.

I considered just making CodeBlock finalization lazy-friendly, but it
turns out that the heap is already way up in CodeBlock's business when
it comes to finalization, so I decided to finish the job and move full
responsibility for CodeBlock finalization into the heap.

will build.

  • debugger/Debugger.cpp: Updated for rename.
  • heap/Heap.cpp:

(JSC::Heap::deleteAllCompiledCode): Renamed for consistency. Fixed a bug
where we would not delete code for a code block that had been previously
jettisoned. I don't know if this happens in practice -- I mostly did
this to improve consistency with deleteUnmarkedCompiledCode.

(JSC::Heap::deleteUnmarkedCompiledCode): New function, responsible for
eager finalization of unmarked code blocks.

(JSC::Heap::collect): Updated for rename. Updated to call
deleteUnmarkedCompiledCode(), which takes care of jettisoned DFG code
blocks too.

(JSC::Heap::addCompiledCode): Renamed, since this points to all code
now, not just functions.

  • heap/Heap.h:

(Heap): Keep track of all user code, not just functions. This is a
negligible additional overhead, since most code is function code.

  • runtime/Executable.cpp:

(JSC::*::finalize): Removed these functions, since we don't rely on
weak pointer finalization anymore.

(JSC::FunctionExecutable::FunctionExecutable): Moved linked-list stuff
into base class so all executables can be in the list.

(JSC::EvalExecutable::clearCode):
(JSC::ProgramExecutable::clearCode):
(JSC::FunctionExecutable::clearCode): All we need to do is delete our
CodeBlock -- that will delete all of its internal data structures.

(JSC::FunctionExecutable::clearCodeIfNotCompiling): Factored out a helper
function to improve clarity.

  • runtime/Executable.h:

(JSC::ExecutableBase): Moved linked-list stuff
into base class so all executables can be in the list.

(JSC::NativeExecutable::create):
(NativeExecutable):
(ScriptExecutable):
(JSC::ScriptExecutable::finishCreation):
(JSC::EvalExecutable::create):
(EvalExecutable):
(JSC::ProgramExecutable::create):
(ProgramExecutable):
(FunctionExecutable):
(JSC::FunctionExecutable::create): Don't use a finalizer -- the heap
will call us back to destroy our code block.

(JSC::FunctionExecutable::discardCode): Renamed to clearCodeIfNotCompiling()
for clarity.

(JSC::FunctionExecutable::isCompiling): New helper function, for clarity.

(JSC::ScriptExecutable::clearCodeVirtual): New helper function, since
the heap needs to make polymorphic calls to clear code.

  • runtime/JSGlobalData.cpp:

(JSC::StackPreservingRecompiler::operator()):

  • runtime/JSGlobalObject.cpp:

(JSC::DynamicGlobalObjectScope::DynamicGlobalObjectScope): Updated for
renames.

11:14 AM Changeset in webkit [119843] by rjkroege@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

More than 8 fingers get ignored by touch
​https://bugs.webkit.org/show_bug.cgi?id=88359

Reviewed by James Robinson.

  • public/WebInputEvent.h: Increase of TouchPoints per TouchEvent from 8 to 12.
10:59 AM Changeset in webkit [119842] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Added svg/filters/color-space-conversion.svg to the Mac skip list. It has been failing since
its addition in r119841. <​http://webkit.org/b/88672> is tracking the failure.

  • platform/mac/Skipped:
10:52 AM Changeset in webkit [119841] by schenney@chromium.org
  • 2 edits in trunk/LayoutTests

WebCore::ImageBuffer.cpp has broken color table code
​https://bugs.webkit.org/show_bug.cgi?id=80321

Unreviewed Chromium expectations update.

Adding tests that are failing pending a rebaseline.

  • platform/chromium/TestExpectations:
10:46 AM Changeset in webkit [119840] by tony@chromium.org
  • 2 edits in trunk/LayoutTests

[chromium] Unreviewed, remove css3/flexbox/child-overflow.html from TestExpectations.
Since bug 77368 is now fixed, this test passes.

  • platform/chromium/TestExpectations:
9:57 AM Changeset in webkit [119839] by vsevik@chromium.org
  • 5 edits in trunk/Source/WebCore

Web Inspector: Fix several compiler warnings and errors.
​https://bugs.webkit.org/show_bug.cgi?id=88660

Reviewed by Pavel Feldman.

Fixed several compiler warnings and errors.
Removed isEmpty method from Object.prototype and made it static method on Object.

  • inspector/InjectedScriptExterns.js:
  • inspector/front-end/FileSystemModel.js:

(WebInspector.FileSystemModel.prototype._detachFrame):

  • inspector/front-end/HeapSnapshot.js:

(WebInspector.HeapSnapshot.prototype._buildDominatorTree):

  • inspector/front-end/utilities.js:

(Object.isEmpty):

9:54 AM Changeset in webkit [119838] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[BlackBerry] Fix assertion failure introduced by bug 87551
​https://bugs.webkit.org/show_bug.cgi?id=88659

Patch by Max Feil <​mfeil@rim.com> on 2012-06-08
Reviewed by Antonio Gomes.

The determineRenderSlider() convenience function added in bug
87551 made an assumption that wasn't always true. This didn't
cause a functional error but it caused an assertion failure,
which indicates a problem with the code. PR164142.

No new tests because this case is already covered by ASSERT's.

  • platform/blackberry/RenderThemeBlackBerry.cpp:

(WebCore::determineRenderSlider):

9:37 AM Changeset in webkit [119837] by Carlos Garcia Campos
  • 11 edits
    3 adds in trunk

[GTK] Add API to get the library version to WebKit2 GTK+
​https://bugs.webkit.org/show_bug.cgi?id=88426

Reviewed by Martin Robinson.

.:

  • configure.ac: Generate WebKitVersion.h from WebKitVersion.h.in

file.

Source/WebKit2:

  • GNUmakefile.am:
  • GNUmakefile.list.am:
  • UIProcess/API/gtk/WebKitVersion.cpp: Added.

(webkit_get_major_version):
(webkit_get_minor_version):
(webkit_get_micro_version):

  • UIProcess/API/gtk/WebKitVersion.h.in: Added.
  • UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add new symbols.
  • UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Add new section

for WebKitVersion.

  • UIProcess/API/gtk/tests/GNUmakefile.am:
  • UIProcess/API/gtk/tests/TestWebKitVersion.cpp: Added.

(testWebKitVersion):
(testWebKitCheckVersion):
(beforeAll):
(afterAll):

  • UIProcess/API/gtk/webkit2.h: Include WebKitVersion.h.

Tools:

  • gtk/generate-gtkdoc:

(get_webkit2_options): Add built sources to the list of files, but
ignore enum types and marshallers.

9:27 AM BuildBot edited by alex.bravo@nokia.com
FIxed a typo (diff)
9:26 AM BuildBot edited by alex.bravo@nokia.com
Fixed formatting of lists (diff)
8:36 AM Changeset in webkit [119836] by schenney@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed fix for Chromium TestExpectations.

A file crept into the expectations that has not yet been submitted.

  • platform/chromium/TestExpectations:
8:35 AM Changeset in webkit [119835] by Simon Hausmann
  • 2 edits in trunk/Source/WebKit/qt

Unreviewed, fix file permissions.

  • WebCoreSupport/FrameLoaderClientQt.cpp:
8:23 AM Changeset in webkit [119834] by commit-queue@webkit.org
  • 4 edits
    1 add in trunk

Some overlay scrollbar API calls in ScrollAnimatorMac can lead to an assertion in RenderBox::mapAbsoluteToLocalPoint
​https://bugs.webkit.org/show_bug.cgi?id=74111

Patch by Ion Rosca <Ion Rosca> on 2012-06-08
Reviewed by Simon Fraser.

.:

  • ManualTests/scrollbar-crash-on-hide-scrolled-area.html: Added.

Source/WebCore:

Added zero-delay timer for AppKit scroll notification that can be called during layout.
Manual test: ManualTests/scrollbar-crash-on-hide-scrolled-area.html
This assertion does not fire when running layout tests. It can be easly reproduced using a debug build by loading the manual test page.

  • platform/mac/ScrollAnimatorMac.h:

(ScrollAnimatorMac):

  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::ScrollAnimatorMac):
(WebCore::ScrollAnimatorMac::notifyContentAreaScrolled):
(WebCore::ScrollAnimatorMac::sendContentAreaScrolledSoon):
(WebCore):
(WebCore::ScrollAnimatorMac::sendContentAreaScrolledTimerFired):

8:20 AM Changeset in webkit [119833] by reni@webkit.org
  • 6 edits in trunk

Adding few already supported features to the FeatureSet in DOMImplementation
​https://bugs.webkit.org/show_bug.cgi?id=86482

Source/WebCore:

Text, Filter and View features are already implemented but they
are not added to the SVG11 feature set. This is corrected in this patch.

Reviewed by Nikolas Zimmermann.

No new testcase is needed because we already have one what checks the svg features via
DOMImplementation (svg/custom/svg-features.html).

  • dom/DOMImplementation.cpp:

(WebCore::isSVG11Feature):

LayoutTests:

Reviewed by Nikolas Zimmermann.

  • platform/qt-5.0/svg/custom/svg-features-expected.txt:
  • platform/qt/svg/custom/svg-features-expected.txt:
  • svg/custom/svg-features-expected.txt:
8:13 AM Changeset in webkit [119832] by scheib@chromium.org
  • 4 edits in trunk

Layout Test pointer-lock/pointerlockchange-pointerlockerror-events.html is failing
​https://bugs.webkit.org/show_bug.cgi?id=88604

Reviewed by Kent Tamura.

Tools:

DumpRenderTree fixed to not call didLosePointerLock if pointer is not currently locked.
Fixes debug only assert crash in pointer-lock/pointerlockchange-pointerlockerror-events.html test.

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::requestPointerUnlock):

LayoutTests:

Removed CRASH expectation.
Test is no longer crashing with this fix to DumpRenderTree.

  • platform/chromium/TestExpectations:
8:01 AM Changeset in webkit [119831] by schenney@chromium.org
  • 5 edits
    2 adds in trunk

WebCore::ImageBuffer.cpp has broken color table code
​https://bugs.webkit.org/show_bug.cgi?id=80321

Reviewed by Dirk Schulze.

Source/WebCore:

The code in ImageBuffer::transformColorSpace had two problems that are
fixed by this patch.

First, it was using member variables for data
that is static constant. This is incredibly wasteful, as every
ImageBuffer that is created (and we make a lot of them) gets its own
copy of the LUT and the table is initialized again and again. The
patch replaces this with a static local, with lazy evaluation, in
the method that uses the tables.

Second, the code for filling the table was mapping 255 to 254, thus
reducing the intensity and alpha of every image it pushed through a
color transform.

Test: svg/filters/color-space-conversion.svg

  • platform/graphics/ImageBuffer.cpp:

(WebCore::ImageBuffer::transformColorSpace):

  • platform/graphics/ImageBuffer.h:

(ImageBuffer):

LayoutTests:

Temporarily marking Chromium tests expected to fail as a result of this patch. And a new test.

  • platform/chromium/TestExpectations:
  • svg/filters/color-space-conversion.svg:
  • svg/filters/color-space-conversion-expected.svg:
7:44 AM Changeset in webkit [119830] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: Better labels positioning on the memory pie chart
​https://bugs.webkit.org/show_bug.cgi?id=88652

Patch by Alexei Filippov <​alexeif@chromium.org> on 2012-06-08
Reviewed by Vsevolod Vlasov.

  • inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.NativeMemoryPieChart.prototype._paint.paintPercentAndLabel):
(WebInspector.NativeMemoryPieChart.prototype._paint):

7:30 AM Changeset in webkit [119829] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

Tried to fix the Snow Leopard build after r119806.

  • WebKit2Prefix.h:
7:23 AM Changeset in webkit [119828] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-06-08

  • DEPS:
7:11 AM Changeset in webkit [119827] by qi.2.zhang@nokia.com
  • 2 edits in trunk/LayoutTests

[Qt] fast/viewport/viewport-133.html fails
​https://bugs.webkit.org/show_bug.cgi?id=80689

Since ​https://bugs.webkit.org/show_bug.cgi?id=88047 remove this
test case, remove it from skip list.

Reviewed by Laszlo Gombos.

  • platform/qt/Skipped:
6:07 AM Changeset in webkit [119826] by caseq@chromium.org
  • 30 edits in trunk

Web Inspector: Annotate timeline records with a frame identifier
​https://bugs.webkit.org/show_bug.cgi?id=86406

Patch by Bryan McQuade <​bmcquade@google.com> on 2012-06-08
Reviewed by Vsevolod Vlasov.

Source/WebCore:

Tests: Updated existing layout tests.

  • inspector/InspectorController.cpp:

(WebCore::InspectorController::InspectorController):

  • inspector/InspectorInstrumentation.cpp:

(WebCore::frameForScriptExecutionContext):
(WebCore):
(WebCore::InspectorInstrumentation::didScheduleResourceRequestImpl):
(WebCore::InspectorInstrumentation::didInstallTimerImpl):
(WebCore::InspectorInstrumentation::didRemoveTimerImpl):
(WebCore::InspectorInstrumentation::willCallFunctionImpl):
(WebCore::InspectorInstrumentation::willChangeXHRReadyStateImpl):
(WebCore::InspectorInstrumentation::willDispatchEventImpl):
(WebCore::InspectorInstrumentation::willDispatchEventOnWindowImpl):
(WebCore::InspectorInstrumentation::willEvaluateScriptImpl):
(WebCore::InspectorInstrumentation::willFireTimerImpl):
(WebCore::InspectorInstrumentation::willLayoutImpl):
(WebCore::InspectorInstrumentation::willLoadXHRImpl):
(WebCore::InspectorInstrumentation::willPaintImpl):
(WebCore::InspectorInstrumentation::willRecalculateStyleImpl):
(WebCore::InspectorInstrumentation::willSendRequestImpl):
(WebCore::InspectorInstrumentation::willReceiveResourceDataImpl):
(WebCore::InspectorInstrumentation::willReceiveResourceResponseImpl):
(WebCore::InspectorInstrumentation::didFinishLoadingImpl):
(WebCore::InspectorInstrumentation::didFailLoadingImpl):
(WebCore::InspectorInstrumentation::domContentLoadedEventFiredImpl):
(WebCore::InspectorInstrumentation::loadEventFiredImpl):
(WebCore::InspectorInstrumentation::willWriteHTMLImpl):
(WebCore::InspectorInstrumentation::didRequestAnimationFrameImpl):
(WebCore::InspectorInstrumentation::didCancelAnimationFrameImpl):
(WebCore::InspectorInstrumentation::willFireAnimationFrameImpl):

  • inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):
(WebCore::InspectorInstrumentation::didScheduleResourceRequest):
(WebCore::InspectorInstrumentation::didInstallTimer):
(WebCore::InspectorInstrumentation::didRemoveTimer):
(WebCore::InspectorInstrumentation::willCallFunction):
(WebCore::InspectorInstrumentation::willChangeXHRReadyState):
(WebCore::InspectorInstrumentation::willDispatchEvent):
(WebCore::InspectorInstrumentation::willEvaluateScript):
(WebCore::InspectorInstrumentation::willFireTimer):
(WebCore::InspectorInstrumentation::willLayout):
(WebCore::InspectorInstrumentation::willLoadXHR):
(WebCore::InspectorInstrumentation::willPaint):
(WebCore::InspectorInstrumentation::willRecalculateStyle):
(WebCore::InspectorInstrumentation::willReceiveResourceData):
(WebCore::InspectorInstrumentation::willReceiveResourceResponse):
(WebCore::InspectorInstrumentation::willWriteHTML):
(WebCore::InspectorInstrumentation::didRequestAnimationFrame):
(WebCore::InspectorInstrumentation::didCancelAnimationFrame):
(WebCore::InspectorInstrumentation::willFireAnimationFrame):

  • inspector/InspectorTimelineAgent.cpp:

(WebCore::InspectorTimelineAgent::pushGCEventRecords):
(WebCore::InspectorTimelineAgent::didBeginFrame):
(WebCore::InspectorTimelineAgent::willCallFunction):
(WebCore::InspectorTimelineAgent::willDispatchEvent):
(WebCore::InspectorTimelineAgent::willLayout):
(WebCore::InspectorTimelineAgent::willRecalculateStyle):
(WebCore::InspectorTimelineAgent::willPaint):
(WebCore::InspectorTimelineAgent::willComposite):
(WebCore::InspectorTimelineAgent::willWriteHTML):
(WebCore::InspectorTimelineAgent::didInstallTimer):
(WebCore::InspectorTimelineAgent::didRemoveTimer):
(WebCore::InspectorTimelineAgent::willFireTimer):
(WebCore::InspectorTimelineAgent::willChangeXHRReadyState):
(WebCore::InspectorTimelineAgent::willLoadXHR):
(WebCore::InspectorTimelineAgent::willEvaluateScript):
(WebCore::InspectorTimelineAgent::didScheduleResourceRequest):
(WebCore::InspectorTimelineAgent::willSendResourceRequest):
(WebCore::InspectorTimelineAgent::willReceiveResourceData):
(WebCore::InspectorTimelineAgent::willReceiveResourceResponse):
(WebCore::InspectorTimelineAgent::didFinishLoadingResource):
(WebCore::InspectorTimelineAgent::didTimeStamp):
(WebCore::InspectorTimelineAgent::didMarkDOMContentEvent):
(WebCore::InspectorTimelineAgent::didMarkLoadEvent):
(WebCore::InspectorTimelineAgent::didRequestAnimationFrame):
(WebCore::InspectorTimelineAgent::didCancelAnimationFrame):
(WebCore::InspectorTimelineAgent::willFireAnimationFrame):
(WebCore::InspectorTimelineAgent::addRecordToTimeline):
(WebCore::InspectorTimelineAgent::innerAddRecordToTimeline):
(WebCore::InspectorTimelineAgent::didCompleteCurrentRecord):
(WebCore::InspectorTimelineAgent::InspectorTimelineAgent):
(WebCore::InspectorTimelineAgent::appendRecord):
(WebCore::InspectorTimelineAgent::pushCurrentRecord):
(WebCore::InspectorTimelineAgent::pushCancelableRecord):
(WebCore::InspectorTimelineAgent::commitCancelableRecords):

  • inspector/InspectorTimelineAgent.h:

(WebCore):
(WebCore::InspectorTimelineAgent::create):
(InspectorTimelineAgent):
(WebCore::InspectorTimelineAgent::TimelineRecordEntry::TimelineRecordEntry):
(TimelineRecordEntry):

  • inspector/WorkerInspectorController.cpp:

(WebCore::WorkerInspectorController::WorkerInspectorController):

  • inspector/front-end/TimelinePresentationModel.js:

(WebInspector.TimelinePresentationModel.isEventDivider):
(WebInspector.TimelinePresentationModel.Record):

LayoutTests:

  • inspector/timeline/timeline-animation-frame-expected.txt:
  • inspector/timeline/timeline-dom-content-loaded-event-expected.txt:
  • inspector/timeline/timeline-event-dispatch-expected.txt:
  • inspector/timeline/timeline-injected-script-eval-expected.txt:
  • inspector/timeline/timeline-layout-expected.txt:
  • inspector/timeline/timeline-load-event-expected.txt:
  • inspector/timeline/timeline-network-resource-expected.txt:
  • inspector/timeline/timeline-paint-expected.txt:
  • inspector/timeline/timeline-parse-html-expected.txt:
  • inspector/timeline/timeline-recalculate-styles-expected.txt:
  • inspector/timeline/timeline-script-tag-1-expected.txt:
  • inspector/timeline/timeline-script-tag-2-expected.txt:
  • inspector/timeline/timeline-test.js:
  • inspector/timeline/timeline-timer-expected.txt:
  • platform/chromium/inspector/timeline/timeline-animation-frame-expected.txt:
  • platform/chromium/inspector/timeline/timeline-event-dispatch-expected.txt:
  • platform/chromium/inspector/timeline/timeline-layout-expected.txt:
  • platform/chromium/inspector/timeline/timeline-network-resource-expected.txt:
  • platform/chromium/inspector/timeline/timeline-paint-expected.txt:
  • platform/chromium/inspector/timeline/timeline-parse-html-expected.txt:
  • platform/chromium/inspector/timeline/timeline-timer-expected.txt:
5:59 AM Changeset in webkit [119825] by loislo@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed update for chromium test expectations.
viewspec-parser-*.html were marked as SLOW.

  • platform/chromium/TestExpectations:

BUGWK87772 SLOW : svg/dom/viewspec-parser-1.html = PASS
BUGWK87772 SLOW : svg/dom/viewspec-parser-2.html = PASS
BUGWK87772 SLOW : svg/dom/viewspec-parser-3.html = PASS
BUGWK87772 SLOW : svg/dom/viewspec-parser-4.html = PASS
BUGWK87772 SLOW : svg/dom/viewspec-parser-5.html = PASS

5:46 AM Changeset in webkit [119824] by kinuko@chromium.org
  • 12 edits in trunk/Source/WebCore

Unreviewed, rolling out r119821.
​http://trac.webkit.org/changeset/119821
​https://bugs.webkit.org/show_bug.cgi?id=88648

Seems to have regression on Mac (Requested by kinuko on
#webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2012-06-08

  • fileapi/File.cpp:

(WebCore::File::File):
(WebCore::File::lastModifiedDate):
(WebCore::File::lastModifiedDateForBinding):
(WebCore):
(WebCore::File::captureSnapshot):

  • fileapi/File.h:

(File):

  • fileapi/File.idl:
  • platform/FileMetadata.h:

(WebCore::FileMetadata::FileMetadata):

  • platform/FileSystem.h:
  • platform/chromium/support/WebHTTPBody.cpp:

(WebKit::WebHTTPBody::elementAt):

  • platform/network/BlobData.cpp:

(WebCore):

  • platform/network/BlobData.h:

(BlobDataItem):
(WebCore::BlobDataItem::BlobDataItem):

  • platform/network/FormData.cpp:

(WebCore::FormData::appendFile):

  • platform/network/cf/FormDataStreamCFNet.cpp:

(WebCore::advanceCurrentStream):

  • platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::blobIsOutOfDate):

5:29 AM Changeset in webkit [119823] by kinuko@chromium.org
  • 2 edits in trunk/Source/WebCore

Unreviewed, removing duplicated entries in gyp.

  • WebCore.gypi:
5:16 AM Changeset in webkit [119822] by arko@motorola.com
  • 3 edits
    2 adds in trunk

Microdata: An element with an empty itemprop= should not get listed in the .properties list.
​https://bugs.webkit.org/show_bug.cgi?id=88235

Reviewed by Ryosuke Niwa.

Source/WebCore:

As per the specification change ​http://html5.org/r/6854, an element should be listed in .properties
if the elelement has an attribute itemprop specified and the element has one or more property names.

Test: fast/dom/MicroData/element-with-empty-itemprop.html

  • html/HTMLPropertiesCollection.cpp:

(WebCore::HTMLPropertiesCollection::itemAfter):

LayoutTests:

  • fast/dom/MicroData/element-with-empty-itemprop-expected.txt: Added.
  • fast/dom/MicroData/element-with-empty-itemprop.html: Added.
4:50 AM Changeset in webkit [119821] by kinuko@chromium.org
  • 12 edits in trunk/Source/WebCore

File::lastModifiedDate should use NaN or separate boolean flag for null Date value
​https://bugs.webkit.org/show_bug.cgi?id=87826

Reviewed by Kent Tamura.

Test: http/tests/local/fileapi/file-last-modified-after-delete.html

  • fileapi/File.cpp:

(WebCore::File::File):
(WebCore::File::captureSnapshot):
(WebCore::File::lastModifiedDate):
(WebCore::File::lastModifiedDateForBinding): Removed.

  • fileapi/File.h:

(File):

  • fileapi/File.idl:
  • platform/FileMetadata.h:

(WebCore::FileMetadata::FileMetadata):

  • platform/FileSystem.h:
  • platform/chromium/support/WebHTTPBody.cpp:

(WebKit::WebHTTPBody::elementAt):

  • platform/network/BlobData.h:

(WebCore::BlobDataItem::doNotCheckFileChange): Removed.

  • platform/network/BlobData.cpp:
  • platform/network/cf/FormDataStreamCFNet.cpp:

(WebCore::advanceCurrentStream):

  • platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::blobIsOutOfDate):

4:44 AM Changeset in webkit [119820] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL] Unskip passing sputnik tests
​https://bugs.webkit.org/show_bug.cgi?id=88641

Unreviewed, unskip sputnik tests which are passing now.

Patch by Sudarsana Nagineni <​sudarsana.nagineni@linux.intel.com> on 2012-06-08

  • platform/efl/TestExpectations:
4:11 AM Changeset in webkit [119819] by zandobersek@gmail.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK gardening, removing crash expectations for two plugins
tests that were fixed after LLInt was enabled for the Gtk port in r119593.

  • platform/gtk/TestExpectations:
4:02 AM Changeset in webkit [119818] by hans@chromium.org
  • 7 edits
    4 adds in trunk

Speech JavaScript API: mock WebSpeechRecognizer for DumpRenderTree
​https://bugs.webkit.org/show_bug.cgi?id=87976

Reviewed by Kent Tamura.

Tools:

Provide a mock implementation of WebSpeechRecognizer for
DumpRenderTree. This will allow better testing of the API via layout
tests.

  • DumpRenderTree/DumpRenderTree.gypi:
  • DumpRenderTree/chromium/LayoutTestController.cpp:

(LayoutTestController::LayoutTestController):
(LayoutTestController::addMockSpeechRecognitionResult):

  • DumpRenderTree/chromium/LayoutTestController.h:

(LayoutTestController):

  • DumpRenderTree/chromium/MockWebSpeechRecognizer.cpp: Added.

(MockWebSpeechRecognizer::create):
(MockWebSpeechRecognizer::start):
(MockWebSpeechRecognizer::stop):
(MockWebSpeechRecognizer::abort):
(MockWebSpeechRecognizer::MockWebSpeechRecognizer):
(MockWebSpeechRecognizer::~MockWebSpeechRecognizer):
(MockWebSpeechRecognizer::ResultTask::runIfValid):

  • DumpRenderTree/chromium/MockWebSpeechRecognizer.h: Added.

(WebKit):
(MockWebSpeechRecognizer):
(MockWebSpeechRecognizer::addMockResult):
(MockWebSpeechRecognizer::taskList):
(ClientCallTask):
(MockWebSpeechRecognizer::ClientCallTask::ClientCallTask):
(ResultTask):
(MockWebSpeechRecognizer::ResultTask::ResultTask):
(NoMatchTask):
(MockWebSpeechRecognizer::NoMatchTask::NoMatchTask):

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::speechRecognizer):

  • DumpRenderTree/chromium/WebViewHost.h:

(WebViewHost):
(WebViewHost::mockSpeechRecognizer):

LayoutTests:

Add a layout test using the new mock, pushing a mock result through
and checking that events fire correctly.

  • fast/speech/scripted/speechrecognition-basics-expected.txt: Added.
  • fast/speech/scripted/speechrecognition-basics.html: Added.
3:59 AM WebKitGTK/WebKit2Roadmap edited by Carlos Garcia Campos
(diff)
3:41 AM Changeset in webkit [119817] by zandobersek@gmail.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK gardening after r119795, adding separate test expectations
for jquery/core.html - the test is crashing on debug builds and failing
with a text diff on release builds.

  • platform/gtk/TestExpectations:
2:53 AM Changeset in webkit [119816] by tkent@chromium.org
  • 15 edits
    2 adds in trunk/Source/WebCore

Move form-related functions of Document to FormController
​https://bugs.webkit.org/show_bug.cgi?id=88497

Reviewed by Hajime Morita.

A lot of files depend on Document.h. We had to build many files when we
touched form-related features in Document.h or
CheckedRadioButtons.h. This patch reduces such pain.

No new tests. Just a refactoring.

  • CMakeLists.txt: Add FormController.cpp and/or FormController.h.
  • GNUmakefile.list.am: ditto.
  • Target.pri: ditto.
  • WebCore.gypi: ditto.
  • WebCore.vcproj/WebCore.vcproj: ditto.
  • WebCore.xcodeproj/project.pbxproj: ditto.
  • dom/Document.cpp: Removed form-releated features except the followings.

(WebCore::Document::formController): Accessor for a FormController.
(WebCore::Document::formElementsState):
Just returns FormController::formElementsState().
We don't create new FormController.
(WebCore::Document::setStateForNewFormElements):
Just returns FormController::setStateForNewFormElements().
We don't create new FormController if the specified vector is empty.

  • dom/Document.h:

Moved form-related functions to FormController.

  • html/FormController.cpp: Added. Moved from Document.cpp.
  • html/FormController.h: Added. Moved from Document.h
  • html/FormAssociatedElement.cpp: Use Document::formController().

(WebCore::FormAssociatedElement::didMoveToNewDocument):
(WebCore::FormAssociatedElement::insertedInto):
(WebCore::FormAssociatedElement::removedFrom):
(WebCore::FormAssociatedElement::formAttributeChanged):

  • html/HTMLFormControlElementWithState.cpp: Use Document::formController().

(WebCore::HTMLFormControlElementWithState::HTMLFormControlElementWithState):
(WebCore::HTMLFormControlElementWithState::~HTMLFormControlElementWithState):
(WebCore::HTMLFormControlElementWithState::didMoveToNewDocument):
(WebCore::HTMLFormControlElementWithState::finishParsingChildren):

  • html/HTMLFormElement.cpp: Use Document::formController().

(WebCore::HTMLFormElement::didNotifyDescendantInsertions):
(WebCore::HTMLFormElement::removedFrom):

  • html/HTMLInputElement.cpp: Use Document::formController().

(WebCore::HTMLInputElement::~HTMLInputElement):
(WebCore::HTMLInputElement::didMoveToNewDocument):
(WebCore::HTMLInputElement::checkedRadioButtons):

  • html/HTMLInputElement.h:

Declare CheckedRadioButtons. It was provided by Document.h.

  • html/parser/HTMLConstructionSite.h:

Declare HTMLFormElement. It was provided by Document.h

2:50 AM Changeset in webkit [119815] by commit-queue@webkit.org
  • 12 edits
    2 adds in trunk/Source

Add FileSystem item to storage tree.
​https://bugs.webkit.org/show_bug.cgi?id=72691

Patch by Taiju Tsuiki <​tzik@chromium.org> on 2012-06-08
Reviewed by Vsevolod Vlasov.

Source/WebCore:

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/compile-front-end.py:
  • inspector/front-end/FileSystemModel.js: Added.
  • inspector/front-end/Images/fileSystem.png: Added.
  • inspector/front-end/ResourcesPanel.js:

(WebInspector.FileSystemListTreeElement):
(WebInspector.FileSystemListTreeElement.prototype.onexpand):
(WebInspector.FileSystemListTreeElement.prototype.onattach):
(WebInspector.FileSystemListTreeElement.prototype._handleContextMenuEvent):
(WebInspector.FileSystemListTreeElement.prototype._fileSystemAdded):
(WebInspector.FileSystemListTreeElement.prototype._fileSystemRemoved):
(WebInspector.FileSystemListTreeElement.prototype._fileSystemTreeElementByName):
(WebInspector.FileSystemListTreeElement.prototype._refreshFileSystem):
(WebInspector.FileSystemTreeElement):
(WebInspector.FileSystemTreeElement.prototype.get fileSystemName):
(WebInspector.FileSystemTreeElement.prototype.get itemURL):

  • inspector/front-end/Settings.js:

(WebInspector.ExperimentsSettings):

  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
  • inspector/front-end/resourcesPanel.css:

(.file-system-storage-tree-item .icon):

  • inspector/front-end/utilities.js:

Source/WebKit/chromium:

  • src/js/DevTools.js:
2:19 AM Changeset in webkit [119814] by haraken@chromium.org
  • 5 edits in trunk/Source/WebCore

REGRESSION r119802: Remove unnecessary ASSERT(!m_deletionHasBegun) from Document.h
​https://bugs.webkit.org/show_bug.cgi?id=88528

Reviewed by Ryosuke Niwa.

m_deletionHasBegun should become true just before 'delete this'.

No tests. No change in behavior.

  • dom/Document.h:

(WebCore::Document::guardRef):
(WebCore::Document::guardDeref):

2:09 AM Changeset in webkit [119813] by peter@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] Re-enable SHARED_WORKERS for Android until a build fix has been resolved.
​https://bugs.webkit.org/show_bug.cgi?id=88637

Unreviewed build fix.

Linking content_shell on the Chromium side fails due to an unresolved
call to WebSharedWorker::create(). Since this blocks WebKit rolls,
temporarily re-enable the feature until this has been solved.

  • features.gypi:
1:29 AM Changeset in webkit [119812] by noel.gordon@gmail.com
  • 1 edit
    1 copy
    1 move in trunk/LayoutTests

[chromium] Run optimize-baselines over canvas-toDataURL-webp.html results

Unreviewed test expectations update.

  • platform/chromium-mac/fast/canvas/canvas-toDataURL-webp-expected.png: Copied from LayoutTests/platform/chromium-mac-snowleopard/fast/canvas/canvas-toDataURL-webp-expected.png.
  • platform/chromium-mac/platform/chromium/virtual/gpu/fast/canvas/canvas-toDataURL-webp-expected.png: Renamed from LayoutTests/platform/chromium-mac-snowleopard/fast/canvas/canvas-toDataURL-webp-expected.png.
1:14 AM Changeset in webkit [119811] by abarth@webkit.org
  • 2 edits in trunk/Source/WebCore

HTTP 204 No Content responses shouldn't trigger MIME type warnings in the web inspector
​https://bugs.webkit.org/show_bug.cgi?id=88603

Reviewed by Pavel Feldman.

A 204 response doesn't have an entity body, so its MIME type is kind of
meaningless. We shouldn't spam the console with a warning about MIME
types.

  • inspector/front-end/NetworkManager.js:

(WebInspector.NetworkDispatcher.prototype._mimeTypeIsConsistentWithType):

1:01 AM Changeset in webkit [119810] by kkristof@inf.u-szeged.hu
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Skip tests using disabled SHADOW_DOM feature on Qt.

Patch by Zoltan Arvai <​zarvai@inf.u-szeged.hu> on 2012-06-08

  • platform/qt/Skipped:
12:51 AM Changeset in webkit [119809] by yosin@chromium.org
  • 11 edits in trunk/Source/WebCore

[Forms] Drop StepRange::NumberWithDecimalPlacesOrMissing
​https://bugs.webkit.org/show_bug.cgi?id=88621

Reviewed by Kent Tamura.

This patch changes NumberWithDecimalPlacesOrMissing to NumberWithDecimalPlaces.
We use NaN as indication of having value or not.

No new tests, because this patch doesn't change behavior.

  • html/DateInputType.cpp:

(WebCore::DateInputType::createStepRange): Replace NumberWithDecimalPlacesOrMissing to NumberWithDecimalPlaces.

  • html/DateTimeInputType.cpp:

(WebCore::DateTimeInputType::createStepRange): ditto.

  • html/DateTimeLocalInputType.cpp:

(WebCore::DateTimeLocalInputType::createStepRange): ditto.

  • html/MonthInputType.cpp:

(WebCore::MonthInputType::createStepRange): ditto.

  • html/NumberInputType.cpp:

(WebCore::NumberInputType::createStepRange): ditto.

  • html/RangeInputType.cpp:

(WebCore::RangeInputType::createStepRange): ditto.

  • html/StepRange.cpp:

(WebCore::StepRange::StepRange): Checks value is finite or not.
(WebCore::StepRange::parseStep): Replace NumberWithDecimalPlacesOrMissing to NumberWithDecimalPlaces.

  • html/StepRange.h:

(StepRange): Removed NumberWithDecimalPlacesOrMissing.

  • html/TimeInputType.cpp:

(WebCore::TimeInputType::createStepRange): NumberWithDecimalPlacesOrMissing to NumberWithDecimalPlaces.

  • html/WeekInputType.cpp:

(WebCore::WeekInputType::createStepRange): ditto.

12:45 AM Changeset in webkit [119808] by haraken@chromium.org
  • 2 edits in trunk/Source/WebCore

Unreviewed. Fixed a comment, pointed out by ap@.

  • platform/TreeShared.h:

(WebCore):

12:42 AM Changeset in webkit [119807] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/WebCore

[chromium] WEBP image blue and red color channels reversed on Android
​https://bugs.webkit.org/show_bug.cgi?id=88613

Reviewed by Kent Tamura.

No new tests. Covered by test fast/images/webp-image-decoding.html though I'm
not sure where/how the Androids run their layout tests.

  • platform/image-decoders/webp/WEBPImageDecoder.cpp:

(outputMode): Detect little-endian Skia with a blue pixel shift (aka Android)
which seems to be the way for them per bug 75861.

12:29 AM Changeset in webkit [119806] by aestes@apple.com
  • 3 edits in trunk/Source/WebKit2

[WebKit2] Move Mac-specific bits from config.h to WebKit2Prefix.h
​https://bugs.webkit.org/show_bug.cgi?id=88625

Reviewed by Dan Bernstein.

Since the Mac supports prefix headers, there is no point in having
Mac-specific code in WebKit2's config.h. Move it to WebKit2Prefix.h.

  • WebKit2Prefix.h:
  • config.h:
12:27 AM Changeset in webkit [119805] by mitz@apple.com
  • 2 edits in trunk/LayoutTests

Fixed fast/files/read-blob-async.html failing after r119791.

  • fast/files/resources/read-blob-test-cases.js:

(testReadingSingleArrayBufferBlob):
(testReadingSlicedArrayBufferBlob):
(testReadingMultipleArrayBufferBlob):
(testReadingHybridBlob):
(testReadingSlicedHybridBlob):
(testReadingTripleSlicedHybridBlob):

12:22 AM Changeset in webkit [119804] by noel.gordon@gmail.com
  • 1 edit
    1 add in trunk/LayoutTests

[chromium] New virtual GPU canvas-toDataURL-webp.html results after r119787

Unreviewed test expectations update.

  • platform/chromium-win/platform/chromium/virtual/gpu/fast/canvas/canvas-toDataURL-webp-expected.png: Added.
12:04 AM Changeset in webkit [119803] by rniwa@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove the assertion that has been moved to TreeScope after r119802.

  • dom/Document.cpp:

(WebCore::Document::removedLastRef):

Note: See TracTimeline for information about the timeline view.