GitLab

Allows the voice search button to display alongside the delete icon in
the omnibox, as long as voice search is enabled. Hides the feature
behind a flag.

Bug: 807726
Change-Id: I3695222b9ca0f74673ca286e052b121fdfde924b
Reviewed-on: https://chromium-review.googlesource.com/896002
Commit-Queue: Troy Hildebrandt <thildebr@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Maria Khomenko <mariakhomenko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537289}

https://chromium.googlesource.com/external/leveldb.git/+log/ca216e493f32..41172a24016b

This CL also adds Clang thread safety annotations [1] to Chrome's
leveldb::port::Mutex. This can't be done in a separate CL because the
primitives are used in Chrome code that is compiled with -Wthread-safety
turned on.

[1] https://cs.chromium.org/chromium/src/base/thread_annotations.h

$ git log ca216e493..41172a240 --date=short --no-merges --format='%ad %ae %s'
2018-02-13 costan Enable thread safety annotations in open source version.
2017-10-25 cmumford Add leveldb_options_set_max_file_size to the C API.
2017-10-23 cmumford Fixed style guide link in CONTRIBUTING.md
2017-10-19 cmumford Deleted unused assignments in Reader.
2017-10-17 cmumford leveldb::DestroyDB will now delete empty directories.
2017-10-10 costan Fix typo (forgotten reference operator) in test.
2017-10-10 costan Replace SSE-optimized CRC32C in POSIX port with external library.

Created with:
  roll-dep src/third_party/leveldatabase/src

Change-Id: I68dc2799673a93253bc338b141e0945b11db097a
Reviewed-on: https://chromium-review.googlesource.com/920773Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537288}

We no longer have WebTaskRunner, and the function is now essentially
identical to GetSingleThreadTaskRunnerForTesting().

This patch removes CreateWebTaskRunnerForTesting() along with its helper
class, and migrates all the call sites to GetSingleThreadTaskRunnerFor-
Testing().

Bug: 794845
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I11094734d79aca4dbf4d7b191df6adb76cce19c1
Reviewed-on: https://chromium-review.googlesource.com/923869Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537287}

In addition to setting request priority in the request itself, add it as
a get parameter to the fetch url.

This is needed for load shedding later.

Bug: 809986
Change-Id: I4d6520218d6ef9d25feada409cf894485e3e2872
Reviewed-on: https://chromium-review.googlesource.com/913618Reviewed-by: Chris Pickel <sfiera@chromium.org>
Commit-Queue: Jan Krcal <jkrcal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537286}

This CL adds a submenu arrow to the "Sync to another account"
button in the user menu to match the mocks under:
https://gallery.googleplex.com/projects/MCHbtQVoQ2HCZWnUS9sdsb0Q/files/MCHtA7U1iMGr68h8u9rYJdKa9KgqJ7geGGg

Bug: 786369
Change-Id: I9c3bad79535ba69bd1dc3bdf775fb059c608bba9
Reviewed-on: https://chromium-review.googlesource.com/919483
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537285}

Of the code pieces left to add jumbo support to, the one which benefit
the most from jumbo is //chrome/browser:browser which goes from 90 to
10 CPU minutes, a reduction of more than a CPU hour. This is about 10%
of the total effort compiling chrome+content_shell+blink_tests.

In jumbo builds many cc files compile in the same translation unit
which reduces the time spent generating debug data and machine code
for headers, as well as the time to actually process the headers.

Bug: 746957
Change-Id: I856f3fca1f16fa2b0738678444c455c303985962
Reviewed-on: https://chromium-review.googlesource.com/899368Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#537284}

1) Audio service is embedded into /content. For that DeferredSequencedTaskRunner for Audio service
is introduced in BrowserMainLoop. This task runner is passed into AudioServiceContext and queues
all the requests to the Audio service and its factory until it is started. And it is started after
AudioManager is created in BrowserMainLoop::BrowserThreadsStarted().

