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

Timeline



Oct 23, 2011:

11:59 PM Changeset in webkit [98219] by Darin Adler
  • 10 edits
    2 deletes in trunk/Source/WebCore

Eliminate OptionElement (second and final part)
​https://bugs.webkit.org/show_bug.cgi?id=70710

Reviewed by Kent Tamura.

  • CMakeLists.txt: Removed OptionElement.h and OptionElement.cpp.
  • GNUmakefile.list.am: Ditto.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • dom/DOMAllInOne.cpp: Ditto.
  • dom/OptionElement.cpp: Removed.
  • dom/OptionElement.h: Removed.
  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::saveLastSelection): Eliminate use of the
toOptionElement function. Use hasTagName and toHTMLOptionElement instead.
(WebCore::HTMLSelectElement::setActiveSelectionAnchorIndex): Ditto.
(WebCore::HTMLSelectElement::updateListBoxSelection): Ditto.
(WebCore::HTMLSelectElement::listBoxOnChange): Ditto.
(WebCore::HTMLSelectElement::recalcListItems): Ditto.
(WebCore::HTMLSelectElement::selectedIndex): Ditto.
(WebCore::HTMLSelectElement::setSelectedIndex): Ditto.
(WebCore::HTMLSelectElement::deselectItemsWithoutValidation): Ditto.
(WebCore::HTMLSelectElement::saveFormControlState): Ditto.
(WebCore::HTMLSelectElement::restoreFormControlState): Ditto.
(WebCore::HTMLSelectElement::appendFormData): Ditto.
(WebCore::HTMLSelectElement::reset): Ditto.
(WebCore::HTMLSelectElement::updateSelectedState): Ditto.
(WebCore::HTMLSelectElement::lastSelectedListIndex): Ditto.
(WebCore::HTMLSelectElement::typeAheadFind): Ditto.
(WebCore::HTMLSelectElement::accessKeySetSelectedIndex): Ditto.

  • html/HTMLSelectElement.h: Changed the argument type of the

deselectItemsWithoutValidation function to HTMLElement* since that's
the type of an item in our items vector now.

11:54 PM Changeset in webkit [98218] by yuzo@google.com
  • 2 edits in trunk/Source/WebCore

[chromium] Attempt to fix compilation error for Chromium OS.

Unreviewed.

  • mediastream/MediaStreamTrack.cpp:

(WebCore::MediaStreamTrack::kind):

10:53 PM Changeset in webkit [98217] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fixed a typo Darin spotted.

  • wtf/StringHasher.h:

(WTF::StringHasher::hash): Expelliarmus!

10:51 PM Changeset in webkit [98216] by ggaren@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

Removed StringImpl::createStrippingNullCharacters
​https://bugs.webkit.org/show_bug.cgi?id=70700

Reviewed by David Levin.

It was unused.

9:36 PM Changeset in webkit [98215] by abarth@webkit.org
  • 3 edits
    2 adds in trunk

<img crossorigin> should fail to load when CORS check fails
​https://bugs.webkit.org/show_bug.cgi?id=69732

Reviewed by Darin Adler.

Source/WebCore:

When loading an image with the crossorigin attribute, the spec says
that we're not supposed to load the image if the CORS check fails.
This "fails fast" behavior is intended to help developers understand
whether they've configured CORS correctly (instead of only catching the
error later when trying to read back the canvas).

Our new behavior matches the spec and Firefox.

Test: http/tests/security/img-with-failed-cors-check-fails-to-load.html

  • loader/ImageLoader.cpp:

(WebCore::ImageLoader::notifyFinished):

LayoutTests:

Test that images loaded with the crossorigin attribute fail to load if
the CORS access check doesn't pass.

  • http/tests/security/img-with-failed-cors-check-fails-to-load-expected.txt: Added.
  • http/tests/security/img-with-failed-cors-check-fails-to-load.html: Added.
8:58 PM Changeset in webkit [98214] by fpizlo@apple.com
  • 9 edits
    3 adds in trunk

DFG should inline constructors
​https://bugs.webkit.org/show_bug.cgi?id=70675

Source/JavaScriptCore:

Reviewed by Oliver Hunt.

Adds support for inlining constructors. Also fixes two pathologies
uncovered along the way: CheckMethod claimed that it never returned a
result (causing CheckMethod -> SetLocal -> GetLocal sequences to
result in the GetLocal doing OSR exit), and get_by_id parsing never
checked if it was hot in slow path. Also fiddled with inlining
heuristics; it appears that for now, the more inlining, the happier
V8 is. Finally, a bug was uncovered where a silent spill of a boxed
integer that had previously been spilled unboxed causes the silent
fill to forget to unbox.

