GitLab

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/ced9010..28f9c60

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=halcanary@google.com

Review URL: https://codereview.chromium.org/782063002

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

Add an observer list to HidService that is notified when a device is
added or removed from the enumeration. This currently works as expected
on Linux and OS X while on Windows devices are only enumerated on request
and so these are not as useful.

BUG=376719

Review URL: https://codereview.chromium.org/779353003

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

This change adds more debug logging to help diagnose issues occurring
during completing pending reads of ChunkDemuxerStream. It also includes
more logs in SourceBufferStream::GetNextBuffer().
These logs are useful for debugging issues like the referenced bug.

R=dalecurtis@chromium.org
BUG=423858

Review URL: https://codereview.chromium.org/761963006

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

BUG=

Review URL: https://codereview.chromium.org/757233006

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

BUG=chromedriver:980

Review URL: https://codereview.chromium.org/783593003

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

In Pepper, glBind*() may generate resource in GPU process, so we have to
flush command buffer to make sure the Bind* command is executed by GPU before
any future Delete* commands.

BUG=437432

Review URL: https://codereview.chromium.org/763383002

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

Move it to PasswordManagerClient, and make it return the AutofillManager for the main frame.

The bug occurred because the PasswordFormManager outlived the driver. The PasswordFormManager can't outlive the main frame.

BUG=438970

Review URL: https://codereview.chromium.org/780183002

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

