GitLab

BUG=641413

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

To determine how much an box needs to shink to avoid a float, the code
was only taking into account the first line of the box. This doesn't
work when the float has a shape, as it can make it seem like there is
more space on the line than there actually is. This patch changes the
calculations to take into account the entire height of the box that
needs to be shrunk, and thus computes the correct amount of space
available.

This is a merge of http://trac.webkit.org/changeset/178045 and
http://trac.webkit.org/changeset/178064 by Bem Jones-Bey <bjonesbe@adobe.com>.

BUG=427989

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

Extract the mouse logic to the extent that
GestureManager can use solely MouseEventManager
instead of being friend with EventHandler and use
private functions. Also this refactoring cleaned
up some parameter passing from EventHandler to
MouseEventManager that was going through
PointerEventManager functions.

Initial refactoring

BUG=625844

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

BUG=651526

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

Can see current guide at:
https://chromium.googlesource.com/chromium/src.git/+/HEAD/tools/metrics/histograms/README.md#Revising-Histograms

Largest change is removal of the guide on exact linear histogram, I think it is less important to give guidance now there is an explicit macro.

BUG=

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

BUG=None

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/4e2b1f13..aaa05f8f

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/2390863003
Cr-Commit-Position: refs/heads/master@{#422777}

Notable changes:
 * CRWWebViewProxyImpl now uses synthesized getter and ivar for
property contentView
 * CRWJSPOSTRequestLoader now does not create a retain cycle of
CRWWKScriptMessageRouter objects. The cycle seems to have been harmless,
but it definitely brings no value, and it's better to not have it.

BUG=651830

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

Before this CL, any task posted to a SequencedWorkerPool redirected to
TaskScheduler after
SequencedWorkerPool::Shutdown(int max_new_blocking_tasks_after_shutdown)
returned was ignored.

With this CL, |max_new_blocking_tasks_after_shutdown| BLOCK_SHUTDOWN
tasks are allowed to be posted after SequencedWorkerPool::Shutdown
returns.

BUG=622400

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

The constructor parameters can be provided directly, and after doing so it
fits on one (80-column) line.

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

https://chromium.googlesource.com/skia.git/+log/2ebd0c80a2a9..ee843b2ae4d7

$ git log 2ebd0c80a..ee843b2ae --date=short --no-merges --format='%ad %ae %s'
2016-10-04 robertphillips Revert of Make GrResourceCache dynamically change between LRU and random replacement strategies. (patchset #8 id:140001 of https://codereview.chromium.org/2321563006/ )

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

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

Before this change, the only way to scan all keys was to call LoadEntries() which loads the whole database. In our use case, this means copying a lot of image data which is wasteful.
With LoadKeys(), we still need to iterate the whole database but we don't have to copy out (and parse) the values -- just the keys. LoadKeys() is useful when storing relatively large values and only access to the keys is required.

We'll use LoadKeys() for NTP content suggestions to garbage collect data. We do that at times when we have a list of all still alive elements and need to intersect that with the elements stored in the db.

BUG=649009

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

BUG=642994
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

It is no longer compiled.

BUG=None

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

Creates FaviconAttributesProvider, FaviconAttributes and FaviconViewNew
classes to present favicons. These classes intend to replace FaviconView
and FaviconViewProvider downstream.

BUG=None

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

This is a follow-up CL to https://codereview.chromium.org/2363183002/.

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

Revert of Automatically copy the DIA DLL in the Syzygy binaries directory. (patchset #11 id:240001 of https://codereview.chromium.org/2365893002/ )

Reason for revert:
Break gclient for the open contributors.

Original issue's description:
> Automatically copy the DIA DLL in the Syzygy binaries directory.
>
> Committed: https://crrev.com/f34c072397b99c313e79cd10a77f5ca397154399
> Cr-Commit-Position: refs/heads/master@{#422655}

TBR=chrisha@chromium.org,siggi@chromium.org,scottmg@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/6b7a634b..4e2b1f13

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/2387383002
Cr-Commit-Position: refs/heads/master@{#422766}

The registered histograms on the native side are exposed
as JSON text from the FeedbackCollector. The histograms are
exposed only if:
(i)  "Include screenshot and logs" is checked when
the feedback was collected, and
(ii) The feedback is submitted from a non-incognito
profile.

In the next CL, these would be attached to the feedback
report.

BUG=651621

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

Otherwise, the overflow menu stays visible while the controls are hidden
which is fairly odd. Hiding the overflow menu would not be a good
experience either because the timer to hide is fairly short.

BUG=None
R=zqzhang@chromium.org

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

Revert of Refactor WebappRegistry into a singleton instance. (patchset #11 id:200001 of https://codereview.chromium.org/2351113005/ )

Reason for revert:
chrome_public_test_apk failed.

https://uberchromegw.corp.google.com/i/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/36476

Original issue's description:
> Refactor WebappRegistry into a singleton instance.
>
> This CL refactors WebappRegistry and WebappDataStorage to make most of
> the methods synchronous. WebappRegistry is now a singleton instance that
> is instantiated at browser startup. This allows all SharedPreferences files to
> be pre-warmed before the class is used; new web apps open new
> SharedPreferences on a background thread when registered, after which the
> preferences are cached automatically.
>
> Most static methods on WebappRegistry and WebappDataStorage have been
> converted to instance methods or removed. This makes the code much
> cleaner and more efficient; each static method had to independently open
> their SharedPreferences, which minimally performs a stat() on the
> underlying XML file to see if it has changed. Now the singleton
> WebappRegistry caches all WebappDataStorage objects on startup and
> whenever new ones are added. This reduces disk IO overhead.
>
> This CL allows all calls to SharedPreferences.Editor.apply() in
> WebappRegistry and WebappDataStorage to occur on the main thread,
> mostly removing the need for unwieldy callback interfaces and bare
> pointer passing across the JNI.
>
> BUG=633791
>
> Committed: https://crrev.com/78cf6596040662c0a73c0e4038bc84fb2b3c470a
> Cr-Commit-Position: refs/heads/master@{#422703}

TBR=dfalcantara@chromium.org,wnwen@chromium.org,msramek@chromium.org,dominickn@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=633791

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

BUG=388194,649710

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

In order to avoid re-entrancy issues, this CL modifies sensor event
dispatching code and uses postTask to schedule event dispatch.
New test is added to ambient-light-sensor.html that checks if generic
sensor code works correctly when sensor is not supported

BUG=606766

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

It is useless to launch a rename process directly from a system-level
Chrome. That process can't rename the Chrome executables since it isn't
elevated.

BUG=650981

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

Notable changes:none

BUG=651816
TEST=none

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

If-statement prevents a crash in a unittest that would occur if rolling
after the WebRTC change.

The WebRTC change: https://codereview.webrtc.org/2387343002/

BUG=chromium:627816

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

1. Update to match new signature of WebContentsCreated.

2. Allow IO in the newly added renderer command prefix test.

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

Revert of Prune createVisibleSelectionDeprecated from SelectionAdjuster (patchset #1 id:1 of https://codereview.chromium.org/2389653002/ )

Reason for revert:
Suspected of causing crbug.com/652301

Original issue's description:
> Prune createVisibleSelectionDeprecated from SelectionAdjuster
>
> BUG=651373
>
> Committed: https://crrev.com/1301c7b47926fa0d435c2f396a6aeba2a4d179c1
> Cr-Commit-Position: refs/heads/master@{#422380}

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

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

BUG=563793

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

These benchmarks were failing for the same underlying reason as
smoothness.tough_webgl_ad_cases, which was fixed by
https://codereview.chromium.org/2373983005.

BUG=647413
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

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

This changes the way the resource score is computed in the
resource_prefetch_predictor. It takes into account the request priority
as the most important feature, then resource type, then average position.

The rationale is that resource priority surfaces the render-blocking
resources, that are the most critical to prefetch.

BUG=631966

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

BUG=651431

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

This does not change the generated code at all.

BUG=460722

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

WebPermissionClient.h was removed in
crrev.com/9f21f26b

Removing some remnants

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

Notable changes:none

BUG=651816
TEST=none

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/eb2b7fdc..6b7a634b

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/2389833003
Cr-Commit-Position: refs/heads/master@{#422748}

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/fee67c6802805c049ddf19e436fb00ad8416032f Revert of Disable local fallback in goma staging environment (patchset #2 id:20001 of https://codereview.chromium.org/2392853002/ ) (tikuta@chromium.org)

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

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

Before this CL the http/tests/webaudio/autoplay-crossorigin.html layout
test would fail if the main frame and the cross-origin frame are in
different renderer processes.  After this CL, the internal settings are
set from the same frame that hosts AudioContext object - this ensures
that setting the settings and getting the settings happens from the same
renderer process (and consequently ensures that the test passes both
with and without --site-per-process flag).

BUG=652366

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

e2df1d70 adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
CQ_INCLUDE_TRYBOTS=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

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

A permission descriptor is a structure describing a permission that an
origin may request. Most permissions are only identified by their names
but some have additional parameters. For example, a Bluetooth device
permission may include the MAC address of the device (or other unique
identifier) so that the permission applies to only the device selected
by the user through some sort of chooser interface.

BUG=638721

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