This appears to be a 4% speed-up on V8 in their harness, or a 1%
speed-up in my harness. The difference is due to warm-up: in my
harness we see significant amounts of time spent in compilation, but
in V8's harness compilation gets amortizes. Profiling indicates that
we have the potential for a 5% win from basic optimizations like
generating OSR exits lazily and holding onto bytecode longer.

  • dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::execute):

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleCall):
(JSC::DFG::ByteCodeParser::handleInlining):
(JSC::DFG::ByteCodeParser::handleMinMax):
(JSC::DFG::ByteCodeParser::parseBlock):
(JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry):
(JSC::DFG::ByteCodeParser::parse):

  • dfg/DFGCapabilities.h:

(JSC::DFG::mightInlineFunctionForConstruct):
(JSC::DFG::canInlineOpcode):
(JSC::DFG::mightInlineFunctionFor):
(JSC::DFG::canInlineFunctionFor):

  • dfg/DFGJITCodeGenerator.h:

(JSC::DFG::JITCodeGenerator::silentFillGPR):

  • runtime/Executable.h:

(JSC::isCall):
(JSC::ExecutableBase::intrinsicFor):

  • runtime/Heuristics.cpp:

(JSC::Heuristics::initializeHeuristics):

  • runtime/Heuristics.h:

LayoutTests:

Reviewed by Oliver Hunt.

Adds test that asserts that register pressure induced by inlining doesn't
result in spilling corrupting values.

  • fast/js/dfg-inlining-reg-alloc-expected.txt: Added.
  • fast/js/dfg-inlining-reg-alloc.html: Added.
  • fast/js/script-tests/dfg-inlining-reg-alloc.js: Added.

(foo):
(bar):
(baz):

8:40 PM Changeset in webkit [98213] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

[chromium] Test expecation change. inspector/elements/edit-dom-actions.html times out since around r98120:98133 (chromium 106751:106770).

Unreviewed.

  • platform/chromium/test_expectations.txt:
7:57 PM Changeset in webkit [98212] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[EFL] Remove unused code from EWebLauncher
​https://bugs.webkit.org/show_bug.cgi?id=70590

Patch by Tomasz Morawski <​t.morawski@samsung.com> on 2011-10-23
Reviewed by Andreas Kling.

Removed unused function on_closeWindow from EWebLauncher

  • EWebLauncher/main.c:
7:54 PM Changeset in webkit [98211] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

[chromium] Test expectation change for "r98183 causes 4 snowleopard tests to fail".
​https://bugs.webkit.org/show_bug.cgi?id=70704

Unreviewed.

  • platform/chromium/test_expectations.txt:
7:23 PM Changeset in webkit [98210] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/JavaScriptCore

[chromium] Remove RopeImpl.{h,cpp} from the gyp projects
​https://bugs.webkit.org/show_bug.cgi?id=70703

Reviewed by Kent Tamura.

runtime/RopeImpl.{h,cpp} were removed in r97872, remove references
to these files from the gyp project files.

7:18 PM Changeset in webkit [98209] by noel.gordon@gmail.com
  • 2 edits in trunk/Source/WebCore

[chromium] Remove GeolocationServiceGtk.{h,cpp} from the gyp projects
​https://bugs.webkit.org/show_bug.cgi?id=70702

Reviewed by Kent Tamura.

platform/gtk/GeolocationServiceGtk.{h,cpp} were removed in r97746, so
remove references to these files from the gyp files.

  • WebCore.gypi:
6:49 PM Changeset in webkit [98208] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

Move StyleSheet::addSubresourceStyleURLs() to CSSStyleSheet.
​https://bugs.webkit.org/show_bug.cgi?id=70690

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

This method isn't part of the StyleSheet DOM interface, nor is it used by
XSLStyleSheet, so move it down to CSSStyleSheet and make it non-virtual.
Also remove some nearby unneeded casts from CSSStyleSheet* to StyleSheet*.

  • css/CSSStyleSheet.h:
  • css/StyleSheet.h:
  • html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::isLoading):
(WebCore::HTMLLinkElement::addSubresourceAttributeURLs):

  • html/HTMLStyleElement.cpp:

(WebCore::HTMLStyleElement::addSubresourceAttributeURLs):
(WebCore::HTMLStyleElement::disabled):
(WebCore::HTMLStyleElement::setDisabled):

6:42 PM Changeset in webkit [98207] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

Remove style sheet constructors that take a style sheet parent.
​https://bugs.webkit.org/show_bug.cgi?id=70692

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

Style sheets are never directly parented by other style sheets, they always
go via a rule (such as @import.)

  • css/CSSStyleSheet.cpp:
  • css/CSSStyleSheet.h:

(WebCore::CSSStyleSheet::create):

  • css/StyleSheet.cpp:
  • css/StyleSheet.h:
5:37 PM Changeset in webkit [98206] by macpherson@chromium.org
  • 2 edits in trunk/Tools

Add watchlist for WebCore/css and put myself on it.
​https://bugs.webkit.org/show_bug.cgi?id=70701

Reviewed by Adam Barth.

  • Scripts/webkitpy/common/config/watchlist:

