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

Timeline



Dec 25, 2011:

11:05 PM Changeset in webkit [103679] by commit-queue@webkit.org
  • 30 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=74067
Refactoring: Unitfy willMoveToNewDocument() and didMoveToNewDocument()

Patch by Hajime Morrita <​morrita@chromium.org> on 2011-12-25
Reviewed by Darin Adler.

No new tests. No behavior change.

This change combines two method Node::willMoveToNewOwnerDocument() and Node::didMoveToNewOwnerDocument()
into Node::didMoveToNewDocument(Document* oldDocument).

The intention of this change is:

  • Making upcoming refactoring (Bug 59816) possible. The refactoring will turn Node::m_document into Node::m_treeScope, and we will no longer have Node::setDocument() where we can invoke both willMoveToNewDocument() and didMoveToNewDocument() at once.
  • Killing one extra virtual method call.
  • Making the concept of "move" clearer by keeping such an operation into the single method.
  • dom/Node.cpp:

(WebCore::setWillMoveToNewDocumentWasCalled):
(WebCore::setDidMoveToNewDocumentWasCalled):
(WebCore::Node::setDocument):
(WebCore::Node::didMoveToNewDocument):

  • dom/Node.h:
  • dom/StyledElement.cpp:

(WebCore::StyledElement::attributeChanged):

  • html/FormAssociatedElement.cpp:

(WebCore::FormAssociatedElement::didMoveToNewDocument):

  • html/FormAssociatedElement.h:
  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::didMoveToNewDocument):
(WebCore::HTMLFormControlElementWithState::didMoveToNewDocument):

  • html/HTMLFormControlElement.h:
  • html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::didMoveToNewDocument):

  • html/HTMLFormElement.h:
  • html/HTMLImageElement.cpp:

(WebCore::HTMLImageElement::didMoveToNewDocument):

  • html/HTMLImageElement.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::didMoveToNewDocument):

  • html/HTMLInputElement.h:
  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::didMoveToNewDocument):

  • html/HTMLMediaElement.h:
  • html/HTMLObjectElement.cpp:

(WebCore::HTMLObjectElement::didMoveToNewDocument):

  • html/HTMLObjectElement.h:
  • html/HTMLPlugInImageElement.cpp:

(WebCore::HTMLPlugInImageElement::didMoveToNewDocument):

  • html/HTMLPlugInImageElement.h:
  • html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::didMoveToNewDocument):

  • html/HTMLVideoElement.h:
  • html/ImageDocument.cpp:

(WebCore::ImageDocumentElement::didMoveToNewDocument):

  • html/ImageInputType.cpp:

(WebCore::ImageInputType::willMoveToNewDocument):

  • loader/ImageLoader.cpp:

(WebCore::ImageLoader::elementDidMoveToNewDocument):

  • loader/ImageLoader.h:
  • svg/SVGImageElement.cpp:

(WebCore::SVGImageElement::didMoveToNewDocument):

  • svg/SVGImageElement.h:
  • svg/SVGSVGElement.cpp:

(WebCore::SVGSVGElement::didMoveToNewDocument):

  • svg/SVGSVGElement.h:
10:16 PM Changeset in webkit [103678] by haraken@chromium.org
  • 8 edits in trunk/Source/WebCore

REGRESSION(r102987): Fix the filename prefix of the generated empty .h
and .cpp files for [Supplemental] IDLs
​https://bugs.webkit.org/show_bug.cgi?id=75082

Reviewed by Darin Adler.

In bug 74481, we changed generate-bindings.pl so that it generates empty .h
and .cpp files for the [Supplemental] IDLs. However, the filename prefixes of
those .h and .cpp files are wrong. This patch fixes the prefixes as follows:

generator=JS => JS*.h, JS*.cpp
generator=V8 => V8*.h, V8*.cpp
generator=ObjC => DOM*.h, DOM*.cpp
generator=GObject => WebKitDOM*.h, WebKitDOM*.cpp
generator=CPP => WebDOM*.h, WebDOM*.cpp

