GitLab

With recent changes, it's dangerous to inject /deep/ selectors
to the page, as they get high priority. This patch stops injecting
backwards compatilibity code to the all-new frontned.

BUG=none

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

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

This patch fixes the raw pointer instances below:

    * StubFrameLoaderClientWithParent::m_parent
    * FrameLoaderClientImpl::m_webFrame
    * RemoteFrameClientImpl::m_webFrame

BUG=509911
R=haraken@chromium.org, oilpan-reviews@chromium.org, peria@chromium.org, sigbjornf@opera.com

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

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

This patch reduces interfaces of WI.CSSProperty and WI.CSSStyleDeclaration
classes.

The patch removes:
  - remove style.getLiveProperty() method and the whole notion of "live"
    properties
  - removes tripple state defined via property.active()/property.inactive()
    flags. (The state was tripple as !property.active() !== property.inactive()).
  - style.shorthandValue/style.shorthandIsImportant public methods
  - removes property.styleBased() property

The patch adds:
  - property.activeInStyle() method which defines if the current property
    has any effect in style.
  - style.leadingProperties() method which computes those properties
    which should be displayed in StylesSidebarPane.

BUG=496263
R=pfeldman
NOTRY=true

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

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

This CL temporary disables tests to be broken by (2).

(1) Blink: THIS PATCH
(2) Chromium: https://codereview.chromium.org/1307133003/
(3) Blink: https://codereview.chromium.org/1311103002/

BUG=523904
NOTRY=true

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

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

