|
|
DescriptionRoll media/cdm/api deps.
This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/
Also, removing now unnecessary lines in CFI blacklist.
BUG=609564, 609175, 515347
Committed: https://crrev.com/c5197d2ab04247a33e6934b576cdbd071ba562c2
Cr-Commit-Position: refs/heads/master@{#393981}
Patch Set 1 #Patch Set 2 : roll media/cdm/api deps #Patch Set 3 : also delete mojo types from blacklist #Patch Set 4 : sync #Patch Set 5 : fix windows with CDM_CLASS_API #Messages
Total messages: 27 (13 generated)
Message was sent while issue was closed.
Description was changed from ========== ... BUG= ========== to ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175 ==========
The CQ bit was checked by krasin@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1959093004/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1959093004/20001
krasin@google.com changed reviewers: + pcc@chromium.org
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: ios-device-gn on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...) ios-simulator on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...) mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...) mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...) mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
Description was changed from ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175 ========== to ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ==========
The CQ bit was checked by krasin@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1959093004/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1959093004/60001
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
The CQ bit was checked by krasin@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1959093004/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1959093004/80001
Peter, please, take a look. Trybots are green.
lgtm
Description was changed from ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ========== to ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ==========
Xiaohan, just an FYI: I am bumping media/cdm/api deps. Things look good so far.
On 2016/05/16 23:10:42, krasin wrote: > Xiaohan, > > just an FYI: I am bumping media/cdm/api deps. Things look good so far. Thanks for the headsup!
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by krasin@google.com
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1959093004/80001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1959093004/80001
Message was sent while issue was closed.
Description was changed from ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ========== to ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ==========
Message was sent while issue was closed.
Committed patchset #5 (id:80001)
Message was sent while issue was closed.
Description was changed from ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 ========== to ========== Roll media/cdm/api deps. This CL brings fixes for visibility of types used across Chromium->CDM bridge. See https://chromiumcodereview.appspot.com/1956123002/ and https://codereview.chromium.org/1964083003/ Also, removing now unnecessary lines in CFI blacklist. BUG=609564, 609175, 515347 Committed: https://crrev.com/c5197d2ab04247a33e6934b576cdbd071ba562c2 Cr-Commit-Position: refs/heads/master@{#393981} ==========
Message was sent while issue was closed.
Patchset 5 (id:??) landed as https://crrev.com/c5197d2ab04247a33e6934b576cdbd071ba562c2 Cr-Commit-Position: refs/heads/master@{#393981} |