GitLab

BUG=641215

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

Now that Oilpan is shipped, we no longer need public/features.gni.

BUG=

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/d046985fcb95..2f63c684f0aa

$ git log d046985fc..2f63c684f --date=short --no-merges --format='%ad %ae %s'
2016-09-08 benjhayden Approve Destructuring and Subclassable Built-ins in style-guide.md.
2016-09-08 benjhayden Display "(empty)" and "(missing)" in value-set-table

TBR=catapult-sheriff@chromium.org

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

Assertions are removed from some of the InputConnection methods that
merely post on UI thread. In doing so, mPendingAccent was changed to be
accessed only on UI thread.

However, beginBatchEdit() / endBatchEdit() implementations are quite
complicated and no evidence calling those on UI thread was found so far.

Also, for get* methods we return cached results.

BUG=643477

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

This removes some duplicated code.

BUG=634443

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

If a _loadCompletions was started while an earlier one was active, the earlier one could still be used.

BUG=none

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

This CL adds a new UMA metric to record the identifier of an infobar
which was visible to the user and hid another infobar behind it. This CL
also updates the visible/hidden check to simply query whether or not
another infobar exists. This is to correct an overcounting issue that
recorded infobars which appeared after another infobar was dismissed
as being hidden.

BUG=589163

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

This is a step towards turning all tree outlines into independent web components,
removing global styles related to it and allowing more components to encapsulate their inner details.

BUG=none

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

BUG=468355

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

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

New:
* win: Speculative fix for LNK1285/delete-pdb-and-rebuild
* win: Support for [] ATL-style uuid() attributes
* win: __nop intrinsic now has a definition
* win: complete codeview debug info for vptrs

Also switch to gnuwin-5, which is identical to gnuwin-4
except that it has gnuwin32's od.exe too.

Also switch to the python version of the Android NDK
"make standalone toolchain" script, since the .sh version
is deprecated and stopped working after the recent NDK
update.

BUG=644351,644976,644977

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

It's more idiomatic in Blink land.

R=esprehn
BUG=

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

BUG=644373

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

In the renderer, extensions use sync IPCs to request port ids and
request the i18n message bundle. We're looking into fixing this, but
it'll be useful to know how much time it's taking now.

BUG=645261

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

Often times we are doing double work parsing color text to handle invalid
colors and then parsing color text again in setColorText. We should just
parse once and pass in the Color object to ColorSwatch.

BUG=642782

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/3d918b93..d87b834b

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_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=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

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

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/3b96eccc03f1e5ab9efa74b1afdfb5edc8c5feb0 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/c958da49797facf429100595bceb4c49e8c94cd3 Switch bot_update to use exponential backoff on clone (agable@chromium.org)

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

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

It is expected to fix flaky build failures in the BUG.

Using source code names, this CL adds a dependency
chrome/test/base/ui_test_utils.cc
--> content/public/browser/render_process_host.h

Other dependencies
content/public/browser/render_process_host.h
--> services/shell/public/cpp/interface_registry.h
--> out/Release/gen/services/shell/public/interfaces/interface_provider.mojom.h
are already written.

BUG=633077

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/d674036d5ba7..d046985fcb95

$ git log d674036d5..d046985fc --date=short --no-merges --format='%ad %ae %s'
2016-09-08 benjhayden Make metric tests examine Histograms instead of Scalars
2016-09-08 benjhayden Move displaying errors from value-set-table to metric-side-panel

TBR=catapult-sheriff@chromium.org

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

https://pdfium.googlesource.com/pdfium.git/+log/cf7ac195266f..cba85abc8a9d

$ git log cf7ac1952..cba85abc8 --date=short --no-merges --format='%ad %ae %s'
2016-09-08 jaepark CPWL_Color::ConvertColorType should compare current color type with the given convert color type.

TBR=dsinclair@chromium.org

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

This change adds a method to the ResourceManager that allows users
to create a tinted version of a static resource. These resources are
cached until the layout changes. In practice, this new functionality
is used to tint the border resource and close button in the tab
switcher.

BUG=641517
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

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

Also removes class toggling hackery.