Reland of Take care of a FIXME in SecurityOrigin.cpp to check the validity (patchset #1 id:1 of https://codereview.chromium.org/1315793002/ )

Reason for revert:
Failure still present.

Original issue's description:
> Revert of Take care of a FIXME in SecurityOrigin.cpp to check the validity of any innerURL during constructio… (patchset #4 id:60001 of https://codereview.chromium.org/1294933004/ )
> 
> Reason for revert:
> Speculative revert for "Too many opened files in the system".
> 
> Original issue's description:
> > Take care of a FIXME in SecurityOrigin.cpp to check the validity of any innerURL during construction and to handle invalidity as a 'unique' origin.
> > 
> > TBR=mkwst
> > BUG=514076
> > 
> > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201022
> 
> TBR=jsbell@chromium.org,palmer@chromium.org,mkwst@chromium.org,brettw@chromium.org,michaeln@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=514076,524248
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201100

TBR=brettw@chromium.org,jsbell@chromium.org,michaeln@chromium.org,mkwst@chromium.org,palmer@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514076,524248

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

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

This patch gets rid of |VisibleSelection| constructor with |Range| parameter
to templatize |VisibleSelection| easier.

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/1307763003

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

Revert of Take care of a FIXME in SecurityOrigin.cpp to check the validity of any innerURL during constructio… (patchset #4 id:60001 of https://codereview.chromium.org/1294933004/ )

Reason for revert:
Speculative revert for "Too many opened files in the system".

Original issue's description:
> Take care of a FIXME in SecurityOrigin.cpp to check the validity of any innerURL during construction and to handle invalidity as a 'unique' origin.
> 
> TBR=mkwst
> BUG=514076
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201022

TBR=jsbell@chromium.org,palmer@chromium.org,mkwst@chromium.org,brettw@chromium.org,michaeln@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=514076,524248

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

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

Now that SkImage sports a lazy-decoding querying API (isLazyGenerated), we no longer
need the external mechanism introduced in http://crrev.com/1253483004.


BUG=523922
R=noel@chromium.org

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

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

After landing https://codereview.chromium.org/1298113003/, we have four types
of GC callbacks:

- A callback for a minor GC
- A callback for incremental marking of a major GC
- A callback for atomic pause of a major GC
- A callback for weak processing

This CL distinguishes the four types of GC callbacks in V8GCController::gcPrologue/gcEpilogue.

This CL basically just moves code around. No substantial changes in behavior.

BUG=521946

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

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

The current flex-grow and flex-shrink interpolation code will not smoothly
interpolate values with 0.
This patch removes this special behaviour as there is no specification text
justifying this behaviour.
Without this behaviour we can simplify the Constraint enum out of AnimatableDouble.

BUG=522817

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

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

BUG=475370

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

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

Change Counter to CSSValueCounter, and remove counters from
CSSPrimitiveValue.

BUG=523893

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

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

WebDatabase used an extern "C" section to declare the monkey-patching
symbols it uses from our patched SQLite VFS code.  Switch to the items
exported from sqlite3.h.

Additionally, various modules were satisfying an implicit dependency on
icu via SQLite's static library (SQLite needs it for the REGEXP override
in sqlite/src/ext/icu/icu.c).  Change the modules depending on the
sqlite library to also depend on icu, so that sqlite can work with
component build.

BUG=489444

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

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

Reland of moved unused CascadeOrder. (patchset #1 id:1 of https://codereview.chromium.org/1306693007/ )

Reason for revert:
I don't think this contributes any of "too many opened files".
(as is commented bay jianli
https://code.google.com/p/chromium/issues/detail?id=524248#c7 )

Will revert and reland the original patch.



Original issue's description:
> Revert of Removed unused CascadeOrder. (patchset #1 id:1 of https://codereview.chromium.org/1291873005/ )
> 
> Reason for revert:
> Speculative revert for causing "Too many opened files in the system".
> 
> https://code.google.com/p/chromium/issues/detail?id=523598
> 
> 
> Original issue's description:
> > Removed unused CascadeOrder.
> > 
> > CascadeOrder is always ignoreCascadeOrder after [1] and can be removed.
> > 
> > [1] https://codereview.chromium.org/1298173004
> > 
> > BUG=487125
> > 
> > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201019
> 
> TBR=kochi@chromium.org,rune@opera.com
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=487125
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201082

TBR=rune@opera.com,jianli@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487125

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

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

" ".repeat(10) will produce string with one space in console output.
This CL changed this behavior by replacing white-space:nowrap rule to white-space:pre.

BUG=516989
R=pfeldman@chromium.org

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

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

It is helpful to let Oilpan report its memory usage at each safe point.

BUG=

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

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

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

BUG=523403
TBR=pfeldman@chromium.org

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

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

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

BUG=509025
TBR=pfeldman@chromium.org

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

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

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

BUG=509025
TBR=leviw@chromium.org

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

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

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

BUG=509025
TBR=leviw@chromium.org

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

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

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

BUG=514751
TBR=drott@chromium.org

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

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

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

BUG=510492
TBR=wangxianzhu@chromium.org

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

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

This patch gets rid of an extra file "core/editing/commands/EditorCommand.h"
from "core.gypi" added by http://crrev.com/1294543005.

TBR=brucedawson@chromium.org
BUG=509527
TEST=n/a; no behavior changes
NOTRY=true

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

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

Revert of Removed unused CascadeOrder. (patchset #1 id:1 of https://codereview.chromium.org/1291873005/ )

Reason for revert:
Speculative revert for causing "Too many opened files in the system".

https://code.google.com/p/chromium/issues/detail?id=523598


Original issue's description:
> Removed unused CascadeOrder.
> 
> CascadeOrder is always ignoreCascadeOrder after [1] and can be removed.
> 
> [1] https://codereview.chromium.org/1298173004
> 
> BUG=487125
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201019

TBR=kochi@chromium.org,rune@opera.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=487125

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

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

With M SDK, the output of "adb version" has changed, from:
"""Android Debug Bridge version 1.0.32"""

to:
"""Android Debug Bridge version 1.0.32
Revision 57224c5cff69-android"""

Update script regex to match both.

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

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

While attempting to add a new css property, I ran into
ASSERT_WITH_SECURITY_IMPLICATION(index < arraySize) while calling
CSSAnimations::calculateTransitionUpdate. It looks like we normally don't
include CSSPropertyInvalid, and use firstCSSProperty to offset the index.

I've done the same here.

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

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

Initial UI is hidden behind "blockedURLs" experiment.

BUG=520259

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

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

BUG=518816

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

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

Use simple Map<string, string> object Instead of utilizing
CSSStyleDeclaration for representation of computed style.

NOTRY=true
BUG=496263

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

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

BUG=523910,490087

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

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

BUG=524160

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

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

BUG=524092
R=pfeldman,paulirish
NOTRY=true

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

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

This patch fixes Japanese line breaking rules before and after ruby
elements. This patch is a port of http://wkb.ug/91588 with the
following changes:
* The WebKit patch sets ruby base text to the prior context for breaks
  after the ruby. Since U+FFFD Object Replacement Character can cover
  most of the common cases that this patch does not include that part.
* canBreakBefore() is fixed to include the correct UAX#14[1] classes
  that prohibit line breaks after. The original patch is a mix of
  before and after, and lacks some important after classes.

This patch fixes the common use cases in CJK, but it's hard to say this
is the right way to fix. There are two issues that block doing so at
this moment:
1. RubyRun is LayoutBlockFlow, but after WebKit implemented, the ruby
   spec was updated to allow line breaking within ruby, so it should be
   like inline, not a replaced element.
2. The CSS WG discussed on line breaking around replaced elements, but
   the discussion is not making much progress.
Changes in this patch should be updated when these issues are resolved.

[1] http://unicode.org/reports/tr14/

BUG=522826

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

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

BUG=523537
NOTRY=true

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

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

R=pdr@chromium.org

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

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

BUG=523073
TEST=Fix test.

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

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

This is a spiritual revert of
https://codereview.chromium.org/1203663003/.

R=adamk@chromium.org
BUG=524009
LOG=N

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

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

The default has been false for at least two years now. Chromium can turn it on,
but it's appropriate for the initial value to be inlined into Settings.in, like
most of the default-off settings.

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

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

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

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

These tests were likely flaky because of a race between setPermission and
the subsequent request.

Utilize the new helper function for setting permissions to rewrite the
tests in terms of promises and remove the races.

BUG=519007

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

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