Skip to content

Commit

Permalink
Remove the Firebug debug class
Browse files Browse the repository at this point in the history
Sadly, until we retire the legacy FirefoxDriver, we cannot
also delete the firebug extension.
  • Loading branch information
shs96c committed Feb 27, 2018
1 parent 08262ae commit 503c44c
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 49 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
import org.openqa.selenium.io.FileHandler;
import org.openqa.selenium.io.Zip;
import org.openqa.selenium.testing.InProject;
import org.openqa.selenium.testing.drivers.Firebug;
import org.openqa.selenium.testing.drivers.SynthesizedFirefoxDriver;

import java.io.BufferedReader;
import java.io.File;
Expand Down Expand Up @@ -190,7 +190,7 @@ public void shouldInstallWebExtensionFromDirectory() throws IOException {

@Test
public void shouldInstallExtensionUsingClasspath() throws IOException {
profile.addExtension(Firebug.class, FIREBUG_RESOURCE_PATH);
profile.addExtension(SynthesizedFirefoxDriver.class, FIREBUG_RESOURCE_PATH);
File profileDir = profile.layoutOnDisk();
File extensionDir = new File(profileDir, "extensions/firebug@software.joehewitt.com");
assertTrue(extensionDir.exists());
Expand Down
1 change: 0 additions & 1 deletion java/client/test/org/openqa/selenium/testing/drivers/BUCK
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ java_library(name = 'browser',

java_library(name = 'drivers',
srcs = glob(['*Driver.java', '*Supplier.java',]) + [
'Firebug.java',
'IgnoreComparator.java',
'OutOfProcessSeleniumServer.java',
'TestIgnorance.java',
Expand Down
46 changes: 0 additions & 46 deletions java/client/test/org/openqa/selenium/testing/drivers/Firebug.java

This file was deleted.

0 comments on commit 503c44c

Please sign in to comment.