GitLab

Currently, there are two important API calls for OffscreenCanvas, which
are transferToImageBitmap and commit. This CL adds usage counter to these
two API calls.

BUG=653599
TBR=kbr@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel

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

BUG=656121, 652345

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

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

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

This patch sets the font family for the "system-ui" generic font family
added in [1] on Linux.

With this change, all platforms use the correct font. On Windows, the
browser already pass menu_font_family_name to the renderer. On Mac and
Android, the render can find the system font without needing the
browser to set.

On Linux and Chrome OS, gfx::Font() calls LinuxFontDelegate to figure
out the current system font. On the default Ubuntu, this resolves to
"sans", which Skia resolves to "DejaVu Sans", or "Arial" in
run-layout-test.

[1] http://crrev.com/2137483004

BUG=654679

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

Adds mobile responsive device list to bluetooth internals page.
Lists device name, address, and latest RSSI.
Refreshes when devices are discovered, removed, or advertising packets are received.

Screenshots
Desktop: https://goo.gl/photos/bot7JsdTuf8P2A328
Mobile: https://goo.gl/photos/G9bb9s4ih5KT3PJw8

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

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

These one were placed incorrectly because of duration is NaN.
drive-by: get rid of highlightTimeRange function.

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

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

BUG=angleproject:1555
TEST=webgl2_conformance on windows
TBR=jmadill@chromium.org
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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

Please see https://crbug.com/658759 for more context.

The TL;DR is I broke some subtle behavior in
https://crrev.com/425358, and this fixes it. Here's what happens
in ProcessDied, in this order:

  1. We reset channel_ so that future Send() calls throw away
     messages. Note that at this point, any Channel-associated
     interfaces remain valid but are intentionally dropping
     all messages as well.

  2. We fire various notifications about the death of the process.

  3. We re-initialize channel_, allowing subsequent Send() calls
     and associated interface calls to be queued and eventually
     delivered to the new process once it's restarted.

The subtlety here is that if a RenderViewHostImpl is created at any
point during step (2) above, we must re-initialize channel_ and
start allowing messages to be queued on it. Formerly this was
accomplished by EnableSendQueue() setting is_initialized_ to
false. This CL establishes the same behavior by restoring
EnableSendQueue() to essentially have the same effect by re-
initializing the channel earlier.

This is likely not the precise behavior we want, since it seems like
this can cause messages intended for the old process to end up getting
delivered to the new process, but it is the behavior we already had in
place before https://crrev.com/425358 and we apparently rely on it.

BUG=658759

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

https://chromium.googlesource.com/skia.git/+log/3d3a65c48816..5934646b4937

$ git log 3d3a65c48..5934646b4 --date=short --no-merges --format='%ad %ae %s'
2016-10-24 msarett Help msvs compile SkColorSpaceXform faster
2016-10-24 scroggo Move third_party/gif's license into its own file GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2448573002
2016-10-24 scroggo Differentiate animated image results
2016-10-24 brianosman Populate ALL copies of a command line flag during parsing

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_precise_blink_rel
TBR=reed@google.com

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

This patch simply cleansup some DataGrid a little.

R=dgozman,lushnikov
BUG=None

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

In order for the browser to correctly reason about suborigins, this adds
support to url::Origin to parse and understand suborigins. It separates
requests for hosts and schemes from the embedded suborigin
serialization, while keeping the same-origin policy checks intact so a
suborigin is a different origin from other suborigins at the same
physical origin.

This updates url/origin.* so that given a suborigin encoded in a GURL,
will correctly deserialize the suborigin and store it accordingly, while
the scheme/host/port tuple is left with the real scheme/host/port.
Additionally, removes the content/public/common/origin_util.h functions
for using suborigins since url::Origin should now be used in their
stead.

BUG=336894,649893

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

Currently the tests are all TEST rather than TEST_F.

Add a test harness (TEST_F) to reduce some boilerplate and encapsulate
some subtle setup code required for overlay scrollers on Mac.

Part of this is to do with ScopedPreferredScrollerStyle: Interleaving
swizzlers like `std::unique_ptr x(new X1); x.reset(new X2);` causes
incorrect behaviour and leakage of the swizzled methods across tests run
in the same process. Specifically, X1's destructor restores the original
methods, but then X2's destructor will restore X1's swizzled methods,
since they were in effect when X2 was constructed. Make it harder for
devs to shoot themselves in the foot with this.

BUG=615948

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

https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/b973c09..fa214ea

BUG=654134

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

MemoryDumpManagerTest didn't join its thread. This made a write in
TearDown() racy with a read from a thread created by the test.

With this CL, threads created by MemoryDumpManagerTest are joined
before TearDown() runs.

BUG=658766

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

chrome/browser/chromeos/arc directory has become very cluttered. In
preparation for a big refactor, let's create intermediate directories
for most of the files. The ones that are still in the top-level
directory will undergo significant change, so it doesn't make sense to
move them just yet since all their callsites will also need to be
changed soon.

BUG=657687
TEST=git cl try

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

Previously, we made all cr-dialogs close on popstate. This is good,
except for navigable dialogs.

For navigable dialogs: It should open if the new route is the dialog's
route, and close otherwise. All the navigable dialogs already have code
that implements this behavior, but it conflicts with the popstate
event handler on cr-dialog.

