GitLab

This patch introduces |character{Before,After}()| function for |VisiblePosition|
as replacement of |VisiblePosition::character{Before,After}()| to simplify
|VisiblePosition| class for ease of templataizing |VisiblePosition| class to
introduce composed tree version of |VisiblePosition|, since an implementation of
|VisiblePosition::character{Before,After}()| can be implemented with public
member functions of |VisiblePosition| class.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

Make updateSelectionForMouseDownDispatchingSelectStart() member function in SelectionController class private

This patch make |updateSelectionForMouseDownDispatchingSelectStart()| member
function for |SelectionController| as private for improving code health
since it is used only in |SelectionController| class.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

Old code assumed that only one level of UA shadow tree
nodes should be excluded from event path, while it is
no longer true after introduction of shadow root
closed mode.

This change properly hides all the nodes behind closed
shadow root in the event path.

BUG=459136
TEST=event-path-closed-shadowroot.html, event-path-closed-shadowroot2.html

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

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

Added STATIC_ONLY, STACK_ALLOCATED, DISALLOW_ALLOCATION, or ALLOW_ONLY_INLINE_ALLOCATION
instead of WTF_MAKE_FAST_ALLOCATED(_WILL_BE_REMOVED) if possible.

BUG=523249

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

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

This patch introduces |nextPositionOf()| function for |VisiblePosition| as
replacement of |VisiblePosition::next()| to simplify |VisiblePosition| class
for ease of templataizing |VisiblePosition| class to introduce composed tree
version of |VisiblePosition|, since an implementation of
|VisiblePosition::next()| can be implemented with public member functions of
|VisiblePosition| class.

Note: http://crrev.com/1310043003introduces |previousPositionOf()|.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

The test was timing out, it's not clear why but the code was fairly
complex. Simplify by avoiding messaging and just return the result in
the response to onfetch.

BUG=518985

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

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

This patch introduces |lineDirectionPointForBlockDirectionNavigationOf()|
function for |VisiblePosition| as replacement of
|VisiblePosition::lineDirectionPointForBlockDirectionNavigation()| to simplify
|VisiblePosition| class for ease of templataizing |VisiblePosition| class to
introduce composed tree version of |VisiblePosition|, since an implementation of
|VisiblePosition::lineDirectionPointForBlockDirectionNavigation()| can be
implemented with public member functions of |VisiblePosition| class.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

as a member variable of WebDevToolsFrontendImpl on Oilpan build.

This CL adds a GC_PLUGIN_IGNORE with a TODO comment 
to notify this issue.


BUG=509911

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

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

Also, add Internals::resetTypeAheadSession().
These changes fix flakiness of fast/forms/select/select-typeahead-with-spacekey.html.

TestExpectatios still has the flaky expectation for the test just in case.

BUG=479202

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

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

It would be helpful to upstream the tests to WPT tests.

BUG=522836

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

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

Added STATIC_ONLY, STACK_ALLOCATED, DISALLOW_ALLOCATION, or ALLOW_ONLY_INLINE_ALLOCATION
instead of WTF_MAKE_FAST_ALLOCATED(_WILL_BE_REMOVED) if possible.

BUG=523249

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

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

This patch introduces |previousPositionOf()| function for |VisiblePosition| as
replacement of |VisiblePosition::previous()| to simplify |VisiblePosition| class
for ease of templataizing |VisiblePosition| class to introduce composed tree
version of |VisiblePosition|, since an implementation of
|VisiblePosition::previous()| can be implemented with public member functions of
|VisiblePosition| class.

This patch also moves |VisiblePosition::characterBefore()| into
"VisiblePosition.cpp" to avoid forward declaration of 
|previousPositionOf()|.

Note: http://crrev.com/1307803003 introduces |nextPositionOf()|.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=201371

BUG=525884
TBR=tkent@chromium.org

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=201370

BUG=410320
TBR=kojii@chromium.org

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

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

Added STATIC_ONLY, STACK_ALLOCATED, DISALLOW_ALLOCATION, or ALLOW_ONLY_INLINE_ALLOCATION
instead of WTF_MAKE_FAST_ALLOCATED(_WILL_BE_REMOVED) if possible.

BUG=523249

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

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

'background: currentColor' not getting repainted when 'color' is changed. 
LayoutObject::adjustStyleDifference doesn't check backgroundColor when
repainting. The related bug filed occurs only on Chrome. Both FF and IE
deal with this correctly.

BUG=487489

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

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

This patch introduces composed tree version of |directionOfEnclosingBlock()|
as a preparation of introducing composed tree version of |VisiblePosition|
for a preparation of making selection to handle granularity for web component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

Blink r201361 shouldn't remove TestExpectation entry.

BUG=525905
TBR=szager@chromium.org

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

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

BUG=525905
TBR=tkent@chromium.org
NOTRY=true

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

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

SkBitmap is being removed from Chrome. There are no longer any callers
of WEBPImageEncoder::encode(SkBitmap,...) so we can remove it and also
the support code it required.

No change in behavior, no new tests.

