GitLab

With this we can see how often different workarounds are being hit.

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

https://chromium.googlesource.com/angle/angle.git/+log/677bb6f..aa7203e

BUG=

TEST=bots

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

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

Whitelisted Linux apps can access chrome.bluetooth. Initialize Bluetooth so
we don't crash on those apps.

BUG=711142
R=rkc@chromium.org,rdevlin.cronin@chromium.org

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

Pure code movement.

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

This CL was generated mechanically with:

$ git grep -l m_callback -- third_party/WebKit/Source | \
      xargs -n 1 sed -i -e 's/\bm_callback\b/callback_/g'

No intended behavior change.

BUG=675877

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

Map ShelfID to AppLaunchId and use that instead of uint32_t.
ShelfModel no longer assigns int ids, clients supply unique strings.

ShelfID/ShelfItem/ShelfModel:
-Add comparison operators and IsNull() helper function.
-Add a corresponding ShelfID mojo struct with traits and test.
-Allow empty app_id strings (iff the launch_id string is also empty).
-Nix ShelfItem's AppLaunchId, use its ShelfID |id| instead.
-DCHECK unique non-empty ShelfIDs (avoid odd ShelfModel behavior).

Properties:
-Remove aura::client::kAppIdKey, use ash::kShelfIdKey instead.
-Move kShelfIDKey to ash/public/cpp to approve Chrome usage.
-Register kShelfIdKey for mus/mash conversion and mirroring.
-Require a kShelfIDKey property for ShelfWindowWatcher item creation.

Misc/Chrome:
-Set non-empty ShelfIDs for AppList, Task Manager, Settings.
-Apply ExtensionWindow* ShelfIDs for panel windows too.
-Use ShelfID in ExtensionAppWindowLauncherController maps.

TODO: Rename AppLaunchId to ShelfID (can do in this CL if desired)
TODO: Remove ShelfModel id conversion functions; audit users.
(also used to check item presence in ShelfModel, map ARC ids...)

OPTIONAL: Keep kShelfIDKey in ash/wm/window_properties for now?
OPTIONAL: CHECK in ShelfModel for prod/arc? Loosen checks for tests?

BUG=557406
TEST=No Chrome OS shelf or app regressions.
R=jamescook@chromium.org,sky@chromium.org,tsepez@chromium.org,reveman@chromium.org

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

https://pdfium.googlesource.com/pdfium.git/+log/b8a8c43f3942..e8c1d4144e84

$ git log b8a8c43f3..e8c1d4144 --date=short --no-merges --format='%ad %ae %s'
2017-05-04 thestig Give a couple of char to int functions better names.
2017-05-04 thestig Reduce 1D barcode DrawPath calls.

Created with:
  roll-dep src/third_party/pdfium


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I0f31d35b9bebe501f81431b5bb0a94e9c91c6509
Reviewed-on: https://chromium-review.googlesource.com/497006
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#469536}

Bug=712331

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

Vulcanize removes HTML imports of global style definitions and inlines
those <style> elements directly in a special <div> on the page. Removing
this <div> in tests means those <style>s go away. Without those styles,
elements being tested no longer have access to the CSS variables or
global style rules they expect, which can break things in all sorts
of ways.

This hasn't affected any tests that I know of (besides one WIP), because
we don't normally test the details of our colors and layouts in tests that
call PolymerTest.clearBody. But it's worth fixing regardless in order to
decrease the delta between tests in vulcanized vs. unvulcanized builds.

BUG=714359
R=dpapad@chromium.org

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

Makes it behave the same as the other iron-lists on the page.

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

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

In https://codereview.chromium.org/2835733002 the ReadHistogramValues
method now returns one value instead of 2, but its call in
UpdateHistogramEnum still expect 2 where the 2nd was ignored.

BUG=711205

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

The gaps have shown to move around, and so the blamed file is likely not
really at fault.

BUG=681694

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

So that we don't need to use global variables.

BUG=718614

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

TEST=unit tests extented. Manually, updated test app via adb and
     its shorcuts were left in app launcher
BUG=718625
BUG=b/34749664

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

There are no flags to be updated

deprecate FLAGS_quic_reloadable_flag_quic_bbr_keep_sending_at_recent_rate in disabled state.

It achieved results that were in general comparable to "measuring
aggregation" approach, but it had worse rtx rate, and worse QoE, and did
not actually seem to be better in other regards.

Merge internal change: 154601526

https://codereview.chromium.org/2859913003/

fix packet conservation to remove packets from recovery window when they are lost.  Protected by FLAGS_quic_reloadable_flag_quic_bbr_fix_conservation2.

This causes a dramatic reduction in losses compared to the previous patch.

The transfer time is approximately the same.

Merge internal change: 154590929

https://codereview.chromium.org/2857243002/