Add watchlist for WebCore/css and put myself on it.

5:21 PM Changeset in webkit [98205] by mhahnenberg@apple.com
  • 6 edits in trunk/Source

Add deleteProperty to the MethodTable
​https://bugs.webkit.org/show_bug.cgi?id=70162

Reviewed by Sam Weinig.

Source/JavaScriptCore:

  • JavaScriptCore.exp:
  • runtime/ClassInfo.h: Added both versions of deleteProperty to the MethodTable.
  • runtime/JSFunction.h: Changed JSFunction::deleteProperty to

be protected rather than private for subclasses who don't provide their own
implementation.

Source/WebCore:

No new tests.

  • WebCore.exp.in:
5:11 PM Changeset in webkit [98204] by abarth@webkit.org
  • 4 edits in trunk/Source/WebCore

Update baselines for run-bindings-tests.

  • bindings/scripts/test/JS/JSTestInterface.cpp:
  • bindings/scripts/test/JS/JSTestInterface.h:
  • bindings/scripts/test/JS/JSTestObj.cpp:
2:14 PM Changeset in webkit [98203] by mhahnenberg@apple.com
  • 54 edits in trunk/Source

Remove getConstructDataVirtual
​https://bugs.webkit.org/show_bug.cgi?id=70638

Reviewed by Darin Adler.

Source/JavaScriptCore:

Removed all declarations and definitions of getConstructDataVirtual.
Also replaced all call sites to getConstructDataVirtual with a
corresponding lookup in the MethodTable.

  • API/JSCallbackConstructor.cpp:
  • API/JSCallbackConstructor.h:
  • API/JSCallbackObject.h:
  • API/JSCallbackObjectFunctions.h:
  • API/JSObjectRef.cpp:

(JSObjectIsConstructor):
(JSObjectCallAsConstructor):

(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/ArrayConstructor.cpp:
  • runtime/ArrayConstructor.h:
  • runtime/BooleanConstructor.cpp:
  • runtime/BooleanConstructor.h:
  • runtime/DateConstructor.cpp:
  • runtime/DateConstructor.h:
  • runtime/Error.h:

(JSC::StrictModeTypeErrorFunction::getConstructData):

  • runtime/ErrorConstructor.cpp:
  • runtime/ErrorConstructor.h:
  • runtime/FunctionConstructor.cpp:
  • runtime/FunctionConstructor.h:
  • runtime/JSCell.cpp:
  • runtime/JSCell.h:
  • runtime/JSFunction.cpp:
  • runtime/JSFunction.h:
  • runtime/JSObject.h:

(JSC::getConstructData):

  • runtime/NativeErrorConstructor.cpp:
  • runtime/NativeErrorConstructor.h:
  • runtime/NumberConstructor.cpp:
  • runtime/NumberConstructor.h:
  • runtime/ObjectConstructor.cpp:
  • runtime/ObjectConstructor.h:
  • runtime/RegExpConstructor.cpp:
  • runtime/RegExpConstructor.h:
  • runtime/StringConstructor.cpp:
  • runtime/StringConstructor.h:

Source/WebCore:

No new tests.

Removed all declarations and definitions of getConstructDataVirtual.
Also replaced all call sites to getConstructDataVirtual with a
corresponding lookup in the MethodTable.

  • WebCore.exp.in:
  • bindings/js/JSAudioConstructor.cpp:
  • bindings/js/JSAudioConstructor.h:
  • bindings/js/JSImageConstructor.cpp:
  • bindings/js/JSImageConstructor.h:
  • bindings/js/JSOptionConstructor.cpp:
  • bindings/js/JSOptionConstructor.h:
  • bindings/js/ScriptFunctionCall.cpp:

(WebCore::ScriptFunctionCall::construct):

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateConstructorDeclaration):
(GenerateConstructorDefinition):

  • bridge/runtime_object.cpp:
  • bridge/runtime_object.h:

Source/WebKit/mac:

Removed all declarations and definitions of getConstructDataVirtual.
Also replaced all call sites to getConstructDataVirtual with a
corresponding lookup in the MethodTable.

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::construct):

Source/WebKit2:

Removed all declarations and definitions of getConstructDataVirtual.
Also replaced all call sites to getConstructDataVirtual with a
corresponding lookup in the MethodTable.

  • WebProcess/Plugins/Netscape/JSNPObject.cpp:
  • WebProcess/Plugins/Netscape/JSNPObject.h:
11:01 AM Changeset in webkit [98202] by ggaren@apple.com
  • 3 edits in trunk/LayoutTests

Not reviewed.

Updated two test results I missed in my last check-in.

These tests depended on non-deterministic effects of hash order, so I
had to update them:

  • http/tests/inspector/search/search-in-resources-expected.txt:
  • platform/mac/fast/dom/prototype-inheritance-2-expected.txt:
12:34 AM Changeset in webkit [98201] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Try to fix the SL build.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::InlineStackEntry::InlineStackEntry): Cast
away int vs unisgned warning.

12:27 AM Changeset in webkit [98200] by commit-queue@webkit.org
  • 5 edits
    1 add in trunk

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

Tools:

Patch by Kaustubh Atrawalkar <Kaustubh Atrawalkar> on 2011-10-23
Reviewed by Martin Robinson.

Implement the KeyDown function on Gtk platform.

  • WebKitTestRunner/GNUmakefile.am:
  • WebKitTestRunner/TestController.cpp:

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

  • WebKitTestRunner/gtk/EventSenderProxyGtk.cpp: Added.

(WTR::EventSenderProxy::EventSenderProxy):
(WTR::dispatchEvent):
(WTR::getModifiers):
(WTR::getGDKKeySymForKeyRef):
(WTR::EventSenderProxy::keyDown):

LayoutTests:

Unskip passed tests.

Patch by Kaustubh Atrawalkar <Kaustubh Atrawalkar> on 2011-10-23
Reviewed by Martin Robinson.

  • platform/gtk-wk2/Skipped:
12:21 AM Changeset in webkit [98199] by ggaren@apple.com
  • 13 edits in trunk

Separated string lifetime bits from character buffer state bits
​https://bugs.webkit.org/show_bug.cgi?id=70673

Source/JavaScriptCore:

Reviewed by Anders Carlsson.

Moved the static/immortal bit into the bottom bit of the refcount, and
moved all other bits into the high bits of the hash code.

This is the first step toward a new Characters/PassString class, and it
makes ref/deref slightly more efficient.

  • create_hash_table:
  • wtf/StringHasher.h:

(WTF::StringHasher::hash): Tweaked the string hashing function to leave
the top bits clear, so they can be used as flags.

Fixed some small differences between the PERL copy of this function and
the C++ copy of this function, which could have in theory caused subtle
crashes.

  • wtf/text/StringImpl.cpp:

(WTF::StringImpl::sharedBuffer):
(WTF::StringImpl::createWithTerminatingNullCharacter):

  • wtf/text/StringImpl.h:

(WTF::StringImpl::StringImpl):
(WTF::StringImpl::cost): Renamed s_refCountFlagShouldReportedCost to
s_didReportExtraCost, since the original name was both self-contradictory
and used as a double-negative.

(WTF::StringImpl::isIdentifier):
(WTF::StringImpl::setIsIdentifier):
(WTF::StringImpl::hasTerminatingNullCharacter):
(WTF::StringImpl::isAtomic):
(WTF::StringImpl::setIsAtomic):
(WTF::StringImpl::setHash):
(WTF::StringImpl::rawHash):
(WTF::StringImpl::hasHash):
(WTF::StringImpl::existingHash):
(WTF::StringImpl::hash):
(WTF::StringImpl::hasOneRef):
(WTF::StringImpl::ref):
(WTF::StringImpl::deref):
(WTF::StringImpl::bufferOwnership):
(WTF::StringImpl::isStatic): Moved the static/immortal bit into the bottom
bit of the refcount. Now, all lifetime information lives in the refcount
field. Moved the other bits into the hash code field.

Source/WebCore:

Reviewed by Anders Carlsson.

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateHashValue): Updated for string hashing changes in JavaScriptCore.

LayoutTests:

Reviewed by Anders Carlsson.

These tests depended on non-deterministic effects of hash order, so I
had to update them:

  • fast/dom/prototype-inheritance-2-expected.txt: Updated for slightly

different order of discovering properties.

  • fast/js/delete-syntax-expected.txt:
  • fast/js/script-tests/delete-syntax.js: Updated not to depend on iteration

order, since that's not what this test is about.

  • http/tests/inspector/inspector-test.js: Updated not to depend on

global function iteration order. This test setup is pretty spaghetti.
I think I avoided making it even more spaghetti.

  • inspector/storage-panel-dom-storage-expected.txt: Updated for changed

iteration order.

Oct 22, 2011:

10:03 PM Changeset in webkit [98198] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

HTTPBodyStream in NSURLRequest gets lost when using CFNetwork loader
​https://bugs.webkit.org/show_bug.cgi?id=70583
<rdar://problem/10281227>

Reviewed by Darin Adler.

Call WebCore::setHTTPBody() only if we actually have a FormData. Unconditionally calling it
results in us passing 0 to wkCFURLRequestSetHTTPRequestBodyParts() which resets any input
stream specified by the user on the NSURLRequest. This code matches the behavior in the
Foundation version of doUpdatePlatformRequest().

  • platform/network/cf/ResourceRequestCFNet.cpp:

(WebCore::ResourceRequest::doUpdatePlatformRequest):

5:40 PM Changeset in webkit [98197] by benjamin@webkit.org
  • 3 edits in trunk/LayoutTests

Make relayout-nested-positioned-elements-crash.html more reliable
​https://bugs.webkit.org/show_bug.cgi?id=70672

