GitLab

Maintain a whitelist of which browsers it makes sense to compare with --browser=reference (doesn't include system)

BUG=416705
TEST=None.
TBR=tonyg
NOTRY=True

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

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

Unlike other permissions, granting notifications for a given domain
will grant them even if such domain happens to be an iframe of
an embedder. Most other permissions are granted only for a combination of
iframe + embedder.

This reverts an unintentional behavioral change, and moves
us back to the situation we currently ship in stable while
we figure out if it is a good idea or not to change the behavior.

BUG=416645

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

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

Following up on http://crbug.com/338289#c13, this CL removes the kill switch for PSL matching. Apart from the linked check with our PM, the consideration is that PSL matching has been in stable for a couple of releases now on all platforms (except for KDE, see below), and the bugs seen so far were of low severity, exposing problems in password manager rather than with PSL matching itself.

It also replaces a couple of callsites of IsMatchingEnabled with ShouldPSLDomainMatchingApply -- the former did not take sites with a unified login origin into consideration, and on the affected callsites it seemed like it should.

One caveat: PSL matching has not been added to KDE on Linux yet (it's planned, just not a top priority). Given that PSL is live on other Linux environments, and all other platforms, the concerns about needing the kill switch for KDE are low.

As a follow-up to this CL, the (never submitted) internal CL with the corresponding Finch config will be closed.

BUG=338289

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

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

BUG=417189
NOTRY=True
TBR=mukai@chromium.org

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

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

Becuase base/strings/string_split.h defines:
typedef std::vector<std::pair<std::string, std::string> > StringPairs;

BUG=412250

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

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

* Check if keyboard mic is available in MediaStreamManager and inform AudioManager if so.
* AudioManager sets the effect in AudioParameters returned from GetInputStreamParameters if available.

BUG=345296

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

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

Removes the superflous cvox.AbstractMsgs class and moves what was rpeviously
ChromeMsgs out of host.

BUG=341953
R=dtseng@chromium.org

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

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

Previous try: https://codereview.chromium.org/137993011
Reverted: https://codereview.chromium.org/139153005

BUG=76280
R=jln@chromium.org

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

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

TBR=machenbach@chromium.org

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

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

expected.

BUG=405413

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

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

We used to just leak everything on Mac. Since r295194, we clean up
references to Profiles being deleted, in Chrome's AppListViewDelegate.
This has caught out the Mac app list trying to access stuff it shouldn't
if Cocoa is still drawing it while Chrome is shutting down. Tests
started failing flakily.

This CL adds AppListViewDelegateObserver::OnShutdown to that Chrome's
AppListViewDelegate can notify the AppListView that it's shutting down.

On Views platforms, the Widget is just closed (which also happens just
after the chrome::NOTIFICATION_APP_TERMINATING broadcast). However, (a)
Cocoa doesn't have a `CloseAllSecondaryWidgets` call to trigger this and
(b) just closing the NSWindow isn't usually enough on Cocoa to avoid
accessing C++ objects due to reference counting.

We could also go through the platform-specific AppListService (e.g. by
passing in the AppListService that creates a Chrome
AppListViewDelegate), but this seemed neater.

This de-flakes a bunch of applist tests on Mac. The Mac-specific
ShowAppListUsingShim test is also augmented with extra checks for the
OnShutdown stuff.

BUG=415264
TEST=AppListServiceMacInteractiveTest.ShowAppListUsingShim,
AppListServiceInteractiveTest.*

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

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

Becuase base/strings/string_split.h defines:
typedef std::vector<std::pair<std::string, std::string> > StringPairs;

BUG=412250

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

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

Revert of DevTools socket tunnel. (patchset #11 id:300001 of https://codereview.chromium.org/517233002/)

Reason for revert:
Fails to compile on Android: https://build.chromium.org/p/chromium.linux/builders/Android%20Arm64%20Builder%20(dbg)/builds/2163

devtools_bridge/android/javatests/src/org/chromium/components/devtools_bridge/SocketTunnelServerTest.java:291: cannot find symbol
symbol: method sleep()
                sleep();
                ^

Original issue's description:
> Implementation of DevTools socket tunneling service for remote debugging.
>
> Test APK in addition to instrumentation tests contains launching activity and foreground service what let duplicate ChromeShell debug socket to a differently named socket. If a device with running ChromeShell and DevToolsBridgeTest's service connect to a Desktop chrome (with USB debugging enabled) then chrome://inspect#devices will show 2 ChromeChell items. Both are debuggable (https://codereview.chromium.org/521573002/ should be applied to Desktop chrome to make the process less surprising). It's suitable for manual tests.
>
> This CL doesn't care of a few known issues:
> 1. Data channel buffer overflow. Data channel automatically closed in this case.
> 2. Uncontrolled number of pending sockets. DevTools throttles opening sockets and and keeps sockets that exceeds a threshold hanging.
> 3. Uncontrolled number of threads (it's actually not a problem if #2 solved, thanks to DevTools throttling).
>
> To keep this CL reasonably simple this issues will be addressed later.
>
> TEST=org.chromium.components.devtools_bridge.SocketTunnelServerTest, see description fro manual testing.
> BUG=383418
>
> Committed: https://crrev.com/15044609893faf9b2c2a533aa9a136f288ea20a1
> Cr-Commit-Position: refs/heads/master@{#296360}

TBR=mnaganov@chromium.org,tedchoc@chromium.org,erikwright@chromium.org,serya@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=383418

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

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

Test APK in addition to instrumentation tests contains launching activity and foreground service what let duplicate ChromeShell debug socket to a differently named socket. If a device with running ChromeShell and DevToolsBridgeTest's service connect to a Desktop chrome (with USB debugging enabled) then chrome://inspect#devices will show 2 ChromeChell items. Both are debuggable (https://codereview.chromium.org/521573002/ should be applied to Desktop chrome to make the process less surprising). It's suitable for manual tests.

This CL doesn't care of a few known issues:
1. Data channel buffer overflow. Data channel automatically closed in this case.
2. Uncontrolled number of pending sockets. DevTools throttles opening sockets and and keeps sockets that exceeds a threshold hanging.
3. Uncontrolled number of threads (it's actually not a problem if #2 solved, thanks to DevTools throttling).

To keep this CL reasonably simple this issues will be addressed later.

TEST=org.chromium.components.devtools_bridge.SocketTunnelServerTest, see description fro manual testing.
BUG=383418

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

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

BUG=402356
TEST=browser_tests --gtest_filter=SyncFileSystemTest.AuthorizationTest

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

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

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

Revert of EventSender: Added checks for missing arguments (patchset #4 id:60001 of https://codereview.chromium.org/584203003/)

Reason for revert:
This makes the fast/events/hit-test-counts.html layout test fail. Reverting for now.

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fevents%2Fhit-test-counts.html&testType=layout-tests

Original issue's description:
> Added checks for missing arguments in event_sender.cc. Also simplified the uses of x & y in EventSender::GestureEvent.
>
> BUG=415970
>
> Committed: https://crrev.com/302907463636d2ddfe25c5381bd1f2b64016d1c6
> Cr-Commit-Position: refs/heads/master@{#296256}

TBR=dpranke@chromium.org,rbyers@chromium.org,earthdok@chromium.org,mustaq@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=415970

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

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

BUG=344769

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

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

The Android link disambiguation popup is rendered by PopupZoomer on the Java side. This CL tries to recreate that UI in Aura using a BubbleDelegate.

This part works just like the Android code:

When Blink detects an ambiguous gesture it calls RenderViewImpl::didTapMultipleTargets(), which uses paintCompositedDeprecated() in the WebWidget to get a zoomed bitmap region containing the ambiguous links the gesture could possibly indicate. RenderViewImpl then sends a ShowDisambiguationPopup message via IPC which ultimately ends up in RenderWidgetHostImpl::OnShowDisambiguationPopup(), which marshals the bitmap and then then calls ShowDisambiguationPopup on the view_.

This part is specific to aura:

After a series of delegation steps the callback ends up in WebContentsViewAura::OnLinkDisambiguationPopupRequested, which converts the target rectangle from pixels to DIPs and then requests the popup be rendered by asking its WebContentsViewDelegate, ChromeWebContentsViewDelegateViews, which owns a LinkDisambiguationPopup and asks it to render.

The LinkDisambiguationPopup uses BubbleDelegate to show the zoomed image. Any GestureEvents it receives converted into the coordinate space of the content view and then are relayed via callback back to WebContentsViewAura, which then converts them in to Blink events and sends them on to the RenderWidgetHostImpl for relaying back to Blink.

jam: content/ parts
sky: everything else?

BUG=385249

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

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

BUG=380213, 416961, 408734, 370175
R=stevenjb@chromium.org
TEST=manually

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

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

This is the preparation for introducing other types of file system supported by
external file system backend into the drive protocol handler.

The CL does not change the format of drive URL.

BUG=367027
TEST=Open HTML, MHTML, PDF, GDOC, OGG, MP4 files on the drive volume.

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

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

This patch will fixed the build errors which is due to while notification flag is
enabled and extension flag is disabled.

BUG=None

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

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

Adding split view divider widget which could be dragged to exit the split view.

BUG=403207,408691

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

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

Previously, this test would crash if the output wasn't quoted properly
(since it assumes that cmd_string.length() >= program_string.length() +
2). Instead, directly compare the string with the expected output. This
also guards against the possibility of mis-constructing the expected
string.

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

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

If you click on the SSL interstitial's error code, the following is
now shown:
* subject
* issuer
* validity end date
* current date
* pem encoded chain

This CL adds a placeholder for debugging information to the security
interstitial. It's intentionally meant to be flexible, so that it could
support other debugging information in the future from other warnings.

BUG=413838

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

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

call OnTerminating in unit test.

BUG=None

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

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

App windows launched in windows ASH should not be messing with the AppUserModelId for the remote HWND.

This causes problems on Windows 7 Chrome OS mode where launching popup app windows in Chrome OS mode
causes the taskbar button for the remote HWND to change to the application button. On Windows 8
these calls silently fail.

BUG=409824

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

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

The close button was setting the AXDescription property, it somehow caused
VoiceOver to use it as the tab view's description. Change it to set the AXTitle
property instead to fix the issue, and to match Safari.

Also change the tab strips' AX children to be sorted from left to right, so
that VoiceOver's "Tab X of Y" order is always ascending left to right.

BUG=415570

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

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

This is to help unblock some of the Blink work to move local frame only
APIs off WebFrame. Strictly speaking, all that was required to unblock
this bug was to manually coerce the WebFrame to a WebLocalFrame for the
call to setIsolatedWorldHumanReadableName.

However, it never makes sense to inject script on a remote frame, so
I've updated ScriptInjection to just work with local frames where
possible. Code that uses ScriptInjection now does the type coercion at
the class boundary. As extensions code is gradually updated to
understand out-of-process iframes, these manual type coercions should
disappear.

BUG=416659

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

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

Cleanup: Remove the instances of "rm_binaries.py" from 'src/tools/gn/bin'. The GN binaries have moved to src/buildtools. So, this script was removing GN binaries from tools/gn/bin that aren't used anymore. It was kept with the same intent till july 2014 (expected), and now think so, it is the good time to remove this script.

BUG=

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

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

BUG=397019

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

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

They were accidentally enabled with the switch to recipes and they don't work
because the older builds don't necessarily work with the generated profile.

BUG=388885
TBR=dtu@chromium.org

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

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

Older versions of Chrome expanded thumbnails to 1024x1024 to work around
an IMG driver bug.  When these were processed through DecompressionTask,
they got corrupted because this method assumed the content size and
buffer size were the same.  This patch changes it to decompress into a
potentially larger buffer then copy if needed.

BUG=416437

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

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

This adds Java code for running junit tests, as well as gyp targets for
both runnable and non-runnable host-side JARs.

BUG=316383

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

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

Rebased on https://codereview.chromium.org/588893006/

Comments that look like standalone line comments but immediately follow
a previous suffix comment should be considered suffix as well so they
can be added to the same node.

R=brettw@chromium.org
BUG=348474

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

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

Looks like a typo from https://codereview.chromium.org/522553002.
NOTRY to green up the perf bots.

BUG=413637
TEST=None.
TBR=nednguyen
NOTRY=True

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

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

StartPage/EndPage from spooling code. Calling this from metafiles unreliable. For example it's not played if for empty page metafiles.
Removed multipage support from windows metafile. We never use that, but it cause issues with empty files.

BUG=412690

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

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

BUG=414781, 416859
TEST=Manual, see bug

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

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

Use sidestep to detour CertVerifyCertificateSignatureEx on Windows versions earlier than Vista (excluding XP SP3), adding in SHA-256 support by deferring to NSS.

The canonical path to supporting SHA-256 is to install XP SP3 or the appropriate hotfixes for XP x64 / Windows Server 2003. However, as not all users may do so, and there's enough of a usability hurdle, provide an interception hook until we fully drop support for these systems.

XP is already outside of MSFT EOL (April 2014). Windows Server 2003 is EOL'd July 2015.

BUG=401365

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

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

 - Request batch attestation instead of individual
 - Updates from U2F API revisions
 - Enumerate HID devieces based on usage page instead of vid/pid
 - Handle wrong keyHandle length and init messages
 - Smaller bug fixes and refactoring

BUG=416998
R=xiyuan,juanlang

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

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