No new tests. No change in behavior.
I confirmed that the names of generated .h and .cpp files are correct.

  • bindings/scripts/CodeGenerator.pm:

(FileNamePrefix): Returns the prefix of file names.

  • bindings/scripts/CodeGeneratorCPP.pm:

(GenerateInterface): Uses CodeGenerator::FileNamePrefix.

  • bindings/scripts/CodeGeneratorGObject.pm:

(GenerateInterface): Ditto.

  • bindings/scripts/CodeGeneratorJS.pm:

(GenerateInterface): Ditto.

  • bindings/scripts/CodeGeneratorObjC.pm:

(GenerateInterface): Ditto.

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateInterface): Ditto.

  • bindings/scripts/generate-bindings.pl:

(generateEmptyHeaderAndCpp): Ditto.

9:20 PM Changeset in webkit [103677] by mitz@apple.com
  • 7 edits in trunk/Source

../WebCore: WebCore changes for: Find indicators overlap when a match spans multiple text boxes
​https://bugs.webkit.org/show_bug.cgi?id=75220

Reviewed by Darin Adler.

  • WebCore.exp.in: Exported new unionRect(const Vector<FloatRect>&) and existing

FloatRect::intersects().

  • platform/graphics/FloatRect.cpp:

(WebCore::unionRect): Added.

  • platform/graphics/FloatRect.h:

../WebKit2: Find indicators overlap when a match spans multiple text boxes
​https://bugs.webkit.org/show_bug.cgi?id=75220

Reviewed by Darin Adler.

  • UIProcess/FindIndicator.cpp:

(WebKit::findIndicatorsForTextRectsOverlap): Added this helper function that checks for
pairwise intersections between all indicator rects.
(WebKit::FindIndicator::FindIndicator): Changed to use a single rect (the union of all text
rects) if any two indicator rects would otherwise overlap. This is similar to what Safari
does, and it eliminates overlapping rects for adjacent text boxes. In rare cases (such as when
a match spans two lines and adjacent text boxes on one of those lines) it results in a find
indicator that is too large and obscures some non-match text.

  • UIProcess/FindIndicator.h:
8:46 PM Changeset in webkit [103676] by Darin Adler
  • 6 edits in trunk/Source/WebCore

Use OwnPtr for CSSFontFace::m_sources
​https://bugs.webkit.org/show_bug.cgi?id=75219

Reviewed by Dan Bernstein.

  • css/CSSFontFace.cpp:

(WebCore::CSSFontFace::isLoaded): Use the same size_t idiom here as in the rest
of the functions.
(WebCore::CSSFontFace::isValid): Ditto. Also removed unneeded empty special casing.
(WebCore::CSSFontFace::addSource): Changed to take a PassOwnPtr. Reordered so the
PassOwnPtr zeroing does not cause trouble.
(WebCore::CSSFontFace::getFontData): Added call to get.
(WebCore::CSSFontFace::hasSVGFontFaceSource): Use the same size_t idiom here as in
the rest of the functions.

  • css/CSSFontFace.h: Use PassOwnPtr for addSource argument, and Vector<OwnPtr> for

the m_sources vector.

  • css/CSSFontSelector.cpp:

(WebCore::CSSFontSelector::addFontFaceRule): Use OwnPtr and PassOwnPtr for font face
sources that are passed to addSource.

  • css/CSSSegmentedFontFace.cpp: Added a now-needed include.
8:02 PM Changeset in webkit [103675] by jarred@sencha.com
  • 3 edits
    6 adds in trunk

Allow XMLHttpRequest responseType to be set at any state up to and including HEADERS_RECEIVED
​https://bugs.webkit.org/show_bug.cgi?id=75190

Source/WebCore:

XMLHttpRequest.responseType should be modifiable at any state up to and including the
HEADERS_RECEIVED state. Therefore, subsequent calls to open() should not reset responseType
to its default value, and calls to open() must follow the same spec mandate set forth in
setResponseType() for synchronous HTTP(S) requests made from the window context.

