GitLab

Raises an NSInvalidArgumentException if call setObject for NSMutableDictionary
with a nil value. Dictionary uses NSNull instead of nil.
Selected text marker range may be nil sometimes
(see BrowserAccessibilityCocoa::selectedTextMarkerRange), that leads to a crashe
in BrowserAccessibilityManagerMac::GetUserInfoForSelectedTextChangedNotification().

R=aboxhall@chromium.org, dmazzoni@chromium.org, dtseng@chromium.org
BUG=

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

BUG=587995

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

This leaves at least 1 day between the value of now() and thisUpdate/nextUpdate in OCSP responses generated by minica.py. This gives more wiggle room if there's a time differential between a test bot and a RemoteTestServer, and should help reduce HTTPSOCSPVerify test flakiness.

This also removes a reference to a non-existant variable in what should be an unreachable code path in minica.py.

BUG=630286

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

Reland of ll buildtools 55638fe5c3..60f7f9a8b4 (patchset #1 id:1 of https://codereview.chromium.org/2171273002/ )

Reason for revert:
GN binaries will not affects GYP's errors about
conditions.

Original issue's description:
> Revert of Roll buildtools 55638fe5c3..60f7f9a8b4 (patchset #1 id:1 of https://codereview.chromium.org/2166993002/ )
>
> Reason for revert:
> Speculative revert - this probably broke the official build - see crbug.com/630514 for more details. Will reland if the suspicion was wrong.
>
> Original issue's description:
> > Roll buildtools 55638fe5c3..60f7f9a8b4
> >
> >   In order to roll GN af92718f..1f417360 (r405547:r406705) and pick up
> >   the following changes:
> >
> >   267b1de9 Make `gn refs` handle files in data directories properly.
> >   5fc3fafd Add TaskPriority as a parameter to SequencedWorkerPool in preparation for TaskScheduler experiment.
> >   aca08753 Make `gn refs` look at script files.
> >   05316e06 Re-land r406064 "[GN] Add JSON project writer".
> >   5a1b4b17 Revert of [GN] Add JSON project writer (patchset #11 id:200001 of https://codereview.chromium.org/2064533002/ )
> >   d2edeb9a [GN] Add JSON project writer
> >   f0544fc6 Remove calls to MessageLoop::current() in tools.
> >   d93e5cf4 Document GN invoker and target name variables.
> >
> > TBR=dpranke@chromium.org
> >
> > Committed: https://crrev.com/d8a18dc3820a4d54d8fc9ad8667d242798f10d33
> > Cr-Commit-Position: refs/heads/master@{#406727}
>
> TBR=dpranke@chromium.org,brettw@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
>
> Committed: https://crrev.com/de249eccb4145727c8eee6ecb737bd50cbbf844e
> Cr-Commit-Position: refs/heads/master@{#407125}

TBR=dpranke@chromium.org,msramek@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

This only affects test targets and Content Shell.

R=sdefresne@chromium.org,mmenke@chromium.org

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

All of the known bugs have been addressed; let's see what isn't known.
This affects:

  chromium.lkgr:
    - ASAN Debug
    - ASAN Release Media
    - ASAN Release
    - ASAN Debug (32-bit x86 with V8-ARM)
    - ASAN Release (32-bit x86 with V8-ARM)
    - ASAN Release Media (32-bit x86 with V8-ARM)
    - TSAN Debug
    - TSAN Release
    - UBSan Release
    - UBSan vptr Release

With this CL, the only remaining linux builders that use GYP are
the 'Closure Compilation' builders. The LKGR waterfall will still
have a few Win GYP builders that will be flipped in a follow-up CL.

R=inferno@chromium.org, mbarbella@chromium.org
BUG=621581

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

This is hoping to achieve better build performance.

BUG=627637

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

platform/ParsingUtilities.h caters to the same needs, so transition
users of skipString(...) to skipToken(...) and remove
platform/text/ParserUtilities.h.

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

Some preliminary work to remove PlatformEvent abstractions.

Rename a constant to pass check-webkit-style.

BUG=625684
TBR=rbyers@chromium.org

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

The font cascade reconfiguration which was introduced as fix for AAT
shaping performance regressions in crbug.com/547912 seems to
occasionally cause CoreText crashes on OS X 10.9. We don't have a better
way of detecting this than by OS or CoreText API version number. This is
one of our top Mac crashers on Mac OS 10.9 with Chrome across versions [1].

This crash does not occur in newer versions of OS X and we can keep this
important performance optimization enabled there.

A big thanks to Robert Sesek (rsesek@) for the patient and thorough
initial investigation. Discussing and working together on this issue we
were able to identify the crash triggering code in HarfBuzz in this
case.

[1] https://bugs.chromium.org/p/chromium/issues/detail?id=576941#c74

BUG=576941, 625902

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

I wrote this while investigating a Promise issue. I think it wouldn't
hurt to land it :)

BUG=None

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

libyuv offers a new API, Android420ToI420, to convert the
Android YUV_420_888 frame (which has interleaved UV planes)
to normal I420 frame.
This can significantly reduce the de-interlacing time on
Android, up to dozens of milliseconds, which can help a lot
on the overall end-to-end video delay.

BUG=629342

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

Android lint tool pointed out that ... should really be … (U+2026).
Tested that this glyph renders well on Mac + Win as well, so
applying to all OS's by default.

TBR=flackr
BUG=621772

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

WindowServer::GetTreeWithId can be null in the case of
|in_nonclient_area|. It looks like |tree| was used without check in
4d3588cb.

BUG=630402

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

BUG=

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

This patch arranges the payment instruments in the following order:
> Non-autofill payment apps
> Complete credit cards from Autofill.
> Incomplete credit card from Autofill (need editing before sending to
  the merchant).

BUG=629882

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

Both Chromium and Blink code can use std::unique_ptr<> recently, so now
we can express the ownership passing on the type of WebTaskRunner::clone.
This CL replaces the return type of WebTaskRunner::clone from a raw pointer
to a std::unique_ptr<>, removes adoptClone, and updates the call-sites.

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

When escape is pressed to hide the spectrum, use commitEdit to return to
the original color text.

R=lushnikov

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

Revert of mus: Rename .*PointerWatcher.* to .*PointerDownWatcher.*. (patchset #5 id:80001 of https://codereview.chromium.org/2158273003/ )

Reason for revert:
As discussed (https://codereview.chromium.org/2163453002/), changing PointerWatcher to watch for all events instead.

Original issue's description:
> mus: Rename .*PointerWatcher.* to .*PointerDownWatcher.* to avoid confusion with PointerEventWatcher (https://codereview.chromium.org/2163453002/).
>
> Committed: https://crrev.com/b2db88d57e677ab3ff4b3fc7f58896603816053c
> Cr-Commit-Position: refs/heads/master@{#406584}

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

The original archives were recorded on Chrome Public (android-chromium)
and used an encoding that is not yet supported by Chrome Stable
(reference).

BUG=630004
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq

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

https://chromium.googlesource.com/skia.git/+log/d01a8f470f43..d50e2401787d

$ git log d01a8f470..d50e24017 --date=short --no-merges --format='%ad %ae %s'
2016-07-22 csmartdalton Fix SkClipStack bug with inverse-filled difference elements
2016-07-22 mtklein Don't ask GLX for EGL procs.
2016-07-22 borenet Change SKPs to new asset management format

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
TBR=jcgregorio@google.com

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

It's not being called, as of https://codereview.chromium.org/2123093002.
And none of the implementations actually do anything.

BUG=479994
TBR=rdsmith@chromium.org

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

Found crash in apk, but there is no offset, like below
DEBUG   :     #04 pc 0000000000209998  /data/app/org.chromium.chrome-1/base.apk

BUG=630620

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

'Cancel' isn't really the right word for what this change does.
In the event of an image decoding failure, ImageResource will
synthesize a call to ResourceLoader::didFinishLoading, killing
the actual network request but reporting it as a successful
completion to the rest of blink. This matches our traditional
behavior of decoding errors looking like a successful
resource load (for the most part).

This requires some plumbing changes to image decoding, because
the decoder selection logic doesn't report why it wasn't able to
create an ImageDecoder. It might be because insufficient data has
been received to sniff the image type, or it may be that we
definitely don't have a valid image type. This change exposes enough
information to tell the difference.

BUG=471272

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

No need to pre-configure the net::DnsClient and it will pick up changes to the
system DNS config automatically via net::NetworkChangeNotifier.

BUG=612439

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

BUG=615157

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

Fix bugs in attrib-type-match.html

BUG=628459
TEST=conformance2/rendering/attrib-type-match.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/2148723004
Cr-Commit-Position: refs/heads/master@{#407168}

https://chromium.googlesource.com/skia.git/+log/da626aadcced..d01a8f470f43

$ git log da626aadc..d01a8f470 --date=short --no-merges --format='%ad %ae %s'
2016-07-22 mtklein Fix SW sRGB dst + LCD coverage bug.
2016-07-22 brianosman Update SkImage_NewFromTexture test, to just test release procs of SkImage::MakeFromTexture.
2016-07-22 bsalomon Revert of Retract PipelineBuilder some more (patchset #9 id:160001 of https://codereview.chromium.org/2092893003/ )
2016-07-22 brianosman Cleanup of code that converts from GPU-backed resources to SkImageInfo
2016-07-22 msarett Add clamp to sk_linear_to_srgb, reorder instructions
2016-07-22 benjaminwagner Add SK_SUPPORT_LEGACY_ACCESSBITMAP to public.bzl.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
TBR=jcgregorio@google.com

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/2cf82064..1852b91a

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

As we continue to split WebView/WebWidget, this change renames
WebTestProxy to WebViewTestProxy, since it should only used for
proxying RenderView calls. A future change will introduce a new
WebWidgetTestProxy for use with RenderWidget.

BUG=583347

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

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

Cleaning up track-webvtt* http tests to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests.

BUG=588956

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

This was lifted in https://codereview.chromium.org/2165873002/,
failing in builds since:

https://build.chromium.org/p/chromium.gpu.fyi/builders/Win7%20Release%20%28New%20Intel%29/builds/1174

BUG=628863
TBR=zmo@chromium.org,kbr@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

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

Revert of BubbleFrameView: add top padding even when close button is hidden (patchset #7 id:120001 of https://codereview.chromium.org/2148963002/ )

Reason for revert:
Caused https://crbug.com/630539

Original issue's description:
> BubbleFrameView: add top padding even when close button is hidden
>
> If there's no title or close button present, BubbleFrameView still needs some
> top padding to avoid the content being hard up against the top edge. Always pad
> as though the close button was visible, even if it's not, unless there's a
> title.
>
> BUG=622859
>
> Committed: https://crrev.com/f06ef4ac6ac3dd2f3a76486d2efde9ec35583ae6
> Cr-Commit-Position: refs/heads/master@{#406856}

TBR=msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=622859

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

Revision b779ce61 broke the
generate_build_files step on the Dr. Memory build:

gyp: Key 'conditions' repeated at level 6 with key path
'targets.0.conditions.5.1' while reading
C:\b\build\slave\drm-cr\build\src\printing\printing.gyp

BUG=607668
TEST=none
TBR=skau@chromium.org

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

BUG=629867

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

This CL modifies the way in which account credentials (URL/username) are displayed on PasswordEntryEditor.java and adds buttons which will allow users to view/copy their credentials.

BUG= 628669

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

BUG=626827

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

This CL retains the maximum autofill profile use-count,
instead of summing the use-counts, when merging two
profiles.

The previous behaviour, summing, inflates the use-count
during profile deduplication, double counts on profile
usage, and falsely counts a fully-typed submission as a
reuse of any profile it matched. All of these skew the
frecency score of the resulting merged profile.

BUG=629978
R=mathp@chromium.org, sebsg@chromium.org

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

The interface has been broken for a long time.

BUG=

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