Fix typo in comments.

Merge internal change: 154539040

https://codereview.chromium.org/2854823004/

QUIC - refactor stream creation. Guarded by --quic_reloadable_flag_quic_refactor_stream_creation.

This CL is in preparation for HTTP stream pairs changes.  A prototype
of the entire HTTP stream pairs work is in progress.  This CL is one
subset of that (2nd in the series).

[ HTTP stream pairs builds upon this, by having new QuicSpdySession
overrides that implicitly create headers streams, and abstracting
stream accounting as needed. ]

Add a new factory like method QuicSession::CreateStream(), that
unconditionally creates a new stream.  It's virtual so subclasses of
QuicSession can create streams that are appropriately specialized from
QuicStream.  There are new versions of Create*DynamicStream() called
MaybeCreate*DynamicStream(), and ShouldCreate*DynamicStream() called
ShouldCreate*DynamicStream2().

Separating this out means that fewer overrides of Create*DynamicStream()
and ShouldCreate*DynamicStream() will necessary, because many of those
were actually only needed to specialize the steam types.  Those will go
away when FLAGS_quic_reloadable_flag_quic_refactor_stream_creation is
deprecated.

Merge internal changes: 154351257, 154373998.

https://codereview.chromium.org/2861673004/

QUIC - stream id refactor for tests.

// AKA - http stream pairs bikeshed #1.

This CL is in preparation for HTTP stream pairs changes.  A prototype
of the entire HTTP stream pairs work is in progress.  This CL is one
subset of that.

Rework QUIC tests so that they do not hard code the assumption that
all headers are on stream 3.  Also, prepare to calculate stream ids
according to whether stream pairs are enabled, i.e. whether they
increment by 2 or 4.

Merge internal change: 150949425

https://codereview.chromium.org/2854833005/