2) audio::CreateAudioSystem() is introduced as a replacement to
media::AudioSystem::CreateInstance(). At this point it still creates media::AudioSystemImpl
instance.

This CL does not change execution path, and (hopefully)  should not be a source of regressions.
The actual switch to using Audio service and audio::AudioSystemToServiceAdapter happens in
the follow-up https://chromium-review.googlesource.com/c/chromium/src/+/918901.

So if something breaks https://chromium-review.googlesource.com/c/chromium/src/+/918901 should
be reverted first (and most likely that should be enough).

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ie69df7bf9f593fda6a2e85f5f57a49a90aeccdb9
Bug: 792441
Reviewed-on: https://chromium-review.googlesource.com/875927Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Max Morin <maxmorin@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Olga Sharonova <olka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537283}

We incorrectly assumed it was called only once.  This patch fixes that
assumption.

Change-Id: I9d8e624634260ca1e352d9419e84b52e64e627a4
Reviewed-on: https://chromium-review.googlesource.com/921082
Commit-Queue: Alex Clarke <alexclarke@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537282}

This is the first in a series of patches to implement tabs.removeCSS

A style sheet injected with tabs.insertCSS will have an internal
autogenerated "key" that will be passed to StyleEngine. In the case of
files, this will likely be just the fully qualified name of the file,
whereas in the case of code it could be the code string itself or a hash
digest of the code string. In either case, a call to tabs.removeCSS with
the same options will deterministically generate the same internal key,
thus enabling the removal of a previously injected style sheet without
the explicit use of IDs and such.

This patch lays the groundwork in StyleEngine that will enable us to
implement behavior similar to that observed in Firefox 57.0.

Changes:

 1.  WebStyleSheetId is now WebStyleSheetKey (WebString)
 2.  WebDocument::InsertStyleSheet now accepts an optional key as a
     parameter
 3.  StyleEngine::InjectSheet and StyleEngine::RemoveInjectedSheet now
     take StyleSheetKey (AtomicString) instead of WebStyleSheetId
 4.  Both WebDocument::RemoveInsertedStyleSheet and
     StyleEngine::RemoveInjectedSheet now take the CSS origin as a
     parameter
 5.  StyleEngine::RemoveInjectedSheet now removes only the last style
     sheet injected with the given key and only from the specified
     CSS origin

BUG=608854

Change-Id: Ie62bfcb0d8309b18cbf0d2cd119c74908e396767
Reviewed-on: https://chromium-review.googlesource.com/877663Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Manish Jethani <m.jethani@eyeo.com>
Cr-Commit-Position: refs/heads/master@{#537281}

kLegacyUserAgentV0 type is gone, and now 2 bits are enough to
hold the type for ShadowRoot, instead of 3.

Bug: 787717
Change-Id: I64b00c9ac79d8d43830ac94cb5bd013a9043dcca
Reviewed-on: https://chromium-review.googlesource.com/923783Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537280}

TBR=jcivelli@chromium.org

Bug: 812998
Change-Id: I34b710517d8c6b066e9f66d6f98863194046ddfd
Reviewed-on: https://chromium-review.googlesource.com/923942Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Christos Froussios <cfroussios@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537279}

This fixes jumbo builds after https://chromium-review.googlesource.com/923604

Change-Id: Ib7030da7e0a6acfdad8200564f9375ba218e8ca6
Reviewed-on: https://chromium-review.googlesource.com/923723Reviewed-by: Fredrik Söderquist <fs@opera.com>
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#537278}

Also, ignore any actions buttons beyond the first two (instead of
silently moving them to the secondary toolbar) if they are explicitly
meant to go to the primary toolbar.

Bug: 810854
Change-Id: I97d4730c884209848096e79e5f2bc414e051a8b1
Reviewed-on: https://chromium-review.googlesource.com/919087Reviewed-by: Yusuf Ozuysal <yusufo@chromium.org>
Commit-Queue: Bernhard Bauer <bauerb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537277}