Reviewed by Alexey Proskuryakov.

Tests: fast/xmlhttprequest/xmlhttprequest-responsetype-before-open-sync-request.html

fast/xmlhttprequest/xmlhttprequest-responsetype-before-open.html
fast/xmlhttprequest/xmlhttprequest-responsetype-set-at-headers-received.html

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::setResponseType):
Prevent setting the value only when in LOADING and DONE states. No longer check if
m_loader is present, which is instantiated on a call to send(), because responseType
can be safely changed after a request is sent.
(WebCore::XMLHttpRequest::open):
Do not reset m_responseTypeCode to the default value, and prevent calls to open()
for synchronous HTTP(S) requests made from the window context when m_responseTypeCode
is not the default value.

LayoutTests:

Reviewed by Alexey Proskuryakov.

  • fast/xmlhttprequest/xmlhttprequest-responsetype-before-open-expected.txt: Added.
  • fast/xmlhttprequest/xmlhttprequest-responsetype-before-open-sync-request-expected.txt: Added.
  • fast/xmlhttprequest/xmlhttprequest-responsetype-before-open-sync-request.html: Added.

Validate that calls to open() for synchronous HTTP(S) requests from the window context will
properly fail when responseType has been previously set to a non-default value.

  • fast/xmlhttprequest/xmlhttprequest-responsetype-before-open.html: Added.

Validate that XMLHttpRequest.responseType can be set prior to a call to open().

  • fast/xmlhttprequest/xmlhttprequest-responsetype-set-at-headers-received-expected.txt: Added.
  • fast/xmlhttprequest/xmlhttprequest-responsetype-set-at-headers-received.html: Added.

Validate that XMLHttpRequest.responseType can be set at the HEADERS_RECEIVED state, but
no state later than that.

6:16 PM Changeset in webkit [103674] by fpizlo@apple.com
  • 10 edits in trunk/Source/JavaScriptCore

The ArgumentCount field in the CallFrame should have its tag left blank for other uses
​https://bugs.webkit.org/show_bug.cgi?id=75199
<rdar://problem/10625105>
<rdar://problem/10625106>

Reviewed by Oliver Hunt.

  • dfg/DFGOSRExitCompiler32_64.cpp:

(JSC::DFG::OSRExitCompiler::compileExit):

  • dfg/DFGOSRExitCompiler64.cpp:

(JSC::DFG::OSRExitCompiler::compileExit):

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::argumentPayloadSlot):

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::emitCall):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::emitCall):

  • interpreter/CallFrame.h:

(JSC::ExecState::argumentCountIncludingThis):
(JSC::ExecState::setArgumentCountIncludingThis):

  • interpreter/Register.h:

(JSC::Register::unboxedInt32):
(JSC::Register::unboxedBoolean):
(JSC::Register::unboxedCell):
(JSC::Register::payload):
(JSC::Register::tag):

  • jit/JITCall.cpp:

(JSC::JIT::compileOpCall):

  • jit/JITCall32_64.cpp:

(JSC::JIT::compileLoadVarargs):
(JSC::JIT::compileOpCall):

1:32 PM Changeset in webkit [103673] by weinig@apple.com
  • 2 edits in trunk/Source/WebCore

Fix tests failing as a result of r103643
​https://bugs.webkit.org/show_bug.cgi?id=75209

Reviewed by Dan Bernstein.

Switch accidental switch of default scroll granularity from
ScrollByPageWheelEvent back to ScrollByPixelWheelEvent.

  • platform/mac/PlatformEventFactory.mm:

(WebCore::PlatformWheelEventBuilder::PlatformWheelEventBuilder):

1:21 PM Changeset in webkit [103672] by kling@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Yarr: Avoid copying vectors in CharacterClassConstructor.
<​http://webkit.org/b/75206>

Reviewed by Darin Adler.

Yarr::CharacterClassConstructor::charClass() was hot when loading twitter
feeds (1.2%), replace the usage of Vector::append() by swap() since we're
always clearing the source vector afterwards anyway.

  • yarr/YarrPattern.cpp:

(JSC::Yarr::CharacterClassConstructor::charClass):

11:03 AM Changeset in webkit [103671] by Darin Adler
  • 2 edits in trunk/Source/WebCore

Change CSS canvas code that does HashMap get/set to use the more efficient add idiom
​https://bugs.webkit.org/show_bug.cgi?id=75204

Reviewed by Dan Bernstein.

  • dom/Document.cpp:

(WebCore::Document::getCSSCanvasContext): Change local variable name of element to
element; it's not the function result, so not good to name it result.
(WebCore::Document::getCSSCanvasElement): Use add instead of get/set so we only do
one hash table lookup.

9:08 AM Changeset in webkit [103670] by Darin Adler
  • 2 edits in trunk/Source/ThirdParty
  • gtest/xcode/gtest.xcodeproj: Added property svn:ignore.
12:24 AM Changeset in webkit [103669] by haraken@chromium.org
  • 3 edits
    2 adds in trunk/Tools

Add unittests for the C++ parser of prepare-ChangeLog
​https://bugs.webkit.org/show_bug.cgi?id=75191

Reviewed by Ryosuke Niwa.

cpp_unittests.cpp is the unittests for get_function_line_ranges_for_cpp()
of prepare-ChangeLog.

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges): Renames get_function_line_ranges_for_c()
to get_function_line_ranges_for_cpp().

  • Scripts/webkitperl/prepare-ChangeLog_unittest/parser_unittests.pl:
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/cpp_unittests-expected.txt: Added.
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/cpp_unittests.cpp: Added.

(func1):
(func2):
(func3):
(func4):
(func5):
(func6):
(func7):
(func8):
(func9):
(func10):
(func11):
(func12):
(func13):
(func14):
(func15):
(funcOverloaded):
(Class::func16):
(Class1::Class2::func17):
(Class2::func18):
(Class2::func19):
(Class2::func20):
(Class2::func21):
(Class2::func22):
(func23):
(func24):
(Class2::func25):
(Class1::func26):
(Class2::func27):
(Class3::func28):
(Class7::operator+):
(Class100::Class100):
(Class101::~Class101):
(Class102::Class102):
(Class103::Class103):
(Struct1::func29):
(Struct2::func30):
(NameSpace1::func30):
(NameSpace1::NameSpace2::func31):

12:15 AM Changeset in webkit [103668] by haraken@chromium.org
  • 3 edits
    2 adds in trunk/Tools

Add unittests for the Java parser of prepare-ChangeLog
​https://bugs.webkit.org/show_bug.cgi?id=75195

Reviewed by Ryosuke Niwa.

java_unittests.java is unittest cases for get_function_line_ranges_for_java()
of prepare-ChangeLog.

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges_for_java): Fixed a parser bug.

interface I
{

void f()
{
}

}

should be recognized as "I.f", and

class C interface I
{

void f()
{
}

}

should be recognized as "C.f".
Without this patch, both are recognized as "I.f".

  • Scripts/webkitperl/prepare-ChangeLog_unittest/parser_unittests.pl:
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/java_unittests-expected.txt: Added.
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/java_unittests.java: Added.

(Simple):
(Simple.func1):
(Simple.func2):
(Simple.func3):
(Simple.func4):
(Simple.func5):
(Simple.func6):
(Simple.func7):
(Simple.func8):
(Simple.func9):
(Simple.func10):
(Simple.funcOverloaded):
(Simple.func11):
(Simple.func12):
(Simple.func13):
(Simple.func14):
(Simple.func15):
(Simple.func16):
(Simple.func17):
(Simple.func18):
(Simple.func19):
(Simple.func20):
(Simple.func21):
(Derived1):
(Derived1.Derived1):
(Derived1.func22):
(Interface1):
(Interface2):
(Interface2.func23):
(Derived2):
(Derived2.Derived2):
(Derived2.func23):