R=alancutter@chromium.org
BUG=643651
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
NOPRESUBMIT=true

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

According to crash dump OOM happens when GetWorkingSetKBytes
is invoked via OomMemoryDetails from tab discarding code
when it reacts to critical memory pressure.
The available memory is already low at that moment so
allocating 1.4-1.5 MiB to gather Working Set details is
likely to fail.

The fix uses UncheckedMalloc to avoid triggering an OOM
crash and allow GetWorkingSetKBytes to fail, which the
code calling GetWorkingSetKBytes should already handle.
Another part of the fix is to reduce the extra buffer size
allocated by this code from 25% to 10%.

BUG=642186

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

The problem was addressed in r403353. We can reenable tests.

BUG=604056

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

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/026e0bb03aa139c5c43be38dd37d1f2f33e90c03 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/3dd7436899f87b0bb1538b43c4332e327fd70a98 Remove unused PATCH_TOOL and resulting dead code from bot_update. (iannucci@chromium.org)

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

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

https://chromium.googlesource.com/skia.git/+log/b3cb2142725f..426a2459290a

$ git log b3cb21427..426a24592 --date=short --no-merges --format='%ad %ae %s'
2016-09-08 halcanary SkPDF: hand SfntlyWrapper::SubsetFont() ttcIndex, not fontName.
2016-09-08 kjlubick Fix strange failures on Android N

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=caryclark@google.com

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

BUG=

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

Remove an unecessary switch exposed during ash::DisplayManager
refactoring.

BUG=581462

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

New minimum height (77 dip) is matched for a single custom notification.

BUG=642804

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

Revert of service worker: Add CHECKs to catch if SWProcessManager isn't releasing ref counts. (patchset #1 id:1 of https://codereview.chromium.org/2318443002/ )

Reason for revert:
SWProcessManager is innocent. The CHECKs aren't failing.

Original issue's description:
> service worker: Add CHECKs to catch if SWProcessManager isn't releasing ref counts.
>
> We're still not sure where the non-zero ref counts on shutdown come from.
>
> BUG=636377
>
> Committed: https://crrev.com/e7ffbf80d3b612521a7c2564b21b96695586274a
> Cr-Commit-Position: refs/heads/master@{#416596}

TBR=horo@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=636377

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

BUG=582312

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

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/9d8799334bb0e9b4082b900d99862d29583b6d48 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
depot_tools:
  https://crrev.com/3454ae85da8913a18a67adaaf8727fa9783127d3 Reland "Delete lots of svn logic from bot_update" (agable@chromium.org)

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

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

https://pdfium.googlesource.com/pdfium.git/+log/0ca160fbf31a..cf7ac195266f

$ git log 0ca160fbf..cf7ac1952 --date=short --no-merges --format='%ad %ae %s'
2016-09-08 tsepez explicit operator bool for CFX_RetainPtr and CFX_CountRef
2016-09-08 weili Reland of Fix leaked internal font

TBR=dsinclair@chromium.org

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/0cb5bdd1..3d918b93

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_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=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

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

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/3a2fd516556039032859dc057036402afdd9c709 Turn on Android N5X Swarm Builder for the chromium gatekeeper. (stip@chromium.org)

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

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

It will allow streaming the profile tree.

BUG=406277

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

The flaky was due to the issue when trying to load OfflinePageModel, it
would crash with SQLite Error 1802. Fixed by mock the OfflinePageModel
if we're in a testing environment.

BUG=644570

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

I had to implement a custom callback dispatch handler.
Used MutationObserver code as a template.

Added a test case.

BUG=641353

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

In very rare circumstance (<5% connection attempt)
GlRenderer::OnPixelTransformationChanged is called when SurfaceCreated has not
been called, which crashes the app since |canvas_| has not been set. This CL
adds check for |canvas_| to prevent crashing. For now we don't know in that
case whether the transformation was set before SurfaceCreated or the surface
had actually been created before the surface callback was registered. If former
is true then this CL will fix the problem since SurfaceCreated will implicitly
cause TransformationChanged. If latter is true then nothing will be drawn on
the view although the app doesn't crash, and we will need to investigate why
this can ever happen.

BUG=644924

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