GitLab

Revert of Move scaling of ui resources for scrollbars to the time of upload (patchset #6 id:100001 of https://codereview.chromium.org/2276633003/ )

Reason for revert:
Sorry, seems to have caused LayerTreeHostScrollbarsPixelTest.HugeTransformScale to fail on Mac10.10 Tests,  Mac10.11 Tests, Win 7 Tests x64 (1), Win10 Tests x64, Mac10.9 Tests and Linux Tests, with failures like:

[8034:1287:0825/033725:3059591139598:ERROR:pixel_comparator.cc(50)] Number of pixel with an error: 672
[8034:1287:0825/033725:3059591187709:ERROR:pixel_comparator.cc(51)] Error Bounding Box : 0,304 368x66
[8034:1287:0825/033725:3059610880597:ERROR:pixel_test_utils.cc(79)] Pixels do not match!
[8034:1287:0825/033725:3059610905829:ERROR:pixel_test_utils.cc(80)] Actual: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAZAAAAGQCAYAAACAvzbMAAAF4ElEQVR4nO3ZQY1CQRRFwccEUWhhgQEMYAED7FGDKtJsxsCcTYdMVQt4d3eS/w9rrTUAX+g+97nNbfeMbS5zmec8t93/2XYZgK8mIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAcnzMY/cGgOQ1r90T/rXjda67NwDwhXzCAiAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREACS43nOuzcAEJzmtPX+4b3ea+sCAJLD79t2f60lIAD8mX8gACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJAICQCIgACQCAkAiIAAkAgJAIiAAJB9NHxfMgTV5VwAAAABJRU5ErkJggg==
[8034:1287:0825/033725:3059610929427:ERROR:pixel_test_utils.cc(81)] Expected: data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAZAAAAGQCAYAAACAvzbMAAAF4klEQVR4nO3ZQY1CQRRFwfcJotDCYhQgAAsIQABqMEXSo2HOpkOmysC7u5N0H2utNQBf6DGPuc9994xtfuZnXvPadv+07TIAX01AAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIDk/Jzn7g0AyXveuyf8a+fb3HZvAOALecICIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQABIBASAREAASAQEgERAAEgEBIBEQAJLzda67NwAQXOay9f7xWZ+1dQEAyWlOc8yx7f6x1hIQAP7MHwgAiYAAkAgIAImAAJAICACJgACQCAgAiYAAkAgIAImAAJAICACJgACQCAgAiYAAkAgIAImAAJAICACJgACQCAgAiYAAkAgIAImAAJAICACJgACQCAgAiYAAkAgIAImAAJAICADJLzUdF8xyVTjQAAAAAElFTkSuQmCC
../../cc/test/layer_tree_pixel_test.cc:124: Failure
Value of: MatchesPNGFile(*result_bitmap_, ref_file_path, *pixel_comparator_)
  Actual: false
Expected: true

Original issue's description:
> Move scaling of ui resources for scrollbars to the time of upload
>
> Instead of trying to figure out what the max texture size is on the
> main thread, and scale raster to meet that, just raster freely. Then
> when uploading the UIResource to a texture, we can easily tell what
> the max texture size is, and if the UIResource is too large, scale it
> down to fit in a texture.
>
> R=enne
> BUG=606056
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> NOTRY=true
>
> Committed: https://crrev.com/941e52edd1522f7c3935aa4f4d763c43578b4944
> Cr-Commit-Position: refs/heads/master@{#414202}

TBR=enne@chromium.org,danakj@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=606056

Review-Url: https://codereview.chromium.org/2278083002
Cr-Commit-Position: refs/heads/master@{#414391}

Move createCrossThreadTask to ExecutionContextTask.h so that
both same thread and cross thread versions of createTask are in the
same header file.
Currently createCrossThreadTask is in CrossThreadTask.h,
but the implementation is a few lines of code, so merging it to
ExecutionContextTask.h and removing CrossThreadTask.h.

BUG=638839

Review-Url: https://codereview.chromium.org/2264543002
Cr-Commit-Position: refs/heads/master@{#414390}

From an out-of-band discussion with rsleevi came the conclusion that
all CT logs, including ones that ceased operation, must be auditable
(since some of their SCTs are accepted).

This change ensures that if logs are added to the known logs list
without a DNS front-end address, it will be detected on start-up.

BUG=624894

Review-Url: https://codereview.chromium.org/2269383003
Cr-Commit-Position: refs/heads/master@{#414389}

Revert of Updates the IME list when the IME has refreshed. (patchset #7 id:120001 of https://codereview.chromium.org/2271483003/ )

Reason for revert:
ImeMenuTrayTest.RefreshImeWithListViewCreated is failing consistently on https://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/builds/15622

Original issue's description:
> Updates the IME list when the IME has refreshed.
>
> When the IME refreshes, we need to update the view of the list in IME menu.
> Saves the created ImeListView and updates it when get IME refreshing event.
> Add ImeMenuTrayTest.RefreshImeWithListViewCreated test.
>
> BUG=640432
> TEST=Verified on local build.
>
> Committed: https://crrev.com/c6f28a1effd401fd7e2e03707e0dd84f5346185b
> Cr-Commit-Position: refs/heads/master@{#414329}

TBR=jamescook@chromium.org,azurewei@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=640432

Review-Url: https://codereview.chromium.org/2281473002
Cr-Commit-Position: refs/heads/master@{#414388}

It's nice to have more information in password manager logging in order to debug password generation in Chrome. This CL updates FormStructure presentation.

For example Linkedin SignUp form has the following presentation in logs:

Adding manager for form: {
Signature of form: 5301209056338178556
Signon realm: https://www.linkedin.com/
Origin: https://www.linkedin.com/
Action: https://www.linkedin.com/
Form name:
Form fields:
firstname: 1855613035, text
lastname: 4163345999, text
emailaddress: 4103469401, email
password: 2051817934, password, off
}

BUG=None

Review-Url: https://codereview.chromium.org/2268313003
Cr-Commit-Position: refs/heads/master@{#414387}

For shadow dom v0, we used a SubtreeStyleChange for propagating
inherited style changes through insertion points to distributed nodes.
LocalStyleChange should suffice. We already use LocalStyleChange in the
HTMLSlotElement case.

We still need to use SubtreeStyleChange where we have a
SubtreeStyleChange/Force from further up the tree like:

<host>
  <:shadow-root>
    <style>.a::content * { background: green }</style>
    <div id="a">
      <content></content>
    </div>
  </:shadow-root>
  <div>Green when #a gets class a.</div>
</host>

R=kochi@chromium.org
BUG=638869

Review-Url: https://codereview.chromium.org/2258793003
Cr-Commit-Position: refs/heads/master@{#414386}

The RunIfWaitingForDebugger command was made stable, breaking this test
which is specifically trying to check the behavior of an experimental
command. Switch to another experimental command.

(The API should have remained compatible even if the command was made
stable. This bug will be fixed separately.)

BUG=640919
NOTRY=true

Review-Url: https://codereview.chromium.org/2277193002
Cr-Commit-Position: refs/heads/master@{#414385}

In PlzNavigate, RenderFrameHostImpl currently requires that when it
gets a DidFailProvisionalLoad IPC, it has previously seen a net error.
This requirement holds for browser-side failed navigations but not for
renderer-side failed navigations. The latter fact causes some layout
tests to fail.

The DidFailProvisionalLoad IPC is scheduled for removal once all
WebContentsObservers have moved to listening for DidFinishNavigation().
As a step in the process (and to move layout tests towards being green
in PlzNavigate), this CL eliminates the sending of this IPC for
browser-side failed navigations. It additionally eliminates the
assertion in RFHI that it has already seen a network error when
receiving a DidFailProvisionalLoad IPC (since it will now be seeing
that IPC only for renderer-side failed navigations).

This CL also updates the browser_tests PlzNavigate filter for the tests
that are now failing because they rely on listening for the deprecated
WCO callbacks from browser-side failed navigations.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2173803003
Cr-Commit-Position: refs/heads/master@{#414384}

These ASSERT()s compare HashMap iterators, and DCHECK*() requires stream printers
for the iterators. This CL adds them.

BUG=596760

Review-Url: https://codereview.chromium.org/2274403002
Cr-Commit-Position: refs/heads/master@{#414383}

This implementation will be invoked from Blink via Mojo and will return the
budget information specified in the BudgetAPI.

Current implementation only support the getCost call, although the GetBudget
call is also defined.

BUG=617971

Review-Url: https://codereview.chromium.org/2265173002
Cr-Commit-Position: refs/heads/master@{#414382}

This patch introduces |DragController::hasCaretIn()| as replacement of
|DragController::caretLayoutObject()| and |DragController::isContentEditable()|
to provide high-level API for improving code health.

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

Review-Url: https://codereview.chromium.org/2272343002
Cr-Commit-Position: refs/heads/master@{#414381}

As of https://crrev.com/414077, we would not recognize swipe gestures when the NTP is scrolled, but the touch event would still be dispatched by the toolbar (which would then ignore it). Instead, we now forward the touch events to the NewTabPageView if the toolbar is moved away from its original position.

BUG=631976

Review-Url: https://codereview.chromium.org/2276793003
Cr-Commit-Position: refs/heads/master@{#414380}

This CL includes following changes
- Add a dismiss button on warning banner about Downloads space. If a user dismiss the banner, we won't show unless 36 hours have passed.
- Show the warning banner about Downloads when the available size is less than 1GB. (It used to be 20%)

BUG=634207
TEST=manually tested by modifying DOWNLOADS_SPACE_WARNING_DISMISS_DURATION from 36 hours to 10 seconds.

Review-Url: https://codereview.chromium.org/2271203002
Cr-Commit-Position: refs/heads/master@{#414379}

It has recently completed its 90d compliance monitoring period.

BUG=605415

Review-Url: https://codereview.chromium.org/2202823003
Cr-Commit-Position: refs/heads/master@{#414378}

BUG=640560

Review-Url: https://codereview.chromium.org/2277833002
Cr-Commit-Position: refs/heads/master@{#414377}

BUG=none
R=brettw@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#414376}

A mostly-automated change to convert instances of WrapUnique(new Foo(...)) to
MakeUnique<Foo>(...). See the thread at
https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/iQgMedVA8-k
for background.

To avoid requiring too many manual fixups, the change skips some cases that are
frequently problematic. In particular, in methods named Foo::Method() it will
not try to change WrapUnique(new Foo()) to MakeUnique<Foo>(). This is because
Foo::Method() may be accessing an internal constructor of Foo.

Cases where MakeUnique<NestedClass>(...) is called within a method of
OuterClass are common but hard to detect automatically, so have been fixed-up
manually.

The only types of manual fix ups applied are:
1) Revert MakeUnique back to WrapUnique
2) Change NULL to nullptr in argument list (MakeUnique cannot forward NULL
   correctly)
3) Add base:: namespace qualifier where missing.

WrapUnique(new Foo) has not been converted to MakeUnique<Foo>() as this might
change behaviour if Foo does not have a user-defined constructor. For example,
WrapUnique(new int) creates an unitialised integer, but MakeUnique<int>()
creates an integer initialised to 0.

git cl format has been been run over the CL. Spot-checking has uncovered no
cases of mis-formatting.

  BUG=637812

Review-Url: https://codereview.chromium.org/2253943004
Cr-Commit-Position: refs/heads/master@{#414375}

m_client is null after frame gets detached. Hence we don't need to check
frame() in ScreenOrientation::isActiveAndVisible.

This is a follow-up after https://codereview.chromium.org/2272043003/.

BUG=

Review-Url: https://codereview.chromium.org/2273193003
Cr-Commit-Position: refs/heads/master@{#414374}

Until now, System Health stories had to be recorded through a benchmark,
e.g. system_health.memory_*:

  tools/perf/record_wpr mobile_memory_system_health
  tools/perf/record_wpr desktop_memory_system_health

This goes against the idea of SH stories being reusable across multiple
benchmarks. Instead, it is now possible (and recommended) to record
directly through the story set:

  tools/perf/record_wpr mobile_system_health_story_set
  tools/perf/record_wpr desktop_system_health_story_set

This will ensure that all SH stories are recorded under the same
conditions (10s wait at the end of the story without a memory dump).

BUG=589726

Review-Url: https://codereview.chromium.org/2276943002
Cr-Commit-Position: refs/heads/master@{#414373}

We observed that if a screenshot is captured while simultaneously the frame
is resized, it is possible that the screenshot is lost. This is because the latency
info (which identifies a CompositorFrame as the screenshot's frame) is sent
from the renderer within a wrongly sized CompositorFrame, which is then
skipped by DelegatedFrameHost.

For situations with an active resize lock, the latency info attached to dropped
frames is saved by the DFH and re-attached to the first frame that is not
dropped. However, this does not happen if the frame is dropped further down
the line, because it is not of the desired size of the RWHV.

This patch also saves the latency info for such wrongly sized frames in DFH,
until a correctly sized frame is submitted.

BUG=637066

Review-Url: https://codereview.chromium.org/2248183003
Cr-Commit-Position: refs/heads/master@{#414372}

getSizeStat() for Downloads used to return
|actual available space| + |evictable Drive cache| - 512MB.
To be consistent with Storage manager (chrome://settings/storage), getSizeStat()
should simplly return |actual available size|.

BUG=640504
TEST=manually compared the avaiable space in Files app and Storage manager.

Review-Url: https://codereview.chromium.org/2274003004
Cr-Commit-Position: refs/heads/master@{#414371}

Add a caution to LeakExpectations and mention that memory leaks must be fixed asap.

BUG=

Review-Url: https://codereview.chromium.org/2277663004
Cr-Commit-Position: refs/heads/master@{#414370}

This CL implements a loading view (aka progress bar) in download manager
UI on Android. It is slightly trickier to handle state changes on
Tablets, because when the download ui is initialized, a filter is
already set in the URL. Therefore we cache the filter temporarily in the
manager, and will show it after we load everything.

There are three possible types of backend the UI needs to wait:
incognito downloads, downloads and offline apges. The UI will only claim
it is loaded if all of them are loaed.

TODO(ianwen): add a timeout mechanisim to either the
DownloadLoadingDelegate or to the backend so that if it takes forever to
load one of the backend, users are still able to see the other two.

BUG=616324

Review-Url: https://codereview.chromium.org/2269353004
Cr-Commit-Position: refs/heads/master@{#414369}

When official builds switched from gyp to gn some smoothness tests
regressed. Investigation showed that some skia functions were taking
up to 15x longer to execute. This was due to /GL /O2 being changed to
/O1 (LTCG and optimize-for-speed changing to optimize-for-size). This
change resolves most of the compiler option differences in skia and
dramatically reduces the CPU usage in the gn build, from 9.4 s to 5.3,
within spitting distance of the gyp build.

This increases the size of chrome_child.dll from 48,593,408 to
49,213,440 bytes. The gyp version is 49,032,192 bytes and if necessary
some more size optimizations can be applied to the gn build.

The discrepancy between gyp and gn optimization settings in skia occurred
when crrev.com/1410883008 landed for gyp only.

There may be other differences - investigation continues - but this fixes
the main issues noticed.

BUG=632651

Review-Url: https://codereview.chromium.org/2270693006
Cr-Commit-Position: refs/heads/master@{#414368}

Each WebUSB function (on the USB and USBDevice IDL objects) is given a
histogram value that is recorded when the function is called.

BUG=492204

Review-Url: https://codereview.chromium.org/2272563003
Cr-Commit-Position: refs/heads/master@{#414367}

Push updating of the animation value into SVGElement. This resembles the
Web Animations code-path to some degree and maybe we can make them even
more similar eventually. This is the first CL in a series that will
remove knowledge of <use>/shadow trees from the SMIL animation code.

BUG=640676

Review-Url: https://codereview.chromium.org/2272033002
Cr-Commit-Position: refs/heads/master@{#414366}

This is similar to earlier https://codereview.chromium.org/2163223002

BUG=584079
R=emso@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#414365}

This will ensure we don't run into missing dependencies.

BUG=584079
R=emso@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#414364}

I had to add them in https://codereview.chromium.org/2052613003

BUG=616502

Review-Url: https://codereview.chromium.org/2276863002
Cr-Commit-Position: refs/heads/master@{#414363}

On Mac all the rendering from the same renderer to the same device runs on the same buffer size which is a minimum of all the requested buffer sizes. FIFO is injected in case that buffer size does not match the one requested by the audio stream. Here we add tracing of the buffer mismatch.
Looks like it's a long enough explanation of 0.5 LOC change :)

TBR=dalecurtis@chromium.org

Review-Url: https://codereview.chromium.org/2278483002
Cr-Commit-Position: refs/heads/master@{#414362}

This patch converts "editing/insert-3778059-fix.html" to use w3c test
harness to simplify test script and renames to "insert_between_block_quote.html"
to provide to provide a hint what this test script verifies, for improving
readability.

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

Review-Url: https://codereview.chromium.org/2280533003
Cr-Commit-Position: refs/heads/master@{#414361}

These tests are currently disabled by TestExpectation
for failing, but will be enabled again in the following CLs.

BUG=594918

Review-Url: https://codereview.chromium.org/2279693002
Cr-Commit-Position: refs/heads/master@{#414360}

This patch converts "editing/before-after-input-element.html" to use w3c test
harness to simplify test script for improving readability.

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

Review-Url: https://codereview.chromium.org/2275223002
Cr-Commit-Position: refs/heads/master@{#414359}

After https://codereview.chromium.org/2251073002 we should no longer
need to walk any <use> instances "manually", since setNeedsStyleRecalc
does that already. This should also avoid the potential double-work
because of this.

TEST=svg/W3C-SVG-1.1/animate-elem-30-t.svg
TEST=svg/W3C-SVG-1.1/animate-elem-40-t.svg

BUG=166438,355359,484552,637310

Review-Url: https://codereview.chromium.org/2270363004
Cr-Commit-Position: refs/heads/master@{#414358}

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/bb0e6881e53477341891c9e3abe91e0abfbec937 Add comma to prevent unintended string concatenation (tikuta@chromium.org)
  https://crrev.com/eac0c99f5e54e20af75ce3277a42f2c31c614338 Make all the rest of the recipes explicitly specify GIT_MODE (agable@chromium.org)
  https://crrev.com/245c516049e221f79dc8cf8fe395138b782308a0 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/4b21db06e48e90335a0a1ae5d5b911b960ea0e5a Default gclient's GIT_MODE to True (agable@chromium.org)

TBR=martiniss@chromium.org,phajdan.jr@chromium.org
BUG=640251

Recipe-Tryjob-Bypass-Reason: Autoroller
Bugdroid-Send-Email: False
Review-Url: https://codereview.chromium.org/2272333002
Cr-Commit-Position: refs/heads/master@{#414357}

By moving the url_formatter methods out of
org.chromium.chrome.browser.UrlUtilities to it's component, other emdedders than
Chrome can benefit from the Java version.

This CL removes the tests originally written for a Java implementation of the
methods replaced by FormatUrlForSecurityDisplay in
https://codereview.chromium.org/1357563002. It's not trivial to move the tests
over to the component as they depend on native library initialization performed
by //content which is disallowed in components/url_formatter/DEPS. Also, the
tests are redundant as the code is thoroughly tested by other means.

Currently there are no users of formatUrlForDisplay() in Chromium, but Opera
would like it exposed.

BUG=624407

Review-Url: https://codereview.chromium.org/2110543004
Cr-Commit-Position: refs/heads/master@{#414356}

The pickle protocol uses __dict__ if __getstate__ and __setstate__
aren't defined. Using __dict__ is enough for IdlArgument.

BUG=569301

Review-Url: https://codereview.chromium.org/2272283002
Cr-Commit-Position: refs/heads/master@{#414355}

BUG=640628
TBR=pthatcher@chromium.org

Review-Url: https://codereview.chromium.org/2274173002
Cr-Commit-Position: refs/heads/master@{#414354}

https://codereview.chromium.org/2110993003 implemented the filtering for empty usernames.

BUG=400674

Review-Url: https://codereview.chromium.org/2277793002
Cr-Commit-Position: refs/heads/master@{#414353}

BUG=624086,640876
TBR=brucedawson@chromium.org

Review-Url: https://codereview.chromium.org/2278843002
Cr-Commit-Position: refs/heads/master@{#414352}