Specifically, this addresses some instances of C4244 ( http://msdn.microsoft.com/en-us/library/th7a07tz.aspx ).

Also does a few misc. cleanups, mostly to test code.

Note: This CL includes server-side change 81444427.

BUG=81439
TEST=none

Review URL: https://codereview.chromium.org/759063003

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

Revert of bluetooth: Make device/bluetooth a shared library component. (patchset #4 id:100001 of https://codereview.chromium.org/778443002/)

Reason for revert:
Chrome target is failing to load due to BluetoothApis.dll not being found on Win7. Likely the DelayLoadDLLs for BluetoothApis.dll is necessary on more than just device_bluetooth.

Original issue's description:
> bluetooth: Make device/bluetooth a shared library component.
>
> device/bluetooth will soon be accessed by content/browser. This
> change converts it from a static library to a shared component.
>
> BUG=438305
>
> Committed: https://crrev.com/ff6eaf5adf4d6dc5a748e2ee4e8de080655d1975
> Cr-Commit-Position: refs/heads/master@{#307049}

TBR=armansito@chromium.org,isherman@chromium.org,benchan@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=438305

Review URL: https://codereview.chromium.org/748873004

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

BUG=439522

Review URL: https://codereview.chromium.org/784563002

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

Include explicit compositor-thread UMA latency coverage for scroll
begin, as well as pinch and fling-related events. Also add an FPS metric
for both main and compositor-thread driven flings.

BUG=437310,436965

Review URL: https://codereview.chromium.org/764403002

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/3ab7158..f02068c

TBR=schenney@chromium.org,dominicc@chromium.org

Review URL: https://codereview.chromium.org/778253002

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

This logs the passphrase type on sync startup if encryption is enabled.

BUG=421623

Review URL: https://codereview.chromium.org/779093002

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

BUG=433988
R=stevenjb@chromium.org,asvitkine@chromium.org

Review URL: https://codereview.chromium.org/747883002

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/69469b5..ced9010

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=halcanary@google.com

Review URL: https://codereview.chromium.org/750983004

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

BUG=438545

Review URL: https://codereview.chromium.org/752003006

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

Support checking RsnFlags as well as WpaFlags so that security type is reported correctly for all networks.

BUG=387869

Review URL: https://codereview.chromium.org/779053002

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

This pulls in the following Native Client changes:

r14148: (dschuff) PNaCl: Update LLVM/clang/binutils-x86 revision in pnacl/COMPONENT_REVISIONS
r14149: (hamaji) Update revision for PNaCl r14144->r14148
r14150: (petarj) Fix issue in DecodeEscaping() in debug_stub test
r14151: (mseaborn) Remove much of the unused parts of pnacl/build.sh
r14152: (bradnelson) Nest the pnacl section of untrusted.gypi to the same level.
r14153: (dyen) The cached results from toolchain_build should be deterministic.
r14154: (jvoung) Add options for base dir and redirecting files to IRT open resource and use it.
r14155: (dschuff) nacl-clang: use gas by default instead of the LLVM assembler
r14156: (dyen) Toolchain testing within the NaCl tree now use the raw variants.
r14157: (teravest) Clean up unused NaClDesc quota code.
r14158: (dschuff) Revert SCons portion of r14155
r14159: (teravest) Remove unnecessary PostMessage logic.
r14160: (mseaborn) Cleanup: Remove TLS_PRESUMED_ALIGNMENT from tls.c
r14161: (mseaborn) Cleanup: Use Bit('build_ARCH') instead of Bit('target_ARCH')
r14162: (dschuff) Update revision for PNaCl r14148->r14155
r14163: (ncbray) GN: add a base target for the NaCl build.
r14164: (dschuff) Switch nacl-clang back to gas in SCons
r14165: (dschuff) PNaCl: Update LLVM/binutils-x86 revision in pnacl/COMPONENT_REVISIONS
r14166: (dyen) Made buildbot PNaCl toolchain test extract using packages script.
r14167: (dyen) Fixed PNaCl buildbot test by adding extract argument to command.
r14168: (dschuff) Update revision for PNaCl r14155->r14165
r14169: (jvoung) Remove __X_LEAST{8,16,32}_TYPE__ hack for appeasing GCC torture tests w/ Clang.
r14170: (dschuff) Re-enable exception_tests for nacl_clang on mac
r14171: (mcgrathr) Update binutils revision in tools/REVISIONS
r14173: (jfb) Use static split module scheduling when using pnacl-translate.py
r14174: (shyamsundarr) cleanup scons test rules for validator proofs a bit.
r14175: (jfb) Update revision for PNaCl r14165->r14173
r14176: (bradnelson) Revert "Nest the pnacl section of untrusted.gypi to the same level."
r14177: (mseaborn) Disable the getdents() syscall by default for security hardening
r14178: (dschuff) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk

Review URL: https://codereview.chromium.org/781103002

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

glInvalidateFramebuffer, glInvalidateSubFramebuffer, glReadBuffer.

All other commands in this group have already been added.

Also, code generator has to be modified to remove the incorrect assumption
that the data pointer is the last arg of a function.  This assumption is not
true for glInvalidateSubFramebuffer.

Also, separate the bindings of glDiscardFramebufferEXT and glInvalidateFramebuffer so we have better control of them.

BUG=429053
TEST=gpu_unittests
R=kbr@chromium.org,bajones@chromium.org,sievers@chromium.org

Review URL: https://codereview.chromium.org/780433006

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

746d0ea..8e7c8a
https://chromium.googlesource.com/angle/angle/+log/746d0ea..8e7c8a

BUG=434033

Review URL: https://codereview.chromium.org/785543003

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

This CL:
* Moves network_event_log.cc -> device_event_log_impl.cc
* Defines macros and initialization code in device_event_log.h
* Modifies network_event_log.h to use device_event_log instead.
* Converts PowerManagerClient logging to use POWER_LOG as an immediately useful example.
* Adds the non-network log entries to a 'device_log' entry in chrome://system and feedback reports.

BUG=293739

Review URL: https://codereview.chromium.org/773703002

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

Non-sequential sparse data writes may end up writing before or after a given
sparse data block. Said data is only stored if the next write is sequential,
and it actually fills a given block.

This CL makes sure that data "before" the start of a given block is properly
discarded (as in, not accounted for when looking for stored data).

BUG=416895

Review URL: https://codereview.chromium.org/770153002

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

BUG=439240

Review URL: https://codereview.chromium.org/780923003

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

BUG=

Review URL: https://codereview.chromium.org/764733005

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

BUG=439526

Review URL: https://codereview.chromium.org/780353002

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

DriSurfaceFactory no longer does any meaningful initialization. Remove
redundant code.

BUG=none
NOTRY=true

Review URL: https://codereview.chromium.org/758853005

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

We should not attempt to create a GLHelper with a null
GLES2Impl (it will crash), which is the case if context
init failed.

BUG=439363
NOTRY=True

Review URL: https://codereview.chromium.org/786473002

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/01aebe5..3ab7158

TBR=schenney@chromium.org,dominicc@chromium.org

Review URL: https://codereview.chromium.org/785613002

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

1) Remove ChromecastConfig. Use PrefService directly.
2) Initialize PrefService after thread creation because it uses blocking IO
   browser thread instead of its own thread pool.
3) Use scoped_ptr for explicit ownership.

