[Chromium] WebFrameImpl.cpp could use a style cleanup
https://bugs.webkit.org/show_bug.cgi?id=97924 Reviewed by Dimitri Glazkov. This file is very old and contains a large number of style errors that are confusing folks contributing new code to this file. Most of these styles are Chromium-isms that crept in when the file was forked in the Chromium repository. * src/WebFrameImpl.cpp: (WebKit): (WebKit::frameContentAsPlainText): (ChromePrintContext): (WebKit::ChromePrintContext::spoolPage): (WebKit::ChromePrintContext::spoolAllPagesWithBoundaries): (ChromePluginPrintContext): (WebKit::ChromePluginPrintContext::spoolPage): (WebKit::WebFrameImpl::DeferredScopeStringMatches::DeferredScopeStringMatches): (WebKit::WebFrameImpl::DeferredScopeStringMatches::doTimeout): (WebKit::WebFrame::fromFrameOwnerElement): (WebKit::WebFrameImpl::uniqueName): (WebKit::WebFrameImpl::assignedName): (WebKit::WebFrameImpl::setName): (WebKit::WebFrameImpl::iconURLs): (WebKit::WebFrameImpl::scrollOffset): (WebKit::WebFrameImpl::minimumScrollOffset): (WebKit::WebFrameImpl::maximumScrollOffset): (WebKit::WebFrameImpl::contentsPreferredWidth): (WebKit::WebFrameImpl::documentElementScrollHeight): (WebKit::WebFrameImpl::hasHorizontalScrollbar): (WebKit::WebFrameImpl::hasVerticalScrollbar): (WebKit::WebFrameImpl::opener): (WebKit::WebFrameImpl::setOpener): (WebKit::WebFrameImpl::parent): (WebKit::WebFrameImpl::top): (WebKit::WebFrameImpl::firstChild): (WebKit::WebFrameImpl::lastChild): (WebKit::WebFrameImpl::nextSibling): (WebKit::WebFrameImpl::previousSibling): (WebKit::WebFrameImpl::traverseNext): (WebKit::WebFrameImpl::traversePrevious): (WebKit::WebFrameImpl::findChildByName): (WebKit::WebFrameImpl::findChildByExpression): (WebKit::WebFrameImpl::document): (WebKit::WebFrameImpl::performance): (WebKit::WebFrameImpl::windowObject): (WebKit::WebFrameImpl::bindToWindowObject): (WebKit::WebFrameImpl::executeScript): (WebKit::WebFrameImpl::executeScriptInIsolatedWorld): (WebKit::WebFrameImpl::setIsolatedWorldSecurityOrigin): (WebKit::WebFrameImpl::collectGarbage): (WebKit::WebFrameImpl::checkIfRunInsecureContent): (WebKit::WebFrameImpl::executeScriptAndReturnValue): (WebKit::WebFrameImpl::callFunctionEvenIfScriptDisabled): (WebKit::WebFrameImpl::mainWorldScriptContext): (WebKit::WebFrameImpl::createFileSystem): (WebKit::WebFrameImpl::createSerializableFileSystem): (WebKit::WebFrameImpl::createFileEntry): (WebKit::WebFrameImpl::reload): (WebKit::WebFrameImpl::reloadWithOverrideURL): (WebKit::WebFrameImpl::loadRequest): (WebKit::WebFrameImpl::loadHistoryItem): (WebKit::WebFrameImpl::loadData): (WebKit::WebFrameImpl::loadHTMLString): (WebKit::WebFrameImpl::isLoading): (WebKit::WebFrameImpl::stopLoading): (WebKit::WebFrameImpl::provisionalDataSource): (WebKit::WebFrameImpl::dataSource): (WebKit::WebFrameImpl::previousHistoryItem): (WebKit::WebFrameImpl::currentHistoryItem): (WebKit::WebFrameImpl::enableViewSourceMode): (WebKit::WebFrameImpl::isViewSourceModeEnabled): (WebKit::WebFrameImpl::setReferrerForRequest): (WebKit::WebFrameImpl::dispatchWillSendRequest): (WebKit::WebFrameImpl::commitDocumentData): (WebKit::WebFrameImpl::insertText): (WebKit::WebFrameImpl::setMarkedText): (WebKit::WebFrameImpl::firstRectForCharacterRange): (WebKit::WebFrameImpl::executeCommand): (WebKit::WebFrameImpl::requestTextChecking): (WebKit::WebFrameImpl::selectWordAroundPosition): (WebKit::WebFrameImpl::selectRange): (WebKit::WebFrameImpl::visiblePositionForWindowPoint): (WebKit::WebFrameImpl::printBegin): (WebKit::WebFrameImpl::getPrintPageShrink): (WebKit::WebFrameImpl::printPage): (WebKit::WebFrameImpl::printEnd): (WebKit::WebFrameImpl::isPrintScalingDisabledForPlugin): (WebKit::WebFrameImpl::pageSizeAndMarginsInPixels): (WebKit::WebFrameImpl::pageProperty): (WebKit::WebFrameImpl::find): (WebKit::WebFrameImpl::scopeStringMatches): (WebKit::WebFrameImpl::reportFindInPageSelection): (WebKit::WebFrameImpl::sendOrientationChangeEvent): (WebKit::WebFrameImpl::addEventListener): (WebKit::WebFrameImpl::removeEventListener): (WebKit::WebFrameImpl::dispatchEvent): (WebKit::WebFrameImpl::dispatchMessageEventWithOriginCheck): (WebKit::WebFrameImpl::isActiveMatchFrameValid): (WebKit::WebFrameImpl::deliverIntent): (WebKit::WebFrameImpl::contentAsText): (WebKit::WebFrameImpl::contentAsMarkup): (WebKit::WebFrameImpl::renderTreeAsText): (WebKit::WebFrameImpl::printPagesWithBoundaries): (WebKit::WebFrameImpl::selectionBoundsRect): (WebKit::WebFrameImpl::selectionStartHasSpellingMarkerFor): (WebKit::WebFrameImpl::layerTreeAsText): (WebKit::WebFrameImpl::initializeAsMainFrame): (WebKit::WebFrameImpl::createChildFrame): (WebKit::WebFrameImpl::createFrameView): (WebKit::WebFrameImpl::fromFrame): (WebKit::WebFrameImpl::fromFrameOwnerElement): (WebKit::WebFrameImpl::viewImpl): (WebKit::WebFrameImpl::setCanHaveScrollbars): (WebKit::WebFrameImpl::setMarkerActive): (WebKit::WebFrameImpl::ordinalOfFirstMatchForFrame): (WebKit::WebFrameImpl::scopeStringMatchesSoon): (WebKit::WebFrameImpl::callScopeStringMatches): (WebKit::WebFrameImpl::invalidateIfNecessary): (WebKit::WebFrameImpl::loadJavaScriptURL): git-svn-id: svn://svn.chromium.org/blink/trunk@129927 bbb929c8-8fbe-4397-9dbb-9b2b20218538
This commit is contained in:
abarth@webkit.org
third_party/WebKit/Source/WebKit/chromium
124
third_party/WebKit/Source/WebKit/chromium/ChangeLog
vendored
124
third_party/WebKit/Source/WebKit/chromium/ChangeLog
vendored
@@ -1,3 +1,127 @@
|
||||
2012-09-28 Adam Barth <abarth@webkit.org>
|
||||
|
||||
[Chromium] WebFrameImpl.cpp could use a style cleanup
|
||||
https://bugs.webkit.org/show_bug.cgi?id=97924
|
||||
|
||||
Reviewed by Dimitri Glazkov.
|
||||
|
||||
This file is very old and contains a large number of style errors that
|
||||
are confusing folks contributing new code to this file. Most of these
|
||||
styles are Chromium-isms that crept in when the file was forked in the
|
||||
Chromium repository.
|
||||
|
||||
* src/WebFrameImpl.cpp:
|
||||
(WebKit):
|
||||
(WebKit::frameContentAsPlainText):
|
||||
(ChromePrintContext):
|
||||
(WebKit::ChromePrintContext::spoolPage):
|
||||
(WebKit::ChromePrintContext::spoolAllPagesWithBoundaries):
|
||||
(ChromePluginPrintContext):
|
||||
(WebKit::ChromePluginPrintContext::spoolPage):
|
||||
(WebKit::WebFrameImpl::DeferredScopeStringMatches::DeferredScopeStringMatches):
|
||||
(WebKit::WebFrameImpl::DeferredScopeStringMatches::doTimeout):
|
||||
(WebKit::WebFrame::fromFrameOwnerElement):
|
||||
(WebKit::WebFrameImpl::uniqueName):
|
||||
(WebKit::WebFrameImpl::assignedName):
|
||||
(WebKit::WebFrameImpl::setName):
|
||||
(WebKit::WebFrameImpl::iconURLs):
|
||||
(WebKit::WebFrameImpl::scrollOffset):
|
||||
(WebKit::WebFrameImpl::minimumScrollOffset):
|
||||
(WebKit::WebFrameImpl::maximumScrollOffset):
|
||||
(WebKit::WebFrameImpl::contentsPreferredWidth):
|
||||
(WebKit::WebFrameImpl::documentElementScrollHeight):
|
||||
(WebKit::WebFrameImpl::hasHorizontalScrollbar):
|
||||
(WebKit::WebFrameImpl::hasVerticalScrollbar):
|
||||
(WebKit::WebFrameImpl::opener):
|
||||
(WebKit::WebFrameImpl::setOpener):
|
||||
(WebKit::WebFrameImpl::parent):
|
||||
(WebKit::WebFrameImpl::top):
|
||||
(WebKit::WebFrameImpl::firstChild):
|
||||
(WebKit::WebFrameImpl::lastChild):
|
||||
(WebKit::WebFrameImpl::nextSibling):
|
||||
(WebKit::WebFrameImpl::previousSibling):
|
||||
(WebKit::WebFrameImpl::traverseNext):
|
||||
(WebKit::WebFrameImpl::traversePrevious):
|
||||
(WebKit::WebFrameImpl::findChildByName):
|
||||
(WebKit::WebFrameImpl::findChildByExpression):
|
||||
(WebKit::WebFrameImpl::document):
|
||||
(WebKit::WebFrameImpl::performance):
|
||||
(WebKit::WebFrameImpl::windowObject):
|
||||
(WebKit::WebFrameImpl::bindToWindowObject):
|
||||
(WebKit::WebFrameImpl::executeScript):
|
||||
(WebKit::WebFrameImpl::executeScriptInIsolatedWorld):
|
||||
(WebKit::WebFrameImpl::setIsolatedWorldSecurityOrigin):
|
||||
(WebKit::WebFrameImpl::collectGarbage):
|
||||
(WebKit::WebFrameImpl::checkIfRunInsecureContent):
|
||||
(WebKit::WebFrameImpl::executeScriptAndReturnValue):
|
||||
(WebKit::WebFrameImpl::callFunctionEvenIfScriptDisabled):
|
||||
(WebKit::WebFrameImpl::mainWorldScriptContext):
|
||||
(WebKit::WebFrameImpl::createFileSystem):
|
||||
(WebKit::WebFrameImpl::createSerializableFileSystem):
|
||||
(WebKit::WebFrameImpl::createFileEntry):
|
||||
(WebKit::WebFrameImpl::reload):
|
||||
(WebKit::WebFrameImpl::reloadWithOverrideURL):
|
||||
(WebKit::WebFrameImpl::loadRequest):
|
||||
(WebKit::WebFrameImpl::loadHistoryItem):
|
||||
(WebKit::WebFrameImpl::loadData):
|
||||
(WebKit::WebFrameImpl::loadHTMLString):
|
||||
(WebKit::WebFrameImpl::isLoading):
|
||||
(WebKit::WebFrameImpl::stopLoading):
|
||||
(WebKit::WebFrameImpl::provisionalDataSource):
|
||||
(WebKit::WebFrameImpl::dataSource):
|
||||
(WebKit::WebFrameImpl::previousHistoryItem):
|
||||
(WebKit::WebFrameImpl::currentHistoryItem):
|
||||
(WebKit::WebFrameImpl::enableViewSourceMode):
|
||||
(WebKit::WebFrameImpl::isViewSourceModeEnabled):
|
||||
(WebKit::WebFrameImpl::setReferrerForRequest):
|
||||
(WebKit::WebFrameImpl::dispatchWillSendRequest):
|
||||
(WebKit::WebFrameImpl::commitDocumentData):
|
||||
(WebKit::WebFrameImpl::insertText):
|
||||
(WebKit::WebFrameImpl::setMarkedText):
|
||||
(WebKit::WebFrameImpl::firstRectForCharacterRange):
|
||||
(WebKit::WebFrameImpl::executeCommand):
|
||||
(WebKit::WebFrameImpl::requestTextChecking):
|
||||
(WebKit::WebFrameImpl::selectWordAroundPosition):
|
||||
(WebKit::WebFrameImpl::selectRange):
|
||||
(WebKit::WebFrameImpl::visiblePositionForWindowPoint):
|
||||
(WebKit::WebFrameImpl::printBegin):
|
||||
(WebKit::WebFrameImpl::getPrintPageShrink):
|
||||
(WebKit::WebFrameImpl::printPage):
|
||||
(WebKit::WebFrameImpl::printEnd):
|
||||
(WebKit::WebFrameImpl::isPrintScalingDisabledForPlugin):
|
||||
(WebKit::WebFrameImpl::pageSizeAndMarginsInPixels):
|
||||
(WebKit::WebFrameImpl::pageProperty):
|
||||
(WebKit::WebFrameImpl::find):
|
||||
(WebKit::WebFrameImpl::scopeStringMatches):
|
||||
(WebKit::WebFrameImpl::reportFindInPageSelection):
|
||||
(WebKit::WebFrameImpl::sendOrientationChangeEvent):
|
||||
(WebKit::WebFrameImpl::addEventListener):
|
||||
(WebKit::WebFrameImpl::removeEventListener):
|
||||
(WebKit::WebFrameImpl::dispatchEvent):
|
||||
(WebKit::WebFrameImpl::dispatchMessageEventWithOriginCheck):
|
||||
(WebKit::WebFrameImpl::isActiveMatchFrameValid):
|
||||
(WebKit::WebFrameImpl::deliverIntent):
|
||||
(WebKit::WebFrameImpl::contentAsText):
|
||||
(WebKit::WebFrameImpl::contentAsMarkup):
|
||||
(WebKit::WebFrameImpl::renderTreeAsText):
|
||||
(WebKit::WebFrameImpl::printPagesWithBoundaries):
|
||||
(WebKit::WebFrameImpl::selectionBoundsRect):
|
||||
(WebKit::WebFrameImpl::selectionStartHasSpellingMarkerFor):
|
||||
(WebKit::WebFrameImpl::layerTreeAsText):
|
||||
(WebKit::WebFrameImpl::initializeAsMainFrame):
|
||||
(WebKit::WebFrameImpl::createChildFrame):
|
||||
(WebKit::WebFrameImpl::createFrameView):
|
||||
(WebKit::WebFrameImpl::fromFrame):
|
||||
(WebKit::WebFrameImpl::fromFrameOwnerElement):
|
||||
(WebKit::WebFrameImpl::viewImpl):
|
||||
(WebKit::WebFrameImpl::setCanHaveScrollbars):
|
||||
(WebKit::WebFrameImpl::setMarkerActive):
|
||||
(WebKit::WebFrameImpl::ordinalOfFirstMatchForFrame):
|
||||
(WebKit::WebFrameImpl::scopeStringMatchesSoon):
|
||||
(WebKit::WebFrameImpl::callScopeStringMatches):
|
||||
(WebKit::WebFrameImpl::invalidateIfNecessary):
|
||||
(WebKit::WebFrameImpl::loadJavaScriptURL):
|
||||
|
||||
2012-09-28 Kent Tamura <tkent@chromium.org>
|
||||
|
||||
Add parseDateTime, formatDateTime, and dateFormatText to Localizer
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user