Start removing some pre-md constants and related code.

Start removing some pre-md constants and related code.

BUG=693282
Review-Url: https://codereview.chromium.org/2698213004
Cr-Commit-Position: refs/heads/master@{#451780}
Committed: https://chromium.googlesource.com/chromium/src/+/304ce3931f9c5c4f63a94de6047b242ae239925d

[Evan Stade, 2017-02-17 00:58:41]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[Evan Stade, 2017-02-17 00:58:51]

estade@chromium.org changed reviewers:
+ tdanderson@chromium.org

[Evan Stade, 2017-02-17 00:58:52]

Stopped here because there are some non-trivial changes.

[commit-bot: I haz the power, 2017-02-17 01:00:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 01:06:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-17 01:06:08]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Evan Stade, 2017-02-17 21:36:38]

The CQ bit was checked by estade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-17 21:37:40]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tdanderson, 2017-02-17 21:46:11]

LGTM with one comment

https://codereview.chromium.org/2698213004/diff/20001/ash/common/system/tray/...
File ash/common/system/tray/tray_details_view.cc (right):

https://codereview.chromium.org/2698213004/diff/20001/ash/common/system/tray/...
ash/common/system/tray/tray_details_view.cc:459:
base::TimeDelta::FromMilliseconds(100), this,
Use kTrayDetailedViewTransitionDelayMs in place of 100

[Evan Stade, 2017-02-17 21:51:33]

https://codereview.chromium.org/2698213004/diff/20001/ash/common/system/tray/...
File ash/common/system/tray/tray_details_view.cc (right):

https://codereview.chromium.org/2698213004/diff/20001/ash/common/system/tray/...
ash/common/system/tray/tray_details_view.cc:459:
base::TimeDelta::FromMilliseconds(100), this,
On 2017/02/17 21:46:11, tdanderson wrote:
> Use kTrayDetailedViewTransitionDelayMs in place of 100

oops, thanks for catching

[Evan Stade, 2017-02-17 21:51:47]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2017-02-17 21:51:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdanderson@chromium.org
Link to the patchset: https://codereview.chromium.org/2698213004/#ps60001
(title: "actually use the constant")

[commit-bot: I haz the power, 2017-02-17 21:52:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 23:55:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-17 23:55:59]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT,
no build URL)

[Evan Stade, 2017-02-21 16:14:56]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2017-02-21 16:15:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-21 17:00:27]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1487693696153930,
"parent_rev": "7633f60f80f749dab9d8f387bb4632b14caebf1e", "commit_rev":
"304ce3931f9c5c4f63a94de6047b242ae239925d"}

[commit-bot: I haz the power, 2017-02-21 17:01:06]

Description was changed from

==========
Start removing some pre-md constants and related code.

BUG=693282
==========

to

==========
Start removing some pre-md constants and related code.

BUG=693282

Review-Url: https://codereview.chromium.org/2698213004
Cr-Commit-Position: refs/heads/master@{#451780}
Committed:
https://chromium.googlesource.com/chromium/src/+/304ce3931f9c5c4f63a94de6047b...
==========

[commit-bot: I haz the power, 2017-02-21 17:01:08]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/304ce3931f9c5c4f63a94de6047b...