GitLab

This CL is a part of https://codereview.chromium.org/665023002/ ("Post-commit
fixes for "stale-while-revalidate..."") that has been separated into a separate
CL because it depends on https://codereview.chromium.org/669083002/ ("Add
logging support for base::Time* types.").

BUG=348877

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

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

BUG=427526

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

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

Previously if drive sync over cellular network is disabled, pinning a drive file
is pending without notificaiton.

This CL lets Files.app show the notifiction.

BUG=279476
TEST=DriveSpecific/FileManagerBrowserTest.Test/4

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

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

Revert of Adds ozone support for ime keyboard. (patchset #7 id:120001 of https://codereview.chromium.org/683773002/)

Reason for revert:
Change doesn't compile on Linux ChromiumOS Ozone Builder

http://build.chromium.org/p/chromium.chromiumos/buildstatus?builder=Linux%20ChromiumOS%20Ozone%20Builder&number=6030

Original issue's description:
> Adds ozone support for ime keyboard.
>
> BUG=342336
>
> Committed: https://crrev.com/8be55caf3d38bba7645de143f7e6b8b655fdd36c
> Cr-Commit-Position: refs/heads/master@{#302057}

TBR=shuchen@chromium.org,fengyuan@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=342336

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

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

Roll src/third_party/WebKit b4fcaad1f8a5c9f7e22f699f41a98dbb954cc4f2:5d4e6a45858acb9904edad8ec8ff267713c2bc4f (svn 184589:184626)

BUG=None
TBR=morrita

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

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

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

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

BUG=424785
TEST=manually. athena_unittests passes.

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

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

This CL adds the canonical cookie that changed and a boolean indicating
if it was removed or added to the cookie changed callback.

It also sets a clear contract when the cookie changed notification callback
is called and documents it in |cookie_store.h|

BUG=NONE

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

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

BUG=342336

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

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

I accidentally committed http://crrev.com/455623003 before getting LGTM from
mmenke@. This CL is for additional fixes that he requested.

BUG=348877
TEST=net_unittests

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

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

Revert of Enable ChannelMojo on Linux (patchset #1 id:1 of https://codereview.chromium.org/684323002/)

Reason for revert:
Caused multiple layout test crashes on Linux

See http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=css3%2Ffilters%2Feffect-reference-subregion-hidpi-hw.html%2Ccss3%2Ffilters%2Feffect-reference-subregion-hidpi.html%2Ccss3%2Fimages%2Fpixelated-hidpi.html%2Cfast%2Fhidpi%2Fimage-set-as-background.html%2Cfast%2Fhidpi%2Fimage-set-background-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-background-repeat-without-size.html%2Cfast%2Fhidpi%2Fimage-set-background-repeat.html%2Cfast%2Fhidpi%2Fimage-set-border-image-comparison.html%2Cfast%2Fhidpi%2Fimage-set-border-image-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-border-image-simple.html%2Cfast%2Fhidpi%2Fimage-set-in-content-dynamic.html%2Cfast%2Fhidpi%2Fimage-set-out-of-order.html%2Cfast%2Fhidpi%2Fimage-set-simple.html%2Cfast%2Fhidpi%2Fimage-set-without-specified-width.html%2Cfast%2Fhistory%2Freplacestate-nocrash.html%2Chttp%2Ftests%2Finspector%2Fnetwork%2Flong-script-content.html%2Chttp%2Ftests%2Fxmlhttprequest%2Fsimple-cross-origin-progress-events.html%2Csvg%2Fas-image%2Fsvg-as-image-canvas.html%2Csvg%2Fcss%2Fsvg-resource-fragment-identifier-img-src.html

Regressions: Unexpected crashes (19)
  css3/filters/effect-reference-subregion-hidpi-hw.html [ Crash ]
  css3/filters/effect-reference-subregion-hidpi.html [ Crash ]
  css3/images/pixelated-hidpi.html [ Crash ]
  fast/hidpi/image-set-as-background.html [ Crash ]
  fast/hidpi/image-set-background-dynamic.html [ Crash ]
  fast/hidpi/image-set-background-repeat-without-size.html [ Crash ]
  fast/hidpi/image-set-background-repeat.html [ Crash ]
  fast/hidpi/image-set-border-image-comparison.html [ Crash ]
  fast/hidpi/image-set-border-image-dynamic.html [ Crash ]
  fast/hidpi/image-set-border-image-simple.html [ Crash ]
  fast/hidpi/image-set-in-content-dynamic.html [ Crash ]
  fast/hidpi/image-set-out-of-order.html [ Crash ]
  fast/hidpi/image-set-simple.html [ Crash ]
  fast/hidpi/image-set-without-specified-width.html [ Crash ]
  fast/history/replacestate-nocrash.html [ Crash ]
  http/tests/inspector/network/long-script-content.html [ Crash ]
  http/tests/xmlhttprequest/simple-cross-origin-progress-events.html [ Crash ]
  svg/as-image/svg-as-image-canvas.html [ Crash ]
  svg/css/svg-resource-fragment-identifier-img-src.html [ Crash ]

Original issue's description:
> Enable ChannelMojo on Linux
>
> This change enables ChannelMojo on Linux.
> There could possibly be a perf slowdown, especially
> on latency metrics.
>
> If you're looking for the cause of some perf bot redness,
> probably this is that one. Please feel free to revert and
> let me know which benchmark was hit.
>
> BUG=664413003
> R=viettrungluu@chromium.org
>
> Committed: https://crrev.com/d4d85c640375ea7f9692e53860f4de058a7f0ea1
> Cr-Commit-Position: refs/heads/master@{#301992}

TBR=viettrungluu@chromium.org,morrita@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=664413003

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

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

Introduce (Refcounted)KeyedServiceFactory classes that deal with the
lifetime management of (Refcounted)KeyedService. Port the classes
(Refcounted)BrowserContextKeyedServiceFactory to inherit from them.

BUG=419366

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

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

Move the fake mounting logic from CrosDisksClientStubImpl
to FakeCrosDsisksClient, with some simplification.

The fake behaviors are used in FileManagerBrowserTest's
OpenZipFiles test. The test is currently disabled for
a performance reason, but I've confirmed that the test
passes with --gtest_also_run_disabled_tests.

TEST=out/Release/browser_tests --gtest_filter='*OpenZipFiles*' --gtest_also_run_disabled_tests
BUG=309506

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

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

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

The Google C++ style guide states:

  Explicitly annotate overrides of virtual functions or virtual
  destructors with an override or (less frequently) final specifier.
  Older (pre-C++11) code will use the virtual keyword as an inferior
  alternative annotation. For clarity, use exactly one of override,
  final, or virtual when declaring an override.

To better conform to these guidelines, the following constructs have
been rewritten:

- if a base class has a virtual destructor, then:
    virtual ~Foo();                   ->  ~Foo() override;
- virtual void Foo() override;        ->  void Foo() override;
- virtual void Foo() override final;  ->  void Foo() final;

This patch was automatically generated. The clang plugin can generate
fixit hints, which are suggested edits when it is 100% sure it knows how
to fix a problem. The hints from the clang plugin were applied to the
source tree using the tool in https://codereview.chromium.org/598073004.

BUG=417463
R=nona@chromium.org

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

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

[1] Chromium-side: THIS PATCH
[2] Blink-side: https://codereview.chromium.org/686353003

BUG=n/a
TEST=compile

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

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

shishir@ and dominich@ no longer work on Chrome and are not responsive
to review requests.

Remove them from OWNERS files.

BUG=
NOTRY=true

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

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

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

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

If keysym is ASCII control or space keys, directly map to keycode instead of go through the fallback maps.

A real case is de(neo) layout, AltGr+V maps to Enter key, so it should map to VKEY_ENTER instead of VKEY_V.

BUG=420544
TEST=Verified on linux_chromeos.

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

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

The Google C++ style guide states:

  Explicitly annotate overrides of virtual functions or virtual
  destructors with an override or (less frequently) final specifier.
  Older (pre-C++11) code will use the virtual keyword as an inferior
  alternative annotation. For clarity, use exactly one of override,
  final, or virtual when declaring an override.

To better conform to these guidelines, the following constructs have
been rewritten:

- if a base class has a virtual destructor, then:
    virtual ~Foo();                   ->  ~Foo() override;
- virtual void Foo() override;        ->  void Foo() override;
- virtual void Foo() override final;  ->  void Foo() final;

This patch was automatically generated. The clang plugin can generate
fixit hints, which are suggested edits when it is 100% sure it knows how
to fix a problem. The hints from the clang plugin were applied to the
source tree using the tool in https://codereview.chromium.org/598073004.

BUG=417463
R=tapted@chromium.org

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

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

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

This change splits the WebRtc observer functionality of RtcDataChannelHandler
from the WebKit related functionality. The former belongs with libjingle's
signaling thread and the latter with the main thread. As is, these threads
are one and the same, but this will change later this week.

This is a re-land of: https://codereview.chromium.org/643693007

BUG=369796
TBR=perkj

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

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

TBR=isherman

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

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

chrome://imageburner was replaced by the new recovery tool
http://google.com/chromeos/recovery.

With this change, chrome://imageburner just redirects to
the new recovery tool page.

Note that the D-Bus client code (ImageBurnerClient) is not removed
because it's used in imageWriterPrivate API, used in the new
recovery tool.

BUG=344279
TEST=open chrome://imageburner and you are redirected to http://google.com/chromeos/recovery

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

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

This CL breaks out the command handling (command creation and session information restore) from |SessionService| into |SessionServiceCommands|, which does have no dependencies on Browser or Content.

Furthermore it breaks out the command handling from |BaseSessionService| into |BaseSessionServiceCommands|. This way the |PersistentTabRestoreService| and the |SessionService| can keep using the base command set which is used by both users.

BUG=424679
TEST=covered by existing tests

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

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

This is applying r301168 to the app-list item in addition to
TileItemView.

BUG=425724
R=calamity@chromium.org
TEST=manually

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

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

BUG=412078

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

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

TBR=jam@chromium.org

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

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

When an HTTPS and/or HTTP proxy was manually configured, WebSocket
connections were mistakenly going direct to the origin server.

According to RFC6455, WebSocket connections should prefer SOCKS proxies,
then HTTPS, then HTTP proxies in that order. Fix the behaviour when
proxies are manually selected to match.

BUG=426736
TEST=net_unittests, manual

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

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

Revert of Remote assistance on Chrome OS Part IV - It2MeHost (patchset #12 id:700001 of https://codereview.chromium.org/639233002/)

Reason for revert:
Reverting due to build failure on Linux Ozone builder.
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Ozone%20Builder/builds/6011

Original issue's description:
> Remote assistance on Chrome OS Part IV - It2MeHost
>
> This CL links the it2me host to the Chrome binary on ChromeOS
> behind a flag.
> The following changes are made to the it2me host so that it can be run in the browser process.
> 1. Initializes SSL server sockets and specific CPU media features on ChromeOS startup.
> 2. Fixes a crash in it2me shutdown by making It2meHost owns the ChromotingHostContext.
> 3. Replace the blocking shutdown wait on PolicyWatcher with a callback.
>    Implements policy_watcher on ChromeOS using policy services.
> 4. Re-use existing threads, url request context getters and policy service on ChromeOS.
> 5. Fixed a incorrect DCHECK regarding the color format of the frames captured on ChromeOS.
>
> BUG=334087
>
> Committed: https://crrev.com/54dde6f02d121ff745e66b57205583087ff720ec
> Cr-Commit-Position: refs/heads/master@{#302034}

TBR=jochen@chromium.org,dpolukhin@chromium.org,jamiewalch@chromium.org,kalman@chromium.org,rmsousa@chromium.org,rockot@chromium.org,wez@chromium.org,agl@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=334087

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

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

Namely, -Winconsistent-missing-override. While here, also
remove "virtual" from overridden methods, as per new style.
No behavior change.

BUG=428099

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

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

This CL links the it2me host to the Chrome binary on ChromeOS
behind a flag.
The following changes are made to the it2me host so that it can be run in the browser process.
1. Initializes SSL server sockets and specific CPU media features on ChromeOS startup.
2. Fixes a crash in it2me shutdown by making It2meHost owns the ChromotingHostContext.
3. Replace the blocking shutdown wait on PolicyWatcher with a callback.
   Implements policy_watcher on ChromeOS using policy services.
4. Re-use existing threads, url request context getters and policy service on ChromeOS.
5. Fixed a incorrect DCHECK regarding the color format of the frames captured on ChromeOS.

BUG=334087

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

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

Also unforks some code and does a TODO(estade).

R=alekseys@chromium.org
TBR=vitalybuka@chromium.org
BUG=383955

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

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

R=alekseys@chromium.org
TBR=vitalybuka@chromium.org
BUG=383955

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

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

BUG=341353

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

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

Not actually visible to the user. It is not supposed to regenerate the
icon if the fifth or greater item changed. Due to an off-by-one error,
it would still regenerate the icon if the fifth item changed (which is
unnecessary).

NB: Tests are incoming (which would have caught this error).

Also harden a DCHECK (fixing a similar off-by-one error, and making it
into a CHECK because it would be a security issue if it failed).

BUG=425444

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

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

This should be useful when debugging resource management.

BUG=None
TBR=sky@chromium.org as an owner of new DEPS dependency ui/wm/public

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

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

To call GetWindow(), make overridden methods of AppActivityProxy public.
(this should be OK as AppActivity is doing the same thing and these methods become public with static_cast<Activity*>).

BUG=None

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

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

TBR=arv@chromium.org
BUG=427867

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

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

BUG=428571
R=cpu@chromium.org

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

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