12:09 AM Changeset in webkit [103667] by haraken@chromium.org
  • 3 edits
    2 adds in trunk/Tools

Add unittests for the Python parser of prepare-ChangeLog
​https://bugs.webkit.org/show_bug.cgi?id=75197

Reviewed by Ryosuke Niwa.

python_unittests.py is the unittest for get_function_line_ranges_for_python()
of prepare-ChangeLog. The unittest just contains classes and defs that are not nested.
This is because the current python parser is wrong, and it cannot correctly parse nested
classes and defs (even one def in a class), nor class inheritance.
We will fix it in another patch.

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges_for_python): Ignores comment lines that starts from #.

  • Scripts/webkitperl/prepare-ChangeLog_unittest/parser_unittests.pl:
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/python_unittests-expected.txt: Added.
  • Scripts/webkitperl/prepare-ChangeLog_unittest/resources/python_unittests.py: Added.

(func1):
(func2):
(func3):
(funcInsideComment):
(func4):
(func5):
(func6):
(funcOverloaded):
(Class1):

Dec 24, 2011:

5:09 PM Changeset in webkit [103666] by mitz@apple.com
  • 1355 edits in trunk/LayoutTests/platform

Set the svn:mime-type property of some PNG files to image/png

2:49 PM Changeset in webkit [103665] by Darin Adler
  • 2 edits in trunk/Source/JavaScriptCore

Specialize HashTraits for RefPtr to use PassRefPtr as "pass type" to reduce reference count churn
​https://bugs.webkit.org/show_bug.cgi?id=72476

Reviewed by Sam Weinig.

  • wtf/HashTraits.h: Defined PassInType and store function in HashTraits<RefPtr>.
2:08 PM Changeset in webkit [103664] by kling@webkit.org
  • 5 edits in trunk/Source/WebCore

CSSElementStyleDeclarations should never move between elements.
<​http://webkit.org/b/75198>

Reviewed by Anders Carlsson.

Have the CSSElementStyleDeclaration subclasses take a StyledElement* in
the constructor and replace setElement(StyledElement*) by clearElement().

No behavior change, just enforcing the current behavior at compile-time.

  • css/CSSElementStyleDeclaration.h:

(WebCore::CSSElementStyleDeclaration::clearElement):
(WebCore::CSSElementStyleDeclaration::CSSElementStyleDeclaration):

  • css/CSSInlineStyleDeclaration.h:

(WebCore::CSSInlineStyleDeclaration::create):
(WebCore::CSSInlineStyleDeclaration::CSSInlineStyleDeclaration):

  • dom/StyledElement.cpp:

(WebCore::StyledElement::createInlineStyleDecl):
(WebCore::StyledElement::destroyInlineStyleDecl):

  • svg/SVGFontFaceElement.cpp:

(WebCore::FontFaceStyleDeclaration::FontFaceStyleDeclaration):

1:24 PM Changeset in webkit [103663] by kling@webkit.org
  • 12 edits in trunk/Source/WebCore

Decouple CSSMappedAttributeDeclaration from element completely.
<​http://webkit.org/b/75187>

Reviewed by Darin Adler.

Let CSSMappedAttributeDeclaration inherit from CSSMutableDeclaration instead
of CSSElementStyleDeclaration. Add methods to CSSMappedAttributeDeclaration
for setting properties that also take a StyledElement* and use that mechanism
instead of temporarily associating an element with the declaration.

This reduces the size of mapped attributes by 4/8 bytes, but more importantly
opens a number of ways to simplify style declarations in future patches.

  • css/CSSMutableStyleDeclaration.h:
  • dom/CSSMappedAttributeDeclaration.cpp:

(WebCore::CSSMappedAttributeDeclaration::setNeedsStyleRecalc):
(WebCore::CSSMappedAttributeDeclaration::setMappedImageProperty):
(WebCore::CSSMappedAttributeDeclaration::setMappedLengthProperty):
(WebCore::CSSMappedAttributeDeclaration::setMappedProperty):
(WebCore::CSSMappedAttributeDeclaration::removeMappedProperty):

  • dom/CSSMappedAttributeDeclaration.h:

(WebCore::CSSMappedAttributeDeclaration::CSSMappedAttributeDeclaration):

Add/move methods to CSSMappedAttributeDeclaration for setting/removing
properties that also take a StyledElement*. That element is used for
scheduling style recalc and passing the right document to CSSParser.

  • css/CSSParser.h:
  • css/CSSParser.cpp:

(WebCore::parseColorValue):
(WebCore::parseSimpleLengthValue):
(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseMappedAttributeValue):

Added a parsedMappedAttributeValue() alternative to parseValue() that
takes a StyledElement*.

  • dom/StyledElement.h:
  • html/HTMLElement.cpp:

(WebCore::HTMLElement::setContentEditable):

Add (and use) a StyledElement::removeCSSProperty() complement to the
addCSS*() functions.

  • dom/StyledElement.cpp:

(WebCore::StyledElement::attributeChanged):
(WebCore::StyledElement::removeCSSProperty):
(WebCore::StyledElement::addCSSProperty):
(WebCore::StyledElement::addCSSImageProperty):
(WebCore::StyledElement::addCSSLength):
(WebCore::StyledElement::addCSSColor):
(WebCore::StyledElement::createMappedDecl):

  • html/HTMLTableElement.cpp:

(WebCore::HTMLTableElement::additionalAttributeStyleDecls):
(WebCore::HTMLTableElement::addSharedCellBordersDecl):
(WebCore::HTMLTableElement::addSharedCellPaddingDecl):
(WebCore::HTMLTableElement::addSharedGroupDecls):

Use the setMapped*Property() functions to plumb the element through.

  • css/CSSElementStyleDeclaration.h:

Update comment about CSSElementStyleDeclaration's subclasses.

1:01 PM Changeset in webkit [103662] by mitz@apple.com
  • 2 edits in trunk/Tools

Set SHARED_PRECOMPS_DIR when Xcode is set to use a custom build product path.

Reviewed by Anders Carlsson.

  • Scripts/webkitdirs.pm:

(determineBaseProductDir):

12:29 PM Changeset in webkit [103661] by jarred@sencha.com
  • 3 edits
    2 adds in trunk

Allow XMLHttpRequest withCredentials to be set prior to a call to open()
​https://bugs.webkit.org/show_bug.cgi?id=75194

Source/WebCore:

XMLHttpRequest.withCredentials attribute should be modifiable prior to the OPENED state per
the W3C spec. See ​http://dvcs.w3.org/hg/xhr/raw-file/tip/Overview.html#the-withcredentials-attribute

Reviewed by Alexey Proskuryakov.

Test: fast/xmlhttprequest/xmlhttprequest-withcredentials-before-open.html

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::setWithCredentials):
Prevent setting the value only after the OPENED state.

LayoutTests:

Reviewed by Alexey Proskuryakov.

  • fast/xmlhttprequest/xmlhttprequest-withcredentials-before-open-expected.txt: Added.
  • fast/xmlhttprequest/xmlhttprequest-withcredentials-before-open.html: Added.

Validate that XMLHttpRequest.withCredentials can be set prior to a call to open().

12:15 PM Changeset in webkit [103660] by kling@webkit.org
  • 2 edits in trunk/Source/WebCore

Remove empty inline RenderStyle destructor.
<​http://webkit.org/b/75188>

Rubber-stamped by Anders "Ordvits" Carlsson.

  • rendering/style/RenderStyle.h:
11:30 AM Changeset in webkit [103659] by mitz@apple.com
  • 2 edits in trunk/Tools

Make build-webkit aware of additional Xcode build location defaults.

Reviewed by Anders Carlsson.

  • Scripts/webkitdirs.pm:

(determineXcodeVersion): Added. This was previously in determineBaseProductDir().
(readXcodeUserDefault): Added. Factored out from determineBaseProductDir().
(determineBaseProductDir): Changed to check for a custom build location
style specified as an absolute location.