Reviewed by Kenneth Rohde Christiansen.

The test relayout-nested-positioned-elements-crash relies on implementation
details of the test runner. If the function runTest() is run
during the test, the result will be different than the expected results.

Change the test to use the Javascript test framework.
Ensure the element has the correct style in the final state.

  • fast/block/positioning/relayout-nested-positioned-elements-crash-expected.txt:
  • fast/block/positioning/relayout-nested-positioned-elements-crash.html:
4:46 PM Changeset in webkit [98196] by commit-queue@webkit.org
  • 19 edits
    30 moves
    1 add
    1 delete in trunk/Source/WebCore

Move all MediaStream and PeerConnection related non-platform files to WebCore/mediastream
​https://bugs.webkit.org/show_bug.cgi?id=70233

Patch by Adam Bergkvist <​adam.bergkvist@ericsson.com> on 2011-10-22
Reviewed by Adam Barth.

Removed affected files from build systems that don't have a MEDIA_STREAM build flag.

No new tests since we're just moving files.

  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • GNUmakefile.list.am:
  • UseJSC.cmake:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • dom/DOMAllInOne.cpp:
  • dom/DispatchTask.h: Removed unused file.
  • dom/ScriptExecutionContext.cpp:
  • mediastream/LocalMediaStream.cpp: Renamed from Source/WebCore/dom/LocalMediaStream.cpp.

(WebCore::LocalMediaStream::create):
(WebCore::LocalMediaStream::LocalMediaStream):
(WebCore::LocalMediaStream::stop):
(WebCore::LocalMediaStream::stopTimerFired):
(WebCore::LocalMediaStream::~LocalMediaStream):
(WebCore::LocalMediaStream::toLocalMediaStream):

  • mediastream/LocalMediaStream.h: Renamed from Source/WebCore/dom/LocalMediaStream.h.
  • mediastream/LocalMediaStream.idl: Renamed from Source/WebCore/dom/LocalMediaStream.idl.
  • mediastream/MediaStream.cpp: Renamed from Source/WebCore/dom/MediaStream.cpp.

(WebCore::MediaStream::create):
(WebCore::MediaStream::MediaStream):
(WebCore::MediaStream::~MediaStream):
(WebCore::MediaStream::readyState):
(WebCore::MediaStream::toMediaStream):
(WebCore::MediaStream::streamEnded):
(WebCore::MediaStream::scriptExecutionContext):
(WebCore::MediaStream::eventTargetData):
(WebCore::MediaStream::ensureEventTargetData):

  • mediastream/MediaStream.h: Renamed from Source/WebCore/dom/MediaStream.h.

(WebCore::MediaStream::label):
(WebCore::MediaStream::tracks):
(WebCore::MediaStream::descriptor):
(WebCore::MediaStream::refEventTarget):
(WebCore::MediaStream::derefEventTarget):

  • mediastream/MediaStream.idl: Renamed from Source/WebCore/dom/MediaStream.idl.
  • mediastream/MediaStreamClient.h: Renamed from Source/WebCore/page/MediaStreamClient.h.

(WebCore::MediaStreamClient::~MediaStreamClient):

  • mediastream/MediaStreamController.cpp: Renamed from Source/WebCore/page/MediaStreamController.cpp.

(WebCore::MediaStreamController::Request::Request):
(WebCore::MediaStreamController::Request::localId):
(WebCore::MediaStreamController::Request::frameController):
(WebCore::MediaStreamController::MediaStreamController):
(WebCore::MediaStreamController::~MediaStreamController):
(WebCore::MediaStreamController::isClientAvailable):
(WebCore::MediaStreamController::unregisterFrameController):
(WebCore::MediaStreamController::registerRequest):
(WebCore::MediaStreamController::generateStream):
(WebCore::MediaStreamController::streamGenerated):
(WebCore::MediaStreamController::streamGenerationFailed):

  • mediastream/MediaStreamController.h: Renamed from Source/WebCore/page/MediaStreamController.h.
  • mediastream/MediaStreamEvent.cpp: Renamed from Source/WebCore/p2p/MediaStreamEvent.cpp.

(WebCore::MediaStreamEvent::create):
(WebCore::MediaStreamEvent::MediaStreamEvent):
(WebCore::MediaStreamEvent::~MediaStreamEvent):
(WebCore::MediaStreamEvent::initMediaStreamEvent):
(WebCore::MediaStreamEvent::stream):

  • mediastream/MediaStreamEvent.h: Renamed from Source/WebCore/p2p/MediaStreamEvent.h.

(WebCore::MediaStreamEvent::isMediaStreamEvent):

  • mediastream/MediaStreamEvent.idl: Renamed from Source/WebCore/p2p/MediaStreamEvent.idl.
  • mediastream/MediaStreamFrameController.cpp: Renamed from Source/WebCore/page/MediaStreamFrameController.cpp.

