RenderObject bits setters don't use the default parameter

Thus we can remove them. It shouldn't matter for the compiler
but it should make the caller more explicit, instead of relying
on the confusing default parameter.

Review URL: https://codereview.chromium.org/87083002

git-svn-id: svn://svn.chromium.org/blink/trunk@162671 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Correctly apply per-keyframe timing functions

Currently, when a per-keyframe timing function is applied to
an element which is targeted by multiple animations, the
timing function may be applied to the wrong CSSAnimationData
object. This causes the per-keyframe timing function to be
ignored.

This patch fixes the problem by making sure the relevant
CSSAnimationData object is always first in the keyframe's
RenderStyle.

R=dstockwell
BUG=288540, 322680

Review URL: https://codereview.chromium.org/24085002

git-svn-id: svn://svn.chromium.org/blink/trunk@162672 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Notify the client of a document access even if document.write is used.

Modal dialogs can usually delay the notification from reaching the client.
notifyIfInitialDocumentAccessed is used to skip the timer if a dialog
is shown (via PageGroupLoadDeferrer), but it shouldn't care whether the
FrameLoader is still in the initial empty document state.

BUG=322959
TEST=See bug for repro steps.

Review URL: https://codereview.chromium.org/86413004

git-svn-id: svn://svn.chromium.org/blink/trunk@162673 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Split FontDataCache out of FontCache.

Get gFontDataCache logic outta FontCache and into its own nice class.

I also grouped other caches into static funcs in preparation for
splitting them off, too.

R=eae,ksakamoto@chromium.org
BUG=315783

Review URL: https://codereview.chromium.org/84773005

git-svn-id: svn://svn.chromium.org/blink/trunk@162674 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Use const Node* in HashMap of DocumentMarkerController

This patch changes key type of HashMap used in DOcumentMarkerController to |const Node*| from |RefPtr<Node>|, which was introduced by r14748(http://src.chromium.org/viewvc/blink?view=revision&revision=14748) to avoid redundant ref counting. 

This patch is a preparation of using |const Node&| parameter in DocumentMarkerController.

BUG=n/a
TEST=n/a; no behavior changes

Review URL: https://codereview.chromium.org/82633002

git-svn-id: svn://svn.chromium.org/blink/trunk@162675 bbb929c8-8fbe-4397-9dbb-9b2b20218538