BUG=

Review URL: https://codereview.chromium.org/767503003

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

Revert of Generalize V4L2 HW video codec device names (patchset #1 id:1 of https://codereview.chromium.org/730693005/)

Reason for revert:
This change breaks video_ChromeHWDecodeUsed, please see crbug.com/439175 for details.

Original issue's description:
> Generalize V4L2 HW video codec device names
>
> Switch to more generalized V4L2 video accelerator device names and update
> sandbox rules to reflect that.
>
> This is another attempt at crrev.com/763743002, reverted in
> crrev.com/767383002 with a fixup for sandbox permissions.
>
> BUG=405861,437598
> TEST=compile
>
> Committed: https://crrev.com/30a3721c82953428d445a93ebe2bf0014e27bc10
> Cr-Commit-Position: refs/heads/master@{#306642}

TBR=jorgelo@chromium.org,scherkus@chromium.org,posciak@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=405861,437598

Review URL: https://codereview.chromium.org/784553004

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

The MediaImportHandler is the overall framework class for coordinating the
import of media from attached media devices to Drive.  This initial
implementation sketches out the process, using FileOperationManager to provide
the actual copy functionality.
- UI and checks for opt-in to Drive->Photos sync and Photos->Drive sync are just
  stubbed out as TODOs.
- The use of FileOperationManager means that the import process is surfaced as a
  normal copy operaion, which is not the final goal.  Follow-up work is needed
  to implement a separate import queue and associated task management and UI.

BUG=420680

Review URL: https://codereview.chromium.org/762593006

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

When gpu context is lost, CopyTextureCHROMIUMResourceManager can be deleted
without releasing resources like GLES2DecoderImpl.

BUG=438950

Review URL: https://codereview.chromium.org/780293002

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

Better error messages on archive failures, and stop running with partial page sets in user_story_runner.

BUG=435063

Review URL: https://codereview.chromium.org/748013003

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/22c9ae1..69469b5

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=halcanary@google.com

Review URL: https://codereview.chromium.org/772363003

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

Waterfall is running on chartjson now, but we've got hacks in the recipes to swap the command lines from buildbot->chartjson.

ie. instead of:

{
  "steps": {
    "blink_perf.animation": {
      "cmd": "/path/to/run_benchmark -v --output-format=buildbot the_test",
      "device_affinity": -1,
      "perf_dashboard_id": "foo"
    },
}

->

{
  "steps": {
    "blink_perf.animation": {
      "cmd": "/path/to/run_benchmark -v --output-format=chartjson --upload-results the_test",
      "device_affinity": -1,
      "perf_dashboard_id": "foo"
    },
}

BUG=422173, 422174

Review URL: https://codereview.chromium.org/779223004

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

The loop that adds prefixes from 1 billion to 3 billion has been broken
since it was modified in Feb 27, 2014, change 2b59b0a6. It didn't survive
the change from signed to unsigned integers meaning that this test
function has effectively been disabled.

This bug was found by VC++'s /analyze which said:
src\chrome\browser\safe_browsing\prefix_set_unittest.cc(280) :
warning C6294: Ill-defined for-loop:
initial condition does not satisfy test.  Loop body not executed.

BUG=427616

Review URL: https://codereview.chromium.org/779163002

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

BUG=

Review URL: https://codereview.chromium.org/780223005

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

This adds more spam in release builds, but will be valuable in
diagnosing bugs.

BUG=435786

Review URL: https://codereview.chromium.org/783713002

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

schemas.  They're unnecessary (controlled by the presence, or lack thereof, of
an HTML file documenting the API) and I keep on seeing code reviews of
cargo-culted APIs that add it.

R=rockot@chromium.org

Review URL: https://codereview.chromium.org/754693004

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

Use the unversioned symlink to ensure we always load instrumented udev.

BUG=437464, 438427
R=thestig@chromium.org, reillyg@chromium.org

Review URL: https://codereview.chromium.org/755873003

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