(WebCore::MediaStreamFrameController::Request::~Request):
(WebCore::MediaStreamFrameController::Request::scriptExecutionContext):
(WebCore::MediaStreamFrameController::Request::isGenerateStreamRequest):
(WebCore::MediaStreamFrameController::Request::Request):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::create):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::~GenerateStreamRequest):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::isGenerateStreamRequest):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::abort):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::successCallback):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::errorCallback):
(WebCore::MediaStreamFrameController::GenerateStreamRequest::GenerateStreamRequest):
(WebCore::MediaStreamFrameController::RequestMap::abort):
(WebCore::MediaStreamFrameController::RequestMap::abortAll):
(WebCore::MediaStreamFrameController::MediaStreamFrameController):
(WebCore::MediaStreamFrameController::~MediaStreamFrameController):
(WebCore::MediaStreamFrameController::securityOrigin):
(WebCore::MediaStreamFrameController::scriptExecutionContext):
(WebCore::MediaStreamFrameController::pageController):
(WebCore::MediaStreamFrameController::enterDetachedState):
(WebCore::MediaStreamFrameController::isClientAvailable):
(WebCore::MediaStreamFrameController::disconnectPage):
(WebCore::MediaStreamFrameController::disconnectFrame):
(WebCore::MediaStreamFrameController::transferToNewPage):
(WebCore::MediaStreamFrameController::parseGenerateStreamOptions):
(WebCore::MediaStreamFrameController::generateStream):
(WebCore::MediaStreamFrameController::streamGenerated):
(WebCore::MediaStreamFrameController::streamGenerationFailed):

  • mediastream/MediaStreamFrameController.h: Renamed from Source/WebCore/page/MediaStreamFrameController.h.

(WebCore::MediaStreamFrameController::IdGenerator::IdGenerator):
(WebCore::MediaStreamFrameController::IdGenerator::getNextId):

  • mediastream/MediaStreamList.cpp: Renamed from Source/WebCore/dom/MediaStreamList.cpp.

(WebCore::MediaStreamList::create):
(WebCore::MediaStreamList::MediaStreamList):
(WebCore::MediaStreamList::~MediaStreamList):
(WebCore::MediaStreamList::length):
(WebCore::MediaStreamList::item):
(WebCore::MediaStreamList::append):
(WebCore::MediaStreamList::remove):
(WebCore::MediaStreamList::contains):

  • mediastream/MediaStreamList.h: Renamed from Source/WebCore/dom/MediaStreamList.h.
  • mediastream/MediaStreamList.idl: Renamed from Source/WebCore/dom/MediaStreamList.idl.
  • mediastream/MediaStreamRegistry.cpp: Renamed from Source/WebCore/platform/MediaStreamRegistry.cpp.

(WebCore::MediaStreamRegistry::registry):
(WebCore::MediaStreamRegistry::registerMediaStreamURL):
(WebCore::MediaStreamRegistry::unregisterMediaStreamURL):
(WebCore::MediaStreamRegistry::mediaStream):

  • mediastream/MediaStreamRegistry.h: Renamed from Source/WebCore/platform/MediaStreamRegistry.h.
  • mediastream/MediaStreamTrack.cpp: Renamed from Source/WebCore/dom/MediaStreamTrack.cpp.

(WebCore::MediaStreamTrack::create):
(WebCore::MediaStreamTrack::MediaStreamTrack):
(WebCore::MediaStreamTrack::~MediaStreamTrack):
(WebCore::MediaStreamTrack::kind):
(WebCore::MediaStreamTrack::label):
(WebCore::MediaStreamTrack::enabled):
(WebCore::MediaStreamTrack::setEnabled):

  • mediastream/MediaStreamTrack.h: Renamed from Source/WebCore/dom/MediaStreamTrack.h.
  • mediastream/MediaStreamTrack.idl: Renamed from Source/WebCore/dom/MediaStreamTrack.idl.
  • mediastream/MediaStreamTrackList.cpp: Renamed from Source/WebCore/dom/MediaStreamTrackList.cpp.

(WebCore::MediaStreamTrackList::create):
(WebCore::MediaStreamTrackList::MediaStreamTrackList):
(WebCore::MediaStreamTrackList::~MediaStreamTrackList):
(WebCore::MediaStreamTrackList::length):
(WebCore::MediaStreamTrackList::item):

  • mediastream/MediaStreamTrackList.h: Renamed from Source/WebCore/dom/MediaStreamTrackList.h.
  • mediastream/MediaStreamTrackList.idl: Renamed from Source/WebCore/dom/MediaStreamTrackList.idl.
  • mediastream/PeerConnection.cpp: Renamed from Source/WebCore/p2p/PeerConnection.cpp.

