GitLab

Termination of a renderer process results in a call to
WebContentsObserver::RenderProcessGone. Navigating from
RenderProcessGone using NavigationControllerImpl::Reload or LoadURL
results in an assertion due to MojoApplicationHost::Init() being called
again on an already initialized host. Reset |mojo_application_host_| in
RenderProcessHostImpl::ProcessDied before calling
ViewHostMsg_RenderProcessGone message handlers to avoid this problem.

BUG=448435

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

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

Missing dependency introduced in r314400 ("Make PointerDeviceObserver
work under Ozone").

TEST=link_freon component build
BUG=450899

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

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

This allows us to run the 'adb shell input X' commands.

See for details:
https://android.googlesource.com/platform/frameworks/base/+/cd92588/cmds/input/src/com/android/commands/input/Input.java

BUG=

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

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

This fixes a reporting bug with the
DataReductionProxy.RequestCompletionErrorCodes histograms. Previously,
these histograms counted requests that encountered errors when retrying
the request after bypassing the data reduction proxy (e.g. block-once).
This is incorrect because the error did not occur when trying to use the
DRP.

The fix is to ignore requests that completed with the LOAD_BYPASS_PROXY
load flag, since these requests did not use the DRP even if the
proxy_server() in the URLRequest is still set to the DRP.

BUG=454947

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

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

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

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

These IPCs hold this param as a gfx::Rect and the RenderWidget and RenderView
set the size. On the receiving end, the size is sometimes used so initial_pos
was a misnomer.

This is a mechanical rename, there should be no functional difference.

TBR=
BUG=

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

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

This is needed by display_controller.cc in ozone builds (to set display
rotation & scale to apply to cursor movement).

These dependencies are needed to enable strict checking of undefined
symbols in the shared_library build.

BUG=445627
TEST=build chrome for link_freon via simplechrome

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

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

If GetAlternateProtocol() is called but there is no alternate protocol, it
now returns one with UNINITIALIZED_ALTERNATE_PROTOCOL.

See https://crrev.com/665083009 for the context of this change and how draft
future changes might rely on this CL.

BUG=392575

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

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

It was renamed to AndroidSyncSettings in http://crrev.com/845543005.

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

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

BUG=448507

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

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

Cube samplers create input sample points in some order. Document the QCMS
order (x,y,z) which is the same as OSX ColorSync and cube LUT interchange
standards such as the Adobe Cube LUT Specification 1.0, 2013. The samples
are RGB order with R changing the slowest, B the fastest.

Hoist the division out of the sample point creation loop: takes 0.0003939
seconds on MacAir for 32^3 sample points, 0.0001619 with hoisting. That's
fine, but small beans compared to the overall costs of cube creation. The
majorty of the work is in qcms_chain_transform (8ms) where floating-point
is used to maintain transform precision.

The output ColorCube LUT produced by qcms_chain_transform is indexed into
transform in RGB order (time cost ~0). Add comments about order.

BUG=443863

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

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

BUG=407778

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

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

BUG=450601
R=kalman@chromium.org

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

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

The extensions with implicit vs. explicit resolve are already
treated differently at the cmdbuffer level. The incorrect
specification of glRenderbufferStorageMultisampleEXT
(multisampled_render_to_texture) with
glRenderbufferStorageMultisample (core MSAA with explicit resolve)
causes an incorrect binding (see removed TODO which says the
same thing).

Also remove glRenderbufferStorageMultisampleANGLE alternative with
the core name.

BUG=454816,438742,325668

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

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

These dependencies are needed to enable strict checking of undefied
symbols in the shared_library build.

BUG=445627
TEST=build chrome for link_freon via simplechrome

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

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

Enable Telemetry to discover devices and find browsers for each device.

BUG=435723

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

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

BUG=453147
R=tengs@chromium.org

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

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

For now a no-op in the renderer.

Enable:
  --enable-threaded-gpu-rasterization

Disable (which takes priority over Enable):
  --disable-threaded-gpu-rasterization

BUG=454500

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

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

createCDM() now uses WeakPtr<> so that it fails gracefully when the
frame is destroyed.

BUG=454278
TEST=fuzzed test doesn't crash

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

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

The changes make it possible to focus individual downloads by using UP/DOWN on the keyboard (like other tables and lists). It is possible to move between the links in a specific download using LEFT/RIGHT and TAB. Focus changes with TAB are achieved by relying on the focusing of elements rather than capturing the keys. This allows screen readers to focus elements without weird behavior.

Changes in FocusRow to support TAB navigation will also affect the history page. This change will keep both pages consistent.

BUG=427670,452870

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

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

* Pull out logic in ShortcutHelper for parsing shortcut properties
  into the ShortcutInfo struct.  This will be used by AppBannerManager.

* Passes ShortcutInfo structs around instead of 4 different parameters
  when creating shortcuts.

BUG=453170

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

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

Android uses a custom UI with the cross-platform strings for address
input. This is a re-land of http://crrev.com/890103002 with a minor
alteration to fix Android GN build: do not build the validator on
Android.

TBR=estade@chromium.org
BUG=454120

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

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

R=lcwu@chromium.org,byungchul@chromium.org
BUG=internal b/18934906

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

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

Since there are no plans to use this API at least for 1H 2015, there is no
point leaving this code in Chrome. Removing this API to reduce code bloat.
We can revisit adding this API again when this work gets prioritized.

R=isherman@chromium.org, kalman@chromium.org, xiyuan@chromium.org
BUG=452562

This flag is specified since we're deleting an enum from histgrams. This currently breaks the presubmit.
NOPRESUBMIT=true

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

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

1- A singleton
2- A logger

Now CAPS depends on base and on chrome/common_version. About 190KB
on plain release build.

BUG=447073

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

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

After https://codereview.chromium.org/896583002/, the isolate step on the
64-bit bot is failing with

    MappingError: Input file C:\b\build\slave\CrWinClang64_dbg_\build\src\out\Debug_x64\x64\msvcp120d.dll doesn't exist

It looks like the x64\msvcp120d.dll only exists in 32-bit builds.
While here, also add the x64 libraries for 32-bit release builds.

BUG=453657,452057
R=maruel@chromium.org, scottmg@chromium.org

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

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

This CL avoids a double registration for content settings in account
rconcilor as this leads to a DCHECK being fired during a reauthentication
operation.

BUG=454507

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

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

Since not all AppRemoting apps will want or need GDrive support to be
enabled, we need to be able to control which apps enable this feature.

This cl adds 'app_features' to the AppRemoting gyp targets and passes
the list of features into the build_webapp script (so the manifest can
have the correct oauth scopes) and the webapp.

Also, because the 'cast' capability was "in the way" for these changes,
it has been converted into a feature as well (default: disabled).

BUG=

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

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

If either the sync experiment OR the command line flag is present, the feature is enabled. (Enabling the feature also gives the user the ability to control it via a separate pref.)

BUG=

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

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

TBR=machenbach@chromium.org

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

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

Summary of changes:
1. Implements a rich UI to prompt for the user's permission to
   accept remote assistance from a hangout participant
2. Move getUrlParameter() to base.js
3. Show an interactive auth prompt if the user doesn't have an
   OAuth2 token for starting the It2Me host in v2 app. For v1
   app, do an auth redirect.
4. Tear down the It2MeHelpee channel if the user decline the
   remote assistance.

BUG=451591

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

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

2683af7 Fix typo and ASN.1 tag number range comment in bytestring.h.
a84f06f Move free from cbb_init() to only CBB_init().
d660b57 runner: Refactor handshake fragmenting slightly.
79ae85e Remove NETSCAPE_HANG_BUG.
e33b9b0 Remove dtls1_timeout_st.
83f9040 Add DTLS timeout and retransmit tests.
d9b091b Revert "Drop retransmits in DTLS tests."
377fc31 Document DTLS timeout API and add current_time_cb hook.
868c7ef Don't assume alignment of ChaCha key on ARM.
64b17cc Fix patch collision of d062c8af and 2b2d66d4.
23343e4 Fix AES-256 NIDs.
d062c8a Probe for NEON support if getauxval is missing.
2b2d66d Remove string.h from base.h.
efed221 Enable more warnings & treat warnings as errors on Windows.
0bb81fc Fix misleading comment.
ab21891 Add a script to generate the ChaCha ARM asm.

BUG=454308

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

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

https://codereview.chromium.org/877293003

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/46c77f7..c9fed24

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

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

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

UpdateCacheFromPrefsOnPrefThread and UpdatePrefsOnPrefThread.
Factor out key strings into variables.

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

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

This CL:
- Makes PointerDeviceObserver get notified when a mouse or touchpad is
  plugged in or removed by making PointerDeviceObserver an
  InputDeviceEventObserver on ozone only.
- Adds InputDeviceEventObserver::OnMouseDeviceConfigurationChanged() and
       InputDeviceEventObserver::OnTouchpadDeviceConfigurationChanged().
- Adds the plumbing from EventFactoryEvdev to notify
  InputDeviceEventObservers when a mouse or touchpad is plugged in or
  removed.

In a later CL I will make PointerDeviceObserver an InputDeviceEventObserver on
X11 and add the X11 plumbing

BUG=450899
TEST=Manual, see bug

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

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

Upcoming changes to filename_util are unreviewable if they include
material code changes and formatting changes. This CL just includes
formatting changes and is the result of running:

  'clang-format -i --style=Chromium filename_util_unittest.cc'.

Depends on: https://codereview.chromium.org/869823003
Needed by : https://codereview.chromium.org/869233006

BUG=444102

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

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

This patch adds composite event recording for requested rects into the
frame timing tracker.

The current approach is to record all rects that intersect the visible
rect for that layer.

R=danakj, mpb@chromium.org
BUG=441555

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

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

UploadFileElementReader relies on checking the modified time of files
being uploaded to determine if a sliced file was modified during upload.
Clients of the net stack (in particular Blink) currently pass around the
expected modified time in a manner which cause the timestamp to lose
precision (E.g. converting to and from a double time_t).

As a result the expected timestamp and the current timestamp as returned
by GetFileInfo() will not match exactly. Current code attempted to
compensate for this by converting both timestamps to (integer) time_t.
However, since the conversion rounds down, this check would only succeed
if the loss of precision of the expected timestamp also caused it to
round down. This is not always the case. (E.g. (epoch + 10.999999) will
become 10 when converted to time_t, but the expected timestamp may have
rounded up to (epoch + 11.0) in the meantime.)

This patch compares the timestamps by checking if the magnitude of the
difference is less than one second.

BUG=426465
R=mmenke

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

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

This change also:
 - adds x86/x86_64 in the list of supported ABIs.
 - adjusts the regexp of the REST endpoints to support the emulator:
   the old regex was using \w+ to match the device serial, which
   doesn't work with the cases such as "emulator-1234".

TBR=primiano@chromium.org
BUG=

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

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