Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(185)

Side by Side Diff: third_party/WebKit/Source/core/html/shadow/MediaControlsTest.cpp

Issue 2721503002: Rename all uses of EmptyFrameLoaderClient with EmptyLocalFrameClient. (Closed)
Patch Set: Change V8 headers to user includes and introduce a space between system and user headers. Headers w… Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/html/shadow/MediaControls.h" 5 #include "core/html/shadow/MediaControls.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <memory> 8 #include <memory>
9 #include "core/HTMLNames.h" 9 #include "core/HTMLNames.h"
10 #include "core/css/StylePropertySet.h" 10 #include "core/css/StylePropertySet.h"
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 void promptCancelled() override {} 75 void promptCancelled() override {}
76 bool remotePlaybackAvailable() const override { 76 bool remotePlaybackAvailable() const override {
77 return m_availability == WebRemotePlaybackAvailability::DeviceAvailable; 77 return m_availability == WebRemotePlaybackAvailability::DeviceAvailable;
78 } 78 }
79 79
80 private: 80 private:
81 WebRemotePlaybackAvailability m_availability = 81 WebRemotePlaybackAvailability m_availability =
82 WebRemotePlaybackAvailability::Unknown; 82 WebRemotePlaybackAvailability::Unknown;
83 }; 83 };
84 84
85 class StubFrameLoaderClient : public EmptyFrameLoaderClient { 85 class StubFrameLoaderClient : public EmptyLocalFrameClient {
haraken 2017/02/27 00:23:17 Consider renaming these "FrameLoaderClient" classe
86 public: 86 public:
87 static StubFrameLoaderClient* create() { return new StubFrameLoaderClient; } 87 static StubFrameLoaderClient* create() { return new StubFrameLoaderClient; }
88 88
89 std::unique_ptr<WebMediaPlayer> createWebMediaPlayer( 89 std::unique_ptr<WebMediaPlayer> createWebMediaPlayer(
90 HTMLMediaElement&, 90 HTMLMediaElement&,
91 const WebMediaPlayerSource&, 91 const WebMediaPlayerSource&,
92 WebMediaPlayerClient*) override { 92 WebMediaPlayerClient*) override {
93 return WTF::wrapUnique(new MockVideoWebMediaPlayer); 93 return WTF::wrapUnique(new MockVideoWebMediaPlayer);
94 } 94 }
95 95
(...skipping 293 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 ASSERT_NE(nullptr, downloadButton); 389 ASSERT_NE(nullptr, downloadButton);
390 390
391 // Download button should not be displayed for HLS streams. 391 // Download button should not be displayed for HLS streams.
392 mediaControls().mediaElement().setSrc("https://example.com/foo.m3u8"); 392 mediaControls().mediaElement().setSrc("https://example.com/foo.m3u8");
393 testing::runPendingTasks(); 393 testing::runPendingTasks();
394 simulateLoadedMetadata(); 394 simulateLoadedMetadata();
395 EXPECT_FALSE(isElementVisible(*downloadButton)); 395 EXPECT_FALSE(isElementVisible(*downloadButton));
396 } 396 }
397 397
398 } // namespace blink 398 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698