The metrics themselves are added in the separate CL in webrtc.

Bug: webrtc:8355
Change-Id: Ic7d98d7e1f5b30d62a79418b110e3e9c7123008f
Reviewed-on: https://chromium-review.googlesource.com/921221Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Commit-Queue: Ilya Nikolaevskiy <ilnik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537276}

During compaction each level is scanned at the same time sequentially.
This means there are files being read from each level at the same
time but each level is read in a sequential manner. Files are read
multiple times during scanning, first reading the end of the file then
contents inside.

Because of this behavior, most-recently-used eviction is the worst
because:
1. The multiple reads in a file (in a non-sequential manner) causes a
   ton of filesystem thrashing here.
2. Files are never re-read once they are finished being scanned in a
   level, so there is no reason to keep it the handle around.

Least-recently-used eviction more closely models the reading behavior
of a leveldb compaction, and this uses leveldb's sharded LRU cache.

File thrashing here is the suspected cause of the assigned bug, where
leveldb never opens. This is a speculative but educated guess fix.

Bug: 795369
Change-Id: I7f4965a2a93b05ae558688e58d220b0fcf549430
Reviewed-on: https://chromium-review.googlesource.com/905575
Commit-Queue: Victor Costan <pwnall@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Reviewed-by: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537275}

The only thing such floats should affect is the block position of
subsequent floats (float rule §5 [1] and clearance). They should not
affect the inline position or inline size of subsequent floats or new
formatting contexts.

The spec isn't actually clear here, although the browsers agree on the
above. Submitted https://github.com/w3c/csswg-drafts/issues/2312

[1] https://www.w3.org/TR/CSS22/visuren.html#float-position

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ia9e724c8a58fcb017f86f26bdf90394f09414f4e
Reviewed-on: https://chromium-review.googlesource.com/919065Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537274}

Our C++ cloneNode() implemented web-exposed Node.prototype.cloneNode()
[1] and DOM "clone" algorithm [2]. This CL makes C++ cloneNode()
responsible only for [1], and moves [2] to virtual Node::Clone().

- Rename Node::importNode override functions to Clone().
- Add Document& argument to support importNode case
- Remove ExceptionState argument. Clone() never throws.

This is a preparation to use [2] for importNode().
This CL has no behavior changes.

[1] https://dom.spec.whatwg.org/#dom-node-clonenode
[2] https://dom.spec.whatwg.org/#concept-node-clone

Bug: 812089
Change-Id: I523e81b32bb8c1e80273e9aa9529cdd6972ca8bf
Reviewed-on: https://chromium-review.googlesource.com/923684Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537273}

TBR=perezju@chromium.org
NOTRY=true

Bug: 812807
Change-Id: If6797e1cf2ad7d3dd776570072294e62593ae70d
Reviewed-on: https://chromium-review.googlesource.com/922922Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537272}

Instead of using a bundled URLLoaderFactory that can selectively get
multiple factories. The bundled URLLoader factory's default factory
for network (e.g. for http/https) scheme may be replaced with custom
factories (e.g. AppCache one), and SW doesn't want to use it when
falling back.

We used to use 'constraints' flag in SharedURLLoaderFactory to control
this behavior but it's needed only one place where Worker makes
SW-controled fetch. Instead we can just pass the network-direct URLLoader
factory in addition to the bundled one, then we can eliminate the
'constraints' flag too.

R=falken@chromium.org, jam@chromium.org

Bug: 715640
Change-Id: I1604de28b22c95e0fd79aa230c5f0cb1c59edc85
Reviewed-on: https://chromium-review.googlesource.com/923621
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537271}

Currently 'git cl uplaod' fails when all files are skipped (e.g. fixing
LayoutTest). This patch makes it pass.

Bug: none
Change-Id: Ib028d39ed63e0cba21918cb565a5c264ad047da7
Reviewed-on: https://chromium-review.googlesource.com/923782Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537270}