R=rch@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2862563003
Cr-Original-Commit-Position: refs/heads/master@{#469218}
Committed: https://chromium.googlesource.com/chromium/src/+/d60018e0b72b5708797cebe57780ad559877164f
Review-Url: https://codereview.chromium.org/2862563003
Cr-Commit-Position: refs/heads/master@{#469528}

This CL separates the site settings browser tests.

(Code health)
BUG=None

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

This simplifies local testing of metrics collection, particularly in dev builds.

BUG=717792

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

BUG=718610

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

Single sample metrics offer a mechanism for clients to modify
a histogram sample repeatedly while having it only reported once;
either upon destruction of a helper class or at process termination.

This is helpful since the fast shutdown path simply kills renderer
processes without going through any destructors. In this case we
lose histogram values that might otherwise be reported at that
time.

In media/ code we've created a cumbersome proxy mechanism which
sends histogram values like this to the browser process and records
them there (see WatchTimeReporter and MediaInternals interactions).

Single sample histograms are implemented through a new mojo service
hosted by the browser process that receives samples and logs the
last received sample upon mojo channel closure.

base:: shims are provided so that these metrics can be created by
anyone in the renderer process. For non-renderer processes a default
implementation is provided without the mojo channel behavior.

Usage looks like the following:
  std::unique_ptr<base::SingleSampleMetric> metric(
      base::SingleSampleMetricsFactory::Get()->CreateCustomCountsMetric(
          "Media.VideoRenderer.CadenceChanges", 1, 10, 10));
  metric->SetSample(1);
  metric->SetSample(2);
  metric->SetSample(3);
  metric.reset();

Only the last sample (3) would end up being reported to the histogram.

BUG=689751
TEST=new tests

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

In https://codereview.chromium.org/2665913002 we set some default
network properties in currentRouteChanged. This was triggering
networkPropertiesChanged_ which udates autoConnect_ and
preferNetwork_ which triggers calls to setNetworkProperties.

To prevent this sort of error, set networkPropertiesReceived_
once properties are received from Chrome and do not call
networkingPrivate.setProperties until that has been set.

BUG=718389

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

This is a prerequisite of whitelisting //third_party/WebKit/public
for header checking in GN.

BUG=716359

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

This CL changes the logic for choosing the loud vs. quiet interstitial.
Instead of comparing against screen dimensions, we instead use window
dimensions, which makes more sense when Android Multi-Window is enabled.

BUG=718195

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

Manually convert tests to use new Audit

BUG=704967
TEST=
Panner/panner-automation-basic.html
Panner/panner-automation-position.html
Panner/panner-distance-clamping.html
Panner/panner-rolloff-clamping.html
Panner/pannernode-basic.html

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

The actual filenames are all lowercase, but src attributes in
execorder.html contains uppercase letters. This causes the scripts not
found and thus timeouts on Linux (which runs on a case-sensitive
filesystem).

This CL fixes the timeout by making the src attributes all lowercased.

BUG=594639

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

This CL removes a hack that adds /usr/lib/gcc/x86_64-linux-gnu/4.6 to
ld.so.conf.d/zz_hack.conf.  The only sysroots that exist currently are
Jessie and Trusty which both use gcc 4.8.

This caused the following flags to be passed to the linker:
-L/path/to/chromium/src/build/linux/debian_jessie_amd64-sysroot/usr/lib/gcc/x86_64-linux-gnu/4.6
-Wl,-rpath-link=/path/to/chromium/src/build/linux/debian_jessie_amd64-sysroot/usr/lib/gcc/x86_64-linux-gnu/4.6
While harmless, it is confusing and should be removed.

R=thestig@chromium.org

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

Not sure why/how this happens, but I was seeing this infrequently on  https://with.in/watch/under-neon-lights/

BUG=718501

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

And improve information provided by out-of-order destruction of task
scheduler and its provided constructs in unit tests.

(comes out of discussion on https://codereview.chromium.org/2852333004/)

BUG=708584, 689520
TBR=avi@chromium.org (documentation update in test_browser_thread_bundle.h)

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

ASAN wraps the kernel dylib on mac. Special case this to avoid tripping
a DCHECK.

BUG=718628
TBR=avi@chromium.org

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

Multi-display support in ARC was punted to the next milestone. This CL
temporarily prevents windows from being dragged to secondary displays
by using display rather than screen coordinates for window bounds.

BUG=714832
TEST=ARC apps cannot be dragged to secondary displays.

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

This is currently a blacklist entry, which means that it's calculated in
the browser and the resulting decision could be incorrect if the GPU
process uses a different GPU from what the browser expects. If this is
a GPU driver bug workaround entry then it'll be determined with more
information in the GPU process.

BUG=687004
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

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

The required texture formats (GL_R16F, etc) are not in the core until
3.0.

We could, in principle, query the particular extensions required for
these formats (GL_ARB_texture_rg, etc), but this query is simpler and
equally effective in practice.

BUG=699278
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

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

https://skia.googlesource.com/skia.git/+log/18f9602094ac..783c43658d91

$ git log 18f960209..783c43658 --date=short --no-merges --format='%ad %ae %s'
2017-05-04 egdaniel Fix comma in vulkan

Created with:
  roll-dep src/third_party/skia


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=scroggo@chromium.org

Change-Id: I12cb923ca36afc50220d839b0c13483ad2f616a1
Reviewed-on: https://chromium-review.googlesource.com/496887Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#469511}

BUG=706802

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

Now that we use the PaintOpBuffer, analyze whether a DisplayItemList
has any discardable images to skip metadata generation if unnecessary.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

Make the order of setting parameters in ConstructSystemRequestContext
exactly match the order in URLRequestContextBuilder.  Also add a few
(trivial) setters to URLRequestContextBuilder for things that
ConstructSystemRequestContext sets that URLRequestContextBuilder does
not support.

This is part of a multi-stage operation to use URLRequestContextBuilder
to create the system URLRequestContext:

1)  Defrag (This CL).
2)  Switch to having a URLRequestContextStorage own everything, and create
everything inline with system URLRequestContext configuration.
3)  Switch the proxy service to using the system URLRequestContext.
4)  Switch to using URLRequestContextBuilder, but having IOThread create
  everything (This could be combined with 2, but 2 will affect a lot of
  accessors, so will probably to do it incrementally)
5)  Switch to having URLRequestContextBuilder create everything it
  reasonably can.

Hopefully the incremental approach will help to protect against regressions.

BUG=715695

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

R=kpaulhamus@chromium.org

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

BUG=647036
TBR=yzshen,dpranke,jbroman

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

Screenshot in bug.

R=dschuyler@chromium.org
BUG=718210

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

https://pdfium.googlesource.com/pdfium.git/+log/d41f4752c81d..b8a8c43f3942

$ git log d41f4752c..b8a8c43f3 --date=short --no-merges --format='%ad %ae %s'
2017-05-04 thestig CPDF_Document::GetPageData() normally does not return NULL.
2017-05-04 thestig Make CPDF_Font member variables protected.

Created with:
  roll-dep src/third_party/pdfium


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


TBR=dsinclair@chromium.org

Change-Id: I62224ef89307f373da6699d28dad3df96e6e658d
Reviewed-on: https://chromium-review.googlesource.com/496926
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#469504}

CPU frequency file exists in [2] for a majority of devices. It can
also exist in [1] for certain devices like chell, cave, caroline,
veyron_speedy, etc. Add a parser to read data from [1] which has
different format from [2].

[1] /sys/devices/system/cpu/cpufreq/all_time_in_state
[2] /sys/devices/system/cpu/cpu*/cpufreq/stats/time_in_state

BUG=548510

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