GitLab

BUG=633253

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

R=tsergeant@chromium.org
BUG=638405
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

We need more OWNERS... move up some command_buffer OWNERS, and add kbr@

BUG=None
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/2348773002
Cr-Commit-Position: refs/heads/master@{#419080}

BUG=646168
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/2347063002
Cr-Commit-Position: refs/heads/master@{#419079}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/7eb57a183493..82d4df2ea564

$ git log 7eb57a183..82d4df2ea --date=short --no-merges --format='%ad %ae %s'
2016-09-15 benjhayden Rename Composition to RelatedHistogramBreakdown.
2016-09-15 lpy Make runtime call stats table grouping consistent with V8 tools.
2016-09-15 prasadv Modify update_bug_with_results to check bisect job status using buildbucket API.
2016-09-15 benjhayden Move scalar-span#significance emoji after #content.
2016-09-15 ccraik Fix frame end detection
2016-09-15 benjhayden Merge NumericValue into Histogram.
2016-09-15 aiolos Disable testTBM2ForSmoke on mac

BUG=646208

TBR=catapult-sheriff@chromium.org

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

This is a step towards removing SidebarTree and moving all
tree outlines into shadow.

BUG=none

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

We were previously unable to see the allocation size in the minidump
because it was optimized out. This should keep it around.

BUG=643845
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

I had the name of one of the input fields wrong, and I had
an error case not returning output correctly.

R=brettw@chromium.org
BUG=555273

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

Reverted in:
https://codereview.chromium.org/2348643002/

Reason for revert:
Fixed gn gen errror

This will allow monochrome to easily define its own set of the same
targets with a different resource whitelist

TBR=brettw@chromium.org,miu@chromium.org
BUG=641032

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

The value of a submit button is appended to a FormData if its activatedSubmit flag
is set.
In a case of form.submit() is called in submit event handler, we did:

1. Set activatedSubmit flag of the submit button
2. Dispatch submit event
2.1. form.submit() builds FormData
3. Build FormData if the submit event was not prevented
4. Clear activatedSubmit flag of the submit button

In step 2.1, we should not include the submit button. So, this CL changes the
process as follows:

1. Dispatch submit event
1.1 form.submit() builds FormData
2. If the submit event was not prevented,
2.1. Set activatedSubmit flag of the submit button
2.2. Build FormData
2.3. Clear activatedSubmit flag of the submit button

Implementation:
We called setActivatedSubmit() before/after prepareForSubmission(). Instead, we
pass a submit button element to prepareForSubmission(), and call
setActivatedSubmit() just before/after building FormData.

BUG=646790

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

BUG=481588

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

This CL is a step toward deep linking to site details in the site
settings. This CL adds the c++ handler backend and does show the
exceptions in the UI. The page title is not updated with the subject URL,
but that is not a regression and will be addressed in a future CL.

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

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

BUG=646443
NO_DEPENDENCY_CHECKS=true

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

This patch renames createVisiblePosition() (all overloads) to
createVisiblePositionDeprecated() to discourage creating a VisiblePosition
in dirty layout and waiting for layout update lazily. This patch also
introduces new createVisiblePosition() which assumes clean layout.

This patch is a preparation for hoisting the layout update call from the old
createVisiblePosition(). In follow up patches, the callers should ensure
clean layout by themselves and call the new createVisiblePosition() instead.

This patch is a mechanical substitution
's/createVisiblePosition(/createVisiblePositionDeprecated(/g', followed
by the following changes:
- Introduce the new createVisiblePosition() in VisiblePosition.h/cpp.
- Revert the substitution in modules/accessibility/AXLayoutObject.cpp,
  since callers there already ensure clean layout

BUG=647219

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

These were disabled ~12 months ago for being flaky. 50 runs
through the linux_chromium_rel_ng trybot revealed no flakiness.

BUG=527437

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

BUG=646323
TESTS=webkit_unit_tests DocumentLoadingRenderingTest.ShouldNotPerformRepeatedLayoutWithPendingImport DocumentLoadingRenderingTest.ShouldClearPlaceholderStyleWhenIgnoringPendingStylesheet

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

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/537cdb699a70d5d55f3eefb923eae1dc5104a8eb Add goma to wasm waterfall (sbc@chromium.org)
  https://crrev.com/b6d326d6f6f9608bba307109cc4a1c5efc5946bb Run the ninja "up-to-date" check for all compiles, but just as an fyi (agrieve@chromium.org)

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

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

Running a task on a worker thread after forcible termination may cause a crash
because of an access to an empty handle returned by V8 API.

BUG=646861

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

BUG=647169,645949
TEST=CompositedLayerMappingTest.BackgroundPaintedIntoGraphicsLayerIfNotCompositedScrolling
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

The TabStripModel is no longer needed inside the event
handlers.

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

This is a step towards removing SidebarTreeElement, and moving
all tree outlines into shadow dom.

BUG=none

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

TBR=kbr@chromium.org
BUG=598902
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/2344983002
Cr-Commit-Position: refs/heads/master@{#419061}

The stack in this crash report indicates that we're inside
UnpackedInstaller::InstallExtension, and we know that function gets
called via a posted task (via OnInstallChecksComplete). The code inside
InstallExtension uses the weak pointer to the ExtensionService without
validating it first, so we strongly suspect a null ExtensionService to
be the root cause of the crash.

BUG=647265

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

BUG=644669
TEST=deqp/functional/gles3/shaderloop_for/while.html
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/2319023002
Cr-Commit-Position: refs/heads/master@{#419059}

This changes the template stamping parts of cr-lazy-render to look more
like iron-list. This simplifies the code and makes it play nicely with
Closure.

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

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

Threads seem to be exiting unexpectedly, so to help debugging set a
crash key if an unexpected WM_QUIT message is received.

BUG=647068
TBR=scottmg@chromium.org

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

This is done using:
./httparchive.py remap-host maps_004.wpr localhost:10020 map-test
(https://codereview.chromium.org/2346853003)

BUG=646924
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/2347483004
Cr-Commit-Position: refs/heads/master@{#419056}

BUG=none

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

Updates TemplateURL to know how to request Now on Tap data for Contextual Search.
Also remove the old unused flag about whether to resolve or not.

BUG=646664,641543

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/596285fbd59f..7eb57a183493

$ git log 596285fbd..7eb57a183 --date=short --no-merges --format='%ad %ae %s'
2016-09-15 aiolos Add chart-legend-tooltip element.

TBR=catapult-sheriff@chromium.org

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

w/o this, WebAudio will always think there are only two channels
available. Untested since I don't have a 5.1 setup on Linux.

BUG=646435
TEST=verified that 2 channels are reported locally.

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

This CL moves the site URL out of the subpage title and into the body of
the subpage. The new title is Site settings. In the new position, the
url has its favicon. This is similar to a previous revision done by
Finnur@.

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/e9b7e550..ad5be95d

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

Replace usage of single-threaded SequencedWorkerPool with a base::Thread in image_fetcher_unittest.mm

Also cleaned up a TODO for 440857 on the way.

BUG=646443, 440857

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

Skia is changing a few signatures to use smart pointers in its API. This
requires that Chromium be updated to match. This define will keep the
old API around while the Skia change is rolling into Chromium.

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

List from
https://cgit.freedesktop.org/vaapi/intel-driver/tree/src/i965_pciids.h

BUG=646135
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/2337863002
Cr-Commit-Position: refs/heads/master@{#419047}

These weren't caught in the original CL
(https://codereview.chromium.org/2310563002) since that CL
intentionally didn't exercise everything.

I neglected to test the follow-up CL completely before
landing the first one, so I missed these bugs:

  1. WebContentsBindingSet::Binding needs to be exported
  2. The RouteProvider proxy in RPHI needs to reset on Init
  3. RenderFrameImpl never implemented
     IPC::Listener::OnAssociatedInterfaceRequest.

*crawls into shame cube*

Landing these fixes separately from the follow-up CL
(https://codereview.chromium.org/2310583002) to retain its
purpose as a minimal example of WebContents IPC conversion.

BUG=612500

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

Needs to be done in MediaCodecUtil otherwise the EME types won't
incorrectly reject encrypted vp8 support.

BUG=612909
TEST=none

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

For better tracking.

BUG=647433,647412,647416,647409,647421,647417
TEST=none

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

We still have one instance where ChildThreadImpl is used without
ChannelMojo (utility processes launched by a service process),
causing a nullptr dereference in ChannelProxy when trying to add
the associated interface.

This CL replaces a DCHECK with a branch to avoid the crash, since
silently ignoring the call is safe and reasonable.

In examining this bug I also realized that ChildThreadImpl was
incorrectly adding the RouteProvider interface after Channel
connection, which is only safe to do in the browser, where the
remote endpoint hasn't been launched yet. So this CL fixes that
too, and clarifies the documentation in ipc_channel_proxy.h.

BUG=647251
R=jam@chromium.org

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