We plan to have WebViewScheduler owned by Page, which will delay the
execution of WebViewScheduler's destructor because Page is garbage
collected.

This patch moves the execution of the destructor to WebViewImpl::
Close() to ensure nothing will break even if we clear the
WebViewScheduler object early.

Bug: 812986
Change-Id: I5de6b7ad3f3598922a408b2a26cec4718a132f6f
Reviewed-on: https://chromium-review.googlesource.com/923246Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537269}

We should never leave any node which has a LayoutObject, but doesn't have a parent in a flat tree
after having run UpdateStyleAndLayoutTree. That means we forget to detach a node which should be
detached.

Update TestExpectations because some tests would hit this assertion when IncrementalShadowDOM is
enabled.

Bug: 776656
Change-Id: I273b9ddd083766fe687aedb178fc6e0c65248fde
Reviewed-on: https://chromium-review.googlesource.com/920329
Commit-Queue: Hayato Ito <hayato@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537268}

When a synchronous profile load is happening (e.g. after chrome restart
on Chrome OS), make sure to invoke UserCloudPolicyManagerChromeOS's
OnStoreLoaded handling.
This is also the place where user affilitaion is determined.

BUG=810013
TEST=browser_tests --gtest_filter=*UserAffiliation*

Change-Id: Ib4a4b1ad3e61eba637f527dfa89cb5cd1fd2159b
Reviewed-on: https://chromium-review.googlesource.com/908565Reviewed-by: Sergey Poromov <poromov@chromium.org>
Reviewed-by: Drew Wilson <atwilson@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537267}

This patch changes "5206311-1.html" to utilize |selection_test()| for
ease of maintenance.

Following patch will rename this test file to
"delete_selection_contains_table_row.html" to provide hint about
what this test file verifies.

Change-Id: Ie0f6e54b998c9679e316f5e488e6a19679236d12
Reviewed-on: https://chromium-review.googlesource.com/914209
Commit-Queue: Hikaru Nishida <hikarun@google.com>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537266}

Change-Id: Ice85ebdd73350f8228a7591496716d73efe3695b
Reviewed-on: https://chromium-review.googlesource.com/922862
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537265}

This patch plus https://chromium-review.googlesource.com/c/chromium/src/+/921082
Fixes the broken headless browswer tests in
https://chromium-review.googlesource.com/c/chromium/src/+/762520

Bug: 777763
Change-Id: I44ef4ea78e673129e65d23fdd4cd1169569e478b
Reviewed-on: https://chromium-review.googlesource.com/922622
Commit-Queue: Alex Clarke <alexclarke@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537264}

We removed this constraint earlier.

TBR=avi@chromium.org

Bug: None
Change-Id: I6ccbfc0d15e520a48921328eb5966147b0aa29dd
Reviewed-on: https://chromium-review.googlesource.com/922504Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537263}

https://chromium.googlesource.com/angle/angle.git/+log/b92f92a780a1..b66de58fc571

$ git log b92f92a78..b66de58fc --date=short --no-merges --format='%ad %ae %s'
2018-02-15 thakis Mark third-party code in ANGLE as no_chromium_code.
2018-02-15 jmadill Add missing is_fuchsia to system_utils selection.
2018-02-15 jmadill Vulkan: Add depth/stencil surfaces.

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=lucferron@chromium.org

Change-Id: Ib19d3fc02e35bb0b490918a34e3b8b48e6c41901
Reviewed-on: https://chromium-review.googlesource.com/923702Reviewed-by: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#537262}

This reverts commit 2319141f.

R=chrishtr@chromium.org
TBR=enne@chromium.org, piman@chromium.org

Bug: 776216
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I36744f2939f565ce2e89a5d7c850dfcd138b70cb
Reviewed-on: https://chromium-review.googlesource.com/918122Reviewed-by: Khushal <khushalsagar@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Khushal <khushalsagar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537261}

This CL changes the animations of the toolbar when the omnibox is
unfocused. The buttons are now appearing after the omnibox is resized.

