Java - закрывающий кадр

frame2 fr2 = new frame2();
fr2.setVisible(false);

тем самым вы создаете новый экземпляр frame2 и скрываете его, и вы ничего не делаете с экземпляром frame2, который вы уже создали, и, следовательно, no reaction

Что вам нужно сделать, так это создать поле класса или что-то подобное для хранения ссылки на экземпляр frame2, который вы создаете сначала (когда вы его показываете), а затем использовать ту же ссылку и вызывать setVisible(false).

class frame1{
    ...

    private frame2 frame2ref;
    ....


    void foo()
    {
       ....
       frame2ref = new frame2();
       frame2ref .setVisible(true);
       // create and save reference and show frame.
       ...
    }

    void bar()
    {
        ...
        frame2ref.setVisible(false);
        // retrieve reference and hide frame. 
        ...

    }

}