GitLab

This is hitting the same bug as ExtensionApiTest.ContentScriptExtensionProcess
(and a few others) where the iframe in the extension page never fully loads in
the test.

TBR=jam@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/13dcae4..21216be

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/200a784..13dcae4

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

BUG=520456

TEST=
1. Launch chrome, Sign in to chrome and  add/import a supervised user.
2. In the signed in user, click on avatar icon in top R.H.S and select 'Exit and childlock' option (Switch person window opens).
3. In Switch person window, double click on locked user (Sign in overlay opens).
4. Click on close button of Sign in overlay and observe.
5. The browser should not crash.

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

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

Includes Some minor fixes to the JS

BUG=none

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

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

Request focus for re-auth webview after re-auth dialog is showing.

BUG=520046.

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

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

R=garykac@chromium.org
BUG=471916

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

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

  In order to roll GN a6029a4c..4c8f3e38 (r341627:r348227) and pick up
  the following changes:

  a88eb18a Throw errors for nested targets in GN.
  d9838e11 Make gyp_flag_compare.py work on Windows
  63a4d153 Fix sbox_validation_tests & Make isolate_driver.py include the executable itself
  789ee894 gn: fix documentation typo
  3473904f Update reference doc page for GN.
  d2cb7552 GN: Fix bootstrap script on Mac.
  7c8964f3 Revert of Fix sbox_validation_tests & Make isolate_driver.py include the executable itself (patchset #3 id:90001 of https://codereview.chromium.org/1245873002/ )
  c6649f2e Fix sbox_validation_tests & Make isolate_driver.py include the executable itself
  5591967b tools/gn: Remove fallback to direct_dependent_configs.
  fe59edcf [GN] move gn doc to proper place
  ba7a73dd Annotate GN executables and shared_libraries with sanitizer deps.
  6aeaa45a GN get_label_info support toolchains for target_out_dir.
  7693bf2b Sync GN unit tests with GYP, add checking script.
  28fccd5c Revert of Sync GN unit tests with GYP, add checking script. (patchset #3 id:40001 of https://codereview.chromium.org/1289423004/ )
  b2f72715 Sync GN unit tests with GYP, add checking script.
  7a694fe1 Fix various stray bugs in the GN auto-roller, and repair the DEPS file.
  83dc1610 Remove remaining legacy SplitString calls.
  f51a763e tools/gn: Remove includes of basictypes.h.
  b35ddf50 clang/win: Try to fix 64-bit build after https://codereview.chromium.org/1261953003.
  e3305701 clang/win: Try to fix build after https://codereview.chromium.org/1261953003.
  c9b8dabc Fix gn bootstrap.
  693c6056 Fix some crashes in GN.
  2a087637 Pull ICU, rename component build GN variables.
  d79f1ac6 Reference written files relatively when possible.

R=brettw@chromium.org
CQ_EXTRA_TRYBOTS=tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg;tryserver.chromium.win:win8_chromium_gn_dbg,win_chromium_gn_x64_rel

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

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

BUG=526268
TBR=jam@chromium.org
Previous Review URL: https://codereview.chromium.org/1326443006

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/dfec292..200a784

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

This doesn't even compile:

../../base/logging.h:523:23: error: invalid operands to binary expression ('basic_ostream<char, std::char_traits<char> >' and 'gfx::BufferUsage')
  ss << names << " (" << v1 << " vs. " << v2 << ")";
  ~~~~~~~~~~~~~~~~~~~ ^  ~~

BUG=530712
TEST=cast_shell_linux tryjob with gbm enabled by default

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

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

Actually, disable all substitutions be default. They're not likely to be
useful for the omnibox.

Since linking to the 10.10 SDK, the omnibox on Mac has started
smartquotifying input. It also replaces "--" with an em-dash. These are
unlikely to be desired for URL or search input, so this restores the old
behavior.

BUG=528014
TEST=Type a quote (") or two dashes (--) in the Omnibox on Mac and wait
a second. It shouldn't get replaced by a different character.

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

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

Revert of [Telemetry] Refactor Chrome devtools tracing (patchset #2 id:20001 of https://codereview.chromium.org/1338033002/ )

Reason for revert:
This CL fails on mojo perf bots. For Chrome, Telemetry waits for Chrome for the devtools port number. But for mojo, telemetry gives the port number to mandoline. Revert for now, will work on a fix.

See https://code.google.com/p/chromium/issues/detail?id=531755

Original issue's description:
> [Telemetry] Refactor Chrome devtools tracing
>
> Platform-level tracing doc:
> https://docs.google.com/document/d/1Uv8SzOq2ugvzLiOcP85g7qPiaxwsn4tgOdxpe3YjUkM/edit?usp=sharing
>
> BUG=515323
>
> Committed: https://crrev.com/cbecca3af4042fbf593dead4c12cfdd22cfd2310
> Cr-Commit-Position: refs/heads/master@{#348740}

TBR=nednguyen@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=515323

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

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

This factors the irt_* toolchain definitions using a template to reduce
copied boilerplate.  It also switches the ARM build to use arm-nacl-clang
rather than arm-nacl-gcc, which matches what the GYP build does now.  Many
of the compilation flags are still not at all right, but this is enough
for the build of the IRT to complete successfully.

BUG=531702
R=dpranke@chromium.org

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

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

If an extension is disabled after it is uninstalled, the extension is not enabled
when it is reinstalled. ThemeService was disabling theme extensions after they
were uninstalled if the extension was uninstalled as a result of the
NOTIFICATION_BROWSER_THEME_CHANGED notification. This bug was causing installing
a theme via the Chrome Web Store to sometimes not work.

BUG=468280
TEST=ThemeServiceTest.UninstallThemeOnThemeChangeNotification

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

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

This patch moves the render_font_warmup_win files into a common
location so that other content processes, especially PPAPI can
share the code. This is needed as a warmup to pushing win32k lockdown
to other content processes.

BUG=523278

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

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

This CL puts a little extra space between the list and the buttons.
It also makes the items paper-icon-items so they line up better.

BUG=531750

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

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/f26e8c0..dfec292

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/0faed58..5dd918b

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=tomhudson@google.com

Commits in this roll:
5dd918b fmalita@chromium.org Suppress test_imagegenerator_factory
2bcab82 reed@google.com remove code from SK_SUPPORT_LEGACY_NEWFROMGENERATOR, eliminates caller of deprecated SkInstallDiscardablePixelRef
48392b7 reed@google.com fix leaked generator
cd56f81 fmalita@chromium.org SkImageSource
506c802 bsalomon@google.com Add helper for creating leaf FPs inside GrFP::TestCreate functions
07344a5 mtklein@chromium.org Revert of SkPx: new approach to fixed-point SIMD (patchset #9 id:160001 of https://codereview.chromium.org/1317233005/ )
ec00d94 egdaniel@google.com Move some of the adding stencil attachment logic of Gpu and into Render Target.
37497dc rmistry@google.com Update SkWhitelistChecksums.cpp with the checksums of the fonts on the CT slave machines.
82c93b4 mtklein@chromium.org SkPx: new approach to fixed-point SIMD
b5b6032 bsalomon@google.com Test that GrFragmentProcessors work without input colors.
0faed58 mtklein@chromium.org cmake_build: support SKIA_OUT and BUILDTYPE
59ce45f bsalomon@google.com Revert of Test that GrFragmentProcessors work without input colors. (patchset #2 id:20001 of https://codereview.chromium.org/1341853002/ )
72c58e7 bsalomon@google.com Test that GrFragmentProcessors work without input colors.
3833daa robertphillips@google.com Fix GPU-only snapping bug in mask blur rendering
efd50da reed@google.com impl preroll for all image backends
5a744b7 mtklein@chromium.org Have SkVarAlloc::alloc() use sk_malloc_throw.
a46ada5 mtklein@chromium.org SkValidatingReadBuffer: make sure we don't call readTypeface().
8725e53 reed@google.com fix leak in test

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

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

Refactor AndroidVideoDecodeAccelerator into a class that manages the MediaCodec buffers, but delegates handling the SurfaceTexture to BackingStrategy implementations.  The AndroidCopyingBackingStrategy class implements the texture copy into the picture buffer, as it did in previous versions.

BUG=507834

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

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

Instead of listening to unreliable WebContentsObserver events, we now
make use of the FaviconDriverObserver.

R=nick@chromium.org
BUG=530486,528924

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

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

This is a spin-off of https://codereview.chromium.org/1294243004/
This exposes the helper functions converting enum types from a blink type to its
corresponding content type.

BUG=440463

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/4edabd3..f26e8c0

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

BUG=354284

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

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

Without this the background thread created by the FontLoader outlives
the HTMLViewer, which leads to random shutdown crashes.

BUG=531120
TEST=none
R=erg@chromium.org

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

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

TBR=scottmg@chromium.org
BUG=529769, 472900, crashpad:51

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

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

This removes the Rect::operator RectF(), and adds an explicit conversion
constructor on RectF instead.

R=sky@chromium.org
BUG=342848

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

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

This is achieved by setting the correct FBO before each kModeDraw and
hardware renderer won't reset it at the end of the draw so that
kModeProcess between kModeDraws also has the correct FBO.

This CL is to solve a bug where kModeProcess could run some GL commands
that binds the wrong FBO, and the following kModeDraw is messed up.

BUG=527113

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/0657feb..4edabd3

Sheriffs: In case of breakage, do NOT revert this roll, revert the
offending commit in the blink repository instead.

NOTRY=true
TBR=hayato@chromium.org,dcheng@chromium.org

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

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

grd files that are sent off for translation cannot contain the
use_name_for_id attribute as described in the bug. This removes
use_name_for_id from ios_strings.grd to fix the extract strings step.

This will need to be fixed for real as a follow-up, either by making
IDS_IOS_ACCEPT_LANGUAGES and IDS_IOS_DEFAULT_ENCODING regular
translatable strings, or by moving them into a separate file as is done
with all the locale_settings_<platform>.grd files.

BUG=531718
TBR=sdefresne@chromium.org

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

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

BUG=523040
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

This CL uses the finch trial group name to set WebKit settings for the android autoplay gesture override experiment.  Most of the code for the experiment itself is in https://codereview.chromium.org/1179223002/ .  The present CL will not build without it.

Summary of the experiment: Our goal is to decide if the requirement for a user gesture can be relaxed on android, to allow some media to autoplay without always requiring the user to interact with it manuall.  The finch experiment will let us try various combinations of heuristics for when to autoplay without a user gesture.  We record metrics based on user behavior, such as "manually started playback", "manually stopped playback", etc.

These options are composible, so we just use the trial group name to specify the options directly.  This lets us try different combinations after we get some initial data from the experiment.

BUG=487345, 402044

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/f8eb3135..d2261936

Please follow these instructions for assigning/CC'ing issues:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

This CL:
- removes the scripts for syncing and relevant comments.
- adds a note in README.chromium that it has been forked from the Mojo repo.

BUG=None

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

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

It's simply an overlook in previous CLs.

BUG=429053
TEST=gpu_unittests, webgl2_conformance
R=piman@chromium.org

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

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

This library is used by Chrome on Chrome OS to allocate accelerated
graphics buffers. The version avaiable on Ubuntu is too old, and as a
result we can't run unit tests for the gbm platform code on developer
workstations or on the waterfall.

This adds the library to third_party and fixes up the build files to use
either the OS copy (for Chrome OS release builds) or gardened copy (for
testing on the waterfall) depending on the value of "use_system_minigbm".

BUG=530712
TEST=
  GYP_DEFINES="use_ozone=1 chromeos=1 ozone_platform_gbm=1 use_system_minigbm=0" gclient runhooks
  ninja -C out/Release ozone_unittests
  ./out/Release/ozone_unittests

  gn gen out_gn_ozone --args='target_os="chromeos" use_ozone=true ozone_platform_gbm=true use_system_minigbm=false'
  ninja -C out_gn_ozone ozone_unittests
  ./out_gn_ozone/Release/ozone_unittests

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

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

Match the native Android TextView handles by using the same PopupWindow
visibility type, namely, TYPE_APPLICATION_SUB_PANEL.

BUG=528365

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

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

If user edited title and url at the same time, url didn't get saved.
The reason is that we save title first, then UI is refreshed by
bookmark observer, which resets the url edit text to the original url.

BUG=530794

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

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

When opening a tab in the background on Svelte devices, tabs are
marked as "frozen" without actually being frozen (no saved state).

* Make CompositorViewHolder load these tabs properly instead of
  trying to unfreeze them.

* Make Tab#unfreezeContents() protected so that nothing can handle
  this.

* Clear out pointers when destroying the objects and clean up
  destruction, in general.

* Stop destroying some TabObservers twice.

* Add a test to catch this.

* Moves process killing to the end of destroy() instead of the middle.

BUG=495877

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

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