Bug: 804748
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I149a004f4aa9e8d643ad2cc6d88de95855a66f83
Reviewed-on: https://chromium-review.googlesource.com/922102
Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org>
Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537260}

This CL increases AudioArray alignment on x86 family architectures so
that blink::VectorMath functions can better utilize AVX optimizations.

Bug: 778262
Change-Id: Ibe94130ba0d0223342ae25f844c69767194370e9
Reviewed-on: https://chromium-review.googlesource.com/738239Reviewed-by: Raymond Toy <rtoy@chromium.org>
Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com>
Cr-Commit-Position: refs/heads/master@{#537259}

After multiple shadow roots are removed, getting a shadow root
can be done by Node::GetShadowRoot() or Element::GetShadowRoot().

Old code used to use get ElementShadow first, then use
ElementShadow::GetShadowRoot(), but it is no longer necessary.

This is a step to merge ElementShadow class into ShadowRoot class.

TBR=dmazzoni@chromium.org, mlamouri@chromium.org

Bug: 624724
Change-Id: I44af0d2de85e45badb36e5ba59facc03f7d81602
Reviewed-on: https://chromium-review.googlesource.com/923245
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537258}

Change-Id: Ibc5e0d946c57a4a3be7fc24a9c0773ec830514f0
Reviewed-on: https://chromium-review.googlesource.com/922822Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Jens Widell <jl@opera.com>
Cr-Commit-Position: refs/heads/master@{#537257}

The pull request has been merged.
https://github.com/WICG/webpackage/pull/119

Bug: 803774
Change-Id: Ic14b0eeb1e63ce687a51709098f049c2dea99cba
Reviewed-on: https://chromium-review.googlesource.com/923764Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537256}

Latest clang rolls require a bit more stack space when calling through
the eager marking visitors.

Bug: chromium:807994
Change-Id: I5bb2a80fdca7f5d536fd965a9cd6151e81573f9a
Reviewed-on: https://chromium-review.googlesource.com/922206Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537255}

Network traffic annotation is added to:
google_apis/gcm/engine/connection_factory_impl.cc

Bug: 656607
Change-Id: Ibc0d1cc04607b67100d9922c27b867db2761c633
Reviewed-on: https://chromium-review.googlesource.com/916001
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537254}

Add |factory| argument to Element::
CloneElementWithoutAttributesAndChildren(), and use it in Document::
importNode().

Bug: 572693, 812089
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I1c89cdf45f9a1f38b9ab6a3e7149692b66e1c44d
Reviewed-on: https://chromium-review.googlesource.com/920965
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537253}

These mock methods are never used by tests and can be safely removed.
If possible the corresponding virtual methods in
DataReductionProxyConfig are also made non-virtual.

Change-Id: I76aa2fe9ff1e8fddb5f9204b1d79681136694a16
Reviewed-on: https://chromium-review.googlesource.com/919322Reviewed-by: Tarun Bansal <tbansal@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537252}

HTMLShadowElement.h is unnecessary.

Change-Id: I5153baa3c3022d67b4f2754a6804e2f45c226495
Reviewed-on: https://chromium-review.googlesource.com/923682Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537251}

When requesting a buffer for OOP raster, the client requests the
largest free size for which no tokens are pending, and can be allocated
without waiting. This is queried from RingBuffer::GetLargestFreeSize.
However, since this size is rounded up to the next alignment when making
the allocation, this results in synchronously waiting on tokens to make
room for the extra padding.

Ensure that ShrinkLastBlock rounds up the new size for the block so
the buffer always has memory aligned free size.

R=enne@chromium.org, piman@chromium.org

Bug: 803925
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Iff1286b6f412f126dbf33300387f85f9c21e91ed
Reviewed-on: https://chromium-review.googlesource.com/922445
Commit-Queue: Khushal <khushalsagar@chromium.org>
Reviewed-by: enne <enne@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#537250}