11:26 AM Changeset in webkit [103658] by rniwa@webkit.org
  • 20 edits
    1 copy
    1 add
    1 delete in trunk

Unreviewed, rolling out r103619.
​http://trac.webkit.org/changeset/103619
​https://bugs.webkit.org/show_bug.cgi?id=66509

Broke chromium clang build

Source/WebKit/chromium:

  • WebKit.gyp:
  • public/WebCommonWorkerClient.h:

(WebKit::WebCommonWorkerClient::postConsoleMessageToWorkerObject):
(WebKit::WebCommonWorkerClient::allowFileSystem):
(WebKit::WebCommonWorkerClient::openFileSystem):
(WebKit::WebCommonWorkerClient::dispatchDevToolsMessage):
(WebKit::WebCommonWorkerClient::saveDevToolsAgentState):
(WebKit::WebCommonWorkerClient::~WebCommonWorkerClient):

  • public/WebSharedWorker.h:
  • public/WebSharedWorkerClient.h: Removed.
  • public/WebWorkerClient.h:

(WebKit::WebWorkerClient::~WebWorkerClient):

  • src/DatabaseObserver.cpp:

(WebKit::AllowDatabaseMainThreadBridge::create):
(WebKit::AllowDatabaseMainThreadBridge::AllowDatabaseMainThreadBridge):
(WebKit::AllowDatabaseMainThreadBridge::allowDatabaseTask):
(WebKit::allowDatabaseForWorker):
(WebCore::DatabaseObserver::canEstablishDatabase):

  • src/IDBFactoryBackendProxy.cpp:

(WebKit::IDBFactoryBackendProxy::openFromWorker):

  • src/LocalFileSystemChromium.cpp:

(WebCore::openFileSystemHelper):

  • src/WebSharedWorkerImpl.cpp:

(WebKit::WebSharedWorkerImpl::WebSharedWorkerImpl):
(WebKit::WebSharedWorkerImpl::~WebSharedWorkerImpl):
(WebKit::WebSharedWorkerImpl::client):
(WebKit::WebSharedWorker::create):

  • src/WebSharedWorkerImpl.h:

(WebKit::WebSharedWorkerImpl::commonClient):
(WebKit::WebSharedWorkerImpl::newCommonClient):

  • src/WebWorkerBase.cpp:

(WebKit::initializeWebKitStaticValues):
(WebKit::WebWorkerBase::WebWorkerBase):
(WebKit::WebWorkerBase::~WebWorkerBase):
(WebKit::WebWorkerBase::stopWorkerThread):
(WebKit::WebWorkerBase::initializeLoader):
(WebKit::WebWorkerBase::dispatchTaskToMainThread):
(WebKit::WebWorkerBase::invokeTaskMethod):
(WebKit::WebWorkerBase::didCreateDataSource):
(WebKit::WebWorkerBase::createApplicationCacheHost):
(WebKit::WebWorkerBase::postMessageToWorkerObject):
(WebKit::WebWorkerBase::postMessageTask):
(WebKit::WebWorkerBase::postExceptionToWorkerObject):
(WebKit::WebWorkerBase::postExceptionTask):
(WebKit::WebWorkerBase::postConsoleMessageToWorkerObject):
(WebKit::WebWorkerBase::postConsoleMessageTask):
(WebKit::WebWorkerBase::postMessageToPageInspector):
(WebKit::WebWorkerBase::postMessageToPageInspectorTask):
(WebKit::WebWorkerBase::updateInspectorStateCookie):
(WebKit::WebWorkerBase::updateInspectorStateCookieTask):
(WebKit::WebWorkerBase::confirmMessageFromWorkerObject):
(WebKit::WebWorkerBase::confirmMessageTask):
(WebKit::WebWorkerBase::reportPendingActivity):
(WebKit::WebWorkerBase::reportPendingActivityTask):
(WebKit::WebWorkerBase::workerContextClosed):
(WebKit::WebWorkerBase::workerContextClosedTask):
(WebKit::WebWorkerBase::workerContextDestroyed):
(WebKit::WebWorkerBase::workerContextDestroyedTask):
(WebKit::WebWorkerBase::postTaskToLoader):
(WebKit::WebWorkerBase::postTaskForModeToWorkerContext):

  • src/WebWorkerBase.h:

(WebKit::WebWorkerBase::view):
(WebKit::WebWorkerBase::setWorkerThread):
(WebKit::WebWorkerBase::workerThread):

  • src/WebWorkerClientImpl.cpp:
  • src/WebWorkerClientImpl.h:

(WebKit::WebWorkerClientImpl::newCommonClient):

  • src/WebWorkerImpl.cpp: Added.

(WebKit::WebWorker::create):
(WebKit::WebWorkerImpl::WebWorkerImpl):
(WebKit::WebWorkerImpl::~WebWorkerImpl):
(WebKit::WebWorkerImpl::newCommonClient):
(WebKit::WebWorkerImpl::commonClient):
(WebKit::WebWorkerImpl::postMessageToWorkerContextTask):
(WebKit::WebWorkerImpl::startWorkerContext):
(WebKit::WebWorkerImpl::terminateWorkerContext):
(WebKit::WebWorkerImpl::postMessageToWorkerContext):
(WebKit::WebWorkerImpl::workerObjectDestroyed):
(WebKit::WebWorkerImpl::clientDestroyed):

  • src/WebWorkerImpl.h: Copied from Source/WebKit/chromium/src/WebWorkerBase.cpp.

(WebKit::WebWorkerImpl::client):

  • src/WorkerFileSystemCallbacksBridge.cpp:

(WebKit::WorkerFileSystemCallbacksBridge::postOpenFileSystemToMainThread):
(WebKit::WorkerFileSystemCallbacksBridge::openFileSystemOnMainThread):

  • src/WorkerFileSystemCallbacksBridge.h:

Tools:

  • DumpRenderTree/chromium/TestWebWorker.h:

(TestWebWorker::createWorker):

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::createWorker):

  • DumpRenderTree/chromium/WebViewHost.h:
10:53 AM Changeset in webkit [103657] by rniwa@webkit.org
  • 2 edits
    1 add in trunk/LayoutTests

Chromium CG Mac rebaseline after r103651.

  • platform/chromium-cg-mac-snowleopard/fast/images/ycbcr-with-cmyk-color-profile-expected.png: Added.
  • platform/chromium/test_expectations.txt:
10:32 AM Changeset in webkit [103656] by kling@webkit.org
  • 3 edits in trunk/Source/WebCore

RenderStyle: Inline the destructor.
<​http://webkit.org/b/75188>

Reviewed by Kenneth Rohde Christiansen.

The (empty) RenderStyle destructor gets a little hot sometimes, reaching up
to 0.4% when loading the full HTML5 spec. Inline it to remove the pointless
function call.

  • rendering/style/RenderStyle.cpp:
  • rendering/style/RenderStyle.h:

(WebCore::RenderStyleBitfields::~RenderStyle):

7:52 AM Changeset in webkit [103655] by jarred@sencha.com
  • 1 edit
    1 delete in trunk/LayoutTests

Unreviewed test result gardening.

  • platform/chromium/fast/xmlhttprequest/xmlhttprequest-responsetype-arraybuffer-expected.txt: Removed.

Identical to the expected result shared by all ports.

1:07 AM Changeset in webkit [103654] by rniwa@webkit.org
  • 2 edits
    1 add
    2 deletes in trunk/LayoutTests

Another rebaseline for font-weight-bold-for-b-and-strong.html.
Also revert a part of r103645.

  • platform/chromium-linux-x86/fast/html: Removed.
  • platform/chromium-linux/fast/html/font-weight-bold-for-b-and-strong-expected.png: Added.
  • platform/chromium/fast/html: Removed.
  • platform/chromium/fast/html/font-weight-bold-for-b-and-strong-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
Note: See TracTimeline for information about the timeline view.