This CL adds a parameter that disables this behavior for navigable
dialogs.

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

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

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

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

It's believed that the flaky GPU process hangs seen on the waterfall
are spurious, not real. Try disabling the GPU watchdog. If the hangs
are real, this will result in test timeouts rather than GPU process
crashes, context loss, and test failures.

BUG=596622, 609252
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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

BUG=538914
NOTRY=true
TBR=nhiroki@chromium.org

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

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/d1882533e19559efc7859154daca12184b1f4db3 Remove unused recipe: dartium_android_builder.py (tikuta@chromium.org)

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

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

Revert of Switching ExtensionSpecialStoragePolicy::IsStorageDurable to use DurableStoragePermissionContext. (patchset #11 id:200001 of https://codereview.chromium.org/2351803002/ )

Reason for revert:
Caused issue 656495.

Original issue's description:
> Switching IsStorageDurable() to use DurableStoragePermissionContext.
>
> ExtensionSpecialStoragePolicy was using cookies_settings to figure out if the
> given origin was allowed to use durable storage. In order to make this work, the
> cookies_settings had to use a lower level api (HostContentSettingsMap) to avoid
> layer violations.
>
> We found that we could remove the cookies_settings::IsStorageDurable() and,
> instead, just have the ExtensionSpecialStoragePolicy access the
> PermissionManager.
>
> BUG=539538
> R=raymes, jochen, pfeldman
>
> Committed: https://crrev.com/4a19a66d254415f7afb67d6f3c6e6354f280109d
> Cr-Commit-Position: refs/heads/master@{#424165}

TBR=jochen@chromium.org,dgrogan@chromium.org,msramek@chromium.org,pfeldman@chromium.org,raymes@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=539538

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

- removes obsolete key configuration code from options; this leaves options as a simple, clean list of documentable settings
- change the tutorial url to point to the Next tutorial

Pending/Optional
- change the update notification to point to a release note (rather than the tutorial)

TBR=dtseng@chromium.org
BUG=618097
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

BUG=654773

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

BUG=649279

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

When there are no network requests recorded, prompt the user to reload,
instead of making them click over to the network panel where they are
prompted to reload.

BUG=648646

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

VP9 driver of elm reported wrong visible size and it was fixed.
Update the md5 of test-25fps.vp9 to reflect the change.

BUG=b:32354993
TEST=Run VDA unittest on elm.

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

This refactors LaunchProcessWithToken to take an explicit list of
handles for the child to inherit (copied from the equivalent process
launch code in //base).

This also changes WorkerProcessLauncherTest to use ChannelMojo.

BUG=604282

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

s/texture_ids/client_texture_ids and s/internal_texture_ids/service_texture_ids/ because
these names are clearer and more consistent with the rest of the gpu code. This also
cleans up GpuVideoDecodeAccelerator and the VDAs, which used internal_texure_ids to refer
to the client ids, and texture_ids to refer to the service ids.

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

BUG=585301

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

BUG=148757,webrtc:6488

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

This CL includes security tests for empty image and undecodable image.

BUG=646083
TEST=third_party/WebKit/LayoutTests/fast/shapedetection/shapedetection-security-test.html

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

The inplace editor's width should account for the left indent.

BUG=657297
R=dgozman

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

on all Android versions before 4.4 (KitKat).

R=aelias@chromium.org
R=zmo@chromium.org
BUG=657925
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/2439253002
Cr-Commit-Position: refs/heads/master@{#427232}

https://pdfium.googlesource.com/pdfium.git/+log/ad2441ef6758..37b203059210

$ git log ad2441ef6..37b203059 --date=short --no-merges --format='%ad %ae %s'
2016-10-24 weili Add ASan bots in commit queue

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

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

BUG=webrtc:6462

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

Revert of gpu: Clarify sized texture format is available only if ES3 context or immutable texture is supported (patchset #7 id:160001 of https://codereview.chromium.org/2117183006/ )

Reason for revert:
Broke playing back recorded data as detailed
in https://crbug.com/657532.  The bug has
step-by-step notes on how to repro (see also #4
in there).

Locally reverting of this CL restores the functionality.

Original issue's description:
> gpu: Clarify the sized texture format is available only if ES3 context or immutable texture is supported.
>
> This clarification makes it possible that gles2 decoder tracks true internal format of immutable texture. So this CL removes internal_format hack in GLES2DecoderImpl::TexStorageImpl()
>
> In addition, it makes CMAA skip redundant copy path because CMAA can know true internal format.
>
> BUG=535198
> 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
>
> Committed: https://crrev.com/730f8089f791fa31a3ee2a7f5059a1052b129d21
> Cr-Commit-Position: refs/heads/master@{#425684}

TBR=dongseong.hwang@chromium.org,zmo@chromium.org,kbr@chromium.org,dongseong.hwang@intel.com
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=535198

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

https://chromium.googlesource.com/angle/angle.git/+log/f017315..3b5ffe5

BUG=none,None,512090

TBR=zmo@chromium.org

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

This is useful in debugging problems.

BUG=652409
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/2440853002
Cr-Commit-Position: refs/heads/master@{#427227}

BUG=618089

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