TBR=fmalita@chromium.org
BUG=449197

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

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

The test just throws an exception.

BUG=525903
TBR=pfeldman, sergeyv
NOTRY=true

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

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

The native button rendering is strange on Yosemite.

BUG=525884

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

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

This patch fixes the shrink-to-fit algorithm when descendants of the
element have orthogonal writing-mode.

Strictly speaking, as the [spec] defines, min-content width of the element
must be determined by logicalHeight of children. This would require either
"layout during preferred width calculation" or "multiple layout passes."

In most cases, however, descendants are not mixed. Since orthogonal
descendants need re-layout only when the parent's logicalHeight changes,
this patch updates logicalWidth after children are laid out but does not
re-layout descendants.

Table cells with orthogonal descendants require additional work and the
fix is not included in this CL. 001q-001x are tests for table cells.

css3/flexbox/flex-flow-margins-auto-size.html that currently expects to
fail passes with this patch: http://wkb.ug/70769

[spec]: http://dev.w3.org/csswg/css-writing-modes-3/#orthogonal-flows

BUG=410320, 359604
TEST=css3/flexbox/flex-flow-margins-auto-size.html
TEST=imported/csswg-test/css-writing-modes-3/orthogonal-parent-shrink-to-fit-001.html

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

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

Reland: Fixed "notEnoughCharacters" not initialized.

Before this CL, SegmentedString::push() had an exotic behavior, where
two consecutive push would swap its order.

This CL changes the push() implementation so that it would always push
the new char in front of the SegmentedString.
This CL also updates the HTMLEntityParser code which relied on the behavior.

BUG=None
TEST={SegmentedStringTest,HTMLEntityParserTest}.*

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201294

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

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

The current documentation was short and didn't really
explain much about the object. The expanded description
introduces more concepts and talks about the different
layouts in more details.

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

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

SkBitmap is being removed from Chrome. There are no longer any callers
of JPEGImageEncoder::encode(SkBitmap,...) so we can remove it and also
the support code it required.

Note: The libjpeg-turbo specialization was the fastest encoder, but it
required premultiplied data (SkBitmap). Thus some performance was lost
when callers were switched over to use the ImageDataBuffer encoder (in
the SkImage-replaces-SkBitmap endeavor).

No change in behavior, no new tests.

BUG=449197

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

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

Added STATIC_ONLY, STACK_ALLOCATED, DISALLOW_ALLOCATION, or ALLOW_ONLY_INLINE_ALLOCATION
instead of WTF_MAKE_FAST_ALLOCATED(_WILL_BE_REMOVED) if possible.

BUG=523249

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

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

Keishi is a frequent contributor in Source/web [1], and the originator of
Source/web/resources [2].

[1] https://blink.lc/blink/stats/Source/web?period=q&ofs=10
[2] https://blink.lc/blink/stats/Source/web/resources?period=y&ofs=10

NOTRY=true
BUG=

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

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

BUG=None

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

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

We currently support -webkit-text as a value in clip properties,
background-clip, -webkit-background-clip and -webkit-mask-clip, as
well as in color properties (only when in quirks mode). I suspect
usage is extremely rare in both cases so this patch adds counters
for these.

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

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

This patch moves global function |intersectsNode()| in
"VisibleSelection.{cpp,h}" to local function in "FrameSelection.cpp" to
narrow visibility for improving code health and ease of implement composed tree
version.

This patch is a preparation of making selection to handle granularity for web
component, http://crrev.com/1277863002

BUG=513568
TEST=n/a; No behavior changes

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

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

Also, remove an obsoleted comment in TestExpectations.

BUG=

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

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

The class was sorely missing some documentation
about the design decisions and its purpose.

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

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

Was changed to failure actually crashing on MSAN bots.

BUG=525824
TBR=szager1, xlai@chromium.org
NOTRY=true

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

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

The current tip wording is the following: "Use 'file:' to define search scope".
The patch changes wording to: "Enter query, use `file:` to filter by path".

BUG=488112
NOTRY=true
TBR=pfeldman, paulirish

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

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

When chrome running on on root-layer-scrolls, page is scaled the style 
is recalculated and thickness is changed. Change in thickness sets 
informs owning renderer to set for childneedsLayout. This layout shall
happen after style reclac is completed, if this sequence is not executed,
then post layout we see the needsForLayout = true, which asserts and
crashes the chrome.

TESTED=fast/scrollbars/custom-scrollbar-thickness-change-on-zoom-crash.html
now passes with root layer scrolling

BUG=522389(https://code.google.com/p/chromium/issues/detail?id=522389#c1)

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=201337

BUG=523056
TBR=aelias@chromium.org

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

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

This is extracted from https://codereview.chromium.org/1307863003
so that it can be landed after all chromium-side changes are landed.

TBR=pfeldman
BUG=456845

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=201342

BUG=525702
TBR=jbroman@chromium.org

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

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

http://src.chromium.org/viewvc/blink?view=revision&revision=201339

BUG=509025,524134
TBR=wangxianzhu@google.com

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

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