(WebCore::PeerConnection::create):
(WebCore::PeerConnection::PeerConnection):
(WebCore::PeerConnection::~PeerConnection):
(WebCore::PeerConnection::processSignalingMessage):
(WebCore::PeerConnection::readyState):
(WebCore::PeerConnection::send):
(WebCore::PeerConnection::addStream):
(WebCore::PeerConnection::removeStream):
(WebCore::PeerConnection::localStreams):
(WebCore::PeerConnection::remoteStreams):
(WebCore::PeerConnection::close):
(WebCore::PeerConnection::iceProcessingCompleted):
(WebCore::PeerConnection::sdpGenerated):
(WebCore::PeerConnection::dataStreamMessageReceived):
(WebCore::PeerConnection::remoteStreamAdded):
(WebCore::PeerConnection::remoteStreamRemoved):
(WebCore::PeerConnection::toPeerConnection):
(WebCore::PeerConnection::scriptExecutionContext):
(WebCore::PeerConnection::stop):
(WebCore::PeerConnection::eventTargetData):
(WebCore::PeerConnection::ensureEventTargetData):
(WebCore::PeerConnection::scheduleInitialNegotiation):
(WebCore::PeerConnection::initialNegotiationTimerFired):
(WebCore::PeerConnection::ensureStreamChangeScheduled):
(WebCore::PeerConnection::streamChangeTimerFired):
(WebCore::PeerConnection::scheduleReadyStateChange):
(WebCore::PeerConnection::readyStateChangeTimerFired):
(WebCore::PeerConnection::changeReadyState):

  • mediastream/PeerConnection.h: Renamed from Source/WebCore/p2p/PeerConnection.h.

(WebCore::PeerConnection::refEventTarget):
(WebCore::PeerConnection::derefEventTarget):

  • mediastream/PeerConnection.idl: Renamed from Source/WebCore/p2p/PeerConnection.idl.
  • mediastream/SignalingCallback.h: Renamed from Source/WebCore/p2p/SignalingCallback.h.

(WebCore::SignalingCallback::~SignalingCallback):

  • mediastream/SignalingCallback.idl: Renamed from Source/WebCore/p2p/SignalingCallback.idl.
  • page/Frame.cpp:
  • page/Navigator.cpp:
  • page/Page.cpp:
3:38 PM Changeset in webkit [98195] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[WK2][GTK] run-launcher script fails after testing WK2 tests
​https://bugs.webkit.org/show_bug.cgi?id=70676

Patch by Zan Dobersek <​zandobersek@gmail.com> on 2011-10-22
Reviewed by Andreas Kling.

Pass the '-2' flag instead of '--webkit-test-runner' when executing
run-launcher script after testing with WebKitTestRunner.

  • Scripts/webkitpy/layout_tests/port/gtk.py:
12:31 PM Changeset in webkit [98194] by pfeldman@chromium.org
  • 40 edits in trunk

Source/WebCore: Web Inspector: get rid of View::hide, View::set visible, View::attach,
View::set visible.
​https://bugs.webkit.org/show_bug.cgi?id=70523

This change also introduces assetions that forces view attach / detach operations
to be performed explicitly.

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-10-20
Reviewed by Yury Semikhatsky.

  • inspector/front-end/ApplicationCacheItemsView.js:

(WebInspector.ApplicationCacheItemsView.prototype.wasShown):
(WebInspector.ApplicationCacheItemsView.prototype.willHide):
(WebInspector.ApplicationCacheItemsView.prototype._updateCallback):

  • inspector/front-end/AuditsPanel.js:

(WebInspector.AuditsPanel.prototype.wasShown):

  • inspector/front-end/ConsolePanel.js:

(WebInspector.ConsolePanel.prototype.wasShown):
(WebInspector.ConsolePanel.prototype.willHide):

  • inspector/front-end/ConsoleView.js:

(WebInspector.ConsoleView.prototype.wasShown):

  • inspector/front-end/CookieItemsView.js:

(WebInspector.CookieItemsView.prototype.wasShown):
(WebInspector.CookieItemsView.prototype.willHide):
(WebInspector.CookieItemsView.prototype._updateWithCookies):

  • inspector/front-end/DOMStorageItemsView.js:

(WebInspector.DOMStorageItemsView.prototype.wasShown):
(WebInspector.DOMStorageItemsView.prototype.willHide):

  • inspector/front-end/DatabaseQueryView.js:
  • inspector/front-end/DatabaseTableView.js:

(WebInspector.DatabaseTableView.prototype.wasShown):

  • inspector/front-end/DetailedHeapshotView.js:

(WebInspector.DetailedHeapshotView.prototype.willHide):
(WebInspector.DetailedHeapshotView.prototype._changeView):

  • inspector/front-end/Drawer.js:

(WebInspector.Drawer.prototype.hide):

  • inspector/front-end/ElementsPanel.js:

(WebInspector.ElementsPanel.prototype.wasShown):
(WebInspector.ElementsPanel.prototype.willHide):

  • inspector/front-end/EmptyView.js:

(WebInspector.EmptyView.prototype.wasShown):

  • inspector/front-end/FontView.js:

(WebInspector.FontView.prototype.wasShown):

  • inspector/front-end/IFrameView.js:

(WebInspector.IFrameView):

  • inspector/front-end/ImageView.js:

(WebInspector.ImageView.prototype.wasShown):

  • inspector/front-end/NetworkItemView.js:

(WebInspector.NetworkItemView.prototype.wasShown):
(WebInspector.ResourceContentView.prototype.wasShown):

  • inspector/front-end/NetworkPanel.js:

(WebInspector.NetworkLogView.prototype.wasShown):
(WebInspector.NetworkLogView.prototype.willHide):
(WebInspector.NetworkPanel.prototype.wasShown):

  • inspector/front-end/Panel.js:

(WebInspector.Panel):
(WebInspector.Panel.prototype.show):
(WebInspector.Panel.prototype.wasShown):
(WebInspector.Panel.prototype.willHide):

  • inspector/front-end/PanelEnablerView.js:
  • inspector/front-end/ProfileView.js:

(WebInspector.CPUProfileView.prototype.willHide):

  • inspector/front-end/ProfilesPanel.js:

(WebInspector.ProfilesPanel.prototype.wasShown):
(WebInspector.ProfilesPanel.prototype._addProfileHeader):
(WebInspector.ProfilesPanel.prototype._updateInterface):
(WebInspector.ProfilesPanel.prototype._enableDetailedHeapProfiles):

  • inspector/front-end/ResourceCookiesView.js:

(WebInspector.ResourceCookiesView.prototype.wasShown):

  • inspector/front-end/ResourceHTMLView.js:

(WebInspector.ResourceHTMLView.prototype.wasShown):
(WebInspector.ResourceHTMLView.prototype.willHide):

  • inspector/front-end/ResourceJSONView.js:

(WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype.wasShown):

  • inspector/front-end/ResourceTimingView.js:

(WebInspector.ResourceTimingView.prototype.wasShown):

  • inspector/front-end/ResourcesPanel.js:

(WebInspector.ResourcesPanel.prototype.wasShown):

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel.prototype.wasShown):
(WebInspector.ScriptsPanel.prototype._updateDebuggerButtons):

  • inspector/front-end/SourceFrame.js:

(WebInspector.SourceFrame.prototype.wasShown):

  • inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel.prototype.wasShown):
(WebInspector.TimelinePanel.prototype.willHide):

  • inspector/front-end/View.js:

(WebInspector.View):
(WebInspector.View.prototype.get visible):
(WebInspector.View.prototype.setHideOnDetach):
(WebInspector.View.prototype._wasShown):
(WebInspector.View.prototype._willHide):
(WebInspector.View.prototype._onResize):
(WebInspector.View.prototype.wasShown):
(WebInspector.View.prototype.willHide):
(WebInspector.View.prototype.onResize):
(WebInspector.View.prototype.show):
(WebInspector.View.prototype.detach):
(WebInspector.View.prototype.doResize):
(WebInspector.View.prototype.dispatchToSelfAndChildren):
(WebInspector.View.prototype.dispatchToChildren):
(WebInspector.View.prototype._handleInsertedIntoDocument):

LayoutTests: Web Inspector: get rid of View::hide, View::set visible, View::attach.
​https://bugs.webkit.org/show_bug.cgi?id=70523

Patch by Pavel Feldman <​pfeldman@google.com> on 2011-10-20
Reviewed by Yury Semikhatsky.

  • inspector/debugger/scripts-panel.html:
  • inspector/view-events-expected.txt:
  • inspector/view-events.html:
9:38 AM Changeset in webkit [98193] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[WK2][GTK] Missing env variables cause WKTR to crash
​https://bugs.webkit.org/show_bug.cgi?id=70674

Patch by Zan Dobersek <​zandobersek@gmail.com> on 2011-10-22
Reviewed by Martin Robinson.

Set two environment variables that the Gtk's port of WebKitTestRunner
depends on and crashes if they're not explicitly set by the user.
Also modify the setting of the WEBKIT_INSPECTOR_PATH variable so it
properly uses the _build_path method.

  • Scripts/webkitpy/layout_tests/port/gtk.py: Set

TEST_RUNNER_INJECTED_BUNDLE_FILENAME and TEST_RUNNER_TEST_PLUGIN_PATH
environment variables.

3:53 AM Changeset in webkit [98192] by tkent@chromium.org
  • 2 edits in trunk/Source/WebCore

REGRESSION(r92480): Passing uninitialized variable in HTMLElement::calculateAndAdjustDirectionality()
​https://bugs.webkit.org/show_bug.cgi?id=70582

Reviewed by Darin Adler.

No new tests. It's very hard to make a reliable test.

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::directionality):
Clear *strongDirectionalityTextNode even if hasStrongDirectionality is false.

Note: See TracTimeline for information about the timeline view.